MD Settings: Fix internet focus and styling

ui/webui/resources/cr_elements/shared_style_css.html

Index: ui/webui/resources/cr_elements/shared_style_css.html
diff --git a/ui/webui/resources/cr_elements/shared_style_css.html b/ui/webui/resources/cr_elements/shared_style_css.html
index bdbfcdcb8386bed851a09ec482fc43346d501d35..e558762c28edd8f5e015ccb92d513e9d7f97f910 100644
--- a/ui/webui/resources/cr_elements/shared_style_css.html
+++ b/ui/webui/resources/cr_elements/shared_style_css.html
@@ -6,6 +6,14 @@
 <dom-module id="cr-shared-style">
   <template>
     <style>
+      button[is='paper-icon-button-light'].subpage-arrow {
+        background-image: url(../images/arrow_right.svg);
+      }
+
+      button[is='paper-icon-button-light'].icon-external {
+        background-image: url(../images/open_in_new.svg);
+      }

[dschuyler, 2016/12/10 01:09:18]

Without [actionable], I believe the arrows will appear enabled when they are
un-clickable. Such as when prefs.profile.password_manager_enmabled.value is
false. There are a few places that may be impacted by this change. I think they
can be found by searching in the settings folder for actionable$=" (i.e. A
binding that conditionally sets actionable).

Maybe those could be changed to a hidden$=" on the button itself?
(I'd also welcome a different disabled behavior for rows that are un-clickable,
though I'd recommend a separate CL for that bit).

Those could be changed in this CL or as a prequel-CL to this CL.

[dschuyler, 2016/12/10 01:14:01]

In case I'm giving the wrong impression when I said, "changed to a hidden". We'd
want to do the conditional hidden$= in addition to the actionable$= (i.e. not
change/replace that with hidden); so that the mouse pointer does the correct
thing as well as hiding/showing the icon.

+
       /* Chrome spinners should be blue. */
       paper-spinner {
         --paper-spinner-layer-1-color: var(--google-blue-500);