Title: [103268] branches/chromium/963/Source/WebCore
Revision
103268
Author
[email protected]
Date
2011-12-19 14:29:54 -0800 (Mon, 19 Dec 2011)

Log Message

Merge 102310 - WebPImageDecoder should not do a full image decode if progressive decoding is active
https://bugs.webkit.org/show_bug.cgi?id=74041

Reviewed by Adam Barth.

If the decoder input data state reaches allDataReceived during a progressive image
decode, the decoder performs a full image decode.

On allDataReceived, check if we already have a decoder, and if so, continue to run
the progressive decoder.

No new tests. No change in behavior.

* platform/image-decoders/webp/WEBPImageDecoder.cpp:
(WebCore::WEBPImageDecoder::decode):


[email protected]
Review URL: http://codereview.chromium.org/8974025

Modified Paths

Diff

Modified: branches/chromium/963/Source/WebCore/ChangeLog (103267 => 103268)


--- branches/chromium/963/Source/WebCore/ChangeLog	2011-12-19 22:23:30 UTC (rev 103267)
+++ branches/chromium/963/Source/WebCore/ChangeLog	2011-12-19 22:29:54 UTC (rev 103268)
@@ -1,770 +1,3 @@
-2011-12-08  Ryosuke Niwa  <[email protected]>
-
-        Line breaks are lost when pasted into textarea text starting with a blank line set while textarea is hidden
-        https://bugs.webkit.org/show_bug.cgi?id=74126
-
-        Reviewed by Tony Chang.
-
-        The bug was caused by the code that generated text out of pre-rendered text was generating div's inside the fragment
-        pasted into textarea even though serialization algorithm in textarea doesn't handle block elements.
-
-        Fixed the bug by special-casing this in createFragmentFromText. In the long run, we should really get rid of this
-        whole pre-rendering trick.
-
-        * editing/markup.cpp:
-        (WebCore::createFragmentFromText):
-
-2011-12-08  Florin Malita  <[email protected]>
-
-        Moving SVG elements on the page doesn't always erase element at the old position
-        https://bugs.webkit.org/show_bug.cgi?id=74002
-
-        Reviewed by Darin Adler.
-
-        Test: svg/repaint/container-repaint.svg
-
-        * rendering/svg/RenderSVGContainer.cpp:
-        (WebCore::RenderSVGContainer::layout):
-        Save the old repaint bounds before updating the viewport.
-
-2011-12-08  Tim Horton  <[email protected]>
-
-        Enable animations of CSS images using -webkit-cross-fade
-        https://bugs.webkit.org/show_bug.cgi?id=74049
-        <rdar://problem/10209303>
-
-        Reviewed by Simon Fraser.
-
-        Add support for animating CSS images in the following properties:
-            - background(-image)
-            - border-image(-source)
-            - list-style(-image)
-            - -webkit-mask-box-image(-source)
-            - -webkit-mask-image(-source)
-        
-        This patch only adds support for transitioning between NinePieceImages
-        where all of the properties except the image itself are equal, and the
-        size of the images are equal. Other cases will not animate.
-        
-        Add animation blend functions for StyleImage and NinePieceImage.
-        
-        Apply the proper compositing operation to -webkit-cross-fade, and
-        fix handling of the destination and source areas to support scaling.
-
-        Tests: animations/cross-fade-background-image.html
-               animations/cross-fade-border-image-source.html
-               animations/cross-fade-list-style-image.html
-               animations/cross-fade-webkit-mask-box-image.html
-               animations/cross-fade-webkit-mask-image.html
-
-        * css/CSSCrossfadeValue.cpp:
-        (WebCore::cachedImageForCSSValue):
-        (WebCore::CSSCrossfadeValue::fixedSize):
-        (WebCore::CSSCrossfadeValue::image):
-        * css/CSSImageValue.cpp:
-        (WebCore::CSSImageValue::CSSImageValue):
-        * css/CSSImageValue.h:
-        (WebCore::CSSImageValue::create):
-        * page/animation/AnimationBase.cpp:
-        (WebCore::crossfadeBlend):
-        (WebCore::blendFunc):
-        (WebCore::RefCountedPropertyWrapper::RefCountedPropertyWrapper):
-        (WebCore::RefCountedPropertyWrapper::blend):
-        (WebCore::FillLayerRefCountedPropertyWrapper::FillLayerRefCountedPropertyWrapper):
-        (WebCore::FillLayerRefCountedPropertyWrapper::blend):
-        (WebCore::FillLayersPropertyWrapper::FillLayersPropertyWrapper):
-        (WebCore::AnimationBase::ensurePropertyMap):
-        (WebCore::addShorthandProperties):
-        * platform/graphics/CrossfadeGeneratedImage.cpp:
-        (WebCore::CrossfadeGeneratedImage::drawCrossfade):
-        (WebCore::CrossfadeGeneratedImage::draw):
-        (WebCore::CrossfadeGeneratedImage::drawPattern):
-        * platform/graphics/CrossfadeGeneratedImage.h:
-        * rendering/style/RenderStyle.h:
-        (WebCore::InheritedFlags::setMaskImage):
-
-2011-12-08  Stephen White  <[email protected]>
-
-        Use Skia's implementation of Gaussian blur when accelerated filters
-        are enabled.
-        https://bugs.webkit.org/show_bug.cgi?id=73949
-
-        Reviewed by Zoltan Herczeg.
-
-        In the future, this will be covered by the SVG tests run in GPU mode.
-
-        * WebCore.gypi:
-        Add FEGaussianBlurSkia.cpp to the build.
-        * platform/graphics/filters/FEGaussianBlur.cpp:
-        (WebCore::FEGaussianBlur::platformApplySoftware):
-        Call out to platformApplySkia() when USE_SKIA is enabled.
-        * platform/graphics/filters/FEGaussianBlur.h:
-        platformApplySkia() declaration.
-        * platform/graphics/filters/skia: Added.
-        * platform/graphics/filters/skia/FEGaussianBlurSkia.cpp: Added.
-        (WebCore::FEGaussianBlur::platformApplySkia):
-        On the Skia port, use SkBlurImageFilter for drawing
-        Gaussian blurs in accelerated mode.
-        * platform/graphics/skia/ImageBufferSkia.cpp:
-        (WebCore::ImageBuffer::copyImage):
-        Implement ImageBuffer::copyImage() with DontCopyBackingStore semantics.
-
-2011-12-08  Erik Arvidsson  <[email protected]>
-
-        CodeGeneratorV8: Fix issue with overloaded static conditional methods
-        https://bugs.webkit.org/show_bug.cgi?id=74114
-
-        Reviewed by Adam Barth.
-
-        The code generator was missing checks for Conditional for overloaded methods.
-
-        * bindings/scripts/CodeGeneratorV8.pm:
-        (GenerateOverloadedFunctionCallback): Generate needed #ifdefs.
-        (GenerateFunctionCallback): Ditto.
-        * bindings/scripts/test/V8/V8TestObj.cpp: Wrap conditional methods with #ifdefs.
-        (WebCore::ConfigureV8TestObjTemplate):
-
-2011-12-08  Dominic Mazzoni  <[email protected]>
-
-        AccessibilityController should support listening to notifications on all elements.
-        https://bugs.webkit.org/show_bug.cgi?id=72866
-
-        Changes accessibilitySetShouldRepostNotifications from an instance method
-        into a class method so that it can be used for global notification listeners,
-        not just for listeners on a particular object.
-
-        Reviewed by Chris Fleizach.
-
-        Test: accessibility/notification-listeners.html
-
-        * accessibility/mac/WebAccessibilityObjectWrapper.h:
-        * accessibility/mac/WebAccessibilityObjectWrapper.mm:
-        (+[WebAccessibilityObjectWrapper accessibilitySetShouldRepostNotifications:]):
-        (-[WebAccessibilityObjectWrapper accessibilityPostedNotification:]):
-
-2011-12-08  Vsevolod Vlasov  <[email protected]>
-
-        Web Inspector: Rename createScriptCallStack() without parameters to createScriptCallStackForInspector().
-        https://bugs.webkit.org/show_bug.cgi?id=74120
-
-        Reviewed by Pavel Feldman.
-
-        * bindings/js/ScriptCallStackFactory.cpp:
-        (WebCore::createScriptCallStackForInspector):
-        * bindings/js/ScriptCallStackFactory.h:
-        * bindings/scripts/CodeGeneratorJS.pm:
-        (GenerateParametersCheck):
-        * bindings/scripts/CodeGeneratorV8.pm:
-        (GenerateFunctionCallback):
-        * bindings/scripts/test/JS/JSTestObj.cpp:
-        (WebCore::jsTestObjPrototypeFunctionCustomArgsAndException):
-        * bindings/scripts/test/V8/V8TestObj.cpp:
-        (WebCore::TestObjInternal::customArgsAndExceptionCallback):
-        * bindings/v8/ScriptCallStackFactory.cpp:
-        (WebCore::createScriptCallStackForInspector):
-        * bindings/v8/ScriptCallStackFactory.h:
-
-2011-12-08  Adrienne Walker  <[email protected]>
-
-        [chromium] Remove dead code in compositor
-        https://bugs.webkit.org/show_bug.cgi?id=74103
-
-        Reviewed by James Robinson.
-
-        Tested via the compiler.
-
-        * platform/graphics/chromium/LayerChromium.h:
-        * platform/graphics/chromium/LayerRendererChromium.cpp:
-        * platform/graphics/chromium/LayerRendererChromium.h:
-
-2011-12-08  Eric Penner  <[email protected]>
-
-        [chromium] Need to adjust memory limit and viewport multipliers.
-        https://bugs.webkit.org/show_bug.cgi?id=74022
-
-        Reviewed by James Robinson.
-
-        * platform/graphics/chromium/TextureManager.cpp:
-        (WebCore::TextureManager::highLimitBytes): Changing constants
-        (WebCore::TextureManager::reclaimLimitBytes): ditto
-
-2011-12-08  Benjamin Poulain  <[email protected]>
-
-        Add a platform EventLoop for iOS
-        https://bugs.webkit.org/show_bug.cgi?id=74043
-
-        Reviewed by David Kilzer.
-
-        * WebCore.xcodeproj/project.pbxproj:
-        * platform/ios/EventLoopIOS.mm: Added.
-        (WebCore::EventLoop::cycle):
-
-2011-12-08  Pavel Feldman  <[email protected]>
-
-        Web Inspector: return node counts on the document / detached root basis
-        https://bugs.webkit.org/show_bug.cgi?id=74104
-
-        Reviewed by Yury Semikhatsky.
-
-        * bindings/js/ScriptProfiler.h:
-        * bindings/v8/ScriptProfiler.cpp:
-        (WebCore::ScriptProfiler::domNodeCount):
-        * bindings/v8/ScriptProfiler.h:
-        * inspector/Inspector.json:
-        * inspector/InspectorController.cpp:
-        (WebCore::InspectorController::InspectorController):
-        * inspector/InspectorMemoryAgent.cpp:
-        (WebCore::InspectorMemoryAgent::getDOMNodeCount):
-        (WebCore::InspectorMemoryAgent::InspectorMemoryAgent):
-        * inspector/InspectorMemoryAgent.h:
-        (WebCore::InspectorMemoryAgent::create):
-
-2011-12-08  Andreas Kling  <[email protected]>
-
-        RenderObject: Rename styleSlowCase() to styleInRegion().
-
-        Rubber-stamped by David Hyatt.
-
-        * WebCore.exp.in:
-        * rendering/RenderObject.cpp:
-        (WebCore::RenderObject::styleInRegion):
-        * rendering/RenderObject.h:
-        (WebCore::RenderObject::style):
-
-2011-12-08  Andreas Kling  <[email protected]>
-
-        Optimize RenderObject::containingBlock().
-        <http://webkit.org/b/74109>
-
-        Reviewed by David Hyatt.
-
-        When climbing the parent chain to locate the containing block-level element,
-        use !isRenderBlock() to reject renderers rather than checking against an arbitrary
-        list of non-block renderers and then rejecting anything that isn't a block anyway.
-
-        RenderObject::containingBlock() was very hot (2.0%) when scrolling on youtube.com.
-        This change takes it down to 1.0% (60% of which is RenderObject::isRenderBlock().)
-
-        * rendering/RenderObject.cpp:
-        (WebCore::RenderObject::containingBlock):
-
-2011-12-08  Nayan Kumar K  <[email protected]>
-
-        Define DEBUG_GL_COMMANDS only in debug builds.
-        https://bugs.webkit.org/show_bug.cgi?id=74083
-
-        Reviewed by Noam Rosenthal.
-
-        No tests added as this change does not affect functionality.
-
-        * platform/graphics/opengl/TextureMapperGL.cpp:
-
-2011-12-08  Ryosuke Niwa  <[email protected]>
-
-        CompositeEditCommand should not be kept alive for undo and redo
-        https://bugs.webkit.org/show_bug.cgi?id=64414
-
-        Reviewed by Enrica Casucci.
-
-        This patch introduces EditCommandComposition that replaces CompositeEditCommand for
-        undo and redo purposes. Furthermore, we now keep a list of commands instead of a tree of commands
-        to unapply and reapply edit commands that composes an undoable action.
-
-        Each top-level CompositeEditCommand holds a ref-pointer to EditCommandComposition,
-        and applyCommandToComposite adds new SimpleEditCommands to the list.
-
-        * editing/CompositeEditCommand.cpp:
-        (WebCore::EditCommandComposition::create):
-        (WebCore::EditCommandComposition::doApply): Never used.
-        (WebCore::EditCommandComposition::doUnapply):
-        (WebCore::EditCommandComposition::doReapply):
-        (WebCore::EditCommandComposition::append):
-        (WebCore::CompositeEditCommand::~CompositeEditCommand): Add an assertion to ensure we didn't create
-        a composition for CompositeEditCommands that have parents.
-        (WebCore::CompositeEditCommand::doUnapply): Never used.
-        (WebCore::CompositeEditCommand::doReapply): Never used.
-        (WebCore::CompositeEditCommand::ensureComposition): Creates and attaches a EditCommandComposition.
-        (WebCore::CompositeEditCommand::applyCommandToComposite): Append a SimpleEditCommand to the composition.
-
-        * editing/CompositeEditCommand.h:
-        (WebCore::EditCommandComposition::EditCommandComposition):
-        (WebCore::CompositeEditCommand::composition):
-        (WebCore::toEditCommandComposition):
-        (WebCore::toCompositeEditCommand):
-
-        * editing/DeleteButtonController.cpp: Wrap RemoveNodeCommand in RemoveTargetCommand since top level
-        commands are now required to be a CompositeEditCommand.
-        (WebCore::RemoveTargetCommand::create):
-        (WebCore::RemoveTargetCommand::RemoveTargetCommand):
-        (WebCore::RemoveTargetCommand::doApply):
-        (WebCore::DeleteButtonController::deleteTarget):
-
-        * editing/EditCommand.cpp:
-        (WebCore::EditCommand::EditCommand): New constructor; used by EditCommandComposition.
-        (WebCore::EditCommand::apply): Create a composition for a top-level command.
-        (WebCore::EditCommand::unapply): Since we clear m_parent of SimpleEditCommand as soon as they are
-        added to EditCommandComposition, we can't use isTopLevelCommand() to differentiate EditCommandComposition
-        from SimpleEditCommand. Use isEditCommandComposition() instead.
-        (WebCore::EditCommand::reapply): Ditto.
-        (WebCore::compositionIfPossible):
-        (WebCore::EditCommand::setStartingSelection): Update the starting selection of EditCommandComposition. 
-        (WebCore::EditCommand::setEndingSelection): Ditto.
-        (WebCore::EditCommand::setParent): Accepts a null pointer in order to avoid keeping a stale pointer in
-        m_parent inside SimpleEditCommand when CompositeEditCommand goes away.
-
-        * editing/EditCommand.h:
-        (WebCore::EditCommand::isSimpleEditCommand):
-        (WebCore::EditCommand::isCompositeEditCommand):
-        (WebCore::EditCommand::isEditCommandComposition):
-        (WebCore::EditCommand::parent):
-        (WebCore::toSimpleEditCommand):
-
-        * editing/Editor.cpp:
-        (WebCore::Editor::appliedEditing): Register a EditCommandComposition, instead of a CompositeEditCommand
-        to the undo stack.
-        (WebCore::Editor::unappliedEditing): Unapplied or reapplied commands are now always EditCommandComposition.
-        (WebCore::Editor::reappliedEditing):
-        * editing/Editor.h:
-
-2011-12-08  Stephen White  <[email protected]>
-
-        Add missing V8 bindings to get CSS_FILTERS to compile in Chromium.
-        https://bugs.webkit.org/show_bug.cgi?id=74091
-
-        Reviewed by Kenneth Russell.
-
-        Will be covered by tests in css3/filters (when enabled).
-
-        * bindings/v8/custom/V8WebKitCSSFilterValueCustom.cpp: Added.
-        (WebCore::V8WebKitCSSFilterValue::indexedPropertyGetter):
-
-2011-12-08  Beth Dakin  <[email protected]>
-
-        https://bugs.webkit.org/show_bug.cgi?id=73348
-        REGRESSION: Assertion when loading a page with a scrollable RenderLayer 
-        -and corresponding-
-        <rdar://problem/10518918>
-
-        Reviewed by Darin Adler.
-
-        The main problem here is that certain delegate calls into AppKit for overlay 
-        scrollbars can cause AppKit to call back into WebKit looking for more information. 
-        The assertion happens when WebKit tells AppKit that the scroll position has 
-        changed during a layout, and AppKit immediately asks WebKit to convert some 
-        coordinates, and WebKit asserts that you shouldn't do that while a layout is still 
-        happening. It's still possible for AppKit to call this delegate method while a 
-        layout is happening, and we should guard against that. This patch, however, does 
-        not do that.
-
-        This change instead addresses the reason this assertion started happening much 
-        more frequently recently, which is that it recently became true that 
-        notifyPositionChanged() can be called when the position has not changed. To fix 
-        the assertion AND the bug that that change was intended to fix, we can just make 
-        sure that either the position OR the scroll origin has changed before calling 
-        notifyPositionChanged(). 
-
-        * platform/mac/ScrollAnimatorMac.mm:
-        (WebCore::ScrollAnimatorMac::immediateScrollToPoint):
-
-        Call resetScrollOriginChanged() after the scroll instead of before so that we know 
-        whether or not to call notifyPositionChanged().
-        * platform/ScrollView.cpp:
-        (WebCore::ScrollView::updateScrollbars):
-
-2011-12-08  Kaustubh Atrawalkar  <[email protected]>
-
-        Fixing support for static conditional overloaded functions.
-        https://bugs.webkit.org/show_bug.cgi?id=74068
-
-        Reviewed by Adam Barth.
-
-        * bindings/scripts/CodeGeneratorJS.pm:
-        (GenerateImplementation): Pushing "static" keyword after condition "#if".
-        * bindings/scripts/test/CPP/WebDOMTestObj.cpp:
-        (WebDOMTestObj::overloadedMethod1): Added newly generated bindings.
-        * bindings/scripts/test/CPP/WebDOMTestObj.h: Ditto.
-        * bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: Ditto.
-        (webkit_dom_test_obj_overloaded_method1):
-        * bindings/scripts/test/GObject/WebKitDOMTestObj.h: Ditto.
-        * bindings/scripts/test/JS/JSTestObj.cpp: Ditto.
-        (WebCore::jsTestObjConstructorFunctionOverloadedMethod11):
-        (WebCore::jsTestObjConstructorFunctionOverloadedMethod12):
-        (WebCore::jsTestObjConstructorFunctionOverloadedMethod1):
-        * bindings/scripts/test/JS/JSTestObj.h: Ditto.
-        * bindings/scripts/test/ObjC/DOMTestObj.h: Ditto.
-        * bindings/scripts/test/ObjC/DOMTestObj.mm: Ditto.
-        (-[DOMTestObj overloadedMethod1:]):
-        * bindings/scripts/test/TestObj.idl: Ditto.
-        * bindings/scripts/test/V8/V8TestObj.cpp: Ditto.
-        (WebCore::TestObjInternal::overloadedMethod11Callback):
-        (WebCore::TestObjInternal::overloadedMethod12Callback):
-        (WebCore::TestObjInternal::overloadedMethod1Callback):
-        (WebCore::ConfigureV8TestObjTemplate):
-
-2011-12-08  Andreas Kling  <[email protected]>
-
-        Remove EventListenerMap destructor.
-        <http://webkit.org/b/74096>
-
-        Reviewed by Darin Adler.
-
-        Let the compiler generate ~EventListenerMap(). We only needed it when the hash map
-        was managing raw pointers.
-
-        We're losing the no-iterators assertion from clear() by doing this, but that was
-        superfluous to begin with - we were just using it to avoid duplicating code.
-
-        * dom/EventListenerMap.cpp:
-        * dom/EventListenerMap.h:
-
-2011-12-08  Takashi Toyoshima  <[email protected]>
-
-        Provide more specific error description for SocketStreamError.
-        https://bugs.webkit.org/show_bug.cgi?id=74066
-
-        Reviewed by Martin Robinson.
-
-        No new tests because this change just improve error messages for unexpected failures.
-
-        * platform/network/soup/SocketStreamError.h: Add an argument for passing error description.
-        (WebCore::SocketStreamError::SocketStreamError):
-        * platform/network/soup/SocketStreamHandleSoup.cpp: Add error description for SocketStreamError.
-        (WebCore::SocketStreamHandle::connected):
-        (WebCore::SocketStreamHandle::readBytes):
-        (WebCore::SocketStreamHandle::platformSend):
-        (WebCore::SocketStreamHandle::platformClose):
-
-2011-12-08  Mihnea Ovidenie  <[email protected]>
-
-        [CSSRegions][CSSOM] Implement NamedFlow interface
-        https://bugs.webkit.org/show_bug.cgi?id=66642
-
-        Reviewed by David Hyatt.
-
-        Add WebKitNamedFlow to support the NamedFlow interface. No methods are
-        yet implemented on this interface. The NamedFlow object is a live object.
-        The first time user asks for it, it gets a valid JS object whose properties
-        will reflect the changes to the flow thread.
-
-        Tests: fast/regions/webkit-named-flow-existing-flow.html
-               fast/regions/webkit-named-flow-flow-added.html
-               fast/regions/webkit-named-flow-modified-flow.html
-               fast/regions/webkit-named-flow-non-existing-flow.html
-               fast/regions/webkit-named-flow-removed-flow.html
-               fast/regions/webkit-named-flow-same-object.html
-
-        * CMakeLists.txt:
-        * DerivedSources.cpp:
-        * DerivedSources.make:
-        * DerivedSources.pri:
-        * GNUmakefile.list.am:
-        * Target.pri:
-        * WebCore.gypi:
-        * WebCore.vcproj/WebCore.vcproj:
-        * WebCore.xcodeproj/project.pbxproj:
-        * bindings/gobject/GNUmakefile.am:
-        * dom/DOMAllInOne.cpp:
-        * dom/Document.cpp:
-        (WebCore::Document::webkitGetFlowByName):
-        * dom/Document.h:
-        * dom/Document.idl:
-        * dom/NodeRenderingContext.cpp:
-        (WebCore::NodeRenderingContext::moveToFlowThreadIfNeeded):
-        * dom/WebKitNamedFlow.cpp: Added.
-        (WebCore::WebKitNamedFlow::WebKitNamedFlow):
-        (WebCore::WebKitNamedFlow::~WebKitNamedFlow):
-        * dom/WebKitNamedFlow.h: Added.
-        (WebCore::WebKitNamedFlow::create):
-        * dom/WebKitNamedFlow.idl: Added.
-        * rendering/RenderFlowThread.cpp:
-        (WebCore::RenderFlowThread::ensureNamedFlow):
-        * rendering/RenderFlowThread.h:
-        * rendering/RenderObject.cpp:
-        (WebCore::RenderObject::createObject):
-        * rendering/RenderView.cpp:
-        (WebCore::RenderView::ensureRenderFlowThreadWithName):
-        * rendering/RenderView.h:
-
-2011-12-08  Sheriff Bot  <[email protected]>
-
-        Unreviewed, rolling out r102321.
-        http://trac.webkit.org/changeset/102321
-        https://bugs.webkit.org/show_bug.cgi?id=74072
-
-        "Breaks all Chromium clobbered builds" (Requested by apavlov_
-        on #webkit).
-
-        * WebCore.gyp/WebCore.gyp:
-        * WebCore.gyp/scripts/action_derivedsourcesallinone.py:
-        (main):
-        * WebCore.gypi:
-        * bindings/scripts/generate-bindings.pl:
-        * page/DOMWindow.idl:
-        * webaudio/DOMWindowWebAudio.idl: Removed.
-
-2011-12-08  Shinya Kawanaka  <[email protected]>
-
-        Refactoring: Editor::requestCheckingFor should take SpellCheckRequest object.
-        https://bugs.webkit.org/show_bug.cgi?id=74033
-
-        Reviewed by Hajime Morita.
-
-        SpellChecker::requestCheckingFor takes SpellCheckRequest object in order to make it easy to
-        pass necessary information to requestCheckingFor.
-
-        No new tests. Covered by existing tests.
-
-        * editing/Editor.cpp:
-        (WebCore::Editor::replaceSelectionWithFragment):
-          Uses the new interface.
-        (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): ditto.
-        * editing/SpellChecker.cpp:
-        (WebCore::SpellCheckRequest::SpellCheckRequest):
-        (WebCore::SpellCheckRequest::~SpellCheckRequest):
-        (WebCore::SpellCheckRequest::create):
-          Creates a new SpellCheckRequest object.
-        (WebCore::SpellChecker::requestCheckingFor):
-          Uses the new interface.
-        (WebCore::SpellChecker::didCheck):
-        * editing/SpellChecker.h:
-        (WebCore::SpellCheckRequest::setSequence):
-        (WebCore::SpellCheckRequest::sequence):
-        (WebCore::SpellCheckRequest::checkingRange):
-        (WebCore::SpellCheckRequest::paragraphRange):
-        (WebCore::SpellCheckRequest::text):
-        (WebCore::SpellCheckRequest::mask):
-        (WebCore::SpellCheckRequest::rootEditableElement):
-
-2011-12-08  Sheriff Bot  <[email protected]>
-
-        Unreviewed, rolling out r102323.
-        http://trac.webkit.org/changeset/102323
-        https://bugs.webkit.org/show_bug.cgi?id=74069
-
-        Caused Chromium and GTK build failure (Requested by bashi on
-        #webkit).
-
-        * testing/Internals.cpp:
-        (WebCore::Internals::getPageScaleFactor):
-        * testing/Internals.h:
-        * testing/Internals.idl:
-
-2011-12-08  Fady Samuel  <[email protected]>
-
-        Move scalePageBy from eventSender to window.internals
-        https://bugs.webkit.org/show_bug.cgi?id=64512
-
-        Reviewed by Simon Fraser.
-
-        Added setPageScaleFactor to window.internals.
-        Renamed window.internals.getPageScaleFactor to window.internals.pageScaleFactor
-        to match the webkit style.
-
-        * testing/Internals.cpp:
-        (WebCore::Internals::pageScaleFactor):
-        (WebCore::Internals::setPageScaleFactor):
-        * testing/Internals.h:
-        * testing/Internals.idl:
-
-2011-12-08  Kentaro Hara  <[email protected]>
-
-        Use the [Supplemental] IDL for webaudio attributes in Chromium
-        https://bugs.webkit.org/show_bug.cgi?id=73394
-
-        Reviewed by Adam Barth.
-
-        - Overview: Using the [Supplemental] IDL, this patch moves the attribute
-        declarations of webaudio from DOMWindow.idl into a new IDL file
-        webaudio/DOMWindowWebAudio.idl, which helps make webaudio a self-contained
-        feature (aka a module).
-
-        - This patch changes the build flow of WebCore.gyp as follows:
-
-            Previous build flow:
-                foreach $idl (all IDL files) {
-                    generate-bindings.pl depends on $idl;
-                    generate-bindings.pl reads $idl;
-                    generate-bindings.pl generates .h and .cpp files for $idl;
-                }
-
-            New build flow (See the discussions in bug 72138 for more details):
-                resolve-supplemental.pl depends on all IDL files;
-                resolve-supplemental.pl reads all IDL files;
-                resolve-supplemental.pl resolves the dependency of [Supplemental=XXXX];
-                resolve-supplemental.pl outputs supplemental_dependency.tmp;
-                foreach $idl (all IDL files) {
-                    generate-bindings.pl depends on $idl and supplemental_dependency.tmp;
-                    generate-bindings.pl reads $idl;
-                    generate-bindings.pl reads supplemental_dependency.tmp;
-                    generate-bindings.pl generates .h and .cpp files for $idl, including all attributes in IDL files whilementing $idl;
-                }
-
-        - This patch introduces a temporary IDL, [Supplemented]. The [Supplemented] IDL
-        will be removed after build scripts for all platforms support the [Supplemental] IDL.
-        The motivation for the [Supplemented] IDL is as follows:
-
-        In order to support the [Supplemental] IDL, we need to
-        (1) run resolve-supplemental.pl and generate supplemental_dependency.tmp
-        (2) and run generate-bindings.pl with the supplemental_dependency.tmp.
-
-        This build flow requires a change on the following build scripts,
-        but changing all the build scripts all at once without any regression is too difficult:
-
-            - DerivedSources.make
-            - DerivedSources.pri
-            - GNUmakefile.am
-            - PlatformBlackBerry.cmake
-            - UseJSC.cmake
-            - UseV8.cmake
-            - WebCore.vcproj/MigrateScripts
-            - WebCore.vcproj/WebCore.vcproj
-            - bindings/gobject/GNUmakefile.am
-            - WebCore.gyp/WebCore.gyp
-
-        Thus, we are planning to change the build scripts one by one, which implies that
-        we need to allow the temporary state in which some build scripts support [Supplemental] IDL
-        but others do not. To accomplish this, we introduce a temporary IDL, [Supplemented].
-        The [Supplemented] IDL on an attribute means that the attribute is marked with [Supplemental]
-        in another IDL file somewhere, like this:
-
-            DOMWindowWebAudio.idl:
-                interface [
-                    Supplemental=DOMWindow
-                ] DOMWindowWebAudio {
-                    attribute attr1;
-                    attribute attr2;
-                };
-
-            DOMWindow.idl:
-                interface [
-                ] DOMWindow {
-                    attribute [Supplemented] attr1; // This line will be removed after all build scripts support the [Su IDL
-                    attribute [Supplemented] attr2; // This line will be removed after all build scripts support the [Su IDL.
-                    attribute attr3;
-                    attribute attr4;
-                };
-
-        Assuming these IDL files, this patch implements the following logic in generate-bindings.pl:
-
-            - If a given build script supports the [Supplemental] IDL,
-            generate-bindings.pl ignores all attributes with the [Supplemented] IDL.
-            - Otherwise, generate-bindings.pl treats all attributes with the [Supplemented] IDL
-            as normal attributes and instead ignores all attributes with the [Supplemental] IDL
-            (i.e. generate-bindings.pl generates nothing from the IDL file with the [Supplemental] IDL).
-
-        Tests: webaudio/*
-
-        * WebCore.gyp/WebCore.gyp: Describes the build flow that I described above.
-        * WebCore.gyp/scripts/action_derivedsourcesallinone.py:
-        (main): Reads the IDL file names from the input file (i.e. supplemental_dependency.tmp), which are described at the first column of each line in the input file. If the file name is a "/cygdrive/c/..."-style path, it is converted to a "C:\cygwin\..."-style path by the cygpath command.
-        * WebCore.gypi: Added DOMWindowWebAudio.idl.
-        * bindings/scripts/generate-bindings.pl: As a temporary solution, if the platform does not support the [Supplemental] IDL, the perl script ignores the [Supplemental] IDL and instead uses the [Supplemented] IDL. Otherwise, the perl script ignores the [Supplemented] IDL and instead uses the [Supplemental] IDL.
-        * page/DOMWindow.idl: Added the [Supplemented] IDL to webaudio-related attributes. As I described above, the [Supplemented] IDL will be removed after all platforms support the [Supplemental] IDL.
-        * webaudio/DOMWindowWebAudio.idl: Added. Describes the [Supplemental=DOMWindow] IDL. The attributes in this IDL file should be treated as if they are written in DOMWindow.idl.
-
-2011-12-07  Yosifumi Inoue  <[email protected]>
-
-        CSS color gets adjusted for disabled input elements
-        https://bugs.webkit.org/show_bug.cgi?id=54643
-
-        Reviewed by Kent Tamura.
-
-        No new tests. covered by existing tests. Need rebasing some existing tests for Chromimum.
-
-        Remove automatic color adjustment for disabled text control for Chromimum.
-
-        * css/themeChromium.css: Add CSS entries for default style for disabled input and textarea elements.
-        * WebCore/rendering/RenderTextControl.cpp:
-        (disabledTextColor): Removed for PLATFORM(CHROMIUM)
-        (RenderTextControl::adjustInnerTextStyle): Don't call disabledTextColor for Chromium.
-
-2011-12-07  Dmitry Lomov  <[email protected]>
-
-        https://bugs.webkit.org/show_bug.cgi?id=74038
-        [V8][Chromium] Support legacy argument order in window.postMessage/window.webkitPostMessage.
-
-        Reviewed by David Levin.
-
-        * bindings/v8/custom/V8DOMWindowCustom.cpp:
-        (WebCore::isLegacyTargetOriginDesignation):
-        (WebCore::handlePostMessageCallback):
-
-2011-12-07  Mary Wu  <[email protected]>
-
-        Upstream 4 files into WebCore/platform/blackberry
-        https://bugs.webkit.org/show_bug.cgi?id=73541
-
-        Reviewed by Antonio Gomes.
-
-        Initial upstream of BlackBerry porting of PlatformScreen/
-        SSLKeyGenerator/Sound/Widget, no new tests.
-
-        * PlatformBlackBerry.cmake: Modified to remove empty file "WheelEventBlackBerry.cpp"
-        * platform/blackberry/PlatformScreenBlackBerry.cpp: Added.
-        (WebCore::screenIsMonochrome):
-        (WebCore::screenDepthPerComponent):
-        (WebCore::screenDepth):
-        (WebCore::screenAvailableRect):
-        (WebCore::screenRect):
-        * platform/blackberry/SSLKeyGeneratorBlackBerry.cpp: Added.
-        (WebCore::getSupportedKeySizes):
-        (WebCore::signedPublicKeyAndChallengeString):
-        * platform/blackberry/SoundBlackBerry.cpp: Added.
-        (WebCore::systemBeep):
-        * platform/blackberry/WidgetBlackBerry.cpp: Added.
-        (WebCore::Widget::Widget):
-        (WebCore::Widget::~Widget):
-        (WebCore::Widget::hide):
-        (WebCore::Widget::paint):
-        (WebCore::Widget::setCursor):
-        (WebCore::Widget::setFocus):
-        (WebCore::Widget::setFrameRect):
-        (WebCore::Widget::setIsSelected):
-        (WebCore::Widget::show):
-        (WebCore::Widget::frameRect):
-
-2011-12-07  Kenichi Ishibashi  <[email protected]>
-
-        Refactor CSSParser::parseFontFaceSrc()
-        https://bugs.webkit.org/show_bug.cgi?id=73989
-
-        Reviewed by Darin Adler.
-
-        Test: fast/css/font-face-src-parsing.html
-
-        * css/CSSParser.cpp:
-        (WebCore::CSSParser::parseFontFaceSrcURI): Added.
-        (WebCore::CSSParser::parseFontFaceSrcLocal): Added.
-        (WebCore::CSSParser::parseFontFaceSrc): Rewrote.
-        * css/CSSParser.h:
-
-2011-12-07  Xingnan Wang  <[email protected]>
-
-        Implement the SSE optimization in SincResampler::process()
-        https://bugs.webkit.org/show_bug.cgi?id=73789
-
-        Reviewed by Benjamin Poulain.
-
-        Here is about 70% performance improvement on the hot spot of sample convolving.
-
-        * platform/audio/SincResampler.cpp:
-
-2011-12-07  Luke Macpherson   <[email protected]>
-
-        Implement border image source properties in CSSStyleApplyProperty.
-        https://bugs.webkit.org/show_bug.cgi?id=73981
-
-        Reviewed by Andreas Kling.
-
-        No new tests / refactoring only.
-
-        * css/CSSStyleApplyProperty.cpp:
-        (WebCore::ApplyPropertyBorderImageSource::applyValue):
-        (WebCore::ApplyPropertyBorderImageSource::createHandler):
-        (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):
-        * css/CSSStyleSelector.cpp:
-        (WebCore::CSSStyleSelector::applyProperty):
-
 2011-12-07  Noel Gordon  <[email protected]>
 
         WebPImageDecoder should not do a full image decode if progressive decoding is active

Modified: branches/chromium/963/Source/WebCore/platform/image-decoders/webp/WEBPImageDecoder.cpp (103267 => 103268)


--- branches/chromium/963/Source/WebCore/platform/image-decoders/webp/WEBPImageDecoder.cpp	2011-12-19 22:23:30 UTC (rev 103267)
+++ branches/chromium/963/Source/WebCore/platform/image-decoders/webp/WEBPImageDecoder.cpp	2011-12-19 22:29:54 UTC (rev 103268)
@@ -111,7 +111,7 @@
         m_rgbOutput.resize(height * stride);
     }
     int newLastVisibleRow = 0; // Last completed row.
-    if (allDataReceived) {
+    if (allDataReceived && !m_decoder) {
         if (!WebPDecodeRGBInto(dataBytes, dataSize, m_rgbOutput.data(), m_rgbOutput.size(), stride))
             return setFailed();
         newLastVisibleRow = height;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to