Implement TouchSelectionEditing controls for OOPIF.

content/browser/frame_host/touch_selection_controller_client_child_frame.h

Index: content/browser/frame_host/touch_selection_controller_client_child_frame.h
diff --git a/content/browser/frame_host/touch_selection_controller_client_child_frame.h b/content/browser/frame_host/touch_selection_controller_client_child_frame.h
new file mode 100644
index 0000000000000000000000000000000000000000..f6abe9b01dadf455b74f888cffd6966570637474
--- /dev/null
+++ b/content/browser/frame_host/touch_selection_controller_client_child_frame.h
@@ -0,0 +1,63 @@
+// Copyright 2017 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_FRAME_HOST_INPUT_TOUCH_SELECTION_CONTROLLER_CLIENT_CHILD_FRAME_H_
+#define CONTENT_BROWSER_FRAME_HOST_INPUT_TOUCH_SELECTION_CONTROLLER_CLIENT_CHILD_FRAME_H_

[kenrb, 2017/05/23 16:19:03]

nit: 'INPUT' isn't in the path or the file name.

[wjmaclean, 2017/05/23 17:00:41]

Done.

(Sorry, another bit of update-cruft.)

+
+#include "base/macros.h"
+#include "cc/input/selection.h"
+#include "content/common/content_export.h"
+#include "ui/touch_selection/touch_selection_controller.h"
+#include "ui/touch_selection/touch_selection_menu_runner.h"
+
+namespace content {
+class RenderWidgetHostViewChildFrame;

[kenrb, 2017/05/23 16:19:03]

I'm not sure about this, but it might be something to run by a content/ owner:
It might be better to put this class in content/browser/renderer_host, and add
an exception to content/browser/renderer_host/DEPS to allow the .cc file to
include RenderWidgetHostViewChildFrame. RWHVCF really belongs in renderer_host,
but nobody has gotten around to moving it because we have to set up some kind of
delegation to CrossProcessFrameConnector. I think it would be better to start
keeping files like this in renderer_host so we don't end up with more improper
dependencies to untangle later on.

[wjmaclean, 2017/05/23 17:00:41]

creis@ - before I move this, can you weigh in please?

[Charlie Reis, 2017/05/23 18:39:51]

I was wondering exactly this when I glanced at the CL earlier, but then saw
RWHVCF is in frame_host as well.  I'm glad to get the context from Ken, and now
I see there's a TODO for moving RWHVCF to renderer_host in renderer_host/DEPS. 
As a result, I agree with Ken's recommendation, but I'd ask for a bug to be
filed and listed on the TODOs in the DEPS file.  Thanks!

[wjmaclean, 2017/05/24 17:10:08]

Done.

+class TouchSelectionControllerClientManager;
+
+// An implementation of |TouchSelectionControllerClient| to be used in Aura's

[kenrb, 2017/05/23 16:19:02]

nit: This shouldn't be Aura-specific, even though it is currently only wired to
work on Aura. I would expect this class to work on any platform where the root
RWHV returns a TouchSelectionControllerClientManager.

[wjmaclean, 2017/05/23 17:00:41]

Done.

+// implementation of touch selection for contents.
+class CONTENT_EXPORT TouchSelectionControllerClientChildFrame
+    : public ui::TouchSelectionControllerClient,
+      public ui::TouchSelectionMenuClient {
+ public:
+  TouchSelectionControllerClientChildFrame(
+      RenderWidgetHostViewChildFrame* rwhv,
+      TouchSelectionControllerClientManager* manager);
+  ~TouchSelectionControllerClientChildFrame() override;
+
+  void UpdateSelectionBoundsIfNeeded(
+      const cc::Selection<gfx::SelectionBound>& selection,
+      float device_scale_factor);
+
+ private:
+  // ui::TouchSelectionControllerClient:
+  bool SupportsAnimation() const override;
+  void SetNeedsAnimate() override;
+  void MoveCaret(const gfx::PointF& position) override;
+  void MoveRangeSelectionExtent(const gfx::PointF& extent) override;
+  void SelectBetweenCoordinates(const gfx::PointF& base,
+                                const gfx::PointF& extent) override;
+  void OnSelectionEvent(ui::SelectionEventType event) override;
+  std::unique_ptr<ui::TouchHandleDrawable> CreateDrawable() override;
+
+  // ui::TouchSelectionMenuClient:
+  bool IsCommandIdEnabled(int command_id) const override;
+  void ExecuteCommand(int command_id, int event_flags) override;
+  void RunContextMenu() override;
+
+  gfx::Point ConvertFromRoot(const gfx::PointF& point) const;
+
+  // Not owned, non-null for the lifetime of this object.
+  RenderWidgetHostViewChildFrame* rwhv_;
+  TouchSelectionControllerClientManager* manager_;
+
+  // The last selection bounds reported by the view.
+  gfx::SelectionBound selection_start_;
+  gfx::SelectionBound selection_end_;
+
+  DISALLOW_COPY_AND_ASSIGN(TouchSelectionControllerClientChildFrame);
+};
+
+}  // namespace content
+#endif  // CONTENT_BROWSER_FRAME_HOST_INPUT_TOUCH_SELECTION_CONTROLLER_CLIENT_CHILD_FRAME_H_