Chromium Code Reviews| Index: chrome/browser/ui/views/screen_capture_notification_ui_views.cc |
| diff --git a/chrome/browser/ui/views/screen_capture_notification_ui_views.cc b/chrome/browser/ui/views/screen_capture_notification_ui_views.cc |
| index 94819e3a4402d30ed542b0fac69216c62b0d540d..995546ab87944c0b81130c887f4599e747ccce57 100644 |
| --- a/chrome/browser/ui/views/screen_capture_notification_ui_views.cc |
| +++ b/chrome/browser/ui/views/screen_capture_notification_ui_views.cc |
| @@ -29,8 +29,9 @@ namespace { |
| const int kMinimumWidth = 460; |
| const int kMaximumWidth = 1000; |
| const int kHorizontalMargin = 10; |
| -const int kPadding = 5; |
| -const int kPaddingLeft = 10; |
| +const int kPaddingVertical = 5; |
| +const int kPaddingHorizontal = 10; |
| +const SkColor kHideLinkColor = SkColorSetRGB(0x55, 0x95, 0xFE); |
|
msw
2014/03/25 00:01:58
It's wrong to use a hard-coded color over a themed
|
| namespace { |
| @@ -150,6 +151,8 @@ ScreenCaptureNotificationUIViews::ScreenCaptureNotificationUIViews( |
| hide_link_ = new views::Link( |
| l10n_util::GetStringUTF16(IDS_PASSWORDS_PAGE_VIEW_HIDE_BUTTON)); |
| hide_link_->set_listener(this); |
| + hide_link_->SetUnderline(false); |
| + hide_link_->SetEnabledColor(kHideLinkColor); |
| AddChildView(hide_link_); |
| } |
| @@ -268,7 +271,10 @@ views::NonClientFrameView* |
| ScreenCaptureNotificationUIViews::CreateNonClientFrameView( |
| views::Widget* widget) { |
| views::BubbleFrameView* frame = new views::BubbleFrameView( |
| - gfx::Insets(kPadding, kPaddingLeft, kPadding, kPadding)); |
| + gfx::Insets(kPaddingVertical, |
| + kPaddingHorizontal, |
| + kPaddingVertical, |
| + kPaddingHorizontal)); |
| SkColor color = widget->GetNativeTheme()->GetSystemColor( |
| ui::NativeTheme::kColorId_DialogBackground); |
| frame->SetBubbleBorder(scoped_ptr<views::BubbleBorder>( |