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

Side by Side Diff: extensions/browser/extension_web_contents_observer.h

Issue 2655853007: Convert ChromeExtensionWebContentsObserver to use the new navigation callbacks. (Closed)
Patch Set: nit Created 3 years, 10 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 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_WEB_CONTENTS_OBSERVER_H_ 5 #ifndef EXTENSIONS_BROWSER_EXTENSION_WEB_CONTENTS_OBSERVER_H_
6 #define EXTENSIONS_BROWSER_EXTENSION_WEB_CONTENTS_OBSERVER_H_ 6 #define EXTENSIONS_BROWSER_EXTENSION_WEB_CONTENTS_OBSERVER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 // ExtensionFunctionDispatcher::Delegate overrides. 79 // ExtensionFunctionDispatcher::Delegate overrides.
80 content::WebContents* GetAssociatedWebContents() const override; 80 content::WebContents* GetAssociatedWebContents() const override;
81 81
82 // content::WebContentsObserver overrides. 82 // content::WebContentsObserver overrides.
83 83
84 // A subclass should invoke this method to finish extension process setup. 84 // A subclass should invoke this method to finish extension process setup.
85 void RenderViewCreated(content::RenderViewHost* render_view_host) override; 85 void RenderViewCreated(content::RenderViewHost* render_view_host) override;
86 86
87 void RenderFrameCreated(content::RenderFrameHost* render_frame_host) override; 87 void RenderFrameCreated(content::RenderFrameHost* render_frame_host) override;
88 void RenderFrameDeleted(content::RenderFrameHost* render_frame_host) override; 88 void RenderFrameDeleted(content::RenderFrameHost* render_frame_host) override;
89 void DidCommitProvisionalLoadForFrame( 89 void DidFinishNavigation(
90 content::RenderFrameHost* render_frame_host, 90 content::NavigationHandle* navigation_handle) override;
91 const GURL& url,
92 ui::PageTransition transition_type) override;
93 void DidNavigateAnyFrame(content::RenderFrameHost* render_frame_host, 91 void DidNavigateAnyFrame(content::RenderFrameHost* render_frame_host,
94 const content::LoadCommittedDetails& details, 92 const content::LoadCommittedDetails& details,
95 const content::FrameNavigateParams& params) override; 93 const content::FrameNavigateParams& params) override;
96 94
97 // Subclasses should call this first before doing their own message handling. 95 // Subclasses should call this first before doing their own message handling.
98 bool OnMessageReceived(const IPC::Message& message, 96 bool OnMessageReceived(const IPC::Message& message,
99 content::RenderFrameHost* render_frame_host) override; 97 content::RenderFrameHost* render_frame_host) override;
100 98
101 // Per the documentation in WebContentsObserver, these two methods are invoked 99 // Per the documentation in WebContentsObserver, these two methods are invoked
102 // when a Pepper plugin instance is attached/detached in the page DOM. 100 // when a Pepper plugin instance is attached/detached in the page DOM.
(...skipping 26 matching lines...) Expand all
129 content::BrowserContext* browser_context_; 127 content::BrowserContext* browser_context_;
130 128
131 ExtensionFunctionDispatcher dispatcher_; 129 ExtensionFunctionDispatcher dispatcher_;
132 130
133 DISALLOW_COPY_AND_ASSIGN(ExtensionWebContentsObserver); 131 DISALLOW_COPY_AND_ASSIGN(ExtensionWebContentsObserver);
134 }; 132 };
135 133
136 } // namespace extensions 134 } // namespace extensions
137 135
138 #endif // EXTENSIONS_BROWSER_EXTENSION_WEB_CONTENTS_OBSERVER_H_ 136 #endif // EXTENSIONS_BROWSER_EXTENSION_WEB_CONTENTS_OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698