DevTools: use recordEnumeratedHistogram instead of recordActionTaken

DevTools: use recordEnumeratedHistogram instead of recordActionTaken

The patch migrates off recordActionTaken method in favor of recordEnumeratedHistogram.
This makes it possible to supply bucket sizes from the javascript code.

R=dgozman
BUG=468026
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=193409

[lushnikov, 2015-04-03 14:18:31]

PTAL.

Chromium-side patch should land first:
https://codereview.chromium.org/1056073003

[dgozman, 2015-04-03 14:23:48]

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/d...
File Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js
(right):

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/d...
Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js:184: *
@override
Should not be override.

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/d...
Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js:187:
recordActionTaken: function(actionCode)
Move to backwards-compat section.

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/h...
File Source/devtools/front_end/host/InspectorFrontendHost.js (right):

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/h...
Source/devtools/front_end/host/InspectorFrontendHost.js:229: recordActionTaken:
function(actionCode) { },
Remove these.

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/h...
Source/devtools/front_end/host/InspectorFrontendHost.js:472: recordActionTaken:
function(actionCode)
Remove these.

[lushnikov, 2015-04-03 14:41:25]

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/d...
File Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js
(right):

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/d...
Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js:184: *
@override
On 2015/04/03 14:23:48, dgozman wrote:
> Should not be override.

Done.

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/d...
Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js:187:
recordActionTaken: function(actionCode)
On 2015/04/03 14:23:48, dgozman wrote:
> Move to backwards-compat section.

Done.

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/h...
File Source/devtools/front_end/host/InspectorFrontendHost.js (right):

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/h...
Source/devtools/front_end/host/InspectorFrontendHost.js:229: recordActionTaken:
function(actionCode) { },
On 2015/04/03 14:23:48, dgozman wrote:
> Remove these.

Done.

https://codereview.chromium.org/1061503002/diff/1/Source/devtools/front_end/h...
Source/devtools/front_end/host/InspectorFrontendHost.js:472: recordActionTaken:
function(actionCode)
On 2015/04/03 14:23:48, dgozman wrote:
> Remove these.

Done.

[dgozman, 2015-04-03 14:43:17]

lgtm

https://codereview.chromium.org/1061503002/diff/20001/Source/devtools/front_e...
File Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js
(right):

https://codereview.chromium.org/1061503002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js:444: *
@param {number} actionCode
Support for legacy front-ends (<M43).

https://codereview.chromium.org/1061503002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js:452: *
@param {number} panelCode
ditto

[lushnikov, 2015-04-03 14:49:09]

https://codereview.chromium.org/1061503002/diff/20001/Source/devtools/front_e...
File Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js
(right):

https://codereview.chromium.org/1061503002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js:444: *
@param {number} actionCode
On 2015/04/03 14:43:17, dgozman wrote:
> Support for legacy front-ends (<M43).

Done.

https://codereview.chromium.org/1061503002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js:452: *
@param {number} panelCode
On 2015/04/03 14:43:17, dgozman wrote:
> ditto

Done.

[lushnikov, 2015-04-08 14:04:14]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2015-04-08 14:04:14]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/1061503002/#ps60001
(title: "rebaseline")

[commit-bot: I haz the power, 2015-04-08 14:04:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1061503002/60001

[commit-bot: I haz the power, 2015-04-08 17:53:35]

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

[commit-bot: I haz the power, 2015-04-08 17:53:36]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/50966)

[lushnikov, 2015-04-08 23:45:39]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2015-04-08 23:45:39]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/1061503002/#ps80001
(title: "fix inspector/user-metrics.html")

[commit-bot: I haz the power, 2015-04-08 23:46:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1061503002/80001

[commit-bot: I haz the power, 2015-04-09 02:50:45]

Committed patchset #5 (id:80001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=193409