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

Unified Diff: chrome/browser/ui/views/find_bar_view.h

Issue 1408223008: [MD] Share button code between find bar and download shelf (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: one more place Created 5 years, 1 month 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
Index: chrome/browser/ui/views/find_bar_view.h
diff --git a/chrome/browser/ui/views/find_bar_view.h b/chrome/browser/ui/views/find_bar_view.h
index 5fac1d1955d17a8777b4600d811d897d6cd75999..b26c753297b96b26b9b44175fbbb6e69d53044be 100644
--- a/chrome/browser/ui/views/find_bar_view.h
+++ b/chrome/browser/ui/views/find_bar_view.h
@@ -22,6 +22,7 @@ class ImageButton;
class Label;
class MouseEvent;
class Painter;
+class Separator;
}
////////////////////////////////////////////////////////////////////////////////
@@ -104,6 +105,9 @@ class FindBarView : public DropdownBarView,
void OnThemeChanged() override;
void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
+ // Returns the color for the icons on the buttons per the current NativeTheme.
+ SkColor GetTextColorForIcon();
+
// We use a hidden view to grab mouse clicks and bring focus to the find
// text box. This is because although the find text box may look like it
// extends all the way to the find button, it only goes as far as to the
@@ -137,6 +141,7 @@ class FindBarView : public DropdownBarView,
scoped_ptr<views::Painter> find_text_border_;
views::Label* match_count_text_;
views::View* focus_forwarder_view_;
+ views::Separator* separator_;
views::ImageButton* find_previous_button_;
views::ImageButton* find_next_button_;
views::ImageButton* close_button_;

Powered by Google App Engine
This is Rietveld 408576698