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

Unified Diff: chrome/browser/ui/views/location_bar/content_setting_image_view.cc

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/content_setting_image_view.cc
diff --git a/chrome/browser/ui/views/location_bar/content_setting_image_view.cc b/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
index 014c3dacf31e2ec523ff0593f2f72468b6d88513..6d05e4fa6a7a3ec83dd6026879005839266391e2 100644
--- a/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
+++ b/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
@@ -83,7 +83,7 @@ ContentSettingImageView::ContentSettingImageView(
text_label_->SetElideBehavior(views::Label::NO_ELIDE);
AddChildView(text_label_);
- TouchableLocationBarView::Init(this);
+ LocationBarView::InitTouchableLocationBarChildView(this);
slide_animator_.SetSlideDuration(kAnimationDurationMS);
slide_animator_.SetTweenType(ui::Tween::LINEAR);
@@ -94,10 +94,6 @@ ContentSettingImageView::~ContentSettingImageView() {
bubble_widget_->RemoveObserver(this);
}
-int ContentSettingImageView::GetBuiltInHorizontalPadding() const {
- return GetBuiltInHorizontalPaddingImpl();
-}
-
void ContentSettingImageView::Update(content::WebContents* web_contents) {
// Note: We explicitly want to call this even if |web_contents| is NULL, so we
// get hidden properly while the user is editing the omnibox.

Powered by Google App Engine
This is Rietveld 408576698