chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc - Issue 107623002: Use the NativeTheme to create the text color for text during the generated card animation.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc

Issue 107623002: Use the NativeTheme to create the text color for text during the generated card animation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Grow alpha for shading & border of overlay background as it approaches black. Created 7 years ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« no previous file with comments | « chrome/browser/ui/autofill/autofill_dialog_controller_impl.h ('k') | chrome/browser/ui/autofill/autofill_dialog_view_delegate.h » ('j') | chrome/browser/ui/views/autofill/autofill_dialog_views.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc

diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc

index 127706e00e66ec3e4b4e5dff3601c6df10de40d0..550b9cbc0290202e296f81c38a4ce947130f9b86 100644

--- a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc

+++ b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc

@@ -87,6 +87,7 @@

#include "ui/gfx/canvas.h"

#include "ui/gfx/image/image_skia_operations.h"

#include "ui/gfx/skia_util.h"

+#include "ui/native_theme/native_theme.h"

namespace autofill {

@@ -772,7 +773,8 @@ bool AutofillDialogControllerImpl::IsDialogButtonEnabled(

return !is_submitting_ || IsSubmitPausedOn(wallet::VERIFY_CVV);

}

-DialogOverlayState AutofillDialogControllerImpl::GetDialogOverlay() {

+DialogOverlayState AutofillDialogControllerImpl::GetDialogOverlay(

+ const ui::NativeTheme* theme) {

bool show_wallet_interstitial = IsPayingWithWallet() && is_submitting_ &&

!(full_wallet_ && !full_wallet_->required_actions().empty());

if (!show_wallet_interstitial) {

@@ -784,7 +786,8 @@ DialogOverlayState AutofillDialogControllerImpl::GetDialogOverlay() {

ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();

DialogOverlayState state;

state.string.font = rb.GetFont(ui::ResourceBundle::BaseFont).DeriveFont(3);

- state.string.text_color = SK_ColorDKGRAY;

+ state.string.text_color = theme->GetSystemColor(

+ ui::NativeTheme::kColorId_TextfieldReadOnlyColor);

const SkColor start_top_color = SkColorSetRGB(0xD6, 0xD6, 0xD6);

const SkColor start_bottom_color = SkColorSetRGB(0x98, 0x98, 0x98);