Pop up requestAutocomplete UI when autofill server hints chrome client that it is in a multipage au…

chrome/browser/autofill/autocheckout_manager.h

+// Copyright (c) 2013 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_AUTOFILL_AUTOCHECKOUT_MANAGER_H_
+#define CHROME_BROWSER_AUTOFILL_AUTOCHECKOUT_MANAGER_H_
+
+#include <string>
+
+#include "base/memory/weak_ptr.h"
+
+class AutofillManager;
+class FormStructure;
+class GURL;
+
+struct FormData;
+struct FormFieldData;

[Ilya Sherman, 2013/01/18 01:45:02]

nit: Not needed.

[Raman Kakilate, 2013/01/18 04:22:30]

Done.

+
+namespace content {
+class WebContents;

[Ilya Sherman, 2013/01/18 01:45:02]

nit: Not needed.

[Raman Kakilate, 2013/01/18 04:22:30]

Done.

+struct SSLStatus;
+}
+
+class AutocheckoutManager {
+ public:
+  explicit AutocheckoutManager(AutofillManager* autofill_manager);
+
+  virtual void ShowAutocheckoutDialog(const GURL& frame_url,
+                                      const content::SSLStatus& ssl_status);
+
+  ~AutocheckoutManager();

[Ilya Sherman, 2013/01/18 01:45:02]

nit: Since this class includes a virtual method, the destructor should be
virtual.

[Raman Kakilate, 2013/01/18 04:22:30]

Done.

+
+ private:
+
+  // Callback called from AutofillDialogController on filling up the UI form.
+  void ReturnAutocheckoutData(const FormStructure* result);
+
+  AutofillManager* autofill_manager_;

[Ilya Sherman, 2013/01/18 01:45:02]

nit: Please add the following comment: "// WEAK; owns us".

[Raman Kakilate, 2013/01/18 04:22:30]

Done.

+  base::WeakPtrFactory<AutocheckoutManager> weak_ptr_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(AutocheckoutManager);
+};
+
+#endif  // CHROME_BROWSER_AUTOFILL_AUTOCHECKOUT_MANAGER_H_