Updates to the license information of third party components.

Updates to the license information of third party components.

LOG=n

Committed: https://crrev.com/e35ff8cd03f4382dc8e59c4801499a94bcbb6c5e
Cr-Commit-Position: refs/heads/master@{#29051}

[Michael Hablich, 2015-06-16 10:40:45]

hablich@chromium.org changed reviewers:
+ danno@chromium.org, ulan@chromium.org

[Michael Hablich, 2015-06-16 10:41:29]

PTAL

I have not deleted the information in /LICENSE as I am not sure if this is ok.

Assembler files are not updated/extracted as discussed with Ulan for now.

[ulan, 2015-06-16 10:55:50]

Thanks a lot for fixing this!

I see that js files are deleted, but not added. Did you forget to "git add" the
moved js files in the new pcre directory?

https://codereview.chromium.org/1183743003/diff/1/LICENSE
File LICENSE (right):

https://codereview.chromium.org/1183743003/diff/1/LICENSE#newcode29
LICENSE:29: Further license information can be found in individual
sub-directories.
Could you please list the sub-directories here?
src/third_party/, test/??

https://codereview.chromium.org/1183743003/diff/1/src/third_party/valgrind/LI...
File src/third_party/valgrind/LICENSE (right):

https://codereview.chromium.org/1183743003/diff/1/src/third_party/valgrind/LI...
src/third_party/valgrind/LICENSE:1: /* -*- c -*-
Can we remove the C++ comments and unindent the text?

https://codereview.chromium.org/1183743003/diff/1/src/third_party/vtune/LICENSE
File src/third_party/vtune/LICENSE (right):

https://codereview.chromium.org/1183743003/diff/1/src/third_party/vtune/LICEN...
src/third_party/vtune/LICENSE:3: /*
Remove the comment?

https://codereview.chromium.org/1183743003/diff/1/test/mjsunit/third_party/re...
File test/mjsunit/third_party/regexp-pcre/LICENSE (right):

https://codereview.chromium.org/1183743003/diff/1/test/mjsunit/third_party/re...
test/mjsunit/third_party/regexp-pcre/LICENSE:2: // ------------
Remove the "//" prefix?

[Michael Hablich, 2015-06-16 11:09:02]

https://codereview.chromium.org/1183743003/diff/1/LICENSE
File LICENSE (right):

https://codereview.chromium.org/1183743003/diff/1/LICENSE#newcode29
LICENSE:29: Further license information can be found in individual
sub-directories.
On 2015/06/16 10:55:49, ulan wrote:
> Could you please list the sub-directories here?
> src/third_party/, test/??

I deliberately do not want to do that or else the master file needs to be kept
in sync too. This information can easily be generated via:

find -iname 'LICENSE'

which currently produces: 

./build/gyp/LICENSE
./LICENSE
./testing/gtest/LICENSE
./testing/gmock/LICENSE
./testing/gmock/scripts/generator/LICENSE
./third_party/icu/LICENSE
./buildtools/LICENSE
./.git/refs/heads/license
./.git/logs/refs/heads/license
./src/third_party/valgrind/LICENSE
./src/third_party/fdlibm/LICENSE
./src/third_party/vtune/LICENSE
./test/mjsunit/third_party/regexp-pcre/LICENSE
./test/mjsunit/third_party/object-keys/LICENSE

https://codereview.chromium.org/1183743003/diff/1/src/third_party/valgrind/LI...
File src/third_party/valgrind/LICENSE (right):

https://codereview.chromium.org/1183743003/diff/1/src/third_party/valgrind/LI...
src/third_party/valgrind/LICENSE:1: /* -*- c -*-
On 2015/06/16 10:55:49, ulan wrote:
> Can we remove the C++ comments and unindent the text?

Done.

https://codereview.chromium.org/1183743003/diff/1/src/third_party/vtune/LICENSE
File src/third_party/vtune/LICENSE (right):

https://codereview.chromium.org/1183743003/diff/1/src/third_party/vtune/LICEN...
src/third_party/vtune/LICENSE:3: /*
On 2015/06/16 10:55:49, ulan wrote:
> Remove the comment?

Done.

https://codereview.chromium.org/1183743003/diff/1/test/mjsunit/third_party/re...
File test/mjsunit/third_party/regexp-pcre/LICENSE (right):

https://codereview.chromium.org/1183743003/diff/1/test/mjsunit/third_party/re...
test/mjsunit/third_party/regexp-pcre/LICENSE:2: // ------------
On 2015/06/16 10:55:50, ulan wrote:
> Remove the "//" prefix?

Done.

[ulan, 2015-06-16 11:39:56]

lgtm with two comments

> NOTRY=true
Please remove this from description, since we are changing the test.

> LOG=n
Why not LOG=Y?

[Michael Hablich, 2015-06-16 11:57:46]

On 2015/06/16 11:39:56, ulan wrote:
> lgtm with two comments
> 
> > NOTRY=true
> Please remove this from description, since we are changing the test.
> 
> > LOG=n
> Why not LOG=Y?

Removed NOTRY;

It isn't a change I would want to add to a changelog. It is just a clean-up.

[Michael Hablich, 2015-06-16 12:14:18]

The CQ bit was checked by hablich@chromium.org

[commit-bot: I haz the power, 2015-06-16 12:14:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1183743003/40001

[commit-bot: I haz the power, 2015-06-16 12:37:43]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-06-16 12:38:01]

Patchset 3 (id:??) landed as
https://crrev.com/e35ff8cd03f4382dc8e59c4801499a94bcbb6c5e
Cr-Commit-Position: refs/heads/master@{#29051}