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

Unified Diff: chrome/browser/ui/views/frame/browser_view.cc

Issue 10917026: Switch Extensions::TabHelper to use WebContents, WebContentsUserData. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase only Created 8 years, 3 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/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index 03d0a58d72c3ae1368155316a9e8c9640a4bfa15..1070473cbd1ef027304a7d967d9ed858200da2e4 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -1537,9 +1537,11 @@ bool BrowserView::ShouldShowWindowTitle() const {
gfx::ImageSkia BrowserView::GetWindowAppIcon() {
if (browser_->is_app()) {
- TabContents* contents = chrome::GetActiveTabContents(browser_.get());
- if (contents && contents->extension_tab_helper()->GetExtensionAppIcon())
- return *contents->extension_tab_helper()->GetExtensionAppIcon();
+ WebContents* contents = chrome::GetActiveWebContents(browser_.get());
+ extensions::TabHelper* extensions_tab_helper =
+ contents ? extensions::TabHelper::FromWebContents(contents) : NULL;
+ if (extensions_tab_helper && extensions_tab_helper->GetExtensionAppIcon())
+ return *extensions_tab_helper->GetExtensionAppIcon();
}
return GetWindowIcon();
« no previous file with comments | « chrome/browser/ui/views/create_application_shortcut_view.cc ('k') | chrome/browser/ui/views/location_bar/location_bar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698