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

Side by Side Diff: chrome/renderer/chrome_content_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
« no previous file with comments | « no previous file | chrome/renderer/chrome_content_renderer_client.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 (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_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_ 5 #ifndef CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_
6 #define CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_ 6 #define CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include <map> 10 #include <map>
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 bool AllowTimerSuspensionWhenProcessBackgrounded() override; 127 bool AllowTimerSuspensionWhenProcessBackgrounded() override;
128 bool AllowPopup() override; 128 bool AllowPopup() override;
129 bool ShouldFork(blink::WebLocalFrame* frame, 129 bool ShouldFork(blink::WebLocalFrame* frame,
130 const GURL& url, 130 const GURL& url,
131 const std::string& http_method, 131 const std::string& http_method,
132 bool is_initial_navigation, 132 bool is_initial_navigation,
133 bool is_server_redirect, 133 bool is_server_redirect,
134 bool* send_referrer) override; 134 bool* send_referrer) override;
135 bool WillSendRequest(blink::WebFrame* frame, 135 bool WillSendRequest(blink::WebFrame* frame,
136 ui::PageTransition transition_type, 136 ui::PageTransition transition_type,
137 const GURL& url, 137 const blink::WebURL& url,
138 const GURL& first_party_for_cookies,
139 GURL* new_url) override; 138 GURL* new_url) override;
140 bool IsPrefetchOnly(content::RenderFrame* render_frame, 139 bool IsPrefetchOnly(content::RenderFrame* render_frame,
141 const blink::WebURLRequest& request) override; 140 const blink::WebURLRequest& request) override;
142 unsigned long long VisitedLinkHash(const char* canonical_url, 141 unsigned long long VisitedLinkHash(const char* canonical_url,
143 size_t length) override; 142 size_t length) override;
144 bool IsLinkVisited(unsigned long long link_hash) override; 143 bool IsLinkVisited(unsigned long long link_hash) override;
145 blink::WebPrescientNetworking* GetPrescientNetworking() override; 144 blink::WebPrescientNetworking* GetPrescientNetworking() override;
146 bool ShouldOverridePageVisibilityState( 145 bool ShouldOverridePageVisibilityState(
147 const content::RenderFrame* render_frame, 146 const content::RenderFrame* render_frame,
148 blink::WebPageVisibilityState* override_state) override; 147 blink::WebPageVisibilityState* override_state) override;
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
253 #endif 252 #endif
254 253
255 #if defined(OS_CHROMEOS) 254 #if defined(OS_CHROMEOS)
256 std::unique_ptr<LeakDetectorRemoteClient> leak_detector_remote_client_; 255 std::unique_ptr<LeakDetectorRemoteClient> leak_detector_remote_client_;
257 #endif 256 #endif
258 257
259 DISALLOW_COPY_AND_ASSIGN(ChromeContentRendererClient); 258 DISALLOW_COPY_AND_ASSIGN(ChromeContentRendererClient);
260 }; 259 };
261 260
262 #endif // CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_ 261 #endif // CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/renderer/chrome_content_renderer_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698