Merge pull request #39 from DrMarcII/master

Minor fixes
google · Apr 10, 2015 · a80022a · a80022a
2 parents d462f9f + 88c729a
commit a80022a
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 13 deletions.
diff --git a/dart/lib/async/webdriver.dart b/dart/lib/async/webdriver.dart
@@ -173,7 +173,7 @@ class _WebElementPageLoaderElement extends WebDriverPageLoaderElement {
   Future<WebDriverPageLoaderElement> get shadowRoot async {
     if (loader.useShadowDom) {
       if ((await context.driver.execute(
-          'arguments[0].shadowRoot != null', [context]))) {
+          'return arguments[0].shadowRoot != null', [context]))) {
         return new _ShadowRootPageLoaderElement(context, loader);
       }
       throw new PageLoaderException('$this does not have a shadowRoot');

diff --git a/dart/test/sync/html_no_shadow_dom_test.dart b/dart/test/sync/html_no_shadow_dom_test.dart
@@ -57,9 +57,3 @@ void main() {
 
   plt.runTests();
 }
-
-class NoOpNodeValidator implements html.NodeValidator {
-  bool allowsAttribute(
-      html.Element element, String attributeName, String value) => true;
-  bool allowsElement(html.Element element) => true;
-}
diff --git a/dart/test/sync/html_test.dart b/dart/test/sync/html_test.dart
@@ -57,9 +57,3 @@ void main() {
 
   plt.runTests();
 }
-
-class NoOpNodeValidator implements html.NodeValidator {
-  bool allowsAttribute(
-      html.Element element, String attributeName, String value) => true;
-  bool allowsElement(html.Element element) => true;
-}