Update TextSelection for non-user initiated events

content/public/renderer/render_frame.h

 // Copyright 2013 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_PUBLIC_RENDERER_RENDER_FRAME_H_
 #define CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_
 
 #include <stddef.h>
 
 #include <memory>
@@ skipping 209 matching lines @@
   // content created by the embedder.
   virtual void AttachGuest(int element_instance_id) = 0;
 
   // Detaches the browser plugin identified by |element_instance_id| from guest
   // content created by the embedder.
   virtual void DetachGuest(int element_instance_id) = 0;
 
   // Notifies the browser of text selection changes made.
   virtual void SetSelectedText(const base::string16& selection_text,
                                size_t offset,
-                               const gfx::Range& range) = 0;
+                               const gfx::Range& range,
+                               bool user_initiated) = 0;

[nasko, 2017/05/04 16:10:24]

It looks like this API is never called outside of content/ code. Let's remove it
from the public API and leave it internal.

[Peter Varga, 2017/05/04 16:33:45]

Shouldn't this be done in another patch? This change doesn't really related to
the original issue.

 
   // Ensures that builtin mojo bindings modules are available in |context|.
   virtual void EnsureMojoBuiltinsAreAvailable(
       v8::Isolate* isolate,
       v8::Local<v8::Context> context) = 0;
 
   // Adds |message| to the DevTools console.
   virtual void AddMessageToConsole(ConsoleMessageLevel level,
                                    const std::string& message) = 0;
 
@@ skipping 26 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class RenderFrameImpl;
   RenderFrame() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_