Harden and move CSS declarations out of the code.

Remove CSS from background_page.html and change getLinkHintCss to only return the user's overrides.
Add vimiumReset class to html generated by completionDialog, helpDialog, linkHints and vimiumFrontEnd
Add vimium.css to the manifest and link to it in the tests_harnesses/automated.html and options.html.
Add an id, vimiumLinkHintsContainer, to linkhints div to allow user overrides to have a higher specificity.
Change the default for the user CSS override.

1 files changed, 23 insertions, 0 deletions

diff --git a/test_harnesses/automated.html b/test_harnesses/automated.html
index 764b0d2f..ac9b9d37 100644
--- a/test_harnesses/automated.html
+++ b/test_harnesses/automated.html
@@ -26,6 +26,7 @@
         border-bottom: dashed 1px #999;
       }
     </style>
+    <link rel="stylesheet" type="text/css" href="../vimium.css" />
     <script type="text/javascript" src="../lib/utils.js"></script>
     <script type="text/javascript" src="../lib/keyboardUtils.js"></script>
     <script type="text/javascript" src="../linkHints.js"></script>
@@ -223,6 +224,28 @@
             assert.equal("5: a label", linkHints.hintMarkers[4].textContent.toLowerCase());
           })
 
+        ),
+
+        context("Input focus",
+
+          setup(function() {
+            var testContent =
+              "<input type='text' id='first'/>" +
+              "<input style='display:none;' id='second'/>" +
+              "<input type='password' id='third' value='some value'/>";
+            document.getElementById("test-div").innerHTML = testContent;
+          }),
+
+          tearDown(function() {
+            document.getElementById("test-div").innerHTML = "";
+          }),
+
+          should("focus the right element", function() {
+            focusInput(1);
+            assert.equal('first', document.activeElement.id);
+            focusInput(100);
+            assert.equal('third', document.activeElement.id);
+          })
         )
 
       );