MD Settings: consolidate .no-outline css rules.

chrome/browser/resources/settings/settings_shared_css.html

Index: chrome/browser/resources/settings/settings_shared_css.html
diff --git a/chrome/browser/resources/settings/settings_shared_css.html b/chrome/browser/resources/settings/settings_shared_css.html
index 4aa064296b0e9e7779b26676d9ea1eecea94f176..4f3e19300161d154b027f6c06c1972eba9d5baf4 100644
--- a/chrome/browser/resources/settings/settings_shared_css.html
+++ b/chrome/browser/resources/settings/settings_shared_css.html
@@ -303,10 +303,9 @@
         flex: auto;
       }
 
-      [scrollable] .no-outline {
+      .no-outline, [scrollable] .no-outline {

[Dan Beam, 2017/04/03 19:51:34]

somehow none of this CSS seems to be checked by our tooling, because the style
guide asks for selectors on multiple lines

[scottchen, 2017/04/03 23:32:44]

Done.

[dpapad, 2017/04/03 23:45:50]

Maybe the presubmit checks don't find CSS inside HTML files? Or at least not
inside Polymer <dom-module> tags?

[Dan Beam, 2017/04/04 19:54:55]

i fixted it https://codereview.chromium.org/2792193003/

         background: none;
         outline: none;
-        cursor: initial;

[scottchen, 2017/04/03 18:22:23]

.no-outline class should just be about outline. Inheriting elements who need to
customize cursor can do it themselves.

[Dan Beam, 2017/04/03 19:51:34]

and also this was out of order :(

[scottchen, 2017/04/03 23:32:44]

Acknowledged.

       }
 
       /* The secondary-action wraps a clickable sub-area of a .settings-box.