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

chrome/browser/autofill/autofill_metrics_unittest.cc

 // 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.
 
 #include <vector>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/string16.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autofill/autofill_cc_infobar_delegate.h"
 #include "chrome/browser/autofill/autofill_common_test.h"
+#include "chrome/browser/autofill/autofill_flow_infobar_delegate.h"
 #include "chrome/browser/autofill/autofill_manager.h"
 #include "chrome/browser/autofill/autofill_manager_delegate.h"
 #include "chrome/browser/autofill/autofill_metrics.h"
 #include "chrome/browser/autofill/personal_data_manager.h"
 #include "chrome/browser/autofill/personal_data_manager_factory.h"
 #include "chrome/browser/ui/autofill/tab_autofill_manager_delegate.h"
 #include "chrome/browser/webdata/web_data_service.h"
 #include "chrome/common/form_data.h"
 #include "chrome/common/form_field_data.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/test/test_browser_thread.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/rect.h"
 
 using content::BrowserThread;
 using ::testing::_;
 using ::testing::AnyNumber;
 using ::testing::Mock;
 using base::TimeTicks;
 using base::TimeDelta;
 
 namespace {
 
 class MockAutofillMetrics : public AutofillMetrics {
  public:
   MockAutofillMetrics() {}
   MOCK_CONST_METHOD1(LogCreditCardInfoBarMetric, void(InfoBarMetric metric));
+  MOCK_CONST_METHOD1(LogAutofillFlowInfoBarMetric, void(InfoBarMetric metric));
   MOCK_CONST_METHOD1(LogDeveloperEngagementMetric,
                      void(DeveloperEngagementMetric metric));
   MOCK_CONST_METHOD3(LogHeuristicTypePrediction,
                      void(FieldTypeQualityMetric metric,
                           AutofillFieldType field_type,
                           const std::string& experiment_id));
   MOCK_CONST_METHOD3(LogOverallTypePrediction,
                      void(FieldTypeQualityMetric metric,
                           AutofillFieldType field_type,
                           const std::string& experiment_id));
@@ skipping 169 matching lines @@
     if (!did_finish_async_form_submit_) {
       // TODO(isherman): It seems silly to need this variable.  Is there some
       // way I can just query the message loop's state?
       message_loop_is_running_ = true;
       MessageLoop::current()->Run();
     } else {
       did_finish_async_form_submit_ = false;
     }
   }
 
+  void ShowAutofillFlowDialog(const FormData& form,
+                              const GURL& frame_url,
+                              const content::SSLStatus& ssl_status) {
+    // no-op. Just used as callback into autofill_flow_infobar_delegate.

[Albert Bodenhamer, 2013/01/14 22:10:18]

Is the callback required?

[Raman Kakilate, 2013/01/14 23:37:46]

Yes, Actual AutofillManager's ShowAutofillFlowDialog will try to show UI, and
hence tests would fail without this override.

+  }
+
   virtual void UploadFormDataAsyncCallback(
       const FormStructure* submitted_form,
       const base::TimeTicks& load_time,
       const base::TimeTicks& interaction_time,
       const base::TimeTicks& submission_time) OVERRIDE {
     if (message_loop_is_running_) {
       MessageLoop::current()->Quit();
       message_loop_is_running_ = false;
     } else {
       did_finish_async_form_submit_ = true;
@@ skipping 24 matching lines @@
   virtual ~AutofillMetricsTest();
 
   virtual void SetUp() OVERRIDE;
   virtual void TearDown() OVERRIDE;
 
  protected:
   scoped_ptr<ConfirmInfoBarDelegate> CreateDelegate(
       MockAutofillMetrics* metric_logger,
       CreditCard** created_card);
 
+  scoped_ptr<ConfirmInfoBarDelegate> CreateAutofillFlowDelegate(
+      MockAutofillMetrics* metric_logger);
+
   content::TestBrowserThread ui_thread_;
   content::TestBrowserThread file_thread_;
 
   scoped_refptr<TestAutofillManager> autofill_manager_;
   TestPersonalDataManager personal_data_;
 
  private:
   std::string default_gmock_verbosity_level_;
 
   DISALLOW_COPY_AND_ASSIGN(AutofillMetricsTest);
@@ skipping 55 matching lines @@
   EXPECT_CALL(*metric_logger,
               LogCreditCardInfoBarMetric(AutofillMetrics::INFOBAR_SHOWN));
   // The delegate created below will take ownership of this object.
   CreditCard* credit_card = new CreditCard();
   if (created_card)
     *created_card = credit_card;
   return AutofillCCInfoBarDelegate::Create(credit_card, &personal_data_,
                                            metric_logger);
 }
 
+scoped_ptr<ConfirmInfoBarDelegate>
+AutofillMetricsTest::CreateAutofillFlowDelegate(
+    MockAutofillMetrics* metric_logger) {
+  EXPECT_CALL(*metric_logger,
+              LogAutofillFlowInfoBarMetric(AutofillMetrics::INFOBAR_SHOWN));
+  GURL url("www.google.com");

[Albert Bodenhamer, 2013/01/14 22:10:18]

I think there's a header for getting google URLs.

[Raman Kakilate, 2013/01/14 23:37:46]

Done.

+  content::SSLStatus ssl_status;
+  return AutofillFlowInfoBarDelegate::Create(autofill_manager_.get(),
+                                             metric_logger,
+                                             url,
+                                             ssl_status);
+}
+
 // Test that we log quality metrics appropriately.
 TEST_F(AutofillMetricsTest, QualityMetrics) {
   // Set up our form data.
   FormData form;
   form.name = ASCIIToUTF16("TestForm");
   form.method = ASCIIToUTF16("POST");
   form.origin = GURL("http://example.com/form.html");
   form.action = GURL("http://example.com/submit.html");
   form.user_submitted = true;
 
@@ skipping 819 matching lines @@
   // Ignore the infobar.
   {
     scoped_ptr<ConfirmInfoBarDelegate> infobar(CreateDelegate(&metric_logger,
                                                               NULL));
     ASSERT_TRUE(infobar);
     EXPECT_CALL(metric_logger,
         LogCreditCardInfoBarMetric(AutofillMetrics::INFOBAR_IGNORED)).Times(1);
   }
 }
 
+// Test that autofill flow infobar metrics are logged correctly.
+TEST_F(AutofillMetricsTest, AutofillFlowInfoBar) {
+  MockAutofillMetrics metric_logger;
+  ::testing::InSequence dummy;
+
+  // Accept the infobar.
+  {
+    scoped_ptr<ConfirmInfoBarDelegate> infobar(
+        CreateAutofillFlowDelegate(&metric_logger));
+    ASSERT_TRUE(infobar);
+    EXPECT_CALL(metric_logger,
+        LogAutofillFlowInfoBarMetric(
+            AutofillMetrics::INFOBAR_ACCEPTED)).Times(1);
+    EXPECT_CALL(metric_logger,
+        LogAutofillFlowInfoBarMetric(
+            AutofillMetrics::INFOBAR_IGNORED)).Times(0);
+    EXPECT_TRUE(infobar->Accept());
+  }
+
+  // Cancel the infobar.
+  {
+    scoped_ptr<ConfirmInfoBarDelegate> infobar(
+        CreateAutofillFlowDelegate(&metric_logger));
+    ASSERT_TRUE(infobar);
+    EXPECT_CALL(metric_logger,
+        LogAutofillFlowInfoBarMetric(
+            AutofillMetrics::INFOBAR_DENIED)).Times(1);
+    EXPECT_CALL(metric_logger,
+        LogAutofillFlowInfoBarMetric(
+            AutofillMetrics::INFOBAR_IGNORED)).Times(0);
+    EXPECT_TRUE(infobar->Cancel());
+  }
+
+  // Dismiss the infobar.
+  {
+    scoped_ptr<ConfirmInfoBarDelegate> infobar(
+        CreateAutofillFlowDelegate(&metric_logger));
+    ASSERT_TRUE(infobar);
+    EXPECT_CALL(metric_logger,
+        LogAutofillFlowInfoBarMetric(
+            AutofillMetrics::INFOBAR_DENIED)).Times(1);
+    EXPECT_CALL(metric_logger,
+        LogAutofillFlowInfoBarMetric(
+            AutofillMetrics::INFOBAR_IGNORED)).Times(0);
+    infobar->InfoBarDismissed();
+  }
+
+  // Ignore the infobar.
+  {
+    scoped_ptr<ConfirmInfoBarDelegate> infobar(
+        CreateAutofillFlowDelegate(&metric_logger));
+    ASSERT_TRUE(infobar);
+    EXPECT_CALL(metric_logger,
+        LogAutofillFlowInfoBarMetric(
+            AutofillMetrics::INFOBAR_IGNORED)).Times(1);
+  }
+}
+
+
 // Test that server query response experiment id metrics are logged correctly.
 TEST_F(AutofillMetricsTest, ServerQueryExperimentIdForQuery) {
   MockAutofillMetrics metric_logger;
   ::testing::InSequence dummy;
 
   // No experiment specified.
   EXPECT_CALL(metric_logger,
               LogServerQueryMetric(AutofillMetrics::QUERY_RESPONSE_RECEIVED));
   EXPECT_CALL(metric_logger,
               LogServerQueryMetric(AutofillMetrics::QUERY_RESPONSE_PARSED));
@@ skipping 363 matching lines @@
     autofill_manager_->OnFormsSeen(forms, TimeTicks::FromInternalValue(1));
     autofill_manager_->OnDidFillAutofillFormData(
         TimeTicks::FromInternalValue(5));
     autofill_manager_->OnTextFieldDidChange(form, form.fields.front(),
                                             TimeTicks::FromInternalValue(3));
     autofill_manager_->FormSubmitted(form, TimeTicks::FromInternalValue(17));
     autofill_manager_->Reset();
     Mock::VerifyAndClearExpectations(autofill_manager_->metric_logger());
   }
 }