Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(120)

Unified Diff: chrome/browser/chromeos/dbus/mus_console_service_provider_delegate.h

Issue 2648433006: Get frecon working with mustash. (Closed)
Patch Set: Fix manifest. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/dbus/mus_console_service_provider_delegate.h
diff --git a/chrome/browser/chromeos/dbus/mus_console_service_provider_delegate.h b/chrome/browser/chromeos/dbus/mus_console_service_provider_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..df52e84c9757e0218bc8bfb767694145231340ab
--- /dev/null
+++ b/chrome/browser/chromeos/dbus/mus_console_service_provider_delegate.h
@@ -0,0 +1,38 @@
+// 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 CHROME_BROWSER_CHROMEOS_DBUS_MUS_CONSOLE_SERVICE_PROVIDER_DELEGATE_H_
+#define CHROME_BROWSER_CHROMEOS_DBUS_MUS_CONSOLE_SERVICE_PROVIDER_DELEGATE_H_
+
+#include "base/macros.h"
+#include "chromeos/dbus/services/console_service_provider.h"
+#include "services/ui/public/interfaces/display/display_controller.mojom.h"
+
+namespace chromeos {
+
+// An implementation of ConsoleServiceProvider::Delegate that communicates with
+// mus-ws via IPC.
+class MusConsoleServiceProviderDelegate
+ : public ConsoleServiceProvider::Delegate {
+ public:
+ MusConsoleServiceProviderDelegate();
+ ~MusConsoleServiceProviderDelegate() override;
+
+ // ConsoleServiceProvider::Delegate overrides:
+ void TakeDisplayOwnership(const UpdateOwnershipCallback& callback) override;
sky 2017/01/24 18:59:03 I'm not familiar with ConsoleServiceProvider::Dele
kylechar 2017/01/24 19:17:31 Yep, DisplayConfigurator::TakeControl/RelinquishCo
+ void ReleaseDisplayOwnership(
+ const UpdateOwnershipCallback& callback) override;
+
+ private:
+ void ConnectDisplayControllerIfNecessary();
+
+ // TODO(kylechar): Can we have one DisplayController for Chrome somewhere?
+ display::mojom::DisplayControllerPtr display_controller_;
+
+ DISALLOW_COPY_AND_ASSIGN(MusConsoleServiceProviderDelegate);
+};
+
+} // namespace chromeos
+
+#endif // CHROME_BROWSER_CHROMEOS_DBUS_MUS_CONSOLE_SERVICE_PROVIDER_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698