Prepare for new EvaluateScript event coming from V8.

Prepare for new EvaluateScript event coming from V8.

BUG=655430
TBR=caseq@chromium.org
NOTRY=true

Committed: https://crrev.com/799eabe2822ffaea3ca8c841d5bca1fbb9fbc979
Cr-Commit-Position: refs/heads/master@{#425398}

[alph, 2016-10-14 18:05:39]

alph@chromium.org changed reviewers:
+ caseq@chromium.org

[caseq, 2016-10-14 18:08:44]

https://codereview.chromium.org/2422643003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js
(right):

https://codereview.chromium.org/2422643003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js:336: var
data = traceEvent.args && (traceEvent.args["beginData"] ||
traceEvent.args["data"]);
This check shouldn't be necessary, we've always got args.

[alph, 2016-10-14 18:26:19]

Description was changed from

==========
DevTools: Prepare to accept trace events with no args

BUG=655430
==========

to

==========
Prepare to new EvaluateScript event coming from V8.

BUG=655430
==========

[alph, 2016-10-14 18:26:35]

Description was changed from

==========
Prepare to new EvaluateScript event coming from V8.

BUG=655430
==========

to

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
==========

[alph, 2016-10-14 18:26:53]

ptal

[alph, 2016-10-14 18:27:53]

Description was changed from

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
==========

to

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
TBR=caseq
NOTRY=true
==========

[alph, 2016-10-14 18:28:15]

Description was changed from

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
TBR=caseq
NOTRY=true
==========

to

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
TBR=caseq@chromium.org
NOTRY=true
==========

[alph, 2016-10-14 18:29:04]

The CQ bit was checked by alph@chromium.org

[commit-bot: I haz the power, 2016-10-14 18:29:48]

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-10-14 18:37:13]

Description was changed from

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
TBR=caseq@chromium.org
NOTRY=true
==========

to

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
TBR=caseq@chromium.org
NOTRY=true
==========

[commit-bot: I haz the power, 2016-10-14 18:37:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-14 18:38:50]

Description was changed from

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
TBR=caseq@chromium.org
NOTRY=true
==========

to

==========
Prepare for new EvaluateScript event coming from V8.

BUG=655430
TBR=caseq@chromium.org
NOTRY=true

Committed: https://crrev.com/799eabe2822ffaea3ca8c841d5bca1fbb9fbc979
Cr-Commit-Position: refs/heads/master@{#425398}
==========

[commit-bot: I haz the power, 2016-10-14 18:38:51]

Patchset 2 (id:??) landed as
https://crrev.com/799eabe2822ffaea3ca8c841d5bca1fbb9fbc979
Cr-Commit-Position: refs/heads/master@{#425398}