Switch from WebDocument::insertStyleText to ::insertUserStyleSheet for programatic CSS injection.

chrome/test/data/extensions/api_test/executescript/basic/test.html

Index: chrome/test/data/extensions/api_test/executescript/basic/test.html
diff --git a/chrome/test/data/extensions/api_test/executescript/basic/test.html b/chrome/test/data/extensions/api_test/executescript/basic/test.html
index a5a2f668fc7f80e186d4829eee894752177b6efe..391c43a6482a0f8e6ef1d64b5aa50daba9c3677c 100644
--- a/chrome/test/data/extensions/api_test/executescript/basic/test.html
+++ b/chrome/test/data/extensions/api_test/executescript/basic/test.html
@@ -72,6 +72,19 @@ chrome.test.getConfig(function(config) {
         });
       },
 
+      function insertCSSTextShouldNotAffectDOM() {
+        chrome.tabs.insertCSS(tabId, {code: 'p {display: none}'}, function() {
+          chrome.tabs.executeScript(
+              tabId,
+              {code: 'document.title = document.styleSheets.length'},
+              function() {
+                chrome.tabs.get(tabId, pass(function(tab) {

[Matt Perry, 2011/07/26 23:07:29]

I think this test could end up being flaky, unfortunately. executeScript fires
its callback as soon as the script is sent to the renderer - it does not wait
for the script to actually run. So tabs.get() could end up querying the title
before the script has a chance to update it.

Then again... it looks like the rest of the tests in this file do the same
thing, so maybe I'm mistaken. Just an FYI.

+                  assertEq(tab.title, '0');
+                }));
+             });
+        });
+      },
+
       function executeJavaScriptCodeShouldFail() {
         chrome.tabs.update(tabId, { url: testFailureUrl }, function() {
           var script_file = {};