1243243 - Use standard license boilerplate in debugger js files

Status: RESOLVED FIXED

(DevTools :: Debugger, defect)

Description

[Lin Clark [:linclark]]

See Bug 1241947

Comment 1

[Lin Clark [:linclark]]

Attachment: Bug1243243.patch

This patch updates the license block in the debugger js files.

I skipped the code_* files because line numbers from those were hardcoded in tests.

Here's a try push that just runs the devtools tests on Linux. Let me know if I should run a full one.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=3b6f973ea735&selectedJob=16009487

Comment 2

[Lin Clark [:linclark]]

Attachment: Bug1243243.patch

:jryans pointed out that tests should be public domain, so updated the patch.

new try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=65e18a5eb7dc

Comment 3

[James Long (:jlongster)]

Sorry, I thought we were removing the emacs/vim mode lines? Nobody said they actually used them and several people said they'd be fine if they were removed.

Comment 4

[Lin Clark [:linclark]]

What I said in the meeting is "We can have that fight later". They are currently in the header because Mozilla Code Standard says they should be there.

My goal is to make everything consistent. Once it's consistent, it will be a lot easier to change what our default is.

Comment 5

[James Long (:jlongster)]

Ok. This looks like more than half the files, and if so maybe the initial consistency should be just removing it (majority wins) :) But this is fine with me.

Comment 6

[Lin Clark [:linclark]]

Yeah, I did notice that debugger was more internally consistent than others. The style seems to vary between components, with each component having its own 2 or 3 styles. In webconsole, there were a lot of files that just had the vim line, but not the emacs line. 

Thanks for the review :)

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/440a72b7b690

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/440a72b7b690