Stop sending serialized SSLStatus to the renderer.

Stop sending serialized SSLStatus to the renderer.

It's not used anymore after previous changes referenced in these bugs. This is working towards removing CertStore.

BUG=598073, 504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/c94a753f93a6b1e7e950d6eaa8814cba7d3aeb54
Cr-Commit-Position: refs/heads/master@{#417032}

[jam, 2016-09-06 04:03:33]

Description was changed from

==========
Stop sending serialized SSLStatus to the renderer

BUG=598073, 504347
==========

to

==========
Stop sending serialized SSLStatus to the renderer

BUG=598073, 504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam, 2016-09-06 04:04:57]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 04:05:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-06 04:24:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-06 04:24:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jam, 2016-09-06 05:55:54]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 05:56:06]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-06 07:23:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-06 07:23:12]

Dry run: This issue passed the CQ dry run.

[jam, 2016-09-06 20:57:23]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 20:58:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jam, 2016-09-06 21:04:48]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[jam, 2016-09-06 21:05:13]

Patchset #3 (id:40001) has been deleted

[commit-bot: I haz the power, 2016-09-06 21:05:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jam, 2016-09-06 21:08:23]

jam@chromium.org changed reviewers:
+ pfeldman@chromium.org, tsepez@chromium.org

[jam, 2016-09-06 21:08:24]

Tom: content/common/resource_messages.h
Pavel: the rest (pretty mechanical after the last cl)

https://codereview.chromium.org/2315443003/diff/60001/content/browser/loader/...
File content/browser/loader/resource_dispatcher_host_unittest.cc (left):

https://codereview.chromium.org/2315443003/diff/60001/content/browser/loader/...
content/browser/loader/resource_dispatcher_host_unittest.cc:2789: // Tests that
during a navigation transferred from one process to
this is redundant with a browser test
(SitePerProcessIgnoreCertErrorsBrowserTest.CrossSiteRedirectCertificateStore)
which does check what we care about (SSLStatus). This unittest looks at
security_info which is now gone.

https://codereview.chromium.org/2315443003/diff/60001/content/browser/loader/...
File content/browser/loader/resource_loader_unittest.cc (left):

https://codereview.chromium.org/2315443003/diff/60001/content/browser/loader/...
content/browser/loader/resource_loader_unittest.cc:1114: // Test that an HTTPS
resource has the expected security info attached
not needed since security info is gone

we do of course have other (browser) tests to ensure that we do have the right
SSLStatus for https loads

[Tom Sepez, 2016-09-06 21:13:56]

messages LGTM.

[jam, 2016-09-06 21:21:55]

Patchset #4 (id:80001) has been deleted

[jam, 2016-09-06 21:21:57]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 21:22:30]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jam, 2016-09-06 21:50:36]

Description was changed from

==========
Stop sending serialized SSLStatus to the renderer

BUG=598073, 504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Stop sending serialized SSLStatus to the renderer.

It's not used anymore after previous changes referenced in these bugs. This is
working towards removing CertStore.

BUG=598073, 504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam, 2016-09-06 23:10:34]

The CQ bit was unchecked by jam@chromium.org

[pfeldman, 2016-09-07 18:21:04]

lgtm

[jam, 2016-09-07 19:33:35]

The CQ bit was checked by jam@chromium.org

[jam, 2016-09-07 19:33:36]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/2315443003/#ps100001
(title: "self review fix and merge fix")

[commit-bot: I haz the power, 2016-09-07 19:34:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 19:40:47]

Description was changed from

==========
Stop sending serialized SSLStatus to the renderer.

It's not used anymore after previous changes referenced in these bugs. This is
working towards removing CertStore.

BUG=598073, 504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Stop sending serialized SSLStatus to the renderer.

It's not used anymore after previous changes referenced in these bugs. This is
working towards removing CertStore.

BUG=598073, 504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2016-09-07 19:40:52]

Committed patchset #4 (id:100001)

[commit-bot: I haz the power, 2016-09-07 19:43:02]

Description was changed from

==========
Stop sending serialized SSLStatus to the renderer.

It's not used anymore after previous changes referenced in these bugs. This is
working towards removing CertStore.

BUG=598073, 504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Stop sending serialized SSLStatus to the renderer.

It's not used anymore after previous changes referenced in these bugs. This is
working towards removing CertStore.

BUG=598073, 504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/c94a753f93a6b1e7e950d6eaa8814cba7d3aeb54
Cr-Commit-Position: refs/heads/master@{#417032}
==========

[commit-bot: I haz the power, 2016-09-07 19:43:04]

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