MacViews: Implement CloseNow() as just -[NSWindow close].

ui/views/widget/native_widget_mac_unittest.mm

 // Copyright 2014 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.
 
 #import "ui/views/widget/native_widget_mac.h"
 
 #import <Cocoa/Cocoa.h>
 
 #import "base/mac/foundation_util.h"
 #import "base/mac/scoped_nsautorelease_pool.h"
@@ skipping 1137 matching lines @@
     EXPECT_TRUE(parent_nswindow_);
     EXPECT_TRUE([parent_nswindow_ delegate]);
   }
 
   ~ParentCloseMonitor() override {
     EXPECT_TRUE(child_closed_);  // Otherwise the observer wasn't removed.
   }
 
   void OnWidgetDestroying(Widget* child) override {
     // Upon a parent-triggered close, the NSWindow relationship will already be
     // removed. The parent should still be open (children are always closed

[karandeepb, 2016/10/18 10:00:29]

"the NSWindow relation will already be removed". Update this comment.

[tapted, 2016/10/19 06:20:25]

Done. And actually added a OnWidgetDestroyed() override as well, so we can check
the old condition as well.

     // first), but not have a delegate (since it is being torn down).
-    EXPECT_FALSE([child->GetNativeWindow() parentWindow]);
+    EXPECT_TRUE([child->GetNativeWindow() parentWindow]);
     EXPECT_TRUE([parent_nswindow_ isVisible]);
     EXPECT_FALSE([parent_nswindow_ delegate]);
 
     EXPECT_FALSE(child_closed_);
     child->RemoveObserver(this);
     child_closed_ = true;
   }
 
   bool child_closed() const { return child_closed_; }
 
@@ skipping 622 matching lines @@
 
 - (void)dealloc {
   if (deallocFlag_) {
     DCHECK(!*deallocFlag_);
     *deallocFlag_ = true;
   }
   [super dealloc];
 }
 
 @end