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

Side by Side Diff: chrome/browser/extensions/api/identity/web_auth_flow.cc

Issue 1008913002: Remove RenderViewHost parameter from WebContentsObserver::Did{Start|Stop}Loading. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 9 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 (c) 2012 The Chromium Authors. All rights reserved. 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 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 "chrome/browser/extensions/api/identity/web_auth_flow.h" 5 #include "chrome/browser/extensions/api/identity/web_auth_flow.h"
6 6
7 #include "base/base64.h" 7 #include "base/base64.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/strings/string_util.h" 10 #include "base/strings/string_util.h"
(...skipping 213 matching lines...) Expand 10 before | Expand all | Expand 10 after
224 TRACE_EVENT_ASYNC_STEP_PAST1("identity", 224 TRACE_EVENT_ASYNC_STEP_PAST1("identity",
225 "WebAuthFlow", 225 "WebAuthFlow",
226 this, 226 this,
227 "DidFailProvisionalLoad", 227 "DidFailProvisionalLoad",
228 "error_code", 228 "error_code",
229 error_code); 229 error_code);
230 if (delegate_) 230 if (delegate_)
231 delegate_->OnAuthFlowFailure(LOAD_FAILED); 231 delegate_->OnAuthFlowFailure(LOAD_FAILED);
232 } 232 }
233 233
234 void WebAuthFlow::DidStopLoading(RenderViewHost* render_view_host) { 234 void WebAuthFlow::DidStopLoading() {
235 AfterUrlLoaded(); 235 AfterUrlLoaded();
236 } 236 }
237 237
238 void WebAuthFlow::DidNavigateMainFrame( 238 void WebAuthFlow::DidNavigateMainFrame(
239 const content::LoadCommittedDetails& details, 239 const content::LoadCommittedDetails& details,
240 const content::FrameNavigateParams& params) { 240 const content::FrameNavigateParams& params) {
241 if (delegate_ && details.http_status_code >= 400) 241 if (delegate_ && details.http_status_code >= 400)
242 delegate_->OnAuthFlowFailure(LOAD_FAILED); 242 delegate_->OnAuthFlowFailure(LOAD_FAILED);
243 } 243 }
244 244
245 } // namespace extensions 245 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/api/identity/web_auth_flow.h ('k') | chrome/browser/policy/policy_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698