[rAC, OSX] Enable requestAutocomplet for OSX

chrome/renderer/chrome_render_process_observer.cc

Index: chrome/renderer/chrome_render_process_observer.cc
diff --git a/chrome/renderer/chrome_render_process_observer.cc b/chrome/renderer/chrome_render_process_observer.cc
index a9020d62d5ac2ea7d49754806d1a5473ffb064be..4b6a1710213e0727072cb47db24b06d9c15cdb2e 100644
--- a/chrome/renderer/chrome_render_process_observer.cc
+++ b/chrome/renderer/chrome_render_process_observer.cc
@@ -165,9 +165,15 @@ ChromeRenderProcessObserver::ChromeRenderProcessObserver(
   }
 
 #if defined(ENABLE_AUTOFILL_DIALOG)
+#if defined(OS_MACOSX)

[sky, 2013/06/07 16:48:03]

Is there some central autofill class we can add a static method to that has this
check. I'm thinking something Autofill::IsEnableInteractiveAutocompleteEnabled()

[Albert Bodenhamer, 2013/06/07 17:03:59]

autofill_switches might work, but the code is only ever needed here.  I'd prefer
to limit the scope.

I'd lean towards pulling the check into a static function here.

+  bool enableAutofill = command_line.HasSwitch(
+      autofill::switches::kEnableInteractiveAutocomplete);
+#else
+  bool enableAutofill = !command_line.HasSwitch(
+      autofill::switches::kDisableInteractiveAutocomplete);
+#endif
   WebRuntimeFeatures::enableRequestAutocomplete(
-      !command_line.HasSwitch(
-          autofill::switches::kDisableInteractiveAutocomplete) ||
+      enableAutofill ||
       command_line.HasSwitch(switches::kEnableExperimentalWebKitFeatures));
 #endif