Title: [140625] branches/chromium/1391a

Diff

Modified: branches/chromium/1391a/LayoutTests/ChangeLog (140624 => 140625)


--- branches/chromium/1391a/LayoutTests/ChangeLog	2013-01-24 02:11:18 UTC (rev 140624)
+++ branches/chromium/1391a/LayoutTests/ChangeLog	2013-01-24 02:14:15 UTC (rev 140625)
@@ -1,50 +1,3 @@
-2013-01-18  Alpha Lam  <[email protected]>
-
-        [chromium] Update test expectations
-
-        Unreviewed. Update test expectations to mark test as flaky.
-
-        * platform/chromium/TestExpectations:
-
-2013-01-18  Alpha Lam  <[email protected]>
-
-        [chromium] Update test expectations
-        https://bugs.webkit.org/show_bug.cgi?id=107328
-
-        Not reviewed. Mark this test as MISSING:
-        http/tests/cache/cancel-multiple-post-xhrs.html
-
-        * platform/chromium/TestExpectations:
-
-2013-01-18  Robert Hogan  <[email protected]>
-
-        fast/inline/fixed-pos-moves-with-abspos-parent-relative-ancestor.html is crashing after r140024
-        https://bugs.webkit.org/show_bug.cgi?id=107209
-
-        Reviewed by David Hyatt.
-
-        * platform/chromium/TestExpectations:
-
-2013-01-18  Alpha Lam  <[email protected]>
-
-        [chromium] Layout test rebaselines
-
-        Unreviewed rebaseline. Update test results after Chromium roll in r140190.
-
-        * platform/chromium-win/css2.1/t100801-c544-valgn-03-d-agi-expected.png:
-        * platform/chromium-win/css2.1/t100801-c544-valgn-03-d-agi-expected.txt:
-        * platform/chromium/css2.1/t100801-c544-valgn-03-d-agi-expected.txt: Renamed from LayoutTests/platform/chromium-mac/css2.1/t100801-c544-valgn-03-d-agi-expected.txt.
-
-2013-01-18  Abhishek Arya  <[email protected]>
-
-        Heap-use-after-free in WebCore::RenderObject::isDescendantOf
-        https://bugs.webkit.org/show_bug.cgi?id=107226
-
-        Reviewed by David Hyatt.
-
-        * fast/block/float/overhanging-float-not-removed-crash-expected.txt: Added.
-        * fast/block/float/overhanging-float-not-removed-crash.html: Added.
-
 2013-01-18  Chris Hopman  <[email protected]>
 
         Fix scrollRectToVisible in the presence of transforms

Deleted: branches/chromium/1391a/LayoutTests/fast/block/float/overhanging-float-not-removed-crash-expected.txt (140624 => 140625)


--- branches/chromium/1391a/LayoutTests/fast/block/float/overhanging-float-not-removed-crash-expected.txt	2013-01-24 02:11:18 UTC (rev 140624)
+++ branches/chromium/1391a/LayoutTests/fast/block/float/overhanging-float-not-removed-crash-expected.txt	2013-01-24 02:14:15 UTC (rev 140625)
@@ -1,2 +0,0 @@
-Bug 107226. Heap-use-after-free in WebCore::RenderObject::isDescendantOf.
-PASS. WebKit didn't crash

Deleted: branches/chromium/1391a/LayoutTests/fast/block/float/overhanging-float-not-removed-crash.html (140624 => 140625)


--- branches/chromium/1391a/LayoutTests/fast/block/float/overhanging-float-not-removed-crash.html	2013-01-24 02:11:18 UTC (rev 140624)
+++ branches/chromium/1391a/LayoutTests/fast/block/float/overhanging-float-not-removed-crash.html	2013-01-24 02:14:15 UTC (rev 140625)
@@ -1,33 +0,0 @@
-<!DOCTYPE html>
-<html>
-<noembed id=test4></noembed>
-<h6 id=test3>
-<h3 id=test7></h3>
-<span id=tCF36>
-<math>
-<and>
-<equivalent>
-<csc id=test5 style="float: left; "> X6DDguzp|m_v c;</csc>
-<otherwise id=test1>
-</equivalent>
-</and>
-<cartesianproduct id=test2>#L    }|2pqynn</cartesianproduct>
-<factorial></factorial>
-<product id=test6>.
-</math>
-</span>
-</h6>
-<script>
-if (window.testRunner)
-    testRunner.dumpAsText();
-
-document.documentElement.offsetTop;
-test1.appendChild(test2);
-test2.appendChild(test3);
-document.documentElement.offsetTop;
-test4.appendChild(test5);
-test6.appendChild(test7);
-document.documentElement.offsetTop;
-document.documentElement.innerHTML = "Bug 107226. Heap-use-after-free in WebCore::RenderObject::isDescendantOf.<br />PASS. WebKit didn't crash";
-</script>
-</html>

Modified: branches/chromium/1391a/Source/WebCore/ChangeLog (140624 => 140625)


--- branches/chromium/1391a/Source/WebCore/ChangeLog	2013-01-24 02:11:18 UTC (rev 140624)
+++ branches/chromium/1391a/Source/WebCore/ChangeLog	2013-01-24 02:14:15 UTC (rev 140625)
@@ -1,149 +1,3 @@
-2013-01-18  Brady Eidson  <[email protected]>
-
-        Rework NetworkProcess resource load identifiers.
-        <rdar://problem/12934449> and https://bugs.webkit.org/show_bug.cgi?id=107192
-
-        Reviewed by Alexey Proskuryakov.
-
-        No new tests (No effect in tested configs).
-
-        * WebCore.exp.in:
-
-        Include an identifier argument for synchronous loads:
-        * loader/LoaderStrategy.cpp:
-        (WebCore::LoaderStrategy::loadResourceSynchronously):
-        * loader/LoaderStrategy.h:
-
-        Pass along a unique identifier to the synchronous load:
-        * loader/FrameLoader.cpp:
-        (WebCore::FrameLoader::loadResourceSynchronously):
-
-        Remove the "setIdentifier" method:
-        * loader/ResourceLoader.cpp:
-        * loader/ResourceLoader.h:
-
-2013-01-18  Shinya Kawanaka  <[email protected]>
-
-        Introduce isHTMLTextFormControlElement and toHTMLTextFormControlElement instead of toTextFormControl
-        https://bugs.webkit.org/show_bug.cgi?id=107089
-
-        Reviewed by Kent Tamura.
-
-        toTextFormControl(node) returns 0 if node is not HTMLTextFormControl. In recent coding convention,
-        we only have ASSERT and statc_cast in this kind of conversion function. So this code looks like
-        having a bad cast, though it does not.
-
-        In this patch, we convert toTextFromControl to isHTMLTextFormControlElement and toHTMLFormControlElement,
-        which aligns our coding convention.
-
-        No new tests, simple refactoring.
-
-        * editing/Editor.cpp:
-        (WebCore::Editor::selectionForCommand):
-        (WebCore::Editor::setBaseWritingDirection):
-        * html/HTMLElement.cpp:
-        (WebCore::HTMLElement::directionality):
-        * html/HTMLTextFormControlElement.cpp:
-        (WebCore::enclosingTextFormControl):
-        * html/HTMLTextFormControlElement.h:
-        (WebCore::isHTMLTextFormControlElement):
-        (WebCore):
-        (WebCore::toHTMLTextFormControlElement):
-        * rendering/RenderTextControl.cpp:
-        (WebCore::RenderTextControl::RenderTextControl):
-        (WebCore::RenderTextControl::layoutSpecialExcludedChild):
-        * testing/Internals.cpp:
-        (WebCore::Internals::visiblePlaceholder):
-
-2013-01-18  Emil A Eklund  <[email protected]>
-
-        Change RenderStyle::getRoundedInnerBorderFor to take ints for border widths
-        https://bugs.webkit.org/show_bug.cgi?id=107331
-
-        Reviewed by Levi Weintraub.
-        
-        Borders are always represented as ints yet RenderStyle::
-        getRoundedInnerBorderFor upcasts from int to LayoutUnit when
-        calling getRoundedInnerBorderFor.
-
-        No new tests, no change in functionality.
-
-        * rendering/style/RenderStyle.cpp:
-        (WebCore::RenderStyle::getRoundedInnerBorderFor):
-        * rendering/style/RenderStyle.h:
-        Change getRoundedInnerBorderFor to use ints for border widths.
-
-2013-01-18  Adam Barth  <[email protected]>
-
-        BackgroundHTMLParser shouldn't create AtomicStrings
-        https://bugs.webkit.org/show_bug.cgi?id=107150
-
-        Reviewed by Tony Gentilcore.
-
-        Previously, we were creating an AtomicString from the HTMLToken for
-        every start and end tag. This caused us to have a static dependency on
-        the parser thread's atomic string table. We want to avoid static
-        dependencies because we want to eventually run the background parser
-        using a library like libdispatch that will let us run many version in
-        parallel.
-
-        This patch uses a different approach whereby we use the tag name from
-        the CompactHTMLToken. In order to avoid calling strcmp excessively, we
-        use StringImpl::hash to quickly reject matches with some common tag
-        names. As a side effect, the hash value will be cached in the
-        StringImpl, speeding up atomization once these StringImpls arrive on
-        the main thread.
-
-        This patch also improves the performance on html-parser-srcdoc.html.
-        With the patch in bug 107236 applied (which focuses the benchmark more
-        tightly on the parser), we spend 2.3% less time on the main thread
-        (likely due to pre-caching the hash values) and 11.5% less time on the
-        parser thread.
-
-        * html/parser/BackgroundHTMLParser.cpp:
-        (WebCore):
-        (WebCore::threadSafeEqual):
-        (WebCore::threadSafeMatch):
-        (WebCore::BackgroundHTMLParser::pumpTokenizer):
-
-2013-01-18  Tony Gentilcore  <[email protected]>
-
-        Run defer scripts in the threaded HTML parser
-        https://bugs.webkit.org/show_bug.cgi?id=107320
-
-        Reviewed by Eric Seidel.
-
-        No new tests because covered by http/tests/misc/script-defer.html.
-
-        * html/parser/HTMLDocumentParser.cpp:
-        (WebCore::HTMLDocumentParser::prepareToStopParsing):
-        (WebCore::HTMLDocumentParser::didReceiveTokensFromBackgroundParser):
-
-2013-01-18  Robert Hogan  <[email protected]>
-
-        fast/inline/fixed-pos-moves-with-abspos-parent-relative-ancestor.html is crashing after r140024
-        https://bugs.webkit.org/show_bug.cgi?id=107209
-
-        Reviewed by David Hyatt.
-
-        * rendering/RenderBlock.cpp:
-        (WebCore::RenderBlock::layoutPositionedObjects): r140024 forgot to layout the fixedpos object after
-        marking it.
-
-2013-01-18  Abhishek Arya  <[email protected]>
-
-        Heap-use-after-free in WebCore::RenderObject::isDescendantOf
-        https://bugs.webkit.org/show_bug.cgi?id=107226
-
-        Reviewed by David Hyatt.
-
-        Test: fast/block/float/overhanging-float-not-removed-crash.html
-
-        * rendering/RenderBox.cpp:
-        (WebCore::RenderBox::removeFloatingOrPositionedChildFromBlockLists):
-        Skip anonymous blocks in the chain to get the enclosing block and
-        be able to correctly mark the overhanging floats in the next siblings.
-
 2013-01-18  Chris Hopman  <[email protected]>
 
         Fix scrollRectToVisible in the presence of transforms

Modified: branches/chromium/1391a/Source/WebCore/rendering/RenderBox.cpp (140624 => 140625)


--- branches/chromium/1391a/Source/WebCore/rendering/RenderBox.cpp	2013-01-24 02:11:18 UTC (rev 140624)
+++ branches/chromium/1391a/Source/WebCore/rendering/RenderBox.cpp	2013-01-24 02:14:15 UTC (rev 140625)
@@ -179,12 +179,6 @@
         }
 
         if (parentBlock) {
-            // Need to skip anonymous blocks in our ancestor chain since our overhanging floats
-            // can be in the next siblings of enclosing block.
-            while (parentBlock && parentBlock->isAnonymousBlock())
-                parentBlock = parentBlock->containingBlock();
-            ASSERT(parentBlock);
-
             RenderObject* parent = parentBlock->parent();
             if (parent && parent->isFlexibleBoxIncludingDeprecated())
                 parentBlock = toRenderBlock(parent);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to