Copying text closes the keyboard and the text input gets unfocused, forcing virtual keyboard is get…

Source/web/WebLocalFrameImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 1824 matching lines @@
         // Finally clear the active match, for two reasons:
         // We just finished the find 'session' and we don't want future (potentially
         // unrelated) find 'sessions' operations to start at the same place.
         // The WebLocalFrameImpl could get reused and the activeMatch could end up pointing
         // to a document that is no longer valid. Keeping an invalid reference around
         // is just asking for trouble.
         m_textFinder->resetActiveMatch();
     }
 }
 
+bool WebLocalFrameImpl::setCaretSelectionFromRangeSelection()
+{
+    if (hasSelection()) {

[yosin_UTC9, 2014/06/13 04:20:15]

nit: We prefer early return, e.g.

if (!hasSelection())
  return false;

....

+        // Check whether Selection exists
+        FrameSelection& selection = frame()->selection();
+        if (selection.isInTextField() && selection.isRange()) {

[yosin_UTC9, 2014/06/13 04:20:15]

We may want not lose selection for contenteditable and TEXTAREA? It seems
isInTextFiled() doesn't return true for TEXTAREA.

How about |if (selection.rootEditablElement() && selection.isRange())|?

[AKVT, 2014/06/16 10:34:34]

Done.

[AKVT, 2014/06/16 10:34:34]

Done.

+            // Check whether it's a range selection. It appears to be reduandant.

[yosin_UTC9, 2014/06/13 04:20:15]

It seems Selection.collapseToEnd() == selection().moveTo(selection.end()) is
enough.

[AKVT, 2014/06/16 10:34:34]

Done.

+            // During copy, Range selection should be available. For safety purpse added this condition.

[Yuta Kitamura, 2014/06/16 07:58:43]

typo: purpse

[AKVT, 2014/06/16 10:34:34]

Done.

+            RefPtrWillBeRawPtr<Range> selectedRange = selection.toNormalizedRange();
+            if (!selectedRange)
+                return false;
+
+            int startOffset = selectedRange->startOffset();
+            int endOffset = selectedRange->endOffset();
+            if (startOffset != endOffset) {
+                // Convert to Caret selection without clearing selection to allow continouous editing after copy.
+                frame()->inputMethodController().setEditableSelectionOffsets(PlainTextRange(endOffset, endOffset));
+                return true;
+            }
+        }
+    }
+
+    return false;
+}
+
 void WebLocalFrameImpl::didFail(const ResourceError& error, bool wasProvisional)
 {
     if (!client())
         return;
     WebURLError webError = error;
     if (wasProvisional)
         client()->didFailProvisionalLoad(this, webError);
     else
         client()->didFailLoad(this, webError);
 }
@@ skipping 91 matching lines @@
 
 void WebLocalFrameImpl::invalidateAll() const
 {
     ASSERT(frame() && frame()->view());
     FrameView* view = frame()->view();
     view->invalidateRect(view->frameRect());
     invalidateScrollbar();
 }
 
 } // namespace blink