Remove code for deprecated and rotted histogram.

Remove code for deprecated and rotted histogram.

WebCore.Scripts.ParsingBlocking.TimeBetweenLoadedAndCompiled is marked
as deprecated in histograms.xml (see
https://chromium.googlesource.com/chromium/src/+/68cde75708853730a1881142d330d573194dc3e4),
but we still have code checked in to report it. The code has also
rotted. For example, the code for this histogram depends on the state
of a boolean member added to HTMLScriptRunner that's always false.

BUG=591399
Committed: https://crrev.com/f1411c1e570b2e44bdd0c8a0b17a9d0aa6f901a6
Cr-Commit-Position: refs/heads/master@{#378926}

[Bryan McQuade, 2016-03-02 17:27:57]

bmcquade@chromium.org changed reviewers:
+ kinuko@chromium.org

[Bryan McQuade, 2016-03-02 17:30:29]

Kinuko, PTAL. Just removing some unused histogram code.

Jochen, Marja, based on my reading of histograms.xml, this histogram is no
longer needed (marked as deprecated) and the code to record it appears to have
rotted a bit, so I'd like to remove it. Please let us know if it should not be
removed.

[Bryan McQuade, 2016-03-02 17:35:48]

Description was changed from

==========
Remove code for deprecated and rotted histogram.

WebCore.Scripts.ParsingBlocking.TimeBetweenLoadedAndCompiled is marked
as deprecated in histograms.xml, but we still have code checked in to
report it. The code has also rotted. For example, the code for this
histogram depends on the state of a boolean member added to
HTMLScriptRunner that's always false.

BUG=591399
==========

to

==========
Remove code for deprecated and rotted histogram.

WebCore.Scripts.ParsingBlocking.TimeBetweenLoadedAndCompiled is marked
as deprecated in histograms.xml (see
https://chromium.googlesource.com/chromium/src/+/68cde75708853730a1881142d330...),
but we still have code checked in to
report it. The code has also rotted. For example, the code for this
histogram depends on the state of a boolean member added to
HTMLScriptRunner that's always false.

BUG=591399
==========

[Bryan McQuade, 2016-03-02 17:36:12]

Description was changed from

==========
Remove code for deprecated and rotted histogram.

WebCore.Scripts.ParsingBlocking.TimeBetweenLoadedAndCompiled is marked
as deprecated in histograms.xml (see
https://chromium.googlesource.com/chromium/src/+/68cde75708853730a1881142d330...),
but we still have code checked in to
report it. The code has also rotted. For example, the code for this
histogram depends on the state of a boolean member added to
HTMLScriptRunner that's always false.

BUG=591399
==========

to

==========
Remove code for deprecated and rotted histogram.

WebCore.Scripts.ParsingBlocking.TimeBetweenLoadedAndCompiled is marked
as deprecated in histograms.xml (see
https://chromium.googlesource.com/chromium/src/+/68cde75708853730a1881142d330...),
but we still have code checked in to report it. The code has also
rotted. For example, the code for this histogram depends on the state
of a boolean member added to HTMLScriptRunner that's always false.

BUG=591399
==========

[jochen (gone - plz use gerrit), 2016-03-02 17:39:51]

jochen@chromium.org changed reviewers:
+ jochen@chromium.org, vogelheim@chromium.org

[jochen (gone - plz use gerrit), 2016-03-02 17:39:51]

+vogelheim

is that correct that we don't need those histograms anymore?

[vogelheim, 2016-03-02 17:46:38]

On 2016/03/02 17:39:51, jochen wrote:
> +vogelheim
> 
> is that correct that we don't need those histograms anymore?

"Our" histograms are in another ScriptRunner; I don't really have an opinion on
the HTMLScriptRunner & friends.

[Bryan McQuade, 2016-03-02 21:46:14]

On 2016/03/02 at 17:46:38, vogelheim wrote:
> On 2016/03/02 17:39:51, jochen wrote:
> > +vogelheim
> > 
> > is that correct that we don't need those histograms anymore?
> 
> "Our" histograms are in another ScriptRunner; I don't really have an opinion
on the HTMLScriptRunner & friends.

Ah, I see the other WebCore.Scripts.* are in
third_party/WebKit/Source/bindings/core/v8/ScriptStreamer.cpp. It appears that
the histogram I am removing is the only one in HTMLScriptRunner.cpp. Given that
this histogram is marked as deprecated and thus not available in the UMA dash
(and hasn't been for about a year), and that it's in a different file from the
currently used Webore.Scripts.* histograms, it seems like we should remove this
one.

[kinuko, 2016-03-03 02:22:55]

lgtm per discussion

[Bryan McQuade, 2016-03-03 02:38:44]

The CQ bit was checked by bmcquade@chromium.org

[commit-bot: I haz the power, 2016-03-03 02:39:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1754213003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1754213003/1

[commit-bot: I haz the power, 2016-03-03 02:44:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-03 02:46:57]

Description was changed from

==========
Remove code for deprecated and rotted histogram.

WebCore.Scripts.ParsingBlocking.TimeBetweenLoadedAndCompiled is marked
as deprecated in histograms.xml (see
https://chromium.googlesource.com/chromium/src/+/68cde75708853730a1881142d330...),
but we still have code checked in to report it. The code has also
rotted. For example, the code for this histogram depends on the state
of a boolean member added to HTMLScriptRunner that's always false.

BUG=591399
==========

to

==========
Remove code for deprecated and rotted histogram.

WebCore.Scripts.ParsingBlocking.TimeBetweenLoadedAndCompiled is marked
as deprecated in histograms.xml (see
https://chromium.googlesource.com/chromium/src/+/68cde75708853730a1881142d330...),
but we still have code checked in to report it. The code has also
rotted. For example, the code for this histogram depends on the state
of a boolean member added to HTMLScriptRunner that's always false.

BUG=591399

Committed: https://crrev.com/f1411c1e570b2e44bdd0c8a0b17a9d0aa6f901a6
Cr-Commit-Position: refs/heads/master@{#378926}
==========

[commit-bot: I haz the power, 2016-03-03 02:46:58]

Patchset 1 (id:??) landed as
https://crrev.com/f1411c1e570b2e44bdd0c8a0b17a9d0aa6f901a6
Cr-Commit-Position: refs/heads/master@{#378926}