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)
Core Graveyard
Plug-ins
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 | ||
Updated•8 years ago
|
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
Assignee | ||
Comment 1•7 years ago
|
||
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)
Assignee | ||
Comment 2•7 years ago
|
||
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)
Assignee | ||
Comment 4•7 years ago
|
||
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 5•7 years ago
|
||
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+
Updated•7 years ago
|
Flags: needinfo?(gerv)
Assignee | ||
Updated•7 years ago
|
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
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 6•7 years ago
|
||
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
Comment 8•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6768b6d8ba5a
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•