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

Unified Diff: chrome/browser/ui/views/location_bar/star_view.h

Issue 10519008: Enlarge size of touch target for several location bar buttons without affecting visual layout. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pure merge to LKGR Created 8 years, 6 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
Index: chrome/browser/ui/views/location_bar/star_view.h
diff --git a/chrome/browser/ui/views/location_bar/star_view.h b/chrome/browser/ui/views/location_bar/star_view.h
index 2a745ac1b7285e92e0616db6a0a852317e2de459..8d011d3808c9771bbd19451a2aedb98cb36f79dd 100644
--- a/chrome/browser/ui/views/location_bar/star_view.h
+++ b/chrome/browser/ui/views/location_bar/star_view.h
@@ -6,6 +6,7 @@
#define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_STAR_VIEW_H_
#pragma once
+#include "chrome/browser/ui/views/location_bar/touchable_location_bar_view.h"
#include "ui/views/controls/image_view.h"
class CommandUpdater;
@@ -15,7 +16,9 @@ class KeyEvent;
class MouseEvent;
}
-class StarView : public views::ImageView {
+class StarView
+ : public views::ImageView,
+ public TouchableLocationBarView {
public:
explicit StarView(CommandUpdater* command_updater);
virtual ~StarView();
@@ -23,6 +26,9 @@ class StarView : public views::ImageView {
// Toggles the star on or off.
void SetToggled(bool on);
+ // TouchableLocationBarView.
+ virtual int GetBuiltInHorizontalPadding() const OVERRIDE;
+
private:
// views::ImageView overrides:
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
« no previous file with comments | « chrome/browser/ui/views/location_bar/page_action_with_badge_view.cc ('k') | chrome/browser/ui/views/location_bar/star_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698