Convert Media Router WebUI <button>s to <paper-button>s.

Convert Media Router WebUI <button>s to <paper-button>s.

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

[apacible, 2015-05-13 01:32:47]

apacible@chromium.org changed reviewers:
+ mfoltz@chromium.org

[apacible, 2015-05-13 01:32:47]

apacible@chromium.org changed reviewers:
+ mfoltz@chromium.org

[apacible, 2015-05-13 01:35:55]

apacible@chromium.org changed reviewers:
+ jhawkins@chromium.org, jlklein@chromium.org, michaelpg@chromium.org

[apacible, 2015-05-13 01:35:55]

PTAL, thanks!

Route Details:
- LTR: https://screenshot.googleplex.com/CLGJ2W1Lhaw.png
- RTL: https://screenshot.googleplex.com/DC3X9r32qzv.png

Issues (1 action):
- LTR: https://screenshot.googleplex.com/czKpzgnZaYT.png
- RTL: https://screenshot.googleplex.com/g6i20xousKg.png

Issues (2 actions):
- LTR: https://screenshot.googleplex.com/cU28vWwU0VG.png
- RTL: https://screenshot.googleplex.com/XLjL5AQUnSk.png

Previous (for reference):
- Route Details: https://screenshot.googleplex.com/bRVJraTeaCq.png
- Issue: https://screenshot.googleplex.com/70qXs1M204m.png

[mark a. foltz, 2015-05-13 01:50:32]

It seems like we are adding some of our own styling to the paper buttons.    Is
that okay?

https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
File chrome/browser/resources/media_router/media_router_common.css (right):

https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
chrome/browser/resources/media_router/media_router_common.css:6:
background-color: rgb(238, 238, 238);
Does paper provide a default color for buttons, or do we have to set it
ourselves?  How are we certain that this color will match other paper buttons in
Chrome?

[Jeremy Klein, 2015-05-13 02:28:15]

lgtm

https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
File chrome/browser/resources/media_router/media_router_common.css (right):

https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
chrome/browser/resources/media_router/media_router_common.css:6:
background-color: rgb(238, 238, 238);
On 2015/05/13 01:50:32, mark a. foltz wrote:
> Does paper provide a default color for buttons, or do we have to set it
> ourselves?  How are we certain that this color will match other paper buttons
in
> Chrome?

When you guys upgrade to 0.8, you'll be able to use these:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/polyme...

[apacible, 2015-05-13 14:31:18]

https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
File chrome/browser/resources/media_router/media_router_common.css (right):

https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
chrome/browser/resources/media_router/media_router_common.css:6:
background-color: rgb(238, 238, 238);
These are current placeholder colors, based on our old design. We'll update
these to match when we get final UX specs/redlines (and switch to 0.8), which
should match the other paper buttons in Chrome.

[mark a. foltz, 2015-05-13 18:08:59]

lgtm

On 2015/05/13 14:31:18, apacible wrote:
>
https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
> File chrome/browser/resources/media_router/media_router_common.css (right):
> 
>
https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
> chrome/browser/resources/media_router/media_router_common.css:6:
> background-color: rgb(238, 238, 238);
> These are current placeholder colors, based on our old design. We'll update
> these to match when we get final UX specs/redlines (and switch to 0.8), which
> should match the other paper buttons in Chrome.

Ok, it would be great to track this - either in a TODO, crbug or task on our
internal spreadsheet.

[apacible, 2015-05-14 16:56:20]

On 2015/05/13 18:08:59, mark a. foltz wrote:
> lgtm
> 
> On 2015/05/13 14:31:18, apacible wrote:
> >
>
https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
> > File chrome/browser/resources/media_router/media_router_common.css (right):
> > 
> >
>
https://codereview.chromium.org/1136353002/diff/1/chrome/browser/resources/me...
> > chrome/browser/resources/media_router/media_router_common.css:6:
> > background-color: rgb(238, 238, 238);
> > These are current placeholder colors, based on our old design. We'll update
> > these to match when we get final UX specs/redlines (and switch to 0.8),
which
> > should match the other paper buttons in Chrome.
> 
> Ok, it would be great to track this - either in a TODO, crbug or task on our
> internal spreadsheet.

Added TODOs and added a task to our spreadsheet.

[apacible, 2015-05-14 17:06:03]

The CQ bit was checked by apacible@chromium.org

[apacible, 2015-05-14 17:06:04]

The patchset sent to the CQ was uploaded after l-g-t-m from mfoltz@chromium.org,
jlklein@chromium.org
Link to the patchset: https://codereview.chromium.org/1136353002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-05-14 17:06:16]

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

[commit-bot: I haz the power, 2015-05-14 18:23:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-14 18:25:05]

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