Commit ad7f4854 authored by Jennifer Apacible's avatar Jennifer Apacible Committed by Commit Bot

[Picture in Picture] Move constructor within file.

Organizational change. No functional change.

Change-Id: Ia67b5ec8ce496562054211930916c6525e00cf45
Reviewed-on: https://chromium-review.googlesource.com/959341Reviewed-by: default avatarCJ DiMeglio <lethalantidote@chromium.org>
Commit-Queue: apacible <apacible@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543592}
parent 1778d698
...@@ -44,6 +44,21 @@ class OverlayWindowWidgetDelegate : public views::WidgetDelegate { ...@@ -44,6 +44,21 @@ class OverlayWindowWidgetDelegate : public views::WidgetDelegate {
DISALLOW_COPY_AND_ASSIGN(OverlayWindowWidgetDelegate); DISALLOW_COPY_AND_ASSIGN(OverlayWindowWidgetDelegate);
}; };
OverlayWindowViews::OverlayWindowViews() {
// TODO(apacible): Change window type to TYPE_WINDOW_FRAMELESS. It is
// temporarily TYPE_WINDOW for resizing purposes.
views::Widget::InitParams params(views::Widget::InitParams::TYPE_WINDOW);
params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
params.bounds = CalculateAndUpdateBounds();
params.keep_on_top = true;
params.visible_on_all_workspaces = true;
// Set WidgetDelegate for more control over |widget_|.
params.delegate = new OverlayWindowWidgetDelegate(this);
Init(params);
}
OverlayWindowViews::~OverlayWindowViews() = default; OverlayWindowViews::~OverlayWindowViews() = default;
gfx::Rect OverlayWindowViews::CalculateAndUpdateBounds() { gfx::Rect OverlayWindowViews::CalculateAndUpdateBounds() {
...@@ -124,18 +139,3 @@ void OverlayWindowViews::OnNativeWidgetWorkspaceChanged() { ...@@ -124,18 +139,3 @@ void OverlayWindowViews::OnNativeWidgetWorkspaceChanged() {
// Picture-in-Picture window. Currently, switching between workspaces on linux // Picture-in-Picture window. Currently, switching between workspaces on linux
// does not trigger this function. http://crbug.com/819673 // does not trigger this function. http://crbug.com/819673
} }
OverlayWindowViews::OverlayWindowViews() {
// TODO(apacible): Change window type to TYPE_WINDOW_FRAMELESS. It is
// temporarily TYPE_WINDOW for resizing purposes.
views::Widget::InitParams params(views::Widget::InitParams::TYPE_WINDOW);
params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
params.bounds = CalculateAndUpdateBounds();
params.keep_on_top = true;
params.visible_on_all_workspaces = true;
// Set WidgetDelegate for more control over |widget_|.
params.delegate = new OverlayWindowWidgetDelegate(this);
Init(params);
}
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment