Cleanup in NTP single-iframe

Cleanup in NTP single-iframe

- Add a few comments
- Rename some functions for (IMO) better clarity
- Remove "file://" as an allowed scheme: support for it has been removed
  a long time ago; see https://codereview.chromium.org/1924773002

BUG=687972
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2693143002
Cr-Commit-Position: refs/heads/master@{#450346}
Committed: https://chromium.googlesource.com/chromium/src/+/cfa2dfc45824fd0b7c912ee050e441807b1d3d09

[Marc Treib, 2017-02-14 12:26:42]

Description was changed from

==========
Cleanup in NTP single-iframe

- Add a few comments
- Rename some functions for (IMO) better clarity
- Remove "file://" as an allowed scheme: support for it has been removed
  a long time ago; see https://codereview.chromium.org/1924773002

BUG=687972
==========

to

==========
Cleanup in NTP single-iframe

- Add a few comments
- Rename some functions for (IMO) better clarity
- Remove "file://" as an allowed scheme: support for it has been removed
  a long time ago; see https://codereview.chromium.org/1924773002

BUG=687972
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Marc Treib, 2017-02-14 12:27:50]

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

[commit-bot: I haz the power, 2017-02-14 12:28:09]

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-02-14 13:28:58]

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

[commit-bot: I haz the power, 2017-02-14 13:28:59]

Dry run: This issue passed the CQ dry run.

[Marc Treib, 2017-02-14 13:35:58]

treib@chromium.org changed reviewers:
+ sfiera@chromium.org

[Marc Treib, 2017-02-14 13:35:59]

PTAL!

[sfiera, 2017-02-14 13:58:43]

LGTM

[Marc Treib, 2017-02-14 14:05:22]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2017-02-14 14:05:47]

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-02-14 14:10:42]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487081122670440, "parent_rev":
"ea69b8739b1bf1a220ea541cca7886597c04bf0b", "commit_rev":
"cfa2dfc45824fd0b7c912ee050e441807b1d3d09"}

[commit-bot: I haz the power, 2017-02-14 14:11:16]

Description was changed from

==========
Cleanup in NTP single-iframe

- Add a few comments
- Rename some functions for (IMO) better clarity
- Remove "file://" as an allowed scheme: support for it has been removed
  a long time ago; see https://codereview.chromium.org/1924773002

BUG=687972
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Cleanup in NTP single-iframe

- Add a few comments
- Rename some functions for (IMO) better clarity
- Remove "file://" as an allowed scheme: support for it has been removed
  a long time ago; see https://codereview.chromium.org/1924773002

BUG=687972
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2693143002
Cr-Commit-Position: refs/heads/master@{#450346}
Committed:
https://chromium.googlesource.com/chromium/src/+/cfa2dfc45824fd0b7c912ee050e4...
==========

[commit-bot: I haz the power, 2017-02-14 14:11:17]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/cfa2dfc45824fd0b7c912ee050e4...