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

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

Issue 21696003: views/location_bar: Eliminate TouchableLocationBarView. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix wrapping Created 7 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
Index: chrome/browser/ui/views/location_bar/touchable_location_bar_view.h
diff --git a/chrome/browser/ui/views/location_bar/touchable_location_bar_view.h b/chrome/browser/ui/views/location_bar/touchable_location_bar_view.h
deleted file mode 100644
index aaba2b944d9e9776acc656f808fe01a9697095e8..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/views/location_bar/touchable_location_bar_view.h
+++ /dev/null
@@ -1,38 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_TOUCHABLE_LOCATION_BAR_VIEW_H_
-#define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_TOUCHABLE_LOCATION_BAR_VIEW_H_
-
-#include "ui/views/view.h"
-
-// An interface for a View in within the LocationBar that knows how to
-// increase its size (without affecting visual layout) in a touch
-// layout, to increase the size of the touch target. Its border
-// extends a few pixels up and down, which doesn't affect layout, and
-// extends half of the padding used between items in the location bar
-// to the left and right.
-//
-// To make your location bar View extend itself into the padding
-// around it to get an enlarged touch target, inherit from
-// TouchableLocationBarView interface, implement its
-// GetBuiltInHorizontalPadding method by calling
-// GetBuiltInHorizontalPaddingImpl, and call
-// TouchableLocationBarView::Init() from your constructor.
-class TouchableLocationBarView {
- public:
- // Returns the number of pixels of built-in padding to the left and
- // right of the image for this view.
- virtual int GetBuiltInHorizontalPadding() const = 0;
-
- protected:
- // Call this from the constructor (or during early initialization)
- // of a class that inherits from TouchableLocationBarView.
- static void Init(views::View* view);
-
- // Use this to implement GetBuiltInHorizontalPadding().
- static int GetBuiltInHorizontalPaddingImpl();
-};
-
-#endif // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_TOUCHABLE_LOCATION_BAR_VIEW_H_
« no previous file with comments | « chrome/browser/ui/views/location_bar/star_view.cc ('k') | chrome/browser/ui/views/location_bar/touchable_location_bar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698