PDF: Small cleanup of Polymer usage

PDF: Small cleanup of Polymer usage

* Replace direct imports of polymer.html with
  chrome://resources/html/polymer.html. This means the manual import of
  polymer_config.js is no longer necessary.
* Avoid importing deprecated iron-flex-layout files by changing
  paper-styles import.

BUG=621736
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/0402963c3f9452446b89d3d7807bcf749a519777
Cr-Commit-Position: refs/heads/master@{#401161}

[tsergeant, 2016-06-21 01:49:17]

Description was changed from

==========
PDF: Small cleanup of Polymer usage

* Replace direct imports of polymer.html with
  chrome://resources/html/polymer.html
* Avoid importing deprecated iron-flex-layout files by changing
  paper-styles import.

BUG=621736
==========

to

==========
PDF: Small cleanup of Polymer usage

* Replace direct imports of polymer.html with
  chrome://resources/html/polymer.html
* Avoid importing deprecated iron-flex-layout files by changing
  paper-styles import.

BUG=621736
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-06-21 01:51:50]

tsergeant@chromium.org changed reviewers:
+ raymes@chromium.org

[tsergeant, 2016-06-21 01:51:50]

PTAL!

[raymes, 2016-06-22 00:07:59]

Thanks Tim

lgtm with one question/request

https://codereview.chromium.org/2084813002/diff/1/chrome/browser/resources/pd...
File chrome/browser/resources/pdf/elements/viewer-bookmark/viewer-bookmark.html
(right):

https://codereview.chromium.org/2084813002/diff/1/chrome/browser/resources/pd...
chrome/browser/resources/pdf/elements/viewer-bookmark/viewer-bookmark.html:4:
<link rel="import" href="chrome://resources/cr_elements/icons.html">
nit: should we order these?

https://codereview.chromium.org/2084813002/diff/1/chrome/browser/resources/pd...
File chrome/browser/resources/pdf/index.html (left):

https://codereview.chromium.org/2084813002/diff/1/chrome/browser/resources/pd...
chrome/browser/resources/pdf/index.html:7: <script
src="chrome://resources/js/polymer_config.js"></script>
I guess this isn't needed anymore? Could you add a comment about this to the CL
description?

[tsergeant, 2016-06-22 00:31:23]

Description was changed from

==========
PDF: Small cleanup of Polymer usage

* Replace direct imports of polymer.html with
  chrome://resources/html/polymer.html
* Avoid importing deprecated iron-flex-layout files by changing
  paper-styles import.

BUG=621736
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
PDF: Small cleanup of Polymer usage

* Replace direct imports of polymer.html with
  chrome://resources/html/polymer.html. This means the manual import of
  polymer_config.js is no longer necessary.
* Avoid importing deprecated iron-flex-layout files by changing
  paper-styles import.

BUG=621736
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-06-22 00:32:39]

https://codereview.chromium.org/2084813002/diff/1/chrome/browser/resources/pd...
File chrome/browser/resources/pdf/elements/viewer-bookmark/viewer-bookmark.html
(right):

https://codereview.chromium.org/2084813002/diff/1/chrome/browser/resources/pd...
chrome/browser/resources/pdf/elements/viewer-bookmark/viewer-bookmark.html:4:
<link rel="import" href="chrome://resources/cr_elements/icons.html">
On 2016/06/22 00:07:58, raymes wrote:
> nit: should we order these?

Done.

https://codereview.chromium.org/2084813002/diff/1/chrome/browser/resources/pd...
File chrome/browser/resources/pdf/index.html (left):

https://codereview.chromium.org/2084813002/diff/1/chrome/browser/resources/pd...
chrome/browser/resources/pdf/index.html:7: <script
src="chrome://resources/js/polymer_config.js"></script>
On 2016/06/22 00:07:58, raymes wrote:
> I guess this isn't needed anymore? Could you add a comment about this to the
CL
> description?

Yup, it's included by the new polymer.html automatically. Added to the CL
description.

[tsergeant, 2016-06-22 00:32:43]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2016-06-22 00:32:44]

The patchset sent to the CQ was uploaded after l-g-t-m from raymes@chromium.org
Link to the patchset: https://codereview.chromium.org/2084813002/#ps20001
(title: "Reorder imports")

[commit-bot: I haz the power, 2016-06-22 00:33:26]

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

[commit-bot: I haz the power, 2016-06-22 02:05:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-22 02:06:57]

Description was changed from

==========
PDF: Small cleanup of Polymer usage

* Replace direct imports of polymer.html with
  chrome://resources/html/polymer.html. This means the manual import of
  polymer_config.js is no longer necessary.
* Avoid importing deprecated iron-flex-layout files by changing
  paper-styles import.

BUG=621736
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
PDF: Small cleanup of Polymer usage

* Replace direct imports of polymer.html with
  chrome://resources/html/polymer.html. This means the manual import of
  polymer_config.js is no longer necessary.
* Avoid importing deprecated iron-flex-layout files by changing
  paper-styles import.

BUG=621736
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/0402963c3f9452446b89d3d7807bcf749a519777
Cr-Commit-Position: refs/heads/master@{#401161}
==========

[commit-bot: I haz the power, 2016-06-22 02:06:58]

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