diff options
author | Kenneth Myhra <kennethmyhra@gmail.com> | 2023-02-15 07:26:32 +0100 |
---|---|---|
committer | Linus Groh <mail@linusgroh.de> | 2023-02-18 00:52:47 +0100 |
commit | 07911958432249b4b435082be301fabc55766486 (patch) | |
tree | 3a7e180788292570d01ad4c6538b5f733ae49386 /Userland/Libraries/LibWeb/Selection/Selection.cpp | |
parent | e3e281adddb0c2a515078851f3ea63f94929289f (diff) | |
download | serenity-07911958432249b4b435082be301fabc55766486.zip |
LibWeb: Make factory methods of DOM::Range fallible
Diffstat (limited to 'Userland/Libraries/LibWeb/Selection/Selection.cpp')
-rw-r--r-- | Userland/Libraries/LibWeb/Selection/Selection.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/Userland/Libraries/LibWeb/Selection/Selection.cpp b/Userland/Libraries/LibWeb/Selection/Selection.cpp index 45297716f1..77c050a246 100644 --- a/Userland/Libraries/LibWeb/Selection/Selection.cpp +++ b/Userland/Libraries/LibWeb/Selection/Selection.cpp @@ -187,7 +187,7 @@ WebIDL::ExceptionOr<void> Selection::collapse(JS::GCPtr<DOM::Node> node, unsigne return {}; // 4. Otherwise, let newRange be a new range. - auto new_range = DOM::Range::create(*m_document); + auto new_range = TRY(DOM::Range::create(*m_document)); // 5. Set the start the start and the end of newRange to (node, offset). TRY(new_range->set_start(*node, offset)); @@ -214,7 +214,7 @@ WebIDL::ExceptionOr<void> Selection::collapse_to_start() } // 2. Otherwise, it must create a new range - auto new_range = DOM::Range::create(*m_document); + auto new_range = TRY(DOM::Range::create(*m_document)); // 3. Set the start both its start and end to the start of this's range TRY(new_range->set_start(*anchor_node(), m_range->start_offset())); @@ -234,7 +234,7 @@ WebIDL::ExceptionOr<void> Selection::collapse_to_end() } // 2. Otherwise, it must create a new range - auto new_range = DOM::Range::create(*m_document); + auto new_range = TRY(DOM::Range::create(*m_document)); // 3. Set the start both its start and end to the start of this's range TRY(new_range->set_start(*anchor_node(), m_range->end_offset())); @@ -266,7 +266,7 @@ WebIDL::ExceptionOr<void> Selection::extend(JS::NonnullGCPtr<DOM::Node> node, un auto new_focus_offset = offset; // 4. Let newRange be a new range. - auto new_range = DOM::Range::create(*m_document); + auto new_range = TRY(DOM::Range::create(*m_document)); // 5. If node's root is not the same as the this's range's root, set the start newRange's start and end to newFocus. if (&node->root() != &m_range->start_container()->root()) { @@ -315,7 +315,7 @@ WebIDL::ExceptionOr<void> Selection::set_base_and_extent(JS::NonnullGCPtr<DOM::N // 3. Let anchor be the boundary point (anchorNode, anchorOffset) and let focus be the boundary point (focusNode, focusOffset). // 4. Let newRange be a new range. - auto new_range = DOM::Range::create(*m_document); + auto new_range = TRY(DOM::Range::create(*m_document)); // 5. If anchor is before focus, set the start the newRange's start to anchor and its end to focus. Otherwise, set the start them to focus and anchor respectively. auto position_of_anchor_relative_to_focus = DOM::position_of_boundary_point_relative_to_other_boundary_point(anchor_node, anchor_offset, focus_node, focus_offset); @@ -348,7 +348,7 @@ WebIDL::ExceptionOr<void> Selection::select_all_children(JS::NonnullGCPtr<DOM::N return {}; // 2. Let newRange be a new range and childCount be the number of children of node. - auto new_range = DOM::Range::create(*m_document); + auto new_range = TRY(DOM::Range::create(*m_document)); auto child_count = node->child_count(); // 3. Set newRange's start to (node, 0). |