Call OnNativeWindowChanged after maximizing and restoring.

ui/views/widget/native_widget_aura.cc

 // Copyright (c) 2012 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.
 
 #include "ui/views/widget/native_widget_aura.h"
 
 #include "base/bind.h"
 #include "base/strings/string_util.h"
 #include "third_party/skia/include/core/SkRegion.h"
 #include "ui/aura/client/aura_constants.h"
@@ skipping 720 matching lines @@
   // handles the case of a maximize animation acquiring the layer (acquiring a
   // layer results in clearing the bounds).
   if (old_bounds.origin() != new_bounds.origin() ||
       (old_bounds == gfx::Rect(0, 0, 0, 0) && !new_bounds.IsEmpty())) {
     delegate_->OnNativeWidgetMove();
   }
   if (old_bounds.size() != new_bounds.size())
     delegate_->OnNativeWidgetSizeChanged(new_bounds.size());
 }
 
+void NativeWidgetAura::OnPropertyChanged(const void* key, intptr_t old) {
+  if (key == aura::client::kShowStateKey) {
+    delegate_->OnNativeWidgetWindowShowStateChanged(

[sky, 2014/08/06 17:23:00]

Won't this result in double the notifications if the bounds do change? Also,
might this be invoked before the bounds have changed? That said, I can't think
of a better way to handle this without adding something on
OnAfterPropertyChanged, which I'm loathe to suggest right now.

[jackhou1, 2014/08/11 05:41:06]

Yeah, SaveWindowPlacement gets called twice. I think the expensive part is
writing to prefs. I don't have any better ideas on how to avoid it though =S.

The bounds are correct in the JS context, so I assume it's being called after
the bounds change, unless there's an asynchronous step.

+        static_cast<ui::WindowShowState>(old));
+  }
+}
+
 gfx::NativeCursor NativeWidgetAura::GetCursor(const gfx::Point& point) {
   return cursor_;
 }
 
 int NativeWidgetAura::GetNonClientComponent(const gfx::Point& point) const {
   return delegate_->GetNonClientComponent(point);
 }
 
 bool NativeWidgetAura::ShouldDescendIntoChildForEventHandling(
       aura::Window* child,
@@ skipping 401 matching lines @@
   l10n_util::AdjustUIFont(&(ncm.lfCaptionFont));
   base::win::ScopedHFONT caption_font(CreateFontIndirect(&(ncm.lfCaptionFont)));
   return gfx::FontList(gfx::Font(caption_font));
 #else
   return gfx::FontList();
 #endif
 }
 
 }  // namespace internal
 }  // namespace views