Commit 8a32858b authored by Tommy C. Li's avatar Tommy C. Li Committed by Commit Bot

Omnibox Material Refresh: Clean LocationBarView::OnLocationIconFetched

Clean up an unnecessary call within:
LocationBarView::OnLocationIconFetched.

Previously we also called LocationIconView::Update, which is
unnecessary.

Bug: 823535
Change-Id: I87e9a83ef2d831ead9d674c3ea7f98bd06830cf4
Reviewed-on: https://chromium-review.googlesource.com/1052917
Commit-Queue: Tommy Li <tommycli@chromium.org>
Reviewed-by: default avatarPeter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557360}
parent 01c6c043
...@@ -856,7 +856,6 @@ void LocationBarView::RefreshLocationIcon() { ...@@ -856,7 +856,6 @@ void LocationBarView::RefreshLocationIcon() {
void LocationBarView::OnLocationIconFetched(const gfx::Image& image) { void LocationBarView::OnLocationIconFetched(const gfx::Image& image) {
location_icon_view_->SetImage(image.AsImageSkia()); location_icon_view_->SetImage(image.AsImageSkia());
location_icon_view_->Update();
} }
bool LocationBarView::RefreshContentSettingViews() { bool LocationBarView::RefreshContentSettingViews() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment