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

Unified Diff: chrome/browser/ui/overlay/overlay_window.h

Issue 2905833004: [OverlayWindow] Add platform-independent window and views implementation. (Closed)
Patch Set: Rename PIP to Overlay. Created 3 years, 6 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/ui/overlay/overlay_window.h
diff --git a/chrome/browser/ui/overlay/overlay_window.h b/chrome/browser/ui/overlay/overlay_window.h
new file mode 100644
index 0000000000000000000000000000000000000000..48e7f8bd67263de34621cfeca9cd04c943543620
--- /dev/null
+++ b/chrome/browser/ui/overlay/overlay_window.h
@@ -0,0 +1,50 @@
+// 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_UI_OVERLAY_OVERLAY_WINDOW_H_
+#define CHROME_BROWSER_UI_OVERLAY_OVERLAY_WINDOW_H_
+
+#include "ui/base/base_window.h"
+#include "ui/compositor/layer.h"
+
+class OverlayWindow : public ui::BaseWindow {
sky 2017/06/20 16:51:39 Please add a description.
apacible 2017/06/24 00:42:03 Done.
+ public:
+ OverlayWindow() {}
+ virtual ~OverlayWindow() {}
+
+ // Returns a created OverlayWindow. This is defined in the platform-specific
+ // implementation for the class.
+ static std::unique_ptr<OverlayWindow> Create();
+
+ virtual void Init() = 0;
+ virtual ui::Layer* GetLayer() = 0;
+
+ // ui::BaseWindow:
+ bool IsActive() const override;
sky 2017/06/20 16:51:39 Why do you have these declarations with no definit
apacible 2017/06/24 00:42:02 These are defined in the platform-specific classes
+ bool IsMaximized() const override;
+ bool IsMinimized() const override;
+ bool IsFullscreen() const override;
+ gfx::NativeWindow GetNativeWindow() const override;
+ gfx::Rect GetRestoredBounds() const override;
+ ui::WindowShowState GetRestoredState() const override;
+ gfx::Rect GetBounds() const override;
+ void Show() override;
+ void Hide() override;
+ void ShowInactive() override;
+ void Close() override;
+ void Activate() override;
+ void Deactivate() override;
+ void Maximize() override;
+ void Minimize() override;
+ void Restore() override;
+ void SetBounds(const gfx::Rect& bounds) override;
+ void FlashFrame(bool flash) override;
+ bool IsAlwaysOnTop() const override;
+ void SetAlwaysOnTop(bool always_on_top) override;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(OverlayWindow);
+};
+
+#endif // CHROME_BROWSER_UI_OVERLAY_OVERLAY_WINDOW_H_

Powered by Google App Engine
This is Rietveld 408576698