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

Side by Side Diff: net/base/network_delegate_impl.h

Issue 2633663003: Implements strict secure cookies as the default behavior in //net (Closed)
Patch Set: Rebase on ToT 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 | « net/base/network_delegate.cc ('k') | net/base/network_delegate_impl.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 NET_BASE_NETWORK_DELEGATE_IMPL_H_ 5 #ifndef NET_BASE_NETWORK_DELEGATE_IMPL_H_
6 #define NET_BASE_NETWORK_DELEGATE_IMPL_H_ 6 #define NET_BASE_NETWORK_DELEGATE_IMPL_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include "base/strings/string16.h" 10 #include "base/strings/string16.h"
(...skipping 179 matching lines...) Expand 10 before | Expand all | Expand 10 after
190 // Returns true if the given |url| has to be requested over connection that 190 // Returns true if the given |url| has to be requested over connection that
191 // is not tracked by the server. Usually is false, unless user privacy 191 // is not tracked by the server. Usually is false, unless user privacy
192 // settings block cookies from being get or set. 192 // settings block cookies from being get or set.
193 bool OnCanEnablePrivacyMode( 193 bool OnCanEnablePrivacyMode(
194 const GURL& url, 194 const GURL& url,
195 const GURL& first_party_for_cookies) const override; 195 const GURL& first_party_for_cookies) const override;
196 196
197 // Returns true if the embedder has enabled experimental cookie features. 197 // Returns true if the embedder has enabled experimental cookie features.
198 bool OnAreExperimentalCookieFeaturesEnabled() const override; 198 bool OnAreExperimentalCookieFeaturesEnabled() const override;
199 199
200 // TODO(jww): Remove this once we ship strict secure cookies:
201 // https://crbug.com/546820
202 bool OnAreStrictSecureCookiesEnabled() const override;
203
204 // Called when the |referrer_url| for requesting |target_url| during handling 200 // Called when the |referrer_url| for requesting |target_url| during handling
205 // of the |request| is does not comply with the referrer policy (e.g. a 201 // of the |request| is does not comply with the referrer policy (e.g. a
206 // secure referrer for an insecure initial target). 202 // secure referrer for an insecure initial target).
207 // Returns true if the request should be cancelled. Otherwise, the referrer 203 // Returns true if the request should be cancelled. Otherwise, the referrer
208 // header is stripped from the request. 204 // header is stripped from the request.
209 bool OnCancelURLRequestWithPolicyViolatingReferrerHeader( 205 bool OnCancelURLRequestWithPolicyViolatingReferrerHeader(
210 const URLRequest& request, 206 const URLRequest& request,
211 const GURL& target_url, 207 const GURL& target_url,
212 const GURL& referrer_url) const override; 208 const GURL& referrer_url) const override;
213 }; 209 };
214 210
215 } // namespace net 211 } // namespace net
216 212
217 #endif // NET_BASE_NETWORK_DELEGATE_IMPL_H_ 213 #endif // NET_BASE_NETWORK_DELEGATE_IMPL_H_
OLDNEW
« no previous file with comments | « net/base/network_delegate.cc ('k') | net/base/network_delegate_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698