From c07cfb808dbd642bb3385f974be296797bd34524 Mon Sep 17 00:00:00 2001 From: "brettw@chromium.org" Date: Thu, 14 May 2009 15:53:13 +0000 Subject: Add match count text to the GTK find bar. This also makes the bar wider. The match count text doesn't currently show up realiably because the data in the find reply is bad. I believe this is bug 11761. The clicker checking code has been moved to the cross-platform FindBarController so it doesn't have to be duplicated for each platform. This also add IntToString16. I didnt add all the variants now. The *Wstring versions should all eventually be changed to string16. http://crbug.com/11750 Review URL: http://codereview.chromium.org/114023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16058 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/find_bar_controller.h | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'chrome/browser/find_bar_controller.h') diff --git a/chrome/browser/find_bar_controller.h b/chrome/browser/find_bar_controller.h index fc286ef..5d8edf8 100644 --- a/chrome/browser/find_bar_controller.h +++ b/chrome/browser/find_bar_controller.h @@ -40,11 +40,20 @@ class FindBarController : public NotificationObserver { FindBar* find_bar() const { return find_bar_.get(); } private: + // Sents an update to the find bar with the tab contents' current result. The + // tab_contents_ must be non-NULL before this call. Theis handles + // de-flickering in addition to just calling the update function. + void UpdateFindBarForCurrentResult(); + scoped_ptr find_bar_; // The TabContents we are currently associated with. Can be NULL. TabContents* tab_contents_; + // The last match count we reported to the user. This is used by + // UpdateFindBarForCurrentResult to avoid flickering. + int last_reported_matchcount_; + DISALLOW_COPY_AND_ASSIGN(FindBarController); }; -- cgit v1.1