MacViews: Don't switch Spaces when calling ShowInactive() for a child Widget.

ui/views/cocoa/bridged_native_widget.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/cocoa/bridged_native_widget.h"
 
 #import <objc/runtime.h>
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 410 matching lines @@
         NativeWidgetMac::GetBridgeForNativeWindow([params.parent window]);
     // If the parent is another BridgedNativeWidget, just add to the collection
     // of child windows it owns and manages. Otherwise, create an adapter to
     // anchor the child widget and observe when the parent NSWindow is closed.
     if (bridged_native_widget_parent) {
       parent_ = bridged_native_widget_parent;
       bridged_native_widget_parent->child_windows_.push_back(this);
     } else {
       parent_ = new WidgetOwnerNSWindowAdapter(this, params.parent);
     }
+    // crbug.com/697829: Widget::ShowInactive() could result in a Space switch
+    // when the widget has a parent, and we're calling -orderWindow:relativeTo:.
+    // Use Transient collection behaviour to prevent that.
+    [window_ setCollectionBehavior:[window_ collectionBehavior] |
+                                   NSWindowCollectionBehaviorTransient];
   }
 
   // OSX likes to put shadows on most things. However, frameless windows (with
   // styleMask = NSBorderlessWindowMask) default to no shadow. So change that.
   // SHADOW_TYPE_DROP is used for Menus, which get the same shadow style on Mac.
   switch (params.shadow_type) {
     case Widget::InitParams::SHADOW_TYPE_NONE:
       [window_ setHasShadow:NO];
       break;
     case Widget::InitParams::SHADOW_TYPE_DEFAULT:
@@ skipping 985 matching lines @@
   [bridged_view_ setMouseDownCanMoveWindow:draggable];
   // AppKit will not update its cache of mouseDownCanMoveWindow unless something
   // changes. Previously we tried adding an NSView and removing it, but for some
   // reason it required reposting the mouse-down event, and didn't always work.
   // Calling the below seems to be an effective solution.
   [window_ setMovableByWindowBackground:NO];
   [window_ setMovableByWindowBackground:YES];
 }
 
 }  // namespace views