Mac: Fix window raising for multiple desktops

chrome/browser/app_controller_mac_browsertest.mm

 // 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.
 
 #import <Cocoa/Cocoa.h>
 
 #include "apps/shell_window_registry.h"
 #include "base/command_line.h"
 #include "base/mac/scoped_nsobject.h"
 #include "chrome/app/chrome_command_ids.h"
@@ skipping 30 matching lines @@
   const BrowserList* active_browser_list_;
 };
 
 // Test that if only a platform app window is open and no browser windows are
 // open then a reopen event does nothing.
 IN_PROC_BROWSER_TEST_F(AppControllerPlatformAppBrowserTest,
                        PlatformAppReopenWithWindows) {
   base::scoped_nsobject<AppController> ac([[AppController alloc] init]);
   NSUInteger old_window_count = [[NSApp windows] count];
   EXPECT_EQ(1u, active_browser_list_->size());
-  BOOL result = [ac applicationShouldHandleReopen:NSApp hasVisibleWindows:YES];
+  [ac applicationShouldHandleReopen:NSApp hasVisibleWindows:YES];
+  // We do not EXPECT_TRUE the result here because the method
+  // deminiaturizes windows manually rather than return YES and have
+  // AppKit do it.
 
-  EXPECT_TRUE(result);
   EXPECT_EQ(old_window_count, [[NSApp windows] count]);
   EXPECT_EQ(1u, active_browser_list_->size());
 }
 
 IN_PROC_BROWSER_TEST_F(AppControllerPlatformAppBrowserTest,
                        ActivationFocusesBrowserWindow) {
   base::scoped_nsobject<AppController> app_controller(
       [[AppController alloc] init]);
 
   ExtensionTestMessageListener listener("Launched", false);
@@ skipping 41 matching lines @@
   EXPECT_FALSE(result);
   EXPECT_EQ(2u, active_browser_list_->size());
 
   Browser* browser = active_browser_list_->get(0);
   GURL current_url =
       browser->tab_strip_model()->GetActiveWebContents()->GetURL();
   EXPECT_EQ(GetAppURL(), current_url.spec());
 }
 
 }  // namespace