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

Side by Side Diff: chrome/renderer/extensions/chrome_extensions_renderer_client.h

Issue 2459143002: Avoid GURL allocations/copies/frees in RenderFrameImpl::willSendRequest (Closed)
Patch Set: s/host/host_piece Created 4 years, 1 month 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 CHROME_RENDERER_EXTENSIONS_CHROME_EXTENSIONS_RENDERER_CLIENT_H_ 5 #ifndef CHROME_RENDERER_EXTENSIONS_CHROME_EXTENSIONS_RENDERER_CLIENT_H_
6 #define CHROME_RENDERER_EXTENSIONS_CHROME_EXTENSIONS_RENDERER_CLIENT_H_ 6 #define CHROME_RENDERER_EXTENSIONS_CHROME_EXTENSIONS_RENDERER_CLIENT_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 10
11 #include "base/macros.h" 11 #include "base/macros.h"
12 #include "extensions/renderer/extensions_renderer_client.h" 12 #include "extensions/renderer/extensions_renderer_client.h"
13 #include "ui/base/page_transition_types.h" 13 #include "ui/base/page_transition_types.h"
14 14
15 class ChromeExtensionsDispatcherDelegate; 15 class ChromeExtensionsDispatcherDelegate;
16 class GURL; 16 class GURL;
17 17
18 namespace blink { 18 namespace blink {
19 class WebFrame; 19 class WebFrame;
20 class WebLocalFrame; 20 class WebLocalFrame;
21 struct WebPluginParams; 21 struct WebPluginParams;
22 class WebURL;
22 } 23 }
23 24
24 namespace content { 25 namespace content {
25 class BrowserPluginDelegate; 26 class BrowserPluginDelegate;
26 class RenderFrame; 27 class RenderFrame;
27 class RenderView; 28 class RenderView;
28 } 29 }
29 30
30 namespace extensions { 31 namespace extensions {
31 class Dispatcher; 32 class Dispatcher;
(...skipping 17 matching lines...) Expand all
49 50
50 // See ChromeContentRendererClient methods with the same names. 51 // See ChromeContentRendererClient methods with the same names.
51 void RenderThreadStarted(); 52 void RenderThreadStarted();
52 void RenderFrameCreated(content::RenderFrame* render_frame); 53 void RenderFrameCreated(content::RenderFrame* render_frame);
53 void RenderViewCreated(content::RenderView* render_view); 54 void RenderViewCreated(content::RenderView* render_view);
54 bool OverrideCreatePlugin(content::RenderFrame* render_frame, 55 bool OverrideCreatePlugin(content::RenderFrame* render_frame,
55 const blink::WebPluginParams& params); 56 const blink::WebPluginParams& params);
56 bool AllowPopup(); 57 bool AllowPopup();
57 bool WillSendRequest(blink::WebFrame* frame, 58 bool WillSendRequest(blink::WebFrame* frame,
58 ui::PageTransition transition_type, 59 ui::PageTransition transition_type,
59 const GURL& url, 60 const blink::WebURL& url,
60 GURL* new_url); 61 GURL* new_url);
61 void SetExtensionDispatcherForTest( 62 void SetExtensionDispatcherForTest(
62 std::unique_ptr<extensions::Dispatcher> extension_dispatcher); 63 std::unique_ptr<extensions::Dispatcher> extension_dispatcher);
63 extensions::Dispatcher* GetExtensionDispatcherForTest(); 64 extensions::Dispatcher* GetExtensionDispatcherForTest();
64 65
65 static bool ShouldFork(blink::WebLocalFrame* frame, 66 static bool ShouldFork(blink::WebLocalFrame* frame,
66 const GURL& url, 67 const GURL& url,
67 bool is_initial_navigation, 68 bool is_initial_navigation,
68 bool is_server_redirect, 69 bool is_server_redirect,
69 bool* send_referrer); 70 bool* send_referrer);
(...skipping 16 matching lines...) Expand all
86 std::unique_ptr<extensions::RendererPermissionsPolicyDelegate> 87 std::unique_ptr<extensions::RendererPermissionsPolicyDelegate>
87 permissions_policy_delegate_; 88 permissions_policy_delegate_;
88 std::unique_ptr<extensions::ExtensionsGuestViewContainerDispatcher> 89 std::unique_ptr<extensions::ExtensionsGuestViewContainerDispatcher>
89 guest_view_container_dispatcher_; 90 guest_view_container_dispatcher_;
90 std::unique_ptr<extensions::ResourceRequestPolicy> resource_request_policy_; 91 std::unique_ptr<extensions::ResourceRequestPolicy> resource_request_policy_;
91 92
92 DISALLOW_COPY_AND_ASSIGN(ChromeExtensionsRendererClient); 93 DISALLOW_COPY_AND_ASSIGN(ChromeExtensionsRendererClient);
93 }; 94 };
94 95
95 #endif // CHROME_RENDERER_EXTENSIONS_CHROME_EXTENSIONS_RENDERER_CLIENT_H_ 96 #endif // CHROME_RENDERER_EXTENSIONS_CHROME_EXTENSIONS_RENDERER_CLIENT_H_
OLDNEW
« no previous file with comments | « chrome/renderer/chrome_content_renderer_client.cc ('k') | chrome/renderer/extensions/chrome_extensions_renderer_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698