Create top-level separate targets for browser and child dlls

chrome/app/child_stubs.cc

Index: chrome/app/child_stubs.cc
diff --git a/chrome/app/child_stubs.cc b/chrome/app/child_stubs.cc
new file mode 100644
index 0000000000000000000000000000000000000000..09cbdb417f40fb04ae11fd1c6777319e71a893b6
--- /dev/null
+++ b/chrome/app/child_stubs.cc
@@ -0,0 +1,28 @@
+// Copyright 2013 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.
+
+// TODO(scottmg): http://crbug.com/237249 This is called from common, but

[jam, 2013/07/17 16:23:37]

i have a very strong aversion to stub files. can we fix this first?

[scottmg, 2013/07/17 16:49:35]

pulled required bluetooth into device_bluetooth_common and removed
child_stubs.cc. it turns out one of the stubs is no longer required, so your
aversion is probably warranted.

[jam, 2013/07/17 18:09:58]

device/bluetooth seems like it's code that should only be used by the browser
process, so it seems like it's working around the problem by propagating
multi-process knowledge into low-level components that shouldn't know about it.

looking at it more, BluetoothPermissionData::Check is actually not called by any
code. so perhaps remove that unused function now so that we don't have to create
new targets in src/device/

+// the caller seems like it probably belongs in the browser. (?)
+
+#include "base/logging.h"
+#include "device/bluetooth/bluetooth_adapter_factory.h"
+#include "device/bluetooth/bluetooth_utils.h"
+
+namespace device {
+
+scoped_refptr<BluetoothAdapter> BluetoothAdapterFactory::MaybeGetAdapter() {
+  NOTREACHED();
+  return NULL;
+}
+
+namespace bluetooth_utils {
+
+std::string CanonicalUuid(std::string uuid) {
+  NOTREACHED();
+  return "";
+}
+
+}  // namespace bluetooth_utils
+
+}  // namespace device