Check and canonicalize CSS selectors before registering PageStateMatchers.

chrome/browser/extensions/api/declarative_content/declarative_content_apitest.cc

 // Copyright (c) 2012 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 "chrome/browser/extensions/extension_action.h"
 #include "chrome/browser/extensions/extension_action_manager.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/extensions/extension_test_message_listener.h"
 #include "chrome/browser/extensions/test_extension_dir.h"
@@ skipping 91 matching lines @@
 
   // Remove it again to make sure that reverts the action.
   ASSERT_TRUE(content::ExecuteScript(
       tab, "document.body.innerHTML = 'Hello world';"));
   // Give the mutation observer a chance to run and send back the
   // matching-selector update.
   ASSERT_TRUE(content::ExecuteScript(tab, std::string()));
   EXPECT_FALSE(page_action->GetIsVisible(tab_id));
 }
 
+IN_PROC_BROWSER_TEST_F(DeclarativeContentApiTest,
+                       CanonicalizesPageStateMatcherCss) {
+  ext_dir_.WriteManifest(kDeclarativeContentManifest);
+  ext_dir_.WriteFile(
+      FILE_PATH_LITERAL("background.js"),
+      "var declarative = chrome.declarative;\n"
+      "\n"
+      "var PageStateMatcher = chrome.declarativeContent.PageStateMatcher;\n"
+      "\n"
+      "chrome.test.runTests([\n"
+      "  function canonicalizesCss() {\n"
+      "    var psm = new PageStateMatcher(\n"
+      "        {css: [\"input[type='password']\"]});\n"
+      "    chrome.test.assertEq(['input[type=\"password\"]'], psm.css);\n"
+      "    chrome.test.succeed();\n"
+      "  },\n"
+      "\n"
+      "  function throwsOnBadCss() {\n"
+      "    function NewPageStateMatcher(obj) {\n"
+      "      return new PageStateMatcher(obj);\n"
+      "    }"
+      "    chrome.test.assertThrows(NewPageStateMatcher,\n"
+      "                             undefined,\n"
+      "                             [{css: [\"input''\"]}],\n"
+      "                             /valid.*: input''$/);\n"
+      "    chrome.test.succeed();\n"
+      "  },\n"
+      "\n"
+      "  function throwsOnComplexSelector() {\n"
+      "    function NewPageStateMatcher(obj) {\n"
+      "      return new PageStateMatcher(obj);\n"
+      "    }"
+      "    chrome.test.assertThrows(NewPageStateMatcher,\n"
+      "                             undefined,\n"
+      "                             [{css: [\"div input\"]}],\n"
+      "                             /compound selector.*: div input$/);\n"

[not at google - send to devlin, 2013/08/29 15:20:25]

invalid types too?

[Jeffrey Yasskin, 2013/08/29 21:46:43]

I'm not sure what you mean here. The actual message is the same, but the
important part of it is that "div input" is not a compound selector. (It's a
"complex selector": http://dev.w3.org/csswg/selectors4/#structure)

I'm going to put that URL in the docs when I update them, so users shouldn't be
too confused by the error message, as long as they read the docs.

[not at google - send to devlin, 2013/08/29 22:24:03]

Oh I meant like rather than passing in a string pass in an object. But if this
goes through schema validation (I wasn't sure) then it's pointless to do so.

+      "    chrome.test.succeed();\n"
+      "  },\n"
+      "]);\n"
+      "\n");
+  ResultCatcher catcher;
+  ASSERT_TRUE(LoadExtension(ext_dir_.unpacked_path()));
+  ASSERT_TRUE(catcher.GetNextResult()) << catcher.message();
+}
+
 }  // namespace
 }  // namespace extensions