Remove unneeded files from third_party/polymer.

Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Updated extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updated create_components_summary.py to use components/ folder
    instead of components-chromium/ folder (which no longer has
    .bower.json files).

Committed: https://crrev.com/68cd128e045e99d3f4d454214ed2b0c2b289858a
Cr-Commit-Position: refs/heads/master@{#384940}

[michaelpg, 2016-03-29 08:38:06]

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

[michaelpg, 2016-03-29 08:38:08]

https://codereview.chromium.org/1834313003/diff/1/third_party/polymer/v1_0/.g...
File third_party/polymer/v1_0/.gitignore (right):

https://codereview.chromium.org/1834313003/diff/1/third_party/polymer/v1_0/.g...
third_party/polymer/v1_0/.gitignore:4: CONTRIBUTING.md
+hero.svg

[Dan Beam, 2016-03-29 17:50:45]

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

[Dan Beam, 2016-03-29 17:50:47]

wait, the **bower.json files are actually kinda useful because it tells you
which element a repo is.

[Dan Beam, 2016-03-29 17:51:06]

which version an element repo is at**

[dpapad, 2016-03-29 17:52:38]

On 2016/03/29 at 17:50:47, dbeam wrote:
> wait, the **bower.json files are actually kinda useful because it tells you
which element a repo is.

@dbeam: See my answer (and related discussion) at
https://groups.google.com/a/chromium.org/d/msg/chromium-polymer/SRu4qlZAZ9w/B....
Let's keep the discussion there until we reach consensus, then we get back to
this CL?

[dpapad, 2016-03-29 17:59:02]

Description was changed from

==========
Remove unneeded files from third_party/polymer
==========

to

==========
Remove unneeded files from third_party/polymer, WIP
==========

[dpapad, 2016-03-29 18:53:09]

Description was changed from

==========
Remove unneeded files from third_party/polymer, WIP
==========

to

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Update extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updating create_components_summary.txt to use components/ folder
    instead of components_chromium/ folder (which no longer has
    .bower.json files).
==========

[dpapad, 2016-03-29 18:55:54]

Description was changed from

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Update extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updating create_components_summary.txt to use components/ folder
    instead of components_chromium/ folder (which no longer has
    .bower.json files).
==========

to

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Update extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updating create_components_summary.txt to use components/ folder
    instead of components-chromium/ folder (which no longer has
    .bower.json files).
==========

[dpapad, 2016-03-29 18:57:34]

Description was changed from

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Update extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updating create_components_summary.txt to use components/ folder
    instead of components-chromium/ folder (which no longer has
    .bower.json files).
==========

to

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Updated extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updated create_components_summary.txt to use components/ folder
    instead of components-chromium/ folder (which no longer has
    .bower.json files).
==========

[dpapad, 2016-03-29 18:57:50]

Description was changed from

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Updated extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updated create_components_summary.txt to use components/ folder
    instead of components-chromium/ folder (which no longer has
    .bower.json files).
==========

to

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Updated extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updated create_components_summary.py to use components/ folder
    instead of components-chromium/ folder (which no longer has
    .bower.json files).
==========

[dpapad, 2016-03-29 23:21:47]

@dbeam: The conversation at
https://groups.google.com/a/chromium.org/d/msg/chromium-polymer/SRu4qlZAZ9w/B...
is pretty quite. It seems that there are no major objections, or people don't
care much either way.

@michaelpg: Per your suggestion to check with licencsing team, did you have in
mind open-source-third-party-reviews@google.com? I'll do that once there is an
LG.


Ideally, I would prefer to get this in before the next Polymer roll, to avoid
resolving conflicts.

https://codereview.chromium.org/1834313003/diff/1/third_party/polymer/v1_0/.g...
File third_party/polymer/v1_0/.gitignore (right):

https://codereview.chromium.org/1834313003/diff/1/third_party/polymer/v1_0/.g...
third_party/polymer/v1_0/.gitignore:4: CONTRIBUTING.md
On 2016/03/29 at 08:38:08, michaelpg wrote:
> +hero.svg

Done.

[Dan Beam, 2016-03-29 23:30:07]

i would consult a lawyer cat before removing some of these .md files

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
File third_party/polymer/v1_0/extract_inline_scripts.sh (right):

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/extract_inline_scripts.sh:32: -not -name
"metadata.html" | \
maybe all the -nots should be in the rsync_exclude.txt?  up to you

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
File third_party/polymer/v1_0/rsync_exclude.txt (right):

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/rsync_exclude.txt:2: bower.json
do you not want to check in at least bower.json files?

i was thinking diffing these to create better roll CL descriptions for each
element (if bower/npm doesn't do that already)

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/rsync_exclude.txt:3: comipiled_resources*.gyp
misspelled

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/rsync_exclude.txt:8: *.md
I would call out the explicit .md files you want to ignore, as I could see a
license file being an .md possibly in the future

[dpapad, 2016-03-30 00:29:55]

Sent email to the third_party reviewers, you have been cc'ed.

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
File third_party/polymer/v1_0/extract_inline_scripts.sh (right):

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/extract_inline_scripts.sh:32: -not -name
"metadata.html" | \
On 2016/03/29 at 23:30:07, Dan Beam wrote:
> maybe all the -nots should be in the rsync_exclude.txt?  up to you

Done.

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
File third_party/polymer/v1_0/rsync_exclude.txt (right):

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/rsync_exclude.txt:2: bower.json
On 2016/03/29 at 23:30:07, Dan Beam wrote:
> do you not want to check in at least bower.json files?
> 
> i was thinking diffing these to create better roll CL descriptions for each
element (if bower/npm doesn't do that already)

Isn't this addressed by the generated
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/polyme...
The diff of this file can be used to generate a good CL description of a roll
CL.
Do we need both components_summary.txt and individual bower.json files?

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/rsync_exclude.txt:3: comipiled_resources*.gyp
On 2016/03/29 at 23:30:07, Dan Beam wrote:
> misspelled

Fixed.

[dpapad, 2016-04-01 20:10:57]

The third_party review team has given a "go ahead" signal. PTAL

[Dan Beam, 2016-04-01 20:38:50]

lgtm

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
File third_party/polymer/v1_0/rsync_exclude.txt (right):

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/rsync_exclude.txt:2: bower.json
On 2016/03/30 00:29:55, dpapad wrote:
> On 2016/03/29 at 23:30:07, Dan Beam wrote:
> > do you not want to check in at least bower.json files?
> > 
> > i was thinking diffing these to create better roll CL descriptions for each
> element (if bower/npm doesn't do that already)
> 
> Isn't this addressed by the generated
>
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/polyme...
> The diff of this file can be used to generate a good CL description of a roll
> CL.
> Do we need both components_summary.txt and individual bower.json files?

ah, true dat

[michaelpg, 2016-04-01 20:49:02]

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
File third_party/polymer/v1_0/rsync_exclude.txt (right):

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/rsync_exclude.txt:2: bower.json
On 2016/03/30 00:29:55, dpapad wrote:
> On 2016/03/29 at 23:30:07, Dan Beam wrote:
> > do you not want to check in at least bower.json files?
> > 
> > i was thinking diffing these to create better roll CL descriptions for each
> element (if bower/npm doesn't do that already)
> 
> Isn't this addressed by the generated
>
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/polyme...
> The diff of this file can be used to generate a good CL description of a roll
> CL.
> Do we need both components_summary.txt and individual bower.json files?

bower.json files list dependencies which I know is useful for me when updating
Polymer elements selectively.

[dpapad, 2016-04-01 22:01:05]

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
File third_party/polymer/v1_0/rsync_exclude.txt (right):

https://codereview.chromium.org/1834313003/diff/40001/third_party/polymer/v1_...
third_party/polymer/v1_0/rsync_exclude.txt:2: bower.json
On 2016/04/01 at 20:49:01, michaelpg wrote:
> On 2016/03/30 00:29:55, dpapad wrote:
> > On 2016/03/29 at 23:30:07, Dan Beam wrote:
> > > do you not want to check in at least bower.json files?
> > > 
> > > i was thinking diffing these to create better roll CL descriptions for
each
> > element (if bower/npm doesn't do that already)
> > 
> > Isn't this addressed by the generated
> >
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/polyme...
> > The diff of this file can be used to generate a good CL description of a
roll
> > CL.
> > Do we need both components_summary.txt and individual bower.json files?
> 
> bower.json files list dependencies which I know is useful for me when updating
Polymer elements selectively.

I restored the bower.json files.

[michaelpg, 2016-04-02 02:15:53]

excellent, lgtm

https://codereview.chromium.org/1834313003/diff/80001/third_party/polymer/v1_...
File third_party/polymer/v1_0/extract_inline_scripts.sh (right):

https://codereview.chromium.org/1834313003/diff/80001/third_party/polymer/v1_...
third_party/polymer/v1_0/extract_inline_scripts.sh:24: rsync -c --delete -r -v
--exclude-from="rsync_exclude.txt" \
also nice

https://codereview.chromium.org/1834313003/diff/80001/third_party/polymer/v1_...
File third_party/polymer/v1_0/generate_gyp.sh (left):

https://codereview.chromium.org/1834313003/diff/80001/third_party/polymer/v1_...
third_party/polymer/v1_0/generate_gyp.sh:9: htmls=$(\ls $dir/*.html 2>/dev/null
| egrep -v '(demo|index)\.html')
nice

[dpapad, 2016-04-04 17:01:20]

The CQ bit was checked by dpapad@chromium.org

[dpapad, 2016-04-04 17:01:22]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/1834313003/#ps80001
(title: "Restore bower.json files.")

[commit-bot: I haz the power, 2016-04-04 17:01:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1834313003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1834313003/80001

[commit-bot: I haz the power, 2016-04-04 17:48:51]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-04-04 17:50:07]

Description was changed from

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Updated extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updated create_components_summary.py to use components/ folder
    instead of components-chromium/ folder (which no longer has
    .bower.json files).
==========

to

==========
Remove unneeded files from third_party/polymer.

Specifically:
 1) Removed unneeded files.
 2) Updated extract_inline_scripts.sh to not rsync those files again
    in the future.
 3) Updated create_components_summary.py to use components/ folder
    instead of components-chromium/ folder (which no longer has
    .bower.json files).

Committed: https://crrev.com/68cd128e045e99d3f4d454214ed2b0c2b289858a
Cr-Commit-Position: refs/heads/master@{#384940}
==========

[commit-bot: I haz the power, 2016-04-04 17:50:09]

Patchset 5 (id:??) landed as
https://crrev.com/68cd128e045e99d3f4d454214ed2b0c2b289858a
Cr-Commit-Position: refs/heads/master@{#384940}