UI polish for certificate viewer

chrome/browser/resources/sync_internals/sync_index.html

Index: chrome/browser/resources/sync_internals/sync_index.html
diff --git a/chrome/browser/resources/sync_internals/sync_index.html b/chrome/browser/resources/sync_internals/sync_index.html
index 74d67f6f91844a298496eb6c563c3dc8fcb9da90..e94809e409952fc83c2642a27b6b68a16fe1f1b4 100644
--- a/chrome/browser/resources/sync_internals/sync_index.html
+++ b/chrome/browser/resources/sync_internals/sync_index.html
@@ -9,6 +9,12 @@ chrome/test/functional/special_tabs.py. -->
 <link rel="stylesheet" href="chrome://resources/css/tree.css">
 <link rel="stylesheet" href="sync_search.css">
 <link rel="stylesheet" href="sync_node_browser.css">
+<style>
+body {
+  /* Remove body margin as the general tabpanels do not have outer border. */
+  margin: 0;
+}

[flackr, 2011/11/21 20:43:31]

Move to CSS file.

[bshe, 2011/11/22 16:26:24]

Should I create a new CSS file for this rule? Or should I put this rule to
sync_search.css/sync_node_browser.css file? sync_search.css and
sync_node_browser.css dont seem like the right place to put this rule. They are
for search and node_browser tabs only. Also not sure create a new css file with
just one rule is good or bad?

[flackr, 2011/11/22 17:02:59]

It's okay to create a new file with one rule, if there are more overall rules
for the page then it can be further used. As you say, both the existing css
files are specific to one tab, so adding a general file for the overall page
makes sense :-)

[bshe, 2011/11/22 19:20:46]

Done.

+</style>
 
 <script src="chrome://resources/css/tree.css.js"></script>