New issue
Advanced search Search tips

Issue 768086 link

Starred by 2 users
Status: Fixed
Owner: ----
Closed: Oct 2017
Cc:
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac , Fuchsia
Pri: 3
Type: Bug



Sign in to add a comment

Add a test that youtube.com is preloaded for HSTS

Project Member Reported by lgar...@chromium.org, Sep 22 2017 
Requries updating unittest0, too.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 18 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/32d266526d8c245ed486d88ec870bb3964877616

commit 32d266526d8c245ed486d88ec870bb3964877616
Author: Lucas Garron <lgarron@chromium.org>
Date: Wed Oct 18 17:19:18 2017

Preload HSTS for youtube.com, including subdomains.

Bug:  768086 
Change-Id: I89023223ff60736a7d51b7b143dd574b68d64fa5
Reviewed-on: https://chromium-review.googlesource.com/719412
Commit-Queue: Lucas Garron <lgarron@chromium.org>
Reviewed-by: Chris Bentzel <cbentzel@chromium.org>
Reviewed-by: Lucas Garron <lgarron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509799}
[modify] https://crrev.com/32d266526d8c245ed486d88ec870bb3964877616/net/http/transport_security_state_static.json
[modify] https://crrev.com/32d266526d8c245ed486d88ec870bb3964877616/net/http/transport_security_state_unittest.cc
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 20 2017

Labels: merge-merged-3239
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/364f0f139159f1cfe798f44b4f1ff6c4e96901dc

commit 364f0f139159f1cfe798f44b4f1ff6c4e96901dc
Author: Lucas Garron <lgarron@chromium.org>
Date: Fri Oct 20 00:12:14 2017

Preload HSTS for youtube.com, including subdomains.

TBR=lgarron@chromium.org

(cherry picked from commit 32d266526d8c245ed486d88ec870bb3964877616)

Bug:  768086 
Change-Id: I89023223ff60736a7d51b7b143dd574b68d64fa5
Reviewed-on: https://chromium-review.googlesource.com/719412
Commit-Queue: Lucas Garron <lgarron@chromium.org>
Reviewed-by: Chris Bentzel <cbentzel@chromium.org>
Reviewed-by: Lucas Garron <lgarron@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#509799}
Reviewed-on: https://chromium-review.googlesource.com/729214
Cr-Commit-Position: refs/branch-heads/3239@{#96}
Cr-Branched-From: adb61db19020ed8ecee5e91b1a0ea4c924ae2988-refs/heads/master@{#508578}
[modify] https://crrev.com/364f0f139159f1cfe798f44b4f1ff6c4e96901dc/net/http/transport_security_state_static.json
[modify] https://crrev.com/364f0f139159f1cfe798f44b4f1ff6c4e96901dc/net/http/transport_security_state_unittest.cc

Comment 3 by lgar...@chromium.org, Oct 23 2017

Status: Fixed (was: Available)
►

Sign in to add a comment