Index: Source/WebCore/ChangeLog |
=================================================================== |
--- Source/WebCore/ChangeLog (revision 103267) |
+++ Source/WebCore/ChangeLog (working copy) |
@@ -1,770 +1,3 @@ |
-2011-12-08 Ryosuke Niwa <rniwa@webkit.org> |
- |
- 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 <fmalita@google.com> |
- |
- 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 <timothy_horton@apple.com> |
- |
- 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 <senorblanco@chromium.org> |
- |
- 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 <arv@chromium.org> |
- |
- 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 <dmazzoni@google.com> |
- |
- 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 <vsevik@chromium.org> |
- |
- 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 <enne@google.com> |
- |
- [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 <epenner@google.com> |
- |
- [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 <bpoulain@apple.com> |
- |
- 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 <pfeldman@google.com> |
- |
- 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 <kling@webkit.org> |
- |
- 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 <kling@webkit.org> |
- |
- 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 <nayankk@motorola.com> |
- |
- 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 <rniwa@webkit.org> |
- |
- 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 <senorblanco@chromium.org> |
- |
- 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 <bdakin@apple.com> |
- |
- 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 <kaustubh@motorola.com> |
- |
- 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 <kling@webkit.org> |
- |
- 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 <toyoshim@chromium.org> |
- |
- 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 <mihnea@adobe.com> |
- |
- [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 <webkit.review.bot@gmail.com> |
- |
- 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 <shinyak@google.com> |
- |
- 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 <webkit.review.bot@gmail.com> |
- |
- 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 <fsamuel@chromium.org> |
- |
- 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 <haraken@chromium.org> |
- |
- 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 <yosin@chromium.org> |
- |
- 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 <dslomov@google.com> |
- |
- 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 <mary.wu@torchmobile.com.cn> |
- |
- 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 <bashi@chromium.org> |
- |
- 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 <xingnan.wang@intel.com> |
- |
- 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 <macpherson@chromium.org> |
- |
- 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 <noel.gordon@gmail.com> |
WebPImageDecoder should not do a full image decode if progressive decoding is active |