Add test for Khronos Group license and whitelist it.

Add test for Khronos Group license and whitelist it.


BUG=

TBR=brettw@chromium.org,cpu@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=190741

[Paweł Hajdan Jr., 2013-03-25 18:29:45]

https://codereview.chromium.org/12502025/diff/1/third_party/devscripts/licens...
File third_party/devscripts/licensecheck.pl (right):

https://codereview.chromium.org/12502025/diff/1/third_party/devscripts/licens...
third_party/devscripts/licensecheck.pl:580: if ($licensetext =~ /The Khronos
Group Inc. All Rights Reserved./) {
This is not enough. Let's remove this and match on "These materials are
protected by copyright laws and contain material proprietary to the Khronos
Group, Inc. You may use these materials for implementing Khronos specifications,
without altering or removing 
any trademark, copyright or other notice from the specification." instead.

https://codereview.chromium.org/12502025/diff/1/tools/checklicenses/checklice...
File tools/checklicenses/checklicenses.py (right):

https://codereview.chromium.org/12502025/diff/1/tools/checklicenses/checklice...
tools/checklicenses/checklicenses.py:59: 'Khronos Group',
Please move it to an exclusion section specific to the files you're adding
(PATH_SPECIFIC...). I don't think we should permit this one globally.

[Raymond Toy (Google), 2013-03-25 21:49:54]

On 2013/03/25 18:29:45, Paweł Hajdan Jr. wrote:
>
https://codereview.chromium.org/12502025/diff/1/third_party/devscripts/licens...
> File third_party/devscripts/licensecheck.pl (right):
> 
>
https://codereview.chromium.org/12502025/diff/1/third_party/devscripts/licens...
> third_party/devscripts/licensecheck.pl:580: if ($licensetext =~ /The Khronos
> Group Inc. All Rights Reserved./) {
> This is not enough. Let's remove this and match on "These materials are
> protected by copyright laws and contain material proprietary to the Khronos
> Group, Inc. You may use these materials for implementing Khronos
specifications,
> without altering or removing 
> any trademark, copyright or other notice from the specification." instead.
> 
>
https://codereview.chromium.org/12502025/diff/1/tools/checklicenses/checklice...
> File tools/checklicenses/checklicenses.py (right):
> 
>
https://codereview.chromium.org/12502025/diff/1/tools/checklicenses/checklice...
> tools/checklicenses/checklicenses.py:59: 'Khronos Group',
> Please move it to an exclusion section specific to the files you're adding
> (PATH_SPECIFIC...). I don't think we should permit this one globally.

Fixed these issues and simplified pattern in patch #3.

[Paweł Hajdan Jr., 2013-03-25 21:52:20]

LGTM

[Raymond Toy (Google), 2013-03-25 22:02:12]

+brettw, +cpu

Please review.

[brettw, 2013-03-25 23:46:02]

LGTM

[Raymond Toy (Google), 2013-03-26 16:18:07]

On 2013/03/25 23:46:02, brettw wrote:
> LGTM

Thanks everyone.

[commit-bot: I haz the power, 2013-03-26 16:20:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtoy@google.com/12502025/6001

[commit-bot: I haz the power, 2013-03-26 19:47:40]

Change committed as 190741