MD WebUI: Remove <style no-process> from Polymer

MD WebUI: Remove <style no-process> from Polymer

<style no-process> was added in crrev.com/391871 as a temporary measure
to allow native custom properties to be used without being overridden by
Polymer's shim.

All Polymer pages now use native custom properties, so this bypass is no
longer necessary. This CL removes it from our Polymer checkout and the
MD History/Downloads pages, with no visible changes as a result.

BUG=608098
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/b8534fb568a8450d5a52589a73343735709d17cd
Cr-Commit-Position: refs/heads/master@{#421119}

[tsergeant, 2016-09-26 03:04:56]

Description was changed from

==========
MD WebUI: Remove <style no-process> from Polymer

<style no-process> was added in crrev.com/391871 as a temporary
measure to allow native custom properties to be used without being
overridden by Polymer's shim.

We are now using native custom properties on all Polymer pages,
so this bypass is no longer necessary. This CL removes it from
our Polymer checkout and the MD History/Downloads pages, with
no visible changes as a result.

BUG=608098
==========

to

==========
MD WebUI: Remove <style no-process> from Polymer

<style no-process> was added in crrev.com/391871 as a temporary
measure to allow native custom properties to be used without being
overridden by Polymer's shim.

We are now using native custom properties on all Polymer pages,
so this bypass is no longer necessary. This CL removes it from
our Polymer checkout and the MD History/Downloads pages, with
no visible changes as a result.

BUG=608098
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-09-26 03:07:24]

Description was changed from

==========
MD WebUI: Remove <style no-process> from Polymer

<style no-process> was added in crrev.com/391871 as a temporary
measure to allow native custom properties to be used without being
overridden by Polymer's shim.

We are now using native custom properties on all Polymer pages,
so this bypass is no longer necessary. This CL removes it from
our Polymer checkout and the MD History/Downloads pages, with
no visible changes as a result.

BUG=608098
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD WebUI: Remove <style no-process> from Polymer

<style no-process> was added in crrev.com/391871 as a temporary measure
to allow native custom properties to be used without being overridden by
Polymer's shim.

All Polymer pages now use native custom properties, so this bypass is no
longer necessary. This CL removes it from our Polymer checkout and the
MD History/Downloads pages, with no visible changes as a result.

BUG=608098
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-09-26 03:07:32]

The CQ bit was checked by tsergeant@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-26 03:07:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tsergeant, 2016-09-26 03:11:13]

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

[tsergeant, 2016-09-26 03:16:42]

PTAL!

[commit-bot: I haz the power, 2016-09-26 04:37:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-26 04:37:45]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Dan Beam, 2016-09-26 07:08:10]

generally lg (yay, kill the forks with fire) but maybe it's time to inline .css
files from downloads into their respective .html files like everywhere else?

https://codereview.chromium.org/2370653002/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_downloads/manager.html (right):

https://codereview.chromium.org/2370653002/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_downloads/manager.html:33: <link rel="import"
type="css" href="chrome://downloads/manager.css">
if the rule you're removing is duplicated in manager.css and we're pulling it in
via <link rel=import type=css>, wont it flicker?  aren't these types of links
polyfilled by JS?

[tsergeant, 2016-09-27 00:25:37]

Bug filed for css migration: https://crbug.com/650504

https://codereview.chromium.org/2370653002/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_downloads/manager.html (right):

https://codereview.chromium.org/2370653002/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_downloads/manager.html:33: <link rel="import"
type="css" href="chrome://downloads/manager.css">
On 2016/09/26 07:08:10, Dan Beam wrote:
> if the rule you're removing is duplicated in manager.css and we're pulling it
in
> via <link rel=import type=css>, wont it flicker?  aren't these types of links
> polyfilled by JS?

I haven't seen any noticeable flicker in the unvulcanized build.

In the vulcanized build, these CSS files are inlined so there's no difference.

[Dan Beam, 2016-09-27 00:37:13]

yeah, you're right: the vulcanized build doesn't care

lgtm

[tsergeant, 2016-09-27 00:45:37]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2016-09-27 00:46:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 04:40:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-27 04:40:07]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[tsergeant, 2016-09-27 04:50:05]

The CQ bit was checked by tsergeant@chromium.org

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

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 05:51:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 05:55:03]

Description was changed from

==========
MD WebUI: Remove <style no-process> from Polymer

<style no-process> was added in crrev.com/391871 as a temporary measure
to allow native custom properties to be used without being overridden by
Polymer's shim.

All Polymer pages now use native custom properties, so this bypass is no
longer necessary. This CL removes it from our Polymer checkout and the
MD History/Downloads pages, with no visible changes as a result.

BUG=608098
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD WebUI: Remove <style no-process> from Polymer

<style no-process> was added in crrev.com/391871 as a temporary measure
to allow native custom properties to be used without being overridden by
Polymer's shim.

All Polymer pages now use native custom properties, so this bypass is no
longer necessary. This CL removes it from our Polymer checkout and the
MD History/Downloads pages, with no visible changes as a result.

BUG=608098
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/b8534fb568a8450d5a52589a73343735709d17cd
Cr-Commit-Position: refs/heads/master@{#421119}
==========

[commit-bot: I haz the power, 2016-09-27 05:55:05]

Patchset 1 (id:??) landed as
https://crrev.com/b8534fb568a8450d5a52589a73343735709d17cd
Cr-Commit-Position: refs/heads/master@{#421119}