Disable etw_manifest in the GN Windows build.

base/trace_event/etw_manifest/BUILD.gn

 # Copyright 2015 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.
 
 import("//build/win/message_compiler.gni")
 
 assert(is_win, "This only runs on Windows.")
 
-message_compiler("chrome_events_win") {
-  visibility = [
-    "//base/trace_event/*",
-    "//chrome:main_dll",
-  ]
+# TOOD(brucedawson) bug 569989: Enable ETW manifest and link it into the
+# proper places. Enabling as-is may add this to too many targets. See the bug
+# for more information.
+if (true) {
+  group("chrome_events_win") {
+  }
+} else {
+  message_compiler("chrome_events_win") {

[brucedawson, 2015/12/15 19:32:54]

Does this control the linking of the .rc/.bin files only, or does it also
control the running of the message compiler?

The message compiler generates a header file which is included by
base\trace_event\trace_event_etw_export_win.cc so not running the message
compiler will lead to compile failures.

These failures may be temporarily hidden if chrome_events_win.h is leftover from
a previous build, so watch out for that.

[brettw, 2015/12/16 19:19:09]

Good catch! This would have been super annoying to find later.

+    visibility = [
+      "//base/trace_event/*",
+      "//chrome:main_dll",
+    ]
 
-  sources = [
-    "chrome_events_win.man",
-  ]
+    sources = [
+      "chrome_events_win.man",
+    ]
 
-  user_mode_logging = true
+    user_mode_logging = true
+  }
 }