Implement fullscreen info bubble on Win and Mac

chrome/browser/ui/cocoa/fullscreen_exit_bubble_controller.mm

 // Copyright (c) 2011 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 "base/logging.h"  // for NOTREACHED()
 #include "base/mac/mac_util.h"
 #include "base/sys_string_conversions.h"
+#include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #import "chrome/browser/ui/cocoa/animatable_view.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #include "chrome/browser/ui/cocoa/event_utils.h"
 #import "chrome/browser/ui/cocoa/fullscreen_exit_bubble_controller.h"
 #import "chrome/browser/ui/cocoa/hyperlink_text_view.h"
+#import "chrome/browser/ui/cocoa/info_bubble_view.h"
 #include "grit/generated_resources.h"
 #include "third_party/GTM/AppKit/GTMUILocalizerAndLayoutTweaker.h"
 #import "third_party/GTM/AppKit/GTMNSAnimation+Duration.h"
 #include "ui/base/models/accelerator_cocoa.h"
 #include "ui/base/l10n/l10n_util_mac.h"
 
 const int kPaddingPx = 8;
 const int kInitialDelayMs = 3800;
 const int kSlideOutDurationMs = 700;
 
 @interface FullscreenExitBubbleController (PrivateMethods)
 // Sets |exitLabel_| based on |exitLabelPlaceholder_|,
 // sets |exitLabelPlaceholder_| to nil.
 - (void)initializeLabel;
 
 - (void)hideSoon;
 
 // Returns the Accelerator for the Toggle Fullscreen menu item.
 + (ui::AcceleratorCocoa)acceleratorForToggleFullscreen;
 
 // Returns a string representation fit for display of
 // +acceleratorForToggleFullscreen.
 + (NSString*)keyCommandString;
 
 + (NSString*)keyCombinationForAccelerator:(const ui::AcceleratorCocoa&)item;
 @end
 
 @implementation FullscreenExitBubbleController
 
-- (id)initWithOwner:(BrowserWindowController*)owner browser:(Browser*)browser {
-  if ((self = [super initWithNibName:@"FullscreenExitBubble"
-                              bundle:base::mac::MainAppBundle()])) {
+- (id)initWithOwner:(BrowserWindowController*)owner
+            browser:(Browser*)browser
+             forURL:(const GURL&)url
+        showButtons:(BOOL)show_buttons {
+  NSString* nibPath =
+      [base::mac::MainAppBundle() pathForResource:@"FullscreenExitBubble"
+                                          ofType:@"nib"];
+  if ((self = [super initWithWindowNibPath:nibPath owner:self])) {
     browser_ = browser;
     owner_ = owner;
+    url_ = url;
+    show_buttons_ = show_buttons;
   }
   return self;
 }
 
-- (void)awakeFromNib {
-  [self initializeLabel];
+- (void)allow:(id)sender {
+  [self hideButtons];
+  browser_->OnAcceptFullscreenPermission(url_);
   [self hideSoon];
 }
 
+- (void)deny:(id)sender {
+  browser_->ToggleFullscreenMode(false);
+}
+
+- (void)hideButtons {
+  [allowButton_ setHidden:YES];
+  [denyButton_ setHidden:YES];
+  [exitLabel_ setHidden:NO];
+}
+
+// We want this to be a child of a browser window.  addChildWindow:
+// (called from this function) will bring the window on-screen;
+// unfortunately, [NSWindowController showWindow:] will also bring it
+// on-screen (but will cause unexpected changes to the window's
+// position).  We cannot have an addChildWindow: and a subsequent
+// showWindow:. Thus, we have our own version.
+- (void)showWindow {
+  BrowserWindowController* bwc = owner_;
+  NSWindow* window = [self window];  // completes nib load
+  [bubble_ setArrowLocation:info_bubble::kNoArrow];
+  if (!show_buttons_) {
+    [self hideButtons];
+    [self hideSoon];
+  }
+  CGFloat maxWidth = NSWidth([bwc window].frame);
+  CGFloat maxY = NSHeight([bwc window].frame);

[Scott Hess - ex-Googler, 2011/10/10 23:19:50]

Capture the frame once in an NSRect then use NSWidth() and NSHeight() inline.

[jeremya, 2011/10/11 05:28:48]

Done.

+  [tweaker_ tweakUI:[self window]];
+  [self positionInWindowAtTop:maxY width:maxWidth];
+  [[bwc window] addChildWindow:window ordered:NSWindowAbove];
+
+  [window orderFront:self];
+}
+
+- (void)awakeFromNib {
+  [self initializeLabel];
+  NSString* title =
+      l10n_util::GetNSStringF(IDS_FULLSCREEN_INFOBAR_REQUEST_PERMISSION,
+          UTF8ToUTF16(url_.host()));
+  [siteInfoLabel_ setStringValue:title];
+  [super awakeFromNib];
+}
+
 - (void)positionInWindowAtTop:(CGFloat)maxY width:(CGFloat)maxWidth {
-  NSRect bubbleFrame = [[self view] frame];
-  bubbleFrame.origin.x = (int)(maxWidth/2 - NSWidth(bubbleFrame)/2);
-  bubbleFrame.origin.y = maxY - NSHeight(bubbleFrame);
-  [[self view] setFrame:bubbleFrame];
+  NSPoint origin = [[self window] frame].origin;
+  origin.x = (int)(maxWidth/2 - NSWidth([self window].frame)/2);
+  origin.y = maxY - NSHeight([self window].frame);

[Scott Hess - ex-Googler, 2011/10/10 23:19:50]

Likewise here, capture the frame once then work on it.

[jeremya, 2011/10/11 05:28:48]

Done.

+  origin.y -= fullscreen_exit_bubble::kBubbleOffsetY;
+  [[self window] setFrameOrigin:origin];
 }
 
 // Called when someone clicks on the embedded link.
 - (BOOL) textView:(NSTextView*)textView
     clickedOnLink:(id)link
           atIndex:(NSUInteger)charIndex {
   browser_->ExecuteCommand(IDC_FULLSCREEN);
   return YES;
 }
 
 - (void)hideTimerFired:(NSTimer*)timer {
-  NSRect endFrame = [[self view] frame];
+  [NSAnimationContext beginGrouping];
+  // The star currently triggers on mouse down, not mouse up.
+  [[NSAnimationContext currentContext]
+      gtm_setDuration:0.2  // TODO: Constantify.
+            eventMask:NSLeftMouseUpMask|NSLeftMouseDownMask];
+  [[[self window] animator] setAlphaValue:0.0];
+  [NSAnimationContext endGrouping];
+  return;
+  NSRect endFrame = [[self window] frame];
   endFrame.origin.y += endFrame.size.height;
-  endFrame.size.height = 0;
   NSDictionary* dict = [NSDictionary dictionaryWithObjectsAndKeys:
-      [self view], NSViewAnimationTargetKey,
+      [self window], NSViewAnimationTargetKey,
       [NSValue valueWithRect:endFrame], NSViewAnimationEndFrameKey, nil];
 
   NSViewAnimation* animation =
       [[NSViewAnimation alloc]
         initWithViewAnimations:[NSArray arrayWithObjects:dict, nil]];
   [animation gtm_setDuration:kSlideOutDurationMs/1000.0
                    eventMask:NSLeftMouseUpMask];
   [animation setDelegate:self];
   [animation startAnimation];
   hideAnimation_.reset(animation);
 }
 
 - (void)animationDidEnd:(NSAnimation*)animation {
   if (animation == hideAnimation_.get()) {
     hideAnimation_.reset();
   }
 }
 
 - (AnimatableView*)animatableView {
-  return static_cast<AnimatableView*>([self view]);
+  return static_cast<AnimatableView*>([self window]);

[Scott Hess - ex-Googler, 2011/10/10 23:19:50]

Is this still a valid conversion?

[jeremya, 2011/10/11 05:28:48]

Nope :) Removed.

 }
 
 - (void)dealloc {
   [hideAnimation_.get() stopAnimation];
   [hideTimer_ invalidate];
   [super dealloc];
 }
 
 @end
 
 @implementation FullscreenExitBubbleController (PrivateMethods)
 
 - (void)initializeLabel {
   // Replace the label placeholder NSTextField with the real label NSTextView.
   // The former doesn't show links in a nice way, but the latter can't be added
   // in IB without a containing scroll view, so create the NSTextView
   // programmatically.
   exitLabel_.reset([[HyperlinkTextView alloc]
       initWithFrame:[exitLabelPlaceholder_ frame]]);
   [exitLabel_.get() setAutoresizingMask:
       [exitLabelPlaceholder_ autoresizingMask]];
+  [exitLabel_.get() setHidden:[exitLabelPlaceholder_ isHidden]];
   [[exitLabelPlaceholder_ superview]
       replaceSubview:exitLabelPlaceholder_ with:exitLabel_.get()];
   exitLabelPlaceholder_ = nil;  // Now released.
   [exitLabel_.get() setDelegate:self];
 
   NSString *message = l10n_util::GetNSStringF(IDS_EXIT_FULLSCREEN_MODE,
       base::SysNSStringToUTF16([[self class] keyCommandString]));
 
+  NSFont* font = [NSFont systemFontOfSize:
+      [NSFont systemFontSizeForControlSize:NSRegularControlSize]];

[Scott Hess - ex-Googler, 2011/10/10 23:19:50]

Merge error?

[jeremya, 2011/10/11 05:28:48]

I don't think so... what makes you say that?

[Scott Hess - ex-Googler, 2011/10/11 05:32:30]

Sorry, brace-counting fail across the lines, compounded by excess instances of
"size".

   [(HyperlinkTextView*)exitLabel_.get()
         setMessageAndLink:@""
                  withLink:message
                  atOffset:0
-                     font:[NSFont systemFontOfSize:18]
-             messageColor:[NSColor whiteColor]
-                linkColor:[NSColor whiteColor]];
-
+                     font:font
+             messageColor:[NSColor blackColor]
+                linkColor:[NSColor blueColor]];
+  [exitLabel_.get() setAlignment:NSRightTextAlignment];
+  // TODO: sizeToFit doesn't seem to get quite the right size.
+  [exitLabel_.get() setVerticallyResizable:NO];
+  [exitLabel_.get() setHorizontallyResizable:YES];
   [exitLabel_.get() sizeToFit];
-  NSLayoutManager* layoutManager = [exitLabel_.get() layoutManager];
-  NSTextContainer* textContainer = [exitLabel_.get() textContainer];
-  [layoutManager ensureLayoutForTextContainer:textContainer];
-  NSRect textFrame = [layoutManager usedRectForTextContainer:textContainer];
-  NSRect frame = [[self view] frame];
-  NSSize textSize = textFrame.size;
-  frame.size.width = textSize.width + 2 * kPaddingPx;
-  [[self view] setFrame:frame];
-  textFrame.origin.x = textFrame.origin.y = kPaddingPx;
-  [exitLabel_.get() setFrame:textFrame];
+  NSSize frameSize = [exitLabel_ frame].size;
+  frameSize.width += 30;
+  [exitLabel_.get() setFrameSize:frameSize];
 }
 
 // This looks at the Main Menu and determines what the user has set as the
 // key combination for quit. It then gets the modifiers and builds an object
 // to hold the data.
 + (ui::AcceleratorCocoa)acceleratorForToggleFullscreen {
   NSMenu* mainMenu = [NSApp mainMenu];
   // Get the application menu (i.e. Chromium).
   for (NSMenuItem* menu in [mainMenu itemArray]) {
     for (NSMenuItem* item in [[menu submenu] itemArray]) {
@@ skipping 39 matching lines @@
 - (void)hideSoon {
   hideTimer_.reset(
       [[NSTimer scheduledTimerWithTimeInterval:kInitialDelayMs/1000.0
                                         target:self
                                       selector:@selector(hideTimerFired:)
                                       userInfo:nil
                                        repeats:NO] retain]);
 }
 
 @end