telemetry: Update tough_path_rendering_cases Chalkboard test URL

telemetry: Update tough_path_rendering_cases Chalkboard test URL

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

[Kimmo Kinnunen, 2016-05-19 07:59:45]

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

[Kimmo Kinnunen, 2016-05-19 07:59:46]

PTAL, note that I can not upload to Cloud Storage. Used --bypass-hooks to upload
this.

[nednguyen, 2016-05-19 15:29:47]

On 2016/05/19 07:59:46, Kimmo Kinnunen wrote:
> PTAL, note that I can not upload to Cloud Storage. Used --bypass-hooks to
upload
> this.

lgtm overall

Though what error did you get when you try to upload this to cloud storage? Note
that you need to use story.PARTNER_BUCKET which is 'chrome-partner-telemetry'

[Stephen White, 2016-05-19 15:33:30]

On 2016/05/19 07:59:46, Kimmo Kinnunen wrote:
> PTAL, note that I can not upload to Cloud Storage. Used --bypass-hooks to
upload
> this.

If the upload failed, then I don't think you can land this successfully.
Referring to a hash that doesn't exist in Cloud Storage won't work, AFAIK.

not LGTM

[nednguyen, 2016-05-19 15:37:17]

On 2016/05/19 15:33:30, Stephen White wrote:
> On 2016/05/19 07:59:46, Kimmo Kinnunen wrote:
> > PTAL, note that I can not upload to Cloud Storage. Used --bypass-hooks to
> upload
> > this.
> 
> If the upload failed, then I don't think you can land this successfully.
> Referring to a hash that doesn't exist in Cloud Storage won't work, AFAIK.
> 
> not LGTM

I send out an email to rschoen@ & cc'ed Kimmo in. Ryan will grant Kimmo
permission to upload to cloud storage soon.

[nednguyen, 2016-05-19 22:24:27]

On 2016/05/19 15:37:17, nednguyen wrote:
> On 2016/05/19 15:33:30, Stephen White wrote:
> > On 2016/05/19 07:59:46, Kimmo Kinnunen wrote:
> > > PTAL, note that I can not upload to Cloud Storage. Used --bypass-hooks to
> > upload
> > > this.
> > 
> > If the upload failed, then I don't think you can land this successfully.
> > Referring to a hash that doesn't exist in Cloud Storage won't work, AFAIK.
> > 
> > not LGTM
> 
> I send out an email to rschoen@ & cc'ed Kimmo in. Ryan will grant Kimmo
> permission to upload to cloud storage soon.

According to Ryan, we don't usually give write permission to partners. Stephen,
can you help Kimmo with uploading the wpr file to cloud storage?

[nednguyen, 2016-05-19 22:24:45]

Description was changed from

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

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

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
==========

[nednguyen, 2016-05-19 22:24:45]

nednguyen@google.com changed reviewers:
+ rschoen@chromium.org

[Stephen White, 2016-05-19 22:26:01]

On 2016/05/19 22:24:27, nednguyen wrote:
> On 2016/05/19 15:37:17, nednguyen wrote:
> > On 2016/05/19 15:33:30, Stephen White wrote:
> > > On 2016/05/19 07:59:46, Kimmo Kinnunen wrote:
> > > > PTAL, note that I can not upload to Cloud Storage. Used --bypass-hooks
to
> > > upload
> > > > this.
> > > 
> > > If the upload failed, then I don't think you can land this successfully.
> > > Referring to a hash that doesn't exist in Cloud Storage won't work, AFAIK.
> > > 
> > > not LGTM
> > 
> > I send out an email to rschoen@ & cc'ed Kimmo in. Ryan will grant Kimmo
> > permission to upload to cloud storage soon.
> 
> According to Ryan, we don't usually give write permission to partners.
Stephen,
> can you help Kimmo with uploading the wpr file to cloud storage?

Sure, will do when I'm back in the office (probably Monday).

[Stephen White, 2016-05-24 15:24:09]

On 2016/05/19 22:26:01, Stephen White wrote:
> On 2016/05/19 22:24:27, nednguyen wrote:
> > On 2016/05/19 15:37:17, nednguyen wrote:
> > > On 2016/05/19 15:33:30, Stephen White wrote:
> > > > On 2016/05/19 07:59:46, Kimmo Kinnunen wrote:
> > > > > PTAL, note that I can not upload to Cloud Storage. Used --bypass-hooks
> to
> > > > upload
> > > > > this.
> > > > 
> > > > If the upload failed, then I don't think you can land this successfully.
> > > > Referring to a hash that doesn't exist in Cloud Storage won't work,
AFAIK.
> > > > 
> > > > not LGTM
> > > 
> > > I send out an email to rschoen@ & cc'ed Kimmo in. Ryan will grant Kimmo
> > > permission to upload to cloud storage soon.
> > 
> > According to Ryan, we don't usually give write permission to partners.
> Stephen,
> > can you help Kimmo with uploading the wpr file to cloud storage?
> 
> Sure, will do when I'm back in the office (probably Monday).

Just to let you know I haven't forgotten about this, but I won't be able to do
it until later in the week.

[Kimmo Kinnunen, 2016-06-08 11:30:34]

On 2016/05/24 15:24:09, Stephen White wrote:
> Just to let you know I haven't forgotten about this, but I won't be able to do
> it until later in the week.

friendly ping

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

On 2016/06/08 11:30:34, Kimmo Kinnunen wrote:
> friendly ping

Sorry for the delay. I've uploaded the WPR, and upload the resulting CL to 
https://codereview.chromium.org/2055823004/ since the hash was different.