Instant Extended: Suppress mouse events for bottom web content

content/browser/renderer_host/render_widget_host_view_mac.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.
 
 #include "content/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include <QuartzCore/QuartzCore.h>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 1710 matching lines @@
   NSView* view = [contentView hitTest:[theEvent locationInWindow]];
   // Traverse the superview hierarchy as the hitTest will return the frontmost
   // view, such as an NSTextView, while nonWebContentView may be specified by
   // its parent view.
   while (view) {
     if ([view respondsToSelector:nonWebContentViewSelector] &&
         [view performSelector:nonWebContentViewSelector]) {
       // The cursor is over a nonWebContentView - ignore this mouse event.
       return YES;
     }
+    if ([view isKindOfClass:[self class]] && ![view isEqual:self]) {

[Alexei Svitkine (slow), 2013/05/22 03:34:43]

Is there a reason to use -isEqual: here rather than just ==?

[sail, 2013/05/22 17:54:10]

This a normal Objective-C convention. For example, if you call -[NSArray
indexOfObject:] it will use isEqual: not ==.

+      // The cursor is over an overlapping render widget.

[Alexei Svitkine (slow), 2013/05/22 03:34:43]

Please expand the comment to mention that given both views will be doing this
check, that events will be processed by one returned by -hitTest:.

[sail, 2013/05/22 17:54:10]

Done.

+      return YES;
+    }
     view = [view superview];
   }
   return NO;
 }
 
 - (void)mouseEvent:(NSEvent*)theEvent {
   TRACE_EVENT0("browser", "RenderWidgetHostViewCocoa::mouseEvent");
   if (delegate_ && [delegate_ respondsToSelector:@selector(handleEvent:)]) {
     BOOL handled = [delegate_ handleEvent:theEvent];
     if (handled)
@@ skipping 1649 matching lines @@
   if (!string) return NO;
 
   // If the user is currently using an IME, confirm the IME input,
   // and then insert the text from the service, the same as TextEdit and Safari.
   [self confirmComposition];
   [self insertText:string];
   return YES;
 }
 
 @end