[webui] get rid of more html[os=] rules

chrome/browser/resources/bookmark_manager/css/bmm.css

Index: chrome/browser/resources/bookmark_manager/css/bmm.css
diff --git a/chrome/browser/resources/bookmark_manager/css/bmm.css b/chrome/browser/resources/bookmark_manager/css/bmm.css
index 890c3e13ea9ad8f502174e49405fdd965332d456..566e789e5c7766b87717e6f2982755f17e4432ae 100644
--- a/chrome/browser/resources/bookmark_manager/css/bmm.css
+++ b/chrome/browser/resources/bookmark_manager/css/bmm.css
@@ -132,20 +132,21 @@ list [editing] .url input {
 }
 
 .tree-item [editing] input {
-  margin: 0;
-  padding: 0;
   -webkit-margin-start: -3px;
   -webkit-padding-end: 7px;
-}
-
-html[os=mac] .tree-label [editing] input {
+  margin: 0;
+  padding: 0;
+<if expr="is_macosx">
   padding-top: 1px;
+</if>
 }
 
-html:not([os=mac]) list .label input,
-html:not([os=mac]) list .url input {
+<if expr="is_macosx">
+list .label input,
+list .url input {
   outline: none;
 }
+</if>
 
 list > [editing] {
   overflow: visible;
@@ -168,18 +169,15 @@ html[dir=rtl] list > .folder > .label {
   background-image: url("../../../../../ui/resources/folder_closed_rtl.png");
 }
 
-html[os=mac] list > .folder > .label {
+<if expr="is_macosx">
+list > .folder > .label,
+.tree-label,
+.tree-row[may-have-children] > .tree-label,
+.tree-item[expanded] > .tree-row > .tree-label {
   background-image: url("../../../../app/theme/bookmark_bar_folder_mac.png");

[Dan Beam, 2012/03/21 03:02:53]

er, wat? seems like this line is duplicated...? also, single quotes if you feel
like it, and there are alpha issues everywhere, :(

[Evan Stade, 2012/03/21 21:51:12]

yea i have no idea. I just copied it. I'm not trying to rewrite or fix anything.

[Dan Beam, 2012/03/21 23:14:51]

I'm saying the exactly same rule is repeated twice according to the diff, so
it'd look like:

  background-image: url("../../../../app/theme/bookmark_bar_folder_mac.png");
  background-image: url("../../../../app/theme/bookmark_bar_folder_mac.png");

does it look like this for you in the code or is this a rietveld (or me) issue?

[Evan Stade, 2012/03/21 23:54:03]

oh, I see what you mean. I will fix it. I thought you were talking about the
selectors, which seem redundant to me.

-}
-
-html[os=mac] .tree-label,
-html[os=mac] .tree-row[may-have-children] > .tree-label {
-  background-image: url("../../../../app/theme/bookmark_bar_folder_mac.png");
-}
-
-html[os=mac] .tree-item[expanded] > .tree-row > .tree-label {
   background-image: url("../../../../app/theme/bookmark_bar_folder_mac.png");
 }
+</if>
 
 .main {
   position: absolute;
@@ -221,10 +219,9 @@ html[os=mac] .tree-item[expanded] > .tree-row > .tree-label {
   width: 5px;
   background-color: #ebeff9;
   cursor: e-resize;
-}
-
-html[os=mac] .splitter {
+<if expr="is_macosx">

[Dan Beam, 2012/03/21 03:02:53]

We should establish how we override styles for just 1 OS, as it seems like we
should be saying <if expr="not is_macosx"> or <else> or something, here. (is
there an <else>?)

[csilv, 2012/03/21 19:35:22]

<else> would be nice, but it doesn't exist yet. 
http://code.google.com/p/grit-i18n/ if you're interested.  :-)

   cursor: col-resize;
+</if>
 }
 
 .logo {