Add repro for crash when opening window with --allow-insecure-localhost.

Add repro for crash when opening window with --allow-insecure-localhost.

BUG=635833
Committed: https://crrev.com/4adf82fd048f68f63308bbb0dd25063cd220237a
Cr-Commit-Position: refs/heads/master@{#420555}

[jam, 2016-09-22 20:12:04]

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

[commit-bot: I haz the power, 2016-09-22 20:12:31]

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-22 20:12:35]

jam@chromium.org changed reviewers:
+ estark@chromium.org

[estark, 2016-09-22 20:34:04]

Thanks! lgtm with a suggestion

https://codereview.chromium.org/2358983004/diff/1/chrome/browser/ssl/ssl_brow...
File chrome/browser/ssl/ssl_browser_tests.cc (right):

https://codereview.chromium.org/2358983004/diff/1/chrome/browser/ssl/ssl_brow...
chrome/browser/ssl/ssl_browser_tests.cc:3248: class SSLUITestAlllowInsecureHost
: public SSLUITest {
There's already SSLUITestIgnoreLocalhostCertErrors if you want to use that
instead.

[commit-bot: I haz the power, 2016-09-22 20:57:01]

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

[commit-bot: I haz the power, 2016-09-22 20:57:02]

Dry run: This issue passed the CQ dry run.

[jam, 2016-09-22 22:25:16]

https://codereview.chromium.org/2358983004/diff/1/chrome/browser/ssl/ssl_brow...
File chrome/browser/ssl/ssl_browser_tests.cc (right):

https://codereview.chromium.org/2358983004/diff/1/chrome/browser/ssl/ssl_brow...
chrome/browser/ssl/ssl_browser_tests.cc:3248: class SSLUITestAlllowInsecureHost
: public SSLUITest {
On 2016/09/22 20:34:03, estark wrote:
> There's already SSLUITestIgnoreLocalhostCertErrors if you want to use that
> instead.

Done

[jam, 2016-09-22 22:25:36]

The CQ bit was checked by jam@chromium.org

[jam, 2016-09-22 22:25:36]

The patchset sent to the CQ was uploaded after l-g-t-m from estark@chromium.org
Link to the patchset: https://codereview.chromium.org/2358983004/#ps20001
(title: "review comment")

[commit-bot: I haz the power, 2016-09-22 22:25:59]

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-22 22:48:26]

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

[commit-bot: I haz the power, 2016-09-22 22:48:28]

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

[jam, 2016-09-23 02:09:19]

The CQ bit was checked by jam@chromium.org

[jam, 2016-09-23 02:09:19]

The patchset sent to the CQ was uploaded after l-g-t-m from estark@chromium.org
Link to the patchset: https://codereview.chromium.org/2358983004/#ps40001
(title: "review comment")

[jam, 2016-09-23 02:09:23]

Patchset #2 (id:20001) has been deleted

[commit-bot: I haz the power, 2016-09-23 02:09:55]

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-23 02:48:12]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-09-23 02:50:05]

Description was changed from

==========
Add repro for crash when opening window with --allow-insecure-localhost.

BUG=635833
==========

to

==========
Add repro for crash when opening window with --allow-insecure-localhost.

BUG=635833

Committed: https://crrev.com/4adf82fd048f68f63308bbb0dd25063cd220237a
Cr-Commit-Position: refs/heads/master@{#420555}
==========

[commit-bot: I haz the power, 2016-09-23 02:50:06]

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