Adding Mouse Support for new GTK Autofill

chrome/browser/autofill/autofill_external_delegate_test.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_EXTERNAL_DELEGATE_TEST_H_
+#define CHROME_BROWSER_AUTOFILL_AUTOFILL_EXTERNAL_DELEGATE_TEST_H_
+#pragma once
+
+#include "chrome/browser/autofill/autofill_external_delegate.h"
+#include "testing/gmock/include/gmock/gmock.h"

[Ilya Sherman, 2012/02/07 23:35:55]

nit: Looks like this isn't used, and it's a pretty expensive class to compile &
link.  (I think Nico once managed to improve the overall link-time memory
consumption when compiling unit tests by something like 25% just by fixing up
gmock.h includes.)

[csharp, 2012/02/08 16:11:16]

Done.

+
+class AutofillManager;
+class TabContentsWrapper;
+
+// This test class is meant to give tests a base AutofillExternalDelegate
+// class that requires no additional work to compile with (i.e. all the
+// pure virtual functions have been giving empty methods).
+class AutofillExternalDelegateTest : public AutofillExternalDelegate {

[Ilya Sherman, 2012/02/07 23:35:55]

nit: Generally a suffix of "Test" means the class is a test fixture class,
derived from testing::Test.  For testing versions of classes, we generally
prefix "Test" rather than suffix it.

[csharp, 2012/02/08 16:11:16]

Done.

+ public:
+  AutofillExternalDelegateTest(TabContentsWrapper* wrapper,
+                               AutofillManager* autofill_manager);
+  virtual ~AutofillExternalDelegateTest();
+
+  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,
+      int separator_index) OVERRIDE;
+
+  virtual void OnQueryPlatformSpecific(int query_id,
+                                       const webkit::forms::FormData& form,
+                                       const webkit::forms::FormField& field,
+                                       const gfx::Rect& bounds) OVERRIDE;
+
+
+  virtual void HideAutofillPopupInternal() OVERRIDE;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(AutofillExternalDelegateTest);
+};
+
+#endif  // CHROME_BROWSER_AUTOFILL_AUTOFILL_EXTERNAL_DELEGATE_TEST_H_