diff options
| author | koz@chromium.org <koz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-17 07:33:32 +0000 |
|---|---|---|
| committer | koz@chromium.org <koz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-17 07:33:32 +0000 |
| commit | c8de167746aa49238fb5d6fe3192fc8a770af827 (patch) | |
| tree | 3b50a24d99f5c7aed368149eed35ea281c116c1b /chrome | |
| parent | 8c9f5ea59f3c73a69a5c57b5f2af49c3f99da807 (diff) | |
| download | chromium_src-c8de167746aa49238fb5d6fe3192fc8a770af827.zip chromium_src-c8de167746aa49238fb5d6fe3192fc8a770af827.tar.gz chromium_src-c8de167746aa49238fb5d6fe3192fc8a770af827.tar.bz2 | |
Add a drop shadow to the fullscreen exit bubble on windows.
BUG=100879
TEST=Visually confirm that when entering fullscreen, the exit bubble has a drop shadow.
Review URL: http://codereview.chromium.org/8565034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110465 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
4 files changed, 7 insertions, 4 deletions
diff --git a/chrome/browser/ui/fullscreen_exit_bubble.cc b/chrome/browser/ui/fullscreen_exit_bubble.cc index fd33364..1a6345a 100644 --- a/chrome/browser/ui/fullscreen_exit_bubble.cc +++ b/chrome/browser/ui/fullscreen_exit_bubble.cc @@ -12,7 +12,7 @@ #include "ui/base/l10n/l10n_util.h" #include "ui/gfx/rect.h" -const int FullscreenExitBubble::kPaddingPx = 8; +const int FullscreenExitBubble::kPaddingPx = 0; const int FullscreenExitBubble::kInitialDelayMs = 3800; const int FullscreenExitBubble::kIdleTimeMs = 2300; const int FullscreenExitBubble::kPositionCheckHz = 10; diff --git a/chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc b/chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc index 0cca0d6..2cfbe68 100644 --- a/chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc +++ b/chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc @@ -226,7 +226,8 @@ AutocompletePopupContentsView::AutocompletePopupContentsView( // The following little dance is required because set_border() requires a // pointer to a non-const object. views::BubbleBorder* bubble_border = - new views::BubbleBorder(views::BubbleBorder::NONE); + new views::BubbleBorder(views::BubbleBorder::NONE, + views::BubbleBorder::NO_SHADOW); bubble_border_ = bubble_border; set_border(bubble_border); // The contents is owned by the LocationBarView. diff --git a/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc b/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc index 1510640..fac2680 100644 --- a/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc +++ b/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc @@ -76,7 +76,8 @@ FullscreenExitBubbleViews::FullscreenExitView::FullscreenExitView( deny_button_(NULL), browser_fullscreen_exit_accelerator_(accelerator) { views::BubbleBorder* bubble_border = - new views::BubbleBorder(views::BubbleBorder::NONE); + new views::BubbleBorder(views::BubbleBorder::NONE, + views::BubbleBorder::SHADOW); set_background(new views::BubbleBackground(bubble_border)); set_border(bubble_border); set_focusable(false); diff --git a/chrome/browser/ui/views/notifications/balloon_view.cc b/chrome/browser/ui/views/notifications/balloon_view.cc index 75b93fa..2ce871c 100644 --- a/chrome/browser/ui/views/notifications/balloon_view.cc +++ b/chrome/browser/ui/views/notifications/balloon_view.cc @@ -109,7 +109,8 @@ BalloonViewImpl::BalloonViewImpl(BalloonCollection* collection) set_parent_owned(false); views::BubbleBorder* bubble_border = - new views::BubbleBorder(views::BubbleBorder::FLOAT); + new views::BubbleBorder(views::BubbleBorder::FLOAT, + views::BubbleBorder::NO_SHADOW); set_border(bubble_border); } |