[ServiceWorker] Don't register ServiceWorker with an invalid HTTPS certificate.

[ServiceWorker] Don't register ServiceWorker with an invalid HTTPS certificate.

BUG=425396, 414026
Committed: https://crrev.com/a8ac964e0cf580c4e72028f81f1cda335436659d
Cr-Commit-Position: refs/heads/master@{#300644}

[horo, 2014-10-21 03:27:30]

Patchset #1 (id:1) has been deleted

[horo, 2014-10-21 03:40:46]

horo@chromium.org changed reviewers:
+ michaeln@chromium.org

[horo, 2014-10-21 03:40:47]

michaeln@
Could you please review this?
Thank you!

[Michael van Ouwerkerk, 2014-10-21 09:48:39]

Drive-by comment: do you need a new test to cover this change?

[michaeln, 2014-10-21 19:35:19]

lgtm!

[horo, 2014-10-22 06:22:29]

On 2014/10/21 09:48:39, Michael van Ouwerkerk wrote:
> Drive-by comment: do you need a new test to cover this change?

Yes we should.
I will add tests in another CL.

[horo, 2014-10-22 06:22:32]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-10-22 06:23:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/643773004/20001

[commit-bot: I haz the power, 2014-10-22 07:13:22]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2014-10-22 07:14:14]

Patchset 1 (id:??) landed as
https://crrev.com/a8ac964e0cf580c4e72028f81f1cda335436659d
Cr-Commit-Position: refs/heads/master@{#300644}

[jsbell, 2014-10-22 16:27:24]

On 2014/10/22 07:14:14, I haz the power (commit-bot) wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/a8ac964e0cf580c4e72028f81f1cda335436659d
> Cr-Commit-Position: refs/heads/master@{#300644}

http/tests/serviceworker/fetch-mixed-content.html is failing following this
change:

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=htt...

[michaeln1, 2014-10-22 16:56:07]

On 2014/10/22 16:27:24, jsbell wrote:
> On 2014/10/22 07:14:14, I haz the power (commit-bot) wrote:
> > Patchset 1 (id:??) landed as
> > https://crrev.com/a8ac964e0cf580c4e72028f81f1cda335436659d
> > Cr-Commit-Position: refs/heads/master@{#300644}
> 
> http/tests/serviceworker/fetch-mixed-content.html is failing following this
> change:
> 
>
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=htt...

Does the https server used for layout tests have a valid ssl cert?

[jsbell, 2014-10-22 18:17:34]

On 2014/10/22 16:56:07, michaeln1 wrote:
> On 2014/10/22 16:27:24, jsbell wrote:
> > On 2014/10/22 07:14:14, I haz the power (commit-bot) wrote:
> > > Patchset 1 (id:??) landed as
> > > https://crrev.com/a8ac964e0cf580c4e72028f81f1cda335436659d
> > > Cr-Commit-Position: refs/heads/master@{#300644}
> > 
> > http/tests/serviceworker/fetch-mixed-content.html is failing following this
> > change:
> > 
> >
>
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=htt...
> 
> Does the https server used for layout tests have a valid ssl cert?

Our code doesn't think so. :)

I filed crbug.com/426047 to track the failure.