Use high resolution icons where possible for streamlined hosted app icons.

chrome/browser/ui/views/frame/browser_view.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/frame/browser_view.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
@@ skipping 158 matching lines @@
 using web_modal::WebContentsModalDialogHost;
 
 namespace {
 // The name of a key to store on the window handle so that other code can
 // locate this object using just the handle.
 const char* const kBrowserViewKey = "__BROWSER_VIEW__";
 
 // The number of milliseconds between loading animation frames.
 const int kLoadingAnimationFrameTimeMs = 30;
 
+// The size of the window icon in pixels.

[tapted, 2013/11/11 08:52:57]

Are these device-independent pixels? (ImageFamily works in DIPs)

[calamity, 2013/11/13 06:37:14]

Removed.

+const int kWindowIconSize = 32;

[tapted, 2013/11/11 08:52:57]

I wonder if this should be IconUtil::kMediumIconSize (48), which icon_util.cc
claims is the Alt+Tab icon size on line 155.
http://msdn.microsoft.com/en-us/library/aa511280.aspx#size suggests this is the
case too.

[calamity, 2013/11/13 06:37:14]

Done.

+
 // TODO(kuan): These functions are temporarily for the bookmark bar while its
 // detached state is at the top of the page;  it'll be moved to float on the
 // content page in the very near future, at which time, these local functions
 // will be removed.
 void PaintDetachedBookmarkBar(gfx::Canvas* canvas,
                               DetachableToolbarView* view,
                               ThemeService* theme_service) {
   // Paint background for detached state; if animating, this is fade in/out.
   canvas->DrawColor(
       chrome::GetDetachedBookmarkBarBackgroundColor(theme_service));
@@ skipping 1371 matching lines @@
     if (extensions_tab_helper && extensions_tab_helper->GetExtensionAppIcon())
       return gfx::ImageSkia::CreateFrom1xBitmap(
           *extensions_tab_helper->GetExtensionAppIcon());
   }
 
   return GetWindowIcon();
 }
 
 gfx::ImageSkia BrowserView::GetWindowIcon() {
   if (browser_->is_app() || browser_->is_type_popup())
-    return browser_->GetCurrentPageIcon().AsImageSkia();
+    return browser_->GetCurrentPageIcon(kWindowIconSize).AsImageSkia();

[tapted, 2013/11/11 08:52:57]

Looks like c/b/ui/gtk/browser_titlebar.cc might use this too... but I guess ToT
is now linux_aura, so maybe you don't need to worry

[calamity, 2013/11/13 06:37:14]

Fixed it just in case.

   return gfx::ImageSkia();
 }
 
 bool BrowserView::ShouldShowWindowIcon() const {
   // For Ash only, app host windows do not show an icon, crbug.com/119411.
   // Child windows (i.e. popups) do show an icon.
   if (browser_->host_desktop_type() == chrome::HOST_DESKTOP_TYPE_ASH &&
       browser_->is_app() && browser_->app_type() == Browser::APP_TYPE_HOST)
     return false;
 
@@ skipping 1125 matching lines @@
     // The +1 in the next line creates a 1-px gap between icon and arrow tip.
     gfx::Point icon_bottom(0, location_icon_view->GetImageBounds().bottom() -
         LocationBarView::kIconInternalPadding + 1);
     ConvertPointToTarget(location_icon_view, this, &icon_bottom);
     gfx::Point infobar_top(0, infobar_container_->GetVerticalOverlap(NULL));
     ConvertPointToTarget(infobar_container_, this, &infobar_top);
     top_arrow_height = infobar_top.y() - icon_bottom.y();
   }
   return top_arrow_height;
 }