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

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

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 #ifndef CHROME_BROWSER_EXTENSIONS_API_IDENTITY_WEB_AUTH_FLOW_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_API_IDENTITY_WEB_AUTH_FLOW_H_
6 #define CHROME_BROWSER_EXTENSIONS_API_IDENTITY_WEB_AUTH_FLOW_H_ 6 #define CHROME_BROWSER_EXTENSIONS_API_IDENTITY_WEB_AUTH_FLOW_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "content/public/browser/notification_observer.h" 10 #include "content/public/browser/notification_observer.h"
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 // ::AppWindowRegistry::Observer implementation. 94 // ::AppWindowRegistry::Observer implementation.
95 void OnAppWindowAdded(AppWindow* app_window) override; 95 void OnAppWindowAdded(AppWindow* app_window) override;
96 void OnAppWindowRemoved(AppWindow* app_window) override; 96 void OnAppWindowRemoved(AppWindow* app_window) override;
97 97
98 // NotificationObserver implementation. 98 // NotificationObserver implementation.
99 void Observe(int type, 99 void Observe(int type,
100 const content::NotificationSource& source, 100 const content::NotificationSource& source,
101 const content::NotificationDetails& details) override; 101 const content::NotificationDetails& details) override;
102 102
103 // WebContentsObserver implementation. 103 // WebContentsObserver implementation.
104 void DidStopLoading(content::RenderViewHost* render_view_host) override; 104 void DidStopLoading() override;
105 void DidNavigateMainFrame( 105 void DidNavigateMainFrame(
106 const content::LoadCommittedDetails& details, 106 const content::LoadCommittedDetails& details,
107 const content::FrameNavigateParams& params) override; 107 const content::FrameNavigateParams& params) override;
108 void RenderProcessGone(base::TerminationStatus status) override; 108 void RenderProcessGone(base::TerminationStatus status) override;
109 void DidStartProvisionalLoadForFrame( 109 void DidStartProvisionalLoadForFrame(
110 content::RenderFrameHost* render_frame_host, 110 content::RenderFrameHost* render_frame_host,
111 const GURL& validated_url, 111 const GURL& validated_url,
112 bool is_error_page, 112 bool is_error_page,
113 bool is_iframe_srcdoc) override; 113 bool is_iframe_srcdoc) override;
114 void DidFailProvisionalLoad(content::RenderFrameHost* render_frame_host, 114 void DidFailProvisionalLoad(content::RenderFrameHost* render_frame_host,
(...skipping 14 matching lines...) Expand all
129 bool embedded_window_created_; 129 bool embedded_window_created_;
130 130
131 content::NotificationRegistrar registrar_; 131 content::NotificationRegistrar registrar_;
132 132
133 DISALLOW_COPY_AND_ASSIGN(WebAuthFlow); 133 DISALLOW_COPY_AND_ASSIGN(WebAuthFlow);
134 }; 134 };
135 135
136 } // namespace extensions 136 } // namespace extensions
137 137
138 #endif // CHROME_BROWSER_EXTENSIONS_API_IDENTITY_WEB_AUTH_FLOW_H_ 138 #endif // CHROME_BROWSER_EXTENSIONS_API_IDENTITY_WEB_AUTH_FLOW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698