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

Unified Diff: views/widget/native_widget_aura.cc

Issue 8387043: [Aura] Support always-on-top top level window. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: adding AuraShellTestBase that creates/deletes Shell and fix win_aura crash Created 9 years, 1 month 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
« ui/base/x/x11_util.h ('K') | « ui/base/x/x11_util.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: views/widget/native_widget_aura.cc
diff --git a/views/widget/native_widget_aura.cc b/views/widget/native_widget_aura.cc
index 0a714443594c014ba755ab6bffb452f2c10c3584..4ff026efd56b3cf2cdc171c849595264aa28fecd 100644
--- a/views/widget/native_widget_aura.cc
+++ b/views/widget/native_widget_aura.cc
@@ -143,6 +143,8 @@ void NativeWidgetAura::InitNativeWidget(const Widget::InitParams& params) {
gfx::NativeView parent = params.GetParent();
if (parent)
parent->AddTransientChild(window_);
+ // SetAlwaysOnTop before SetParent so that always-on-top container is used.
+ SetAlwaysOnTop(params.keep_on_top);
window_->SetParent(NULL);
}
// TODO(beng): do this some other way.
@@ -410,8 +412,7 @@ bool NativeWidgetAura::IsActive() const {
}
void NativeWidgetAura::SetAlwaysOnTop(bool on_top) {
- // http://crbug.com/102582
- NOTIMPLEMENTED();
+ window_->SetIntProperty(aura::kAlwaysOnTopKey, on_top);
}
void NativeWidgetAura::Maximize() {
« ui/base/x/x11_util.h ('K') | « ui/base/x/x11_util.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698