Tables with display:inline rendered in wrong position.

Source/core/rendering/RenderObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 148 matching lines @@
 
     if (element->hasTagName(rubyTag)) {
         if (style->display() == INLINE)
             return new RenderRubyAsInline(element);
         else if (style->display() == BLOCK)
             return new RenderRubyAsBlock(element);
     }
     // treat <rt> as ruby text ONLY if it still has its default treatment of block
     if (element->hasTagName(rtTag) && style->display() == BLOCK)
         return new RenderRubyText(element);
+    if (element->hasTagName(tableTag) && style->display() == INLINE)
+        return new RenderTable(element);

[esprehn, 2013/08/28 18:17:54]

This does not appear correct, if you make a table display: inline why shouldn't
it be inline?

This will break things like:

<div>Hello <table><tr><td>World</td></tr></table> </div>

table, tr, td { display: inline; }

[suchit.agrawal, 2013/08/29 04:31:09]

I have tried this example and check the behavior.
You are right, now it's behavior is changed but it is matching with Firfox.

[suchit.agrawal, 2013/08/29 05:49:05]

Our previous behavior of this example was wrong. 'World' was rendered in the
next line. But It should be in same line of 'Hello'.
After this patch, Behavior is correct.  

     if (RuntimeEnabledFeatures::cssRegionsEnabled() && style->isDisplayRegionType() && !style->regionThread().isEmpty() && doc->renderView())
         return new RenderRegion(element, 0);
 
     if (style->display() == RUN_IN)
         UseCounter::count(doc, UseCounter::CSSDisplayRunIn);
     else if (style->display() == COMPACT)
         UseCounter::count(doc, UseCounter::CSSDisplayCompact);
 
     switch (style->display()) {
     case NONE:
@@ skipping 3153 matching lines @@
 {
     if (object1) {
         const WebCore::RenderObject* root = object1;
         while (root->parent())
             root = root->parent();
         root->showRenderTreeAndMark(object1, "*", object2, "-", 0);
     }
 }
 
 #endif