Chromium Code Reviews
DescriptionRevert of [perf] Change google.com to google.ca in top_10_mobile page set. (patchset #2 id:20001 of https://codereview.chromium.org/2092443002/ )
Reason for revert:
Caused artificial memory drop in memory.top_10_mobile benchmark.
Sorry if this breaks or flakes other benchmarks. We can re-land (with re-recorded pages) when the dust settles after health plan for this milestone is completed.
Original issue's description:
> [perf] Change google.com to google.ca in top_10_mobile page set.
>
> In the top10_mobile page set google.com gets redirected to google.ca.
> This sometimes causes problems in telemetry with scrolling. As a short term
> fix, replace google.com with google.ca in the page set.
>
> BUG=chromium:617914
> LOG=N
> CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq
>
> Committed: https://crrev.com/d0c95636111161400a576472427de8eeb2330d43
> Cr-Commit-Position: refs/heads/master@{#401863}
TBR=petrcermak@chromium.org,nednguyen@google.com,mythria@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:617914
Committed: https://crrev.com/8e45aa437f44a72f971913dd1a3ceb20a6f672ad
Cr-Commit-Position: refs/heads/master@{#402865}
Patch Set 1 #
Messages
Total messages: 8 (3 generated)
|
||||||||||||||||||||||||||||||||||||||||||||||