Stop using chrome/browser/ui/views/bubble/bubble.h in fullscreen exit bubble.

chrome/browser/ui/views/fullscreen_exit_bubble_views.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/views/fullscreen_exit_bubble_views.h"
 
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
-#include "chrome/browser/ui/views/bubble/bubble.h"
 #include "grit/generated_resources.h"
 #include "ui/base/animation/slide_animation.h"
 #include "ui/base/keycodes/keyboard_codes.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/canvas_skia.h"
 #include "ui/gfx/screen.h"
 #include "views/bubble/bubble_border.h"
 #include "views/controls/button/text_button.h"
 #include "views/controls/link.h"
@@ skipping 53 matching lines @@
 #if !defined(OS_CHROMEOS)
   browser_fullscreen_exit_text_ =
       l10n_util::GetStringFUTF16(IDS_EXIT_FULLSCREEN_MODE, accelerator);
 #else
   browser_fullscreen_exit_text_ =
       l10n_util::GetStringUTF16(IDS_EXIT_FULLSCREEN_MODE);
 #endif
 
   views::BubbleBorder* bubble_border =
       new views::BubbleBorder(views::BubbleBorder::NONE);
-  bubble_border->set_background_color(Bubble::kBackgroundColor);
   set_background(new views::BubbleBackground(bubble_border));
   set_border(bubble_border);
   set_focusable(false);
 
   message_label_.set_parent_owned(false);
   message_label_.SetFont(ResourceBundle::GetSharedInstance().GetFont(
       ResourceBundle::MediumFont));
 
   link_.set_parent_owned(false);
   link_.set_collapse_when_hidden(false);
@@ skipping 121 matching lines @@
   size_animation_->Reset(1);
 
   // Create the contents view.
   ui::Accelerator accelerator(ui::VKEY_UNKNOWN, false, false, false);
   bool got_accelerator = frame->GetAccelerator(IDC_FULLSCREEN, &accelerator);
   DCHECK(got_accelerator);
   view_ = new FullscreenExitView(
       this, accelerator.GetShortcutText(), url, bubble_type_);
 
   // Initialize the popup.
   popup_ = new views::Widget;

[msw, 2011/11/16 22:05:40]

This wouldn't be necessary if we used the views bubble infrastructure.
Add a TODO or some comment to that effect here.

[yzshen1, 2011/11/17 00:55:45]

Done.

   views::Widget::InitParams params(views::Widget::InitParams::TYPE_POPUP);
   params.transparent = true;
   params.can_activate = false;
   params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
   params.parent = frame->GetNativeView();
   params.bounds = GetPopupRect(false);
   popup_->Init(params);
   gfx::Size size = GetPopupRect(true).size();
   popup_->SetContentsView(view_);
   // We set layout manager to NULL to prevent the widget from sizing its
@@ skipping 110 matching lines @@
     size.set_height(size.height() - popup_bottom + y_offset);
     origin.set_y(origin.y() - y_offset);
   }
   return gfx::Rect(origin, size);
 }
 
 void FullscreenExitBubbleViews::StartWatchingMouseIfNecessary() {
   if (!fullscreen_bubble::ShowButtonsForType(bubble_type_))
     StartWatchingMouse();
 }