[OSX, OmniTheatre] Handle OriginChip click properly.

chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_cell.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 "chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_cell.h"
 
 #include "base/logging.h"
 #include "base/mac/foundation_util.h"
 #include "base/mac/mac_logging.h"
 #include "chrome/browser/search/search.h"
@@ skipping 147 matching lines @@
   std::reverse(decorations->begin() + left_count, decorations->end());
   std::reverse(decoration_frames->begin() + left_count,
                decoration_frames->end());
 
   *remaining_frame = frame;
   return left_count;
 }
 
 }  // namespace
 
+@interface AutocompleteTextFieldCell ()
+// Post an OnSetFocus notification to the observer of |controlView|.
+- (void)focusNotificationFor:(NSEvent*)event
+                      ofView:(AutocompleteTextField*)controlView;
+@end
+
 @implementation AutocompleteTextFieldCell
 
 @synthesize isPopupMode = isPopupMode_;
 
 - (CGFloat)topTextFrameOffset {
   return 3.0;
 }
 
 - (CGFloat)bottomTextFrameOffset {
   return 3.0;
@@ skipping 226 matching lines @@
   LocationBarDecoration* decoration =
       [self decorationForEvent:theEvent inRect:cellFrame ofView:controlView];
   if (decoration)
     return decoration->GetMenu();
   return nil;
 }
 
 - (BOOL)mouseDown:(NSEvent*)theEvent
            inRect:(NSRect)cellFrame
            ofView:(AutocompleteTextField*)controlView {
+  // Check if this mouseDown was the reason the control became firstResponder.
+  // If not, discard focus event.
+  base::scoped_nsobject<NSEvent> focusEvent(focusEvent_.release());
+  if (![theEvent isEqual:focusEvent_])

[Scott Hess - ex-Googler, 2014/02/27 21:09:36]

focusEvent (no _), focusEvent_ is nil.

+    focusEvent.reset();
+
   LocationBarDecoration* decoration =
       [self decorationForEvent:theEvent inRect:cellFrame ofView:controlView];
   if (!decoration || !decoration->AcceptsMousePress())
     return NO;
 
   NSRect decorationRect =
       [self frameForDecoration:decoration inFrame:cellFrame];
 
   // If the decoration is draggable, then initiate a drag if the user
   // drags or holds the mouse down for awhile.
@@ skipping 50 matching lines @@
 
     button->SetButtonState(ButtonDecoration::kButtonStatePressed);
     [controlView setNeedsDisplay:YES];
 
     // Track the mouse until the user releases the button.
     [self trackMouse:theEvent
               inRect:cellFrame
               ofView:controlView
         untilMouseUp:YES];
 
+    // Post delayed focus notification, if necessary.
+    [self focusNotificationFor:focusEvent ofView:controlView];

[Scott Hess - ex-Googler, 2014/02/27 21:09:36]

Yeah, I think so.  Nothing has occurred to me about clean ways to not focus in
the current design, probably it would have to just integrate a real button
somehow.  Or, possibly, a multi-purpose sub-view which exists only to intercept
specific regions.

[groby-ooo-7-16, 2014/02/27 23:15:27]

Well, we need to accept firstResponder for the top level view, since we _do_
want mouseDown's for button clicks. The bigger issue is probably that upstream
code wants to hide buttons in OnFocus(). Breaking up in OnFocus/OnReadyToEdit
might be another way to tackle that, but I don't want to bloat the top-level API
either.

+
     // Set the proper state (hover or normal) once the mouse has been released,
     // and call |OnMousePressed| if the button was released while the mouse was
     // within the bounds of the button.
     const NSPoint mouseLocation =
         [[controlView window] mouseLocationOutsideOfEventStream];
     const NSPoint point = [controlView convertPoint:mouseLocation fromView:nil];
     if (NSMouseInRect(point, cellFrame, [controlView isFlipped])) {
       button->SetButtonState(ButtonDecoration::kButtonStateHover);
       [controlView setNeedsDisplay:YES];
       handled = decoration->AsButtonDecoration()->OnMousePressed(
@@ skipping 306 matching lines @@
   [controlView setNeedsDisplay:YES];
   NSTextView* fieldEditor =
       base::mac::ObjCCastStrict<NSTextView>([controlView currentEditor]);
   [fieldEditor updateInsertionPointStateAndRestartTimer:YES];
 }
 
 - (BOOL)showsFirstResponder {
   return [super showsFirstResponder] && !hideFocusState_;
 }
 
+- (void)focusNotificationFor:(NSEvent*)event
+                      ofView:(AutocompleteTextField*)controlView {
+  if ([controlView observer] && event) {
+    const bool controlDown = ([event modifierFlags] & NSControlKeyMask) != 0;
+    [controlView observer]->OnSetFocus(controlDown);
+  }
+}

[Scott Hess - ex-Googler, 2014/02/27 21:09:36]

I can't tell if the following is clearer or not, so I'm just throwing it out
there to potentially be ignored.  I certainly don't feel strongly about it (I
suspect this all is six-or-half-a-dozen in terms of subtleness).

If the code at the top of -mouseDown: were removed, then event here would always
be non-nil.  So it could be:

if ([controlView observer] && (!focusEvent_ || [focusEvent_ isEqual:event])) {
  // OnSetFocus().
}
focusEvent_.reset();

Then these two methods which are close to each other are the sole source of this
knowledge.

[groby-ooo-7-16, 2014/02/27 23:15:27]

Except this is only called from button decorations.  I could call it from a few
other places to take care of non-button-decorations, but that's equally
confusing. I suppose what I really want is factor out mouseDown, to

target = [self decorationForEvent:event];
[self postDelayedFocusNotificationForEvent:event];
[self mouseDownForDecoration:target];

I'll leave a TODO() in there - it seems the "done thing" in Omnibox ;)

+
+- (void)handleFocusEvent:(NSEvent*)event
+                  ofView:(AutocompleteTextField*)controlView {
+  // Only intercept left button click. All other events cause immediate focus.
+  if ([event type] == NSLeftMouseDown) {
+    LocationBarDecoration* decoration =
+        [self decorationForEvent:event
+                          inRect:[controlView bounds]
+                          ofView:controlView];
+    // Only ButtonDecorations need a delayed focus handling.
+    if (decoration && decoration->AsButtonDecoration()) {
+      focusEvent_.reset([event retain]);
+      return;
+    }
+  }
+
+  // Handle event immediately.
+  [self focusNotificationFor:event ofView:controlView];
+}
+
 @end