summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authornektar <nektar@chromium.org>2015-11-02 15:52:52 -0800
committerCommit bot <commit-bot@chromium.org>2015-11-02 23:53:44 +0000
commite04afb73ccb8dd14665b9a0622daf1d93e122a2d (patch)
treeca6491ce19d2fcc406ae002eab24b6d0e55f7a8d
parent03071f16072310e22958c3b4af699fcf69166686 (diff)
downloadchromium_src-e04afb73ccb8dd14665b9a0622daf1d93e122a2d.zip
chromium_src-e04afb73ccb8dd14665b9a0622daf1d93e122a2d.tar.gz
chromium_src-e04afb73ccb8dd14665b9a0622daf1d93e122a2d.tar.bz2
Switched to checking node() != nullptr instead of layoutObject() in AXLayoutObject::selectionUnderObject().
There are cases when node is nullptr but layoutObject is not. Note that checking node() encompasses checking layoutObject since the node() function implicitly checks layoutObject != nullptr. BUG=518810 R=dmazzoni@chromium.org, aboxhall@chromium.org Review URL: https://codereview.chromium.org/1415463014 Cr-Commit-Position: refs/heads/master@{#357473}
-rw-r--r--third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp b/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp
index 286c8d5..54b384b 100644
--- a/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp
+++ b/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp
@@ -1890,7 +1890,7 @@ AXObject::AXRange AXLayoutObject::selectionUnderObject() const
if (textSelection.isValid())
return textSelection;
- if (!layoutObject() || !layoutObject()->frame())
+ if (!node() || !layoutObject()->frame())
return AXRange();
VisibleSelection selection = layoutObject()->frame()->selection().selection();