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)
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•8 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•8 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•8 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•8 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•8 years ago
|
Flags: needinfo?(gerv)
Assignee | ||
Updated•8 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•8 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 6•8 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•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•