Stub out ScreenManagerOzoneExternal

services/ui/display/screen_manager_ozone_external.h

Index: services/ui/display/screen_manager_ozone_external.h
diff --git a/services/ui/display/screen_manager_ozone_external.h b/services/ui/display/screen_manager_ozone_external.h
new file mode 100644
index 0000000000000000000000000000000000000000..e2c9c86ea869ee2f7ce849d5ad88a14a3960501e
--- /dev/null
+++ b/services/ui/display/screen_manager_ozone_external.h
@@ -0,0 +1,39 @@
+// Copyright 2017 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.
+
+#ifndef SERVICES_UI_DISPLAY_SCREEN_MANAGER_OZONE_EXTERNAL_H_
+#define SERVICES_UI_DISPLAY_SCREEN_MANAGER_OZONE_EXTERNAL_H_
+
+#include "base/memory/weak_ptr.h"
+#include "services/ui/display/screen_manager.h"
+
+namespace display {
+
+// In external window mode, the purpose of having a ScreenManager
+// does not apply: there is not a ScreenManagerDelegate manager
+// responsible for creating Dislpay instances.
+// Basically, in this mode WindowTreeHost creates the display instance.
+//
+// ScreenManagerOzoneExternal provides the stub out implementation
+// of ScreenManager for Ozone non-chromeos platforms.
+class ScreenManagerOzoneExternal : public ScreenManager {
+ public:
+  ScreenManagerOzoneExternal();
+  ~ScreenManagerOzoneExternal() override;
+
+ private:
+  // ScreenManager.
+  void AddInterfaces(service_manager::InterfaceRegistry* registry) override;
+  void Init(ScreenManagerDelegate* delegate) override;
+  int64_t GetPrimaryDisplayId() const override;
+  void RequestCloseDisplay(int64_t display_id) override;
+
+  base::WeakPtrFactory<ScreenManagerOzoneExternal> weak_ptr_factory_;

[kylechar, 2017/02/13 21:45:59]

I don't think you need the WeakPtrFactory? It's not used.

+
+  DISALLOW_COPY_AND_ASSIGN(ScreenManagerOzoneExternal);
+};
+
+}  // namespace display
+
+#endif  // SERVICES_UI_DISPLAY_SCREEN_MANAGER_OZONE_EXTERNAL_H_