Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(143)

Issue 110813003: Revert of [Telemetry] Fix page_cycler for pages with client side redirects. (Closed)

Created:
7 years ago by pdr.
Modified:
7 years ago
CC:
chromium-reviews, chrome-speed-team+watch_google.com, telemetry+watch_chromium.org
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -40 lines) Patch
M tools/telemetry/telemetry/core/backends/chrome/inspector_page.py View 4 chunks +27 lines, -40 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
pdr.
Created Revert of [Telemetry] Fix page_cycler for pages with client side redirects.
7 years ago (2013-12-09 23:08:53 UTC) #1
achuithb
lgtm.
7 years ago (2013-12-09 23:11:03 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pdr@chromium.org/110813003/1
7 years ago (2013-12-09 23:42:26 UTC) #3
commit-bot: I haz the power
List of reviewers changed. rmistry@google.com did a drive-by without LGTM'ing!
7 years ago (2013-12-10 01:34:13 UTC) #4
rmistry
Sorry about that, I was trying to figure out why the CQ did not do ...
7 years ago (2013-12-10 01:38:36 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pdr@chromium.org/110813003/1
7 years ago (2013-12-10 01:42:23 UTC) #6
commit-bot: I haz the power
Change committed as 239644
7 years ago (2013-12-10 02:01:02 UTC) #7
Tom Hudson
7 years ago (2013-12-13 14:17:20 UTC) #8
Message was sent while issue was closed.
This revert seems to have fixed some, but not all, of Chrome for Android's
internal test bots?

Powered by Google App Engine
This is Rietveld 408576698