diff options
author | Lorry Tar Creator <lorry-tar-importer@lorry> | 2017-06-27 06:07:23 +0000 |
---|---|---|
committer | Lorry Tar Creator <lorry-tar-importer@lorry> | 2017-06-27 06:07:23 +0000 |
commit | 1bf1084f2b10c3b47fd1a588d85d21ed0eb41d0c (patch) | |
tree | 46dcd36c86e7fbc6e5df36deb463b33e9967a6f7 /Source/WebCore/editing/SplitTextNodeCommand.cpp | |
parent | 32761a6cee1d0dee366b885b7b9c777e67885688 (diff) | |
download | WebKitGtk-tarball-master.tar.gz |
webkitgtk-2.16.5HEADwebkitgtk-2.16.5master
Diffstat (limited to 'Source/WebCore/editing/SplitTextNodeCommand.cpp')
-rw-r--r-- | Source/WebCore/editing/SplitTextNodeCommand.cpp | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/Source/WebCore/editing/SplitTextNodeCommand.cpp b/Source/WebCore/editing/SplitTextNodeCommand.cpp index e7108f514..b0a4b21f9 100644 --- a/Source/WebCore/editing/SplitTextNodeCommand.cpp +++ b/Source/WebCore/editing/SplitTextNodeCommand.cpp @@ -10,10 +10,10 @@ * notice, this list of conditions and the following disclaimer in the * documentation and/or other materials provided with the distribution. * - * THIS SOFTWARE IS PROVIDED BY APPLE COMPUTER, INC. ``AS IS'' AND ANY + * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR - * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE COMPUTER, INC. OR + * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR @@ -54,11 +54,14 @@ void SplitTextNodeCommand::doApply() if (!parent || !parent->hasEditableStyle()) return; - String prefixText = m_text2->substringData(0, m_offset, IGNORE_EXCEPTION); + auto result = m_text2->substringData(0, m_offset); + if (result.hasException()) + return; + auto prefixText = result.releaseReturnValue(); if (prefixText.isEmpty()) return; - m_text1 = Text::create(document(), prefixText); + m_text1 = Text::create(document(), WTFMove(prefixText)); ASSERT(m_text1); document().markers().copyMarkers(m_text2.get(), 0, m_offset, m_text1.get(), 0); @@ -74,10 +77,10 @@ void SplitTextNodeCommand::doUnapply() String prefixText = m_text1->data(); - m_text2->insertData(0, prefixText, ASSERT_NO_EXCEPTION); + m_text2->insertData(0, prefixText); document().markers().copyMarkers(m_text1.get(), 0, prefixText.length(), m_text2.get(), 0); - m_text1->remove(ASSERT_NO_EXCEPTION); + m_text1->remove(); } void SplitTextNodeCommand::doReapply() @@ -94,19 +97,19 @@ void SplitTextNodeCommand::doReapply() void SplitTextNodeCommand::insertText1AndTrimText2() { - ExceptionCode ec = 0; - m_text2->parentNode()->insertBefore(m_text1.get(), m_text2.get(), ec); - if (ec) + if (m_text2->parentNode()->insertBefore(*m_text1, m_text2.get()).hasException()) return; - m_text2->deleteData(0, m_offset, ec); + m_text2->deleteData(0, m_offset); } #ifndef NDEBUG + void SplitTextNodeCommand::getNodesInCommand(HashSet<Node*>& nodes) { addNodeAndDescendants(m_text1.get(), nodes); addNodeAndDescendants(m_text2.get(), nodes); } + #endif } // namespace WebCore |