Misc. cleanup

ui/webui/resources/css/tree.css

 /* Copyright (c) 2012 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file. */
 
 tree {
   display: block;
   outline: none;
   overflow: auto;
 }
 
@@ skipping 88 matching lines @@
   -webkit-padding-start: 20px;
   background-position: 0 50%;
   background-repeat: no-repeat;
   white-space: pre;
 }
 
 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(
       url(../../../resources/default_100_percent/common/folder_closed_rtl.png) 1x,
       url(../../../resources/default_200_percent/common/folder_closed_rtl.png) 2x);
 }
 
 html[dir=rtl] .tree-item[expanded] > .tree-row > .tree-label {
   background-image: -webkit-image-set(
      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;
   background-image: none !important;
 }
 
 tree[icon-visibility=parent] .tree-label,
 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: ' ';
   white-space: pre;
 }
 
 .tree-rename > .tree-row > .tree-label {
   -webkit-user-modify: read-write-plaintext-only;
   -webkit-user-select: auto;
   background: white;
@@ skipping 10 matching lines @@
 <if expr="not is_macosx and not is_ios">
   outline: none;
 </if>
   padding: 1px 7px 1px 1px;
 }
 
 html[dir=rtl] .tree-item[editing] input {
   margin: -2px -3px -2px -8px;
   padding: 1px 1px 1px 7px;
 }