Start of New Autofill UI for Windows

chrome/browser/autofill/autofill_external_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autofill/autocomplete_history_manager.h"
 #include "chrome/browser/autofill/autofill_external_delegate.h"
 #include "chrome/browser/autofill/autofill_manager.h"
 #include "chrome/common/autofill_messages.h"
 #include "chrome/common/chrome_constants.h"
@@ skipping 327 matching lines @@
   autofill_icons->insert(autofill_icons->begin(),
                          data_list_icons_.begin(),
                          data_list_icons_.end());
   autofill_unique_ids->insert(autofill_unique_ids->begin(),
                               data_list_unique_ids_.begin(),
                               data_list_unique_ids_.end());
 }
 
 // Add a "!defined(OS_YOUROS) for each platform that implements this
 // in an autofill_external_delegate_YOUROS.cc.  Currently there are
 // none, so all platforms use the default.

[Ilya Sherman, 2012/08/14 05:12:09]

nit: Please remove the "Currently there are none sentence.  Also, maybe it's
worth reversing this check to just be checking for platforms that *don't* define
the UI?  (So, just Mac.)

[csharp, 2012/08/14 19:06:13]

Done.

 
-#if !defined(OS_ANDROID) && !defined(TOOLKIT_GTK)
+#if !defined(OS_ANDROID) && !defined(TOOLKIT_GTK) && !defined(TOOLKIT_VIEWS)
 
 AutofillExternalDelegate* AutofillExternalDelegate::Create(
     TabContents*, AutofillManager*) {
   return NULL;
 }
 
 #endif