Make EditorCommand to use VisibleSelection::toNormalizedEphemeralRange() instead of toNormalizedRang

Source/core/editing/commands/EditorCommand.cpp

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2009 Igalia S.L.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 376 matching lines @@
 static bool executeDeleteToEndOfParagraph(LocalFrame& frame, Event*, EditorCommandSource, const String&)
 {
     // Despite its name, this command should delete the newline at the end of
     // a paragraph if you are at the end of a paragraph.
     frame.editor().deleteWithDirection(DirectionForward, ParagraphBoundary, true, false);
     return true;
 }
 
 static bool executeDeleteToMark(LocalFrame& frame, Event*, EditorCommandSource, const String&)
 {
-    // TODO(yosin) We should use |EphemeralRange| version of
-    // |VisibleSelection::toNormalizedRange()|.
-    RefPtrWillBeRawPtr<Range> mark = frame.editor().mark().toNormalizedRange();
-    if (mark) {
-        bool selected = frame.selection().setSelectedRange(unionEphemeralRanges(EphemeralRange(mark.get()), frame.editor().selectedRange()), TextAffinity::Downstream, FrameSelection::NonDirectional, FrameSelection::CloseTyping);
+    const EphemeralRange mark = frame.editor().mark().toNormalizedEphemeralRange();
+    if (mark.isNotNull()) {
+        bool selected = frame.selection().setSelectedRange(unionEphemeralRanges(mark, frame.editor().selectedRange()), TextAffinity::Downstream, FrameSelection::NonDirectional, FrameSelection::CloseTyping);
         ASSERT(selected);
         if (!selected)
             return false;
     }
     frame.editor().performDelete();
     frame.editor().setMark(frame.selection().selection());
     return true;
 }
 
 static bool executeDeleteWordBackward(LocalFrame& frame, Event*, EditorCommandSource, const String&)
@@ skipping 623 matching lines @@
     return expandSelectionToGranularity(frame, ParagraphGranularity);
 }
 
 static bool executeSelectSentence(LocalFrame& frame, Event*, EditorCommandSource, const String&)
 {
     return expandSelectionToGranularity(frame, SentenceGranularity);
 }
 
 static bool executeSelectToMark(LocalFrame& frame, Event*, EditorCommandSource, const String&)
 {
-    // TODO(yosin) We should use |EphemeralRange| version of
-    // |VisibleSelection::toNormalizedRange()|.
-    RefPtrWillBeRawPtr<Range> mark = frame.editor().mark().toNormalizedRange();
+    const EphemeralRange mark = frame.editor().mark().toNormalizedEphemeralRange();
     EphemeralRange selection = frame.editor().selectedRange();
-    if (!mark || selection.isNull())
+    if (mark.isNull() || selection.isNull())
         return false;
-    frame.selection().setSelectedRange(unionEphemeralRanges(EphemeralRange(mark.get()), selection), TextAffinity::Downstream, FrameSelection::NonDirectional, FrameSelection::CloseTyping);
+    frame.selection().setSelectedRange(unionEphemeralRanges(mark, selection), TextAffinity::Downstream, FrameSelection::NonDirectional, FrameSelection::CloseTyping);
     return true;
 }
 
 static bool executeSelectWord(LocalFrame& frame, Event*, EditorCommandSource, const String&)
 {
     return expandSelectionToGranularity(frame, WordGranularity);
 }
 
 static bool executeSetMark(LocalFrame& frame, Event*, EditorCommandSource, const String&)
 {
@@ skipping 729 matching lines @@
 {
     return m_command && m_command->isTextInsertion;
 }
 
 int Editor::Command::idForHistogram() const
 {
     return isSupported() ? m_command->idForUserMetrics : 0;
 }
 
 } // namespace blink