mac: Makeshift UI for scroll feedback.

chrome/browser/renderer_host/render_widget_host_view_mac.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.
 
 #include "chrome/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include <QuartzCore/QuartzCore.h>
 
 #include <cmath>
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/mac/mac_util.h"
 #include "base/mac/scoped_cftyperef.h"
 #import "base/mac/scoped_nsautorelease_pool.h"
 #include "base/metrics/histogram.h"
 #import "base/memory/scoped_nsobject.h"
 #include "base/string_util.h"
 #include "base/sys_info.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/browser/browser_trial.h"
 #include "chrome/browser/mac/closure_blocks_leopard_compat.h"
 #import "chrome/browser/renderer_host/accelerated_plugin_view_mac.h"
 #import "chrome/browser/renderer_host/text_input_client_mac.h"
 #include "chrome/browser/spellchecker/spellchecker_platform_engine.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
+#import "chrome/browser/ui/cocoa/history_overlay_controller.h"
 #import "chrome/browser/ui/cocoa/rwhvm_editcommand_helper.h"
 #import "chrome/browser/ui/cocoa/view_id_util.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/spellcheck_messages.h"
 #import "content/browser/accessibility/browser_accessibility_cocoa.h"
 #include "content/browser/browser_thread.h"
 #include "content/browser/gpu/gpu_process_host.h"
 #include "content/browser/gpu/gpu_process_host_ui_shim.h"
 #include "content/browser/plugin_process_host.h"
 #include "content/browser/renderer_host/backing_store_mac.h"
@@ skipping 1542 matching lines @@
         // If "forward" is inactive and the user rubber-bands to the right,
         // "isPinnedLeft" will be false.  When the user then rubber-bands to
         // the left in the same gesture, that should trigger history
         // immediately if there's no scrollbar, hence the check for
         // hasHorizontalScrollbar_.
         (!hasHorizontalScrollbar_ ||
         // One would think we have to check canGoBack / canGoForward here, but
         // that's actually done in the renderer
         // (ChromeClientImpl::shouldRubberBand()), when it decides if it should
         // rubberband or send back an event unhandled.
+        // FIXME: Is this true with the UI up?
             (isPinnedLeft_ && !isRightScroll) ||
             (isPinnedRight_ && isRightScroll))) {
-      // The way this api works: gestureAmount is between -1 and 1 (float). If
+
+      // Released by the tracking handler once the gesture is complete.
+      HistoryOverlayController* historyOverlay =

[Alexei Svitkine (slow), 2011/08/16 14:10:04]

I was going to ask you about this - but Mark beat me to it by trying it out and
noticing the issue with multiple windows being created.

You probably want to keep a scoped ptr to the history controller so the already
open window can be re-used (or maybe just closed) if its still open.

+          [[HistoryOverlayController alloc]
+            initForMode:goForward ? kHistoryOverlayModeForward :
+                                    kHistoryOverlayModeBack];
+
+      // The way this api works: gestureAmount is between -1 and 1 (float).  If
       // the user does the gesture for more than about 25% (i.e. < -0.25 or >
       // 0.25) and then lets go, it is accepted, we get a NSEventPhaseEnded,
       // and after that the block is called with amounts animating towards 1
       // (or -1, depending on the direction).  If the user lets go below that
       // threshold, we get NSEventPhaseCancelled, and the amount animates
-      // toward 0.
+      // toward 0.  When gestureAmount has reaches its final value, i.e. the
+      // track animation is done, the handler is called with |isComplete| set
+      // to |YES|.
       [theEvent trackSwipeEventWithOptions:0
-                  dampenAmountThresholdMin:-1
-                                       max:1
+                  dampenAmountThresholdMin:-50

[Mark Mentovai, 2011/08/16 05:47:22]

Why the change to ±50 here? Learn anything you’re not sharing?

[Nico, 2011/08/16 15:55:43]

Hah, no. I played with this and couldn't really see a difference (if I make the
number 0.0001, then the swipe feels very slow, but 1 and 50 feel mostly the
same). I'll change this back to 1 before landing.

+                                       max:50
                               usingHandler:^(CGFloat gestureAmount,
                                              NSEventPhase phase,
                                              BOOL isComplete,
                                              BOOL *stop) {
+          if (phase == NSEventPhaseBegan) {
+            [historyOverlay showPanelForWindow:[self window]];
+            return;
+          }
+
           // |gestureAmount| obeys -[NSEvent isDirectionInvertedFromDevice]
           // automatically.
-          // TODO(thakis): UI.
           Browser* browser = BrowserList::GetLastActive();
           if (phase == NSEventPhaseEnded && browser) {
             if (goForward)
               browser->GoForward(CURRENT_TAB);
             else
               browser->GoBack(CURRENT_TAB);
           }
+
+          [historyOverlay setProgress:gestureAmount];
+          if (isComplete) {
+            [historyOverlay dismiss];
+            [historyOverlay release];
+          }
         }];
       return YES;
     }
   }
   return NO;
 }
 
 - (void)scrollWheel:(NSEvent*)theEvent {
   [self cancelChildPopups];
 
@@ skipping 1261 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