views::Textfield: Implement yank editing command.

ui/views/views_delegate.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 UI_VIEWS_VIEWS_DELEGATE_H_
 #define UI_VIEWS_VIEWS_DELEGATE_H_
 
 #include <string>
 
 #if defined(OS_WIN)
@@ skipping 84 matching lines @@
 
   // Call this method to set a factory callback that will be used to construct
   // NativeWidget implementations overriding the platform defaults.
   void set_native_widget_factory(NativeWidgetFactory factory) {
     native_widget_factory_ = factory;
   }
   const NativeWidgetFactory& native_widget_factory() {
     return native_widget_factory_;
   }
 
+  const base::string16& kill_buffer() {

[tapted, 2016/07/20 06:32:21]

nit: declare method const

[karandeepb, 2016/07/20 07:51:30]

Done.

+    return kill_buffer_;
+  }
+
+  void set_kill_buffer(const base::string16& kill_buffer) {

[tapted, 2016/07/20 06:32:21]

nit: swap these around and remove the blank line between to be consistent with
native_widget_factory. It should have a brief comment too, like "Access the text
buffer used to implement the "Yank" text edit command."

[karandeepb, 2016/07/20 07:51:30]

Done.

+    kill_buffer_ = kill_buffer;
+  }
+
   // Saves the position, size and "show" state for the window with the
   // specified name.
   virtual void SaveWindowPlacement(const Widget* widget,
                                    const std::string& window_name,
                                    const gfx::Rect& bounds,
                                    ui::WindowShowState show_state);
 
   // Retrieves the saved position and size and "show" state for the window with
   // the specified name.
   virtual bool GetSavedWindowPlacement(const Widget* widget,
@@ skipping 83 matching lines @@
 
  private:
   std::unique_ptr<ViewsTouchEditingControllerFactory> views_tsc_factory_;
 
 #if defined(USE_AURA)
   std::unique_ptr<TouchSelectionMenuRunnerViews> touch_selection_menu_runner_;
 #endif
 
   NativeWidgetFactory native_widget_factory_;
 
+  // Buffer containing the text to be inserted on executing yank command for
+  // Views::Texfield. Member of ViewsDelegate since this needs to be persisted

[tapted, 2016/07/20 06:32:21]

nit: Views -> views

[karandeepb, 2016/07/20 07:51:30]

Done.

+  // across multiple textfield instances.
+  // On Mac, the size of the kill ring (no. of buffers) is controlled by
+  // NSTextKillRingSize, a text system default. However to keep things simple,
+  // the default kill ring size of 1 (i.e. a single buffer) is assumed.
+  base::string16 kill_buffer_;
+
   DISALLOW_COPY_AND_ASSIGN(ViewsDelegate);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_VIEWS_DELEGATE_H_