Hide AXPlatformNode on ChromeOS.

ui/accessibility/BUILD.gn

Index: ui/accessibility/BUILD.gn
diff --git a/ui/accessibility/BUILD.gn b/ui/accessibility/BUILD.gn
index 06a95cc9e6e5f65a7051eda4a44e79ed95c502e4..2752c17279136233be6534792d239da76635d6f1 100644
--- a/ui/accessibility/BUILD.gn
+++ b/ui/accessibility/BUILD.gn
@@ -8,6 +8,7 @@ import("//build/config/ui.gni")
 import("//testing/libfuzzer/fuzzer_test.gni")
 import("//testing/test.gni")
 import("//tools/json_schema_compiler/json_schema_api.gni")
+import("//ui/base/ui_features.gni")
 
 if (is_android) {
   import("//build/config/android/rules.gni")
@@ -49,17 +50,24 @@ component("accessibility") {
     "ax_tree_update.h",
     "platform/ax_android_constants.cc",
     "platform/ax_android_constants.h",
-    "platform/ax_platform_node.cc",
-    "platform/ax_platform_node.h",
-    "platform/ax_platform_node_base.cc",
-    "platform/ax_platform_node_base.h",
-    "platform/ax_platform_node_delegate.h",
-    "platform/ax_platform_node_mac.h",
-    "platform/ax_platform_node_mac.mm",
-    "platform/ax_platform_node_win.cc",
-    "platform/ax_platform_node_win.h",
+    "platform/ax_platform_unique_id.cc",
+    "platform/ax_platform_unique_id.h",
   ]
 
+  if (has_platform_accessibility) {
+    sources += [
+      "platform/ax_platform_node.cc",
+      "platform/ax_platform_node.h",
+      "platform/ax_platform_node_base.cc",
+      "platform/ax_platform_node_base.h",
+      "platform/ax_platform_node_delegate.h",
+      "platform/ax_platform_node_mac.h",
+      "platform/ax_platform_node_mac.mm",
+      "platform/ax_platform_node_win.cc",
+      "platform/ax_platform_node_win.h",
+    ]
+  }
+
   defines = [ "ACCESSIBILITY_IMPLEMENTATION" ]
 
   public_deps = [
@@ -116,12 +124,17 @@ if (is_android) {
 
 static_library("test_support") {
   sources = [
-    "platform/test_ax_node_wrapper.cc",
-    "platform/test_ax_node_wrapper.h",
     "tree_generator.cc",
     "tree_generator.h",
   ]
 
+  if (is_win) {

[dmazzoni, 2017/03/21 15:42:04]

No concerns for this change, but just a note:

This should probably be if (has_platform_accessibility) because we
ought to have Mac tests at this layer too. Seeing this
made me realize we don't have any.

It looks like all of the Mac accessibility tests are in 
native_widget_mac_accessibility_unittest.mm, and while
there's nothing wrong with any of these tests, we should
try to do more unit-testing at the level of AXPlatformNode
too.

Tests in ui/views should focus on interactions between
accessibility and views, tests in ui/accessibility should focus
on getting all of the corner cases of the NSAccessibility
implementation right.

[tapted, 2017/03/22 01:47:27]

Filed http://crbug.com/703893 for this.

+    sources += [
+      "platform/test_ax_node_wrapper.cc",
+      "platform/test_ax_node_wrapper.h",
+    ]
+  }
+
   deps = [
     ":accessibility",
   ]