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

Unified Diff: chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc

Issue 13139004: Deprecate Browser::TYPE_PANEL (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 9 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/ash/launcher/browser_launcher_item_controller.cc
diff --git a/chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc b/chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc
index 83b2b4ed6253bbad6a9bb04e5929ad044c5cefcc..9bb5e44696f33ffbaba5b18f929f415a5ee5f780 100644
--- a/chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc
+++ b/chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc
@@ -104,14 +104,7 @@ BrowserLauncherItemController* BrowserLauncherItemController::Create(
type = TYPE_WINDOWED_APP;
}
} else if (browser->is_app()) {
- if (browser->is_type_panel()) {
- if (browser->app_type() == Browser::APP_TYPE_CHILD)
- type = TYPE_EXTENSION_PANEL;
- else
- type = TYPE_APP_PANEL;
- } else {
- type = TYPE_TABBED;
- }
+ type = TYPE_TABBED;
app_id = web_app::GetExtensionIdFromApplicationName(browser->app_name());
} else {
return NULL;
@@ -134,7 +127,7 @@ void BrowserLauncherItemController::BrowserActivationStateChanged() {
}
string16 BrowserLauncherItemController::GetTitle() {
- if (type() == TYPE_TABBED || type() == TYPE_EXTENSION_PANEL) {
+ if (type() == TYPE_TABBED) {
if (tab_model_->active_index() != TabStripModel::kNoTab) {
const content::WebContents* contents = tab_model_->GetActiveWebContents();
if (contents)
@@ -301,38 +294,17 @@ void BrowserLauncherItemController::UpdateLauncher(content::WebContents* tab) {
return;
ash::LauncherItem item = launcher_model()->items()[item_index];
- if (type() == TYPE_EXTENSION_PANEL) {
- if (!favicon_loader_.get() || favicon_loader_->web_contents() != tab)
- favicon_loader_.reset(new LauncherFaviconLoader(this, tab));
-
- // Update the icon for extension panels.
- extensions::TabHelper* extensions_tab_helper =
- extensions::TabHelper::FromWebContents(tab);
- gfx::ImageSkia new_image = gfx::ImageSkia::CreateFrom1xBitmap(
- favicon_loader_->GetFavicon());
- if (new_image.isNull() && extensions_tab_helper->GetExtensionAppIcon()) {
- new_image = gfx::ImageSkia::CreateFrom1xBitmap(
- *extensions_tab_helper->GetExtensionAppIcon());
- }
- // Only update the icon if we have a new image, or none has been set yet.
- // This avoids flickering to an empty image when a pinned app is opened.
- if (!new_image.isNull())
- item.image = new_image;
- else if (item.image.isNull())
- item.image = extensions::IconsInfo::GetDefaultAppIcon();
- } else {
- DCHECK_EQ(TYPE_TABBED, type());
- ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
- FaviconTabHelper* favicon_tab_helper =
- FaviconTabHelper::FromWebContents(tab);
- if (favicon_tab_helper->ShouldDisplayFavicon()) {
- item.image = favicon_tab_helper->GetFavicon().AsImageSkia();
- if (item.image.isNull()) {
- item.image = *rb.GetImageSkiaNamed(IDR_DEFAULT_FAVICON);
- }
- } else {
+ DCHECK_EQ(TYPE_TABBED, type());
+ ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
+ FaviconTabHelper* favicon_tab_helper =
+ FaviconTabHelper::FromWebContents(tab);
+ if (favicon_tab_helper->ShouldDisplayFavicon()) {
+ item.image = favicon_tab_helper->GetFavicon().AsImageSkia();
+ if (item.image.isNull()) {
item.image = *rb.GetImageSkiaNamed(IDR_DEFAULT_FAVICON);
}
+ } else {
+ item.image = *rb.GetImageSkiaNamed(IDR_DEFAULT_FAVICON);
}
launcher_model()->Set(item_index, item);
}

Powered by Google App Engine
This is Rietveld 408576698