Switch the about:gpu implementation from an about handler to dom_ui....

chrome/browser/resources/net_internals/tabswitcherview.css

+.tab-switcher-view * {

[arv (Not doing code reviews), 2010/12/08 19:52:37]

You should be able to do 

.tab-switcher-view {
  font-family: sans-serif;
}

since font-family is inherited.

+  font-family: sans-serif;
+}
+
+.tab-switcher-view ul {
+  list-style: none;
+  padding: 0;
+  margin: 0;
+}
+
+.tab-switcher-view {
+  border-bottom: 1px solid #555;
+  background: #aaa;
+  overflow: hidden;
+}
+
+.tab-switcher-view li {
+  float: left;
+  -webkit-margin-start: 5px;
+  direction: rtl;

[arv (Not doing code reviews), 2010/12/08 17:55:19]

I don't know why this is here. Can you add a comment?

[nduca, 2010/12/08 19:08:22]

Umm, in a previous review you asked me to add it? Or did I misunderstand your
comment? You wrote, "rtl"
I have to admit to being blind wrt internationalization.


On 2010/12/08 17:55:19, arv wrote:
Done.

[arv (Not doing code reviews), 2010/12/08 19:52:37]

Anytime I see a hardcoded left/right and no handling of RTL I add that comment.
The classic solution is to use direction aware properties (like
-webkit-margin-start) where possible and for the other cases set the dir
attribute on the html element and have a more specific rule.

.tab-switcher-view li {
  float: left;
}

html[dir=rtl] .tab-switcher-view li {
  float: right;
}

This requires you to set the dir on the HTML element though. Almost all of the
DOMUI pages do this using the i18n template system (like js templates but
simpler)

+}
+
+.tab-switcher-view a {
+  text-decoration: none;
+  text-align: center;
+  display: inline-block;
+  margin-top: 4px;
+  padding: 5px 10px 3px 10px;
+  -webkit-border-top-right-radius: 8px;

[arv (Not doing code reviews), 2010/12/08 17:55:19]

Remove -webkit- prefix for border radius

[nduca, 2010/12/08 19:08:22]

Done.

+  -webkit-border-top-left-radius: 8px;
+  background-clip: border-box;
+  background: #ccc;
+}
+
+.tab-switcher-view a:hover {
+  background: #eee;
+}
+
+
+.tab-switcher-view a:visited,
+.tab-switcher-view a  {
+  color: blue;
+}
+
+.tab-switcher-view .selected {
+  background: white;
+}
+
+.tab-switcher-view a.selected {
+  position:relative;

[arv (Not doing code reviews), 2010/12/08 17:55:19]

whitespace after :

[nduca, 2010/12/08 19:08:22]

Done.

+  top: 3px;
+  color: black;
+}