Remove function wrappers in DOMSelection
There is no point in doing the feature detection on every call, unless there is an IE bug where it is not sure about which support for selection it has (totally plausible).
Christoph Pojer committed
Apr 18, 2014 at 12:47 UTC
9a13393ce367a06258f0ee2b155b02493632d472
1 file changed
+2
-8
src/browser/ui/ReactDOMSelection.js
+2
-8
@@ -171,19 +171,13 @@ var ReactDOMSelection = {
171
/**
172
* @param {DOMElement} node
173
*/
174
- getOffsets: function(node) {
175
- var getOffsets = document.selection ? getIEOffsets : getModernOffsets;
176
- return getOffsets(node);
177
- },
174
+ getOffsets: document.selection ? getIEOffsets : getModernOffsets,
175
176
/**
177
* @param {DOMElement|DOMTextNode} node
178
* @param {object} offsets
179
*/
183
- setOffsets: function(node, offsets) {
184
- var setOffsets = document.selection ? setIEOffsets : setModernOffsets;
185
- setOffsets(node, offsets);
186
- }
180
+ setOffsets: document.selection ? setIEOffsets : setModernOffsets
181
};
182
183
module.exports = ReactDOMSelection;