EnumerateModules: Address UI review comments....

chrome/browser/resources/about_conflicts.html

Index: chrome/browser/resources/about_conflicts.html
===================================================================
--- chrome/browser/resources/about_conflicts.html	(revision 67442)
+++ chrome/browser/resources/about_conflicts.html	(working copy)
@@ -128,7 +128,7 @@
 }
 
 .suspected-bad {
-  color: orange;
+  color: #DD7700;
 }
 
 .confirmed-bad {
@@ -231,62 +231,42 @@
       </div>
 
       <div jsdisplay="moduleList.length > 0">
-      <div class="module" jsselect="moduleList">
-        <table width="100%" cellpadding="0" cellspacing="0">
-        <tr class="module-loaded">
-        <td valign="top">
-          <table cellpadding="2" cellspacing="0">
-            <tr>
-              <td colspan="2"><span class="module-name" dir="ltr"
-                                   jscontent="name">NAME</span>
-                              <span dir="ltr"
-                                    jsdisplay="type_description.length > 0"
-                              >(<span dir="ltr"
-                                      jscontent="type_description">MODULE_TYPE</span
-                              >)</span>
-              </td>
-            </tr>
-            <tr>
-              <td width="75"><span i18n-content="headerSoftware" /></td>
-              <td><span dir="ltr" jsvalues=".innerHTML:description"></span>
-                  <span dir="ltr" jsdisplay="version.length > 0"> - </span>
-                  <span dir="ltr" jscontent="version">VERSION</span></td>
-            </tr>
-            <tr jsdisplay="digital_signer.length > 0">
-              <td><span i18n-content="headerSignedBy" /></td>
-              <td><span dir="ltr" jscontent="digital_signer">SIGNER</span></td>
-            </tr>
-            <tr>
-              <td><span i18n-content="headerLocation" /></td>
-              <td><span dir="ltr"
-                        jscontent="location">LOCATION</span><span
-                            dir="ltr" jscontent="name">NAME</span></td>
-            </tr>
-            <tr jsdisplay="status == 2 || status == 3">
-              <td><span i18n-content="headerWarning" /></td>
-              <td><span jsdisplay="status == 2"
-                        i18n-content="moduleSuspectedBad"
-                        class="suspected-bad">SUSPECTED_BAD</span>
-                  <span jsdisplay="status == 3"
-                        i18n-content="moduleConfirmedBad"
-                        class="confirmed-bad">CONFIRMED_BAD</span>
-                  <a jsdisplay="help_url.length > 0"
-                     jsvalues=".href:help_url"><span
-                         i18n-content="helpCenterLink">HELP_CENTER</span></a>
-              </td>
-            </tr>
-            <tr jsdisplay="possibleResolution.length > 0">
-              <td><span i18n-content="headerHelpTip" /></td>
-              <td><span
-                  jscontent="possibleResolution">POSSIBLE_RESOLUTION</span></td>
-            </tr>
-          </table>
-
-        </td>
-        </tr>
+      <table width="100%" cellpadding="0" cellspacing="0">
+      <tr class="module-loaded">
+      <td valign="top">
+        <table cellpadding="2" cellspacing="0" border="0">
+          <tr jsselect="moduleList">
+            <td valign="top"><span dir="ltr" jsvalues=".innerHTML:description"></span></div>
+                <span jsdisplay="status == 2 || status == 3"><br><nobr>

[Nico, 2010/11/26 17:54:36]

can use use css instead of <br> tags for vertical layout? same question for nobr
(it's white-space: nowrap in this case i believe)

+                <span jsdisplay="status == 2"
+                      i18n-content="moduleSuspectedBad"
+                      class="suspected-bad">SUSPECTED_BAD</span>
+                <span jsdisplay="status == 3"
+                      i18n-content="moduleConfirmedBad"
+                      class="confirmed-bad">CONFIRMED_BAD</span>
+                <a jsdisplay="help_url.length > 0"

[Nico, 2010/11/26 17:54:36]

do you need to escape the >, i.e. >? (
http://htmlhelp.com/tools/validator/problems.html#amp ).

+                      jsvalues=".href:help_url"><span

[Nico, 2010/11/26 17:54:36]

dedent 3?

+                          i18n-content="helpCenterLink">HELP_CENTER</span></a></nobr>
+                <span jsdisplay="possibleResolution.length > 0"><br>
+                <span jscontent="possibleResolution">POSSIBLE_RESOLUTION</span></span><br><br></span>
+            </td>
+            <td valign="top"><nobr><span dir="ltr" jscontent="digital_signer">SIGNER</span></nobr></td>
+            <td valign="top"><span dir="ltr" jscontent="version">VERSION</span></td>
+            <td valign="top"><span dir="ltr"
+                                   jscontent="location">LOCATION</span><strong><span
+                          dir="ltr" jscontent="name">NAME</span></strong>
+                <span dir="ltr"
+                      jsdisplay="type_description.length > 0"
+                    >(<span dir="ltr"
+                            jscontent="type_description">MODULE_TYPE</span
+                    >)</span>
+            </td>
+          </tr>
         </table>
+      </td>
+      </tr>
+      </table>
       </div>
-      </div>
     </div>
   </div>
 </div>