Start of New Autofill UI for Windows

chrome/browser/ui/views/autofill/autofill_external_delegate_views.h

Index: chrome/browser/ui/views/autofill/autofill_external_delegate_views.h
diff --git a/chrome/browser/ui/views/autofill/autofill_external_delegate_views.h b/chrome/browser/ui/views/autofill/autofill_external_delegate_views.h
new file mode 100644
index 0000000000000000000000000000000000000000..9367c0a3fc8f12713e89b233e12b610d8c1a4bf1
--- /dev/null
+++ b/chrome/browser/ui/views/autofill/autofill_external_delegate_views.h
@@ -0,0 +1,51 @@
+// 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.
+
+#ifndef CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_EXTERNAL_DELEGATE_VIEWS_H_
+#define CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_EXTERNAL_DELEGATE_VIEWS_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/autofill/autofill_external_delegate.h"
+
+class AutofillPopupViewViews;

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

Wonderful. :P

+
+namespace content {
+class WebContents;
+}
+
+class AutofillExternalDelegateViews : public AutofillExternalDelegate {
+ public:
+  AutofillExternalDelegateViews(TabContents* tab_contents,
+                                AutofillManager* autofill_manager);
+
+  virtual ~AutofillExternalDelegateViews();
+
+ protected:
+  // AutofillExternalDelegate implementations.

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

nit: "implementations" -> "implementation"

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

Done.

+  virtual void HideAutofillPopupInternal() OVERRIDE;
+  virtual void OnQueryPlatformSpecific(
+      int query_id,
+      const webkit::forms::FormData& form,
+      const webkit::forms::FormField& field,
+      const gfx::Rect& bounds) OVERRIDE;
+  virtual void ApplyAutofillSuggestions(
+      const std::vector<string16>& autofill_values,
+      const std::vector<string16>& autofill_labels,
+      const std::vector<string16>& autofill_icons,
+      const std::vector<int>& autofill_unique_ids) OVERRIDE;
+  virtual void SetBounds(const gfx::Rect& bounds) OVERRIDE;
+
+ private:
+  // Create a valid view to display the autofill results if one doesn't
+  // currently exist.
+  void CreateViewIfNeeded();
+
+  scoped_ptr<AutofillPopupViewViews> view_;
+
+  content::WebContents* web_contents_;  // Weak reference.

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

nit: The parent class should probably just have a protected accessor to its
WebContents, rather than needing to store it here.

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

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(AutofillExternalDelegateViews);
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_EXTERNAL_DELEGATE_VIEWS_H_