Closed Bug 1323120 Opened 8 years ago Closed 8 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
Status: NEW → RESOLVED
Closed: 8 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: