Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(268)

Unified Diff: ui/touch_selection/touch_selection_controller.h

Issue 2201853002: Blink handle selection handle visibility (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: default handle visibility should be false Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/touch_selection/touch_selection_controller.h
diff --git a/ui/touch_selection/touch_selection_controller.h b/ui/touch_selection/touch_selection_controller.h
index bbffc87c80dd030f7785673f2778f492a29fe98c..e102ac59f617c6d260617eb139ba9e452555df40 100644
--- a/ui/touch_selection/touch_selection_controller.h
+++ b/ui/touch_selection/touch_selection_controller.h
@@ -106,10 +106,6 @@ class UI_TOUCH_SELECTION_EXPORT TouchSelectionController
// long-press drag.
void OnScrollBeginEvent();
- // Allow showing the selection handles from the most recent selection bounds
- // update (if valid), or a future valid bounds update.
- void AllowShowingFromCurrentSelection();
-
// Hide the handles and suppress bounds updates until the next explicit
// showing allowance.
void HideAndDisallowShowingAutomatically();
@@ -147,10 +143,6 @@ class UI_TOUCH_SELECTION_EXPORT TouchSelectionController
ActiveStatus active_status() const { return active_status_; }
- bool insertion_active_or_requested() const {
- return activate_insertion_automatically_;
- }
-
private:
friend class TouchSelectionControllerTestApi;
@@ -174,8 +166,6 @@ class UI_TOUCH_SELECTION_EXPORT TouchSelectionController
gfx::PointF GetSelectionStart() const override;
gfx::PointF GetSelectionEnd() const override;
- void ShowInsertionHandleAutomatically();
- void ShowSelectionHandlesAutomatically();
bool WillHandleTapOrLongPress(const gfx::PointF& location);
void OnInsertionChanged();
@@ -219,11 +209,9 @@ class UI_TOUCH_SELECTION_EXPORT TouchSelectionController
ActiveStatus active_status_;
std::unique_ptr<TouchHandle> insertion_handle_;
- bool activate_insertion_automatically_;
std::unique_ptr<TouchHandle> start_selection_handle_;
std::unique_ptr<TouchHandle> end_selection_handle_;
- bool activate_selection_automatically_;
bool selection_empty_;
bool selection_editable_;

Powered by Google App Engine
This is Rietveld 408576698