Commit 5e207050 authored by derat@chromium.org's avatar derat@chromium.org

aura: Comment noisy NOTIMPLEMENTEDs in NativeWidgetAura.

This also wires up GetRestoredBounds().

BUG=99729
TEST=manual: less spam

Review URL: http://codereview.chromium.org/8417014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107735 0039d316-1c4b-4281-b951-d872f2087c98
parent 70fda3cd
...@@ -77,6 +77,7 @@ class AURA_EXPORT Window : public ui::LayerDelegate { ...@@ -77,6 +77,7 @@ class AURA_EXPORT Window : public ui::LayerDelegate {
WindowDelegate* delegate() { return delegate_; } WindowDelegate* delegate() { return delegate_; }
const gfx::Rect& bounds() const; const gfx::Rect& bounds() const;
const gfx::Rect& restore_bounds() const { return restore_bounds_; }
Window* parent() { return parent_; } Window* parent() { return parent_; }
const Window* parent() const { return parent_; } const Window* parent() const { return parent_; }
......
...@@ -164,7 +164,7 @@ void NativeWidgetAura::ReorderLayers() { ...@@ -164,7 +164,7 @@ void NativeWidgetAura::ReorderLayers() {
} }
void NativeWidgetAura::ViewRemoved(View* view) { void NativeWidgetAura::ViewRemoved(View* view) {
// NOTIMPLEMENTED(); //NOTIMPLEMENTED();
} }
void NativeWidgetAura::SetNativeWindowProperty(const char* name, void* value) { void NativeWidgetAura::SetNativeWindowProperty(const char* name, void* value) {
...@@ -181,14 +181,14 @@ TooltipManager* NativeWidgetAura::GetTooltipManager() const { ...@@ -181,14 +181,14 @@ TooltipManager* NativeWidgetAura::GetTooltipManager() const {
} }
bool NativeWidgetAura::IsScreenReaderActive() const { bool NativeWidgetAura::IsScreenReaderActive() const {
NOTIMPLEMENTED(); //NOTIMPLEMENTED();
return false; return false;
} }
void NativeWidgetAura::SendNativeAccessibilityEvent( void NativeWidgetAura::SendNativeAccessibilityEvent(
View* view, View* view,
ui::AccessibilityTypes::Event event_type) { ui::AccessibilityTypes::Event event_type) {
NOTIMPLEMENTED(); //NOTIMPLEMENTED();
} }
void NativeWidgetAura::SetMouseCapture() { void NativeWidgetAura::SetMouseCapture() {
...@@ -228,24 +228,24 @@ void NativeWidgetAura::GetWindowPlacement( ...@@ -228,24 +228,24 @@ void NativeWidgetAura::GetWindowPlacement(
} }
void NativeWidgetAura::SetWindowTitle(const string16& title) { void NativeWidgetAura::SetWindowTitle(const string16& title) {
NOTIMPLEMENTED(); // Aura doesn't have native window frames.
} }
void NativeWidgetAura::SetWindowIcons(const SkBitmap& window_icon, void NativeWidgetAura::SetWindowIcons(const SkBitmap& window_icon,
const SkBitmap& app_icon) { const SkBitmap& app_icon) {
NOTIMPLEMENTED(); // Aura doesn't have window icons.
} }
void NativeWidgetAura::SetAccessibleName(const string16& name) { void NativeWidgetAura::SetAccessibleName(const string16& name) {
NOTIMPLEMENTED(); //NOTIMPLEMENTED();
} }
void NativeWidgetAura::SetAccessibleRole(ui::AccessibilityTypes::Role role) { void NativeWidgetAura::SetAccessibleRole(ui::AccessibilityTypes::Role role) {
NOTIMPLEMENTED(); //NOTIMPLEMENTED();
} }
void NativeWidgetAura::SetAccessibleState(ui::AccessibilityTypes::State state) { void NativeWidgetAura::SetAccessibleState(ui::AccessibilityTypes::State state) {
NOTIMPLEMENTED(); //NOTIMPLEMENTED();
} }
void NativeWidgetAura::BecomeModal() { void NativeWidgetAura::BecomeModal() {
...@@ -257,15 +257,12 @@ gfx::Rect NativeWidgetAura::GetWindowScreenBounds() const { ...@@ -257,15 +257,12 @@ gfx::Rect NativeWidgetAura::GetWindowScreenBounds() const {
} }
gfx::Rect NativeWidgetAura::GetClientAreaScreenBounds() const { gfx::Rect NativeWidgetAura::GetClientAreaScreenBounds() const {
// TODO(beng): // In Aura, the entire window is the client area.
NOTIMPLEMENTED();
return window_->GetScreenBounds(); return window_->GetScreenBounds();
} }
gfx::Rect NativeWidgetAura::GetRestoredBounds() const { gfx::Rect NativeWidgetAura::GetRestoredBounds() const {
// TODO(beng): return window_->restore_bounds();
NOTIMPLEMENTED();
return window_->bounds();
} }
void NativeWidgetAura::SetBounds(const gfx::Rect& bounds) { void NativeWidgetAura::SetBounds(const gfx::Rect& bounds) {
......
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