Test nextOnLine does not return ignored nodes

Bug: 1001562 Change-Id: I0e0eb5ab1b04f20605946c525ea4abdea831c11f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791439Reviewed-by: Aaron Leventhal <aleventhal@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#695289}

Test nextOnLine does not return ignored nodes
Bug: 1001562 Change-Id: I0e0eb5ab1b04f20605946c525ea4abdea831c11f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791439Reviewed-by: Aaron Leventhal <aleventhal@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#695289}
9257c53d · David Tseng · Commit Bot · 1ae5304c · 9257c53d · 9257c53d
Commit 9257c53d authored Sep 10, 2019 by David Tseng Committed by Commit Bot Sep 10, 2019
4 changed files
--- a/chrome/browser/extensions/api/automation/automation_apitest.cc
+++ b/chrome/browser/extensions/api/automation/automation_apitest.cc
@@ -414,6 +414,13 @@ IN_PROC_BROWSER_TEST_F(AutomationApiTestWithLanguageDetection,
      << message_;
 }

+IN_PROC_BROWSER_TEST_F(AutomationApiTest, IgnoredNodesNotReturned) {
+  StartEmbeddedTestServer();
+  ASSERT_TRUE(RunExtensionSubtest("automation/tests/tabs",
+                                  "ignored_nodes_not_returned.html"))
+      << message_;
+}
+
 #if defined(OS_CHROMEOS)

 class AutomationApiTestWithDeviceScaleFactor : public AutomationApiTest {

--- a/chrome/test/data/extensions/api_test/automation/sites/ignored_nodes_not_returned.html
+++ b/chrome/test/data/extensions/api_test/automation/sites/ignored_nodes_not_returned.html
+<!--
+ * Copyright 2019 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.
+-->
+<html>
+  <body>
+    <div role="textbox" contenteditable>
+      This is a test<br role="presentation"> of a content editable.
+    </div>
+  </body>
+</html>
--- a/chrome/test/data/extensions/api_test/automation/tests/tabs/ignored_nodes_not_returned.html
+++ b/chrome/test/data/extensions/api_test/automation/tests/tabs/ignored_nodes_not_returned.html
+<!--
+ * Copyright 2019 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.
+-->
+<script src="common.js"></script>
+<script src="ignored_nodes_not_returned.js"></script>
--- a/chrome/test/data/extensions/api_test/automation/tests/tabs/ignored_nodes_not_returned.js
+++ b/chrome/test/data/extensions/api_test/automation/tests/tabs/ignored_nodes_not_returned.js
+// Copyright 2019 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.
+
+var allTests = [
+  function testIgnoredNodesNotReturned() {
+    var node = rootNode.find({role: chrome.automation.RoleType.STATIC_TEXT});
+    assertEq('This is a test', node.name);
+    assertEq(1, node.children.length);
+    assertEq(chrome.automation.RoleType.INLINE_TEXT_BOX, node.firstChild.role);
+    assertEq('This is a test', node.firstChild.name);
+
+    // The line break is ignored and should not show up.
+    assertEq(undefined, node.nextOnLine);
+    assertEq(undefined, node.firstChild.nextOnLine);
+
+    node = node.nextSibling;
+    assertEq('of a content editable.', node.name);
+    assertEq(1, node.children.length);
+    assertEq(chrome.automation.RoleType.INLINE_TEXT_BOX, node.firstChild.role);
+    assertEq('of a content editable.', node.firstChild.name);
+    chrome.test.succeed();
+  }
+];
+
+setUpAndRunTests(allTests, 'ignored_nodes_not_returned.html');