Tidy up in Polymer directory.

Tidy up in Polymer directory.

* Updated README.chromium.
* Suppressed non-zero exit status of some commands in reproduce.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit complains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true

Committed: https://crrev.com/8071389f8e027b45339bdfd678d862799b8d6691
Cr-Commit-Position: refs/heads/master@{#371361}

[dzhioev (left Google), 2016-01-23 02:07:20]

Description was changed from

==========
Tidy up in Polymer directory.

* Updated README.chromium.
* Suppresed non-zero exit status of some commands in reproduse.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit coplains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true
==========

to

==========
Tidy up in Polymer directory.

* Updated README.chromium.
* Suppresed non-zero exit status of some commands in reproduse.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit coplains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true
==========

[dzhioev (left Google), 2016-01-23 02:07:20]

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

[dzhioev (left Google), 2016-01-23 02:07:49]

Michael, PTAL.

[michaelpg, 2016-01-23 04:28:15]

nice!

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/README....
File third_party/polymer/README.chromium (right):

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/README....
third_party/polymer/README.chromium:17:
https://github.com/PolymerElements/<component-name>
+PolymerLabs for polymer-externs

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/v1_0/cr...
File third_party/polymer/v1_0/create_components_summary.py (right):

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/v1_0/cr...
third_party/polymer/v1_0/create_components_summary.py:23: if not
os.path.exists(bower_path):
isfile

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/v1_0/cr...
third_party/polymer/v1_0/create_components_summary.py:25: info =
json.load(open(bower_path))
nit: with open(...

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/v1_0/cr...
third_party/polymer/v1_0/create_components_summary.py:27: tree =
'https%s/tree/%s' % (repository[3:][:-4], info['_resolution']['tag'])
[3:-4]

[dzhioev (left Google), 2016-01-25 23:31:35]

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/README....
File third_party/polymer/README.chromium (right):

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/README....
third_party/polymer/README.chromium:17:
https://github.com/PolymerElements/<component-name>
On 2016/01/23 04:28:15, michaelpg wrote:
> +PolymerLabs for polymer-externs

Done.

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/v1_0/cr...
File third_party/polymer/v1_0/create_components_summary.py (right):

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/v1_0/cr...
third_party/polymer/v1_0/create_components_summary.py:23: if not
os.path.exists(bower_path):
On 2016/01/23 04:28:15, michaelpg wrote:
> isfile

Done.

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/v1_0/cr...
third_party/polymer/v1_0/create_components_summary.py:25: info =
json.load(open(bower_path))
On 2016/01/23 04:28:15, michaelpg wrote:
> nit: with open(...

Done.

https://codereview.chromium.org/1623693002/diff/1/third_party/polymer/v1_0/cr...
third_party/polymer/v1_0/create_components_summary.py:27: tree =
'https%s/tree/%s' % (repository[3:][:-4], info['_resolution']['tag'])
On 2016/01/23 04:28:15, michaelpg wrote:
> [3:-4]

Done.

[michaelpg, 2016-01-25 23:37:12]

lgtm!

[dzhioev (left Google), 2016-01-25 23:37:20]

The CQ bit was checked by dzhioev@chromium.org

[michaelpg, 2016-01-25 23:37:48]

Description was changed from

==========
Tidy up in Polymer directory.

* Updated README.chromium.
* Suppresed non-zero exit status of some commands in reproduse.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit coplains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true
==========

to

==========
Tidy up in Polymer directory.

* Updated README.chromium.
* Suppressed non-zero exit status of some commands in reproduce.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit complains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-01-25 23:39:13]

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

[commit-bot: I haz the power, 2016-01-26 00:11:14]

Description was changed from

==========
Tidy up in Polymer directory.

* Updated README.chromium.
* Suppressed non-zero exit status of some commands in reproduce.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit complains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true
==========

to

==========
Tidy up in Polymer directory.

* Updated README.chromium.
* Suppressed non-zero exit status of some commands in reproduce.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit complains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-01-26 00:11:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-26 00:12:10]

Description was changed from

==========
Tidy up in Polymer directory.

* Updated README.chromium.
* Suppressed non-zero exit status of some commands in reproduce.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit complains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true
==========

to

==========
Tidy up in Polymer directory.

* Updated README.chromium.
* Suppressed non-zero exit status of some commands in reproduce.sh (this caused
  premature exit).
* Added script which generates a short summary of presenting components.

Presubmit complains about removed copy of README.chromium, that is why
NOPRESUBMIT is set.

BUG=none
NOPRESUBMIT=true

Committed: https://crrev.com/8071389f8e027b45339bdfd678d862799b8d6691
Cr-Commit-Position: refs/heads/master@{#371361}
==========

[commit-bot: I haz the power, 2016-01-26 00:12:11]

Patchset 2 (id:??) landed as
https://crrev.com/8071389f8e027b45339bdfd678d862799b8d6691
Cr-Commit-Position: refs/heads/master@{#371361}

[Dan Beam, 2016-02-09 02:06:02]

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

[Dan Beam, 2016-02-09 02:06:02]

https://codereview.chromium.org/1623693002/diff/20001/third_party/polymer/v1_...
File third_party/polymer/v1_0/create_components_summary.py (right):

https://codereview.chromium.org/1623693002/diff/20001/third_party/polymer/v1_...
third_party/polymer/v1_0/create_components_summary.py:33: 'tag':
info['_resolution']['tag'],
probably doesn't matter much, but when trying to test branches from the Polymer
folks (i.e. https://github.com/PolymerElements/iron-list/tree/focus-patch), at
least 'tag' isn't set

maybe we should be using .get('tag', '')?

this is an uncommon use case (so far) though