Update Panel UI looks on Windows.

chrome/browser/ui/panels/panel_browser_frame_view.cc

 // Copyright (c) 2011 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/panels/panel_browser_frame_view.h"
 
 #include <algorithm>
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/themes/theme_service.h"
+#include "chrome/browser/themes/theme_service_factory.h"
 #include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/panels/panel_browser_view.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
 #include "chrome/browser/ui/panels/panel_settings_menu_model.h"
 #include "chrome/common/extensions/extension.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "grit/theme_resources_standard.h"
 #include "grit/ui_resources.h"
 #include "third_party/skia/include/effects/SkGradientShader.h"
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/animation/slide_animation.h"
 #include "ui/base/hit_test.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/canvas_skia.h"
 #include "ui/gfx/path.h"
 #include "ui/gfx/screen.h"
+#include "ui/gfx/skia_util.h"
 #include "ui/views/widget/widget_delegate.h"
 #include "views/controls/button/image_button.h"
 #include "views/controls/button/menu_button.h"
 #include "views/controls/label.h"
 #include "views/controls/menu/menu_item_view.h"
 #include "views/controls/menu/menu_model_adapter.h"
 #include "views/controls/menu/menu_runner.h"
 #include "views/painter.h"
 
 namespace {
 
 // The height in pixels of the titlebar.
 const int kTitlebarHeight = 24;
 
-// The thickness in pixels of the frame border.
-const int kFrameBorderThickness = 1;
+// The thickness in pixels of the border.
+const int kBorderThickness = 1;
 
-// The spacing in pixels between the icon and the border/text.
-const int kIconSpacing = 4;
+// No client edge is present.
+const int kPanelClientEdgeThickness = 0;
+
+// The spacing in pixels between the icon and the left border.
+const int kIconAndBorderSpacing = 4;
 
 // The height and width in pixels of the icon.
 const int kIconSize = 16;
 
-// The spacing in pixels between buttons or the button and the adjacent control.
-const int kButtonSpacing = 6;
+// The spacing in pixels between the title and the icon on the left, or the
+// button on the right.
+const int kTitleSpacing = 8;
+
+// The spacing in pixels between the close button and the right border.
+const int kCloseButtonAndBorderSpacing = 8;
+
+// The spacing in pixels between the close button and the settings button.
+const int kSettingsButtonAndCloseButtonSpacing = 8;
 
 // This value is experimental and subjective.
 const int kUpdateSettingsVisibilityAnimationMs = 120;
 
-// Colors used in painting the titlebar for drawing attention.
-const SkColor kBackgroundColorForAttention = 0xfffa983a;
-const SkColor kTitleTextColorForAttention = SK_ColorWHITE;
+// Colors used to draw active titlebar under default theme.
+const SkColor kActiveTitleTextDefaultColor = SK_ColorBLACK;
+const SkColor kActiveBackgroundDefaultColorStart = 0xfff0f8fa;
+const SkColor kActiveBackgroundDefaultColorEnd = 0xffd1e2ed;
+
+// Colors used to draw inactive titlebar under default theme.
+const SkColor kInactiveTitleTextDefaultColor = 0xc0000000;
+const SkColor kInactiveBackgroundDefaultColorStart = 0xffffffff;
+const SkColor kInactiveBackgroundDefaultColorEnd = 0xffe7edf1;
+
+// Alpha value used in drawing inactive titlebar under default theme.
+const U8CPU kInactiveAlphaBlending = 0xc0;
+
+// Colors used to draw titlebar for drawing attention regardless of the theme.
+const SkColor kAttentionTitleTextColor = SK_ColorWHITE;
+const SkColor kAttentionBackgroundColorStart = 0xffffab57;
+const SkColor kAttentionBackgroundColorEnd = 0xfff59338;
+
+// Color used to draw the border.
+const SkColor kBorderColor = 0xc0000000;
+
+// Color used to draw the divider line between the titlebar and the client area.
+const SkColor kDividerColor = 0xffb5b5b5;
 
 struct ButtonResources {
   SkBitmap* normal_image;
   SkBitmap* mask_image;
   SkBitmap* hover_image;
   SkBitmap* pushed_image;
 
   void SetResources(int normal_image_id, int mask_image_id, int hover_image_id,
                     int pushed_image_id) {
     ResourceBundle& rb = ResourceBundle::GetSharedInstance();
@@ skipping 25 matching lines @@
     bottom_right = rb.GetBitmapNamed(bottom_right_id);
     bottom = rb.GetBitmapNamed(bottom_id);
     bottom_left = rb.GetBitmapNamed(bottom_left_id);
     left = rb.GetBitmapNamed(left_id);
   }
 };
 
 ButtonResources settings_button_resources;
 ButtonResources close_button_resources;
 EdgeResources frame_edges;
-EdgeResources client_edges;
 gfx::Font* title_font = NULL;
-SkBitmap* background_bitmap_for_attention = NULL;
+SkPaint active_background_default_paint;

[Dmitry Titov, 2011/11/29 04:04:03]

I wonder if this is a good thing to have objects with complex destructors as
static. They tend to have unpredictable timing of destruction during process
shutdown. 
It would be better to use CR_DEFINE_STATIC_LOCAL (see for example,
browser_list.cc)

[jianli, 2011/11/29 19:30:15]

Done.

+SkPaint inactive_background_default_paint;
+SkPaint attention_background_paint;
+
+void InitializeGradientPaint(SkPaint* paint,

[Dmitry Titov, 2011/11/29 04:04:03]

why not return the SkPaint*, especially if statics are not
statically-initialized?

[jianli, 2011/11/29 19:30:15]

I still need this to do the delay initialization when I change to use
CR_DEFINE_STATIC_LOCAL.

+                             SkColor start_color,
+                             SkColor end_color) {
+  SkShader* shader = gfx::CreateGradientShader(
+      0, kTitlebarHeight, start_color, end_color);
+  paint->setStyle(SkPaint::kFill_Style);
+  paint->setAntiAlias(true);
+  paint->setShader(shader);
+  shader->unref();
+}
 
 void LoadImageResources() {
   settings_button_resources.SetResources(
       IDR_BALLOON_WRENCH, 0, IDR_BALLOON_WRENCH_H, IDR_BALLOON_WRENCH_P);
 
   close_button_resources.SetResources(
       IDR_TAB_CLOSE, IDR_TAB_CLOSE_MASK, IDR_TAB_CLOSE_H, IDR_TAB_CLOSE_P);
 
   frame_edges.SetResources(
       IDR_WINDOW_TOP_LEFT_CORNER, IDR_WINDOW_TOP_CENTER,
       IDR_WINDOW_TOP_RIGHT_CORNER, IDR_WINDOW_RIGHT_SIDE,
       IDR_PANEL_BOTTOM_RIGHT_CORNER, IDR_WINDOW_BOTTOM_CENTER,
       IDR_PANEL_BOTTOM_LEFT_CORNER, IDR_WINDOW_LEFT_SIDE);
-
-  client_edges.SetResources(
-      IDR_APP_TOP_LEFT, IDR_APP_TOP_CENTER,
-      IDR_APP_TOP_RIGHT, IDR_CONTENT_RIGHT_SIDE,
-      IDR_CONTENT_BOTTOM_RIGHT_CORNER, IDR_CONTENT_BOTTOM_CENTER,
-      IDR_CONTENT_BOTTOM_LEFT_CORNER, IDR_CONTENT_LEFT_SIDE);
 }
 
 void EnsureResourcesInitialized() {
   static bool resources_initialized = false;
   if (resources_initialized)
     return;
   resources_initialized = true;
 
   ResourceBundle& rb = ResourceBundle::GetSharedInstance();
-  title_font = new gfx::Font(rb.GetFont(ResourceBundle::BaseFont));
+  title_font = new gfx::Font(rb.GetFont(ResourceBundle::BoldFont));
 
-  // Creates a bitmap of the specified color.
-  background_bitmap_for_attention = new SkBitmap();
-  background_bitmap_for_attention->setConfig(
-      SkBitmap::kARGB_8888_Config, 16, 16);
-  background_bitmap_for_attention->allocPixels();
-  background_bitmap_for_attention->eraseColor(kBackgroundColorForAttention);
+  InitializeGradientPaint(&active_background_default_paint,
+                          kActiveBackgroundDefaultColorStart,
+                          kActiveBackgroundDefaultColorEnd);
+  InitializeGradientPaint(&inactive_background_default_paint,
+                          kInactiveBackgroundDefaultColorStart,
+                          kInactiveBackgroundDefaultColorEnd);
+  InitializeGradientPaint(&attention_background_paint,
+                          kAttentionBackgroundColorStart,
+                          kAttentionBackgroundColorEnd);
 
   LoadImageResources();
 }
 
 }  // namespace
 
 // PanelBrowserFrameView::MouseWatcher -----------------------------------------
 
 PanelBrowserFrameView::MouseWatcher::MouseWatcher(PanelBrowserFrameView* view)
     : view_(view),
@@ skipping 177 matching lines @@
   int window_component = GetHTComponentForFrame(point,
       NonClientBorderThickness(), NonClientBorderThickness(),
       0, 0,
       frame()->widget_delegate()->CanResize());
   // Fall back to the caption if no other component matches.
   return (window_component == HTNOWHERE) ? HTCAPTION : window_component;
 }
 
 void PanelBrowserFrameView::GetWindowMask(const gfx::Size& size,
                                           gfx::Path* window_mask) {
-  // For panel, the window shape is rectangle with top-left and top-right
-  // corners rounded.
-  if (size.height() <= Panel::kMinimizedPanelHeight) {
-    // For minimize panel, we need to produce the window mask applicable to
-    // the 3-pixel lines.
-    window_mask->moveTo(0, SkIntToScalar(size.height()));
-    window_mask->lineTo(0, 1);
-    window_mask->lineTo(1, 0);
-    window_mask->lineTo(SkIntToScalar(size.width()) - 1, 0);
-    window_mask->lineTo(SkIntToScalar(size.width()), 1);
-    window_mask->lineTo(SkIntToScalar(size.width()),
-                        SkIntToScalar(size.height()));
-  } else {
-    window_mask->moveTo(0, 3);
-    window_mask->lineTo(1, 2);
-    window_mask->lineTo(1, 1);
-    window_mask->lineTo(2, 1);
-    window_mask->lineTo(3, 0);
-    window_mask->lineTo(SkIntToScalar(size.width() - 3), 0);
-    window_mask->lineTo(SkIntToScalar(size.width() - 2), 1);
-    window_mask->lineTo(SkIntToScalar(size.width() - 1), 1);
-    window_mask->lineTo(SkIntToScalar(size.width() - 1), 2);
-    window_mask->lineTo(SkIntToScalar(size.width()), 3);
-    window_mask->lineTo(SkIntToScalar(size.width()),
-                        SkIntToScalar(size.height()));
-    window_mask->lineTo(0, SkIntToScalar(size.height()));
-  }
+  window_mask->moveTo(0, 3);
+  window_mask->lineTo(1, 2);
+  window_mask->lineTo(1, 1);
+  window_mask->lineTo(2, 1);
+  window_mask->lineTo(3, 0);
+  window_mask->lineTo(SkIntToScalar(size.width() - 3), 0);
+  window_mask->lineTo(SkIntToScalar(size.width() - 2), 1);
+  window_mask->lineTo(SkIntToScalar(size.width() - 1), 1);
+  window_mask->lineTo(SkIntToScalar(size.width() - 1), 2);
+  window_mask->lineTo(SkIntToScalar(size.width() - 1), 3);
+  window_mask->lineTo(SkIntToScalar(size.width()),
+                      SkIntToScalar(size.height()));
+  window_mask->lineTo(0, SkIntToScalar(size.height()));
   window_mask->close();
 }
 
 void PanelBrowserFrameView::ResetWindowControls() {
   // The close button isn't affected by this constraint.
 }
 
 void PanelBrowserFrameView::UpdateWindowIcon() {
   title_icon_->SchedulePaint();
 }
 
 void PanelBrowserFrameView::OnPaint(gfx::Canvas* canvas) {
   // The font and color need to be updated depending on the panel's state.
   PaintState paint_state;
   if (panel_browser_view_->panel()->IsDrawingAttention())
     paint_state = PAINT_FOR_ATTENTION;
   else if (panel_browser_view_->focused())
     paint_state = PAINT_AS_ACTIVE;
   else
     paint_state = PAINT_AS_INACTIVE;
 
   UpdateControlStyles(paint_state);
   PaintFrameBorder(canvas);
-  PaintClientEdge(canvas);
 }
 
 void PanelBrowserFrameView::OnThemeChanged() {
   LoadImageResources();
 }
 
 gfx::Size PanelBrowserFrameView::GetMinimumSize() {
-  // This makes the panel be able to shrink to very small, like 3-pixel lines.
+  // This makes the panel be able to shrink to very small, like 4-pixel lines.
   // Since the panel cannot be resized by the user, we do not need to enforce
   // the minimum size.
   return gfx::Size();
 }
 
 void PanelBrowserFrameView::Layout() {
   // Cancel the settings button animation if the layout of titlebar is being
   // updated.
   if (settings_button_animator_.get() && settings_button_animator_->IsShowing())
     settings_button_animator_->Reset();
 
   // Layout the close button.
   gfx::Size close_button_size = close_button_->GetPreferredSize();
   close_button_->SetBounds(
-      width() - kFrameBorderThickness - kButtonSpacing -
+      width() - kBorderThickness - kCloseButtonAndBorderSpacing -
           close_button_size.width(),
       (NonClientTopBorderHeight() - close_button_size.height()) / 2,
       close_button_size.width(),
       close_button_size.height());
 
   // Layout the settings button.
   gfx::Size settings_button_size = settings_button_->GetPreferredSize();
   settings_button_->SetBounds(
-      close_button_->x() - kButtonSpacing - settings_button_size.width(),
+      close_button_->x() - kSettingsButtonAndCloseButtonSpacing -
+          settings_button_size.width(),
       (NonClientTopBorderHeight() - settings_button_size.height()) / 2,
       settings_button_size.width(),
       settings_button_size.height());
 
   // Trace the full bounds and zero-size bounds for animation purpose.
   settings_button_full_bounds_ = settings_button_->bounds();
   settings_button_zero_bounds_.SetRect(
       settings_button_full_bounds_.x() +
           settings_button_full_bounds_.width() / 2,
       settings_button_full_bounds_.y() +
           settings_button_full_bounds_.height() / 2,
       0,
       0);
 
   // Layout the icon.
   int icon_y = (NonClientTopBorderHeight() - kIconSize) / 2;
   title_icon_->SetBounds(
-      kFrameBorderThickness + kIconSpacing,
+      kBorderThickness + kIconAndBorderSpacing,
       icon_y,
       kIconSize,
       kIconSize);
 
   // Layout the title.
-  int title_x = title_icon_->bounds().right() + kIconSpacing;
+  int title_x = title_icon_->bounds().right() + kTitleSpacing;
   int title_height = BrowserFrame::GetTitleFont().GetHeight();
   title_label_->SetBounds(
       title_x,
       icon_y + ((kIconSize - title_height - 1) / 2),
-      std::max(0, settings_button_->x() - kButtonSpacing - title_x),
+      std::max(0, settings_button_->x() - kTitleSpacing - title_x),
       title_height);
 
   // Calculate the client area bounds.
   int top_height = NonClientTopBorderHeight();
   int border_thickness = NonClientBorderThickness();
   client_view_bounds_.SetRect(
       border_thickness,
       top_height,
       std::max(0, width() - (2 * border_thickness)),
       std::max(0, height() - top_height - border_thickness));
@@ skipping 81 matching lines @@
     AnimationEnded(animation);
   else
     settings_button_->SetBoundsRect(new_bounds);
 }
 
 void PanelBrowserFrameView::AnimationCanceled(const ui::Animation* animation) {
   AnimationEnded(animation);
 }
 
 int PanelBrowserFrameView::NonClientBorderThickness() const {
-  return kFrameBorderThickness + kClientEdgeThickness;
+  return kBorderThickness + kPanelClientEdgeThickness;
 }
 
 int PanelBrowserFrameView::NonClientTopBorderHeight() const {
-  return kFrameBorderThickness + kTitlebarHeight + kClientEdgeThickness;
+  return kBorderThickness + kTitlebarHeight + kPanelClientEdgeThickness;
 }
 
 gfx::Size PanelBrowserFrameView::NonClientAreaSize() const {
   return gfx::Size(NonClientBorderThickness() * 2,
                    NonClientTopBorderHeight() + NonClientBorderThickness());
 }
 
+bool PanelBrowserFrameView::UsingDefaultTheme() const {
+  ThemeService* theme_service = ThemeServiceFactory::GetForProfile(
+      panel_browser_view_->panel()->browser()->profile());
+  return theme_service->UsingDefaultTheme();
+}
+
+SkColor PanelBrowserFrameView::GetDefaultTitleColor(
+    PaintState paint_state) const {
+  switch (paint_state) {
+    case PAINT_AS_INACTIVE:
+      return kActiveTitleTextDefaultColor;
+    case PAINT_AS_ACTIVE:
+      return kInactiveTitleTextDefaultColor;
+    case PAINT_FOR_ATTENTION:
+      return kAttentionTitleTextColor;
+    default:
+      NOTREACHED();
+      return SkColor();
+  }
+}
+
 SkColor PanelBrowserFrameView::GetTitleColor(PaintState paint_state) const {
   switch (paint_state) {
     case PAINT_AS_INACTIVE:
-      return GetThemeProvider()->GetColor(
-          ThemeService::COLOR_BACKGROUND_TAB_TEXT);
+      return SkColorSetA(
+          GetThemeProvider()->GetColor(ThemeService::COLOR_BACKGROUND_TAB_TEXT),
+          kInactiveAlphaBlending);
     case PAINT_AS_ACTIVE:
       return GetThemeProvider()->GetColor(ThemeService::COLOR_TAB_TEXT);
     case PAINT_FOR_ATTENTION:
-      return kTitleTextColorForAttention;
+      return kAttentionTitleTextColor;
     default:
       NOTREACHED();
       return SkColor();
   }
 }
 
 gfx::Font* PanelBrowserFrameView::GetTitleFont() const {
   return title_font;
 }
 
+SkPaint* PanelBrowserFrameView::GetDefaultFrameTheme(
+    PaintState paint_state) const {
+  switch (paint_state) {
+    case PAINT_AS_INACTIVE:
+      return &inactive_background_default_paint;
+    case PAINT_AS_ACTIVE:
+      return &active_background_default_paint;
+    case PAINT_FOR_ATTENTION:
+      return &attention_background_paint;
+    default:
+      NOTREACHED();
+      return NULL;
+  }
+}
+
 SkBitmap* PanelBrowserFrameView::GetFrameTheme(PaintState paint_state) const {
   switch (paint_state) {
     case PAINT_AS_INACTIVE:
       return GetThemeProvider()->GetBitmapNamed(IDR_THEME_TAB_BACKGROUND);
     case PAINT_AS_ACTIVE:
       return GetThemeProvider()->GetBitmapNamed(IDR_THEME_TOOLBAR);
     case PAINT_FOR_ATTENTION:
-      return background_bitmap_for_attention;
+      // Background color for drawing attention is same regardless of the
+      // theme. GetDefaultFrameTheme should be used.
     default:
       NOTREACHED();
       return NULL;
   }
 }
 
 void PanelBrowserFrameView::UpdateControlStyles(PaintState paint_state) {
   DCHECK(paint_state != NOT_PAINTED);
 
   if (paint_state == paint_state_)
     return;
   paint_state_ = paint_state;
 
   SkColor title_color = GetTitleColor(paint_state_);
   title_label_->SetEnabledColor(title_color);
   title_label_->SetFont(*GetTitleFont());
 
   close_button_->SetBackground(title_color,
                                close_button_resources.normal_image,
                                close_button_resources.mask_image);
 }
 
 void PanelBrowserFrameView::PaintFrameBorder(gfx::Canvas* canvas) {
-  SkBitmap* theme_frame = GetFrameTheme(paint_state_);
-
-  // Draw the theme frame.
-  canvas->TileImageInt(*theme_frame, 0, 0, width(), height());
-
-  // No need to paint other stuff if panel is minimized.
-  if (height() <= Panel::kMinimizedPanelHeight)
-    return;
+  // Paint the background.
+  if (paint_state_ == PAINT_FOR_ATTENTION || UsingDefaultTheme()) {
+    SkPaint* paint = GetDefaultFrameTheme(paint_state_);
+    canvas->DrawRectInt(0, 0, width(), kTitlebarHeight, *paint);
+  } else {
+    SkBitmap* bitmap = GetFrameTheme(paint_state_);
+    canvas->TileImageInt(*bitmap, 0, 0, width(), kTitlebarHeight);
+  }
 
   // Draw the top border.
   canvas->DrawBitmapInt(*(frame_edges.top_left), 0, 0);
   canvas->TileImageInt(
       *(frame_edges.top), frame_edges.top_left->width(), 0,
       width() - frame_edges.top_right->width(), frame_edges.top->height());
   canvas->DrawBitmapInt(
       *(frame_edges.top_right),
       width() - frame_edges.top_right->width(), 0);
 
@@ skipping 18 matching lines @@
   canvas->DrawBitmapInt(
       *(frame_edges.bottom_left), 0,
       height() - frame_edges.bottom_left->height());
 
   // Draw the left border.
   canvas->TileImageInt(
       *(frame_edges.left), 0, frame_edges.top_left->height(),
       frame_edges.left->width(),
       height() - frame_edges.top_left->height() -
           frame_edges.bottom_left->height());
-}
 
-void PanelBrowserFrameView::PaintClientEdge(gfx::Canvas* canvas) {
-  int client_area_top = client_view_bounds_.y();
-
-  // No need to paint other stuff if panel is minimized.
-  if (height() <= Panel::kMinimizedPanelHeight)
-    return;
-
-  // Draw the top edge.
-  int top_edge_y = client_area_top - client_edges.top->height();
-  canvas->DrawBitmapInt(
-      *(client_edges.top_left),
-      client_view_bounds_.x() - client_edges.top_left->width(),
-      top_edge_y);
-  canvas->TileImageInt(
-      *(client_edges.top), client_view_bounds_.x(), top_edge_y,
-      client_view_bounds_.width(), client_edges.top->height());
-  canvas->DrawBitmapInt(
-      *(client_edges.top_right), client_view_bounds_.right(), top_edge_y);
-
-  // Draw the right edge.
-  int client_area_bottom =
-      std::max(client_area_top, client_view_bounds_.bottom());
-  int client_area_height = client_area_bottom - client_area_top;
-  canvas->TileImageInt(
-      *(client_edges.right), client_view_bounds_.right(), client_area_top,
-      client_edges.right->width(), client_area_height);
-
-  // Draw the bottom edge.
-  canvas->DrawBitmapInt(
-      *(client_edges.bottom_right), client_view_bounds_.right(),
-      client_area_bottom);
-  canvas->TileImageInt(
-      *(client_edges.bottom), client_view_bounds_.x(), client_area_bottom,
-      client_view_bounds_.width(), client_edges.bottom_right->height());
-  canvas->DrawBitmapInt(
-      *(client_edges.bottom_left),
-      client_view_bounds_.x() - client_edges.bottom_left->width(),
-      client_area_bottom);
-
-  // Draw the left edge.
-  canvas->TileImageInt(
-      *(client_edges.left),
-      client_view_bounds_.x() - client_edges.left->width(),
-      client_area_top, client_edges.left->width(), client_area_height);
+  // Draw the divider between the titlebar and the client area.
+  if (height() > kTitlebarHeight) {
+    canvas->DrawRectInt(kDividerColor, kBorderThickness, kTitlebarHeight,
+                        width() - 1 - 2 * kBorderThickness, kBorderThickness);
+  }
 }
 
 string16 PanelBrowserFrameView::GetTitleText() const {
   return frame()->widget_delegate()->GetWindowTitle();
 }
 
 void PanelBrowserFrameView::UpdateTitleBar() {
   title_label_->SetText(GetTitleText());
 }
 
@@ skipping 45 matching lines @@
 
   settings_menu_model_.reset(
       new PanelSettingsMenuModel(panel_browser_view_->panel()));
   settings_menu_adapter_.reset(
       new views::MenuModelAdapter(settings_menu_model_.get()));
   settings_menu_ = new views::MenuItemView(settings_menu_adapter_.get());
   settings_menu_adapter_->BuildMenu(settings_menu_);
   settings_menu_runner_.reset(new views::MenuRunner(settings_menu_));
   return true;
 }