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

Side by Side Diff: components/autofill/content/renderer/password_autofill_agent.cc

Issue 2680163006: Eliminate PasswordAutofillAgentConstructed() in mojo interface PasswordManagerDriver (Closed)
Patch Set: Address comments from vabr@ Created 3 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/autofill/content/renderer/password_autofill_agent.h" 5 #include "components/autofill/content/renderer/password_autofill_agent.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 558 matching lines...) Expand 10 before | Expand all | Expand 10 after
569 569
570 PasswordAutofillAgent::PasswordAutofillAgent(content::RenderFrame* render_frame) 570 PasswordAutofillAgent::PasswordAutofillAgent(content::RenderFrame* render_frame)
571 : content::RenderFrameObserver(render_frame), 571 : content::RenderFrameObserver(render_frame),
572 logging_state_active_(false), 572 logging_state_active_(false),
573 was_username_autofilled_(false), 573 was_username_autofilled_(false),
574 was_password_autofilled_(false), 574 was_password_autofilled_(false),
575 binding_(this) { 575 binding_(this) {
576 // PasswordAutofillAgent is guaranteed to outlive |render_frame|. 576 // PasswordAutofillAgent is guaranteed to outlive |render_frame|.
577 render_frame->GetInterfaceRegistry()->AddInterface( 577 render_frame->GetInterfaceRegistry()->AddInterface(
578 base::Bind(&PasswordAutofillAgent::BindRequest, base::Unretained(this))); 578 base::Bind(&PasswordAutofillAgent::BindRequest, base::Unretained(this)));
579 GetPasswordManagerDriver()->PasswordAutofillAgentConstructed();
580 } 579 }
581 580
582 PasswordAutofillAgent::~PasswordAutofillAgent() { 581 PasswordAutofillAgent::~PasswordAutofillAgent() {
583 } 582 }
584 583
585 void PasswordAutofillAgent::BindRequest( 584 void PasswordAutofillAgent::BindRequest(
586 mojom::PasswordAutofillAgentRequest request) { 585 mojom::PasswordAutofillAgentRequest request) {
587 binding_.Bind(std::move(request)); 586 binding_.Bind(std::move(request));
588 } 587 }
589 588
(...skipping 927 matching lines...) Expand 10 before | Expand all | Expand 10 after
1517 PasswordAutofillAgent::GetPasswordManagerDriver() { 1516 PasswordAutofillAgent::GetPasswordManagerDriver() {
1518 if (!password_manager_driver_) { 1517 if (!password_manager_driver_) {
1519 render_frame()->GetRemoteInterfaces()->GetInterface( 1518 render_frame()->GetRemoteInterfaces()->GetInterface(
1520 mojo::MakeRequest(&password_manager_driver_)); 1519 mojo::MakeRequest(&password_manager_driver_));
1521 } 1520 }
1522 1521
1523 return password_manager_driver_; 1522 return password_manager_driver_;
1524 } 1523 }
1525 1524
1526 } // namespace autofill 1525 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698