Roll Polymer

Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
NOTRY=true  # see patchset 2's try jobs
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/134cd0d0669376815daebd5a170184ffef3b8ebd
Cr-Commit-Position: refs/heads/master@{#400318}

[Dan Beam, 2016-06-16 18:23:32]

Description was changed from

==========
Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
==========

to

==========
Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[michaelpg, 2016-06-16 19:35:22]

lgtm

[Dan Beam, 2016-06-16 23:01:54]

https://codereview.chromium.org/2074813002/diff/20001/third_party/polymer/v1_...
File
third_party/polymer/v1_0/components-chromium/iron-collapse/iron-collapse-extracted.js
(right):

https://codereview.chromium.org/2074813002/diff/20001/third_party/polymer/v1_...
third_party/polymer/v1_0/components-chromium/iron-collapse/iron-collapse-extracted.js:99:
* @param {string} size The new value for `maxWidth`/`maxHeight` as css property
value, usually `auto` or `0px`.
this was !String, which was a compiler error

i sent this PR https://github.com/PolymerElements/iron-collapse/pull/56

and fixted it manually here (assuming it'll be in the next polymer roll)

[michaelpg, 2016-06-16 23:03:25]

https://codereview.chromium.org/2074813002/diff/20001/third_party/polymer/v1_...
File
third_party/polymer/v1_0/components-chromium/iron-collapse/iron-collapse-extracted.js
(right):

https://codereview.chromium.org/2074813002/diff/20001/third_party/polymer/v1_...
third_party/polymer/v1_0/components-chromium/iron-collapse/iron-collapse-extracted.js:99:
* @param {string} size The new value for `maxWidth`/`maxHeight` as css property
value, usually `auto` or `0px`.
On 2016/06/16 23:01:54, Dan Beam wrote:
> this was !String, which was a compiler error
> 
> i sent this PR https://github.com/PolymerElements/iron-collapse/pull/56
> 
> and fixted it manually here (assuming it'll be in the next polymer roll)

add this to chromium.patch

[Dan Beam, 2016-06-17 00:41:51]

Description was changed from

==========
Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
NOTRY=true  # see patchset 2's try jobs
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-06-17 00:42:10]

https://codereview.chromium.org/2074813002/diff/20001/third_party/polymer/v1_...
File
third_party/polymer/v1_0/components-chromium/iron-collapse/iron-collapse-extracted.js
(right):

https://codereview.chromium.org/2074813002/diff/20001/third_party/polymer/v1_...
third_party/polymer/v1_0/components-chromium/iron-collapse/iron-collapse-extracted.js:99:
* @param {string} size The new value for `maxWidth`/`maxHeight` as css property
value, usually `auto` or `0px`.
On 2016/06/16 23:03:25, michaelpg wrote:
> On 2016/06/16 23:01:54, Dan Beam wrote:
> > this was !String, which was a compiler error
> > 
> > i sent this PR https://github.com/PolymerElements/iron-collapse/pull/56
> > 
> > and fixted it manually here (assuming it'll be in the next polymer roll)
> 
> add this to chromium.patch

Done.

[Dan Beam, 2016-06-17 00:42:18]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-06-17 00:42:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
michaelpg@chromium.org
Link to the patchset: https://codereview.chromium.org/2074813002/#ps40001
(title: "chromium.patch")

[commit-bot: I haz the power, 2016-06-17 00:43:04]

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

[commit-bot: I haz the power, 2016-06-17 00:44:35]

Description was changed from

==========
Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
NOTRY=true  # see patchset 2's try jobs
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
NOTRY=true  # see patchset 2's try jobs
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-06-17 00:44:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-17 00:46:09]

Description was changed from

==========
Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
NOTRY=true  # see patchset 2's try jobs
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Roll Polymer

See components_summary.txt for versioning details

R=michaelpg@chromium.org
BUG=619797
NOPRESUBMIT=true  # crisper.js
NOTRY=true  # see patchset 2's try jobs
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/134cd0d0669376815daebd5a170184ffef3b8ebd
Cr-Commit-Position: refs/heads/master@{#400318}
==========

[commit-bot: I haz the power, 2016-06-17 00:46:10]

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