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

Issue 220593005: Include subdomains for AppEngine preloaded HSTS/pinning. (Closed)

Created:
6 years, 8 months ago by palmer
Modified:
6 years, 8 months ago
Reviewers:
agl, Ryan Sleevi
CC:
chromium-reviews, cbentzel+watch_chromium.org
Visibility:
Public.

Description

Include subdomains for AppEngine preloaded HSTS/pinning. BUG=358571 R=agl@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260919

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M net/http/transport_security_state_static.h View 1 chunk +1 line, -1 line 0 comments Download
M net/http/transport_security_state_static.json View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
palmer
PTAL.
6 years, 8 months ago (2014-04-01 18:57:07 UTC) #1
agl
lgtm
6 years, 8 months ago (2014-04-01 19:05:15 UTC) #2
palmer
6 years, 8 months ago (2014-04-01 19:15:45 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r260919 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698