Remove obsolete patching step from runtime callstats.html

Remove obsolete patching step from runtime callstats.html

BUG=
R=cbruni@chromium.org

Committed: https://crrev.com/9d66b3f3d3a0135ca8e324e2f24f3a1d9cb70d43
Cr-Commit-Position: refs/heads/master@{#37534}

[Camillo Bruni, 2016-07-05 13:55:23]

sorry... forgot to press the "send" button this morning.

https://codereview.chromium.org/2119823002/diff/60001/tools/callstats.py
File tools/callstats.py (right):

https://codereview.chromium.org/2119823002/diff/60001/tools/callstats.py#newc...
tools/callstats.py:322: ('Group-Callback', re.compile("Callback$")),
This won't match FunctionCallback, so we do need the .* (just checked,
apparently python is quite different from JS)

[jochen (gone - plz use gerrit), 2016-07-05 14:09:39]

https://codereview.chromium.org/2119823002/diff/60001/tools/callstats.py
File tools/callstats.py (right):

https://codereview.chromium.org/2119823002/diff/60001/tools/callstats.py#newc...
tools/callstats.py:322: ('Group-Callback', re.compile("Callback$")),
On 2016/07/05 at 13:55:23, Camillo Bruni wrote:
> This won't match FunctionCallback, so we do need the .* (just checked,
apparently python is quite different from JS)

I copied this from the v8-perf repo, so I guess it's then broken there?

[jochen (gone - plz use gerrit), 2016-07-05 14:24:27]

ptal

[Camillo Bruni, 2016-07-05 14:25:02]

On 2016/07/05 at 14:09:39, jochen wrote:
> https://codereview.chromium.org/2119823002/diff/60001/tools/callstats.py
> File tools/callstats.py (right):
> 
>
https://codereview.chromium.org/2119823002/diff/60001/tools/callstats.py#newc...
> tools/callstats.py:322: ('Group-Callback', re.compile("Callback$")),
> On 2016/07/05 at 13:55:23, Camillo Bruni wrote:
> > This won't match FunctionCallback, so we do need the .* (just checked,
apparently python is quite different from JS)
> 
> I copied this from the v8-perf repo, so I guess it's then broken there?

right, that's the outdated version... should remove it.

[Camillo Bruni, 2016-07-05 14:30:11]

lgtm

[jochen (gone - plz use gerrit), 2016-07-05 14:34:08]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-07-05 14:34:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-05 14:58:35]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-07-05 14:58:37]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-05 14:59:17]

Description was changed from

==========
Remove obsolete patching step from runtime callstats.html

BUG=
R=cbruni@chromium.org
==========

to

==========
Remove obsolete patching step from runtime callstats.html

BUG=
R=cbruni@chromium.org

Committed: https://crrev.com/9d66b3f3d3a0135ca8e324e2f24f3a1d9cb70d43
Cr-Commit-Position: refs/heads/master@{#37534}
==========

[commit-bot: I haz the power, 2016-07-05 14:59:18]

Patchset 6 (id:??) landed as
https://crrev.com/9d66b3f3d3a0135ca8e324e2f24f3a1d9cb70d43
Cr-Commit-Position: refs/heads/master@{#37534}