Telemetry: Replace actual groupcloned page with test page.

Telemetry: Replace actual groupcloned page with test page.

Disable actual http://groupcloned.com and add test page
http://groupcloned.com/test/list-images-variable/index.html instead.

The actual groupcloned page keeps failing in the key_mobile_sites page set.
It seems to behave non-deterministically, i.e. it's trying to load different
items on different invocations. This is despite Telemetry trying to get rid
of all randomness. The problem with the non-determinism is that the page often
tries to load resources that haven't been loaded during recording.

BUG=238864
R=tonyg@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221468

[Dominik Grewe, 2013-09-05 11:35:03]

There's a corresponding CL which adds the recording of the new page:
https://chromereviews.googleplex.com/9917013

PTAL, thanks!

[tonyg, 2013-09-05 15:32:13]

lgtm

https://codereview.chromium.org/23500008/diff/1/tools/perf/page_sets/key_mobi...
File tools/perf/page_sets/key_mobile_sites.json (right):

https://codereview.chromium.org/23500008/diff/1/tools/perf/page_sets/key_mobi...
tools/perf/page_sets/key_mobile_sites.json:215: "disabled": "Page behaves
non-deterministically, replaced with test version for now",
Should we just remove this instead of disabling it?

Also, perhaps if you could pinpoint the source of the non-determinism, we could
improve web page replay to be able to make that deterministic.

[Dominik Grewe, 2013-09-05 15:38:11]

https://codereview.chromium.org/23500008/diff/1/tools/perf/page_sets/key_mobi...
File tools/perf/page_sets/key_mobile_sites.json (right):

https://codereview.chromium.org/23500008/diff/1/tools/perf/page_sets/key_mobi...
tools/perf/page_sets/key_mobile_sites.json:215: "disabled": "Page behaves
non-deterministically, replaced with test version for now",
On 2013/09/05 15:32:13, tonyg wrote:
> Should we just remove this instead of disabling it?
> 
> Also, perhaps if you could pinpoint the source of the non-determinism, we
could
> improve web page replay to be able to make that deterministic.

I tried to figure out what exactly is going on but the page has quite a lot of
obscure JS which doesn't make it easy to understand what's going on. We're
trying to get a deterministic version of the page from the developers, just like
the newly added test page but more similar to the actual page.

I disabled rather than removed it for now because we still don't know what we're
going to do with this page. Should I leave it like that or should I remove it?

[tonyg, 2013-09-05 15:43:13]

On 2013/09/05 15:38:11, Dominik Grewe wrote:
>
https://codereview.chromium.org/23500008/diff/1/tools/perf/page_sets/key_mobi...
> File tools/perf/page_sets/key_mobile_sites.json (right):
> 
>
https://codereview.chromium.org/23500008/diff/1/tools/perf/page_sets/key_mobi...
> tools/perf/page_sets/key_mobile_sites.json:215: "disabled": "Page behaves
> non-deterministically, replaced with test version for now",
> On 2013/09/05 15:32:13, tonyg wrote:
> > Should we just remove this instead of disabling it?
> > 
> > Also, perhaps if you could pinpoint the source of the non-determinism, we
> could
> > improve web page replay to be able to make that deterministic.
> 
> I tried to figure out what exactly is going on but the page has quite a lot of
> obscure JS which doesn't make it easy to understand what's going on. We're
> trying to get a deterministic version of the page from the developers, just
like
> the newly added test page but more similar to the actual page.

Thanks for trying.
> 
> I disabled rather than removed it for now because we still don't know what
we're
> going to do with this page. Should I leave it like that or should I remove it?

I guess it doesn't hurt anything to leave it.

[Dominik Grewe, 2013-09-05 15:46:12]

On 2013/09/05 15:43:13, tonyg wrote:
> On 2013/09/05 15:38:11, Dominik Grewe wrote:
> >
>
https://codereview.chromium.org/23500008/diff/1/tools/perf/page_sets/key_mobi...
> > File tools/perf/page_sets/key_mobile_sites.json (right):
> > 
> >
>
https://codereview.chromium.org/23500008/diff/1/tools/perf/page_sets/key_mobi...
> > tools/perf/page_sets/key_mobile_sites.json:215: "disabled": "Page behaves
> > non-deterministically, replaced with test version for now",
> > On 2013/09/05 15:32:13, tonyg wrote:
> > > Should we just remove this instead of disabling it?
> > > 
> > > Also, perhaps if you could pinpoint the source of the non-determinism, we
> > could
> > > improve web page replay to be able to make that deterministic.
> > 
> > I tried to figure out what exactly is going on but the page has quite a lot
of
> > obscure JS which doesn't make it easy to understand what's going on. We're
> > trying to get a deterministic version of the page from the developers, just
> like
> > the newly added test page but more similar to the actual page.
> 
> Thanks for trying.
> > 
> > I disabled rather than removed it for now because we still don't know what
> we're
> > going to do with this page. Should I leave it like that or should I remove
it?
> 
> I guess it doesn't hurt anything to leave it.

Thanks. I'll ask Sami to submit this and the other CL again so we can land them
more or less at the same time.

[Sami, 2013-09-05 18:26:25]

Committed patchset #1 manually as r221468 (presubmit successful).