[InputEvent] Introduce |StaticRange| and use in |InputEvent::getRanges()|

third_party/WebKit/Source/core/events/InputEvent.cpp

Index: third_party/WebKit/Source/core/events/InputEvent.cpp
diff --git a/third_party/WebKit/Source/core/events/InputEvent.cpp b/third_party/WebKit/Source/core/events/InputEvent.cpp
index c485ea8057f2c7e1f1ab98def97e89f7c00d5be5..9154fc74b89a8d5dde3a1589d25ad883670f8d98 100644
--- a/third_party/WebKit/Source/core/events/InputEvent.cpp
+++ b/third_party/WebKit/Source/core/events/InputEvent.cpp
@@ -90,6 +90,15 @@ String InputEvent::inputType() const
     return convertInputTypeToString(m_inputType);
 }
 
+StaticRangeVector InputEvent::getRanges() const
+{
+    StaticRangeVector staticRanges;
+    for (const auto& range : m_ranges) {

[dtapuska, 2016/06/02 15:26:29]

brackets not needed.

[chongz, 2016/06/06 19:43:44]

Done.

+        staticRanges.append(StaticRange::create(range->startContainer(), range->startOffset(), range->endContainer(), range->endOffset()));
+    }
+    return staticRanges;

[dtapuska, 2016/06/02 15:26:29]

Does this create a new object; or should std::move be used here... Hopefully the
compiler is being smart.

[chongz, 2016/06/06 19:43:43]

According to go/totw/11 I'd assume compiler is smart enough to optimize it?

+}
+
 bool InputEvent::isInputEvent() const
 {
     return true;