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

Side by Side Diff: extensions/browser/extension_host.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
« no previous file with comments | « content/test/test_web_contents.cc ('k') | extensions/browser/extension_host.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 EXTENSIONS_BROWSER_EXTENSION_HOST_H_ 5 #ifndef EXTENSIONS_BROWSER_EXTENSION_HOST_H_
6 #define EXTENSIONS_BROWSER_EXTENSION_HOST_H_ 6 #define EXTENSIONS_BROWSER_EXTENSION_HOST_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 10
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 // finished. 96 // finished.
97 void OnNetworkRequestDone(uint64 request_id); 97 void OnNetworkRequestDone(uint64 request_id);
98 98
99 // content::WebContentsObserver: 99 // content::WebContentsObserver:
100 bool OnMessageReceived(const IPC::Message& message) override; 100 bool OnMessageReceived(const IPC::Message& message) override;
101 void RenderViewCreated(content::RenderViewHost* render_view_host) override; 101 void RenderViewCreated(content::RenderViewHost* render_view_host) override;
102 void RenderViewDeleted(content::RenderViewHost* render_view_host) override; 102 void RenderViewDeleted(content::RenderViewHost* render_view_host) override;
103 void RenderViewReady() override; 103 void RenderViewReady() override;
104 void RenderProcessGone(base::TerminationStatus status) override; 104 void RenderProcessGone(base::TerminationStatus status) override;
105 void DocumentAvailableInMainFrame() override; 105 void DocumentAvailableInMainFrame() override;
106 void DidStartLoading(content::RenderViewHost* render_view_host) override; 106 void DidStartLoading() override;
107 void DidStopLoading(content::RenderViewHost* render_view_host) override; 107 void DidStopLoading() override;
108 108
109 // content::WebContentsDelegate: 109 // content::WebContentsDelegate:
110 content::JavaScriptDialogManager* GetJavaScriptDialogManager( 110 content::JavaScriptDialogManager* GetJavaScriptDialogManager(
111 content::WebContents* source) override; 111 content::WebContents* source) override;
112 void AddNewContents(content::WebContents* source, 112 void AddNewContents(content::WebContents* source,
113 content::WebContents* new_contents, 113 content::WebContents* new_contents,
114 WindowOpenDisposition disposition, 114 WindowOpenDisposition disposition,
115 const gfx::Rect& initial_rect, 115 const gfx::Rect& initial_rect,
116 bool user_gesture, 116 bool user_gesture,
117 bool* was_blocked) override; 117 bool* was_blocked) override;
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
203 ObserverList<ExtensionHostObserver> observer_list_; 203 ObserverList<ExtensionHostObserver> observer_list_;
204 ObserverList<DeferredStartRenderHostObserver> 204 ObserverList<DeferredStartRenderHostObserver>
205 deferred_start_render_host_observer_list_; 205 deferred_start_render_host_observer_list_;
206 206
207 DISALLOW_COPY_AND_ASSIGN(ExtensionHost); 207 DISALLOW_COPY_AND_ASSIGN(ExtensionHost);
208 }; 208 };
209 209
210 } // namespace extensions 210 } // namespace extensions
211 211
212 #endif // EXTENSIONS_BROWSER_EXTENSION_HOST_H_ 212 #endif // EXTENSIONS_BROWSER_EXTENSION_HOST_H_
OLDNEW
« no previous file with comments | « content/test/test_web_contents.cc ('k') | extensions/browser/extension_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698