commit | 9c9e124bfa74d4b6ddb7f429b916285721c4f609 | [log] [tgz] |
---|---|---|
author | msw@chromium.org <msw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | Mon Sep 24 19:34:54 2012 |
committer | msw@chromium.org <msw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | Mon Sep 24 19:34:54 2012 |
tree | ba5ea4947184055f38a78c9da10b629edd2fe7da | |
parent | e1b37ddd46483117541127c4b8cc9931e149659b [diff] |
Check ChromeToMobileService OriginalURLs before handling. Example one-day UMA shows "Send Success" > "Sending URL" + "Sending Snapshot": https://gfsviewer.corp.google.com/cns/pa-d/home/uma-data/public/dashboards/histograms.html?dayCount=1&endDate=09-21-2012&version=24.0.1273.0-W&group=ChromeToMobile&histogram=Service&otherVersions=10 Perhaps these URLs may be redirected? This CL checks the *original* URLs before handling. This speculative fix should be safe even if it doesn't address the root issue. BUG=102709,120941 TEST=UMA data doesn't report (send success+failure) > (send URL+snapshot). R=nyquist@chromium.org,sky@chromium.org Review URL: https://chromiumcodereview.appspot.com/10970054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158352 0039d316-1c4b-4281-b951-d872f2087c98