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

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

Issue 18339006: Rename RenderViewGone IPC/methods to better reflect reality (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes for reviews by creis@ and palmer@. Created 7 years, 5 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 | Annotate | Revision Log
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 "apps/shell_window.h" 7 #include "apps/shell_window.h"
8 #include "base/base64.h" 8 #include "base/base64.h"
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/message_loop.h" 10 #include "base/message_loop.h"
(...skipping 189 matching lines...) Expand 10 before | Expand all | Expand 10 after
200 break; 200 break;
201 } 201 }
202 default: 202 default:
203 NOTREACHED() 203 NOTREACHED()
204 << "Got a notification that we did not register for: " << type; 204 << "Got a notification that we did not register for: " << type;
205 break; 205 break;
206 } 206 }
207 } 207 }
208 } 208 }
209 209
210 void WebAuthFlow::RenderViewGone(base::TerminationStatus status) { 210 void WebAuthFlow::RenderProcessGone(base::TerminationStatus status) {
211 if (delegate_) 211 if (delegate_)
212 delegate_->OnAuthFlowFailure(WebAuthFlow::WINDOW_CLOSED); 212 delegate_->OnAuthFlowFailure(WebAuthFlow::WINDOW_CLOSED);
213 } 213 }
214 214
215 void WebAuthFlow::DidStartProvisionalLoadForFrame( 215 void WebAuthFlow::DidStartProvisionalLoadForFrame(
216 int64 frame_id, 216 int64 frame_id,
217 int64 parent_frame_id, 217 int64 parent_frame_id,
218 bool is_main_frame, 218 bool is_main_frame,
219 const GURL& validated_url, 219 const GURL& validated_url,
220 bool is_error_page, 220 bool is_error_page,
(...skipping 18 matching lines...) Expand all
239 } 239 }
240 240
241 void WebAuthFlow::DidNavigateMainFrame( 241 void WebAuthFlow::DidNavigateMainFrame(
242 const content::LoadCommittedDetails& details, 242 const content::LoadCommittedDetails& details,
243 const content::FrameNavigateParams& params) { 243 const content::FrameNavigateParams& params) {
244 if (delegate_ && details.http_status_code >= 400) 244 if (delegate_ && details.http_status_code >= 400)
245 delegate_->OnAuthFlowFailure(LOAD_FAILED); 245 delegate_->OnAuthFlowFailure(LOAD_FAILED);
246 } 246 }
247 247
248 } // namespace extensions 248 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/api/identity/web_auth_flow.h ('k') | chrome/browser/extensions/extension_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698