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

Unified Diff: ui/views/widget/native_widget_aura.cc

Issue 9428018: Create BaseWindow and ExtensionWindowWrapper for extension API access to Panels (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Address comments. Created 8 years, 10 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: ui/views/widget/native_widget_aura.cc
diff --git a/ui/views/widget/native_widget_aura.cc b/ui/views/widget/native_widget_aura.cc
index 098eeaed904ba314b516efa0355ac5cc8ee5e6b7..4e94a2314a84e584d07807a2ad73cd2c83da2170 100644
--- a/ui/views/widget/native_widget_aura.cc
+++ b/ui/views/widget/native_widget_aura.cc
@@ -415,6 +415,8 @@ gfx::Rect NativeWidgetAura::GetRestoredBounds() const {
}
void NativeWidgetAura::SetBounds(const gfx::Rect& bounds) {
+ if (GetRestoreBounds(window_))
sky 2012/02/24 04:38:24 I don't think this makes sense here. Some layout m
stevenjb 2012/02/24 19:25:06 See http://codereview.chromium.org/9372126/
+ SetRestoreBounds(window_, bounds);
window_->SetBounds(bounds);
}
@@ -516,8 +518,9 @@ void NativeWidgetAura::Maximize() {
}
void NativeWidgetAura::Minimize() {
- // No minimized window for aura. crbug.com/104571.
- NOTREACHED();
+ // Note: This currently does not do anything except set the property,
+ // see crbug.com/104571.
+ window_->SetProperty(aura::client::kShowStateKey, ui::SHOW_STATE_MINIMIZED);
}
bool NativeWidgetAura::IsMaximized() const {
« chrome/browser/ui/views/aura/panel_view_aura.cc ('K') | « chrome/chrome_browser.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698