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

Unified Diff: components/password_manager/content/browser/content_password_manager_driver_factory.cc

Issue 1415533013: Fix password manager internals renderer reporting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: All comments addressed Created 5 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: components/password_manager/content/browser/content_password_manager_driver_factory.cc
diff --git a/components/password_manager/content/browser/content_password_manager_driver_factory.cc b/components/password_manager/content/browser/content_password_manager_driver_factory.cc
index 222c743b3b837c3ebea7758f38f6a83700dbff56..5d09f4823d76c16a98d8a365b32f116da0e32995 100644
--- a/components/password_manager/content/browser/content_password_manager_driver_factory.cc
+++ b/components/password_manager/content/browser/content_password_manager_driver_factory.cc
@@ -77,8 +77,11 @@ ContentPasswordManagerDriverFactory::GetDriverForFrame(
void ContentPasswordManagerDriverFactory::RenderFrameCreated(
content::RenderFrameHost* render_frame_host) {
// This is called twice for the main frame.
- if (!ContainsKey(frame_driver_map_, render_frame_host))
- CreateDriverForFrame(render_frame_host);
+ if (!ContainsKey(frame_driver_map_, render_frame_host)) {
+ ContentPasswordManagerDriver* new_driver =
+ CreateDriverForFrame(render_frame_host);
+ new_driver->SendLoggingAvailability();
+ }
}
void ContentPasswordManagerDriverFactory::RenderFrameDeleted(
@@ -101,13 +104,16 @@ void ContentPasswordManagerDriverFactory::DidNavigateAnyFrame(
->second->DidNavigateFrame(details, params);
}
-void ContentPasswordManagerDriverFactory::CreateDriverForFrame(
+ContentPasswordManagerDriver*
+ContentPasswordManagerDriverFactory::CreateDriverForFrame(
content::RenderFrameHost* render_frame_host) {
DCHECK(!ContainsKey(frame_driver_map_, render_frame_host));
- frame_driver_map_.set(
- render_frame_host,
- make_scoped_ptr(new ContentPasswordManagerDriver(
- render_frame_host, password_client_, autofill_client_)));
+ scoped_ptr<ContentPasswordManagerDriver> driver(
+ new ContentPasswordManagerDriver(render_frame_host, password_client_,
+ autofill_client_));
+ ContentPasswordManagerDriver* weak_return_pointer = driver.get();
+ frame_driver_map_.set(render_frame_host, driver.Pass());
+ return weak_return_pointer;
}
void ContentPasswordManagerDriverFactory::TestingSetDriverForFrame(
@@ -116,4 +122,10 @@ void ContentPasswordManagerDriverFactory::TestingSetDriverForFrame(
frame_driver_map_.set(render_frame_host, driver.Pass());
}
+void ContentPasswordManagerDriverFactory::RequestSendLoggingAvailability() {
+ for (auto key_val_iterator : frame_driver_map_) {
+ key_val_iterator.second->SendLoggingAvailability();
+ }
+}
+
} // namespace password_manager

Powered by Google App Engine
This is Rietveld 408576698