Revert "platform_window: Fix component builds"

This reverts commit 8229a222.

Reason for revert: Tentatively reverting because it seems like a possible culprit for this tree-closing failure: https://ci.chromium.org/p/chromium/builders/ci/fuchsia-x64-dbg/2029

Original change's description:
> platform_window: Fix component builds
> 
> The platform_window target is used from multiple components and should
> therefore be a component itself. This is needed to avoid ODR violations.
> 
> One ODR problem that currently can happen is the duplication of the
> global boolean backing UseTestConfigForPlatformWindows().
> 
> Bug: 1019015
> 
> Change-Id: I935df9a2fb2c6df1fec887c45c332bc833db91b9
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1889293
> Reviewed-by: Scott Violet <sky@chromium.org>
> Commit-Queue: Michael Spang <spang@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#710905}

TBR=sky@chromium.org,spang@chromium.org

Change-Id: I2c39a3dd1655cd20273ee82778b2421628b6fac9
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1019015
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1891330Reviewed-by: anthonyvd <anthonyvd@chromium.org>
Commit-Queue: anthonyvd <anthonyvd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710924}

ui/platform_window/BUILD.gn

@@ -4,7 +4,7 @@
 
 import("//build/config/ui.gni")
 
-component("platform_window") {
+source_set("platform_window") {
   sources = [
     "platform_window_base.cc",
     "platform_window_base.h",
@@ -15,8 +15,6 @@ component("platform_window") {
     "platform_window_init_properties.h",
   ]
 
-  defines = [ "IS_PLATFORM_WINDOW_IMPL" ]
-
   deps = [
     "//base",
     "//ui/base",

ui/platform_window/common/BUILD.gn

@@ -4,17 +4,7 @@
 
 import("//build/config/ui.gni")
 
-assert(is_linux)
-
-component("common") {
-  output_name = "platform_window_common"
-
-  defines = [ "IS_PLATFORM_WINDOW_COMMON_IMPL" ]
-
-  public_deps = [
-    "//base",
-  ]
-
+source_set("common") {
   sources = [
     "platform_window_defaults.cc",
     "platform_window_defaults.h",

ui/platform_window/common/platform_window_defaults.h

@@ -5,14 +5,12 @@
 #ifndef UI_PLATFORM_WINDOW_COMMON_PLATFORM_WINDOW_DEFAULTS_H_
 #define UI_PLATFORM_WINDOW_COMMON_PLATFORM_WINDOW_DEFAULTS_H_
 
-#include "base/component_export.h"
-
 namespace ui {
 
 // Returns true if PlatformWindow should use test configuration. Will return
 // false by default, unless test::EnableTestConfigForPlatformWindows() has been
 // called, then it will return true.
-COMPONENT_EXPORT(PLATFORM_WINDOW_COMMON) bool UseTestConfigForPlatformWindows();
+bool UseTestConfigForPlatformWindows();
 
 namespace test {
 
@@ -24,7 +22,6 @@ namespace test {
 // various tests, otherwise the call to Show() blocks because it never receives
 // the MapNotify event. It is unclear why this is necessary, but might be
 // related to calls to XInitThreads().
-COMPONENT_EXPORT(PLATFORM_WINDOW_COMMON)
 void EnableTestConfigForPlatformWindows();
 
 }  // namespace test

ui/platform_window/platform_window_base.h

@@ -7,7 +7,6 @@
 
 #include <memory>
 
-#include "base/component_export.h"
 #include "base/strings/string16.h"
 #include "ui/base/class_property.h"
 #include "ui/base/cursor/cursor.h"
@@ -29,8 +28,7 @@ namespace ui {
 //
 // Each instance of PlatformWindowBase represents a single window in the
 // underlying platform windowing system (i.e. X11/Win/OSX).
-class COMPONENT_EXPORT(PLATFORM_WINDOW) PlatformWindowBase
-    : public PropertyHandler {
+class PlatformWindowBase : public PropertyHandler {
  public:
   PlatformWindowBase();
   ~PlatformWindowBase() override;

ui/platform_window/platform_window_delegate_base.h

@@ -5,7 +5,6 @@
 #ifndef UI_PLATFORM_WINDOW_PLATFORM_WINDOW_DELEGATE_BASE_H_
 #define UI_PLATFORM_WINDOW_PLATFORM_WINDOW_DELEGATE_BASE_H_
 
-#include "base/component_export.h"
 #include "base/optional.h"
 #include "ui/gfx/native_widget_types.h"
 
@@ -29,7 +28,7 @@ enum class PlatformWindowState {
 // This is the bare minimum for PlatformWindowDeelegate, but some platforms may
 // require more, and should do so in a subclass. Please refer to
 // PlatformWindowDelegateLinux for an example.
-class COMPONENT_EXPORT(PLATFORM_WINDOW) PlatformWindowDelegateBase {
+class PlatformWindowDelegateBase {
  public:
   PlatformWindowDelegateBase();
   virtual ~PlatformWindowDelegateBase();

ui/platform_window/platform_window_delegate_linux.h

@@ -5,7 +5,6 @@
 #ifndef UI_PLATFORM_WINDOW_PLATFORM_WINDOW_DELEGATE_LINUX_H_
 #define UI_PLATFORM_WINDOW_PLATFORM_WINDOW_DELEGATE_LINUX_H_
 
-#include "base/component_export.h"
 #include "ui/base/buildflags.h"
 #include "ui/platform_window/platform_window_delegate_base.h"
 
@@ -24,8 +23,7 @@ namespace ui {
 // This is an optional linux delegate interface, which should be implemented by
 // linux-based platforms. It contains both Wayland and X11 specific and common
 // interfaces.
-class COMPONENT_EXPORT(PLATFORM_WINDOW) PlatformWindowDelegateLinux
-    : public PlatformWindowDelegateBase {
+class PlatformWindowDelegateLinux : public PlatformWindowDelegateBase {
  public:
   PlatformWindowDelegateLinux();
   ~PlatformWindowDelegateLinux() override;

ui/platform_window/platform_window_init_properties.h

@@ -7,7 +7,6 @@
 
 #include <string>
 
-#include "base/component_export.h"
 #include "base/optional.h"
 #include "build/build_config.h"
 #include "ui/gfx/geometry/rect.h"
@@ -41,7 +40,7 @@ enum class PlatformWindowOpacity {
 
 // Initial properties which are passed to PlatformWindow to be initialized
 // with a desired set of properties.
-struct COMPONENT_EXPORT(PLATFORM_WINDOW) PlatformWindowInitProperties {
+struct PlatformWindowInitProperties {
   PlatformWindowInitProperties();
 
   // Initializes properties with the specified |bounds|.

ui/platform_window/platform_window_linux.h

@@ -5,7 +5,6 @@
 #ifndef UI_PLATFORM_WINDOW_PLATFORM_WINDOW_LINUX_H_
 #define UI_PLATFORM_WINDOW_PLATFORM_WINDOW_LINUX_H_
 
-#include "base/component_export.h"
 #include "base/optional.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/platform_window/platform_window_base.h"
@@ -13,8 +12,7 @@
 namespace ui {
 
 // Linux extensions to the PlatformWindowBase.
-class COMPONENT_EXPORT(PLATFORM_WINDOW) PlatformWindowLinux
-    : public PlatformWindowBase {
+class PlatformWindowLinux : public PlatformWindowBase {
  public:
   PlatformWindowLinux();
   ~PlatformWindowLinux() override;

ui/platform_window/platform_window_win.h

@@ -5,14 +5,12 @@
 #ifndef UI_PLATFORM_WINDOW_PLATFORM_WINDOW_WIN_H_
 #define UI_PLATFORM_WINDOW_PLATFORM_WINDOW_WIN_H_
 
-#include "base/component_export.h"
 #include "ui/platform_window/platform_window_base.h"
 
 namespace ui {
 
 // Windows extensions to the PlatformWindowBase.
-class COMPONENT_EXPORT(PLATFORM_WINDOW) PlatformWindowWin
-    : public PlatformWindowBase {
+class PlatformWindowWin : public PlatformWindowBase {
  public:
   PlatformWindowWin() = default;
   ~PlatformWindowWin() override = default;