Discussion:
Issue 177262 in chromium: find-in-page should update as the page changes
(too old to reply)
c***@googlecode.com
2013-02-20 20:12:28 UTC
Permalink
Raw Message
Status: Untriaged
Owner: ----
CC: ***@chromium.org, ***@chromium.org, ***@chromium.org
Labels: OS-All Area-UI Type-Bug Pri-2

New issue 177262 by ***@chromium.org: find-in-page should update as the
page changes
http://code.google.com/p/chromium/issues/detail?id=177262

What steps will reproduce the problem?
1. Load
http://plexode.com/eval3/#ht=%3Cdiv%3EOjan%20is%20awesome%3C%2Fdiv%3E&ohh=0&ohj=0&ojh=0&ojj=0
2. Search for "ojan"
3. Click the "set innerHTML now" button.
4. Hit the up/down arrows in the find-bar

What is the expected result?
After step 4, both instances of "Ojan" should get cycled through and the
number of matches should be updated.

What happens instead of that?
It cycles off/on the first instance of "Ojan" but doesn't find the second.

With MutationObservers, I think we could actually do a reasonable
implementation that updates the find results when the DOM changes without a
big perf hit.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs
c***@googlecode.com
2013-02-20 21:33:55 UTC
Permalink
Raw Message
Updates:
Cc: ***@chromium.org

Comment #1 on issue 177262 by ***@chromium.org: find-in-page should update
as the page changes
http://code.google.com/p/chromium/issues/detail?id=177262

Adam convinced me that MutationObservers are the wrong solution since what
we really care about is changes in the render tree. I believe we could hook
addChild/removeChild/setText in the render tree and give a notification
across the WebKit API that a subtree needs to be reprocesses.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs
c***@googlecode.com
2013-03-18 00:21:59 UTC
Permalink
Raw Message
Updates:
Labels: Cr-UI-Browser-FindInPage

Comment #3 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
http://code.google.com/p/chromium/issues/detail?id=177262

(No comment was entered for this change.)
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs
c***@googlecode.com
2013-04-28 06:49:06 UTC
Permalink
Raw Message
Comment #4 on issue 177262 by ***@gmail.com: find-in-page should
update as the page changes
http://code.google.com/p/chromium/issues/detail?id=177262

In case a real-life use case is desired: on pages with ‘infinite scroll’,
you would like new instances of the search term to come up as you scroll.
So this affects Google+, Facebook and Twitter.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs
c***@googlecode.com
2013-06-11 00:10:41 UTC
Permalink
Raw Message
Comment #5 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
http://code.google.com/p/chromium/issues/detail?id=177262

Issue 82276 has been merged into this issue.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs
c***@googlecode.com
2013-06-11 00:13:41 UTC
Permalink
Raw Message
Comment #6 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
http://code.google.com/p/chromium/issues/detail?id=177262

Issue 226040 has been merged into this issue.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs
c***@googlecode.com
2013-06-11 00:16:41 UTC
Permalink
Raw Message
Updates:
Cc: ***@chromium.org

Comment #7 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
http://code.google.com/p/chromium/issues/detail?id=177262

Issue 21309 has been merged into this issue.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs
c***@googlecode.com
2013-06-11 00:18:41 UTC
Permalink
Raw Message
Updates:
Cc: ***@chromium.org

Comment #8 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
http://code.google.com/p/chromium/issues/detail?id=177262

Issue 236346 has been merged into this issue.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs
c***@googlecode.com
2015-10-19 18:00:39 UTC
Permalink
Raw Message
Updates:
Cc: ***@chromium.org

Comment #9 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
https://code.google.com/p/chromium/issues/detail?id=177262

Issue 517665 has been merged into this issue.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs

To unsubscribe from this group and stop receiving emails from it, send an email to chromium-bugs+***@chromium.org.
c***@googlecode.com
2015-10-20 16:02:08 UTC
Permalink
Raw Message
Updates:
Cc: ***@chromium.org

Comment #13 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
https://code.google.com/p/chromium/issues/detail?id=177262

Issue 174487 has been merged into this issue.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs

To unsubscribe from this group and stop receiving emails from it, send an email to chromium-bugs+***@chromium.org.
c***@googlecode.com
2015-10-20 16:04:08 UTC
Permalink
Raw Message
Comment #14 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
https://code.google.com/p/chromium/issues/detail?id=177262

Issue 536632 has been merged into this issue.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs

To unsubscribe from this group and stop receiving emails from it, send an email to chromium-bugs+***@chromium.org.
c***@googlecode.com
2016-02-18 08:13:42 UTC
Permalink
Raw Message
Comment #15 on issue 177262 by ***@chromium.org: find-in-page should
update as the page changes
https://code.google.com/p/chromium/issues/detail?id=177262

Issue 584724 has been merged into this issue.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/a/chromium.org/group/chromium-bugs

---
You received this message because you are subscribed to the Google Groups "Chromium-bugs" group.
To unsubscribe from this group and stop receiving emails from it, send an email to chromium-bugs+***@chromium.org.
Loading...