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

Side by Side Diff: chrome/browser/ui/views/location_bar/location_bar_view.cc

Issue 10821114: Refactor the content setting view to allow a subclass for the web intents button. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Consume tap-down gesture Created 8 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/ui/views/location_bar/content_setting_image_view.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/location_bar/location_bar_view.h" 5 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <map> 8 #include <map>
9 9
10 #include "base/stl_util.h" 10 #include "base/stl_util.h"
(...skipping 1069 matching lines...) Expand 10 before | Expand all | Expand 10 after
1080 view->SetBounds( 1080 view->SetBounds(
1081 leading ? bounds->x() : (bounds->right() - view_size.width()), 1081 leading ? bounds->x() : (bounds->right() - view_size.width()),
1082 view->y(), view_size.width(), view->height()); 1082 view->y(), view_size.width(), view->height());
1083 bounds->set_width(bounds->width() - desired_width); 1083 bounds->set_width(bounds->width() - desired_width);
1084 } 1084 }
1085 } 1085 }
1086 1086
1087 void LocationBarView::RefreshContentSettingViews() { 1087 void LocationBarView::RefreshContentSettingViews() {
1088 for (ContentSettingViews::const_iterator i(content_setting_views_.begin()); 1088 for (ContentSettingViews::const_iterator i(content_setting_views_.begin());
1089 i != content_setting_views_.end(); ++i) { 1089 i != content_setting_views_.end(); ++i) {
1090 (*i)->UpdateFromWebContents(model_->input_in_progress() ? NULL : 1090 (*i)->Update(model_->input_in_progress() ? NULL : GetTabContents());
1091 GetWebContentsFromDelegate(delegate_));
1092 } 1091 }
1093 } 1092 }
1094 1093
1095 void LocationBarView::DeletePageActionViews() { 1094 void LocationBarView::DeletePageActionViews() {
1096 for (PageActionViews::const_iterator i(page_action_views_.begin()); 1095 for (PageActionViews::const_iterator i(page_action_views_.begin());
1097 i != page_action_views_.end(); ++i) 1096 i != page_action_views_.end(); ++i)
1098 RemoveChildView(*i); 1097 RemoveChildView(*i);
1099 STLDeleteElements(&page_action_views_); 1098 STLDeleteElements(&page_action_views_);
1100 } 1099 }
1101 1100
(...skipping 396 matching lines...) Expand 10 before | Expand all | Expand 10 after
1498 } 1497 }
1499 1498
1500 void LocationBarView::CleanupFadeAnimation() { 1499 void LocationBarView::CleanupFadeAnimation() {
1501 // Since we're no longer animating we don't need our layer. 1500 // Since we're no longer animating we don't need our layer.
1502 SetPaintToLayer(false); 1501 SetPaintToLayer(false);
1503 // Bubble labels don't need a transparent background anymore. 1502 // Bubble labels don't need a transparent background anymore.
1504 ev_bubble_view_->SetLabelBackgroundColor(SK_ColorWHITE); 1503 ev_bubble_view_->SetLabelBackgroundColor(SK_ColorWHITE);
1505 selected_keyword_view_->SetLabelBackgroundColor(SK_ColorWHITE); 1504 selected_keyword_view_->SetLabelBackgroundColor(SK_ColorWHITE);
1506 } 1505 }
1507 #endif // USE_AURA 1506 #endif // USE_AURA
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/location_bar/content_setting_image_view.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698