Start of New Autofill UI for Windows

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

Index: chrome/browser/ui/views/autofill/autofill_popup_view_views.h
diff --git a/chrome/browser/ui/views/autofill/autofill_popup_view_views.h b/chrome/browser/ui/views/autofill/autofill_popup_view_views.h
new file mode 100644
index 0000000000000000000000000000000000000000..fe464638eba51fdc4e62bf4eedc259aa0c37139d
--- /dev/null
+++ b/chrome/browser/ui/views/autofill/autofill_popup_view_views.h
@@ -0,0 +1,57 @@
+// 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_POPUP_VIEW_VIEWS_H_
+#define CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_POPUP_VIEW_VIEWS_H_
+
+#include "chrome/browser/autofill/autofill_popup_view.h"
+#include "content/public/browser/keyboard_listener.h"
+#include "ui/views/widget/widget_delegate.h"
+
+class AutofillExternalDelegateViews;
+
+namespace content {
+class RenderViewHost;
+class WebContents;
+}
+
+namespace gfx {
+class Rect;
+}
+
+// The View Autofill popup implementation.
+class AutofillPopupViewViews : public views::WidgetDelegateView,
+                               public AutofillPopupView,
+                               public KeyboardListener {
+ public:
+  AutofillPopupViewViews(content::WebContents* web_contents,
+                         AutofillExternalDelegateViews* external_delegate);
+  virtual ~AutofillPopupViewViews();

[sky, 2012/08/29 20:56:41]

If you can, make this private.

[csharp, 2012/08/31 19:36:07]

Done.

+
+ private:
+  class AutofillPopupWidget;
+
+  // views:Views implementation.
+  virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
+
+  // views::WidgetDelegateView implementation.
+  virtual void WindowClosing() OVERRIDE;
+
+  // AutofillPopupView implementation.
+  virtual void ShowInternal() OVERRIDE;
+  virtual void HideInternal() OVERRIDE;
+  virtual void InvalidateRow(size_t row) OVERRIDE;
+  virtual void ResizePopup() OVERRIDE;
+
+  // Used to determine if our already closing, which would mean we don't
+  // need to close the widget.
+  bool popup_closing_;

[Ilya Sherman, 2012/08/29 22:36:44]

nit: I don't see this used anywhere -- is it vestigial?

[csharp, 2012/08/31 19:36:07]

Correct, removed.

+
+  AutofillExternalDelegateViews* external_delegate_;  // Weak reference.
+  content::WebContents* web_contents_;  // Weak reference.
+
+  DISALLOW_COPY_AND_ASSIGN(AutofillPopupViewViews);
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_POPUP_VIEW_VIEWS_H_