Mac: Fix window ordering for certificate viewer/selector sheets and overlays.

chrome/browser/ui/cocoa/ssl_client_certificate_selector_cocoa.mm

Index: chrome/browser/ui/cocoa/ssl_client_certificate_selector_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/ssl_client_certificate_selector_cocoa.mm b/chrome/browser/ui/cocoa/ssl_client_certificate_selector_cocoa.mm
index aa06b2d989ff696541645153fad4f79cd81ca48b..f7c6bc2026d270272bf9ddac00cbbda930157033 100644
--- a/chrome/browser/ui/cocoa/ssl_client_certificate_selector_cocoa.mm
+++ b/chrome/browser/ui/cocoa/ssl_client_certificate_selector_cocoa.mm
@@ -286,6 +286,9 @@ void ClearTableViewDataSourcesIfNeeded(NSWindow*) {}
 
   oldResizesSubviews_ = [[sheetWindow contentView] autoresizesSubviews];
   [[sheetWindow contentView] setAutoresizesSubviews:NO];
+
+  // Ensure this overlay doesn't interfere with overlays with hidden sheets.
+  [overlayWindow_ setLevel:0];

[tapted, 2016/12/09 04:02:00]

We should be passing an NSFooWindowLevel to setLevel. E.g. NSNormalWindowLevel,
NSFloatingWindowLevel, etc. There's a `CGWindowLevel.h` which uses the same
constants -- I don't know whether 0/1 will do the right thing  (NSFooWindowLevel
is actually a macro that results in a function call returning some unknown
integer). But! I don't think setLevel is right.

Can we use NSWindow's

- (void)orderWindow:(NSWindowOrderingMode)place 
         relativeTo:(NSInteger)otherWin;

instead?

[Patti Lor, 2016/12/09 05:02:49]

Yeah, I considered this. It adds some complexity because the child windows will
need to be retrieved via the parent window, and there always seems to be 1
mysterious child window (the count is always # of overlay windows + 1). I don't
think it works either, I tried just looping through |overlayWindow_|'s parent's
child windows (excluding overlayWindow_) and calling orderWindow:relativeTo: on
them (passing in [child {level,orderedLevel,windowNumber}], not sure which is
the right one), and it still has the buggy behaviour.

 }
 
 - (void)unhideSheet {
@@ -293,6 +296,9 @@ void ClearTableViewDataSourcesIfNeeded(NSWindow*) {}
   [[sheetWindow contentView] setAutoresizesSubviews:oldResizesSubviews_];
   [sheetWindow setAlphaValue:1.0];
   [sheetWindow setIgnoresMouseEvents:NO];
+
+  // Set the current overlay's level higher than overlays with hidden sheets.
+  [overlayWindow_ setLevel:1];
 }
 
 - (void)pulseSheet {