Checkout Polymer using bower instead of the raw repository.

Checkout Polymer using bower instead of the raw repository.

This adds Polymer to third_party using bower rather than the raw repository.
Bower is the recommended way to manage Polymer. It makes it easy to update
this dependency in chrome as well. Instructions for updating polymer are
included in the README.chromium file.

The existing references to polymer.min.js have been updated.

This change is so that more polymer components can easily be added which  are
needed for the PDF extension.

R=bshe@chromium.org

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

[bryeung, 2013-12-16 10:49:49]

I'm definitely not the right reviewer for this...

Bryan


On Mon, Dec 16, 2013 at 1:26 AM, <raymes@chromium.org> wrote:

> Reviewers: bryeung,
>
> Description:
> Checkout Polymer using bower instead of the raw repository.
>
> This adds Polymer to third_party using bower rather than the raw
> repository.
> Bower is the recommended way to manage Polymer. It makes it easy to update
> this dependency in chrome as well. Instructions for updating polymer are
> included in the README.chromium file.
>
> The existing references to polymer.min.js have been updated.
>
> This change is so that more polymer components can easily be added which
>  are
> needed for the PDF extension.
>
> Please review this at https://codereview.chromium.org/98583008/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files (+430, -151 lines):
>   D third_party/polymer/platform.js
>   M ui/keyboard/resources/polymer_loader.js
>   A third_party/polymer/platform/.bower.json
>   D third_party/polymer/PATENTS
>   A third_party/polymer/polymer/polymer.html
>   A third_party/polymer/platform/platform.js.map
>   A third_party/polymer/polymer/README.md
>   A third_party/polymer/platform/AUTHORS
>   A third_party/polymer/platform/CONTRIBUTING.md
>   A third_party/polymer/polymer/CONTRIBUTING.md
>   D third_party/polymer/polymer.js
>   A third_party/polymer/polymer/polymer.js.map
>   A third_party/polymer/polymer/.bower.json
>   A third_party/polymer/polymer/bower.json
>   M third_party/polymer/README.chromium
>   A third_party/polymer/platform/bower.json
>   D third_party/polymer/README.md
>   D third_party/polymer/build.log
>   A third_party/polymer/polymer/AUTHORS
>   A third_party/polymer/platform/README.md
>   M third_party/polymer/OWNERS
>   A + third_party/polymer/polymer/PATENTS
>   A third_party/polymer/polymer/polymer.js
>   A + third_party/polymer/platform/PATENTS
>   A + third_party/polymer/platform/LICENSE
>   A + third_party/polymer/polymer/build.log
>   A third_party/polymer/platform/build.log
>   A + third_party/polymer/polymer/LICENSE
>   A third_party/polymer/.bowerrc
>   A third_party/polymer/platform/platform.js
>
>
>

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

[bshe, 2013-12-16 18:25:26]

-bryeung

+kevers, rsadam and myself

thanks for doing this. Would you mind to update this doc to include instructions
about how to uprev polymer through bower? the doc is here:
https://docs.google.com/a/google.com/document/d/1SGxfTL2wjzQqmavGCN3T6g-RwBqg...

lgtm

https://codereview.chromium.org/98583008/diff/20001/third_party/polymer/OWNERS
File third_party/polymer/OWNERS (right):

https://codereview.chromium.org/98583008/diff/20001/third_party/polymer/OWNER...
third_party/polymer/OWNERS:3: raymes@chromium.org
do you mind to replace bryeung with kevers@chromium.org ? bryeung didn't work on
this project anymore.

Thanks!

[bshe, 2013-12-16 18:26:29]

really + kevers and rsadam :p

[raymes, 2013-12-16 23:15:59]

bryeung: I removed you from the OWNERS file.

On Tue, Dec 17, 2013 at 5:26 AM,  <bshe@chromium.org> wrote:
> really + kevers and rsadam :p
>
> https://codereview.chromium.org/98583008/

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

[raymes, 2013-12-16 23:17:34]

Thanks!

On Tue, Dec 17, 2013 at 5:25 AM,  <bshe@chromium.org> wrote:
> -bryeung
>
> +kevers, rsadam and myself
>
> thanks for doing this. Would you mind to update this doc to include
> instructions
> about how to uprev polymer through bower? the doc is here:
>
https://docs.google.com/a/google.com/document/d/1SGxfTL2wjzQqmavGCN3T6g-RwBqg...
>

Done.

> lgtm
>
>
>
https://codereview.chromium.org/98583008/diff/20001/third_party/polymer/OWNERS
> File third_party/polymer/OWNERS (right):
>
>
https://codereview.chromium.org/98583008/diff/20001/third_party/polymer/OWNER...
> third_party/polymer/OWNERS:3: raymes@chromium.org
> do you mind to replace bryeung with kevers@chromium.org ? bryeung didn't
> work on this project anymore.
>
> Thanks!

Done.

>
> https://codereview.chromium.org/98583008/

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

[bryeung, 2013-12-16 23:24:45]

Thanks!

Bryan
On Dec 16, 2013 6:15 PM, "Raymes Khoury" <raymes@chromium.org> wrote:

> bryeung: I removed you from the OWNERS file.
>
> On Tue, Dec 17, 2013 at 5:26 AM,  <bshe@chromium.org> wrote:
> > really + kevers and rsadam :p
> >
> > https://codereview.chromium.org/98583008/
>

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

[commit-bot: I haz the power, 2013-12-16 23:24:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/98583008/40001

[commit-bot: I haz the power, 2013-12-17 06:49:23]

List of reviewers changed. bryeung@chromium.org did a drive-by without LGTM'ing!

[raymes, 2013-12-17 23:38:38]

Committed patchset #5 manually as r241411 (presubmit successful).