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

Unified Diff: chrome/browser/ui/views/location_bar/page_action_with_badge_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/page_action_with_badge_view.h
diff --git a/chrome/browser/ui/views/location_bar/page_action_with_badge_view.h b/chrome/browser/ui/views/location_bar/page_action_with_badge_view.h
index f56c4a27670f077588baa70bca6199c24689f49e..90096b250f945b953ae5cdf2bb360c8650410a30 100644
--- a/chrome/browser/ui/views/location_bar/page_action_with_badge_view.h
+++ b/chrome/browser/ui/views/location_bar/page_action_with_badge_view.h
@@ -7,6 +7,7 @@
#pragma once
#include "base/compiler_specific.h"
+#include "chrome/browser/ui/views/location_bar/touchable_location_bar_view.h"
#include "ui/gfx/size.h"
#include "ui/views/view.h"
@@ -18,7 +19,9 @@ class WebContents;
}
// A container for the PageActionImageView plus its badge.
-class PageActionWithBadgeView : public views::View {
+class PageActionWithBadgeView
+ : public views::View,
+ public TouchableLocationBarView {
public:
explicit PageActionWithBadgeView(PageActionImageView* image_view);
@@ -28,6 +31,9 @@ class PageActionWithBadgeView : public views::View {
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual gfx::Size GetPreferredSize() OVERRIDE;
+ // TouchableLocationBarView.
+ virtual int GetBuiltInHorizontalPadding() const OVERRIDE;
+
void UpdateVisibility(content::WebContents* contents, const GURL& url);
private:

Powered by Google App Engine
This is Rietveld 408576698