chore: fix ShouldFork impl, referrer stripping has been removed

refs: 1124329
This commit is contained in:
Samuel Attard 2018-10-25 17:12:12 +11:00 committed by deepak1556
parent c9007557e5
commit 74d1685523
2 changed files with 2 additions and 5 deletions

View file

@ -163,13 +163,11 @@ bool AtomRendererClient::ShouldFork(blink::WebLocalFrame* frame,
const GURL& url, const GURL& url,
const std::string& http_method, const std::string& http_method,
bool is_initial_navigation, bool is_initial_navigation,
bool is_server_redirect, bool is_server_redirect) {
bool* send_referrer) {
// Handle all the navigations and reloads in browser. // Handle all the navigations and reloads in browser.
// FIXME We only support GET here because http method will be ignored when // FIXME We only support GET here because http method will be ignored when
// the OpenURLFromTab is triggered, which means form posting would not work, // the OpenURLFromTab is triggered, which means form posting would not work,
// we should solve this by patching Chromium in future. // we should solve this by patching Chromium in future.
*send_referrer = true;
return http_method == "GET"; return http_method == "GET";
} }

View file

@ -51,8 +51,7 @@ class AtomRendererClient : public RendererClientBase {
const GURL& url, const GURL& url,
const std::string& http_method, const std::string& http_method,
bool is_initial_navigation, bool is_initial_navigation,
bool is_server_redirect, bool is_server_redirect) override;
bool* send_referrer) override;
void DidInitializeWorkerContextOnWorkerThread( void DidInitializeWorkerContextOnWorkerThread(
v8::Local<v8::Context> context) override; v8::Local<v8::Context> context) override;
void WillDestroyWorkerContextOnWorkerThread( void WillDestroyWorkerContextOnWorkerThread(