Consistent fading behavior for touch editing handles

content/browser/renderer_host/render_widget_host_view_aura.h

 // 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.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_AURA_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_AURA_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 102 matching lines @@
   };
 
   // Displays and controls touch editing elements such as selection handles.
   class TouchEditingClient {
    public:
     TouchEditingClient() {}
 
     // Tells the client to start showing touch editing handles.
     virtual void StartTouchEditing() = 0;
 
-    // Notifies the client that touch editing is no longer needed.
-    virtual void EndTouchEditing() = 0;
+    // Notifies the client that touch editing is no longer needed. |quick|
+    // determines whether the handles should fade out quickly or slowly.
+    virtual void EndTouchEditing(bool quick) = 0;

[sky, 2014/01/16 23:15:02]

If !quick means no animation, how about renaming to animate? |animate| is a lot
clearer as to what is going to happen.

[mohsen, 2014/01/17 02:09:41]

Actually it was first named |animate|. But, since it still has a super quick
animation, I renamed it to |quick|. If you think that we can consider this super
quick animation as "no animation", then I can rename it back to |animate|.

 
     // Notifies the client that the selection bounds need to be updated.
     virtual void OnSelectionOrCursorChanged(const gfx::Rect& anchor,
                                             const gfx::Rect& focus) = 0;
 
     // Notifies the client that the current text input type as changed.
     virtual void OnTextInputTypeChanged(ui::TextInputType type) = 0;
 
     // Notifies the client that an input event is about to be sent to the
     // renderer. Returns true if the client wants to stop event propagation.
@@ skipping 666 matching lines @@
   // passed to windowless plugins like Flash/Silverlight, etc as the
   // container window.
   HWND plugin_parent_window_;
 #endif
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewAura);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_AURA_H_