Source/WebCore/ChangeLog - Issue 9111020: Merge 101903 - REGRESSION (r91125): Polyline tool in google docs is broken

Unified Diff: Source/WebCore/ChangeLog

Side-by-side diff isn't available for this file because of its large size.

Issue 9111020: Merge 101903 - REGRESSION (r91125): Polyline tool in google docs is broken (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/963/

Patch Set: Created 8 years, 11 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

Download patch

« no previous file with comments | « LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.txt ('k') | Source/WebCore/platform/graphics/cg/PathCG.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: Source/WebCore/ChangeLog

===================================================================

--- Source/WebCore/ChangeLog (revision 104173)

+++ Source/WebCore/ChangeLog (working copy)

@@ -1,5259 +1,3 @@

-2011-12-10 Noel Gordon <noel.gordon@gmail.com>

- WebPImageDecoder progressive decodes fail to decode valid images

- https://bugs.webkit.org/show_bug.cgi?id=74062

- Reviewed by Adam Barth.

- The WEBP header is followed by a so-called P0 header, then some data to

- decode. If a partial P0 header is received during progressive decodes,

- WebPIDecGetRGB() returns false; that makes the decoder enter the failed

- state, no image appears on the page.

- James Zern (webp) recommended the following via e-mail:

- WebPIUpdate() validates input data, and will return an error status for

- malformed data (bit-stream error, invalid data). Otherwise, it returns

- OK or SUSPENDED. OK means that decoding is done/complete/no-error, and

- SUSPENDED means more input data is needed to complete decoding. A NULL

- return from WebPIDecGetRGB() is valid at this time due to a partial P0,

- and should not be interpreted as a decoding failure.

- No new tests. Not something DumpRenderTree can easily test.

- * platform/image-decoders/webp/WEBPImageDecoder.cpp:

- (WebCore::WEBPImageDecoder::decode): A NULL WebPIDecGetRGB() return is

- acceptable here. Return false instead of failing the decoder.

-2011-12-09 Benjamin Poulain <bpoulain@apple.com>

- Add the FileSystem functions of iOS

- https://bugs.webkit.org/show_bug.cgi?id=74164

- Reviewed by David Kilzer.

- Two functions are needed on iOS for temporary files and directories.

- * WebCore.exp.in:

- * WebCore.xcodeproj/project.pbxproj:

- * platform/ios/FileSystemIOS.h: Added.

- * platform/ios/FileSystemIOS.mm: Added.

- (WebCore::createTemporaryDirectory):

- (WebCore::createTemporaryFile):

-2011-12-09 Jacky Jiang <zhajiang@rim.com>

- Remove ResourceHandle::bufferedData() from ResourceHandleBlackBerry.cpp

- https://bugs.webkit.org/show_bug.cgi?id=74197

- The bufferedData() was removed in r95120.

- Reviewed by Rob Buis.

- Trivial fix, so no new tests.

- * platform/network/blackberry/ResourceHandleBlackBerry.cpp:

-2011-12-09 Eric Penner <epenner@google.com>

- [chromium] Prevent ASSERT in legitimate out-of-memory case.

- https://bugs.webkit.org/show_bug.cgi?id=74215

- Reviewed by James Robinson.

- * platform/graphics/chromium/TiledLayerChromium.cpp:

- (WebCore::TiledLayerChromium::updateCompositorResources):

-2011-12-09 Arko Saha <arko@motorola.com>

- NameNodeListCache should be invalidated when name attribute changes/modified.

- https://bugs.webkit.org/show_bug.cgi?id=70810

- Reviewed by Ryosuke Niwa.

- Test: fast/dom/getelementsbyname-invalidation-cache.html

- * html/HTMLAnchorElement.cpp:

- (WebCore::HTMLAnchorElement::parseMappedAttribute):

- * html/HTMLAppletElement.cpp:

- (WebCore::HTMLAppletElement::parseMappedAttribute):

- * html/HTMLElement.cpp:

- (WebCore::HTMLElement::parseMappedAttribute):

- * html/HTMLEmbedElement.cpp:

- (WebCore::HTMLEmbedElement::parseMappedAttribute):

- * html/HTMLFormElement.cpp:

- (WebCore::HTMLFormElement::parseMappedAttribute):

- * html/HTMLFrameElementBase.cpp:

- (WebCore::HTMLFrameElementBase::parseMappedAttribute):

- * html/HTMLIFrameElement.cpp:

- (WebCore::HTMLIFrameElement::parseMappedAttribute):

- * html/HTMLImageElement.cpp:

- (WebCore::HTMLImageElement::parseMappedAttribute):

- * html/HTMLMapElement.cpp:

- (WebCore::HTMLMapElement::parseMappedAttribute):

- * html/HTMLMetaElement.cpp:

- (WebCore::HTMLMetaElement::parseMappedAttribute):

- * html/HTMLObjectElement.cpp:

- (WebCore::HTMLObjectElement::parseMappedAttribute):

- * html/HTMLParamElement.cpp:

- (WebCore::HTMLParamElement::parseMappedAttribute):

-2011-12-09 Anders Carlsson <andersca@apple.com>

- Fix Lion release build.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterDelegate mouseLocationInScrollerForScrollerImp:]):

-2011-12-09 Mark Pilgrim <pilgrim@chromium.org>

- [FileSystem API] Entry.remove successCallback is required

- https://bugs.webkit.org/show_bug.cgi?id=69639

- Reviewed by Adam Barth.

- Test: fast/filesystem/simple-required-arguments-remove.html

- * fileapi/Entry.idl: remove [Optional] flag from Entry.remove.successCallback parameter

-2011-12-09 Tim Horton <timothy_horton@apple.com>

- background-image transitions trigger between equivalent images

- https://bugs.webkit.org/show_bug.cgi?id=74229

- <rdar://problem/10558627>

- Reviewed by Darin Adler.

- Patch by Simon Fraser.

- For animation property wrappers around StyleImage properties,

- test the equivalence of the image itself, instead of equality of

- the StyleImage pointer.

- Test: transitions/equivalent-background-image-no-transition.html

- * page/animation/AnimationBase.cpp:

- (WebCore::StyleImagePropertyWrapper::StyleImagePropertyWrapper):

- (WebCore::StyleImagePropertyWrapper::equals):

- (WebCore::FillLayerStyleImagePropertyWrapper::FillLayerStyleImagePropertyWrapper):

- (WebCore::FillLayerStyleImagePropertyWrapper::equals):

- (WebCore::FillLayersPropertyWrapper::FillLayersPropertyWrapper):

- (WebCore::AnimationBase::ensurePropertyMap):

-2011-12-09 Mary Wu <mary.wu@torchmobile.com.cn>

- Small style fix on DragDataBlackBerry.cpp

- https://bugs.webkit.org/show_bug.cgi?id=74171

- Reviewed by Rob Buis.

- Style fix, no function impact, no new tests.

- * platform/blackberry/DragDataBlackBerry.cpp:

- (WebCore::DragData::containsURL):

- (WebCore::DragData::asFilenames):

- (WebCore::DragData::asURL):

- (WebCore::DragData::asFragment):

-2011-12-09 Tony Chang <tony@chromium.org>

- add css parsing for flex-flow: wrap and wrap-reverse

- https://bugs.webkit.org/show_bug.cgi?id=74008

- Reviewed by Ojan Vafai.

- Also save 2 bits in StyleFlexibleBoxData by changing the size of m_flexFlow (there are only 4 enum values).

- * css/CSSComputedStyleDeclaration.cpp:

- (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Print the wrap value if it exists.

- * css/CSSParser.cpp:

- (WebCore::CSSParser::parseValue): Parse a second token and put the values into a CSSValueList.

- * css/CSSPrimitiveValueMappings.h:

- (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

- (WebCore::CSSPrimitiveValue::operator EFlexWrap):

- * css/CSSStyleApplyProperty.cpp:

- (WebCore::ApplyPropertyFlexFlow::applyInheritValue): Does not inherit.

- (WebCore::ApplyPropertyFlexFlow::applyInitialValue):

- (WebCore::ApplyPropertyFlexFlow::applyValue): Special handler for setting two render style values from one

- CSS property.

- (WebCore::ApplyPropertyFlexFlow::createHandler):

- (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):

- * css/CSSValueKeywords.in:

- * rendering/style/RenderStyle.h:

- (WebCore::InheritedFlags::flexWrap):

- (WebCore::InheritedFlags::setFlexWrap):

- (WebCore::InheritedFlags::initialFlexWrap):

- * rendering/style/RenderStyleConstants.h: EFlexWrap to hold flex wrap values.

- * rendering/style/StyleFlexibleBoxData.cpp:

- (WebCore::StyleFlexibleBoxData::StyleFlexibleBoxData):

- (WebCore::StyleFlexibleBoxData::operator==):

- * rendering/style/StyleFlexibleBoxData.h: 2 bits is enough to hold the 4 flexFlow values.

-2011-12-09 KwangHyuk Kim <hyuki.kim@samsung.com>

- [EFL] Add RefPtrEfl specialization for evas_object.

- https://bugs.webkit.org/show_bug.cgi?id=73790

- Reviewed by Ryosuke Niwa.

- As evas_object is also based on reference count, RefPtr is applied for evas_object.

- * PlatformEfl.cmake:

- * platform/efl/RefPtrEfl.cpp: Added.

- (WTF::refIfNotNull):

- (WTF::derefIfNotNull):

- * platform/efl/RefPtrEfl.h: Added.

-2011-12-09 Tony Chang <tony@chromium.org>

- REGRESSION(102234): 2-3% layout regression

- https://bugs.webkit.org/show_bug.cgi?id=74141

- Reviewed by David Hyatt.

- Don't allocate a RuleSet when there are no regions.

- * css/CSSStyleSelector.cpp:

- (WebCore::CSSStyleSelector::initForRegionStyling):

-2011-12-09 Anders Carlsson <andersca@apple.com>

- Fix assertion failure in ScrollAnimatorMac

- https://bugs.webkit.org/show_bug.cgi?id=74222

- Reviewed by Andreas Kling.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterControllerDelegate scrollerImpPair:convertContentPoint:toScrollerImp:]):

- This can be called with a nil scrollerImp, just return NSZeroPoint when that happens.

-2011-12-09 Anders Carlsson <andersca@apple.com>

- Move the "is currently drawing into layer" flag out into ScrollbarThemeMac

- https://bugs.webkit.org/show_bug.cgi?id=74217

- Reviewed by Beth Dakin.

- There's no need to store this flag inside ScrollAnimatorMac, just make it a global and put it in ScrollbarThemeMac instead.

- * platform/mac/ScrollAnimatorMac.h:

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterDelegate layer]):

- (WebCore::ScrollAnimatorMac::ScrollAnimatorMac):

- * platform/mac/ScrollbarThemeMac.h:

- * platform/mac/ScrollbarThemeMac.mm:

- (WebCore::ScrollbarThemeMac::isCurrentlyDrawingIntoLayer):

- (WebCore::ScrollbarThemeMac::paint):

-2011-12-09 Anders Carlsson <andersca@apple.com>

- Minor cleanup in ScrollAnimatorMac.mm

- https://bugs.webkit.org/show_bug.cgi?id=74211

- Reviewed by Andreas Kling.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterDelegate mouseLocationInScrollerForScrollerImp:]):

- No need to get the scrollbar from the scroll animator anymore.

- (-[WebScrollbarPainterDelegate setUpAlphaAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateUIStateTransitionWithDuration:]):

- Try to get data from the scrollbar and/or the scrollable area instead of the scrollbar painter.

-2011-12-09 David Levin <levin@chromium.org>

- Regression(r53595): Sync xhr requests in workers aren't terminated on worker close.

- https://bugs.webkit.org/show_bug.cgi?id=71695

- Reviewed by Zoltan Herczeg.

- Overview: Message loops rely on the message queue being killed in order

- to exit. r53595 stopped this from happening because killing a message loop

- would also stop it from doing database clean up tasks. The database clean up

- tasks needed to be tasks due to ordering issues. (They wanted to run after

- certain order tasks were run.) This was solved by once again terminating

- the message queue but then still runnning clean-up tasks from the killed

- message queue.

- * workers/WorkerRunLoop.cpp:

- (WebCore::WorkerRunLoop::run): Added the call to run clean-up tasks.

- (WebCore::WorkerRunLoop::runInMode):

- (WebCore::WorkerRunLoop::runCleanupTasks): Loop to simply clear out all clean up tasks.

- (WebCore::WorkerRunLoop::Task::performTask): Stop non-clean up tasks

- from running after the loop has been terminated.

- * workers/WorkerRunLoop.h:

- (WebCore::WorkerRunLoop::terminated): Just made it const.

- * workers/WorkerThread.cpp:

- (WebCore::WorkerThreadShutdownFinishTask::performTask): Removed

- the terminate clause since it was put back in stop.

- (WebCore::WorkerThread::stop): Terminate the run loop so

- that all loops will exit and clean up tasks will run. Also removed a comment

- about nested workers because nested workers are no longer imminent and the

- issue mentioned is one of many that should logically be investigated -- behavior correctness

- in the face of different orderings of shutdown between the document and each worker --

- when implementing them.

-2011-12-09 Tony Chang <tony@chromium.org>

- Unreviewed, rolling out r102416.

- http://trac.webkit.org/changeset/102416

- https://bugs.webkit.org/show_bug.cgi?id=73394

- Chromium Win clobber builds are failing.

- * 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-09 Eric Carlson <eric.carlson@apple.com>

- JSC wrappers for TextTrack and TextTrackCue should not be collected during event dispatch or when owner is reachable

- https://bugs.webkit.org/show_bug.cgi?id=72179

- Reviewed by Geoff Garen.

- Tests: media/track/text-track-cue-is-reachable.html

- media/track/text-track-is-reachable.html

- * GNUmakefile.list.am: Add JSTextTrackCueCustom.cpp and JSTextTrackCustom.cpp.

- * Target.pri: Ditto.

- * WebCore.gypi: Ditto.

- * WebCore.xcodeproj/project.pbxproj: Ditto

- * bindings/js/JSBindingsAllInOne.cpp: Ditto.

- * bindings/js/JSTextTrackCueCustom.cpp: Added.

- (WebCore::JSTextTrackCueOwner::isReachableFromOpaqueRoots): New.

- (WebCore::JSTextTrackCueOwner::visitChildren): New.

- * bindings/js/JSTextTrackCustom.cpp: Added.

- (WebCore::JSTextTrackOwner::isReachableFromOpaqueRoots): New.

- (WebCore::JSTextTrack::visitChildren): New, mark all cues.

- * bindings/js/JSTextTrackCustom.h: Added.

- (WebCore::root): New.

- * bindings/js/JSTextTrackListCustom.cpp:

- (WebCore::JSTextTrackList::visitChildren): New, mark all tracks.

- * html/HTMLMediaElement.cpp:

- (WebCore::HTMLMediaElement::trackWillBeRemoved): TextTracks::remove now takes a TextTrack*.

- * html/LoadableTextTrack.h:

- (WebCore::LoadableTextTrack::trackElement): New, return the <track>.

- * html/TextTrack.cpp:

- (WebCore::TextTrack::TextTrack): Initialize m_mediaElement.

- * html/TextTrack.h:

- (WebCore::TextTrack::setMediaElement): New.

- (WebCore::TextTrack::mediaElement): Ditto.

- * html/TextTrack.idl: Add CustomIsReachable and CustomMarkFunction.

- * html/TextTrackCue.idl: Add CustomIsReachable.

- * html/track/TextTrackList.cpp:

- (TextTrackList::append): Set track's media element.

- (TextTrackList::remove): Clear track's media element. Take a raw ptr, not a PassRefPtr.

- * html/track/TextTrackList.h:

- * html/track/TextTrackList.idl: Add CustomMarkFunction

-2011-12-09 Chris Fleizach <cfleizach@apple.com>

- WebKit should ignore images with @alt matching only whitespace

- https://bugs.webkit.org/show_bug.cgi?id=74189

- Reviewed by Darin Adler.

- Test: accessibility/img-alt-tag-only-whitespace.html

- * accessibility/AccessibilityRenderObject.cpp:

- (WebCore::AccessibilityRenderObject::accessibilityIsIgnored):

-2011-12-09 Anders Carlsson <andersca@apple.com>

- Remove NSAnimationContext calls

- https://bugs.webkit.org/show_bug.cgi?id=74207

- Reviewed by Sam Weinig.

- NSAnimationContext is not used for NSAnimation subclasses, so the calls to beginGrouping/endGrouping and setDuration:

- are essentially no-ops. Remove them.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterDelegate setUpAlphaAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateUIStateTransitionWithDuration:]):

-2011-12-09 Anders Carlsson <andersca@apple.com>

- Rename scrollAnimatorDestroyed to invalidate

- https://bugs.webkit.org/show_bug.cgi?id=74206

- Reviewed by Sam Weinig.

- Since these methods can be called when both scrollbars are destroyed and the scroll animator itself is

- destroyed, rename it to something more neutral.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollAnimationHelperDelegate invalidate]):

- (-[WebScrollbarPartAnimation invalidate]):

- (-[WebScrollbarPainterDelegate invalidate]):

- (WebCore::ScrollAnimatorMac::~ScrollAnimatorMac):

- (WebCore::ScrollAnimatorMac::willRemoveVerticalScrollbar):

- (WebCore::ScrollAnimatorMac::willRemoveHorizontalScrollbar):

-2011-12-09 Anders Carlsson <andersca@apple.com>

- WebScrollbarPainterControllerDelegate should know about the ScrollableArea, not the ScrollAnimatorMac

- https://bugs.webkit.org/show_bug.cgi?id=74204

- Reviewed by Sam Weinig.

- It makes more logical sense to associate the WebScrollbarPainterControllerDelegate object with its ScrollableArea, since

- painting has nothing to do with animation.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterControllerDelegate initWithScrollableArea:]):

- (-[WebScrollbarPainterControllerDelegate invalidate]):

- (-[WebScrollbarPainterControllerDelegate contentAreaRectForScrollerImpPair:]):

- (-[WebScrollbarPainterControllerDelegate inLiveResizeForScrollerImpPair:]):

- (-[WebScrollbarPainterControllerDelegate mouseLocationInContentAreaForScrollerImpPair:]):

- (-[WebScrollbarPainterControllerDelegate scrollerImpPair:convertContentPoint:toScrollerImp:]):

- (-[WebScrollbarPainterControllerDelegate scrollerImpPair:updateScrollerStyleForNewRecommendedScrollerStyle:]):

- (WebCore::ScrollAnimatorMac::ScrollAnimatorMac):

- (WebCore::ScrollAnimatorMac::~ScrollAnimatorMac):

-2011-12-09 Jarred Nicholls <jarred@sencha.com>

- [JSC] Allow cached attributes in bindings that declare a custom mark function

- https://bugs.webkit.org/show_bug.cgi?id=74187

- Reviewed by Oliver Hunt.

- No new tests, current binding tests are sufficient.

- * bindings/scripts/CodeGeneratorJS.pm:

- (GenerateImplementation):

-2011-12-09 Anders Carlsson <andersca@apple.com>

- Remove duplicate animation ivars

- https://bugs.webkit.org/show_bug.cgi?id=74194

- Reviewed by Sam Weinig.

- Since we now have one WebScrollbarPainterDelegate for each scrollbar, we no longer need separate

- vertical/horizontal animation objects, so get rid of them.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterDelegate cancelAnimations]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateKnobAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateTrackAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateUIStateTransitionWithDuration:]):

- (-[WebScrollbarPainterDelegate scrollAnimatorDestroyed]):

-2011-12-09 Anders Carlsson <andersca@apple.com>

- WebScrollbarPartAnimation should only know about the scrollbar it's animating

- https://bugs.webkit.org/show_bug.cgi?id=74192

- Reviewed by Sam Weinig.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPartAnimation initWithScrollbar:featureToAnimate:animateFrom:animateTo:duration:]):

- Change the designated initializer to just take the scrollbar. Also, make the animation non-blocking here

- so we don't have to do it in all the call sites.

- (-[WebScrollbarPartAnimation startAnimation]):

- Update the scrollbar painter.

- (-[WebScrollbarPartAnimation setCurrentProgress:]):

- Just invalidate the scrollbar we're animating.

- (-[WebScrollbarPainterDelegate setUpAlphaAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateUIStateTransitionWithDuration:]):

- Update call sites to use the new designated initializer.

-2011-12-08 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

- Inspector: Don't translate the context when rendering the highlights on a tiled layer.

- https://bugs.webkit.org/show_bug.cgi?id=74085

- Reviewed by Kenneth Rohde Christiansen.

- When the frame view is using fixed layouting, the page overlay is the size of the whole

- page and the context shouldn't be translated. The visible rect is still used in that

- case to display element titles within the visible rect.

- Also:

- - Rename overlayRect to visibleRect to reduce confusion in this case.

- - Remove the superfluous boundingBox check.

- * inspector/DOMNodeHighlighter.cpp:

-2011-12-09 Vsevolod Vlasov <vsevik@chromium.org>

- Unreviewed inspector utilities syntax fix.

- * inspector/front-end/utilities.js:

- ():

-2011-12-09 Joone Hur <joone.hur@collabora.co.uk>, Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

- [GTK] Initial implementation of Accelerated Compositing using Clutter

- https://bugs.webkit.org/show_bug.cgi?id=73319

- Reviewed by Gustavo Noronha Silva.

- No new tests added as this feature will be able to reuse the existing

- CSS3 transforms layout tests.

- * GNUmakefile.am: Include WebCore/platform/graphics/clutter path.

- * GNUmakefile.list.am: Add GraphicsLayerClutter.

- * platform/clutter/GRefPtrClutter.cpp: Added.

- (WTF::adoptGRef):

- (WTF::ClutterActor):

- * platform/clutter/GRefPtrClutter.h: Added.

- * platform/graphics/GraphicsLayer.h: Define PlatformLayer type, which represents ClutterActor.

- * platform/graphics/clutter/GraphicsLayerClutter.cpp: Boilerplate implementation.

- (WebCore::GraphicsLayerClutter::GraphicsLayerClutter):

- (WebCore::GraphicsLayerClutter::platformLayer):

- * platform/graphics/clutter/GraphicsLayerClutter.h: Boilerplate implementation.

-2011-12-08 Vsevolod Vlasov <vsevik@chromium.org>

- Web Inspector: Introduce a Map class allowing to store values indexed by arbitrary objects.

- https://bugs.webkit.org/show_bug.cgi?id=74084

- Reviewed by Pavel Feldman.

- Test: inspector/map.html

- * inspector/front-end/treeoutline.js:

- (TreeOutline):

- ():

- (TreeElement.prototype.collapse):

- (TreeElement.prototype.expand):

- * inspector/front-end/utilities.js:

- ():

-2011-12-09 Peter Rybin <peter.rybin@gmail.com>

- Web Inspector: [protocol] generate C++ classes for protocol JSON named types

- https://bugs.webkit.org/show_bug.cgi?id=72835

- Reviewed by Yury Semikhatsky.

- Extends python generator functionality.

- Makes constructor in InspectorObject public.

- * inspector/CodeGeneratorInspector.py:

- * inspector/InspectorValues.h:

-2011-12-08 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

- [Qt] [WK2] Webkit should release TextureMapper GL objects if page paint node is deallocated.

- https://bugs.webkit.org/show_bug.cgi?id=73591

- Reviewed by Noam Rosenthal.

- Implementation of helper function to remove all GL allocated tiles

- when QQuickWebView gets removed from canvas.

- Tested by new API test in WK2 (tst_QQuickWebView::removeFromCanvas).

- * platform/graphics/texmap/TextureMapperNode.cpp:

- (WebCore::TextureMapperNode::purgeNodeTexturesRecursive):

- * platform/graphics/texmap/TextureMapperNode.h:

-2011-12-08 Leo Yang <leo.yang@torchmobile.com.cn>

- Upstream platform/network/blackberry/DeferredData.{h, cpp}, NetworkJob.{h, cpp} and NetworkManager.{h, cpp}

- https://bugs.webkit.org/show_bug.cgi?id=73791

- Reviewed by Rob Buis.

- Other main contributors:

- Joe Mason <jmason@rim.com>

- Lianghui Chen <liachen@rim.com>

- Charles Wei <charles.wei@torchmobile.com.cn>

- Initial upstream, can't be built yet, no new tests.

- * platform/network/blackberry/DeferredData.cpp: Added.

- * platform/network/blackberry/DeferredData.h: Added.

- * platform/network/blackberry/NetworkJob.cpp: Added.

- * platform/network/blackberry/NetworkJob.h: Added.

- * platform/network/blackberry/NetworkManager.cpp: Added.

- * platform/network/blackberry/NetworkManager.h: Added.

-2011-12-08 Ryosuke Niwa <rniwa@webkit.org>

- It's semantically incorrect to call notifyNodeListsAttributeChanged in dispatchSubtreeModifiedEvent

- https://bugs.webkit.org/show_bug.cgi?id=74028

- Reviewed by Darin Adler.

- Remove a call to notifyNodeListsAttributeChanged in dispatchSubtreeModified and add explicit calls

- to notifyNodeListsAttributeChanged at appropriate places.

- Also merge notifyNodeListsChildrenChanged with notifyLocalNodeListsChildrenChanged, and

- notifyNodeListsAttributeChanged with notifyLocalNodeListsAttributeChanged, and rename them to

- invalidateNodeListsCacheAfterAttributeChanges and invalidateNodeListsCacheAfterNodeChanges respectively.

- * dom/Attr.cpp:

- (WebCore::Attr::childrenChanged):

- * dom/ContainerNode.cpp:

- (WebCore::ContainerNode::childrenChanged):

- * dom/Document.cpp:

- (WebCore::Document::updateRangesAfterNodeChanges):

- * dom/Document.h:

- * dom/NamedNodeMap.cpp:

- (WebCore::NamedNodeMap::addAttribute):

- (WebCore::NamedNodeMap::removeAttribute):

- * dom/Node.cpp:

- (WebCore::removeNodeListCacheIfPossible):

- (WebCore::Node::unregisterDynamicNodeList):

- (WebCore::Node::invalidateNodeListsCacheAfterAttributeChanges):

- (WebCore::Node::invalidateNodeListsCacheAfterNodeChanges):

- (WebCore::Node::dispatchSubtreeModifiedEvent):

- * dom/Node.h:

- * dom/NodeRareData.h:

- * dom/StyledElement.cpp:

- (WebCore::StyledElement::classAttributeChanged):

-2011-12-08 Kenichi Ishibashi <bashi@chromium.org>

- Unreviewed, rolling out r102418.

- http://trac.webkit.org/changeset/102418

- https://bugs.webkit.org/show_bug.cgi?id=71870

- Caused Chromium build failure.

- * CMakeLists.txt:

- * DerivedSources.make:

- * DerivedSources.pri:

- * GNUmakefile.list.am:

- * Target.pri:

- * WebCore.gypi:

- * WebCore.xcodeproj/project.pbxproj:

- * bindings/js/JSWebGLRenderingContextCustom.cpp:

- (WebCore::toJS):

- * bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:

- (WebCore::toV8Object):

- * html/canvas/WebGLExtension.h:

- * html/canvas/WebGLRenderingContext.cpp:

- (WebCore::WebGLRenderingContext::getExtension):

- (WebCore::WebGLRenderingContext::getSupportedExtensions):

- (WebCore::WebGLRenderingContext::maybeRestoreContext):

- * html/canvas/WebGLRenderingContext.h:

- * html/canvas/WebKitLoseContext.cpp: Renamed from Source/WebCore/html/canvas/WebGLLoseContext.cpp.

- (WebCore::WebKitLoseContext::WebKitLoseContext):

- (WebCore::WebKitLoseContext::~WebKitLoseContext):

- (WebCore::WebKitLoseContext::getName):

- (WebCore::WebKitLoseContext::create):

- (WebCore::WebKitLoseContext::loseContext):

- (WebCore::WebKitLoseContext::restoreContext):

- * html/canvas/WebKitLoseContext.h: Renamed from Source/WebCore/html/canvas/WebGLLoseContext.h.

- * html/canvas/WebKitLoseContext.idl: Renamed from Source/WebCore/html/canvas/WebGLLoseContext.idl.

-2011-12-08 Fady Samuel <fsamuel@chromium.org>

- [Chromium] Enable viewport metatag

- https://bugs.webkit.org/show_bug.cgi?id=73495

- Reviewed by Darin Fisher.

- Recompute viewpot parameters on frame rect resize.

- * page/FrameView.cpp:

- (WebCore::FrameView::setFrameRect):

-2011-12-08 Kent Tamura <tkent@chromium.org>

- Build fix for r102419.

- https://bugs.webkit.org/show_bug.cgi?id=73916

- * platform/PopupMenuClient.h:

- (WebCore::PopupMenuClient::listBoxSelectItem):

- Remove unused argument names.

-2011-12-08 Kentaro Hara <haraken@chromium.org>

- Unreviewed. Rebaselined run-bindings-tests results.

- * bindings/scripts/test/JS/JSFloat64Array.cpp:

- * bindings/scripts/test/JS/JSTestEventConstructor.cpp:

- * bindings/scripts/test/JS/JSTestInterface.cpp:

- * bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp:

- * bindings/scripts/test/JS/JSTestNamedConstructor.cpp:

- * bindings/scripts/test/JS/JSTestObj.cpp:

- * bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp:

-2011-12-08 Mary Wu <mary.wu@torchmobile.com.cn>

- Upstream BlackBerry porting of MIMETypeRegistry/KeyboardEvent

- https://bugs.webkit.org/show_bug.cgi?id=73534

- Reviewed by Rob Buis.

- Other main contributors:

- Mike Fenton <mifenton@rim.com>

- Joe Mason <jmason@rim.com>

- Max Feil <mfeil@qnx.com>

- Lukas Sydorowski <lsydorowski@rim.com>

- Crystal Zhang <haizhang@rim.com>

- Initial upstream, no new tests.

- * platform/blackberry/MIMETypeRegistryBlackBerry.cpp: Added.

- (WebCore::MIMETypeRegistry::getMIMETypeForExtension):

- (WebCore::MIMETypeRegistry::getPreferredExtensionForMIMEType):

- (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):

- * platform/blackberry/PlatformKeyboardEventBlackBerry.cpp: Added.

- (WebCore::keyIdentifierForBlackBerryCharacter):

- (WebCore::windowsKeyCodeForBlackBerryCharacter):

- (WebCore::adjustCharacterFromOS):

- (WebCore::toWebCorePlatformKeyboardEventType):

- (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):

- (WebCore::PlatformKeyboardEvent::currentCapsLockState):

- (WebCore::PlatformKeyboardEvent::disambiguateKeyDownEvent):

- (WebCore::PlatformKeyboardEvent::getCurrentModifierState):

-2011-12-08 Sheriff Bot <webkit.review.bot@gmail.com>

- Unreviewed, rolling out r101619.

- http://trac.webkit.org/changeset/101619

- https://bugs.webkit.org/show_bug.cgi?id=74158

- this patch produces bad behaviour on mac (Requested by

- jeremya_ on #webkit).

- * page/EventHandler.cpp:

- (WebCore::EventHandler::handleMouseMoveEvent):

- (WebCore::EventHandler::dragSourceEndedAt):

- * rendering/RenderLayer.cpp:

- (WebCore::RenderLayer::updateHoverActiveState):

-2011-12-08 Adam Klein <adamk@chromium.org>

- [MutationObservers] V8LazyEventHandler breaks microtask delivery semantics

- https://bugs.webkit.org/show_bug.cgi?id=73492

- Reviewed by Adam Barth.

- Test: fast/mutation/inline-event-listener.html

- * bindings/v8/V8LazyEventListener.cpp:

- (WebCore::V8LazyEventListener::prepareListenerObject): Call v8::Script::Run directly instead of going through V8Proxy.

-2011-12-08 Hayato Ito <hayato@chromium.org>

- Suppress rendering of light children when ShadowRoot is dynamically created.

- https://bugs.webkit.org/show_bug.cgi?id=72441

- Reviewed by Ryosuke Niwa.

- Tests: fast/dom/shadow/dynamically-created-shadow-root-expected.html

- fast/dom/shadow/dynamically-created-shadow-root.html

- * dom/Element.cpp:

- (WebCore::Element::setShadowRoot):

-2011-12-08 Pierre Rossi <pierre.rossi@gmail.com>

- Drop ENABLE_NO_LISTBOX_RENDERING, and make it a runtime decision.

- https://bugs.webkit.org/show_bug.cgi?id=73916

- This was needed for Qt since the mobile theme, which can be picked

- up at runtime, delegates the rendering of list boxes.

- Reviewed by Kent Tamura.

- No new tests, there's no functional change.

- * html/HTMLSelectElement.cpp:

- (WebCore::HTMLSelectElement::usesMenuList):

- * html/HTMLSelectElement.h:

- * platform/PopupMenuClient.h:

- (WebCore::PopupMenuClient::listBoxSelectItem):

- (WebCore::PopupMenuClient::multiple):

- * platform/qt/RenderThemeQtMobile.h:

- (WebCore::RenderThemeQtMobile::delegatesMenuListRendering):

- * rendering/RenderMenuList.cpp:

- (WebCore::RenderMenuList::multiple):

- * rendering/RenderMenuList.h:

- * rendering/RenderTheme.h:

- (WebCore::RenderTheme::delegatesMenuListRendering):

-2011-12-08 Kenneth Russell <kbr@google.com>

- Rename WEBKIT_lose_context to WEBKIT_WEBGL_lose_context

- https://bugs.webkit.org/show_bug.cgi?id=71870

- Reviewed by James Robinson.

- Rename largely done with do-webcore-rename with a couple of

- necessary manual fixups. Ran WebGL layout tests.

- * CMakeLists.txt:

- * DerivedSources.make:

- * DerivedSources.pri:

- * GNUmakefile.list.am:

- * Target.pri:

- * WebCore.gypi:

- * WebCore.xcodeproj/project.pbxproj:

- * bindings/js/JSWebGLRenderingContextCustom.cpp:

- (WebCore::toJS):

- * bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:

- (WebCore::toV8Object):

- * html/canvas/WebGLExtension.h:

- * html/canvas/WebGLLoseContext.cpp: Copied from Source/WebCore/html/canvas/WebKitLoseContext.cpp.

- (WebCore::WebGLLoseContext::WebGLLoseContext):

- (WebCore::WebGLLoseContext::~WebGLLoseContext):

- (WebCore::WebGLLoseContext::getName):

- (WebCore::WebGLLoseContext::create):

- (WebCore::WebGLLoseContext::loseContext):

- (WebCore::WebGLLoseContext::restoreContext):

- * html/canvas/WebGLLoseContext.h: Copied from Source/WebCore/html/canvas/WebKitLoseContext.h.

- * html/canvas/WebGLLoseContext.idl: Copied from Source/WebCore/html/canvas/WebKitLoseContext.idl.

- * html/canvas/WebGLRenderingContext.cpp:

- (WebCore::WebGLRenderingContext::getExtension):

- (WebCore::WebGLRenderingContext::getSupportedExtensions):

- (WebCore::WebGLRenderingContext::maybeRestoreContext):

- * html/canvas/WebGLRenderingContext.h:

- * html/canvas/WebKitLoseContext.cpp: Removed.

- * html/canvas/WebKitLoseContext.h: Removed.

- * html/canvas/WebKitLoseContext.idl: Removed.

-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-08 Van Lam <vanlam@google.com>

- Caret keeps blinking during forward-delete

- https://bugs.webkit.org/show_bug.cgi?id=38564

- Reviewed by Darin Adler.

- Currently updateAppearance determines if the caret should stop blinking

- based on whether or not the editing operation changed the position of

- the caret; so the caret stops blinking in case of typing text and

- backwards delete (which always displace the caret) but does not stop

- blinking in the case of forward delete (which does not displace the

- caret).

- Added a boolean member function shouldStopCaretBlinking in EditCommand

- which will return true if the object is a TypingCommand (my

- understanding here is that all TypingCommands should stop the caret

- from blinking for a cycle, currently 0.5 seconds). Then used this

- function to stop the caret from blinking if the last editing command

- is a TypingCommand.

- * editing/EditCommand.h:

- (WebCore::EditCommand::shouldStopCaretBlinking):

- * editing/FrameSelection.cpp:

- (WebCore::FrameSelection::updateAppearance):

- * editing/TypingCommand.h:

- (WebCore::TypingCommand::shouldStopCaretBlinking):

-2011-12-08 Adam Klein <adamk@chromium.org>

- Use HashMap<Node*, OwnPtr<...>> in ChildListMutationScope

- https://bugs.webkit.org/show_bug.cgi?id=73964

- Reviewed by Darin Adler.

- Re-landing r102267 with a fix for the clang build.

- No new tests, refactoring only.

- * dom/ChildListMutationScope.cpp:

- (WebCore::ChildListMutationAccumulator::MutationAccumulationRouter::childAdded):

- (WebCore::ChildListMutationAccumulator::MutationAccumulationRouter::willRemoveChild):

- (WebCore::ChildListMutationAccumulator::MutationAccumulationRouter::incrementScopingLevel):

- (WebCore::ChildListMutationAccumulator::MutationAccumulationRouter::decrementScopingLevel):

-2011-12-08 Anders Carlsson <andersca@apple.com>

- WebScrollbarPainterDelegate should have a pointer to its Scrollbar

- https://bugs.webkit.org/show_bug.cgi?id=74149

- Reviewed by Darin Adler.

- This is another step towards making the scroll animation code more robust.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterDelegate initWithScrollbar:WebCore::]):

- (-[WebScrollbarPainterDelegate scrollAnimator]):

- (-[WebScrollbarPainterDelegate layer]):

- (-[WebScrollbarPainterDelegate mouseLocationInScrollerForScrollerImp:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateKnobAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateTrackAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateUIStateTransitionWithDuration:]):

- (-[WebScrollbarPainterDelegate scrollAnimatorDestroyed]):

- (WebCore::ScrollAnimatorMac::didAddVerticalScrollbar):

- (WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar):

-2011-12-08 Rakesh KN <rakesh.kn@motorola.com>

- keyboard event doesn't fire while moving mouse with button pressed

- https://bugs.webkit.org/show_bug.cgi?id=73821

- Reviewed by Alexey Proskuryakov.

- Autoscroll should not stop on key press.

- Test: fast/events/autoscroll-should-not-stop-on-keypress.html

- * page/EventHandler.cpp:

- (WebCore::EventHandler::keyEvent):

- Removed the check for autoscroll so that autoscroll is not stopped on

- key press and key event is processed.

-2011-12-08 Anders Carlsson <andersca@apple.com>

- Add scrollAnimator getter method to WebScrollbarPainterDelegate

- https://bugs.webkit.org/show_bug.cgi?id=74146

- Reviewed by Beth Dakin.

- * platform/mac/ScrollAnimatorMac.mm:

- (-[WebScrollbarPainterDelegate scrollAnimator]):

- (-[WebScrollbarPainterDelegate layer]):

- (-[WebScrollbarPainterDelegate mouseLocationInScrollerForScrollerImp:]):

- (-[WebScrollbarPainterDelegate setUpAlphaAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateKnobAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateTrackAlphaTo:duration:]):

- (-[WebScrollbarPainterDelegate scrollerImp:animateUIStateTransitionWithDuration:]):

-2011-12-08 James Robinson <jamesr@chromium.org>

- Improve handling of frame removal during requestAnimationFrame callback invocation

- https://bugs.webkit.org/show_bug.cgi?id=74036

- Reviewed by Adam Barth.

- See bug for details.

- Test: fast/animation/request-animation-frame-detach-element.html

- * dom/Document.cpp:

- (WebCore::Document::removedLastRef):

- (WebCore::Document::detach):

- * dom/Document.h:

- * dom/ScriptedAnimationController.cpp:

- (WebCore::ScriptedAnimationController::~ScriptedAnimationController):

- (WebCore::ScriptedAnimationController::serviceScriptedAnimations):

- (WebCore::ScriptedAnimationController::scheduleAnimation):

- * dom/ScriptedAnimationController.h:

- (WebCore::ScriptedAnimationController::create):

- (WebCore::ScriptedAnimationController::clearDocumentPointer):

- * page/FrameView.cpp:

- (WebCore::FrameView::serviceScriptedAnimations):

-2011-12-08 Yongjun Zhang <yongjun_zhang@apple.com>

- Use bitfield for bool data members in BitmapImage.

- https://bugs.webkit.org/show_bug.cgi?id=74102

- Reviewed by Darin Adler.

- Class BitmapImage and FrameData has bool data members, we can use bitfield for those data

- members to reduce the BitmapImage's memory footprint.

- * platform/graphics/BitmapImage.cpp:

- (WebCore::BitmapImage::BitmapImage):

- * platform/graphics/BitmapImage.h:

- (WebCore::FrameData::FrameData):

- * platform/graphics/cairo/ImageCairo.cpp:

- (WebCore::BitmapImage::BitmapImage):

- * platform/graphics/cg/ImageCG.cpp:

- (WebCore::BitmapImage::BitmapImage):

- * platform/graphics/openvg/ImageOpenVG.cpp:

- (WebCore::BitmapImage::BitmapImage):

- * platform/graphics/qt/ImageQt.cpp:

- (WebCore::BitmapImage::BitmapImage):

-2011-12-08 Anders Carlsson <andersca@apple.com>

- Fix a paste-o in ScrollAnimatorMac::updateScrollerStyle

- https://bugs.webkit.org/show_bug.cgi?id=74145

- Reviewed by Sam Weinig.

- Call setHorizontalScrollerImp to set the horizontal scrollbar painter.

- * platform/mac/ScrollAnimatorMac.mm:

- (WebCore::ScrollAnimatorMac::updateScrollerStyle):

-2011-12-08 Anders Carlsson <andersca@apple.com>

- Create one WebScrollbarPainterDelegate for each scrollbar

- https://bugs.webkit.org/show_bug.cgi?id=74142

- Reviewed by Sam Weinig.

- Create and destroy WebScrollbarPainterDelegate objects as scrollbars come and go.

- This is a step towards simplifying the WebScrollbarPainterDelegate object.

- * platform/mac/ScrollAnimatorMac.h:

- * platform/mac/ScrollAnimatorMac.mm:

- (WebCore::ScrollAnimatorMac::ScrollAnimatorMac):

- (WebCore::ScrollAnimatorMac::~ScrollAnimatorMac):

- (WebCore::ScrollAnimatorMac::didAddVerticalScrollbar):

- (WebCore::ScrollAnimatorMac::willRemoveVerticalScrollbar):

- (WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar):

- (WebCore::ScrollAnimatorMac::willRemoveHorizontalScrollbar):

- (WebCore::ScrollAnimatorMac::cancelAnimations):

-2011-12-08 David Reveman <reveman@chromium.org>

- [Chromium] Add per-tile painting flag to DumpRenderTree and rename AcceleratedDrawing to AcceleratedPainting in chromium specific code.

- https://bugs.webkit.org/show_bug.cgi?id=74017

- Reviewed by James Robinson.

- Add per-tile drawing to page settings.

- No new tests.

- * page/Settings.h:

- (WebCore::Settings::setPerTileDrawingEnabled):

- (WebCore::Settings::perTileDrawingEnabled):

- * testing/Internals.cpp:

- (WebCore::Internals::setPerTileDrawingEnabled):

- * testing/Internals.h:

-2011-12-08 Anders Carlsson <andersca@apple.com>

- Add a scrollbarPainterForScrollbar helper function

- https://bugs.webkit.org/show_bug.cgi?id=74139

- Reviewed by Sam Weinig.

- * platform/mac/ScrollAnimatorMac.mm:

- (macScrollbarTheme):

- (scrollbarPainterForScrollbar):

- (WebCore::ScrollAnimatorMac::mouseEnteredScrollbar):

- (WebCore::ScrollAnimatorMac::mouseExitedScrollbar):

- (WebCore::ScrollAnimatorMac::didAddVerticalScrollbar):

- (WebCore::ScrollAnimatorMac::willRemoveVerticalScrollbar):

- (WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar):

- (WebCore::ScrollAnimatorMac::willRemoveHorizontalScrollbar):

-2011-12-08 James Robinson <jamesr@chromium.org>

- [chromium] Move NonCompositedContentHost to WebKit

- https://bugs.webkit.org/show_bug.cgi?id=74047

- Reviewed by Kenneth Russell.

- Updates WebCore.gypi to remove files no longer within WebCore and removes unused forward declaration and include

- from LayerRendererChromium.

- * WebCore.gypi:

- * platform/graphics/chromium/LayerRendererChromium.cpp:

- * platform/graphics/chromium/LayerRendererChromium.h:

-2011-12-08 Sami Kyostila <skyostil@google.com>

- [chromium] Layer contents scale change should trigger invalidation

- https://bugs.webkit.org/show_bug.cgi?id=74086

- When the contents scale of a layer is changed, the entire contents of

- the layer should be marked dirty.

- Reviewed by James Robinson.

- * platform/graphics/chromium/LayerChromium.cpp:

- (WebCore::LayerChromium::setContentsScale):

-2011-12-08 Stephen White <senorblanco@chromium.org>

- [chromium] Add CSS_FILTERS custom binding file to WebCore.gypi.

- https://bugs.webkit.org/show_bug.cgi?id=74091

- Reviewed by Adam Barth.

- If it builds, you're happy.

- * WebCore.gypi:

-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:

- * WebCore.gypi: Added DOMWindowWebAudio.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

- 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):

-2011-12-07 Alexandre Elias <aelias@google.com>

- [chromium] Add page-scale animation support to Impl thread

- https://bugs.webkit.org/show_bug.cgi?id=72996

- Reviewed by James Robinson.

- This adds a new math helper class to compute the progress of the

- animation, and code in the CCLayerTreeHostImpl to apply the animation

- frame by frame.

- No new tests. (https://bugs.webkit.org/show_bug.cgi?id=71529 filed.)

- * WebCore.gypi:

- * platform/graphics/chromium/cc/CCInputHandler.h:

- * platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:

- (WebCore::CCLayerTreeHostImpl::animate):

- (WebCore::CCLayerTreeHostImpl::startPageScaleAnimation):

- (WebCore::CCLayerTreeHostImpl::processScrollDeltas):

- * platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:

- * platform/graphics/chromium/cc/CCPageScaleAnimation.cpp: Added.

- (WebCore::CCPageScaleAnimation::CCPageScaleAnimation):

- (WebCore::CCPageScaleAnimation::initialize):

- (WebCore::CCPageScaleAnimation::zoomTo):

- (WebCore::CCPageScaleAnimation::zoomWithAnchor):

- (WebCore::CCPageScaleAnimation::zoomElsewhere):

- (WebCore::CCPageScaleAnimation::scrollOffsetAtTime):

- (WebCore::CCPageScaleAnimation::pageScaleAtTime):

- (WebCore::CCPageScaleAnimation::isAnimationCompleteAtTime):

- (WebCore::CCPageScaleAnimation::progressRatioForTime):

- (WebCore::CCPageScaleAnimation::scrollOffsetAtRatio):

- (WebCore::CCPageScaleAnimation::pageScaleAtRatio):

- * platform/graphics/chromium/cc/CCPageScaleAnimation.h: Added.

- (WebCore::CCPageScaleAnimation::startTime):

- (WebCore::CCPageScaleAnimation::duration):

- (WebCore::CCPageScaleAnimation::endTime):

- (WebCore::CCPageScaleAnimation::finalScrollOffset):

- (WebCore::CCPageScaleAnimation::finalPageScale):

-2011-12-07 Shinya Kawanaka <shinyak@google.com>

- Editor::markAndReplaceFor should take Range instead of TextCheckingParagraph.

- https://bugs.webkit.org/show_bug.cgi?id=74035

- Reviewed by Hajime Morita.

- Editor::markAndReplaceFor takes chekcing range and paragraph range instead of

- spelling paragraph and grammar paragraph.

- No new tests. Covered by existing tests.

- * editing/Editor.cpp:

- (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges):

- (WebCore::Editor::markAndReplaceFor):

- * editing/Editor.h:

- * editing/TextCheckingHelper.cpp:

- (WebCore::TextCheckingParagraph::TextCheckingParagraph):

- * editing/TextCheckingHelper.h:

-2011-12-07 Ami Fischman <fischman@chromium.org>

- Force LTR layout for media controls even in RTL documents.

- https://bugs.webkit.org/show_bug.cgi?id=74024

- Reviewed by Darin Adler.

- Test: media/controls-layout-direction.html

- * css/mediaControls.css:

- (::-webkit-media-controls):

-2011-12-07 Mark Pilgrim <pilgrim@chromium.org>

- [FileSystem API] requestFileSystem successCallback is required

- https://bugs.webkit.org/show_bug.cgi?id=69637

- Reviewed by Darin Adler.

- * page/DOMWindow.idl: remove [Optional] flag from successCallback parameter

-2011-12-07 Leo Yang <leo.yang@torchmobile.com.cn>

- Upstream the BlackBerry porting of network Resource

- https://bugs.webkit.org/show_bug.cgi?id=73388

- Reviewed by Rob Buis.

- Other main contributors:

- Joe Mason <jmason@rim.com>

- Yong Li <yoli@rim.com>

- Gary Simmons <gsimmons@rim.com>

- Genevieve Mak <gmak@rim.com>

- Chris Guan <chris.guan@torchmobile.com.cn>

- Mike Lattanzio <mlattanzio@rim.com>

- Initial upstream, can't be built yet, no new tests.

- * platform/network/blackberry/ResourceError.h: Added.

- * platform/network/blackberry/ResourceErrorBlackBerry.cpp: Added.

- * platform/network/blackberry/ResourceHandleBlackBerry.cpp: Added.

- * platform/network/blackberry/ResourceRequest.h: Added.

- * platform/network/blackberry/ResourceRequestBlackBerry.cpp: Added.

- * platform/network/blackberry/ResourceResponse.h: Added.

- * platform/network/blackberry/ResourceResponseBlackBerry.cpp: Added.

-2011-12-07 Fady Samuel <fsamuel@chromium.org>

- [Chromium] Plumb DPI info into PlatformScreen

- https://bugs.webkit.org/show_bug.cgi?id=70556

- Reviewed by Darin Fisher.

- Make DPI information accessible from WebKit through

- PlatformScreen. This is useful when making scaling

- computations on various devices (e.g. Viewport meta tag).

- This patch adds DPI plumbing on Chromium Win/Mac/Linux

- platforms.

- * page/Screen.cpp:

- (WebCore::Screen::horizontalDPI):

- (WebCore::Screen::verticalDPI):

- * page/Screen.h:

- * platform/PlatformScreen.h:

- * platform/chromium/PlatformScreenChromium.cpp:

- (WebCore::screenHorizontalDPI):

- (WebCore::screenVerticalDPI):

- * platform/chromium/PlatformSupport.h:

- * platform/efl/PlatformScreenEfl.cpp:

- (WebCore::screenHorizontalDPI):

- (WebCore::screenVerticalDPI):

- * platform/gtk/PlatformScreenGtk.cpp:

- (WebCore::screenHorizontalDPI):

- (WebCore::screenVerticalDPI):

- * platform/mac/PlatformScreenMac.mm:

- (WebCore::screenHorizontalDPI):

- (WebCore::screenVerticalDPI):

- * platform/qt/PlatformScreenQt.cpp:

- (WebCore::screenHorizontalDPI):

- (WebCore::screenVerticalDPI):

- * platform/win/PlatformScreenWin.cpp:

- (WebCore::screenHorizontalDPI):

- (WebCore::screenVerticalDPI):

-2011-12-07 Aaron Colwell <acolwell@chromium.org>

- Revert mixed content handling for video fix and follow-up test expectations & Skipped changes.

- (r101883, r101918, r101927, r101981, r101986, r101997)

- https://bugs.webkit.org/show_bug.cgi?id=72178

- Reviewed by Adam Barth.

- * loader/SubresourceLoader.cpp:

- (WebCore::SubresourceLoader::willSendRequest):

- * loader/cache/CachedRawResource.cpp:

- * loader/cache/CachedRawResource.h:

- * loader/cache/CachedResource.cpp:

- (WebCore::defaultPriorityForResourceType):

- (WebCore::cachedResourceTypeToTargetType):

- * loader/cache/CachedResource.h:

- * loader/cache/CachedResourceLoader.cpp:

- (WebCore::createResource):

- (WebCore::CachedResourceLoader::checkInsecureContent):

- (WebCore::CachedResourceLoader::canRequest):

- (WebCore::CachedResourceLoader::requestResource):

-2011-12-07 Ryuan Choi <ryuan.choi@samsung.com>

- [EFL] Introduce AssertMatchingEnums.cpp.

- https://bugs.webkit.org/show_bug.cgi?id=65238

- Reviewed by Filip Pizlo.

- Remove switch statement which convert EWK_TOUCH_PointType enum values to

- WebCore::PlatformTouchPoint::State enum values.

- Newly added AssertMatchingEnums.cpp assure that they are equal.

- No new tests, no new functionality.

- * platform/efl/PlatformTouchEventEfl.cpp:

- (WebCore::PlatformTouchEvent::PlatformTouchEvent):

-2011-12-07 Shawn Singh <shawnsingh@chromium.org>

- [chromium] Clearing root surface should happen after damage tracking

- https://bugs.webkit.org/show_bug.cgi?id=73958

- Reviewed by James Robinson.

- No new semantics, covered by existing layout tests.

- * platform/graphics/chromium/LayerRendererChromium.cpp:

- (WebCore::LayerRendererChromium::clearSurfaceForDebug):

- (WebCore::LayerRendererChromium::drawLayersOntoRenderSurfaces):

- (WebCore::LayerRendererChromium::drawLayersInternal):

- * platform/graphics/chromium/LayerRendererChromium.h:

-2011-12-07 Florin Malita <fmalita@google.com>

- <li value="0"> behaves like <li> (the same for negative numbers)

- https://bugs.webkit.org/show_bug.cgi?id=73911

- Reviewed by Alexey Proskuryakov.

- Allow LI values <= 0 and consolidate the value processing logic.

- * html/HTMLLIElement.cpp:

- (WebCore::HTMLLIElement::parseMappedAttribute):

- Delegated value parsing to parseValue().

- (WebCore::HTMLLIElement::attach):

- Ditto. Explicit value null testing is no longer necessary,

- as parseValue's toInt() performs an equivalent check.

- (WebCore::HTMLLIElement::parseValue):

- Consolidated value parsing logic.

- * html/HTMLLIElement.h:

-2011-12-07 Joshua Bell <jsbell@chromium.org>

- IndexedDB: Replace bool args in IDBKeyRange private methods with enum

- https://bugs.webkit.org/show_bug.cgi?id=70743

- Reviewed by Tony Chang.

- No new tests - no functional changes.

- * storage/IDBKeyRange.cpp:

- (WebCore::IDBKeyRange::IDBKeyRange):

- (WebCore::IDBKeyRange::bound):

- * storage/IDBKeyRange.h:

- (WebCore::IDBKeyRange::create):

- (WebCore::IDBKeyRange::lowerOpen):

- (WebCore::IDBKeyRange::upperOpen):

-2011-12-07 Sheriff Bot <webkit.review.bot@gmail.com>

- Unreviewed, rolling out r102267.

- http://trac.webkit.org/changeset/102267

- https://bugs.webkit.org/show_bug.cgi?id=74032

- Breaks build on Chromium Mac Debug (Requested by aklein on

- #webkit).

- * dom/ChildListMutationScope.cpp:

- (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::childAdded):

- (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::willRemoveChild):

- (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::incrementScopingLevel):

- (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::decrementScopingLevel):

-2011-12-07 Kentaro Hara <haraken@chromium.org>

- REGRESSION (r95249): Right side can be truncated when printing

- https://bugs.webkit.org/show_bug.cgi?id=73868

- Reviewed by Darin Adler.

- When we print a page with an overflowed width, the right side of the page

- can be truncated. This is due to a wrong rendering calculation.

- Since 'maximumShrinkFactor' is a ratio based on 'pageSize',

- 'maximumShrinkFactor' should multiply (not 'originalPageSize') but 'pageSize'.

- This bug happens if all the following conditions are met:

- - pageLogicalWidth < docLogicalWidth

- - originalPageSize.width * maximumShrinkFactor < docLogicalWidth

- - docLogicalWidth < pageLogicalWidth * maximumShrinkFactor

- Test: printing/width-overflow.html

- * page/FrameView.cpp:

- (WebCore::FrameView::forceLayoutForPagination):

-2011-12-07 Yong Li <yoli@rim.com>

- Defer ScriptExecutionContext::Task's in Document when page loading is deferred.

- Schedule them with timer when page loading is resumed. The tasks will be performed

- in the original order. This fixes the problem that database callbacks could be missed

- when page loading was deferred.

- https://bugs.webkit.org/show_bug.cgi?id=49401

- Reviewed by Darin Adler.

- Manual test added: ManualTests/database-callback-deferred.html.

- * dom/Document.cpp:

- (WebCore::Document::Document):

- (WebCore::Document::didReceiveTask):

- (WebCore::Document::postTask):

- (WebCore::Document::pendingTasksTimerFired):

- (WebCore::Document::suspendScheduledTasks):

- (WebCore::Document::resumeScheduledTasks):

- * dom/Document.h:

- * page/PageGroupLoadDeferrer.cpp:

- (WebCore::PageGroupLoadDeferrer::PageGroupLoadDeferrer):

- (WebCore::PageGroupLoadDeferrer::~PageGroupLoadDeferrer):

-2011-12-07 Andreas Kling <kling@webkit.org>

- RenderObject::style(): Inline early-return condition.

- <http://webkit.org/b/74019>

- Reviewed by Anders Carlsson.

- style() was very hot (6.1%) when scrolling around on youtube.com,

- and 100% of the calls were taking the early return path.

- Inlined the !isRenderFlowThread() check and renamed the function to

- styleSlowCase().

- * rendering/RenderObject.cpp:

- (WebCore::RenderObject::styleSlowCase):

- * rendering/RenderObject.h:

- (WebCore::RenderObject::style):

-2011-12-07 Adam Klein <adamk@chromium.org>

- Use HashMap<Node*, OwnPtr<...>> in ChildListMutationScope

- https://bugs.webkit.org/show_bug.cgi?id=73964

- Reviewed by Ryosuke Niwa.

- No new tests, refactoring only.

- * dom/ChildListMutationScope.cpp:

- (WebCore::ChildListMutationAccumulator::MutationAccumulationRouter::childAdded):

- (WebCore::ChildListMutationAccumulator::MutationAccumulationRouter::willRemoveChild):

- (WebCore::ChildListMutationAccumulator::MutationAccumulationRouter::incrementScopingLevel):

- (WebCore::ChildListMutationAccumulator::MutationAccumulationRouter::decrementScopingLevel):

-2011-12-07 Andreas Kling <kling@webkit.org>

- RenderLayer::updateZOrderLists(): Inline early-return condition.

- <http://webkit.org/b/74013>

- Reviewed by Anders Carlsson.

- updateZOrderLists() was hot (1.2%) when scrolling around on youtube.com,

- and 85% of the calls were taking the early return path.

- Inlined the two checks for the early return and renamed the function

- to updateZOrderListsSlowCase(). Also reversed their order to avoid the

- virtual call (RenderObject::isRenderView()) if possible.

- * rendering/RenderLayer.cpp:

- (WebCore::RenderLayer::updateZOrderListsSlowCase):

- * rendering/RenderLayer.h:

- (WebCore::RenderLayer::updateZOrderLists):

-2011-12-07 Sheriff Bot <webkit.review.bot@gmail.com>

- Unreviewed, rolling out r102244.

- http://trac.webkit.org/changeset/102244

- https://bugs.webkit.org/show_bug.cgi?id=74016

- caused debug test timeouts (Requested by simonjam on #webkit).

- * platform/graphics/skia/ImageBufferSkia.cpp:

- (WebCore::putImageData):

- (WebCore::ImageBuffer::putUnmultipliedImageData):

- (WebCore::ImageBuffer::putPremultipliedImageData):

-2011-12-07 Adam Klein <adamk@chromium.org>

- Layout Test inspector/debugger/dom-breakpoints.html fails on chromium linux debug with ENABLE(MUTATION_OBSERVERS)

- https://bugs.webkit.org/show_bug.cgi?id=73919

- Reviewed by Ojan Vafai.

- Use StyleAttributeMutationScope to manage DOM breakpoints for style property changes.

- Instead of calling InspectorInstrumentation::didInvalidateStyleAttr()

- directly in setNeedsStyleRecalc, set a bool in the current

- StyleAttributeMutationScope, and delay the call until the scope's

- counter runs down to zero. This keeps the inspector JS from re-entering

- CSSMutableStyleDeclaration until all StyleAttributeMutationScope work is done.

- Also fix a small bug in StyleAttributeMutationScope, where

- s_shouldDeliver wasn't getting reset properly to false.

- * css/CSSMutableStyleDeclaration.cpp:

- (WebCore::CSSMutableStyleDeclaration::setNeedsStyleRecalc):

-2011-12-07 Ken Buchanan <kenrb@chromium.org>

- Crash from multicol spans with layers

- https://bugs.webkit.org/show_bug.cgi?id=68030

- Reviewed by David Hyatt.

- The layer tree diverges from the render tree when a span is being split

- between columns. This patch causes the layer tree to be updated when necessary.

- * rendering/RenderBlock.cpp:

- (WebCore::RenderBlock::splitFlow)

- (WebCore::RenderBlock::splitBlocks)

-2011-12-07 Alexey Proskuryakov <ap@apple.com>

- Handling of !important in inline style sets is broken

- https://bugs.webkit.org/show_bug.cgi?id=73941

- Reviewed by Dave Hyatt.

- This behavior was introduced in bug 8223 to match IE and Firefox. But it doesn't appear that we're matching

- any browser today, and CSSOM spec agrees with them.

- * WebCore.exp.in: Don't export CSSStyleDeclaration::setProperty(), no one is using it.

- * bindings/js/JSCSSStyleDeclarationCustom.cpp: (WebCore::JSCSSStyleDeclaration::putDelegate): Use regular

- setProperty(), not the incorrect version that's being removed. Properties set via IDL attributes are never

- important.

- * bindings/v8/custom/V8CSSStyleDeclarationCustom.cpp: (WebCore::V8CSSStyleDeclaration::namedPropertySetter):

- Made the same fix for v8. Why did v8 bindings authors copy/paste the code?!

- * css/CSSStyleDeclaration.cpp:

- * css/CSSStyleDeclaration.h:

- Removed a version of setProperty() that attempted to parse the value and extract !important from it.

- * html/ImageDocument.cpp:

- (WebCore::ImageDocument::resizeImageToFit):

- (WebCore::ImageDocument::restoreImageSize):

- (WebCore::ImageDocument::windowSizeChanged):

- * page/DragController.cpp:

- (WebCore::DragController::concludeEditDrag):

- We never needed to use this version of setProperty() here, it was just unnecessarily slower.

-2011-12-07 Xianzhu Wang <wangxianzhu@chromium.org>

- In FontCacheAndroid.cpp should keep the pointer valid returned from CString::data()

- https://bugs.webkit.org/show_bug.cgi?id=73849

- The changed code has been covered by many existing layout tests.

- Reviewed by Adam Barth.

- * platform/graphics/chromium/FontCacheAndroid.cpp:

- (WebCore::FontCache::createFontPlatformData):

-2011-12-07 Xiaomei Ji <xji@chromium.org>

- Turn on move caret by word visually for Windows platform.

- https://bugs.webkit.org/show_bug.cgi?id=59652

- Reviewed by Ryosuke Niwa.

- We already support (arrow key) moving cursor by character in visual order.

- This patch implements (ctrl/alt-arrow) moving cursor by word in visual order (in Windows).

- It matches Firefox's default behavior.

- Without this patch, ctrl(alt for mac)-arrow key or

- selection.modify("move", "left"/"right", "word") moves cursor by word in logical order.

- IE implements moving cursor by logical order for both arrow key and ctrl-arrow key.

- Firefox implements moving cursor by visual order for both operations.

- From Chromium bug report, native speakers would like moving cursor by visual order since it

- is more intuitive.

- The patch is only enabled for Windows (by EditingBehavior) because current implementation

- matches Windows' native behavior.

- For exmaple, if the logical text is "abc def hij", the cursor positions are

- "|abc |def |hij|" no matter pressing ctrl-left-arrow or ctrl-right-arrow.

- Mac and Linux's native behavior is slightly different. In which, when pressing

- ctrl-left-arrow, the cursor positions are "|abc |def |hij|". When pressing ctrl-right-arrow,

- the cursor positions are "|abc| def| hij|". We will implement it next.

- Test: editing/selection/move-left-right-by-word-mac.html

- * editing/EditingBehavior.h:

- (WebCore::EditingBehavior::shouldMoveLeftRightByWordInVisualOrder):

- * editing/FrameSelection.cpp: Remove experimental enum WebKitVisualWordGranularity.

- (WebCore::FrameSelection::modifyExtendingRight):

- (WebCore::FrameSelection::modifyExtendingForward):

- (WebCore::FrameSelection::modifyMovingRight):

- (WebCore::FrameSelection::modifyMovingForward):

- (WebCore::FrameSelection::modifyExtendingLeft):

- (WebCore::FrameSelection::modifyExtendingBackward):

- (WebCore::FrameSelection::modifyMovingLeft):

- (WebCore::FrameSelection::modifyMovingBackward):

- * editing/TextGranularity.h: Remove experimental enum WebKitVisualWordGranularity.

- * editing/VisibleSelection.cpp: Remove experimental enum WebKitVisualWordGranularity.

- (WebCore::VisibleSelection::setStartAndEndFromBaseAndExtentRespectingGranularity):

- * page/DOMSelection.cpp: Remove experimental experimental flag -webkit-visual-word.

- (WebCore::DOMSelection::modify):

-2011-12-07 Jonathan Backer <backer@chromium.org>

- [chromium] Plumb damage from WebExternalTextureLayer and WebPluginContainer to CCDamageTracker

- https://bugs.webkit.org/show_bug.cgi?id=73485

- Reviewed by Darin Fisher.

- * platform/graphics/chromium/PluginLayerChromium.cpp:

- (WebCore::PluginLayerChromium::updateCompositorResources):

- (WebCore::PluginLayerChromium::invalidateRect):

- * platform/graphics/chromium/PluginLayerChromium.h:

-2011-12-07 Mary Wu <mary.wu@torchmobile.com.cn>

- Upstream 5 files into WebCore/platform/blackberry

- https://bugs.webkit.org/show_bug.cgi?id=73632

- Reviewed by Rob Buis.

- Initial upstream, no new tests.

- * PlatformBlackBerry.cmake: Remove two empty files from build list.

- * platform/blackberry/PopupMenuBlackBerry.cpp: Added.

- (WebCore::PopupMenuBlackBerry::PopupMenuBlackBerry):

- (WebCore::PopupMenuBlackBerry::~PopupMenuBlackBerry):

- (WebCore::PopupMenuBlackBerry::show):

- (WebCore::PopupMenuBlackBerry::hide):

- (WebCore::PopupMenuBlackBerry::updateFromElement):

- (WebCore::PopupMenuBlackBerry::disconnectClient):

- * platform/blackberry/PopupMenuBlackBerry.h: Added.

- (WebCore::PopupMenuBlackBerry::client):

- * platform/blackberry/ScrollbarThemeBlackBerry.cpp: Added.

- (WebCore::ScrollbarTheme::nativeTheme):

- * platform/blackberry/SearchPopupMenuBlackBerry.cpp: Added.

- (WebCore::SearchPopupMenuBlackBerry::SearchPopupMenuBlackBerry):

- (WebCore::SearchPopupMenuBlackBerry::popupMenu):

- (WebCore::SearchPopupMenuBlackBerry::enabled):

- (WebCore::SearchPopupMenuBlackBerry::saveRecentSearches):

- (WebCore::SearchPopupMenuBlackBerry::loadRecentSearches):

- * platform/blackberry/SearchPopupMenuBlackBerry.h: Added.

-2011-12-07 Dan Bernstein <mitz@apple.com>

- Fixed the definition of BUTTON_CELL_DRAW_WITH_FRAME_DRAWS_FOCUS_RING from r102246.

- * platform/mac/ThemeMac.mm: Added parentheses.

-2011-12-07 Dan Bernstein <mitz@apple.com>

- <rdar://problem/10542095> Focus rings are not drawn around push buttons, radio buttons and checkboxes

- Reviewed by Darin Adler.

- Instead of relying on -setShowsFirstResponder: to make -drawWithFrame:inView: draw the focus

- ring, use -drawFocusRingMaskWithFrame:inView:.

- * platform/mac/ThemeMac.mm:

- (-[NSCell _web_drawFocusRingWithFrame:inView:]): Added. Sets up the focus ring style and a

- transparency layer, then uses -drawFocusRingMaskWithFrame:inView: to draw the focus ring.

- (WebCore::updateStates): Eliminated calls to get and set showsFirstResponder.

- (WebCore::paintCheckbox): Changed to use -_web_drawFocusRingWithFrame:inView:.

- (WebCore::paintRadio): Ditto.

- (WebCore::paintButton): Ditto.

-2011-12-07 Brian Salomon <bsalomon@google.com>

- [CHROMIUM/SKIA] Handle put[Un/Pre]multipliedImageData conversions in Skia rather than ImageBuffer

- https://bugs.webkit.org/show_bug.cgi?id=73953

- Reviewed by Stephen White.

- Tested by existing canvas2d layout tests.

- * platform/graphics/skia/ImageBufferSkia.cpp:

- (WebCore::putImageData):

- (WebCore::ImageBuffer::putUnmultipliedImageData):

- (WebCore::ImageBuffer::putPremultipliedImageData):

-2011-12-07 Andreas Kling <kling@webkit.org>

- Micro-optimize ScrollView::visibleContentRect().

- <http://webkit.org/b/74001>

- Reviewed by Anders Carlsson.

- Reorder the scrollbar exclusion code to minimize the number of virtual calls

- to ScrollableArea::verticalScrollbar(), ScrollableArea::horizontalScrollbar()

- and Scrollbar::isOverlayScrollbar().

- * platform/ScrollView.cpp:

- (WebCore::ScrollView::visibleContentRect):

-2011-12-07 Andreas Kling <kling@webkit.org>

- ApplyPropertyBorderImage: Remove unneeded template argument for mapNinePieceImage().

- <http://webkit.org/b/73998>

- Reviewed by Antti Koivisto.

- Have ApplyPropertyBorderImage call mapNinePieceImage() directly now that it's

- public (instead of passing it as a template argument.)

- * css/CSSStyleApplyProperty.cpp:

- (WebCore::ApplyPropertyBorderImage::applyValue):

- (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):

-2011-12-07 Jessie Berlin <jberlin@apple.com>

- Mac build fix after r102235.

- * WebCore.exp.in:

-2011-11-30 Simon Hausmann <simon.hausmann@nokia.com>

- [Qt] V8 debug build fixes.

- Reviewed by Tor Arne Vestbø.

- * Target.pri: Add missing files to the build.

- * loader/SubresourceLoader.cpp: Add missing CString.h header file inclusion,

- that is implicitly included with Chromium builds and only needed in ASSERTS

- in debug builds.

- * loader/cache/CachedResource.cpp: Ditto.

- * page/FrameTree.cpp: Ditto.

- * platform/graphics/MediaPlayer.cpp: Ditto.

-2011-11-30 Simon Hausmann <simon.hausmann@nokia.com>

- [Qt] V8 build fixes.

- Reviewed by Tor Arne Vestbø.

- * Target.pri: Don't load(javascriptcore) if we're building with v8.

-2011-12-07 Mary Wu <mary.wu@torchmobile.com.cn>

- Change function name InitializeLoggingChannelsIfNecessary to follow coding style guideline

- https://bugs.webkit.org/show_bug.cgi?id=73986

- Reviewed by Kenneth Rohde Christiansen.

- Just function name change, no new tests.

- * platform/Logging.h:

- * platform/efl/LoggingEfl.cpp:

- (WebCore::initializeLoggingChannelsIfNecessary):

- * platform/gtk/LoggingGtk.cpp:

- (WebCore::initializeLoggingChannelsIfNecessary):

- * platform/mac/LoggingMac.mm:

- (WebCore::initializeLoggingChannelsIfNecessary):

- * platform/qt/LoggingQt.cpp:

- (WebCore::initializeLoggingChannelsIfNecessary):

- * platform/win/LoggingWin.cpp:

- (WebCore::initializeLoggingChannelsIfNecessary):

- * platform/wx/LoggingWx.cpp:

- (WebCore::initializeLoggingChannelsIfNecessary):

-2011-12-07 Mihnea Ovidenie <mihnea@adobe.com>

- [CSSRegions]Add support for background-color in region styling

- https://bugs.webkit.org/show_bug.cgi?id=71488

- Reviewed by David Hyatt.

- Tests: fast/regions/region-style-block-background-color.html

- fast/regions/region-style-block-background-color2.html

- fast/regions/region-style-image-background-color.html

- fast/regions/region-style-inline-background-color.html

- * WebCore.exp.in:

- * css/CSSStyleSelector.cpp:

- (WebCore::RuleData::regionStyleRule):

- (WebCore::CSSStyleSelector::CSSStyleSelector):

- (WebCore::CSSStyleSelector::addMatchedDeclaration):

- (WebCore::CSSStyleSelector::matchRules):

- (WebCore::CSSStyleSelector::matchAllRules):

- (WebCore::CSSStyleSelector::initForRegionStyling):

- (WebCore::CSSStyleSelector::styleForElement):

- (WebCore::CSSStyleSelector::pseudoStyleForElement):

- (WebCore::RuleData::RuleData):

- (WebCore::RuleSet::RuleSet):

- (WebCore::RuleSet::addToRuleSet):

- (WebCore::CSSStyleSelector::applyDeclarations):

- (WebCore::isValidRegionStyleProperty):

- (WebCore::CSSStyleSelector::applyProperty):

- * css/CSSStyleSelector.h:

- (WebCore::CSSStyleSelector::setRegionForStyling):

- (WebCore::CSSStyleSelector::regionForStyling):

- (WebCore::CSSStyleSelector::applyPropertyToRegionStyle):

- * rendering/RenderFlowThread.cpp:

- (WebCore::RenderFlowThread::clearRenderRegionRangeMap):

- (WebCore::RenderFlowThread::~RenderFlowThread):

- (WebCore::RenderFlowThread::layout):

- (WebCore::RenderFlowThread::clearRenderObjectCustomStyle):

- (WebCore::RenderFlowThread::setRegionRangeForBox):

- * rendering/RenderFlowThread.h:

- * rendering/RenderLayer.cpp:

- (WebCore::CurrentRenderRegionMaintainer::CurrentRenderRegionMaintainer):

- (WebCore::CurrentRenderRegionMaintainer::~CurrentRenderRegionMaintainer):

- (WebCore::RenderLayer::paint):

- (WebCore::RenderLayer::hitTest):

- * rendering/RenderObject.cpp:

- (WebCore::RenderObject::style):

- * rendering/RenderObject.h:

- (WebCore::RenderObject::canHaveRegionStyle):

- * rendering/RenderObjectChildList.cpp:

- (WebCore::RenderObjectChildList::removeChildNode):

- * rendering/RenderRegion.cpp:

- (WebCore::RenderRegion::layout):

- (WebCore::RenderRegion::renderObjectRegionStyle):

- (WebCore::RenderRegion::computeStyleInRegion):

- (WebCore::RenderRegion::clearObjectStyleInRegion):

- * rendering/RenderRegion.h:

- * rendering/RenderView.cpp:

- (WebCore::RenderView::RenderView):

- * rendering/RenderView.h:

- (WebCore::RenderView::currentRenderRegion):

- (WebCore::RenderView::setCurrentRenderRegion):

-2011-12-01 Vsevolod Vlasov <vsevik@chromium.org>

- Web Inspector: Extract default call stack creation and check for front-end from console.

- https://bugs.webkit.org/show_bug.cgi?id=73566

- Reviewed by Yury Semikhatsky.

- * bindings/js/ScriptCallStackFactory.cpp:

- (WebCore::createScriptCallStack):

- * 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::createScriptCallStack):

- * bindings/v8/ScriptCallStackFactory.h:

- * inspector/InspectorInstrumentation.cpp:

- (WebCore::InspectorInstrumentation::hasFrontendForScriptContext):

- * inspector/InspectorInstrumentation.h:

- (WebCore::InspectorInstrumentation::hasFrontendForScriptContext):

- * inspector/WorkerInspectorController.h:

- (WebCore::WorkerInspectorController::hasFrontend):

- * page/Console.cpp:

- * page/Console.h:

-2011-12-07 Shinya Kawanaka <shinyak@google.com>

- Internals should have a method to reutrn the max sequence number of spellcheck reqeust.

- https://bugs.webkit.org/show_bug.cgi?id=73511

- Reviewed by Hajime Morita.

- Internal state of SpellChecker should be able to be exposed for testing SpellChecker.

- This patch will enable us to know asynchronous spellcheck has finished or not.

- Test: editing/spelling/spellcheck-sequencenum.html

- * editing/SpellChecker.cpp:

- (WebCore::SpellChecker::SpellChecker):

- (WebCore::SpellChecker::createRequest):

- (WebCore::SpellChecker::didCheck):

- * editing/SpellChecker.h:

- (WebCore::SpellChecker::lastRequestSequence):

- Interface to take SpellCheck sequence numbers.

- (WebCore::SpellChecker::lastProcessedSequence): ditto.

- * testing/Internals.cpp:

- (WebCore::spellchecker):

- (WebCore::Internals::lastSpellCheckRequestSequence):

- (WebCore::Internals::lastSpellCheckProcessedSequence):

- * testing/Internals.h:

- * testing/Internals.idl:

-2011-12-07 Ryosuke Niwa <rniwa@webkit.org>

- TypingCommand duplicates code to obtain the last typing command

- https://bugs.webkit.org/show_bug.cgi?id=73984

- Reviewed by Kent Tamura.

- Extracted lastTypingCommandIfStillOpenForTyping out of isOpenForMoreTypingCommand

- and a bunch of TypingCommand static member functions.

- Also made more member functions of TypingCommand private.

- * editing/Editor.cpp:

- (WebCore::Editor::setComposition):

- * editing/FrameSelection.cpp:

- (WebCore::FrameSelection::setSelection):

- * editing/TypingCommand.cpp:

- (WebCore::TypingCommand::deleteSelection):

- (WebCore::TypingCommand::deleteKeyPressed):

- (WebCore::TypingCommand::forwardDeleteKeyPressed):

- (WebCore::TypingCommand::insertText):

- (WebCore::TypingCommand::insertLineBreak):

- (WebCore::TypingCommand::insertParagraphSeparatorInQuotedContent):

- (WebCore::TypingCommand::insertParagraphSeparator):

- (WebCore::TypingCommand::lastTypingCommandIfStillOpenForTyping):

- (WebCore::TypingCommand::closeTyping):

- * editing/TypingCommand.h:

- (WebCore::TypingCommand::isOpenForMoreTyping):

- (WebCore::TypingCommand::closeTyping):

-2011-12-06 Mary Wu <mary.wu@torchmobile.com.cn>

- upstream BlackBerry porting of KURL/Logging

- https://bugs.webkit.org/show_bug.cgi?id=73524

- Reviewed by Antonio Gomes.

- * platform/blackberry/KURLBlackBerry.cpp: Added.

- (WebCore::KURL::fileSystemPath):

- * platform/blackberry/LoggingBlackBerry.cpp: Added.

- (WebCore::initializeWithUserDefault):

- (WebCore::InitializeLoggingChannelsIfNecessary):

-2011-12-06 Leo Yang <leo.yang@torchmobile.com.cn>

- [BlackBerry] Remove redundant files in PlatformBlackBerry.cmake

- https://bugs.webkit.org/show_bug.cgi?id=73976

- Reviewed by Antonio Gomes.

- The listing of the following files in PlatformBlackBerry.cmake are redundant. They should be removed.

- platform/network/blackberry/MultipartResponseDelegate.cpp

- platform/network/blackberry/NetworkManager.cpp

- platform/network/blackberry/NetworkStateNotifierBlackBerry.cpp

- platform/network/blackberry/ResourceErrorBlackBerry.cpp

- platform/network/blackberry/ResourceRequestBlackBerry.cpp

- * PlatformBlackBerry.cmake:

-2011-12-06 Shinya Kawanaka <shinyak@google.com>

- Refactoring: Editor::markAllMisspellingsAndBadGrammarInRanges should be refactored.

- https://bugs.webkit.org/show_bug.cgi?id=73628

- Reviewed by Hajime Morita.

- Extracted a code for adding markers and replacing misspelled words from WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges.

- No new tests. covered by existing tests.

- * editing/Editor.cpp:

- (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges):

- Extracted a code for adding markers and replacing missplled words, and moved to markAndReplaceFor.

- (WebCore::Editor::markAndReplaceFor):

- * editing/Editor.h:

-2011-12-06 Julien Chaffraix <jchaffraix@webkit.org>

- Avoid calling calculateRects in RenderLayer::paintLayer when the rectangles are not needed

- https://bugs.webkit.org/show_bug.cgi?id=73754

- Reviewed by Simon Fraser.

- Performance change, no change in behavior.

- RenderLayer::paintLayer can easily be called a million time when scrolling on a big table with

- td { overflow: hidden; }. We would spend a lot of time recomputing the rectangles that we never

- unused for painting as our layer was not self-painting (clipping layer only) and we did not paint

- some overlay scrollbars.

- * rendering/RenderLayer.cpp:

- (WebCore::RenderLayer::paintLayer):

- Simplified and moved the shouldPaint logic earlier in the function. Now the branches

- are checking the same boolean which makes the logic more obvious. A consequence of

- filling shouldPaint earlier is that we call |calculateRects| only if there is a chance

- the rectangles will used. Also cached the result of isSelfPaintingLayer() in a local

- variable (isSelfPaintingLayer() is fairly expensive due to several virtual calls).

- * rendering/RenderLayerBacking.cpp:

- (WebCore::RenderLayerBacking::paintIntoLayer):

- For coherency, applied the same optimizations here too: added an early return instead

- of conditionaly call |calculateRects| as we don't have to restore any clip.

-2011-12-06 Benjamin Poulain <benjamin@webkit.org>

- Simplify KURL's checkEncodedString()

- https://bugs.webkit.org/show_bug.cgi?id=73890

- Reviewed by Andreas Kling.

- The Macro UNUSED_PARAM is not supposed to be used for this case,

- use ASSERT_UNUSED instead.

- * platform/KURL.cpp:

- (WebCore::checkEncodedString):

-2011-12-06 Ryosuke Niwa <rniwa@webkit.org>

- The code to create a NodeListsNodeData is duplicated everywhere

- https://bugs.webkit.org/show_bug.cgi?id=73961

- Reviewed by Darin Adler.

- Extracted the logic to create NodeListsNodeData as NodeRareData::ensureNodeLists.

- * dom/Document.cpp:

- (WebCore::Document::getItems):

- * dom/Node.cpp:

- (WebCore::Node::childNodes):

- (WebCore::Node::registerDynamicNodeList):

- (WebCore::Node::getElementsByTagName):

- (WebCore::Node::getElementsByTagNameNS):

- (WebCore::Node::getElementsByName):

- (WebCore::Node::getElementsByClassName):

- * dom/NodeRareData.h:

- (WebCore::NodeRareData::ensureNodeLists):

- * html/HTMLFormControlElement.cpp:

- (WebCore::HTMLFormControlElement::labels):

-2011-12-06 Leo Yang <leo.yang@torchmobile.com.cn>

- Upstream about: feature in WebKit/blackberry/WebCoreSupport/

- https://bugs.webkit.org/show_bug.cgi?id=73612

- Reviewed by Antonio Gomes.

- * PlatformBlackBerry.cmake: Move platform/network/blackberry/AboutData.{h, cpp}

- to WebKit/blackberry/WebCoreSupport

-2011-12-06 Benjamin Poulain <bpoulain@apple.com>

- WebKit Mac does not build without CONTEXT MENU

- https://bugs.webkit.org/show_bug.cgi?id=73962

- Reviewed by Pavel Feldman.

- In the patch r100903, the symbols were exported under ENABLE(CONTEXT_MENUS)

- because the feature is triggered from the menus.

- The implementation has no dependency on the context menu but is necessary to build

- when the inspector is enabled.

- This patch moves the exported symbols from ENABLE(CONTEXT_MENUS) to ENABLE(INSPECTOR).

- * WebCore.exp.in:

-2011-12-06 Adrienne Walker <enne@google.com>

- [chromium] setNeedsCommit on non-composited host layers should trigger commit

- https://bugs.webkit.org/show_bug.cgi?id=73711

- Reviewed by James Robinson.

- Pipe non-composited content host syncs to setNeedsCommit.

- Since now the NonCompositedContentHost generates setNeedsCommit, don't

- call it unnecessarily, e.g. calling setBackgroundColor to the same

- color each frame should not retrigger more commits.

- * platform/graphics/chromium/GraphicsLayerChromium.cpp:

- (WebCore::GraphicsLayerChromium::setAnchorPoint):

- (WebCore::GraphicsLayerChromium::setTransform):

- (WebCore::GraphicsLayerChromium::setChildrenTransform):

- (WebCore::GraphicsLayerChromium::setMasksToBounds):

- (WebCore::GraphicsLayerChromium::setBackgroundColor):

- (WebCore::GraphicsLayerChromium::clearBackgroundColor):

- (WebCore::GraphicsLayerChromium::setContentsOpaque):

- (WebCore::GraphicsLayerChromium::setBackfaceVisibility):

- (WebCore::GraphicsLayerChromium::setOpacity):

- * platform/graphics/chromium/LayerChromium.cpp:

- (WebCore::LayerChromium::setNeedsCommit):

- (WebCore::LayerChromium::setAnchorPoint):

- (WebCore::LayerChromium::setAnchorPointZ):

- (WebCore::LayerChromium::setBackgroundColor):

- (WebCore::LayerChromium::setMasksToBounds):

- (WebCore::LayerChromium::setMaskLayer):

- (WebCore::LayerChromium::setOpacity):

- (WebCore::LayerChromium::setOpaque):

- (WebCore::LayerChromium::setPosition):

- (WebCore::LayerChromium::setSublayerTransform):

- (WebCore::LayerChromium::setTransform):

- (WebCore::LayerChromium::setScrollPosition):

- (WebCore::LayerChromium::setScrollable):

- (WebCore::LayerChromium::setDoubleSided):

- * platform/graphics/chromium/LayerChromium.h:

- (WebCore::LayerChromium::setReplicaLayer):

- * platform/graphics/chromium/NonCompositedContentHost.cpp:

- (WebCore::NonCompositedContentHost::notifySyncRequired):

- * platform/graphics/chromium/NonCompositedContentHost.h:

-2011-12-06 Kenichi Ishibashi <bashi@chromium.org>

- [Chromium] unknown characters symbol on \n in complex script text (RTL and LTR)

- https://bugs.webkit.org/show_bug.cgi?id=73806

- Reviewed by Tony Chang.

- Sets fMergeNeutralItems to 1 instead of merging script items based on their tags.

- Tests: platform/chromium/fast/text/international/chromium-complex-text-non-printable-expected.html

- platform/chromium/fast/text/international/chromium-complex-text-non-printable.html

- * platform/graphics/chromium/UniscribeHelper.cpp:

- (WebCore::UniscribeHelper::fillRuns): Removed a block which merges script items.

-2011-12-06 Luke Macpherson <macpherson@chromium.org>

- Implement remaining border-image and webkit-maskbox-image properties in CSSStyleApplyProperty.

- https://bugs.webkit.org/show_bug.cgi?id=73391

- Reviewed by Hajime Morita.

- No new tests / refacoring only.

- * css/CSSStyleApplyProperty.cpp:

- (WebCore::ApplyPropertyBorderImageModifier::getValue):

- (WebCore::ApplyPropertyBorderImageModifier::setValue):

- (WebCore::ApplyPropertyBorderImageModifier::applyInheritValue):

- (WebCore::ApplyPropertyBorderImageModifier::applyInitialValue):

- (WebCore::ApplyPropertyBorderImageModifier::applyValue):

- (WebCore::ApplyPropertyBorderImageModifier::createHandler):

- (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):

- * css/CSSStyleSelector.cpp:

- (WebCore::CSSStyleSelector::applyProperty):

- * css/CSSStyleSelector.h:

-2011-12-06 Pavel Feldman <pfeldman@google.com>

- Web Inspector: introduce a memory agent stub.

- https://bugs.webkit.org/show_bug.cgi?id=73930

- Reviewed by Timothy Hatcher.

- We'd like to experiment with the memory stats and hence need a

- nice home for that. Adding this undocumented agent / domain for now.

- * CMakeLists.txt:

- * GNUmakefile.list.am:

- * Target.pri:

- * WebCore.gypi:

- * WebCore.vcproj/WebCore.vcproj:

- * WebCore.xcodeproj/project.pbxproj:

- * bindings/js/ScriptProfiler.h:

- (WebCore::ScriptProfiler::nodeCount):

- * bindings/v8/ScriptProfiler.cpp:

- (WebCore::ScriptProfiler::nodeCount):

- * bindings/v8/ScriptProfiler.h:

- * inspector/Inspector.json:

- * inspector/InspectorController.cpp:

- (WebCore::InspectorController::InspectorController):

- * inspector/InspectorMemoryAgent.cpp: Added.

- (WebCore::InspectorMemoryAgent::~InspectorMemoryAgent):

- (WebCore::InspectorMemoryAgent::getNodeCounter):

- (WebCore::InspectorMemoryAgent::InspectorMemoryAgent):

- * inspector/InspectorMemoryAgent.h: Added.

- (WebCore::InspectorMemoryAgent::create):

-2011-12-06 Julien Chaffraix <jchaffraix@webkit.org>

- Unreviewed build fix after 102183.

- * rendering/style/RenderStyle.h:

- (WebCore::InheritedFlags::initialGridTrackValue):

- Use DEFINE_STATIC_LOCAL to avoid having an exit-time destructor.

-2011-12-06 Julien Chaffraix <jchaffraix@webkit.org>

- Inline RenderObject::view()

- https://bugs.webkit.org/show_bug.cgi?id=73733

- Reviewed by Darin Adler.

- Micro-performance optimization, no change in behavior.

- RenderObject::view() is super hot and is taking ~4-5% of the time in some

- benchmarks as it is called several hundred thousands times. For some reason,

- the compiler did not inline it even though it is very simple in release builds.

- * WebCore.exp.in: Removed RenderObject::view() as it is inlined now.

- * rendering/RenderObject.cpp: Moved the implementation from here ...

- * rendering/RenderView.h:

- (WebCore::RenderObject::view): ... to here to avoid a cyclic

- dependency between RenderObject and RenderView. Also marked the

- function as ALWAYS_INLINE.

- * rendering/RenderObject.h:

- * rendering/svg/RenderSVGResourceContainer.cpp:

- Added #include "RenderView.h" as the code checks for view() during repaint.

-2011-12-06 Julien Chaffraix <jchaffraix@webkit.org>

- CSS Grid Layout: Add support for parsing multiple grid-columns or grid-rows

- https://bugs.webkit.org/show_bug.cgi?id=73272

- Reviewed by Tony Chang.

- Test: fast/css-grid-layout/grid-columns-rows-get-set-multiple.html

- Updated our supported syntax to match the following:

- <track-list> := [ <track-breadth> ]+ | 'none'

- <track-breadth> := <length> | <percentage> | 'auto'

- (the naming loosely matches the specification)

- * css/CSSComputedStyleDeclaration.cpp:

- (WebCore::valueForGridTrackBreadth): Added function to handle a breadth

- (extended with 'auto' that the spec puts in <track-minmax>).

- (WebCore::valueForGridTrackList): Create a space seperated list of

- track breadth or none.

- (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Updated

- to use the new functions.

- * css/CSSParser.cpp:

- (WebCore::CSSParser::parseGridTrackList): Extended the function to

- match the new syntax.

- * css/CSSStyleApplyProperty.cpp:

- (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): Removed our

- simple implementation. Replaced by the CSSStyleSelector functions.

- * css/CSSStyleSelector.cpp:

- (WebCore::createGridTrackBreadth):

- (WebCore::createGridTrackList):

- Added those 2 functions to convert the CSSPrimitiveValue to a Vector

- as expected by RenderStyle.

- (WebCore::CSSStyleSelector::applyProperty): Added our 2 properties

- now that it is not handled by CSSStyleApplyProperty.

- * rendering/style/RenderStyle.h:

- (WebCore::InheritedFlags::gridColumns):

- (WebCore::InheritedFlags::gridRows):

- (WebCore::InheritedFlags::setGridColumns):

- (WebCore::InheritedFlags::setGridRows):

- (WebCore::InheritedFlags::initialGridColumns):

- (WebCore::InheritedFlags::initialGridRows):

- Updated the previous methods to take a Vector of Length.

- (WebCore::InheritedFlags::initialGridTrackValue):

- Needed function to return a Vector with one 'none' Length (the initial

- value per the specification).

- * rendering/style/StyleGridData.h: Updated to use a Vector.

-2011-12-06 David Reveman <reveman@chromium.org>

- [Chromium] Implement tile-sized painting using SkPicture.

- https://bugs.webkit.org/show_bug.cgi?id=71869

- Reviewed by James Robinson.

- Add texture uploader that paints tile-sized chunks using SkPicture

- recording and playback. Expose setting which allows this texture

- updater to be enabled.

- No new tests. Covered by existing tests.

- * WebCore.gypi:

- * platform/graphics/chromium/BitmapSkPictureCanvasLayerTextureUpdater.cpp: Added.

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::Texture::Texture):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::Texture::prepareRect):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::Texture::updateRect):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::create):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::BitmapSkPictureCanvasLayerTextureUpdater):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::~BitmapSkPictureCanvasLayerTextureUpdater):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::createTexture):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::sampledTexelFormat):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::prepareToUpdate):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::paintContentsRect):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::updateTextureRect):

- * platform/graphics/chromium/BitmapSkPictureCanvasLayerTextureUpdater.h: Added.

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::Texture::textureUpdater):

- (WebCore::BitmapSkPictureCanvasLayerTextureUpdater::orientation):

- * platform/graphics/chromium/ContentLayerChromium.cpp:

- (WebCore::ContentLayerChromium::createTextureUpdater):

- * platform/graphics/chromium/cc/CCLayerTreeHost.h:

- (WebCore::CCSettings::CCSettings):

-2011-12-06 Adrienne Walker <enne@google.com>

- [chromium] Don't crash if tile upload happens without painting first

- https://bugs.webkit.org/show_bug.cgi?id=73939

- Reviewed by James Robinson.

- Remove at least one place (in ImageLayerChromium) where this could

- happen.

- Although this shouldn't happen, we should be robust to it in the

- chance that other code causes it to.

- * platform/graphics/chromium/ImageLayerChromium.cpp:

- (WebCore::ImageLayerChromium::paintContentsIfDirty):

- * platform/graphics/chromium/TiledLayerChromium.cpp:

- (WebCore::TiledLayerChromium::updateCompositorResources):

-2011-12-06 Ruben <chromium@hybridsource.org>

- Enable web audio by default on non-Mac POSIX platforms

- https://bugs.webkit.org/show_bug.cgi?id=73491

- Reviewed by Tony Chang.

- No new tests, just changing gyp includes.

- * WebCore.gyp/WebCore.gyp:

-2011-12-06 Benjamin Poulain <benjamin@webkit.org>

- Put length in its own variable in KURL copyASCII

- https://bugs.webkit.org/show_bug.cgi?id=73928

- Reviewed by Darin Adler.

- * platform/KURL.cpp:

- (WebCore::copyASCII):

-2011-12-06 Dana Jansens <danakj@chromium.org>

- [chromium] Set opaque flag for ImageLayerChromium

- https://bugs.webkit.org/show_bug.cgi?id=72964

- Reviewed by James Robinson.

- Unit test in tests/ImageLayerChromiumTest.cpp.

- * platform/graphics/chromium/GraphicsLayerChromium.cpp:

- (WebCore::GraphicsLayerChromium::setContentsToImage):

- * platform/graphics/chromium/GraphicsLayerChromium.h:

- (WebCore::GraphicsLayerChromium::contentsLayer):

- * platform/graphics/chromium/ImageLayerChromium.cpp:

- (WebCore::ImageLayerChromium::setContents):

-2011-12-06 Alexandre Elias <aelias@google.com>

- [chromium] Apply sent deltas on finishCommit

- https://bugs.webkit.org/show_bug.cgi?id=73884

- Reviewed by James Robinson.

- This moves scroll and pageScale "sent" deltas to be applied to

- the layer at the end of the commit, instead of the beginning.

- This has several advantages, especially for page scale:

- - When pageScale changes, no longer any need to change the scroll's

- coordinate space at beginning of commit, which is complex and prone to

- bugs (this fixes a problem where we were forgetting to modify the

- scrollPosition before).

- - No need for non-commit-related code to consider the "sent" values.

- m_pageScale is now always the content scale factor, and

- m_pageScaleDelta is the scale to be on the impl-side matrix.

- - This will make it easy to send arbitrary fake or future delta

- values for example while pinch zooming out.

- The scroll logic is similarly altered for consistency's sake. Note that

- I also moved the tree synchronize to the beginning of finishCommit

- in order to avoid having to change the pageScale coordinate space of

- sentScrollDelta in adjustScrollsForPageScaleChange().

- No new tests. (Refactoring of existing code.)

- * platform/graphics/chromium/LayerChromium.cpp:

- (WebCore::LayerChromium::pushPropertiesTo):

- * platform/graphics/chromium/cc/CCLayerTreeHost.cpp:

- (WebCore::CCLayerTreeHost::finishCommitOnImplThread):

- * platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:

- (WebCore::CCLayerTreeHostImpl::setPageScaleFactorAndLimits):

- (WebCore::CCLayerTreeHostImpl::applyPageScaleDeltaToScrollLayer):

- (WebCore::CCLayerTreeHostImpl::processScrollDeltas):

-2011-12-06 Gavin Barraclough <barraclough@apple.com>

- https://bugs.webkit.org/show_bug.cgi?id=68328

- The generator and intrinsic fields in HashTableValue/HashEntry and associated structures and methods are redundant

- Reviewed by Geoff Garen.

- Intrinsic is no longer in the DFG namespace, is always in the

- hash table. Removed ThunkGenerator.

- * bindings/scripts/CodeGeneratorJS.pm:

- (GenerateHashTable):

-2011-12-06 Dimitri Glazkov <dglazkov@chromium.org>

- Unreviewed, rolling out r102091.

- http://trac.webkit.org/changeset/102091

- https://bugs.webkit.org/show_bug.cgi?id=73711

- Caused Clang Linux compile failure.

- * platform/graphics/chromium/GraphicsLayerChromium.cpp:

- (WebCore::GraphicsLayerChromium::setAnchorPoint):

- (WebCore::GraphicsLayerChromium::setTransform):

- (WebCore::GraphicsLayerChromium::setChildrenTransform):

- (WebCore::GraphicsLayerChromium::setMasksToBounds):

- (WebCore::GraphicsLayerChromium::setBackgroundColor):

- (WebCore::GraphicsLayerChromium::clearBackgroundColor):

- (WebCore::GraphicsLayerChromium::setContentsOpaque):

- (WebCore::GraphicsLayerChromium::setBackfaceVisibility):

- (WebCore::GraphicsLayerChromium::setOpacity):

- * platform/graphics/chromium/LayerChromium.cpp:

- (WebCore::LayerChromium::setNeedsCommit):

- * platform/graphics/chromium/LayerChromium.h:

- (WebCore::LayerChromium::setAnchorPoint):

- (WebCore::LayerChromium::setAnchorPointZ):

- (WebCore::LayerChromium::setBackgroundColor):

- (WebCore::LayerChromium::setMasksToBounds):

- (WebCore::LayerChromium::setMaskLayer):

- (WebCore::LayerChromium::setOpacity):

- (WebCore::LayerChromium::setOpaque):

- (WebCore::LayerChromium::setPosition):

- (WebCore::LayerChromium::setSublayerTransform):

- (WebCore::LayerChromium::setTransform):

- (WebCore::LayerChromium::setScrollPosition):

- (WebCore::LayerChromium::setScrollable):

- (WebCore::LayerChromium::setDoubleSided):

- (WebCore::LayerChromium::setReplicaLayer):

- * platform/graphics/chromium/NonCompositedContentHost.cpp:

- (WebCore::NonCompositedContentHost::notifySyncRequired):

- * platform/graphics/chromium/NonCompositedContentHost.h:

-2011-12-06 Dana Jansens <danakj@chromium.org>

- [Chromium] Make root layer always opaque

- https://bugs.webkit.org/show_bug.cgi?id=70564

- Reviewed by James Robinson.

- * platform/graphics/chromium/NonCompositedContentHost.cpp:

- (WebCore::NonCompositedContentHost::NonCompositedContentHost):

- * platform/graphics/chromium/cc/CCTiledLayerImpl.cpp:

- (WebCore::CCTiledLayerImpl::draw):

-2011-12-06 Noel Gordon <noel.gordon@gmail.com>

- WebPImageDecoder computes image width and height multiple times

- https://bugs.webkit.org/show_bug.cgi?id=73796

- Reviewed by Adam Barth.

- Once sufficient image data arrives, we can compute the decoded image height

- and width from the WEBP image header data.

- From then on, the decoded image size is known so there's no need to re-read

- it from the WEBP image header again.

- No change in behavior, so no new tests.

- * platform/image-decoders/webp/WEBPImageDecoder.cpp:

- (WebCore::WEBPImageDecoder::decode):

-2011-12-06 Mike Reed <reed@google.com>

- optimize TransformationMatrix::scale by not calling through to generic multiply

- https://bugs.webkit.org/show_bug.cgi?id=73830

- Reviewed by Kenneth Russell.

- No new tests. Optimization only, existing tests exercise the code

- * platform/graphics/transforms/TransformationMatrix.cpp:

- (WebCore::TransformationMatrix::scaleNonUniform):

- (WebCore::TransformationMatrix::scale3d):

-2011-12-06 Eric Carlson <eric.carlson@apple.com>

- Revert WebCore track Settings changes made in r101977

- https://bugs.webkit.org/show_bug.cgi?id=73879

- Reviewed by Sam Weinig.

- No new tests yet, still nothing to test.

- * page/Settings.cpp: Move the preference setters back into the .h file.

- * page/Settings.h:

- (WebCore::Settings::setShouldDisplaySubtitles):

- (WebCore::Settings::setShouldDisplayCaptions):

- (WebCore::Settings::setShouldDisplayTextDescriptions):

-2011-12-06 Andreas Kling <kling@webkit.org>

- MediaList: Remove constructor that takes a CSSImportRule*.

- <http://webkit.org/b/73833>

- Reviewed by Antti Koivisto.

- * css/MediaList.h:

- * css/MediaList.cpp:

- Remove MediaList(CSSImportRule*, ...) constructor.

- * css/CSSImportRule.cpp:

- (WebCore::CSSImportRule::CSSImportRule):

- Have CSSImportRule construct its MediaList by passing the parent

- style sheet, which is exactly what the old constructor accomplished.

- Also assert that we're always created with a non-null parent sheet.

-2011-12-06 Jarred Nicholls <jarred@sencha.com>

- getComputedStyle returns wrong value for margin-*

- https://bugs.webkit.org/show_bug.cgi?id=73334

- margin-* getComputedStyle values should return the "used" absolute value when there is a renderer

- and the specified value is relative (percentage, auto, etc.).

- When there is no renderer, the specified value should be returned.

- See http://dev.w3.org/csswg/cssom/#resolved-values.

- Reviewed by Darin Adler.

- Test: fast/css/getComputedStyle/getComputedStyle-resolved-values.html

- * css/CSSComputedStyleDeclaration.cpp:

- (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

-2011-12-06 Alexey Proskuryakov <ap@apple.com>

- REGRESSION (WebKit2): Kill ring is not cleared when selection changes

- https://bugs.webkit.org/show_bug.cgi?id=73888

- <rdar://problem/10532310>

- Reviewed by Mark Rowe.

- Test: editing/pasteboard/emacs-ctrl-k-with-move.html

- * editing/Editor.cpp: (WebCore::Editor::respondToChangedSelection): Moved the code to clear

- kill ring from Mac WebKit, as it's needed in all Mac ports at least.

-2011-12-06 Darin Adler <darin@apple.com>

- Use HashMap<OwnPtr> in CollectionCache

- https://bugs.webkit.org/show_bug.cgi?id=73784

- Reviewed by Andreas Kling.

- * html/CollectionCache.cpp:

- (WebCore::CollectionCache::copyCacheMap): Use adoptPtr.

- (WebCore::CollectionCache::reset): Removed now-unneeded calls to deleteAllValues.

- (WebCore::append): Added. Helper function for appending elements to the maps from

- the collection cache.

- * html/CollectionCache.h: Changed mapped type in NodeCacheMap to OwnPtr.

- Added append function.

- * html/HTMLCollection.cpp:

- (WebCore::nameShouldBeVisibleInDocumentAll): Added, to factor out common code in

- two functions below.

- (WebCore::HTMLCollection::checkForNameMatch): Changed to call nameShouldBeVisibleInDocumentAll.

- (WebCore::HTMLCollection::updateNameCache): Ditto. Also updated cache code to use the append

- function, so it will work with OwnPtr. Also eliminated an unneeded get call before

- each hash table add; we do both at once in the new append function.

- * html/HTMLFormCollection.cpp:

- (WebCore::HTMLFormCollection::updateNameCache): More of the same.

-2011-12-06 Yury Semikhatsky <yurys@chromium.org>

- [Chromium] Web Inspector: getFunctionLocation should return scriptId as String not as int

- https://bugs.webkit.org/show_bug.cgi?id=73892

- Reviewed by Pavel Feldman.

- * bindings/v8/custom/V8InjectedScriptHostCustom.cpp:

- (WebCore::V8InjectedScriptHost::functionLocationCallback): scriptId should be a string not a number

- * inspector/Inspector.json: removed unused parameter

-2011-11-21 Balazs Kelemen <kbalazs@webkit.org>

- Enable ParallelJobs by default

- https://bugs.webkit.org/show_bug.cgi?id=70032

- Reviewed by Zoltan Herczeg.

- Covered by existing tests.

- According to measurements on Mac and Linux it is a

- considerable speedup for SVG on multicore.

- Remove the ENABLE(PARALLEL_JOBS) guard. Fix the Windows build

- by qualifying ParallelJobs with the WTF namespace (otherwise

- MSVC believes it belongs to WebCore which is likely a compiler bug).

- * platform/graphics/filters/FEConvolveMatrix.cpp:

- (WebCore::FEConvolveMatrix::setInteriorPixelsWorker):

- (WebCore::FEConvolveMatrix::platformApplySoftware):

- * platform/graphics/filters/FEConvolveMatrix.h:

- * platform/graphics/filters/FEGaussianBlur.cpp:

- (WebCore::FEGaussianBlur::platformApplyWorker):

- (WebCore::FEGaussianBlur::platformApply):

- * platform/graphics/filters/FEGaussianBlur.h:

- * platform/graphics/filters/FELighting.cpp:

- (WebCore::FELighting::platformApplyGenericWorker):

- (WebCore::FELighting::platformApplyGeneric):

- * platform/graphics/filters/FELighting.h:

- * platform/graphics/filters/FEMorphology.cpp:

- (WebCore::FEMorphology::platformApplyWorker):

- (WebCore::FEMorphology::platformApply):

- * platform/graphics/filters/FEMorphology.h:

- * platform/graphics/filters/FETurbulence.cpp:

- (WebCore::FETurbulence::fillRegionWorker):

- (WebCore::FETurbulence::platformApplySoftware):

- * platform/graphics/filters/FETurbulence.h:

- * platform/graphics/filters/arm/FELightingNEON.cpp:

- (WebCore::FELighting::platformApplyNeonWorker):

- * platform/graphics/filters/arm/FELightingNEON.h:

- (WebCore::FELighting::platformApplyNeon):

-2011-12-06 Andreas Kling <kling@webkit.org>

- Unreviewed assertion fix for r102123.

- * platform/KURL.cpp:

- (WebCore::checkEncodedString):

-2011-12-06 Benjamin Poulain <benjamin@webkit.org>

- Simplify KURL's checkEncodedString()

- https://bugs.webkit.org/show_bug.cgi?id=73890

- Reviewed by Andreas Kling.

- The method was reimplementing String::containsOnlyASCII().

- Use the method from String and we can remove the #if NDEBUG.

- * platform/KURL.cpp:

- (WebCore::checkEncodedString):

-2011-12-06 Sheriff Bot <webkit.review.bot@gmail.com>

- Unreviewed, rolling out r102111.

- http://trac.webkit.org/changeset/102111

- https://bugs.webkit.org/show_bug.cgi?id=73902

- Breaks compilation (Requested by vsevik on #webkit).

- * editing/Editor.cpp:

- (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges):

- * editing/Editor.h:

-2011-12-06 Hajime Morrita <morrita@chromium.org>

- [Refactoring] Accessing Node::m_document should be minimized.

- https://bugs.webkit.org/show_bug.cgi?id=73800

- Reviewed by Kent Tamura.

- No new tests. No behavioral change.

- Replaced m_document reference with the document() accessor

- or temporaril variables. This is a preparation for using

- m_document space to point a shadow root pointer.

- * dom/Document.h:

- (WebCore::Node::Node):

- * dom/Node.cpp:

- (WebCore::Node::~Node):

-2011-12-06 Shinya Kawanaka <shinyak@google.com>

- https://bugs.webkit.org/show_bug.cgi?id=73889

- TextCheckingParagraph::offsetTo should not have a side effect.

- Reviewed by Hajime Morita.

- Since TextCheckingParagraph::offsetTo had a side effect, its cache often became inconsistent.

- This is likely to cause a bug when changing SpellChecker and Editor.

- No new tests. Covered by existing tests.

- * editing/TextCheckingHelper.cpp:

- (WebCore::TextCheckingParagraph::offsetTo):

-2011-12-06 Eric Penner <epenner@google.com>

- [chromium] Set texture limits as multiples of viewport size instead of hardcoded values

- https://bugs.webkit.org/show_bug.cgi?id=72202

- Reviewed by James Robinson.

- * platform/graphics/chromium/LayerRendererChromium.cpp:

- (WebCore::LayerRendererChromium::drawLayers): added viewport param

- (WebCore::LayerRendererChromium::initializeSharedObjects): ditto

- * platform/graphics/chromium/TextureManager.cpp:

- (WebCore::TextureManager::highLimitBytes): calculated based on viewport

- (WebCore::TextureManager::reclaimLimitBytes): ditto

- (WebCore::TextureManager::lowLimitBytes): ditto

- (WebCore::TextureManager::TextureManager): added viewport param

- (WebCore::TextureManager::setMaxMemoryLimitBytes): changed name

- (WebCore::TextureManager::setPreferredMemoryLimitBytes): added function

- (WebCore::TextureManager::requestTexture): added viewport param

- * platform/graphics/chromium/TextureManager.h: ditto

- (WebCore::TextureManager::create): ditto

- * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: ditto

- (WebCore::CCLayerTreeHost::initialize): ditto

- (WebCore::CCLayerTreeHost::beginCommitOnImplThread): ditto

- (WebCore::CCLayerTreeHost::setViewport): ditto

- (WebCore::CCLayerTreeHost::setVisible): ditto

- (WebCore::CCLayerTreeHost::didBecomeInvisibleOnImplThread): ditto

-2011-12-06 Huang Dongsung <luxtella@company100.net>

- [TexMap][QT] Draw the borders of media and webgl elements in TexMap.

- https://bugs.webkit.org/show_bug.cgi?id=73817

- GraphicsContext3D only draws the content of the WebGL canvas, not the additional

- CSS such as the borders. TextureMapper should render the content of a

- media/webgl layer before drawing the actual canvas.

- This makes LayoutTests/compositing/webgl/webgl-reflection.html work.

- Reviewed by Noam Rosenthal.

- * platform/graphics/texmap/GraphicsLayerTextureMapper.cpp:

- (WebCore::GraphicsLayerTextureMapper::setContentsNeedsDisplay):

- * platform/graphics/texmap/GraphicsLayerTextureMapper.h:

- * platform/graphics/texmap/TextureMapperNode.cpp:

- (WebCore::TextureMapperNode::renderContent):

- (WebCore::TextureMapperNode::paintSelf):

-2011-12-06 Sheriff Bot <webkit.review.bot@gmail.com>

- Unreviewed, rolling out r102043.

- http://trac.webkit.org/changeset/102043

- https://bugs.webkit.org/show_bug.cgi?id=73898

- Breaks chromium mac-cg compilation. (Requested by vsevik on

- #webkit).

- * platform/graphics/chromium/GraphicsLayerChromium.cpp:

- (WebCore::GraphicsLayerChromium::setContentsToImage):

- * platform/graphics/chromium/GraphicsLayerChromium.h:

- (WebCore::GraphicsLayerChromium::contentsLayer):

- * platform/graphics/chromium/ImageLayerChromium.cpp:

- (WebCore::ImageLayerChromium::setContents):

-2011-12-06 Shinya Kawanaka <shinyak@google.com>

- Refactoring: Editor::markAllMisspellingsAndBadGrammarInRanges should be refactored.

- https://bugs.webkit.org/show_bug.cgi?id=73628

- Reviewed by Hajime Morita.

- Extracted a code for adding markers and replacing misspelled words from WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges.

- No new tests. covered by existing tests.

- * editing/Editor.cpp:

- (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges):

- Extracted a code for adding markers and replacing missplled words, and moved to markAndReplaceFor.

- (WebCore::Editor::markAndReplaceFor):

- * editing/Editor.h:

-2011-12-05 Alexander Pavlov <apavlov@chromium.org>

- Web Inspector: [Audits] Implement "Stop" button and progress bar instead of spinner.

- https://bugs.webkit.org/show_bug.cgi?id=73626

- Reviewed by Yury Semikhatsky.

- * English.lproj/localizedStrings.js:

- * inspector/front-end/AuditLauncherView.js:

- (WebInspector.AuditLauncherView):

- (WebInspector.AuditLauncherView.prototype._setAuditRunning):

- (WebInspector.AuditLauncherView.prototype._launchButtonClicked):

- (WebInspector.AuditLauncherView.prototype._createLauncherUI):

- (WebInspector.AuditLauncherView.prototype._updateResourceProgress):

- (WebInspector.AuditLauncherView.prototype._updateButton):

- * inspector/front-end/AuditRules.js:

- (WebInspector.AuditRules.GzipRule.prototype.doRun):

- (WebInspector.AuditRules.CombineExternalResourcesRule.prototype.doRun):

- (WebInspector.AuditRules.MinimizeDnsLookupsRule.prototype.doRun):

- (WebInspector.AuditRules.UnusedCssRule.prototype.doRun.evalCallback.selectorsCallback):

- (WebInspector.AuditRules.UnusedCssRule.prototype.doRun.evalCallback.documentLoaded):

- (WebInspector.AuditRules.UnusedCssRule.prototype.doRun.evalCallback):

- (WebInspector.AuditRules.UnusedCssRule.prototype.doRun.styleSheetCallback):

- (WebInspector.AuditRules.UnusedCssRule.prototype.doRun.allStylesCallback):

- (WebInspector.AuditRules.UnusedCssRule.prototype.doRun):

- (WebInspector.AuditRules.CacheControlRule.prototype.doRun):

- (WebInspector.AuditRules.ImageDimensionsRule.prototype.doRun):

- (WebInspector.AuditRules.CssInHeadRule.prototype.doRun):

- (WebInspector.AuditRules.CssInHeadRule.prototype.doRun.externalStylesheetsReceived):

- (WebInspector.AuditRules.CssInHeadRule.prototype.doRun.inlineStylesReceived):

- (WebInspector.AuditRules.CssInHeadRule.prototype.doRun.onDocumentAvailable):

- (WebInspector.AuditRules.StylesScriptsOrderRule.prototype.doRun):

- (WebInspector.AuditRules.StylesScriptsOrderRule.prototype.doRun.cssBeforeInlineReceived):

- (WebInspector.AuditRules.StylesScriptsOrderRule.prototype.doRun.lateStylesReceived):

- (WebInspector.AuditRules.StylesScriptsOrderRule.prototype.doRun.onDocumentAvailable):

- (WebInspector.AuditRules.CookieRuleBase.prototype.doRun.resultCallback):

- (WebInspector.AuditRules.CookieRuleBase.prototype.doRun):

- * inspector/front-end/AuditsPanel.js:

- (WebInspector.AuditsPanel):

- (WebInspector.AuditsPanel.prototype._executeAudit.ruleResultReadyCallback):

- (WebInspector.AuditsPanel.prototype._executeAudit):

- (WebInspector.AuditsPanel.prototype._auditFinishedCallback):

- (WebInspector.AuditsPanel.prototype.terminateAudit):

- (WebInspector.AuditCategory.prototype.run):

- (WebInspector.AuditRule.prototype.run):

- (WebInspector.AuditRule.prototype.doRun):

- (WebInspector.AuditProgressMonitor):

- (WebInspector.AuditProgressMonitor.prototype.setTotalWork):

- (WebInspector.AuditProgressMonitor.prototype.worked):

- (WebInspector.AuditProgressMonitor.prototype.get indeterminate):

- (WebInspector.AuditProgressMonitor.prototype.done):

- (WebInspector.AuditProgressMonitor.prototype.get canceled):

- (WebInspector.AuditProgressMonitor.prototype.set canceled):

-2011-12-06 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

- [Qt] [WK2] MiniBrowser assert on startup in debug build after r101713

- https://bugs.webkit.org/show_bug.cgi?id=73874

- This change partially reverts r101713 restoring original behaviour for

- KUrl creation from empty string and fixes asserts in debug build.

- Reviewed by Alexey Proskuryakov.

- No new tests. Tests from r101713 pass.

- * platform/KURL.cpp:

- (WebCore::KURL::init):

- (WebCore::KURL::parse):

- * platform/KURL.h:

-2011-12-06 Andreas Kling <kling@webkit.org>

- Use HashMap<OwnPtr> for EventListenerMap's internal map.

- <http://webkit.org/b/73761>

- Reviewed by Benjamin Poulain.

- Changed the value type of EventListenerMap::m_hashMap to OwnPtr<EventListenerVector>.

- This means we no longer need to manually delete the vectors when taking them out of

- the map, which makes the code a little prettier.

- A few tweaks were necessary; release() instead of leakPtr() when switching modes

- and adoptPtr()/get() sprinkled as needed.

- * dom/EventListenerMap.h:

- * dom/EventListenerMap.cpp:

- (WebCore::EventListenerMap::clear):

- (WebCore::EventListenerMap::add):

- (WebCore::EventListenerMap::remove):

- (WebCore::EventListenerMap::find):

- (WebCore::EventListenerMap::removeFirstEventListenerCreatedFromMarkup):

- (WebCore::EventListenerMap::copyEventListenersNotCreatedFromMarkupToTarget):

-2011-12-05 Rafael Weinstein <rafaelw@chromium.org>

- [MutationObservers] Support 'attributes' mutation records for element.removeAttribute

- https://bugs.webkit.org/show_bug.cgi?id=73880

- Reviewed by Ojan Vafai.

- * dom/Element.cpp:

- (WebCore::enqueueAttributesMutationRecord):

- (WebCore::Element::removeAttribute):

-2011-12-05 Dana Jansens <danakj@chromium.org>

- Set opaque flag for WebGLLayerChromium

- https://bugs.webkit.org/show_bug.cgi?id=73876

- Reviewed by James Robinson.

- New unit test in tests/WebGLLayerChromiumTest.cpp

- * platform/graphics/chromium/DrawingBufferChromium.cpp:

- (WebCore::DrawingBuffer::platformLayer):

-2011-12-05 Benjamin Poulain <bpoulain@apple.com>

- Upstream the Cursor implementation of iOS

- https://bugs.webkit.org/show_bug.cgi?id=73724

- Reviewed by David Kilzer.

- iOS does not need to support the Cursor of WebKit. For compatibility, Cursor

- is implemented as an empty class on the platform.

- * Configurations/WebCore.xcconfig:

- * WebCore.xcodeproj/project.pbxproj:

- * platform/Cursor.h:

- (WebCore::Cursor::Cursor):

- * platform/ios/CursorIOS.cpp: Added.

- (WebCore::cursor):

- (WebCore::pointerCursor):

- (WebCore::crossCursor):

- (WebCore::handCursor):

- (WebCore::moveCursor):

- (WebCore::iBeamCursor):

- (WebCore::waitCursor):

- (WebCore::helpCursor):

- (WebCore::eastResizeCursor):

- (WebCore::northResizeCursor):

- (WebCore::northEastResizeCursor):

- (WebCore::northWestResizeCursor):

- (WebCore::southResizeCursor):

- (WebCore::southEastResizeCursor):

- (WebCore::southWestResizeCursor):

- (WebCore::westResizeCursor):

- (WebCore::northSouthResizeCursor):

- (WebCore::eastWestResizeCursor):

- (WebCore::northEastSouthWestResizeCursor):

- (WebCore::northWestSouthEastResizeCursor):

- (WebCore::columnResizeCursor):

- (WebCore::rowResizeCursor):

- (WebCore::middlePanningCursor):

- (WebCore::eastPanningCursor):

- (WebCore::northPanningCursor):

- (WebCore::northEastPanningCursor):

- (WebCore::northWestPanningCursor):

- (WebCore::southPanningCursor):

- (WebCore::southEastPanningCursor):

- (WebCore::southWestPanningCursor):

- (WebCore::westPanningCursor):

- (WebCore::verticalTextCursor):

- (WebCore::cellCursor):

- (WebCore::contextMenuCursor):

- (WebCore::noDropCursor):

- (WebCore::notAllowedCursor):

- (WebCore::progressCursor):

- (WebCore::aliasCursor):

- (WebCore::zoomInCursor):

- (WebCore::zoomOutCursor):

- (WebCore::copyCursor):

- (WebCore::noneCursor):

- (WebCore::grabCursor):

- (WebCore::grabbingCursor):

- (WebCore::determineHotSpot):

-2011-12-05 Noel Gordon <noel.gordon@gmail.com>

- [GTK] GIF image test crashes on 32- and 64-bit Release

- https://bugs.webkit.org/show_bug.cgi?id=73812

- Reviewed by Adam Barth.

- Call resize() on the image pixel backing store after allocation to see if

- that stops the GIF image decoder animation tests crashes on GTK.

- No new tests, covered by exiting tests ...

- fast/backgrounds/animated-gif-as-background.html

- fast/images/dont-crash-with-null-gif-frames.html

- fast/images/gif-loop-count.html

- * platform/image-decoders/ImageDecoder.cpp:

- (WebCore::ImageFrame::setSize):

-2011-12-05 Erik Arvidsson <arv@chromium.org>

- Could save a lot of memory in CharacterData by not always storing a String

- https://bugs.webkit.org/show_bug.cgi?id=72404

- Reviewed by Ojan Vafai.

- When a Text node is created by the parser we check if the string is all whitespace

- and if so we put it in the AtomicString table so that all future identical whitespace

- strings can share the StringImpl.

- No new tests. Covered by existing tests.

- * html/parser/HTMLConstructionSite.cpp:

- (WebCore::HTMLNames::isAllWhitespace):

- (WebCore::HTMLConstructionSite::insertTextNode):

- If we do not know whether the string is all whitespace this now checks the string

- If the string is all whitespace we create an AtomicString for it.

- * html/parser/HTMLConstructionSite.h:

- * html/parser/HTMLTreeBuilder.cpp:

- (WebCore::HTMLTreeBuilder::ExternalCharacterTokenBuffer::skipLeadingNonWhitespace): We never cared about the return value here.

- (WebCore::HTMLTreeBuilder::processCharacterBuffer): Pass WhitespaceMode in the case we know whether the string is all whitespace or not.

- (WebCore::HTMLTreeBuilder::defaultForInTableText): Ditto.

-2011-12-05 Benjamin Poulain <benjamin@webkit.org>

- Update KURL's copy copyASCII to avoid String::characters()

- https://bugs.webkit.org/show_bug.cgi?id=73794

- Reviewed by Andreas Kling.

- When the String is already on 8 bits, we can simply copy the

- data. In the 16 bits case, everything remains the same.

- * platform/KURL.cpp:

- (WebCore::copyASCII):

- (WebCore::appendASCII):

- (WebCore::KURL::init):

- (WebCore::KURL::parse):

- (WebCore::KURL::copyToBuffer):

-2011-12-05 Yong Li <yoli@rim.com>

- https://bugs.webkit.org/show_bug.cgi?id=73683

- Implement KeyframeValueList::operator=() and KeyframeValueList::swap().

- Reviewed by Darin Adler.

- No new tests as no functional change.

- * platform/graphics/GraphicsLayer.h:

- (WebCore::KeyframeValueList::operator=): Added

- (WebCore::KeyframeValueList::swap): Added

-2011-12-05 Adrienne Walker <enne@google.com>

- [chromium] setNeedsCommit on non-composited host layers should trigger commit

- https://bugs.webkit.org/show_bug.cgi?id=73711

- Reviewed by James Robinson.

- Pipe non-composited content host syncs to setNeedsCommit.

- Since now the NonCompositedContentHost generates setNeedsCommit, don't

- call it unnecessarily, e.g. calling setBackgroundColor to the same

- color each frame should not retrigger more commits.

- * platform/graphics/chromium/GraphicsLayerChromium.cpp:

- (WebCore::GraphicsLayerChromium::setAnchorPoint):

- (WebCore::GraphicsLayerChromium::setTransform):

- (WebCore::GraphicsLayerChromium::setChildrenTransform):

- (WebCore::GraphicsLayerChromium::setMasksToBounds):

- (WebCore::GraphicsLayerChromium::setBackgroundColor):

- (WebCore::GraphicsLayerChromium::clearBackgroundColor):

- (WebCore::GraphicsLayerChromium::setContentsOpaque):

- (WebCore::GraphicsLayerChromium::setBackfaceVisibility):

- (WebCore::GraphicsLayerChromium::setOpacity):

- * platform/graphics/chromium/LayerChromium.cpp:

- (WebCore::LayerChromium::setNeedsCommit):

- (WebCore::LayerChromium::setAnchorPoint):

- (WebCore::LayerChromium::setAnchorPointZ):

- (WebCore::LayerChromium::setBackgroundColor):

- (WebCore::LayerChromium::setMasksToBounds):

- (WebCore::LayerChromium::setMaskLayer):

- (WebCore::LayerChromium::setOpacity):

- (WebCore::LayerChromium::setOpaque):

- (WebCore::LayerChromium::setPosition):

- (WebCore::LayerChromium::setSublayerTransform):

- (WebCore::LayerChromium::setTransform):

- (WebCore::LayerChromium::setScrollPosition):

- (WebCore::LayerChromium::setScrollable):

- (WebCore::LayerChromium::setDoubleSided):

- * platform/graphics/chromium/LayerChromium.h:

- (WebCore::LayerChromium::setReplicaLayer):

- * platform/graphics/chromium/NonCompositedContentHost.cpp:

- (WebCore::NonCompositedContentHost::notifySyncRequired):

- * platform/graphics/chromium/NonCompositedContentHost.h:

-2011-12-05 Tony Chang <tony@chromium.org>

- small refactor of RenderFlexibleBox

- https://bugs.webkit.org/show_bug.cgi?id=73854

- Reviewed by Darin Adler.

- No new tests, just a refactor.

- * rendering/RenderFlexibleBox.cpp:

- (WebCore::RenderFlexibleBox::isLeftToRightFlow): Inline isReverseFlow since it's only used in one place.

- (WebCore::RenderFlexibleBox::layoutAndPlaceChildren): Rename startEdge

- to mainAxisOffset. Rename logicalTop to crossAxisOffset. Get rid of

- logicalLeft local variable since it's confusing. Move shouldFlipMainAxis

- out of the for loop to avoid computing it each iteration.

- * rendering/RenderFlexibleBox.h:

-2011-12-05 Florin Malita <fmalita@google.com>

- Heap-buffer-overflow in WebCore::HTMLTreeBuilder::processEndTag

- https://bugs.webkit.org/show_bug.cgi?id=73765

- Reviewed by Adam Barth.

- Test: fast/parser/foreign-content-crash.html

- Use m_tree.currentNode() instead of m_tree.currentElement() as the top node is not always an Element.

- * html/parser/HTMLTreeBuilder.cpp:

- (WebCore::HTMLTreeBuilder::processEndTag):

-2011-12-05 Stephen White <senorblanco@chromium.org>

- Allow the ImageBuffers used by SVG filters to be accelerated

- https://bugs.webkit.org/show_bug.cgi?id=73842

- Reviewed by Kenneth Russell.

- Regressions covered by existing SVG tests; new functionality to be

- tested by the API exposed on Internals.

- * page/Settings.cpp:

- (WebCore::Settings::Settings):

- * page/Settings.h:

- (WebCore::Settings::setAcceleratedFiltersEnabled):

- (WebCore::Settings::acceleratedFiltersEnabled):

- * platform/graphics/filters/FETile.cpp:

- (WebCore::FETile::platformApplySoftware):

- * platform/graphics/filters/Filter.h:

- (WebCore::Filter::Filter):

- (WebCore::Filter::renderingMode):

- (WebCore::Filter::setRenderingMode):

- * platform/graphics/filters/FilterEffect.cpp:

- (WebCore::FilterEffect::asImageBuffer):

- (WebCore::FilterEffect::createImageBufferResult):

- * platform/graphics/skia/ImageBufferSkia.cpp:

- (WebCore::ImageBuffer::platformTransformColorSpace):

- * rendering/svg/RenderSVGResourceClipper.cpp:

- (WebCore::RenderSVGResourceClipper::applyClippingToContext):

- * rendering/svg/RenderSVGResourceFilter.cpp:

- (WebCore::RenderSVGResourceFilter::applyResource):

- * rendering/svg/RenderSVGResourceMasker.cpp:

- (WebCore::RenderSVGResourceMasker::applyResource):

- * rendering/svg/RenderSVGResourcePattern.cpp:

- (WebCore::RenderSVGResourcePattern::createTileImage):

- * rendering/svg/SVGImageBufferTools.cpp:

- (WebCore::SVGImageBufferTools::createImageBuffer):

- * rendering/svg/SVGImageBufferTools.h:

- * testing/Internals.cpp:

- (WebCore::Internals::setAcceleratedFiltersEnabled):

- * testing/Internals.h:

- * testing/Internals.idl:

-2011-12-05 Benjamin Poulain <bpoulain@apple.com>

- Upstream htmlSelectMultipleItems needed for <select multiple> by iOS

- https://bugs.webkit.org/show_bug.cgi?id=73734

- Reviewed by David Kilzer.

- * platform/DefaultLocalizationStrategy.cpp:

- (WebCore::DefaultLocalizationStrategy::htmlSelectMultipleItems):

- * platform/DefaultLocalizationStrategy.h:

-2011-12-05 Darin Adler <darin@apple.com>

- Use HashMap<OwnPtr> in CrossOriginPreflightResultCache

- https://bugs.webkit.org/show_bug.cgi?id=73785

- Reviewed by Andreas Kling.

- * loader/CrossOriginPreflightResultCache.cpp:

- (WebCore::CrossOriginPreflightResultCache::appendEntry): Changed code to use set

- instead of add, since it wants to replace existing entries. Also removed leakPtr

- and removed the FIXME that documented the memory leak now fixed here.

- (WebCore::CrossOriginPreflightResultCache::canSkipPreflight): Removed unneeded

- std:: prefix here and also unneeded explicit delete call.

- (WebCore::CrossOriginPreflightResultCache::empty): Removed unneeded deleteAllValues

- call here.

- * loader/CrossOriginPreflightResultCache.h: Make mapped value of the

- CrossOriginPreflightResultHashMap be OwnPtr instead of raw pointer.

-2011-12-05 Darin Adler <darin@apple.com>

- Some small improvements to ContainerNode.h

- https://bugs.webkit.org/show_bug.cgi?id=73786

- Reviewed by Alexey Proskuryakov.

- * dom/ContainerNode.cpp:

- (WebCore::ContainerNode::suspendPostAttachCallbacks): Added a FIXME comment about the

- peculiar behavior of this function. Somehow the post-attach suspend state is both

- global and specific to a certain Page object. That can't be right. If it was truly

- global then this would be a static member function. If it was truly per-page, then

- the related functions could not be static.

- * dom/ContainerNode.h: Removed some unneeded argument names. Moved the hasChildNodes

- function up with the other basic getters. Put the other getters, childNodeCount and

- childNode, right after the basic getters. Used ASSERT_NO_EXCEPTION in all the basic

- mutation functions so they can be used in a cleaner fashion in C++ code where we have

- some reason to know an exception won't occur. Grouped all the overrides of functions

- from Node into a single paragraph and used the OVERRIDE macro on all of them. Made the

- queuePostAttachCallback and postAttachCallbacksAreSuspended functions protected.

- * dom/Element.h: Moved the include of ExceptionCodePlaceholder.h into ContainerNode.h.

- * dom/Node.cpp:

- (WebCore::Node::lazyAttach): Use hasChildNodes instead of firstChild for clarity.

- (WebCore::Node::isDescendantOf): Ditto.

-2011-12-05 Benjamin Poulain <benjamin@webkit.org>

- Get rid of KURL::KURL(ParsedURLStringTag, const char*);

- https://bugs.webkit.org/show_bug.cgi?id=73792

- Reviewed by Andreas Kling.

- In all cases where the constructor is used, the constructor

- taking a String is as effective because the string is valid and converted

- to String for m_string.

- This patch remove the constructor KURL::KURL(ParsedURLStringTag, const char*)

- and change the call sites that were using that constructor to build

- empty URLs.

- * dom/Document.cpp:

- (WebCore::Document::initSecurityContext):

- * loader/FrameLoader.cpp:

- (WebCore::FrameLoader::init):

- * loader/archive/cf/LegacyWebArchive.cpp:

- (WebCore::LegacyWebArchive::create):

- * page/DOMWindow.cpp:

- (WebCore::DOMWindow::createWindow):

- * platform/KURL.cpp:

- * platform/KURL.h:

- * platform/KURLGoogle.cpp:

-2011-12-05 Benjamin Poulain <benjamin@webkit.org>

- Remove methods declared but never implemented with GOOGLEURL

- https://bugs.webkit.org/show_bug.cgi?id=73795

- Reviewed by Adam Barth.

- * platform/KURL.h:

-2011-12-05 Darin Adler <darin@apple.com>

- Change CSSFontSelector to use HashMap<OwnPtr>

- https://bugs.webkit.org/show_bug.cgi?id=73781

- Reviewed by Dan Bernstein.

- * css/CSSFontSelector.cpp:

- (WebCore::CSSFontSelector::~CSSFontSelector): Removed calls to deleteAllValues.

- (WebCore::CSSFontSelector::addFontFaceRule): Updated to use OwnPtr instead of raw

- pointer for the entry in m_fontFaces and m_locallyInstalledFontFaces.

- (WebCore::CSSFontSelector::getFontData): Updated to use OwnPtr instead of raw

- pointer for the entry in m_fonts. Also removed an unneeded std:: prefix.

- * css/CSSFontSelector.h: Made m_fontFaces, m_locallyInstalledFontFaces, and m_fonts

- be HashMap<OwnPtr>.

-2011-12-05 Adam Klein <adamk@chromium.org>

- V8RecursionScope should call didLeaveScriptContext when recursionLevel reaches zero

- https://bugs.webkit.org/show_bug.cgi?id=73867

- Reviewed by Adam Barth.

- Moved V8RecursionScope into its own file, and moved

- V8Proxy::didLeaveScriptContext into that file, along with a static

- recursionLevel accessor, hiding the V8BindingPerIsolateData methods

- from V8Proxy.

- This will make it easy and less error-prone to use V8RecursionScope

- properly. I plan to make use of it in V8LazyEventListener to fix

- https://bugs.webkit.org/show_bug.cgi?id=73492.

- No new tests, refactoring only.

- * Target.pri:

- * UseV8.cmake:

- * WebCore.gypi:

- * bindings/v8/V8Binding.h:

- (WebCore::V8BindingPerIsolateData::incrementRecursionLevel): return the new recursion level.

- (WebCore::V8BindingPerIsolateData::decrementRecursionLevel): return the new recursion level.

- * bindings/v8/V8Proxy.cpp: remove didLeaveScriptContext.

- (WebCore::V8Proxy::runScript): remove explicit call to didLeaveScriptContext.

- (WebCore::V8Proxy::instrumentedCallFunction): remove explicit call to didLeaveScriptContext.

- * bindings/v8/V8Proxy.h: remove didLeaveScriptContext.

- * bindings/v8/V8RecursionScope.cpp: Added.

- (WebCore::V8RecursionScope::didLeaveScriptContext): copied from V8Proxy.cpp.

- * bindings/v8/V8RecursionScope.h: Added.

- (WebCore::V8RecursionScope::V8RecursionScope):

- (WebCore::V8RecursionScope::~V8RecursionScope):

- (WebCore::V8RecursionScope::recursionLevel):

-2011-12-05 Benjamin Poulain <bpoulain@apple.com>

- Build fix for SecurityOrigin.cpp when neither BLOB nor FILE_SYSTEM are defined

- Reviewed by David Kilzer.

- When neither BLOB nor FILE_SYSTEM are defined, the parameter is unused.

- * page/SecurityOrigin.cpp:

- (WebCore::shouldUseInnerURL):

-2011-12-05 Darin Adler <darin@apple.com>

- Change RuleSet to use HashMap<OwnPtr>

- https://bugs.webkit.org/show_bug.cgi?id=73783

- Reviewed by Andreas Kling.

- * css/CSSStyleSelector.cpp: Make RuleSet::AtomRuleMap use OwnPtr for the mapped values.

- (WebCore::RuleSet::addToRuleSet): Use add instead of get/set to set up a new entry in the

- map or find the old entry in the map.

-2011-12-05 Mario Sanchez Prada <msanchez@igalia.com>

- [GTK] Move emissions of AtkDocument signals down to WebCore

- https://bugs.webkit.org/show_bug.cgi?id=73750

- Reviewed by Chris Fleizach.

- Implement the needed infrastructure to allow notifying

- accessibility, in a cross-platform way, when a event related to

- the load of a document happens. Added a generic method, which will

- be called from the FrameLoader, and platform specific versions of

- it so every port has a chance to decide what to do with those

- notifications.

- This patch doesn't include a new test because the one testing this

- functionality is the GTK-specific unit test added along with patch

- for bug 73746: testWebkitAtkDocumentLoadingEvents.

- * accessibility/AXObjectCache.h:

- (WebCore::AXObjectCache::frameLoadingEventNotification): New, called

- from the FrameLoader to notify accessibility when an event happens.

- (WebCore::AXObjectCache::frameLoadingEventPlatformNotification): New,

- platform specific function to let ports decide what to do.

- * accessibility/AXObjectCache.cpp:

- (WebCore::AXObjectCache::frameLoadingEventNotification): New.

- * accessibility/chromium/AXObjectCacheChromium.cpp:

- (WebCore::AXObjectCache::frameLoadingEventPlatformNotification): Dummy

- implementation of the platform specific function for chromium.

- * accessibility/gtk/AXObjectCacheAtk.cpp:

- (WebCore::AXObjectCache::frameLoadingEventPlatformNotification):

- * accessibility/mac/AXObjectCacheMac.mm:

- (WebCore::AXObjectCache::frameLoadingEventPlatformNotification): Dummy

- implementation of the platform specific function for the Mac.

- * accessibility/win/AXObjectCacheWin.cpp:

- (WebCore::AXObjectCache::frameLoadingEventPlatformNotification): Dummy

- implementation of the platform specific function for Windows.

- * loader/FrameLoader.cpp:

- (WebCore::FrameLoader::prepareForLoadStart): Notify accessibility

- by calling the new frameLoadingEventNotification() function.

- (WebCore::FrameLoader::checkLoadCompleteForThisFrame): Ditto.

-2011-12-05 Benjamin Poulain <benjamin@webkit.org>

- Update String::containsOnlyASCII() to handle 8 bits strings

- https://bugs.webkit.org/show_bug.cgi?id=73799

- Reviewed by Darin Adler.

- When possible, change the call sites from charactersAreAllASCII()

- to the optimized version String::containsOnlyASCII().

- * platform/KURL.cpp:

- (WebCore::KURL::init):

- * platform/cf/BinaryPropertyList.cpp:

- (WebCore::BinaryPropertyListPlan::writeStringObject):

- * platform/graphics/chromium/FontCacheChromiumWin.cpp:

- (WebCore::FontCodepage::if):

-2011-12-01 Vangelis Kokkevis <vangelis@chromium.org>

- [chromium] Use ANGLE's texture_usage and texture_storage extensions when allocating compositor textures

- https://bugs.webkit.org/show_bug.cgi?id=73621

- When the extensions are available, compositor textures are allocated via

- glTexStorage2DEXT instead of glTexImage2D to eliminate creation of unnecessary

- mip levels on the service side. In addition, the GL_FRAMEBUFFER_ATTACHMENT_ANGLE

- is specified for all textures used by RenderSurfaces to eliminate the need for

- a system memory bitmap allocation.

- Reviewed by Kenneth Russell.

- * platform/graphics/chromium/Extensions3DChromium.h:

- * platform/graphics/chromium/LayerRendererChromium.cpp:

- (WebCore::LayerRendererChromium::initialize):

- (WebCore::LayerRendererChromium::initializeSharedObjects):

- * platform/graphics/chromium/TrackingTextureAllocator.cpp:

- (WebCore::TrackingTextureAllocator::TrackingTextureAllocator):

- (WebCore::textureToStorageFormat):

- (WebCore::isTextureFormatSupportedForStorage):

- (WebCore::TrackingTextureAllocator::createTexture):

- * platform/graphics/chromium/TrackingTextureAllocator.h:

- (WebCore::TrackingTextureAllocator::setTextureUsageHint):

- (WebCore::TrackingTextureAllocator::setUseTextureStorageExt):

- * platform/graphics/chromium/cc/CCLayerTreeHost.h:

- (WebCore::LayerRendererCapabilities::LayerRendererCapabilities):

-2011-12-05 Chris Fleizach <cfleizach@apple.com>

- AX: aria-hidden doesn't work on iframe elements

- https://bugs.webkit.org/show_bug.cgi?id=73857

- Reviewed by Darin Adler.

- Test: platform/mac/accessibility/iframe-aria-hidden.html

- * accessibility/AccessibilityRenderObject.cpp:

- (WebCore::AccessibilityRenderObject::addAttachmentChildren):

- (WebCore::AccessibilityRenderObject::addChildren):

- * accessibility/AccessibilityRenderObject.h:

- * accessibility/AccessibilityScrollView.cpp:

- (WebCore::AccessibilityScrollView::accessibilityIsIgnored):

- (WebCore::AccessibilityScrollView::addChildren):

- * accessibility/AccessibilityScrollView.h:

-2011-12-05 Chris Fleizach <cfleizach@apple.com>

- AX: web search mechanism does not work with frames

- https://bugs.webkit.org/show_bug.cgi?id=73836

- Reviewed by Beth Dakin.

- This allows searching for, and within, frames for elements using the accessibility

- search mechanism.

- Test: platform/mac/accessibility/search-with-frames.html

- * accessibility/AccessibilityObject.cpp:

- (WebCore::AccessibilityObject::isAccessibilityObjectSearchMatch):

- (WebCore::appendAccessibilityObject):

- (WebCore::appendChildrenToArray):

-2011-12-05 Anders Carlsson <andersca@apple.com>

- Make LayerFlushSchedulerClient::flushLayers indicate whether the flush was successful or not

- https://bugs.webkit.org/show_bug.cgi?id=73862

- Reviewed by Andy Estes.

- Change LayerFlushSchedulerClient::flushLayers to return a boolean. If it returns true, the flush was

- successful and the run loop observer will be invalidated.

- * platform/graphics/ca/LayerFlushScheduler.h:

- * platform/graphics/ca/LayerFlushSchedulerClient.h:

- * platform/graphics/ca/mac/LayerFlushSchedulerMac.cpp:

- (WebCore::LayerFlushScheduler::runLoopObserverCallback):

-2011-12-05 Dana Jansens <danakj@chromium.org>

- [chromium] Set opaque flag for ImageLayerChromium

- https://bugs.webkit.org/show_bug.cgi?id=72964

- Reviewed by James Robinson.

- New unit test in tests/ImageLayerChromiumTest.cpp.

- * platform/graphics/chromium/GraphicsLayerChromium.cpp:

- (WebCore::GraphicsLayerChromium::setContentsToImage):

- * platform/graphics/chromium/GraphicsLayerChromium.h:

- (WebCore::GraphicsLayerChromium::contentsLayer):

- * platform/graphics/chromium/ImageLayerChromium.cpp:

- (WebCore::ImageLayerChromium::setContents):

-2011-12-05 Julien Chaffraix <jchaffraix@webkit.org>

- TD width in precentage doesn't work.

- https://bugs.webkit.org/show_bug.cgi?id=34758

- Reviewed by David Hyatt.

- Test: fast/table/colspan-with-all-percent-cells.html

- * rendering/AutoTableLayout.cpp:

- (WebCore::AutoTableLayout::calcEffectiveLogicalWidth):

- In the case where all our columns have percent lengths, split the colspan

- logical width using the percentages from the lengths. This should be properly

- handled by the massive 'else' case but it is not and I did not feel like refactor

- that due to massive compatibility issues that would arise.

-2011-12-05 Chris Fleizach <cfleizach@apple.com>

- platform/mac/accessibility/search-when-element-starts-in-table.html is failing

- https://bugs.webkit.org/show_bug.cgi?id=73751

- When encountering a table, the rows() and not the cells() should be queried (otherwise

- we can end up finding the element we started with).

- When searching in reverse, we also need to account for searching elements within the

- parent hierarchy. because technically it is "behind" the start element.

- Reviewed by Darin Adler.

- * accessibility/AccessibilityObject.cpp:

- (WebCore::appendChildrenToArray):

- (WebCore::AccessibilityObject::objectMatchesSearchCriteriaWithResultLimit):

- (WebCore::AccessibilityObject::findMatchingObjects):

- * accessibility/AccessibilityObject.h:

-2011-12-02 Jer Noble <jer.noble@apple.com>

- <video> elements should disable the system and display sleep when playing on OS X.

- https://bugs.webkit.org/show_bug.cgi?id=73730

- <rdar://problem/9502155>

- Reviewed by Alexey Proskuryakov.

- No new tests; platform specific system behavior only.

- Create a new DisplaySleepDisabler object when the playback rate becomes non-zero, and destroy

- that object when the playback rate drops back to zero.

- * html/HTMLMediaElement.cpp:

- (WebCore::HTMLMediaElement::mediaPlayerRateChanged):

- * html/HTMLMediaElement.h:

-2011-12-05 Peter Beverloo <peter@chromium.org>

- [Chromium] Add Android keycodes and build Linux clipboard/filesystem files.

- https://bugs.webkit.org/show_bug.cgi?id=73672

- Add the KeyCodeConversionAndroid.cpp file, which is a partial re-land

- of an older file from the Android port:

- http://trac.webkit.org/browser/trunk/WebCore/platform/android/KeyEventAndroid.cpp?rev=56704

- Add IconChromiumAndroid.cpp which nullifies the rendering of icons in

- file upload boxes, which is not something we want to support now. Also

- include ClipboardChromiumLinux.cpp and FileSystemChromiumLinux.cpp

- as their functionality can be re-used for Android.

- Reviewed by Adam Barth.

- * WebCore.gyp/WebCore.gyp:

- * WebCore.gypi:

- * platform/chromium/KeyCodeConversionAndroid.cpp: Added.

- (WebCore::windowsKeyCodeForKeyEvent):

- * platform/graphics/chromium/IconChromiumAndroid.cpp: Added.

- (WebCore::Icon::Icon):

- (WebCore::Icon::~Icon):

- (WebCore::Icon::paint):

-2011-12-05 Steve Falkenburg <sfalken@apple.com>

- Reviewed by Sam Weinig.

- On Windows, filenames not properly preserved when copied into a file list exposed by Event.dataTransfer

- https://bugs.webkit.org/show_bug.cgi?id=73841

- <rdar://problem/10521879>

- No test since repro case involves dropping a file onto the WebView.

- Calling characters() explicitly causes a non-terminated string buffer to get passed back

- to the String() constructor that expects a terminated buffer. The characters() call isn't

- necessary at all, since we have a String and the method we're calling expects a String.

- * platform/win/ClipboardWin.cpp:

- (WebCore::ClipboardWin::files): Remove characters() since it doesn't null terminate.

-2011-12-05 Timothy Hatcher <timothy@apple.com>

- Keep both InspectorBackend.js and InspectorBackendStub.js in Release builds after

- they have been combined into inspector.js.

- The InspectorBackend.js file split out of the generated InspectorBackendStub.js

- in r101670, and both files are needed to be useful.

- https://webkit.org/b/73839

- Reviewed by Joseph Pecoraro and Brian Weinstein.

- * WebCore.xcodeproj/project.pbxproj: Pass -not -name "InspectorBackend*.js" to find

- instead of -not -name InspectorBackendStub.js.

-2011-12-05 Mikhail Naganov <mnaganov@chromium.org>

- Web Inspector: [Chromium] Heap profiler should designate weak references.

- https://bugs.webkit.org/show_bug.cgi?id=69948

- Weak references are now ignored when tracing paths to GC roots.

- Reviewed by Yury Semikhatsky.

- * inspector/front-end/HeapSnapshot.js:

- (WebInspector.HeapSnapshotEdge.prototype.get isWeak):

- (WebInspector.HeapSnapshotEdge.prototype.toString):

- (WebInspector.HeapSnapshotEdge.prototype.get _hasStringName):

- (WebInspector.HeapSnapshotRetainerEdge.prototype.get isWeak):

- (WebInspector.HeapSnapshot.prototype._init):

- (WebInspector.HeapSnapshotPathFinder.prototype._fillRootChildren):

- (WebInspector.HeapSnapshotPathFinder.prototype._skipEdge):

-2011-12-05 Sheriff Bot <webkit.review.bot@gmail.com>

- Unreviewed, rolling out r102004.

- http://trac.webkit.org/changeset/102004

- https://bugs.webkit.org/show_bug.cgi?id=73835

- Breaks qt minimal release compilation (Requested by vsevik on

- #webkit).

- * inspector/CodeGeneratorInspector.py:

- (Capitalizer.upper_camel_case_to_lower):

- (RawTypes.get):

- (RawTypes.String.get_c_param_type):

- (RawTypes.Object.get_c_param_type):

- (RawTypes.Object):

- (ParamType):

- (TypeData.__init__):

- (TypeData.get_raw_type):

- (TypeMap.__init__):

- (InspectorFrontend_h):

- (InspectorArray):

- (InspectorObject):

- (String):

- (InspectorBackendDispatcher_h):

- (Generator.process_command):

- * inspector/InspectorValues.h:

-2011-12-05 Peter Rybin <peter.rybin@gmail.com>

- Web Inspector: [protocol] generate C++ classes for protocol JSON named types

- https://bugs.webkit.org/show_bug.cgi?id=72835

- Reviewed by Yury Semikhatsky.

- Extends python generator functionality.

- Makes constructor in InspectorObject public.

- * inspector/CodeGeneratorInspector.py:

- * inspector/InspectorValues.h:

-2011-12-05 Pavel Podivilov <podivilov@chromium.org>

- Web Inspector: fix fronted compilation.

- https://bugs.webkit.org/show_bug.cgi?id=73831

- Reviewed by Yury Semikhatsky.

- * inspector/front-end/CompilerSourceMapping.js:

- (WebInspector.ClosureCompilerSourceMappingPayload):

-2011-12-05 Eric Carlson <eric.carlson@apple.com>

- Out-of-band text tracks may only load from same origin as the media element's Document's origin

- https://bugs.webkit.org/show_bug.cgi?id=73184

- Reviewed by Sam Weinig.

- Test: http/tests/security/text-track-crossorigin.html

- * html/HTMLTrackElement.cpp:

- (WebCore::urlForLogging): Debug-only function for logging urls.

- (WebCore::HTMLTrackElement::scheduleLoad): Call canLoadUrl() before passing control off to

- the Track.

- (WebCore::HTMLTrackElement::canLoadUrl): Don't ask HTMLMediaElement to validate the url, the

- requirements for <track> are different from <video>.

- (WebCore::HTMLTrackElement::didCompleteLoad): Change bool param to enum.

- (WebCore::HTMLTrackElement::mediaElementCrossOriginAttribute): New, return parent 'crossorigin'

- attribute value.

- * html/HTMLTrackElement.h:

- * html/LoadableTextTrack.cpp:

- (WebCore::LoadableTextTrack::scheduleLoad): Add comments from the spec.

- (WebCore::LoadableTextTrack::loadTimerFired): Always cancel pending loads. Let the caller know

- if the loader refuses the url immediately.

- (WebCore::LoadableTextTrack::cueLoadingStarted): The <track> deals with readyState.

- (WebCore::LoadableTextTrack::cueLoadingCompleted): HTMLTrackElement::didCompleteLoad takes

- an enum, not a bool.

- * loader/TextTrackLoader.cpp:

- (WebCore::TextTrackLoader::corsPolicyPreventedLoad): New, log the error and set m_state.

- (WebCore::TextTrackLoader::notifyFinished): Check for CORS failure.

- (WebCore::TextTrackLoader::load): Take media element cross-origin attribute as a param so we

- can make the correct checks.

- * loader/TextTrackLoader.h:

-2011-12-05 Roland Steiner <rolandsteiner@chromium.org>

- "Raw" pseudo selectors don't match if immediately after a child or descendant combinator

- https://bugs.webkit.org/show_bug.cgi?id=72933

- Remove shortcut that prevents universal selectors from being created before shadow pseudo-elements.

- Reviewed by Antti Koivisto.

- * css/CSSParser.cpp:

- (WebCore::CSSParser::updateSpecifiersWithElementName):

-2011-12-05 Sheriff Bot <webkit.review.bot@gmail.com>

- Unreviewed, rolling out r101983.

- http://trac.webkit.org/changeset/101983

- https://bugs.webkit.org/show_bug.cgi?id=73827

- It broke all tests on GTK and on Qt in debug mode (Requested

- by Ossy on #webkit).

- * dom/Document.h:

- (WebCore::Node::Node):

- * dom/Node.cpp:

- (WebCore::Node::~Node):

- * dom/Node.h:

- (WebCore::Node::inDocument):

-2011-12-05 Roland Steiner <rolandsteiner@chromium.org>

- <style scoped>: Add 'scoped' attribute

- https://bugs.webkit.org/show_bug.cgi?id=67718

- Add 'scoped' attribute to IDL and attribute list,

- implement and test setting/resetting of the attribute.

- Reviewed by Antti Koivisto.

- Test: fast/css/style-scoped/basic-attribute.html

- * html/HTMLAttributeNames.in:

- * html/HTMLStyleElement.cpp:

- (WebCore::HTMLStyleElement::scoped):

- (WebCore::HTMLStyleElement::setScoped):

- (WebCore::HTMLStyleElement::scopingElement):

- * html/HTMLStyleElement.h:

- * html/HTMLStyleElement.idl:

-2011-12-05 Hajime Morrita <morrita@chromium.org>

- [Refactoring] Accessing Node::m_document should be minimized.

- https://bugs.webkit.org/show_bug.cgi?id=73800

- Reviewed by Kent Tamura.

- No new tests. No behavioral change.

- Replaced m_document reference with the document() accessor

- or temporaril variables. This is a preparation for using

- m_document space to point a shadow root pointer.

- * dom/Document.h:

- (WebCore::Node::Node):

- * dom/Node.cpp:

- (WebCore::Node::~Node):

- * dom/Node.h:

- (WebCore::Node::inDocument):

-2011-12-05 Shinya Kawanaka <shinyak@google.com>

- Asynchronous SpellChecker should consider multiple requests.

- https://bugs.webkit.org/show_bug.cgi?id=72939

- Reviewed by Hajime Morita.

- Now SpellChecker saves a request when it is processing the previous spellcheck request.

- If there is a request having the same root editable element, the older request is replaced by newer request

- Test: editing/spelling/spellcheck-queue.html

- * editing/SpellChecker.cpp:

- (WebCore::SpellChecker::SpellCheckRequest::SpellCheckRequest):

- A structure to have spell check request.

- (WebCore::SpellChecker::SpellCheckRequest::sequence):

- (WebCore::SpellChecker::SpellCheckRequest::range):

- (WebCore::SpellChecker::SpellCheckRequest::text):

- (WebCore::SpellChecker::SpellCheckRequest::mask):

- (WebCore::SpellChecker::SpellCheckRequest::rootEditableElement):

- (WebCore::SpellChecker::SpellChecker):

- (WebCore::SpellChecker::createRequest):

- (WebCore::SpellChecker::timerFiredToProcessQueuedRequest):

- When timer is fired, queued request is processed if any.

- (WebCore::SpellChecker::canCheckAsynchronously):

- (WebCore::SpellChecker::requestCheckingFor):

- When the spellchecker is processing another request, the latest request is queued.

- (WebCore::SpellChecker::invokeRequest):

- (WebCore::SpellChecker::enqueueRequest):

- Enqueues a request. If there is an older request whose root editable element is the same as the request,

- it will be replaced.

- (WebCore::SpellChecker::didCheck):

- * editing/SpellChecker.h:

-2011-12-05 Eric Carlson <eric.carlson@apple.com>

- WebCore part of: Add WebKit preferences for text track settings

- https://bugs.webkit.org/show_bug.cgi?id=73721

- Reviewed by John Sullivan.

- No new tests yet, still nothing to test.

- * page/Settings.cpp:

- (WebCore::Settings::setShouldDisplaySubtitles): Move the setters to the .cpp file so they

- aren't inlined.

- (WebCore::Settings::setShouldDisplayCaptions): Ditto.

- (WebCore::Settings::setShouldDisplayTextDescriptions): Ditto.

- * page/Settings.h:

-2011-12-05 Noel Gordon <noel.gordon@gmail.com>

- ImageDecoder setSize() should check for backing store allocation failure

- https://bugs.webkit.org/show_bug.cgi?id=72864

- Reviewed by Adam Barth.

- The backing store of a decoded image is a Vector<PixelData> on the affected

- ports. And Vector<> provides a resize capacity member that returns false if

- memory allocation fails.

- setSize() should be called once only during an image decode - add an ASSERT

- for that. Resize the backing store capacity to the requested image size and

- return false if memory allocation fails.

- ImageDecoder::isOverSize(width, height) is called to check that the decoded

- width and height won't overflow 'width x height x sizeof(PixelData)' before

- calls to setSize(). Refer to http://webkit.org/b/48634

- No new tests. Covered by fast/images/size-failure.html

- * platform/image-decoders/ImageDecoder.cpp:

- (WebCore::ImageFrame::setSize):

-2011-12-04 Andreas Kling <kling@webkit.org>

- CSSStyleSheet can't be reparented, enforce this at compile time.

- <http://webkit.org/b/73793>

- Reviewed by Benjamin Poulain.

- * css/StyleSheet.h:

- (WebCore::StyleSheet::clearOwnerRule):

- Changed setParentRule(CSSImportRule*) to clearOwnerRule() to document and

- enforce the fact that style sheets should never be reparented after creation.

- (WebCore::StyleSheet::ownerRule):

- Renamed parentRule() to ownerRule() to match the CSSOM name.

- (WebCore::StyleSheet::ownerNode):

- (WebCore::StyleSheet::clearOwnerNode):

- Also renamed StyleSheet::m_parentNode to m_ownerNode to match its accessors.

- * css/CSSStyleSheet.h:

- Removed ownerRule() as we now inherit it from StyleSheet.

- * bindings/js/JSDOMBinding.h:

- (WebCore::root):

- * css/CSSImportRule.cpp:

- (WebCore::CSSImportRule::~CSSImportRule):

- (WebCore::CSSImportRule::setCSSStyleSheet):

- * css/StyleSheet.cpp:

- (WebCore::StyleSheet::StyleSheet):

- (WebCore::StyleSheet::parentStyleSheet):

- (WebCore::StyleSheet::baseURL):

- * inspector/InspectorStyleSheet.cpp:

- (WebCore::fillMediaListChain):

- Update call sites to use the new names.

-2011-12-04 Andreas Kling <kling@webkit.org>

- border-width: initial and border-color: initial cannot be removed via CSSOM.

- <http://webkit.org/b/68551>

- Reviewed by Darin Adler.

- Test: fast/css/cssom-remove-shorthand-property.html

- When asked to remove a shorthand property, we should toss out both the

- sub-properties of that shorthand as well as the shorthand itself, should the

- declaration contain it. The latter part was missing in our implementation.

- * css/CSSMutableStyleDeclaration.h:

- * css/CSSMutableStyleDeclaration.cpp:

- (WebCore::CSSMutableStyleDeclaration::removePropertiesInSet):

- (WebCore::CSSMutableStyleDeclaration::removeShorthandProperty):

- Changed to return true only if something was actually removed.

-2011-12-04 Ryosuke Niwa <rniwa@webkit.org>

- HIERARCHY_REQUEST_ERR check in checkAcceptChild should be optimized for newChild without children

- https://bugs.webkit.org/show_bug.cgi?id=73737

- Reviewed by Darin Adler.

- It turned out that 50-70% of nodes inserted by DOM APIs such as insertBefore and appendChild

- don't have any descendent nodes. Optimize isDescendantOf which is used by checkAcceptChild for this case.

- On a test case attached on the bug, we see a 40% improvement.

- Also optimize for cases where either new child or new parent but not both are in document as suggested

- by Erik Arvidsson. This appears to happen about 40-70% of the time, and the symmetric difference between

- the two cases is about 50% so it's worth implementing both optimizations.

- Unfortunately no tests because we still have a O(n) algorithm somewhere.

- * dom/Node.cpp:

- (WebCore::Node::isDescendantOf):

- (WebCore::Node::contains):

-2011-12-04 Andreas Kling <kling@webkit.org>

- CSSValuePool: Inline trivial getters.

- <http://webkit.org/b/73763>

- Reviewed by Anders Carlsson.

- * css/CSSValuePool.cpp:

- * css/CSSValuePool.h:

- (WebCore::CSSValuePool::createInheritedValue): Inlined.

- (WebCore::CSSValuePool::createImplicitInitialValue): Ditto.

- (WebCore::CSSValuePool::createExplicitInitialValue): Ditto.

-2011-12-03 Noel Gordon <noel.gordon@gmail.com>

- Fix WebPImageDecoder decoder leak.

- https://bugs.webkit.org/show_bug.cgi?id=73756

- Reviewed by Andreas Kling.

- Delete the m_decoder member in the destructor if needed with WebPIDelete.

- No new tests, it's valgrind territory.

- * platform/image-decoders/webp/WEBPImageDecoder.cpp:

- (WebCore::WEBPImageDecoder::~WEBPImageDecoder):

-2011-12-03 Andreas Kling <kling@webkit.org>

- CSSStyleSheet: Parent rule can only ever be @import, enforce this at compile time.

- <http://webkit.org/b/73725>

- Reviewed by Darin Adler.

- Made StyleSheet::m_parentRule a CSSImportRule* rather than a CSSRule*

- and updated getters, setters and constructors accordingly.

- There is no change in behavior, this simply enforces the status quo.

- * WebCore.xcodeproj/project.pbxproj:

- * bindings/js/JSDOMBinding.h:

- * bindings/scripts/CodeGeneratorV8.pm:

- (AddIncludesForType):

- * css/CSSStyleSheet.cpp:

- (WebCore::CSSStyleSheet::CSSStyleSheet):

- * css/CSSStyleSheet.h:

- (WebCore::CSSStyleSheet::create):

- (WebCore::CSSStyleSheet::ownerRule):

- * css/StyleSheet.cpp:

- (WebCore::StyleSheet::StyleSheet):

- * css/StyleSheet.h:

- (WebCore::StyleSheet::parentRule):

- (WebCore::StyleSheet::setParentRule):

-2011-12-03 Andreas Kling <kling@webkit.org>

- Unreviewed, revert accidental change from r101932.

- * bindings/scripts/CodeGeneratorV8.pm:

- (AddIncludesForType):

-2011-12-03 Mary Wu <mary.wu@torchmobile.com.cn>

- Upstream 4 files into WebCore/platform/blackberry

- https://bugs.webkit.org/show_bug.cgi?id=73614

- Reviewed by Eric Seidel.

- * platform/blackberry/CursorBlackBerry.cpp: Added.

- (WebCore::AllCursors::AllCursors):

- (WebCore::getCursor):

- (WebCore::Cursor::Cursor):

- (WebCore::Cursor::~Cursor):

- (WebCore::Cursor::operator=):

- (WebCore::aliasCursor):

- (WebCore::cellCursor):

- (WebCore::columnResizeCursor):

- (WebCore::contextMenuCursor):

- (WebCore::copyCursor):

- (WebCore::crossCursor):

- (WebCore::eastResizeCursor):

- (WebCore::eastWestResizeCursor):

- (WebCore::grabbingCursor):

- (WebCore::grabCursor):

- (WebCore::handCursor):

- (WebCore::helpCursor):

- (WebCore::iBeamCursor):

- (WebCore::moveCursor):

- (WebCore::noDropCursor):

- (WebCore::noneCursor):

- (WebCore::northEastResizeCursor):

- (WebCore::northEastSouthWestResizeCursor):

- (WebCore::northResizeCursor):

- (WebCore::northSouthResizeCursor):

- (WebCore::northWestResizeCursor):

- (WebCore::northWestSouthEastResizeCursor):

- (WebCore::notAllowedCursor):

- (WebCore::pointerCursor):

- (WebCore::progressCursor):

- (WebCore::rowResizeCursor):

- (WebCore::southEastResizeCursor):

- (WebCore::southResizeCursor):

- (WebCore::southWestResizeCursor):

- (WebCore::verticalTextCursor):

- (WebCore::waitCursor):

- (WebCore::westResizeCursor):

- (WebCore::zoomInCursor):

- (WebCore::zoomOutCursor):

- (WebCore::middlePanningCursor):

- (WebCore::eastPanningCursor):

- (WebCore::northPanningCursor):

- (WebCore::northEastPanningCursor):

- (WebCore::northWestPanningCursor):

- (WebCore::southPanningCursor):

- (WebCore::southEastPanningCursor):

- (WebCore::southWestPanningCursor):

- (WebCore::westPanningCursor):

- * platform/blackberry/DragDataBlackBerry.cpp: Added.

- (WebCore::DragData::canSmartReplace):

- (WebCore::DragData::containsColor):

- (WebCore::DragData::containsCompatibleContent):

- (WebCore::DragData::containsFiles):

- (WebCore::DragData::containsPlainText):

- (WebCore::DragData::containsURL):

- (WebCore::DragData::asFilenames):

- (WebCore::DragData::asColor):

- (WebCore::DragData::asPlainText):

- (WebCore::DragData::asURL):

- (WebCore::DragData::asFragment):

- * platform/blackberry/DragImageBlackBerry.cpp: Added.

- (WebCore::createDragImageFromImage):

- (WebCore::createDragImageIconForCachedImage):

- (WebCore::deleteDragImage):

- (WebCore::dissolveDragImageToFraction):

- (WebCore::scaleDragImage):

- (WebCore::dragImageSize):

- * platform/blackberry/EventLoopBlackBerry.cpp: Added.

- (WebCore::EventLoop::platformInit):

- (WebCore::EventLoop::cycle):

-2011-12-03 Andreas Kling <kling@webkit.org>

- Cache CSSInitialValue instances per-document.

- <http://webkit.org/b/73745>

- Reviewed by Oliver Hunt.

- Test: http/tests/security/cross-origin-css-primitive.html

- Have CSSValuePool manage the caching of CSSInitialValue objects.

- * css/CSSInitialValue.h:

- (WebCore::CSSInitialValue::createExplicit):

- (WebCore::CSSInitialValue::createImplicit):

- (WebCore::CSSInitialValue::CSSInitialValue):

- * css/CSSParser.cpp:

- (WebCore::CSSParser::parseValue):

- (WebCore::CSSParser::parseFillShorthand):

- (WebCore::CSSParser::parseAnimationShorthand):

- (WebCore::CSSParser::parseTransitionShorthand):

- (WebCore::CSSParser::parseShorthand):

- * css/CSSValuePool.cpp:

- (WebCore::CSSValuePool::CSSValuePool):

- (WebCore::CSSValuePool::createImplicitInitialValue):

- (WebCore::CSSValuePool::createExplicitInitialValue):

- * css/CSSValuePool.h:

-2011-12-03 Andreas Kling <kling@webkit.org>

- Keep CSSInheritedValue in the CSS value pool.

- <http://webkit.org/b/73747>

- Reviewed by Antti Koivisto.

- We only need one CSSInheritedValue instance per document, so cache it

- in CSSValuePool and have the parser create it through there.

- * css/CSSParser.cpp:

- (WebCore::CSSParser::parseValue):

- * css/CSSValuePool.cpp:

- (WebCore::CSSValuePool::CSSValuePool):

- (WebCore::CSSValuePool::createInheritedValue):

- * css/CSSValuePool.h:

-2011-12-03 Andreas Kling <kling@webkit.org>

- Rename CSSPrimitiveValueCache to CSSValuePool.

- <http://webkit.org/b/73742>

- Reviewed by Antti Koivisto.

- CSSPrimitiveValueCache -> CSSValuePool

- Document::primitiveValueCache() -> Document::cssValuePool()

- This is in preparation for sharing more than just primitive values.

- * CMakeLists.txt:

- * GNUmakefile.list.am:

- * Target.pri:

- * WebCore.gypi:

- * WebCore.vcproj/WebCore.vcproj:

- * WebCore.xcodeproj/project.pbxproj:

- * bindings/scripts/CodeGeneratorV8.pm:

- (AddIncludesForType):

- * css/CSSComputedStyleDeclaration.cpp:

- (WebCore::valueForNinePieceImageSlice):

- (WebCore::valueForNinePieceImageQuad):

- (WebCore::valueForNinePieceImageRepeat):

- (WebCore::valueForNinePieceImage):

- (WebCore::zoomAdjustedPixelValue):

- (WebCore::zoomAdjustedNumberValue):

- (WebCore::zoomAdjustedPixelValueForLength):

- (WebCore::valueForReflection):

- (WebCore::getPositionOffsetValue):

- (WebCore::CSSComputedStyleDeclaration::currentColorOrValidColor):

- (WebCore::getBorderRadiusCornerValue):

- (WebCore::computedTransform):

- (WebCore::CSSComputedStyleDeclaration::valueForFilter):

- (WebCore::valueForGridTrackList):

- (WebCore::getDelayValue):

- (WebCore::getDurationValue):

- (WebCore::createLineBoxContainValue):

- (WebCore::CSSComputedStyleDeclaration::getFontSizeCSSValuePreferringKeyword):

- (WebCore::CSSComputedStyleDeclaration::valueForShadow):

- (WebCore::valueForFamily):

- (WebCore::renderTextDecorationFlagsToCSSValue):

- (WebCore::fillRepeatToCSSValue):

- (WebCore::fillSizeToCSSValue):

- (WebCore::contentToCSSValue):

- (WebCore::counterToCSSValue):

- (WebCore::fontFamilyFromStyle):

- (WebCore::lineHeightFromStyle):

- (WebCore::fontSizeFromStyle):

- (WebCore::fontStyleFromStyle):

- (WebCore::fontVariantFromStyle):

- (WebCore::fontWeightFromStyle):

- (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

- * css/CSSParser.cpp:

- (WebCore::parseColorValue):

- (WebCore::parseSimpleLengthValue):

- (WebCore::CSSParser::setStyleSheet):

- (WebCore::CSSParser::createPrimitiveNumericValue):

- (WebCore::CSSParser::createPrimitiveStringValue):

- (WebCore::CSSParser::parseValidPrimitive):

- (WebCore::CSSParser::parseValue):

- (WebCore::parseBackgroundClip):

- (WebCore::CSSParser::parseFillShorthand):

- (WebCore::CSSParser::parsePage):

- (WebCore::CSSParser::parseSizeParameter):

- (WebCore::CSSParser::parseContent):

- (WebCore::CSSParser::parseAttr):

- (WebCore::CSSParser::parseBackgroundColor):

- (WebCore::CSSParser::parseFillPositionX):

- (WebCore::CSSParser::parseFillPositionY):

- (WebCore::CSSParser::parseFillPositionComponent):

- (WebCore::CSSParser::parseFillPosition):

- (WebCore::CSSParser::parseFillRepeat):

- (WebCore::CSSParser::parseFillSize):

- (WebCore::CSSParser::parseFillProperty):

- (WebCore::CSSParser::parseAnimationDirection):

- (WebCore::CSSParser::parseAnimationFillMode):

- (WebCore::CSSParser::parseAnimationIterationCount):

- (WebCore::CSSParser::parseAnimationName):

- (WebCore::CSSParser::parseAnimationPlayState):

- (WebCore::CSSParser::parseAnimationProperty):

- (WebCore::CSSParser::parseAnimationTimingFunction):

- (WebCore::CSSParser::parseGridTrackList):

- (WebCore::CSSParser::parseDashboardRegions):

- (WebCore::CSSParser::parseCounterContent):

- (WebCore::CSSParser::parseShape):

- (WebCore::CSSParser::parseWrapShape):

- (WebCore::CSSParser::parseFont):

- (WebCore::CSSParser::parseFontFamily):

- (WebCore::CSSParser::parseFontStyle):

- (WebCore::CSSParser::parseFontVariant):

- (WebCore::CSSParser::parseFontWeight):

- (WebCore::CSSParser::parseColor):

- (WebCore::ShadowParseContext::ShadowParseContext):

- (WebCore::ShadowParseContext::commitLength):

- (WebCore::ShadowParseContext::commitStyle):

- (WebCore::CSSParser::parseShadow):

- (WebCore::CSSParser::parseReflect):

- (WebCore::CSSParser::parseFlex):

- (WebCore::BorderImageParseContext::BorderImageParseContext):

- (WebCore::CSSParser::parseBorderImage):

- (WebCore::CSSParser::parseBorderImageRepeat):

- (WebCore::BorderImageSliceParseContext::BorderImageSliceParseContext):

- (WebCore::BorderImageSliceParseContext::commitNumber):

- (WebCore::BorderImageSliceParseContext::commitBorderImageSlice):

- (WebCore::CSSParser::parseBorderImageSlice):

- (WebCore::BorderImageQuadParseContext::BorderImageQuadParseContext):

- (WebCore::BorderImageQuadParseContext::commitNumber):

- (WebCore::BorderImageQuadParseContext::commitBorderImageQuad):

- (WebCore::CSSParser::parseBorderImageQuad):

- (WebCore::CSSParser::parseBorderRadius):

- (WebCore::CSSParser::parseAspectRatio):

- (WebCore::CSSParser::parseCounter):

- (WebCore::parseDeprecatedGradientPoint):

- (WebCore::parseDeprecatedGradientColorStop):

- (WebCore::CSSParser::parseDeprecatedGradient):

- (WebCore::valueFromSideKeyword):

- (WebCore::parseGradientColorOrKeyword):

- (WebCore::CSSParser::parseLinearGradient):

- (WebCore::CSSParser::parseRadialGradient):

- (WebCore::CSSParser::parseCrossfade):

- (WebCore::CSSParser::parseCustomFilter):

- (WebCore::CSSParser::parseFilter):

- (WebCore::CSSParser::parseFlowThread):

- (WebCore::CSSParser::parseRegionThread):

- (WebCore::CSSParser::parseTextEmphasisStyle):

- (WebCore::CSSParser::parseFontFeatureSettings):

- * css/CSSParser.h:

- (WebCore::CSSParser::cssValuePool):

- * css/CSSValuePool.cpp: Renamed from Source/WebCore/css/CSSPrimitiveValueCache.cpp.

- (WebCore::CSSValuePool::CSSValuePool):

- (WebCore::CSSValuePool::~CSSValuePool):

- (WebCore::CSSValuePool::createIdentifierValue):

- (WebCore::CSSValuePool::createColorValue):

- (WebCore::CSSValuePool::createValue):

- * css/CSSValuePool.h: Renamed from Source/WebCore/css/CSSPrimitiveValueCache.h.

- (WebCore::CSSValuePool::create):

- (WebCore::CSSValuePool::createValue):

- * css/mediaControlsGtk.css:

- (audio::-webkit-media-controls-volume-slider, video::-webkit-media-controls-volume-slider):

- * dom/Document.cpp:

- (WebCore::Document::cssValuePool):

- * dom/Document.h:

-2011-12-03 Andreas Kling <kling@webkit.org>

- Shrink CSSValueList.

- <http://webkit.org/b/73732>

- Reviewed by Antti Koivisto.

- Packed CSSValueList::m_isSpaceSeparated into the CSSValue bit field

- and renamed it to m_isSpaceSeparatedValue, shrinking CSSValueList

- by one CPU word.

- Also renamed CSSValue::m_isImplicit to m_isImplicitInitialValue

- for good measure.

- * css/CSSInitialValue.h:

- (WebCore::CSSInitialValue::CSSInitialValue):

- * css/CSSValue.h:

- (WebCore::CSSValue::isImplicitInitialValue):

- (WebCore::CSSValue::CSSValue):

- * css/CSSValueList.cpp:

- (WebCore::CSSValueList::CSSValueList):

- (WebCore::CSSValueList::copy):

- (WebCore::CSSValueList::customCssText):

- * css/CSSValueList.h:

- (WebCore::CSSValueList::isSpaceSeparated):

-2011-12-02 Philippe Normand <pnormand@igalia.com>

- [GStreamer] Fullscreen controller support for the new WebKit Fullscreen API

- https://bugs.webkit.org/show_bug.cgi?id=66968

- Reviewed by Martin Robinson.

- * GNUmakefile.am: Don't use the Quicktime fullscreen CSS anymore,

- it wasn't suited for GTK in the first place anyway.

- * platform/gtk/RenderThemeGtk.cpp:

- (WebCore::RenderThemeGtk::extraFullScreenStyleSheet): Simply reuse

- GTK CSS.

-2011-12-03 Alejandro G. Castro <alex@igalia.com>

- [GTK] TextureMapperNode should not use Qt types and functions

- https://bugs.webkit.org/show_bug.cgi?id=73713

- Replace qreal with double and qMin with std::min.

- Reviewed by Noam Rosenthal.

- * platform/graphics/texmap/TextureMapperNode.cpp:

- (WebCore::solveCubicBezierFunction):

- (WebCore::solveStepsFunction):

-2011-12-03 Dan Winship <danw@gnome.org>

- [GTK] Remove lots of code that is now unnecessary after

- SoupRequestHTTP API changes.

- https://bugs.webkit.org/show_bug.cgi?id=71611

- Reviewed by Martin Robinson.

- No new tests; behavior is unchanged

- * platform/network/ResourceHandleInternal.h:

- (WebCore::ResourceHandleInternal::ResourceHandleInternal): remove

- no-longer-needed fields.

- * platform/network/soup/ResourceHandleSoup.cpp:

- (WebCore::finishedCallback): remove this, and m_finished, which

- we no longer need to explicitly track

- (WebCore::statusWillBeHandledBySoup):

- (WebCore::soupErrorShouldCauseLoadFailure): No longer needed;

- SoupRequestHTTP's logic is now aligned with WebKit's.

- (WebCore::gotChunkCallback): remove gotChunkCallback, which

- is no longer needed

- (WebCore::startHTTPRequest):

- (WebCore::sendRequestCallback):

- (WebCore::gotHeadersCallback):

- (WebCore::contentSniffedCallback): merge the code from

- gotHeadersCallback and contentSniffedCallback into

- sendRequestCallback

- * platform/network/soup/ResourceResponseSoup.cpp:

- (WebCore::ResourceResponse::updateFromSoupMessage): move the

- sniffing override logic here

-2011-12-03 Sheriff Bot <webkit.review.bot@gmail.com>

- Unreviewed, rolling out r101904.

- http://trac.webkit.org/changeset/101904

- https://bugs.webkit.org/show_bug.cgi?id=73739

- It broke zillions of tests on all bot (Requested by

- Ossy_weekend on #webkit).

- * bindings/js/JSDOMWindowCustom.cpp:

- (WebCore::handlePostMessage):

-2011-12-02 Kentaro Hara <haraken@chromium.org>

- [Refactoring] Use join(", ", @arguments) to build a method argument string in CodeGenerator*.pm

- https://bugs.webkit.org/show_bug.cgi?id=73651

- Reviewed by Darin Adler.

- The code in CodeGenerator*.pm to build a method argument string is really dirty

- and error-prone. It is building an argument string incrementally judging whether

- ", " is necessary or not, like this:

- my $method = ... ? "func(" : "func(a";

- if (...) {

- $method .= $method =~ /\($/ ? "b" : ", b";

- }

- $method .= ")";

- Alternatively, we can refactor the code as follows:

- my $funcName = "func";

- my @arguments;

- push(@arguments, "a") if (...);

- push(@arguments, "b") if (...);

- my $method = $funcName . "(" . join(", ", @arguments) . ")";

- This patch just refactors the code, and generated .h and .cpp files should be

- exactly the same as the current .h and .cpp files.

- Tests: bindings/scripts/test/*

- * bindings/scripts/CodeGenerator.pm:

- (GetterExpression):

- (SetterExpression):

- * bindings/scripts/CodeGeneratorCPP.pm:

- (GenerateImplementation):

- * bindings/scripts/CodeGeneratorGObject.pm:

- ():

- * bindings/scripts/CodeGeneratorJS.pm:

- (GenerateImplementation):

- * bindings/scripts/CodeGeneratorObjC.pm:

- (GenerateImplementation):

- * bindings/scripts/CodeGeneratorV8.pm:

- (GenerateNormalAttrGetter):

- (GenerateNormalAttrSetter):

-2011-12-02 David Levin <levin@chromium.org>

- Rename WTF class from TemporarilyChange to TemporaryChange.

- https://bugs.webkit.org/show_bug.cgi?id=73479

- Reviewed by Eric Seidel.

- * ForwardingHeaders/wtf/TemporarilyChange.h: Removed.

- * ForwardingHeaders/wtf/TemporaryChange.h: Added.

- * page/FrameView.cpp:

- (WebCore::FrameView::forceLayoutParentViewIfNeeded):

- (WebCore::FrameView::layout):

- (WebCore::FrameView::setScrollPosition):

- (WebCore::FrameView::autoSizeIfEnabled):

-2011-12-02 Armand Navabi <armand.navabi@gmail.com>

- ASSERT fails in updateState ACTION_DRAW case

- https://bugs.webkit.org/show_bug.cgi?id=73351

- Reviewed by James Robinson.

- Changed ASSERT in CCSchedulerStateMachine to include ( || !m_visible) as discussed in bug

- report. m_commitState is set to COMMIT_STATE_WAITING_FOR_FIRST_DRAW if m_needsCommit or

- !m_visible, so in ACTION_DRAW the assert should have both conditions.

- * platform/graphics/chromium/cc/CCSchedulerStateMachine.cpp:

- (WebCore::CCSchedulerStateMachine::updateState):

-2011-12-02 Shawn Singh <shawnsingh@chromium.org>

- [chromium] Scissor rect optimization for chromium compositor

- https://bugs.webkit.org/show_bug.cgi?id=67341

- Reviewed by James Robinson.

- Mostly covered by damage tracker tests. Currently this relies on

- manually running layout tests, because this patch requires partial

- swaps that are not supported by DumpRenderTree. The feature

- automatically disables if partial swap is not supported.

- * platform/graphics/chromium/LayerRendererChromium.cpp:

- (WebCore::LayerRendererChromium::initialize):

- (WebCore::LayerRendererChromium::trackDamageForAllSurfaces):

- (WebCore::LayerRendererChromium::drawLayersOntoRenderSurfaces):

- (WebCore::LayerRendererChromium::drawLayersInternal):

- (WebCore::LayerRendererChromium::swapBuffers):

- (WebCore::LayerRendererChromium::drawLayer):

- * platform/graphics/chromium/LayerRendererChromium.h:

- * platform/graphics/chromium/cc/CCLayerTreeHost.h:

- (WebCore::CCSettings::CCSettings):

- (WebCore::LayerRendererCapabilities::LayerRendererCapabilities):

- * platform/graphics/chromium/cc/CCRenderSurface.cpp:

- (WebCore::CCRenderSurface::draw):

- * platform/graphics/chromium/cc/CCRenderSurface.h:

-2011-12-02 Dmitry Lomov <dslomov@google.com>

- https://bugs.webkit.org/show_bug.cgi?id=73691

- [JSC] Implement correct order of window.postMessage arguments.

- Reviewed by Geoffrey Garen.

- * bindings/js/JSDOMWindowCustom.cpp:

- (WebCore::handlePostMessage):

2011-12-02 Stephen Chenney <schenney@chromium.org>

REGRESSION (r91125): Polyline tool in google docs is broken