Closed Bug 1323120 Opened 8 years ago Closed 7 years ago

[jsplugins] Add license for Chromium and third party code in mortar project

Categories

(Core Graveyard :: Plug-ins, defect)

defect
Not set
normal

Tracking

(firefox53 fixed)

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: lochang, Assigned: lochang)

References

Details

Attachments

(1 file, 1 obsolete file)

In this bug, we would:
1) Add license for Chromium code inside browser/extensions/mortar/ppapi/
2) Add license for third party code inside browser/extensions/mortar/json/
3) Add license for third party code inside browser/extensions/mortar/host/common/opengles2-utils.jsm
Assignee: nobody → lochang
No longer blocks: 1309444
Summary: [jsplugins] Add license for Chromium and third party code for mortar project → [jsplugins] Add license for Chromium and third party code in mortar project
Based on our offline discussion with Elvin, i submit the patch which add license for chromium and third party code in mortar project.

Hi Vance, would you please take a look at the patch see if there is still something missing and also ask for the right person to review? Thanks!
Flags: needinfo?(vchen)
Rename the JSON license to JSMN license.
Attachment #8818183 - Attachment is obsolete: true
Hi Gerv, could you help to review this patch ? 

And to provide some context:

We are going to land Project Mortar[1] into m-c. We have submitted Bug#1313295 for code review and Bug#1314029 for legal discussions. In Hawaii we have met with Elvin and he pointed out that in order to land Mortar project on m-c we should modify the license.html file to cover:

1. The PPAPI source code we got from Google Chromium project and use in the Mortar project.
2. The json source files.
3. The opengles2-utils.jsm files.

So please help to check if the modification is OK

[1]https://wiki.mozilla.org/Mortar_Project
Flags: needinfo?(vchen) → needinfo?(gerv)
Comment on attachment 8818211 [details] [diff] [review]
[Final] Bug 1323120 - Add license for Chromium and thrid party code in mortar project, r=gerv

Directly set review flag to Gerv.
Attachment #8818211 - Flags: review?(gerv)
Comment on attachment 8818211 [details] [diff] [review]
[Final] Bug 1323120 - Add license for Chromium and thrid party code in mortar project, r=gerv

Review of attachment 8818211 [details] [diff] [review]:
-----------------------------------------------------------------

r=gerv.

Gerv
Attachment #8818211 - Flags: review?(gerv) → review+
Flags: needinfo?(gerv)
Attachment #8818211 - Attachment description: Bug 1323120 - Add license for Chromium and thrid party code in mortar project → [Final] Bug 1323120 - Add license for Chromium and thrid party code in mortar project, r=gerv
Keywords: checkin-needed
Thanks for Gerv's review.
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/6768b6d8ba5a
Add license for Chromium and third party code in mortar project. r=gerv
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/6768b6d8ba5a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Depends on: 1347440
No longer depends on: 1347440
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: