1168077 - Remove remaining SpiderMonkey specific JS syntax in browser/devtools

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[Patrick Brosset <:pbro>]

From bug 887895, and looking at its blocking bugs, the remaining offenders in browser/devtools should be:

tilt
====

  tilt/tilt-gl.js:838:11
  Unexpected identifier

webaudioeditor
==============

  webaudioeditor/panel.js:55:16
  Unexpected token this

  webaudioeditor/test/browser_wa_properties-view-edit-01.js:60:11
  Unexpected identifier

sourceeditor
============

  sourceeditor/editor.js:85:54
  Unexpected token for

storage
=======

  storage/panel.js:34:16
  Unexpected token this

netmonitor
==========

  netmonitor/netmonitor-controller.js:371:16
  Unexpected identifier

  netmonitor/netmonitor-view.js:951:30
  Unexpected token (

  netmonitor/panel.js:59:16
  Unexpected token this

  netmonitor/test/browser_net_complex-params.js:20:13
  Unexpected identifier

  netmonitor/test/browser_net_simple-request-details.js:18:13
  Unexpected identifier

  netmonitor/test/head.js:408:49
  Unexpected token for

commandline
===========

  commandline/test/browser_cmd_screenshot.js:29:23
  Unexpected identifier

Comment 1

[Patrick Brosset <:pbro>]

+ all the tests in browser/devools/performance/test that miss the * for the spawTest generator function.

Comment 2

[Patrick Brosset <:pbro>]

+ things in shadereditor too.

Comment 3

[Patrick Brosset <:pbro>]

Attachment: bug1168077-spidermonkey-remaining-browser-devtools.diff

Comment 4

[Patrick Brosset <:pbro>]

Comment on attachment 8610097 [details] [diff] [review]
bug1168077-spidermonkey-remaining-browser-devtools.diff

https://treeherder.mozilla.org/#/jobs?repo=try&revision=315440d4ea04

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/30945769c039

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/30945769c039