AutofillDriverFactory manages information about user gestures

components/autofill/core/browser/autofill_driver_factory.cc

 // Copyright 2017 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 "components/autofill/core/browser/autofill_driver_factory.h"
 
 #include "base/callback.h"
 #include "components/autofill/core/browser/autofill_client.h"
 #include "components/autofill/core/browser/autofill_driver.h"
 
 namespace autofill {
 
 AutofillDriverFactory::AutofillDriverFactory(AutofillClient* client)
     : client_(client) {}
 
 AutofillDriverFactory::~AutofillDriverFactory() {}
 
 AutofillDriver* AutofillDriverFactory::DriverForKey(void* key) {
   auto mapping = driver_map_.find(key);
   return mapping == driver_map_.end() ? nullptr : mapping->second.get();
 }
 
+void AutofillDriverFactory::NavigationFinished() {
+  user_gesture_seen_ = false;
+  client_->HideAutofillPopup();
+}
+
+void AutofillDriverFactory::TabHidden() {
+  client_->HideAutofillPopup();
+}
+
+void AutofillDriverFactory::OnFirstUserGestureObserved() {
+  if (user_gesture_seen_)
+    return;
+
+  for (auto& driver : driver_map_) {

[dvadym, 2017/02/24 12:48:21]

Nit: do we need {} here?

[vabr (Chromium), 2017/02/24 12:54:25]

We don't need them, but they are allowed [1]. However, I should at least be
consistent with lines 50-51 below, so I'm removing the braces here. Thanks for
pointing this out.

[1]
https://google.github.io/styleguide/cppguide.html#Loops_and_Switch_Statements

+    driver.second->NotifyFirstUserGestureObservedInTab();
+  }
+
+  user_gesture_seen_ = true;
+}
+
 void AutofillDriverFactory::AddForKey(
     void* key,
     base::Callback<std::unique_ptr<AutofillDriver>()> factory_method) {
   auto insertion_result = driver_map_.insert(std::make_pair(key, nullptr));
   // This can be called twice for the key representing the main frame.
-  if (insertion_result.second)
+  if (insertion_result.second) {
     insertion_result.first->second = factory_method.Run();
+    if (user_gesture_seen_)
+      insertion_result.first->second->NotifyFirstUserGestureObservedInTab();
+  }
 }
 
 void AutofillDriverFactory::DeleteForKey(void* key) {
   driver_map_.erase(key);
 }
 
-void AutofillDriverFactory::NavigationFinished() {
-  client_->HideAutofillPopup();
-}
-
-void AutofillDriverFactory::TabHidden() {
-  client_->HideAutofillPopup();
-}
-
 }  // namespace autofill