Add paper-dropdown and paper-dropdown-menu and run reproduce.sh

Add paper-dropdown and paper-dropdown-menu and run reproduce.sh

R=dzhioev@chromium.org
BUG=

Committed: https://crrev.com/150281d62a5fc771503d45a00746c62c7bb31535
Cr-Commit-Position: refs/heads/master@{#317235}

[Jeremy Klein, 2015-02-09 23:34:27]

Note that most of the files changed in this CL are just due to the new version
of vulcanize adding utf-8 encoding.

Should I make a note about web-animations-js in README.chromium?

[michaelpg, 2015-02-09 23:35:24]

michaelpg@chromium.org changed reviewers:
+ michaelpg@chromium.org

[michaelpg, 2015-02-09 23:35:25]

did you forget to "git add -f" paper-dropdown and paper-dropdown-menu in
components and component-chromium?

https://codereview.chromium.org/864173009/diff/1/third_party/polymer/bower.json
File third_party/polymer/bower.json (right):

https://codereview.chromium.org/864173009/diff/1/third_party/polymer/bower.js...
third_party/polymer/bower.json:71: "web-animations-js":
"web-animations/web-animations-js#1.0.5"
add a line in the README explaining this polyfill

[Jeremy Klein, 2015-02-10 00:08:39]

jlklein@chromium.org changed reviewers:
+ dzhioev@chromium.org
- raymes@chromium.org

[Jeremy Klein, 2015-02-10 04:47:48]

Note: web-animations-js/test/* is huge. Would it be ok to remove it in
reproduce.sh?

[michaelpg, 2015-02-10 05:47:07]

Yeah, if it's just a test suite for web-animations-js itself, I think
that's a reasonable thing to do.

On Mon, Feb 9, 2015 at 8:47 PM, <jlklein@chromium.org> wrote:

> Note: web-animations-js/test/* is huge. Would it be ok to remove it in
> reproduce.sh?
>
> https://codereview.chromium.org/864173009/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[dzhioev (left Google), 2015-02-10 10:14:05]

On 2015/02/09 23:27:26, Jeremy Klein wrote:

Hello Jeremy,

the things are harder than they look. Formally, web-animations is a
separate third-party library so you need to follow instructions from
page [1] to add it. Last time when I tried to do it [2] I didn't get
approval from Chromium security guys, because web-animations was not
stable enough. The things have changed since then, so security guy
shouldn't have objections now.
[3] is a tracking bug for adding web-animations (never mind it says
"web animations *next*"). Note: you need to place the library to separate
directory in src/third_party with separate README.chromium, otherwise
'tools/licenses.py' couldn't generate license file correctly.
I propose to move web-animations sources from 'third-party/polymer/components'
to
'third-party/web-animations' using reproduce.sh script, right after 'bower
install'
step.

Good luck.

[1] http://www.chromium.org/developers/adding-3rd-party-libraries
[2] https://codereview.chromium.org/603683003/
[3] http://crbug.com/424574

[Jeremy Klein, 2015-02-10 19:33:21]

On 2015/02/10 10:14:05, dzhioev wrote:
> On 2015/02/09 23:27:26, Jeremy Klein wrote:
> 
> Hello Jeremy,
> 
> the things are harder than they look. Formally, web-animations is a
> separate third-party library so you need to follow instructions from
> page [1] to add it. Last time when I tried to do it [2] I didn't get
> approval from Chromium security guys, because web-animations was not
> stable enough. The things have changed since then, so security guy
> shouldn't have objections now.
> [3] is a tracking bug for adding web-animations (never mind it says
> "web animations *next*"). Note: you need to place the library to separate
> directory in src/third_party with separate README.chromium, otherwise
> 'tools/licenses.py' couldn't generate license file correctly.
> I propose to move web-animations sources from 'third-party/polymer/components'
> to
> 'third-party/web-animations' using reproduce.sh script, right after 'bower
> install'
> step.
> 
> Good luck.
> 
> [1] http://www.chromium.org/developers/adding-3rd-party-libraries
> [2] https://codereview.chromium.org/603683003/
> [3] http://crbug.com/424574

Thanks for all the info, Pavel. I've added some questions and comments to that
crbug. Should core-animation be using web-animations-next? Who is the security
person I should talk to about all this?

[Jeremy Klein, 2015-02-17 23:31:16]

This should be fully up-to-date now. Did you guys have any comments on this that
were separate from web-animations-js stuff?

[michaelpg, 2015-02-19 00:06:36]

lgtm

[Jeremy Klein, 2015-02-19 21:39:52]

New patchsets have been uploaded after l-g-t-m from michaelpg@chromium.org

[Jeremy Klein, 2015-02-19 21:40:08]

jlklein@chromium.org changed reviewers:
+ dbeam@chromium.org

[Jeremy Klein, 2015-02-20 01:45:42]

The CQ bit was checked by jlklein@chromium.org

[commit-bot: I haz the power, 2015-02-20 01:46:48]

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

[commit-bot: I haz the power, 2015-02-20 04:05:26]

Committed patchset #7 (id:90001)

[commit-bot: I haz the power, 2015-02-20 04:06:21]

Patchset 7 (id:??) landed as
https://crrev.com/150281d62a5fc771503d45a00746c62c7bb31535
Cr-Commit-Position: refs/heads/master@{#317235}