Chromium Code Reviews| Index: Source/core/rendering/HitTestResult.cpp |
| diff --git a/Source/core/rendering/HitTestResult.cpp b/Source/core/rendering/HitTestResult.cpp |
| index 63f71c09e4fceeb2b6fd70a57a1efe9393783ba1..6bc60977c4dda8b17e1ed51068f5d9fd0a79c342 100644 |
| --- a/Source/core/rendering/HitTestResult.cpp |
| +++ b/Source/core/rendering/HitTestResult.cpp |
| @@ -378,10 +378,10 @@ bool HitTestResult::isContentEditable() const |
| if (!m_innerNonSharedNode) |
| return false; |
| - if (isHTMLTextAreaElement(*m_innerNonSharedNode)) |
| + if (isHTMLTextAreaElement(*m_innerNonSharedNode) && !toHTMLFormControlElement(m_innerNonSharedNode)->isDisabledOrReadOnly()) |
|
yosin_UTC9
2014/05/07 01:12:03
nit: Can we move |isDisabledOrReadOnly()| call to
|
| return true; |
| - if (isHTMLInputElement(*m_innerNonSharedNode)) |
| + if (isHTMLInputElement(*m_innerNonSharedNode) && !toHTMLFormControlElement(m_innerNonSharedNode)->isDisabledOrReadOnly()) |
|
yosin_UTC9
2014/05/07 01:12:03
nit: Can we move |isDisabledOrReadOnly()| call to
|
| return toHTMLInputElement(*m_innerNonSharedNode).isTextField(); |
| return m_innerNonSharedNode->rendererIsEditable(); |