Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(166)

Side by Side Diff: chrome/browser/ui/autofill/autofill_dialog_controller_browsertest.cc

Issue 12091086: [Autofill] Add UMA timing metrics for requestAutocomplete dialog. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: DialogRequester -> DialogType Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/bind.h"
6 #include "base/message_loop.h"
7 #include "base/time.h"
8 #include "base/utf_string_conversions.h"
9 #include "chrome/browser/autofill/autofill_metrics.h"
10 #include "chrome/browser/ui/autofill/autofill_dialog_controller_impl.h"
11 #include "chrome/browser/ui/autofill/autofill_dialog_view.h"
12 #include "chrome/browser/ui/browser.h"
13 #include "chrome/browser/ui/tabs/tab_strip_model.h"
14 #include "chrome/common/form_data.h"
15 #include "chrome/common/form_field_data.h"
16 #include "chrome/test/base/in_process_browser_test.h"
17 #include "content/public/test/test_utils.h"
18 #include "testing/gtest/include/gtest/gtest.h"
19
20 namespace autofill {
21
22 namespace {
23
24 void MockCallback(const FormStructure*) {}
25
26 class MockAutofillMetrics : public AutofillMetrics {
27 public:
28 MockAutofillMetrics()
29 : dialog_type_(static_cast<DialogType>(-1)),
30 metric_(static_cast<AutofillMetrics::DialogDismissalAction>(-1)) {}
31 virtual ~MockAutofillMetrics() {}
32
33 // AutofillMetrics:
34 virtual void LogRequestAutocompleteUiDuration(
35 const base::TimeDelta& duration,
36 DialogType dialog_type,
37 DialogDismissalAction dismissal_action) const OVERRIDE {
38 // Ignore constness for testing.
39 MockAutofillMetrics* mutable_this = const_cast<MockAutofillMetrics*>(this);
40 mutable_this->dialog_type_ = dialog_type;
41 mutable_this->metric_ = dismissal_action;
42 }
43
44 DialogType dialog_type() const { return dialog_type_; }
45 AutofillMetrics::DialogDismissalAction metric() const { return metric_; }
46
47 private:
48 DialogType dialog_type_;
49 AutofillMetrics::DialogDismissalAction metric_;
50
51 DISALLOW_COPY_AND_ASSIGN(MockAutofillMetrics);
52 };
53
54 class TestAutofillDialogController : public AutofillDialogControllerImpl {
55 public:
56 TestAutofillDialogController(content::WebContents* contents,
57 const FormData& form_data,
58 const AutofillMetrics& metric_logger,
59 const DialogType dialog_type)
60 : AutofillDialogControllerImpl(contents,
61 form_data,
62 GURL(),
63 content::SSLStatus(),
64 metric_logger,
65 dialog_type,
66 base::Bind(&MockCallback)) {
67 }
68
69 virtual ~TestAutofillDialogController() {}
70
71 virtual void ViewClosed(DialogAction action) OVERRIDE {
72 AutofillDialogControllerImpl::ViewClosed(action);
73 MessageLoop::current()->Quit();
74 }
75
76 // Increase visibility for testing.
77 AutofillDialogView* view() { return AutofillDialogControllerImpl::view(); }
78
79 private:
80 DISALLOW_COPY_AND_ASSIGN(TestAutofillDialogController);
81 };
82
83 } // namespace
84
85 class AutofillDialogControllerTest : public InProcessBrowserTest {
86 public:
87 AutofillDialogControllerTest() {}
88 virtual ~AutofillDialogControllerTest() {}
89
90 content::WebContents* GetActiveWebContents() {
91 return browser()->tab_strip_model()->GetActiveWebContents();
92 }
93
94 private:
95 DISALLOW_COPY_AND_ASSIGN(AutofillDialogControllerTest);
96 };
97
98 // TODO(isherman): Enable this test on other platforms once the UI is
99 // implemented on those platforms.
100 #if defined(TOOLKIT_VIEWS)
101 #define MAYBE_RequestAutocompleteUiDurationMetrics \
102 RequestAutocompleteUiDurationMetrics
103 #else
104 #define MAYBE_RequestAutocompleteUiDurationMetrics \
105 DISABLED_RequestAutocompleteUiDurationMetrics
106 #endif // defined(TOOLKIT_VIEWS)
107 IN_PROC_BROWSER_TEST_F(AutofillDialogControllerTest,
108 MAYBE_RequestAutocompleteUiDurationMetrics) {
109 FormData form;
110 form.name = ASCIIToUTF16("TestForm");
111 form.method = ASCIIToUTF16("POST");
112 form.origin = GURL("http://example.com/form.html");
113 form.action = GURL("http://example.com/submit.html");
114 form.user_submitted = true;
115
116 FormFieldData field;
117 field.autocomplete_attribute = "email";
118 form.fields.push_back(field);
119
120 // Submit the form data.
121 {
122 MockAutofillMetrics metric_logger;
123 TestAutofillDialogController* dialog_controller =
124 new TestAutofillDialogController(
125 GetActiveWebContents(), form, metric_logger,
126 DIALOG_TYPE_REQUEST_AUTOCOMPLETE);
127 dialog_controller->Show();
128 dialog_controller->view()->SubmitForTesting();
129
130 content::RunMessageLoop();
131
132 EXPECT_EQ(AutofillMetrics::DIALOG_ACCEPTED, metric_logger.metric());
133 EXPECT_EQ(DIALOG_TYPE_REQUEST_AUTOCOMPLETE, metric_logger.dialog_type());
134 }
135
136 // Cancel out of the dialog.
137 {
138 MockAutofillMetrics metric_logger;
139 TestAutofillDialogController* dialog_controller =
140 new TestAutofillDialogController(
141 GetActiveWebContents(), form, metric_logger,
142 DIALOG_TYPE_AUTOCHECKOUT);
143 dialog_controller->Show();
144 dialog_controller->view()->CancelForTesting();
145
146 content::RunMessageLoop();
147
148 EXPECT_EQ(AutofillMetrics::DIALOG_CANCELED, metric_logger.metric());
149 EXPECT_EQ(DIALOG_TYPE_AUTOCHECKOUT, metric_logger.dialog_type());
150 }
151
152 // Take some other action that dismisses the dialog.
153 {
154 MockAutofillMetrics metric_logger;
155 TestAutofillDialogController* dialog_controller =
156 new TestAutofillDialogController(
157 GetActiveWebContents(), form, metric_logger,
158 DIALOG_TYPE_AUTOCHECKOUT);
159 dialog_controller->Show();
160 dialog_controller->Hide();
161
162 content::RunMessageLoop();
163
164 EXPECT_EQ(AutofillMetrics::DIALOG_CANCELED, metric_logger.metric());
165 EXPECT_EQ(DIALOG_TYPE_AUTOCHECKOUT, metric_logger.dialog_type());
166 }
167 }
168
169 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698