DevTools: (anonymous function) ==> (anonymous)

DevTools: (anonymous function) ==> (anonymous)

(anonymous function) is oft-seen and could use a better signal:noise ratio. Furthermore, in the case of callstacks, we don't want to elide a filename to prioritize showing this function "name".

BUG=644580
Committed: https://crrev.com/f1080c137ac52fd7cc673f89c3f91affa5a880f3
Cr-Commit-Position: refs/heads/master@{#417451}

[paulirish, 2016-09-07 01:19:56]

The CQ bit was checked by paulirish@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 01:20:16]

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

[paulirish, 2016-09-07 01:27:57]

Description was changed from

==========
DevTools: (anonymous function) ==> (anonymous)
==========

to

==========
DevTools: (anonymous function) ==> (anonymous)

(anonymous function) is oft-seen and can use higher signal:noise. Furthermore,
in the case of callstacks, we don't want to elide a filename to prioritize
showing this function "name".
==========

[paulirish, 2016-09-07 01:28:45]

Description was changed from

==========
DevTools: (anonymous function) ==> (anonymous)

(anonymous function) is oft-seen and can use higher signal:noise. Furthermore,
in the case of callstacks, we don't want to elide a filename to prioritize
showing this function "name".
==========

to

==========
DevTools: (anonymous function) ==> (anonymous)

(anonymous function) is oft-seen and could use a better signal:noise ratio.
Furthermore, in the case of callstacks, we don't want to elide a filename to
prioritize showing this function "name".
==========

[paulirish, 2016-09-07 01:29:39]

paulirish@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org

[paulirish, 2016-09-07 01:29:40]

kozyatinskiy, you're first (of many) to review. 

Any concerns here?

[kozy, 2016-09-07 01:47:35]

lgtm

https://codereview.chromium.org/2297903005/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/console/console-uncaught-exception-in-eval-expected.txt
(right):

https://codereview.chromium.org/2297903005/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/console/console-uncaught-exception-in-eval-expected.txt:18:

Why new line here?

[paulirish, 2016-09-07 01:52:26]

Description was changed from

==========
DevTools: (anonymous function) ==> (anonymous)

(anonymous function) is oft-seen and could use a better signal:noise ratio.
Furthermore, in the case of callstacks, we don't want to elide a filename to
prioritize showing this function "name".
==========

to

==========
DevTools: (anonymous function) ==> (anonymous)

(anonymous function) is oft-seen and could use a better signal:noise ratio.
Furthermore, in the case of callstacks, we don't want to elide a filename to
prioritize showing this function "name". 

BUG=644580
==========

[paulirish, 2016-09-07 01:52:36]

paulirish@chromium.org changed reviewers:
+ caseq@chromium.org, pfeldman@chromium.org

[paulirish, 2016-09-07 01:52:37]

https://codereview.chromium.org/2297903005/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/console/console-uncaught-exception-in-eval-expected.txt
(right):

https://codereview.chromium.org/2297903005/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/console/console-uncaught-exception-in-eval-expected.txt:18:

On 2016/09/07 at 01:47:34, kozyatinskiy wrote:
> Why new line here?

fixed. :)

[paulirish, 2016-09-08 18:49:45]

The CQ bit was checked by paulirish@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-08 18:50:11]

Dry run: 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-09-08 20:39:14]

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

[commit-bot: I haz the power, 2016-09-08 20:39:15]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[paulirish, 2016-09-08 22:54:17]

The CQ bit was checked by paulirish@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-08 22:54:46]

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

[caseq, 2016-09-08 22:55:53]

lgtm

[paulirish, 2016-09-08 22:56:53]

The CQ bit was unchecked by paulirish@chromium.org

[paulirish, 2016-09-08 22:56:55]

The CQ bit was checked by paulirish@chromium.org

[paulirish, 2016-09-08 22:56:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
kozyatinskiy@chromium.org
Link to the patchset: https://codereview.chromium.org/2297903005/#ps80001
(title: "tests")

[commit-bot: I haz the power, 2016-09-08 22:57:44]

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-09-09 00:34:56]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-09-09 00:37:20]

Description was changed from

==========
DevTools: (anonymous function) ==> (anonymous)

(anonymous function) is oft-seen and could use a better signal:noise ratio.
Furthermore, in the case of callstacks, we don't want to elide a filename to
prioritize showing this function "name". 

BUG=644580
==========

to

==========
DevTools: (anonymous function) ==> (anonymous)

(anonymous function) is oft-seen and could use a better signal:noise ratio.
Furthermore, in the case of callstacks, we don't want to elide a filename to
prioritize showing this function "name".

BUG=644580

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

[commit-bot: I haz the power, 2016-09-09 00:37:22]

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