Add early-injection capability to user scripts. I haven't yet

chrome/common/extensions/user_script_unittest.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/file_path.h"
 #include "base/logging.h"
 #include "chrome/common/extensions/user_script.h"
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 60 matching lines @@
 }
 
 TEST(UserScriptTest, Pickle) {
   URLPattern pattern1;
   URLPattern pattern2;
   ASSERT_TRUE(pattern1.Parse("http://*/foo*"));
   ASSERT_TRUE(pattern2.Parse("http://bar/baz*"));
 
   UserScript script1;
   script1.set_url(GURL("chrome-user-script:/foo.user.js"));
+  script1.set_run_location(UserScript::DOCUMENT_START);
   script1.add_url_pattern(pattern1);
   script1.add_url_pattern(pattern2);
 
   Pickle pickle;
   script1.Pickle(&pickle);
 
   void* iter = NULL;
   UserScript script2;
   script2.Unpickle(pickle, &iter);
 
   EXPECT_EQ(script1.url(), script2.url());
   ASSERT_EQ(script1.globs().size(), script2.globs().size());
   for (size_t i = 0; i < script1.globs().size(); ++i) {
     EXPECT_EQ(script1.globs()[i], script2.globs()[i]);
   }
   ASSERT_EQ(script1.url_patterns().size(), script2.url_patterns().size());
   for (size_t i = 0; i < script1.url_patterns().size(); ++i) {
     EXPECT_EQ(script1.url_patterns()[i].GetAsString(),
               script2.url_patterns()[i].GetAsString());
   }
 }
+
+TEST(UserScriptTest, Defaults) {

[Matt Perry, 2009/02/11 19:28:46]

Do these defaults need to be this way for compatibility?  If so, add a comment. 
Otherwise, it seems strange that you have a unit test for variable
initialization :).

+  UserScript script;
+  ASSERT_EQ(FILE_PATH_LITERAL(""), script.path().value());
+  ASSERT_EQ("", script.url().spec());
+  ASSERT_EQ(0, script.url_patterns().size());
+  ASSERT_EQ(0, script.globs().size());
+  ASSERT_EQ(UserScript::DOCUMENT_END, script.run_location());
+}