Get rid of system_export.h, etc.

mojo/edk/system/BUILD.gn

 # Copyright 2014 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("../mojo_edk.gni")
 import("//testing/test.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
   import("//build/config/android/rules.gni")
 }
 
+# TODO(vtl): Should we get rid of this?

[jamesr, 2015/10/23 23:12:55]

yes?

[viettrungluu, 2015/10/24 00:04:57]

Yeah, but then there's a cascade of things that refers to it (and then the
question of whether we should get rid of them). (If we think that there'll never
need to be config for stuff in the EDK, then we can get rid of them too. But if
we do need config for the EDK, then it'd be slightly annoying to bring back.)

So I might get rid of it as a separate change, which would at least make it
easier to revert.

 config("system_config") {
-  defines = [
-    # Ensures that dependent projects import the core functions on Windows.
-    "MOJO_USE_SYSTEM_IMPL",
-  ]
 }
 
 component("system") {
   output_name = "mojo_system_impl"
 
   sources = [
     "async_waiter.cc",
     "async_waiter.h",
     "awakable.h",
     "awakable_list.cc",
@@ skipping 85 matching lines @@
     "slave_connection_manager.h",
     "thread_annotations.h",
     "transport_data.cc",
     "transport_data.h",
     "unique_identifier.cc",
     "unique_identifier.h",
     "waiter.cc",
     "waiter.h",
   ]
 
-  defines = [ "MOJO_SYSTEM_IMPLEMENTATION" ]
-
   all_dependent_configs = [ ":system_config" ]
 
   public_deps = [
     "../embedder",
     "../embedder:delegates",
     "../embedder:platform",
     "../util",
     "../../public/c/system",
     "../../public/cpp/system",
   ]
@@ skipping 109 matching lines @@
   deps = [
     ":system",
     ":test_utils",
     "../test:test_support",
     "//base",
     "//base/test:test_support",
     "//base/test:test_support_perf",
     "//testing/gtest",
   ]
 }