diff options
author | Teng-Hui Zhu <ztenghui@google.com> | 2012-04-16 15:24:57 -0700 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2012-04-16 15:24:57 -0700 |
commit | 93b8c711084f01037af0c1c42c7ff57ae8711f84 (patch) | |
tree | 56cf09a05028bbd83d854018f8c2b271de28b6e9 /Source | |
parent | 2662ec568b9fe4333e7ad443b588aa0dfae579a8 (diff) | |
parent | 95217a662a9bdf75bcd22acb92bccce312ea974b (diff) | |
download | external_webkit-93b8c711084f01037af0c1c42c7ff57ae8711f84.zip external_webkit-93b8c711084f01037af0c1c42c7ff57ae8711f84.tar.gz external_webkit-93b8c711084f01037af0c1c42c7ff57ae8711f84.tar.bz2 |
Merge "Revert "Cherry-pick WebKit change r92139 to fix a LayoutTest crash""
Diffstat (limited to 'Source')
-rw-r--r-- | Source/WebCore/css/CSSStyleSelector.cpp | 22 | ||||
-rw-r--r-- | Source/WebCore/dom/Node.cpp | 2 |
2 files changed, 10 insertions, 14 deletions
diff --git a/Source/WebCore/css/CSSStyleSelector.cpp b/Source/WebCore/css/CSSStyleSelector.cpp index e49ad2a..2424349 100644 --- a/Source/WebCore/css/CSSStyleSelector.cpp +++ b/Source/WebCore/css/CSSStyleSelector.cpp @@ -1237,15 +1237,6 @@ PassRefPtr<RenderStyle> CSSStyleSelector::styleForDocument(Document* document) return documentStyle.release(); } -static inline bool isAtShadowBoundary(Element* element) -{ - if (!element) - return false; - - ContainerNode* parentNode = element->parentNode(); - return parentNode && parentNode->isShadowBoundary(); -} - // If resolveForRootDefault is true, style based on user agent style sheet only. This is used in media queries, where // relative units are interpreted according to document root element style, styled only with UA stylesheet @@ -1288,10 +1279,6 @@ PassRefPtr<RenderStyle> CSSStyleSelector::styleForElement(Element* e, RenderStyl initForStyleResolve(e, defaultParent); } - // Don't propagate user-modify into shadow DOM - if (isAtShadowBoundary(e)) - m_style->setUserModify(RenderStyle::initialUserModify()); - m_checker.m_matchVisitedPseudoClass = matchVisitedPseudoClass; m_style = RenderStyle::create(); @@ -1786,6 +1773,15 @@ static void addIntrinsicMargins(RenderStyle* style) } } +static inline bool isAtShadowBoundary(Element* element) +{ + if (!element) + return false; + + ContainerNode* parentNode = element->parentNode(); + return parentNode && parentNode->isShadowBoundary(); +} + void CSSStyleSelector::adjustRenderStyle(RenderStyle* style, RenderStyle* parentStyle, Element *e) { // Cache our original display. diff --git a/Source/WebCore/dom/Node.cpp b/Source/WebCore/dom/Node.cpp index ec5c423..da4312c 100644 --- a/Source/WebCore/dom/Node.cpp +++ b/Source/WebCore/dom/Node.cpp @@ -770,7 +770,7 @@ bool Node::isContentEditable() const bool Node::rendererIsEditable(EditableLevel editableLevel) const { - if (document()->frame() && document()->frame()->page() && document()->frame()->page()->isEditable() && !shadowTreeRootNode()) + if (document()->frame() && document()->frame()->page() && document()->frame()->page()->isEditable()) return true; // Ideally we'd call ASSERT(!needsStyleRecalc()) here, but |