telemetry: Update tough_path_rendering_cases Chalkboard test URL

telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928
Committed: https://crrev.com/d0d9b578d0fce1a364f157c6cd669fd4439a916a
Cr-Commit-Position: refs/heads/master@{#398985}

[Stephen White, 2016-06-09 18:25:30]

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

[commit-bot: I haz the power, 2016-06-09 18:26:24]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2055823004/1

[Stephen White, 2016-06-09 18:27:01]

Description was changed from

==========
telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928

patch from issue 1998543002 at patchset 1 (http://crrev.com/1998543002#ps1)
==========

to

==========
telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928

patch from issue 1998543002 at patchset 1 (http://crrev.com/1998543002#ps1)
==========

[Stephen White, 2016-06-09 18:27:01]

senorblanco@chromium.org changed reviewers:
+ kkinnunen@nvidia.com, nednguyen@google.com

[Stephen White, 2016-06-09 18:27:10]

Description was changed from

==========
telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928

patch from issue 1998543002 at patchset 1 (http://crrev.com/1998543002#ps1)
==========

to

==========
telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928
==========

[Stephen White, 2016-06-09 18:27:28]

nednguyen@: PTAL. Thanks!

[nednguyen, 2016-06-09 18:33:35]

lgtm

https://codereview.chromium.org/2055823004/diff/1/tools/perf/page_sets/tough_...
File tools/perf/page_sets/tough_path_rendering_cases.py (right):

https://codereview.chromium.org/2055823004/diff/1/tools/perf/page_sets/tough_...
tools/perf/page_sets/tough_path_rendering_cases.py:41: # Chalkboard content
linked from http://ie.microsoft.com/testdrive/Performance/Chalkboard/.
nits: split the url to the newline

[Stephen White, 2016-06-09 18:36:16]

Thanks for your review.

https://codereview.chromium.org/2055823004/diff/1/tools/perf/page_sets/tough_...
File tools/perf/page_sets/tough_path_rendering_cases.py (right):

https://codereview.chromium.org/2055823004/diff/1/tools/perf/page_sets/tough_...
tools/perf/page_sets/tough_path_rendering_cases.py:41: # Chalkboard content
linked from http://ie.microsoft.com/testdrive/Performance/Chalkboard/.
On 2016/06/09 18:33:35, nednguyen wrote:
> nits: split the url to the newline

Done.

[Stephen White, 2016-06-09 18:36:35]

The CQ bit was checked by senorblanco@chromium.org

[Stephen White, 2016-06-09 18:36:35]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2055823004/#ps20001
(title: "Fix formatting")

[commit-bot: I haz the power, 2016-06-09 18:37:54]

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

[commit-bot: I haz the power, 2016-06-09 19:53:37]

Description was changed from

==========
telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928
==========

to

==========
telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928
==========

[commit-bot: I haz the power, 2016-06-09 19:53:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-09 19:55:12]

Description was changed from

==========
telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928
==========

to

==========
telemetry: Update tough_path_rendering_cases Chalkboard test URL

[Copy of https://codereview.chromium.org/1998543002/ by
kkinnunen@nvidia.org with the WPR uploaded.]

Update tough_path_rendering_cases Chalkboard test URL. The old
url would go to a page that would load the content in an iframe.
The injected script would fail.

Load the iframe URL instead. Loading the main document would inject
the script to the parent main document. The injected code would
start the test by click()ing an element cross domains, which
would be rejected.

Fixing is needed in order to record new content to
tough_path_rendering_cases.

BUG=594928

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

[commit-bot: I haz the power, 2016-06-09 19:55:13]

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

[Stephen White, 2016-06-09 19:59:02]

NOTE TO PERF SHERIFFS: the change in URL may impact the results of the
Chalkboard test (either negatively or positively). However, it should affect the
ref build as well, so the change should be ignored.