Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2115)

Unified Diff: content/browser/find_request_manager.h

Issue 2363993003: Revert of Handling new frames and frame navigations with find-in-page during a find session. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/find_bar/find_bar_controller.cc ('k') | content/browser/find_request_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/find_request_manager.h
diff --git a/content/browser/find_request_manager.h b/content/browser/find_request_manager.h
index 5a6b3988d1083017d35dc8b6bd59e33a4c7a315e..7dddcc4e0ad267a31eaf1e0ad4dde74386f69bab 100644
--- a/content/browser/find_request_manager.h
+++ b/content/browser/find_request_manager.h
@@ -103,7 +103,6 @@
};
// WebContentsObserver implementation.
- void DidFinishLoad(RenderFrameHost* rfh, const GURL& validated_url) override;
void RenderFrameDeleted(RenderFrameHost* rfh) override;
void RenderFrameHostChanged(RenderFrameHost* old_host,
RenderFrameHost* new_host) override;
@@ -146,11 +145,8 @@
// Adds a frame to the set of frames that are being searched. The new frame
// will automatically be searched when added, using the same options (stored
- // in |current_request_.options|). |force| should be set to true when a
- // dynamic content change is suspected, which will treat the frame as a newly
- // added frame even if it has already been searched. This will force a
- // re-search of the frame.
- void AddFrame(RenderFrameHost* rfh, bool force);
+ // in |current_request_.options|).
+ void AddFrame(RenderFrameHost* rfh);
// Returns whether |rfh| is in the set of frames being searched in the current
// find session.
« no previous file with comments | « chrome/browser/ui/find_bar/find_bar_controller.cc ('k') | content/browser/find_request_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698