Changed the default value for Empty Dir Attribute

third_party/WebKit/Source/core/html/HTMLElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004-2008, 2013, 2014 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2011 Motorola Mobility. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 216 matching lines @@
             addPropertyToPresentationAttributeStyle(style, CSSPropertyWebkitUserSelect, CSSValueNone);
         } else if (equalIgnoringCase(value, "false")) {
             addPropertyToPresentationAttributeStyle(style, CSSPropertyWebkitUserDrag, CSSValueNone);
         }
     } else if (name == dirAttr) {
         if (equalIgnoringCase(value, "auto")) {
             addPropertyToPresentationAttributeStyle(style, CSSPropertyUnicodeBidi, unicodeBidiAttributeForDirAuto(this));
         } else {
             if (isValidDirAttribute(value))
                 addPropertyToPresentationAttributeStyle(style, CSSPropertyDirection, value);
+            else if (isHTMLBodyElement(*this))
+                addPropertyToPresentationAttributeStyle(style, CSSPropertyDirection, "ltr");
             else
-                addPropertyToPresentationAttributeStyle(style, CSSPropertyDirection, "ltr");
+                addPropertyToPresentationAttributeStyle(style, CSSPropertyDirection, "");

[kojii, 2016/02/16 07:39:01]

I should have noticed before, sorry, but is this line necessary? Adding an empty
string means it's not a valid style, so this line should be no-op. Can you try
if it still works without this?

[chakshu, 2016/02/18 09:00:58]

Oh yes, even I didn't realize that.
Made the change now. It works !

             if (!hasTagName(bdiTag) && !hasTagName(bdoTag) && !hasTagName(outputTag))
                 addPropertyToPresentationAttributeStyle(style, CSSPropertyUnicodeBidi, CSSValueIsolate);
         }
     } else if (name.matches(XMLNames::langAttr)) {
         mapLanguageAttributeToLocale(value, style);
     } else if (name == langAttr) {
         // xml:lang has a higher priority than lang.
         if (!fastHasAttribute(XMLNames::langAttr))
             mapLanguageAttributeToLocale(value, style);
     } else {
@@ skipping 787 matching lines @@
 #ifndef NDEBUG
 
 // For use in the debugger
 void dumpInnerHTML(blink::HTMLElement*);
 
 void dumpInnerHTML(blink::HTMLElement* element)
 {
     printf("%s\n", element->innerHTML().ascii().data());
 }
 #endif