MacViews: Audit TOOLKIT_VIEWS in chrome/browser/ui

chrome/browser/ui/autofill/card_unmask_prompt_view.cc

Index: chrome/browser/ui/autofill/card_unmask_prompt_view.cc
diff --git a/chrome/browser/ui/autofill/card_unmask_prompt_view.cc b/chrome/browser/ui/autofill/card_unmask_prompt_view.cc
index c420f916d805b0536eed54c702c2038b7e92cf13..469aa16c8cac263fdb83e7f3825e14a9936953eb 100644
--- a/chrome/browser/ui/autofill/card_unmask_prompt_view.cc
+++ b/chrome/browser/ui/autofill/card_unmask_prompt_view.cc
@@ -7,7 +7,7 @@
 
 namespace autofill {
 
-#if !defined(OS_ANDROID) && !defined(TOOLKIT_VIEWS)
+#if !defined(OS_ANDROID) && !defined(TOOLKIT_VIEWS) || defined(OS_MACOSX)

[msw, 2015/01/13 20:22:24]

nit: This seems specific to Mac and iOS, can we do this instead?:
#if defined(OS_MACOSX) || defined(OS_IOS)
(that seems sufficient from the list at src/build/build_config.h)

[tapted, 2015/01/14 00:07:38]

ooh - much nicer. In fact OS_IOS is only defined when OS_MACOSX already is, so
it can be just #if defined(OS_MACOSX), which is more consistent with other #ifs
under c/b/ui/autofill.

Done && checked on a bunch of builds to make sure it's all good.

[msw, 2015/01/14 00:09:13]

Nice.

 // static
 CardUnmaskPromptView* CardUnmaskPromptView::CreateAndShow(
     CardUnmaskPromptController* controller) {