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

Unified Diff: chrome/browser/ui/browser.cc

Issue 7828025: Handle the ViewHostMsg_FindReply message which is sent by content(renderer) via the TabContentsDe... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 4 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/browser.h ('k') | chrome/browser/ui/find_bar/find_tab_helper.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser.cc
===================================================================
--- chrome/browser/ui/browser.cc (revision 99051)
+++ chrome/browser/ui/browser.cc (working copy)
@@ -2518,6 +2518,23 @@
tcw->web_intent_picker_controller()->ShowDialog(action, type);
}
+// static
+void Browser::FindReplyHelper(TabContents* tab,
+ int request_id,
+ int number_of_matches,
+ const gfx::Rect& selection_rect,
+ int active_match_ordinal,
+ bool final_update) {
+ TabContentsWrapper* tcw = TabContentsWrapper::GetCurrentWrapperForContents(
+ tab);
+ if (!tcw || !tcw->find_tab_helper())
+ return;
+
+ tcw->find_tab_helper()->HandleFindReply(request_id, number_of_matches,
+ selection_rect, active_match_ordinal,
+ final_update);
+}
+
void Browser::ExecuteCommandWithDisposition(
int id, WindowOpenDisposition disposition) {
// No commands are enabled if there is not yet any selected tab.
@@ -3773,6 +3790,16 @@
intent_id);
}
+void Browser::FindReply(TabContents* tab,
+ int request_id,
+ int number_of_matches,
+ const gfx::Rect& selection_rect,
+ int active_match_ordinal,
+ bool final_update) {
+ Browser::FindReplyHelper(tab, request_id, number_of_matches, selection_rect,
jam 2011/09/02 00:05:26 nit: the Browser:: is redundant
ananta 2011/09/02 00:28:28 Done. I also removed the Browser prefix from other
+ active_match_ordinal, final_update);
+}
+
void Browser::ExitTabbedFullscreenModeIfNecessary() {
if (tab_caused_fullscreen_)
ToggleFullscreenMode();
« no previous file with comments | « chrome/browser/ui/browser.h ('k') | chrome/browser/ui/find_bar/find_tab_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698