Allows MHTM page to open a popup

Allows MHTM page to open a popup

BUG=704722
TEST=test updated

Review-Url: https://codereview.chromium.org/2837053004
Cr-Commit-Position: refs/heads/master@{#467446}
Committed: https://chromium.googlesource.com/chromium/src/+/3024459c9f7cd7783b63535623244a21e9d4d4e6

[jianli, 2017-04-24 21:12:30]

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

[jianli, 2017-04-24 21:12:53]

jianli@chromium.org changed reviewers:
+ mkwst@chromium.org

[commit-bot: I haz the power, 2017-04-24 21:13:20]

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, 2017-04-24 22:49:50]

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

[commit-bot: I haz the power, 2017-04-24 22:49:51]

Dry run: This issue passed the CQ dry run.

[Mike West, 2017-04-25 07:38:01]

Code change looks correct, but please add tests for this behavior.

[jianli, 2017-04-25 23:30:02]

Description was changed from

==========
Allows MHTM page to open a popup

BUG=704722
==========

to

==========
Allows MHTM page to open a popup

BUG=704722
TEST=test updated
==========

[jianli, 2017-04-25 23:30:19]

Updated test.

[jianli, 2017-04-25 23:30:28]

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

[commit-bot: I haz the power, 2017-04-25 23:31:21]

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, 2017-04-26 01:41:22]

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

[commit-bot: I haz the power, 2017-04-26 01:41:23]

Dry run: This issue passed the CQ dry run.

[Mike West, 2017-04-26 09:30:08]

lgtm

[jianli, 2017-04-26 20:41:34]

The CQ bit was checked by jianli@chromium.org

[commit-bot: I haz the power, 2017-04-26 20:42:19]

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, 2017-04-26 20:47:58]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493239294425540,
"parent_rev": "0d2d43615b4553f00ac711f54dbcc0835f04366b", "commit_rev":
"3024459c9f7cd7783b63535623244a21e9d4d4e6"}

[commit-bot: I haz the power, 2017-04-26 20:48:12]

Description was changed from

==========
Allows MHTM page to open a popup

BUG=704722
TEST=test updated
==========

to

==========
Allows MHTM page to open a popup

BUG=704722
TEST=test updated

Review-Url: https://codereview.chromium.org/2837053004
Cr-Commit-Position: refs/heads/master@{#467446}
Committed:
https://chromium.googlesource.com/chromium/src/+/3024459c9f7cd7783b6353562324...
==========

[commit-bot: I haz the power, 2017-04-26 20:48:13]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/3024459c9f7cd7783b6353562324...