Collect data reduction proxy UMA on all platforms

Collect data reduction proxy UMA on all platforms

UMA for the data reduction proxy was gated behind preprocessor
directives that resticted the logic to Android and iOS. This
change opens UMA calculations and collection up to all platforms.

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

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

[bengr, 2014-05-06 20:27:41]

mef: chrome/browser/net/*
marq: components/data_reduction_proxy/*

[marq (ping after 24h), 2014-05-06 20:45:56]

LGTM.

[mef, 2014-05-06 21:17:31]

lgtm

[bengr, 2014-05-06 21:19:55]

The CQ bit was checked by bengr@chromium.org

[commit-bot: I haz the power, 2014-05-06 21:20:17]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bengr@chromium.org/266033002/40001

[commit-bot: I haz the power, 2014-05-07 03:25:15]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  mac_chromium_rel on tryserver.chromium

[commit-bot: I haz the power, 2014-05-07 10:00:12]

FYI, CQ is re-trying this CL (attempt #2).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  win_chromium_rel on tryserver.chromium

[commit-bot: I haz the power, 2014-05-07 12:03:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-07 12:03:46]

Try jobs failed on following builders:
  win_chromium_rel on tryserver.chromium

[bengr, 2014-05-07 18:08:58]

The CQ bit was checked by bengr@chromium.org

[commit-bot: I haz the power, 2014-05-07 18:11:57]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bengr@chromium.org/266033002/40001

[bengr, 2014-05-08 00:00:06]

The CQ bit was checked by bengr@chromium.org

[commit-bot: I haz the power, 2014-05-08 00:03:53]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bengr@chromium.org/266033002/70001

[commit-bot: I haz the power, 2014-05-08 04:12:09]

Change committed as 269026

[tapted, 2014-05-08 05:57:19]

A revert of this CL has been created in
https://codereview.chromium.org/274453011/ by tapted@chromium.org.

The reason for reverting is: Causes a sizes regression, starting in
http://build.chromium.org/p/chromium/builders/Linux%20x64/builds/64712

+# data_reduction_proxy_settings.cc
data_reduction_proxy::DataReductionProxySettings::key_
+# data_reduction_proxy_settings.cc __cxa_atexit@plt [registers a dtor to run at
exit]
+# data_reduction_proxy_settings.cc __init_array_end+0x3868
+# data_reduction_proxy_settings.cc __init_array_end+0x3b88
+# data_reduction_proxy_settings.cc __init_array_end+0x530.

[tapted, 2014-05-08 06:01:25]

https://codereview.chromium.org/266033002/diff/70001/components/data_reductio...
File components/data_reduction_proxy/browser/data_reduction_proxy_metrics.cc
(right):

https://codereview.chromium.org/266033002/diff/70001/components/data_reductio...
components/data_reduction_proxy/browser/data_reduction_proxy_metrics.cc:305:
DataReductionProxySettings::GetDataReductionProxies();
did a double-take due since data_reduction_proxy_settings.cc didn't actually
change in this CL.. but I think it must be this line.

[bengr, 2014-05-08 18:00:51]

The CQ bit was checked by bengr@chromium.org

[commit-bot: I haz the power, 2014-05-08 18:06:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bengr@chromium.org/266033002/90001

[bengr, 2014-05-08 18:32:48]

The CQ bit was unchecked by bengr@chromium.org

[bengr, 2014-05-08 18:33:27]

The CQ bit was checked by bengr@chromium.org

[commit-bot: I haz the power, 2014-05-08 18:40:53]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bengr@chromium.org/266033002/90001

[commit-bot: I haz the power, 2014-05-08 23:08:51]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/18...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-05-08 23:34:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-08 23:34:02]

Try jobs failed on following builders:
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)

[bengr, 2014-05-10 01:55:19]

The CQ bit was checked by bengr@chromium.org

[commit-bot: I haz the power, 2014-05-10 01:56:25]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bengr@chromium.org/266033002/170001

[commit-bot: I haz the power, 2014-05-10 05:17:18]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-05-10 05:20:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-10 05:20:33]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[bengr, 2014-05-10 05:27:21]

The CQ bit was checked by bengr@chromium.org

[commit-bot: I haz the power, 2014-05-10 05:30:33]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bengr@chromium.org/266033002/170001

[commit-bot: I haz the power, 2014-05-10 06:24:01]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-05-11 01:09:37]

Change committed as 269670