Basic Drawn text for new GTK Autofill popup.

chrome/browser/autofill/autofill_popup_view.h

 // 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_AUTOFILL_AUTOFILL_POPUP_VIEW_H_
 #define CHROME_BROWSER_AUTOFILL_AUTOFILL_POPUP_VIEW_H_
 #pragma once
 
+#include <vector>
+
 #include "base/compiler_specific.h"
 #include "base/string16.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/notification_observer.h"
 #include "ui/gfx/rect.h"
 
 namespace content {
 class WebContents;
 }
 
 class AutofillPopupView : public content::NotificationObserver {
  public:
   explicit AutofillPopupView(content::WebContents* web_contents);
   virtual ~AutofillPopupView();
 
   // Hide the popup from view.
   virtual void Hide() = 0;
 
   // Display the autofill popup and fill it in with the values passed in.
-  virtual void Show(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,
-                    int separator_index) = 0;
+  // Platform independent work.

[Ilya Sherman, 2012/01/11 23:36:29]

nit: "Platform-independent"

[csharp, 2012/01/12 19:39:11]

Done.

+  void Show(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,
+            int separator_index);
 
 
   void set_element_bounds(const gfx::Rect& bounds) {
     element_bounds_ = bounds;
   }
 
   const gfx::Rect& element_bounds() { return element_bounds_; }
 
+ protected:
+  // Display the autofill popup and fill it in with the values passed in.
+  // Platform dependent work.

[Ilya Sherman, 2012/01/11 23:36:29]

nit: "Platform-dependent"

[csharp, 2012/01/12 19:39:11]

Done.

+  virtual void ShowInternal(const std::vector<string16>& autofill_values,
+                            const std::vector<string16>& autofill_labels,
+                            const std::vector<string16>& autofill_icons) = 0;

[Ilya Sherman, 2012/01/11 23:36:29]

nit: Hmm, since all of these vectors are exposed via methods below, do we really
need to pass them in as params?  Conversely, if we do need to pass them in as
params, do we really need them to be exposed via the methods below?

[csharp, 2012/01/12 19:39:11]

I removed passing them in, since they do need to be stored for them to still be
valid when actually drawing the text.

+
+  const std::vector<string16>& autofill_values() { return autofill_values_; }
+  const std::vector<string16>& autofill_labels() { return autofill_labels_; }
+  const std::vector<string16>& autofill_icons() { return autofill_icons_; }
+  const int separator_index() { return separator_index_; }

[Ilya Sherman, 2012/01/11 23:36:29]

nit: These methods should be marked const.  (Ah, const, you are such a virus :)

[csharp, 2012/01/12 19:39:11]

Done.

+
  private:
   // content::NotificationObserver method override.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   // A scoped container for notification registries.
   content::NotificationRegistrar registrar_;
 
   // The bounds of the text element that is the focus of the Autofill.
   gfx::Rect element_bounds_;
+
+  // The current autofill query values.

[Ilya Sherman, 2012/01/11 23:36:29]

nit: "autofill" -> "Autofill"

[csharp, 2012/01/12 19:39:11]

Done.

+  std::vector<string16> autofill_values_;
+  std::vector<string16> autofill_labels_;
+  std::vector<string16> autofill_icons_;
+  std::vector<int> autofill_unique_ids_;
+
+  // The location of the separator index (which separator the returned values

[Ilya Sherman, 2012/01/11 23:36:29]

nit: "which separator" -> "which separates"

[csharp, 2012/01/12 19:39:11]

Done.

+  // from autofill options).

[Ilya Sherman, 2012/01/11 23:36:29]

nit: "from autofill options" -> "from the Autofill options"

[csharp, 2012/01/12 19:39:11]

Done.

+  int separator_index_;
 };
 
 #endif  // CHROME_BROWSER_AUTOFILL_AUTOFILL_POPUP_VIEW_H_