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

Issue 9347031: Merge 120807 - Revert 116685 - Make incognito windows not inherit HSTS state from the main profile. (Closed)

Created:
8 years, 10 months ago by palmer
Modified:
8 years, 10 months ago
Reviewers:
palmer
CC:
chromium-reviews
Visibility:
Public.

Description

Merge 120807 - Revert 116685 - Make incognito windows not inherit HSTS state from the main profile. The dynamic HSTS state is loaded by the TransportSecurityPersister. By moving it up into profile_impl (which isn't shared by Incognito profiles), the Incognito profiles don't ever load dynamic state. BUG=104935 TEST=Inject an HSTS entry for a non-HTTPS site in the main profile using about:net-internals. Open an incognito window and verify that you can still access the HTTP site. Review URL: http://codereview.chromium.org/8997012 TBR=agl@chromium.org,cevans@google.com Review URL: https://chromiumcodereview.appspot.com/9349029 TBR=palmer@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=120816

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -13 lines) Patch
M chrome/browser/profiles/profile_impl_io_data.h View 3 chunks +1 line, -4 lines 0 comments Download
M chrome/browser/profiles/profile_impl_io_data.cc View 3 chunks +1 line, -7 lines 0 comments Download
M chrome/browser/profiles/profile_io_data.h View 3 chunks +5 lines, -1 line 0 comments Download
M chrome/browser/profiles/profile_io_data.cc View 3 chunks +6 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
palmer
8 years, 10 months ago (2012-02-07 20:23:28 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698