Misc. cleanup

ui/webui/resources/css/tree.css

Index: ui/webui/resources/css/tree.css
diff --git a/ui/webui/resources/css/tree.css b/ui/webui/resources/css/tree.css
index 103f1aaae2d0672d578eeab2c1b01e64c38ae638..2681f5e70f99f65564cf422d361a4679c7a1b0cb 100644
--- a/ui/webui/resources/css/tree.css
+++ b/ui/webui/resources/css/tree.css
@@ -106,6 +106,20 @@ html[dir=rtl] .tree-label {
   background-position: 100% 50%;
 }
 
+.tree-label,
+.tree-row[may-have-children] > .tree-label {
+  background-image: -webkit-image-set(
+      url(../../../resources/default_100_percent/common/folder_closed.png) 1x,
+      url(../../../resources/default_200_percent/common/folder_closed.png) 2x);
+}
+
+.tree-item[expanded] > .tree-row > .tree-label {
+  background-image: -webkit-image-set(
+      url(../../../views/resources/default_100_percent/common/folder_open.png) 1x,
+      url(../../../views/resources/default_200_percent/common/folder_open.png) 2x);
+}
+
+<if expr="not is_macosx and not is_ios">
 html[dir=rtl] .tree-label,
 html[dir=rtl] .tree-row[may-have-children] > .tree-label {
   background-image: -webkit-image-set(
@@ -118,6 +132,7 @@ html[dir=rtl] .tree-item[expanded] > .tree-row > .tree-label {
      url(../../../views/resources/default_100_percent/common/folder_open_rtl.png) 1x,
      url(../../../views/resources/default_200_percent/common/folder_open_rtl.png) 2x);
 }
+</if>
 
 tree[icon-visibility=hidden] .tree-label {
   -webkit-padding-start: 0;
@@ -129,19 +144,6 @@ tree[icon-visibility=parent] .tree-row[has-children=false] > .tree-label {
   background-image: none;
 }
 
-.tree-label,
-.tree-row[may-have-children] > .tree-label {
-  background-image: -webkit-image-set(
-      url(../../../resources/default_100_percent/common/folder_closed.png) 1x,
-      url(../../../resources/default_200_percent/common/folder_closed.png) 2x);
-}
-
-.tree-item[expanded] > .tree-row > .tree-label {
-  background-image: -webkit-image-set(
-      url(../../../views/resources/default_100_percent/common/folder_open.png) 1x,
-      url(../../../views/resources/default_200_percent/common/folder_open.png) 2x);
-}

[Dan Beam, 2016/08/30 01:27:29]

why are you moving these?  changing the order of the statements may have a
functional affect (i.e. if there are 2 statements with similar specificity, the
last one declared wins)

[Peter Kasting, 2016/08/30 01:48:47]

I'm trying to put them next to the similar RTL code in preparation for combining
and replacing all those rules.

(And I don't think the two selector blocks above have similar specificity to
these, but if they did, we'd want them to win; so we'd want them declared last.)

-
 /* We need to ensure that even empty labels take up space */
 .tree-label:empty::after {
   content: ' ';