Chromium Code Reviews| Index: third_party/WebKit/Source/core/html/HTMLElement.cpp |
| diff --git a/third_party/WebKit/Source/core/html/HTMLElement.cpp b/third_party/WebKit/Source/core/html/HTMLElement.cpp |
| index bc3c1bd86be98f88dc2d7ee1f7e9544a13ba772f..090d095dc9c05ba2d3c1880d06541ea4e77c95a2 100644 |
| --- a/third_party/WebKit/Source/core/html/HTMLElement.cpp |
| +++ b/third_party/WebKit/Source/core/html/HTMLElement.cpp |
| @@ -232,10 +232,13 @@ void HTMLElement::collectStyleForPresentationAttribute(const QualifiedName& name |
| if (equalIgnoringCase(value, "auto")) { |
| addPropertyToPresentationAttributeStyle(style, CSSPropertyUnicodeBidi, unicodeBidiAttributeForDirAuto(this)); |
| } else { |
| - if (isValidDirAttribute(value)) |
| + if (isValidDirAttribute(value)) { |
| addPropertyToPresentationAttributeStyle(style, CSSPropertyDirection, value); |
| - else |
| - addPropertyToPresentationAttributeStyle(style, CSSPropertyDirection, "ltr"); |
| + } else { |
|
chakshu
2016/02/11 05:55:37
@kojii Can you look at this fix for default value
|
| + addPropertyToPresentationAttributeStyle(style, CSSPropertyDirection, ""); |
| + if (hasTagName(bodyTag)) |
|
kojii
2016/02/11 13:51:16
Probably isHTMLBodyElement() is better? I'm not ce
chakshu
2016/02/12 06:04:10
isHTMLBodyElement() is basically the wrapper funct
|
| + addPropertyToPresentationAttributeStyle(style, CSSPropertyDirection, "ltr"); |
| + } |
| if (!hasTagName(bdiTag) && !hasTagName(bdoTag) && !hasTagName(outputTag)) |
| addPropertyToPresentationAttributeStyle(style, CSSPropertyUnicodeBidi, CSSValueIsolate); |
| } |