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

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

Issue 2851173004: Eliminate bind callback that doesn't take a BindSourceInfo parameter. (Closed)
Patch Set: . Created 3 years, 7 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/autofill_agent.h" 5 #include "components/autofill/content/renderer/autofill_agent.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <tuple> 9 #include <tuple>
10 10
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 render_frame->GetWebFrame()->SetAutofillClient(this); 158 render_frame->GetWebFrame()->SetAutofillClient(this);
159 password_autofill_agent->SetAutofillAgent(this); 159 password_autofill_agent->SetAutofillAgent(this);
160 160
161 // AutofillAgent is guaranteed to outlive |render_frame|. 161 // AutofillAgent is guaranteed to outlive |render_frame|.
162 render_frame->GetInterfaceRegistry()->AddInterface( 162 render_frame->GetInterfaceRegistry()->AddInterface(
163 base::Bind(&AutofillAgent::BindRequest, base::Unretained(this))); 163 base::Bind(&AutofillAgent::BindRequest, base::Unretained(this)));
164 } 164 }
165 165
166 AutofillAgent::~AutofillAgent() {} 166 AutofillAgent::~AutofillAgent() {}
167 167
168 void AutofillAgent::BindRequest(mojom::AutofillAgentRequest request) { 168 void AutofillAgent::BindRequest(
169 const service_manager::BindSourceInfo& source_info,
170 mojom::AutofillAgentRequest request) {
169 binding_.Bind(std::move(request)); 171 binding_.Bind(std::move(request));
170 } 172 }
171 173
172 bool AutofillAgent::FormDataCompare::operator()(const FormData& lhs, 174 bool AutofillAgent::FormDataCompare::operator()(const FormData& lhs,
173 const FormData& rhs) const { 175 const FormData& rhs) const {
174 return std::tie(lhs.name, lhs.origin, lhs.action, lhs.is_form_tag) < 176 return std::tie(lhs.name, lhs.origin, lhs.action, lhs.is_form_tag) <
175 std::tie(rhs.name, rhs.origin, rhs.action, rhs.is_form_tag); 177 std::tie(rhs.name, rhs.origin, rhs.action, rhs.is_form_tag);
176 } 178 }
177 179
178 void AutofillAgent::DidCommitProvisionalLoad(bool is_new_navigation, 180 void AutofillAgent::DidCommitProvisionalLoad(bool is_new_navigation,
(...skipping 605 matching lines...) Expand 10 before | Expand all | Expand 10 after
784 return autofill_driver_; 786 return autofill_driver_;
785 } 787 }
786 788
787 const mojom::PasswordManagerDriverPtr& 789 const mojom::PasswordManagerDriverPtr&
788 AutofillAgent::GetPasswordManagerDriver() { 790 AutofillAgent::GetPasswordManagerDriver() {
789 DCHECK(password_autofill_agent_); 791 DCHECK(password_autofill_agent_);
790 return password_autofill_agent_->GetPasswordManagerDriver(); 792 return password_autofill_agent_->GetPasswordManagerDriver();
791 } 793 }
792 794
793 } // namespace autofill 795 } // namespace autofill
OLDNEW
« no previous file with comments | « components/autofill/content/renderer/autofill_agent.h ('k') | components/autofill/content/renderer/password_autofill_agent.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698