Adjust ChromeToMobile error logging and mitigation.

Adjust ChromeToMobile error logging and mitigation.

Increase kMaxRetries for URLFetcher to 5 (common elsewhere).
Decrease kDelayHours for URLFetcher to 1hr (w/exp backoff).

Clear |access_token_| with new kGaiaOAuth2LoginAccessToken.
Clear |access_token_| *and devices* on OnGetTokenFailure().
Handle invalid login refresh tokens as OnGetTokenFailure().
Clear token, devices, retry on 403 (auth) search responses.

Add and log some new BAD_* UMA Metric enum values.
(407 related to Issue 146685 and perhaps Issue 137267)
LOG (Chrome, not UMA) send response data on failure.

Corresponding src/tools/histograms/histograms.xml change:
https://chromereviews.googleplex.com/4782033

BUG=102709, 120941, 146685
TEST=Less Chrome To Mobile failures, additional insightful logging.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=155317

[msw, 2012-09-06 02:34:58]

Hey Tommy and Scott, please take a look; thanks!

[sky, 2012-09-06 15:59:32]

LGTM

[nyquist, 2012-09-06 19:09:15]

https://chromiumcodereview.appspot.com/10913089/diff/11001/chrome/browser/chr...
File chrome/browser/chrome_to_mobile_service.cc (right):

https://chromiumcodereview.appspot.com/10913089/diff/11001/chrome/browser/chr...
chrome/browser/chrome_to_mobile_service.cc:571:
auth_retry_timer_.GetCurrentDelay() * 2);
Should this not be for search_retry_timer_?

[nyquist, 2012-09-06 19:21:16]

lgtm after fixing delta delya

[msw, 2012-09-06 20:07:10]

Comment addressed, thanks for the catch! Landing.

https://chromiumcodereview.appspot.com/10913089/diff/11001/chrome/browser/chr...
File chrome/browser/chrome_to_mobile_service.cc (right):

https://chromiumcodereview.appspot.com/10913089/diff/11001/chrome/browser/chr...
chrome/browser/chrome_to_mobile_service.cc:571:
auth_retry_timer_.GetCurrentDelay() * 2);
On 2012/09/06 19:09:15, nyquist wrote:
> Should this not be for search_retry_timer_?

Done.

[commit-bot: I haz the power, 2012-09-06 20:09:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/msw@chromium.org/10913089/1006

[commit-bot: I haz the power, 2012-09-06 21:31:31]

Try job failure for 10913089-1006 (retry) on linux_rel for step
"interactive_ui_tests".
It's a second try, previously, step "interactive_ui_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-09-06 21:43:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/msw@chromium.org/10913089/1006

[commit-bot: I haz the power, 2012-09-06 23:17:31]

Try job failure for 10913089-1006 (retry) on win for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

[commit-bot: I haz the power, 2012-09-06 23:19:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/msw@chromium.org/10913089/1006

[commit-bot: I haz the power, 2012-09-07 05:33:44]

Change committed as 155317