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

Side by Side Diff: chrome/browser/extensions/chrome_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
« no previous file with comments | « no previous file | chrome/browser/extensions/chrome_extension_web_contents_observer.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 CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_WEB_CONTENTS_OBSERVER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_WEB_CONTENTS_OBSERVER_H_
6 #define CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_WEB_CONTENTS_OBSERVER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_WEB_CONTENTS_OBSERVER_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 21 matching lines...) Expand all
32 explicit ChromeExtensionWebContentsObserver( 32 explicit ChromeExtensionWebContentsObserver(
33 content::WebContents* web_contents); 33 content::WebContents* web_contents);
34 ~ChromeExtensionWebContentsObserver() override; 34 ~ChromeExtensionWebContentsObserver() override;
35 35
36 // ExtensionWebContentsObserver: 36 // ExtensionWebContentsObserver:
37 void InitializeRenderFrame( 37 void InitializeRenderFrame(
38 content::RenderFrameHost* render_frame_host) override; 38 content::RenderFrameHost* render_frame_host) override;
39 39
40 // content::WebContentsObserver overrides. 40 // content::WebContentsObserver overrides.
41 void RenderViewCreated(content::RenderViewHost* render_view_host) override; 41 void RenderViewCreated(content::RenderViewHost* render_view_host) override;
42 void DidCommitProvisionalLoadForFrame( 42 void DidFinishNavigation(
43 content::RenderFrameHost* render_frame_host, 43 content::NavigationHandle* navigation_handle) override;
44 const GURL& url,
45 ui::PageTransition transition_type) override;
46 44
47 // Silence a warning about hiding a virtual function. 45 // Silence a warning about hiding a virtual function.
48 bool OnMessageReceived(const IPC::Message& message, 46 bool OnMessageReceived(const IPC::Message& message,
49 content::RenderFrameHost* render_frame_host) override; 47 content::RenderFrameHost* render_frame_host) override;
50 48
51 // Adds a message to the extensions ErrorConsole. 49 // Adds a message to the extensions ErrorConsole.
52 void OnDetailedConsoleMessageAdded( 50 void OnDetailedConsoleMessageAdded(
53 content::RenderFrameHost* render_frame_host, 51 content::RenderFrameHost* render_frame_host,
54 const base::string16& message, 52 const base::string16& message,
55 const base::string16& source, 53 const base::string16& source,
56 const StackTrace& stack_trace, 54 const StackTrace& stack_trace,
57 int32_t severity_level); 55 int32_t severity_level);
58 56
59 // Reloads an extension if it is on the terminated list. 57 // Reloads an extension if it is on the terminated list.
60 void ReloadIfTerminated(content::RenderViewHost* render_view_host); 58 void ReloadIfTerminated(content::RenderViewHost* render_view_host);
61 59
62 // Determines which bucket of a synthetic field trial this client belongs 60 // Determines which bucket of a synthetic field trial this client belongs
63 // to and sets it. 61 // to and sets it.
64 void SetExtensionIsolationTrial(content::RenderFrameHost* render_frame_host); 62 void SetExtensionIsolationTrial(content::RenderFrameHost* render_frame_host);
65 63
66 DISALLOW_COPY_AND_ASSIGN(ChromeExtensionWebContentsObserver); 64 DISALLOW_COPY_AND_ASSIGN(ChromeExtensionWebContentsObserver);
67 }; 65 };
68 66
69 } // namespace extensions 67 } // namespace extensions
70 68
71 #endif // CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_WEB_CONTENTS_OBSERVER_H_ 69 #endif // CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_WEB_CONTENTS_OBSERVER_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/extensions/chrome_extension_web_contents_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698