Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(313)

Unified Diff: Source/WebCore/ChangeLog

Side-by-side diff isn't available for this file because of its large size.
Issue 9309085: Merge 106610 - Avoid crashing renderer when GPU process dies by not caching textures between vide... (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/1025/
Patch Set: Created 8 years, 10 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 | « no previous file | Source/WebCore/platform/graphics/chromium/cc/CCVideoLayerImpl.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/WebCore/ChangeLog
===================================================================
--- Source/WebCore/ChangeLog (revision 106674)
+++ Source/WebCore/ChangeLog (working copy)
@@ -1,5 +1,3507 @@
+2012-02-02 Ami Fischman <fischman@chromium.org>
+
+ Avoid crashing renderer when GPU process dies by not caching textures between video frames.
+ https://bugs.webkit.org/show_bug.cgi?id=77654
+
+ Reviewed by James Robinson.
+
+ * platform/graphics/chromium/cc/CCVideoLayerImpl.cpp:
+ (WebCore::CCVideoLayerImpl::draw):
+
+2012-02-02 Anders Carlsson <andersca@apple.com>
+
+ Add ScrollingTree class
+ https://bugs.webkit.org/show_bug.cgi?id=77695
+
+ Reviewed by Andreas Kling.
+
+ * WebCore.xcodeproj/project.pbxproj:
+ Add new files.
+
+ * page/scrolling/ScrollingCoordinator.cpp:
+ (WebCore::ScrollingCoordinator::ScrollingCoordinator):
+ Create a scrolling tree.
+
+ (WebCore::ScrollingCoordinator::~ScrollingCoordinator):
+ Assert that the scrolling tree is null.
+
+ (WebCore::ScrollingCoordinator::pageDestroyed):
+ Null out the scrolling tree member variable and tell it to invalidate itself.
+
+ * page/scrolling/ScrollingTree.cpp: Added.
+
+ (WebCore::ScrollingTree::invalidate):
+ Null out the scrolling coordinator, breaking the reference cycle between the scrolling
+ coordinator and the scrolling tree.
+
+ * page/scrolling/ScrollingTree.h: Added.
+
+2012-02-02 Tim Dresser <tdresser@chromium.org>
+
+ Refactor plugin drawing to be more data driven
+ https://bugs.webkit.org/show_bug.cgi?id=76715
+
+ Reviewed by James Robinson.
+
+ CCPluginLayerImpl no longer handles drawing itself, but produces a list of CCPluginDrawQuads.
+ These quads are then drawn by LayerRendererChromium.
+
+ CCLayerImpl::willDraw(LayerRendererChromium*) is called directly before appendQuads.
+ This allows for CCLayerImpl objects to allocate textures before appendQuads is called.
+
+ This is a refactor, so no new tests were added.
+ Flash was tested manually on Linux and Mac.
+
+ * platform/graphics/chromium/LayerRendererChromium.cpp:
+ (WebCore::PluginProgramBinding::set):
+ (PluginProgramBinding):
+ (WebCore):
+ (WebCore::TexStretchPluginProgramBinding::set):
+ (TexStretchPluginProgramBinding):
+ (WebCore::TexTransformPluginProgramBinding::set):
+ (TexTransformPluginProgramBinding):
+ (WebCore::LayerRendererChromium::drawPluginQuad):
+ * platform/graphics/chromium/cc/CCLayerImpl.h:
+ (WebCore::CCLayerImpl::willDraw):
+ * platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
+ (WebCore::CCLayerTreeHostImpl::calculateRenderPasses):
+ * platform/graphics/chromium/cc/CCPluginDrawQuad.cpp:
+ (WebCore::CCPluginDrawQuad::create):
+ (WebCore::CCPluginDrawQuad::CCPluginDrawQuad):
+ * platform/graphics/chromium/cc/CCPluginDrawQuad.h:
+ (CCPluginDrawQuad):
+ (WebCore::CCPluginDrawQuad::uvRect):
+ (WebCore::CCPluginDrawQuad::textureId):
+ (WebCore::CCPluginDrawQuad::flipped):
+ (WebCore::CCPluginDrawQuad::ioSurfaceWidth):
+ (WebCore::CCPluginDrawQuad::ioSurfaceHeight):
+ (WebCore::CCPluginDrawQuad::ioSurfaceTextureId):
+ * platform/graphics/chromium/cc/CCPluginLayerImpl.cpp:
+ (WebCore::CCPluginLayerImpl::willDraw):
+ (WebCore::CCPluginLayerImpl::appendQuads):
+ * platform/graphics/chromium/cc/CCPluginLayerImpl.h:
+ (CCPluginLayerImpl):
+
+2012-01-29 Pablo Flouret <pablof@motorola.com>
+
+ V8 idl code generator doesn't handle SerializedScriptValue attributes properly.
+ https://bugs.webkit.org/show_bug.cgi?id=77295
+
+ Reviewed by Kentaro Hara.
+
+ Only the case of one (and only one) SerializedScriptValue attribute was
+ handled, and it was deserialized eagerly in the constructor instead of
+ generating getters/setters. This patch gets rid of that behavior and
+ generates the getters and setters instead (similar to what the JSC
+ generator does).
+ Also handle the case of SSV attributes with [CachedAttribute]
+ declaration, caching the deserialized value in a hidden attribute on the
+ object.
+
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (GenerateNormalAttrGetter):
+ (GenerateNormalAttrSetter):
+ (GenerateConstructorCallback):
+ (GenerateNamedConstructorCallback):
+ (GenerateSingleBatchedAttribute):
+ (GenerateImplementation):
+ (GenerateToV8Converters):
+ (JSValueToNative):
+ (NativeToJSValue):
+ * bindings/scripts/test/CPP/WebDOMTestSerializedScriptValueInterface.cpp:
+ (WebDOMTestSerializedScriptValueInterface::setValue):
+ (WebDOMTestSerializedScriptValueInterface::readonlyValue):
+ (WebDOMTestSerializedScriptValueInterface::cachedValue):
+ (WebDOMTestSerializedScriptValueInterface::setCachedValue):
+ (WebDOMTestSerializedScriptValueInterface::cachedReadonlyValue):
+ * bindings/scripts/test/CPP/WebDOMTestSerializedScriptValueInterface.h:
+ (WebDOMTestSerializedScriptValueInterface):
+ * bindings/scripts/test/GObject/WebKitDOMTestSerializedScriptValueInterface.cpp:
+ (webkit_dom_test_serialized_script_value_interface_set_value):
+ (webkit_dom_test_serialized_script_value_interface_get_readonly_value):
+ (webkit_dom_test_serialized_script_value_interface_get_cached_value):
+ (webkit_dom_test_serialized_script_value_interface_set_cached_value):
+ (webkit_dom_test_serialized_script_value_interface_get_cached_readonly_value):
+ (webkit_dom_test_serialized_script_value_interface_get_property):
+ (webkit_dom_test_serialized_script_value_interface_class_init):
+ * bindings/scripts/test/GObject/WebKitDOMTestSerializedScriptValueInterface.h:
+ * bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp:
+ ():
+ (WebCore::jsTestSerializedScriptValueInterfaceReadonlyValue):
+ (WebCore):
+ (WebCore::jsTestSerializedScriptValueInterfaceCachedValue):
+ (WebCore::jsTestSerializedScriptValueInterfaceCachedReadonlyValue):
+ (WebCore::JSTestSerializedScriptValueInterface::put):
+ (WebCore::setJSTestSerializedScriptValueInterfaceValue):
+ (WebCore::setJSTestSerializedScriptValueInterfaceCachedValue):
+ * bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.h:
+ (JSTestSerializedScriptValueInterface):
+ (JSTestSerializedScriptValueInterfacePrototype):
+ (WebCore):
+ * bindings/scripts/test/ObjC/DOMTestSerializedScriptValueInterface.h:
+ * bindings/scripts/test/ObjC/DOMTestSerializedScriptValueInterface.mm:
+ (-[DOMTestSerializedScriptValueInterface setValue:]):
+ (-[DOMTestSerializedScriptValueInterface readonlyValue]):
+ (-[DOMTestSerializedScriptValueInterface cachedValue]):
+ (-[DOMTestSerializedScriptValueInterface setCachedValue:]):
+ (-[DOMTestSerializedScriptValueInterface cachedReadonlyValue]):
+ * bindings/scripts/test/TestSerializedScriptValueInterface.idl:
+ * bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:
+ (WebCore::TestSerializedScriptValueInterfaceInternal::valueAttrGetter):
+ (TestSerializedScriptValueInterfaceInternal):
+ (WebCore::TestSerializedScriptValueInterfaceInternal::valueAttrSetter):
+ (WebCore::TestSerializedScriptValueInterfaceInternal::readonlyValueAttrGetter):
+ (WebCore::TestSerializedScriptValueInterfaceInternal::cachedValueAttrGetter):
+ (WebCore::TestSerializedScriptValueInterfaceInternal::cachedValueAttrSetter):
+ (WebCore::TestSerializedScriptValueInterfaceInternal::cachedReadonlyValueAttrGetter):
+ ():
+ (WebCore::V8TestSerializedScriptValueInterface::constructorCallback):
+ (WebCore::ConfigureV8TestSerializedScriptValueInterfaceTemplate):
+ (WebCore::V8TestSerializedScriptValueInterface::wrapSlow):
+
+2012-02-02 Adam Barth <abarth@webkit.org>
+
+ BMW Car Configuration Page doesn't work (Only manifests in Chromium)
+ https://bugs.webkit.org/show_bug.cgi?id=77312
+
+ Reviewed by Dimitri Glazkov.
+
+ When we autogenerated the event factory, we missed the check for
+ whether touch events were enabled at runtime:
+ http://trac.webkit.org/changeset/97933/trunk/Source/WebCore/dom/Document.cpp
+
+ This patch adds the check back.
+
+ Unfortunately, there isn't a testing frame work for
+ RuntimeEnabledFeatures. The main difficulty is that these static bools
+ need to be set when WebKit is initialized and can't be changed (which
+ is why they're not part of WebCore::Settings). To test them properly,
+ we'd need a testing framework that booted up WebKit for each test.
+
+ We could test this particular change (which doesn't need the bool to be
+ constant through the runtime of WebKit), but that would create a
+ sandtrap for future patches who might thing that this testing framework
+ can really be used to test RuntimeEnabledFeatures.
+
+ The net result is that Chromium is going to end up living with the
+ non-default codepath for these settings being untested, which will lead
+ to regressions like this one. If we ened up with a bunch of these
+ regressions, we'll likely end up with a testing framework similar to
+ Chromium's browsers_tests, which create a fresh subprocess for each
+ test.
+
+ * dom/EventFactory.in:
+ * dom/make_event_factory.pl:
+ (defaultItemFactory):
+ (generateImplementation):
+
+2012-02-02 Justin Novosad <junov@chromium.org>
+
+ [Chromium] Use SkCanvas::flush in skia port
+ https://bugs.webkit.org/show_bug.cgi?id=77463
+
+ Reviewed by Stephen White.
+
+ Code cleanup in skia port. Using SkCanvas::flush to remove
+ unnecessary dependency on GrContext. Removed unnecessary inclusions
+ and forward declarations of GrContext in several source files.
+
+ * platform/graphics/chromium/Canvas2DLayerChromium.cpp:
+ (WebCore::Canvas2DLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/LayerRendererChromium.cpp:
+ * platform/graphics/chromium/LayerRendererChromium.h:
+ * platform/graphics/chromium/cc/CCLayerTreeHost.h:
+ * platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:
+ * platform/graphics/chromium/cc/CCProxy.h:
+ * platform/graphics/skia/PlatformContextSkia.cpp:
+
+2012-02-02 Chris Marrin <cmarrin@apple.com>
+
+ Turn on CSS Filters on Windows
+ https://bugs.webkit.org/show_bug.cgi?id=76667
+
+ Turning on CSS_FILTERS flag for Windows and fixed a couple of resultant build errors
+
+ Reviewed by Adele Peterson.
+
+ * WebCore.vcproj/copyForwardingHeaders.cmd:
+ * platform/graphics/ca/win/PlatformCALayerWin.cpp:
+ (PlatformCALayer::setFilters):
+ (PlatformCALayer::filtersCanBeComposited):
+
+2012-02-02 Jon Lee <jonlee@apple.com>
+
+ Clear shown notifications when context is no longer active
+ https://bugs.webkit.org/show_bug.cgi?id=77363
+ <rdar://problem/10568907>
+
+ Reviewed by Darin Adler.
+
+ * notifications/NotificationPresenter.h: Add new virtual function to clear notifications
+ associated with a given execution context. By default the notifications are left alone, as
+ before. Individual implementations can override to allow notifications to clear them.
+
+ * notifications/NotificationCenter.cpp:
+ (WebCore::NotificationCenter::disconnectFrame): When disconnecting the page from the frame, we
+ call clearNotifications().
+ * page/Frame.cpp:
+ (WebCore::Frame::pageDestroyed): When the page is destroyed, tell the DOM window to reset notifications.
+
+2012-02-02 Anders Carlsson <andersca@apple.com>
+
+ The overhang area layer should have a linen background
+ https://bugs.webkit.org/show_bug.cgi?id=77670
+ <rdar://problem/10797727>
+
+ Reviewed by Andreas Kling.
+
+ * page/scrolling/mac/ScrollingCoordinatorMac.mm:
+ (WebCore::ScrollingCoordinator::scrollByOnScrollingThread):
+ Add an #ifdef so that scroll position clamping can be disabled. This will be
+ removed once rubber-banding works properly.
+
+ * platform/ScrollbarTheme.h:
+ (WebCore::ScrollbarTheme::setUpOverhangAreasLayerContents):
+ Add new empty function.
+
+ * platform/mac/ScrollbarThemeMac.h:
+ (ScrollbarThemeMac):
+ * platform/mac/ScrollbarThemeMac.mm:
+ (WebCore::linenBackgroundColor):
+ Helper function for getting the CGColorRef that represents the linen background color.
+
+ (WebCore::ScrollbarThemeMac::setUpOverhangAreasLayerContents):
+ Set the linen background color as the overhang areas layer background color.
+
+ * rendering/RenderLayerCompositor.cpp:
+ (WebCore::RenderLayerCompositor::updateOverflowControlsLayers):
+ Call ScrollbarTheme::setUpOverhangAreasLayerContents.
+
+2012-02-02 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r106566.
+ http://trac.webkit.org/changeset/106566
+ https://bugs.webkit.org/show_bug.cgi?id=77673
+
+ Broke the Windows build (Requested by jessieberlin on
+ #webkit).
+
+ * WebCore.vcproj/copyForwardingHeaders.cmd:
+ * platform/graphics/ca/win/PlatformCALayerWin.cpp:
+
+2012-02-02 Tommy Widenflycht <tommyw@google.com>
+
+ [chromium] MediaStream API: Adding the embedding code for MediaStreamCenter
+ https://bugs.webkit.org/show_bug.cgi?id=73130
+
+ Reviewed by Darin Fisher.
+
+ Tests for the Media Stream API will be provided by the bug 56587, pending enough landed code.
+
+ * mediastream/UserMediaClient.h:
+ (UserMediaClient):
+ * mediastream/UserMediaRequest.cpp:
+ (WebCore::UserMediaRequest::didCompleteQuery):
+ * mediastream/UserMediaRequest.h:
+ (UserMediaRequest):
+ (WebCore::UserMediaRequest::audio):
+ (WebCore::UserMediaRequest::video):
+ (WebCore::UserMediaRequest::cameraPreferenceUser):
+ (WebCore::UserMediaRequest::cameraPreferenceEnvironment):
+ * platform/mediastream/MediaStreamCenter.cpp:
+ (WebCore):
+ (WebCore::MediaStreamCenter::queryMediaStreamSources):
+ * platform/mediastream/MediaStreamCenter.h:
+ (WebCore):
+ (MediaStreamSourcesQueryClient):
+ (MediaStreamCenter):
+
+2012-02-02 Raymond Toy <rtoy@google.com>
+
+ Constant values to set "distanceModel" are undefined
+ https://bugs.webkit.org/show_bug.cgi?id=74273
+
+ Reviewed by Kenneth Russell.
+
+ Tests: webaudio/distance-exponential.html
+ webaudio/distance-inverse.html
+ webaudio/distance-linear.html
+
+ * webaudio/AudioPannerNode.h: Define enum for the new constants
+ for the distance models.
+ * webaudio/AudioPannerNode.idl: Define matching constants for the
+ distance models.
+
+2012-02-02 Raymond Toy <rtoy@google.com>
+
+ Illegal panner model values should throw an exception
+ https://bugs.webkit.org/show_bug.cgi?id=77235
+
+ Reviewed by Kenneth Russell.
+
+ Modified existing panner-set-model test to catch exceptions.
+ Debug build should not crash anymore.
+
+ * webaudio/AudioPannerNode.cpp:
+ (WebCore::AudioPannerNode::setPanningModel): Throw exception for
+ invalid model values.
+ * webaudio/AudioPannerNode.h:
+ (AudioPannerNode): Update declaration
+ * webaudio/AudioPannerNode.idl: Setting panner model can throw
+ exception.
+
+2012-02-02 Kentaro Hara <haraken@chromium.org>
+
+ Rename [ConvertUndefinedOrNullToNullString] to
+ [TreatNullAs=EmptyString, TreatUndefinedAs=EmptyString]
+ https://bugs.webkit.org/show_bug.cgi?id=77611
+
+ Reviewed by Adam Barth.
+
+ This patch renames [ConvertUndefinedOrNullToNullString] to
+ [TreatNullAs=EmptyString, TreatUndefinedAs=EmptyString], according to the spec
+ (http://dev.w3.org/2006/webapi/WebIDL/#TreatNullAs,
+ http://dev.w3.org/2006/webapi/WebIDL/#TreatUndefinedAs).
+
+ No tests. No changes in behavior.
+
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (JSValueToNative):
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (GetNativeTypeFromSignature):
+ * dom/DOMImplementation.idl:
+ * dom/Document.idl:
+ * fileapi/Blob.idl:
+ * fileapi/DirectoryEntry.idl:
+ * fileapi/DirectoryEntrySync.idl:
+ * fileapi/Entry.idl:
+ * fileapi/EntrySync.idl:
+ * fileapi/WebKitBlobBuilder.idl:
+ * html/HTMLButtonElement.idl:
+ * html/HTMLCanvasElement.idl:
+ * html/HTMLFieldSetElement.idl:
+ * html/HTMLInputElement.idl:
+ * html/HTMLKeygenElement.idl:
+ * html/HTMLObjectElement.idl:
+ * html/HTMLOutputElement.idl:
+ * html/HTMLSelectElement.idl:
+ * html/HTMLTextAreaElement.idl:
+ * page/Console.idl:
+ * page/DOMWindow.idl:
+
+ * bindings/scripts/test/TestObj.idl: No changes in run-bindings-tests results.
+
+2012-02-02 Anders Carlsson <andersca@apple.com>
+
+ NPAPI will not send mouse up events when mouse is outside plugin area
+ https://bugs.webkit.org/show_bug.cgi?id=77657
+ <rdar://problem/10160674>
+
+ Reviewed by Andreas Kling.
+
+ Export EventHandler::setCapturingMouseEventsNode.
+
+ * WebCore.exp.in:
+
+2012-02-02 Antti Koivisto <antti@apple.com>
+
+ Move remaining implementation from CSSStyleDeclaration to subclasses
+ https://bugs.webkit.org/show_bug.cgi?id=77621
+
+ Reviewed by Andreas Kling.
+
+ This leaves CSSStyleDeclaration a near-pure CSSOM interface.
+
+ * bindings/js/JSCSSStyleDeclarationCustom.cpp:
+ (WebCore::isCSSPropertyName):
+ (WebCore::JSCSSStyleDeclaration::putDelegate):
+ * css/CSSComputedStyleDeclaration.cpp:
+ (WebCore::CSSComputedStyleDeclaration::parentRule):
+ (WebCore):
+ * css/CSSComputedStyleDeclaration.h:
+ (CSSComputedStyleDeclaration):
+ * css/CSSMutableStyleDeclaration.cpp:
+ (WebCore::CSSMutableStyleDeclaration::CSSMutableStyleDeclaration):
+ (WebCore::CSSMutableStyleDeclaration::contextStyleSheet):
+ (WebCore):
+ (WebCore::CSSMutableStyleDeclaration::parentRule):
+ * css/CSSMutableStyleDeclaration.h:
+ (WebCore::CSSMutableStyleDeclaration::isInlineStyleDeclaration):
+ (WebCore::CSSMutableStyleDeclaration::parentRuleInternal):
+ (WebCore::CSSMutableStyleDeclaration::clearParentRule):
+ (CSSMutableStyleDeclaration):
+ (WebCore::CSSMutableStyleDeclaration::parentElement):
+ (WebCore::CSSMutableStyleDeclaration::clearParentElement):
+ (WebCore::CSSMutableStyleDeclaration::parentStyleSheet):
+ * css/CSSParser.cpp:
+ (WebCore::parseColorValue):
+ (WebCore::parseSimpleLengthValue):
+ (WebCore::CSSParser::parseValue):
+ (WebCore::CSSParser::parseDeclaration):
+ * css/CSSStyleDeclaration.cpp:
+ (WebCore):
+ * css/CSSStyleDeclaration.h:
+ (CSSStyleDeclaration):
+ (WebCore::CSSStyleDeclaration::parentStyleSheet):
+ (WebCore::CSSStyleDeclaration::CSSStyleDeclaration):
+ * css/CSSStyleRule.cpp:
+ (WebCore::CSSStyleRule::setSelectorText):
+
+ Eliminate unnecessary call to CSSMutableStyleDeclaration::parentStyleSheet()
+
+ * css/CSSStyleRule.h:
+ (WebCore::CSSStyleRule::setDeclaration):
+ * css/CSSStyleSelector.cpp:
+ (WebCore::isInsideRegionRule):
+ (WebCore::CSSStyleSelector::applyDeclaration):
+ * css/WebKitCSSKeyframeRule.cpp:
+ (WebCore::WebKitCSSKeyframeRule::setDeclaration):
+ * page/PageSerializer.cpp:
+ (WebCore::PageSerializer::serializeFrame):
+ (WebCore::PageSerializer::serializeCSSStyleSheet):
+ (WebCore::PageSerializer::retrieveResourcesForCSSRule):
+ (WebCore::PageSerializer::retrieveResourcesForCSSDeclaration):
+ * page/PageSerializer.h:
+
+ Eliminate unnecessary call to CSSMutableStyleDeclaration::parentStyleSheet()
+
+ (WebCore):
+ (PageSerializer):
+
+2012-02-02 Chris Marrin <cmarrin@apple.com>
+
+ Turn on CSS Filters on Windows
+ https://bugs.webkit.org/show_bug.cgi?id=76667
+
+ Turning on CSS_FILTERS flag for Windows and fixed a couple of resultant build errors
+
+ Reviewed by Adele Peterson.
+
+ * WebCore.vcproj/copyForwardingHeaders.cmd:
+ * platform/graphics/ca/win/PlatformCALayerWin.cpp:
+ (PlatformCALayer::setFilters):
+ (PlatformCALayer::filtersCanBeComposited):
+
+2012-02-02 Anders Carlsson <andersca@apple.com>
+
+ Move ScrollingThread to its own file
+ https://bugs.webkit.org/show_bug.cgi?id=77652
+
+ Reviewed by Sam Weinig.
+
+ This just shuffles some classes around and makes ScrollingCoordinator call into ScrollingThread.
+ Hopefully we'll be able to make ScrollingThread use the new WebCore::RunLoop class eventually.
+
+ * WebCore.xcodeproj/project.pbxproj:
+ * page/scrolling/ScrollingCoordinator.cpp:
+ (WebCore::ScrollingCoordinator::handleWheelEvent):
+ * page/scrolling/ScrollingCoordinator.h:
+ (ScrollingCoordinator):
+ * page/scrolling/ScrollingThread.cpp: Added.
+ (WebCore):
+ (WebCore::ScrollingThread::ScrollingThread):
+ (WebCore::ScrollingThread::isCurrentThread):
+ (WebCore::ScrollingThread::dispatch):
+ (WebCore::ScrollingThread::shared):
+ (WebCore::ScrollingThread::createThreadIfNeeded):
+ (WebCore::ScrollingThread::threadCallback):
+ (WebCore::ScrollingThread::threadBody):
+ (WebCore::ScrollingThread::dispatchFunctionsFromScrollingThread):
+ * page/scrolling/ScrollingThread.h: Added.
+ (WebCore):
+ (ScrollingThread):
+ * page/scrolling/mac/ScrollingCoordinatorMac.mm:
+ (WebCore::ScrollingCoordinator::scrollByOnScrollingThread):
+ (WebCore::ScrollingCoordinator::updateMainFrameScrollLayerPositionOnScrollingThread):
+ * page/scrolling/mac/ScrollingThreadMac.mm: Added.
+ (WebCore):
+ (WebCore::ScrollingThread::initializeRunLoop):
+ (WebCore::ScrollingThread::wakeUpRunLoop):
+ (WebCore::ScrollingThread::threadRunLoopSourceCallback):
+
+2012-02-02 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r106551.
+ http://trac.webkit.org/changeset/106551
+ https://bugs.webkit.org/show_bug.cgi?id=77648
+
+ Breaking mac and gtk tests due to font differences. (Requested
+ by _pdr_ on #webkit).
+
+ * platform/graphics/SVGGlyph.cpp:
+ (WebCore::charactersWithArabicForm):
+ * svg/SVGFontData.cpp:
+ (WebCore::SVGFontData::applySVGGlyphSelection):
+ * svg/SVGFontData.h:
+ (SVGFontData):
+
+2012-02-02 Alexander Pavlov <apavlov@chromium.org>
+
+ Web Inspector: [REGRESSION] Slow continuous DOM traversal with Up/Down keys
+ https://bugs.webkit.org/show_bug.cgi?id=77643
+
+ Reviewed by Pavel Feldman.
+
+ Test: inspector/styles/updates-during-dom-traversal.html
+
+ * inspector/front-end/StylesSidebarPane.js:
+ (WebInspector.StylesSidebarPane.prototype._executeRebuildUpdate):
+
+2012-02-02 Claudio Saavedra <csaavedra@igalia.com> and Martin Robinson <mrobinson@igalia.com>
+
+ [GTK] WebKitWebView won't work in a GtkOffscreenWindow
+ https://bugs.webkit.org/show_bug.cgi?id=76911
+
+ Reviewed by Philippe Normand.
+
+ * platform/gtk/GtkUtilities.cpp:
+ (WebCore::widgetIsOnscreenToplevelWindow): Added this helper.
+ * platform/gtk/GtkUtilities.h:
+ (WebCore): Added helper declaration.
+
+2012-02-02 Allan Sandfeld Jensen <allan.jensen@nokia.com>
+
+ Update active and hover state on touch release.
+ https://bugs.webkit.org/show_bug.cgi?id=77620
+
+ Reviewed by Kenneth Rohde Christiansen.
+
+ * rendering/HitTestRequest.h:
+ (WebCore::HitTestRequest::move):
+ (WebCore::HitTestRequest::release):
+ (WebCore::HitTestRequest::touchEvent):
+ (WebCore::HitTestRequest::mouseEvent):
+ (WebCore::HitTestRequest::touchMove):
+ (WebCore::HitTestRequest::touchRelease):
+ Rename the enum values in HitTestRequest to be mouse/touch
+ agnostic, and add value for recognizing touch events.
+ * rendering/RenderFrameSet.cpp:
+ (WebCore::RenderFrameSet::nodeAtPoint): Update for HitTestRequest rename.
+ * rendering/RenderLayer.cpp:
+ (WebCore::RenderLayer::hitTest): ditto
+ (WebCore::RenderLayer::updateHoverActiveState): Reset hoverstate on touch release.
+ * page/EventHandler.cpp:
+ (WebCore::EventHandler::updateSelectionForMouseDrag): Update for HitTestRequest rename.
+ (WebCore::EventHandler::handleMouseMoveEvent): ditto
+ (WebCore::EventHandler::handleMouseReleaseEvent): ditto
+ (WebCore::EventHandler::hoverTimerFired): ditto
+ (WebCore::EventHandler::dragSourceEndedAt): ditto
+ (WebCore::EventHandler::handleTouchEvent): Hittest touch release to reset
+ active and hover states and add touch enum to all touch hittests.
+
+2012-02-02 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: enable editing of selected rows on single click in elements panel.
+ https://bugs.webkit.org/show_bug.cgi?id=77627
+
+ Reviewed by Vsevolod Vlasov.
+
+ * inspector/front-end/ElementsTreeOutline.js:
+ (WebInspector.ElementsTreeElement.prototype.onattach):
+ (WebInspector.ElementsTreeElement.prototype.onselect):
+ (WebInspector.ElementsTreeElement.prototype._mouseDown):
+ * inspector/front-end/treeoutline.js:
+ (TreeElement.prototype.selectOnMouseDown):
+ (TreeElement.prototype.select):
+
+2012-02-02 Philip Rogers <pdr@google.com>
+
+ Fix mirroring with SVG fonts
+ https://bugs.webkit.org/show_bug.cgi?id=77067
+
+ Reviewed by Nikolas Zimmermann.
+
+ SVG fonts were incorrectly handling mirrored characters in bidi text.
+ In this change I added the function createStringWithMirroredCharacters
+ which handles mirroring the characters when selecting glyphs for SVG
+ fonts. I also made a small cosmetic change in the function
+ charactersWithArabicForm, changing the bool parameter "mirror" to "rtl"
+ which better reflects what it actually does.
+
+ Several new tests were added to test mirroring with SVG fonts in the
+ presence of Arabic forms and non-BMP characters.
+
+ Tests: svg/custom/glyph-selection-arabic-forms.svg
+ svg/custom/glyph-selection-bidi-mirror.svg
+ svg/custom/glyph-selection-non-bmp.svg
+
+ * platform/graphics/SVGGlyph.cpp:
+ (WebCore::charactersWithArabicForm):
+ * svg/SVGFontData.cpp:
+ (WebCore::SVGFontData::applySVGGlyphSelection):
+ (WebCore::SVGFontData::createStringWithMirroredCharacters):
+ * svg/SVGFontData.h:
+ (SVGFontData):
+
+2012-02-02 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: add experiment for single click styles editing.
+ https://bugs.webkit.org/show_bug.cgi?id=77624
+
+ Reviewed by Vsevolod Vlasov.
+
+ * inspector/front-end/Settings.js:
+ (WebInspector.ExperimentsSettings):
+ * inspector/front-end/StylesSidebarPane.js:
+ (WebInspector.StylePropertiesSection):
+ (WebInspector.StylePropertyTreeElement.prototype.onattach):
+ (WebInspector.StylePropertyTreeElement.prototype._mouseDown):
+ (WebInspector.StylePropertyTreeElement.prototype._resetMouseDownElement):
+ (WebInspector.StylePropertyTreeElement.prototype):
+ (WebInspector.StylePropertyTreeElement.prototype.selectElement.context):
+
+2012-02-02 Kenneth Rohde Christiansen <kenneth@webkit.org>
+
+ Make the tap highlighting work for all test cases
+ https://bugs.webkit.org/show_bug.cgi?id=77626
+
+ Reviewed by Simon Hausmann.
+
+ Clean up of the current code to make it more generic. Now uses
+ addFocusRingRects for finding the areas to highlight.
+
+ Tested by current manual tests.
+
+ * page/GestureTapHighlighter.cpp:
+ (WebCore::GestureTapHighlighter::pathForNodeHighlight):
+
+2012-02-02 Mario Sanchez Prada <msanchez@igalia.com>
+
+ [Gtk] atk_text_get_text_at_offset() sometimes fails to provide the correct line
+ https://bugs.webkit.org/show_bug.cgi?id=72382
+
+ Reviewed by Martin Robinson.
+
+ Do not add unnecesary blanks at the end of a line of text.
+
+ * accessibility/gtk/WebKitAccessibleInterfaceText.cpp:
+ (textForRenderer): Do not just append a '\n' at the end of a line
+ if the linebreak for that line was already considered.
+
+2012-02-02 Raul Hudea <rhudea@adobe.com>
+
+ Regions should ignore the saved currentRenderFlowThread during repainting
+ because if there are imbricated flow threads, it might end using the wrong one.
+
+ [CSSRegions] Assert failure in RenderView::computeRectForRepaint
+ https://bugs.webkit.org/show_bug.cgi?id=77430
+
+ Reviewed by David Hyatt.
+
+ Test: fast/regions/imbricated-flow-threads-crash.html
+
+ * rendering/RenderFlowThread.cpp:
+ (CurrentRenderFlowThreadDisabler):
+ (WebCore::CurrentRenderFlowThreadDisabler::CurrentRenderFlowThreadDisabler):
+ (WebCore::CurrentRenderFlowThreadDisabler::~CurrentRenderFlowThreadDisabler):
+ (WebCore):
+ (WebCore::RenderFlowThread::repaintRectangleInRegions):
+
+2012-02-02 Kinuko Yasuda <kinuko@chromium.org>
+
+ Cleanup: Move chrome-specific filesystem type handling code (for FileSystem API) under chromium directory (re-landing r105395)
+ https://bugs.webkit.org/show_bug.cgi?id=76551
+
+ Reviewed by David Levin.
+
+ Moved the implementation of crackFileSystemURL() and toURL() from
+ WebCore/fileapi/DOMFileSystemBase into WebCore/platform/AsyncFileSystem
+ so that each platform can extend/implement their behavior if necessary.
+
+ No new tests as it has no functional changes.
+
+ * fileapi/DOMFileSystemBase.cpp: Moved crackFileSystemURL() to AsyncFileSystem.
+ * fileapi/DOMFileSystemBase.h:
+ (DOMFileSystemBase):
+ * fileapi/EntryBase.cpp: Moved toURL() to AsyncFileSystem.
+ (WebCore::EntryBase::toURL):
+ * page/DOMWindow.cpp: Made corresponding callsite changes.
+ (WebCore::DOMWindow::webkitRequestFileSystem):
+ (WebCore::DOMWindow::webkitResolveLocalFileSystemURL):
+ * page/DOMWindow.h:
+ * platform/AsyncFileSystem.cpp:
+ (WebCore::AsyncFileSystem::isValidType): Added.
+ * platform/AsyncFileSystem.h:
+ (AsyncFileSystem):
+ * workers/WorkerContext.cpp: Made corresponding callsite changes.
+ (WebCore::WorkerContext::webkitRequestFileSystem):
+ (WebCore::WorkerContext::webkitRequestFileSystemSync):
+ (WebCore::WorkerContext::webkitResolveLocalFileSystemURL):
+ (WebCore::WorkerContext::webkitResolveLocalFileSystemSyncURL):
+ * workers/WorkerContext.h:
+
+2012-02-02 Yury Semikhatsky <yurys@chromium.org>
+
+ Web Inspector: pause on uncaugh exceptions state is not properly restored
+ https://bugs.webkit.org/show_bug.cgi?id=77558
+
+ 'Pause on exceptions' state is now stored in InspectorState object so that it is
+ properly restored on inspected process change.
+
+ Reviewed by Vsevolod Vlasov.
+
+ * inspector/InspectorDebuggerAgent.cpp:
+ (DebuggerAgentState):
+ (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent):
+ (WebCore::InspectorDebuggerAgent::disable):
+ (WebCore::InspectorDebuggerAgent::restore):
+ (WebCore::InspectorDebuggerAgent::setPauseOnExceptions):
+ (WebCore):
+ (WebCore::InspectorDebuggerAgent::setPauseOnExceptionsImpl):
+ * inspector/InspectorDebuggerAgent.h:
+ (InspectorDebuggerAgent):
+
+2012-02-02 Kentaro Hara <haraken@chromium.org>
+
+ The third argument of addEventListener/removeEventListener of PeerConnection should be optional
+ https://bugs.webkit.org/show_bug.cgi?id=77606
+
+ Reviewed by Adam Barth.
+
+ This patch corrects a typo in PeerConnection.idl, i.e. [optional] => [Optional].
+
+ Test: fast/mediastream/peerconnection-eventlistener-optional-argument.html
+
+ * mediastream/PeerConnection.idl:
+
+2012-02-02 Rakesh KN <rakesh.kn@motorola.com>
+
+ hidden attribute on <input type=file /> suppresses the file selection dialog
+ https://bugs.webkit.org/show_bug.cgi?id=58208
+
+ Reviewed by Ryosuke Niwa.
+
+ We want to allow opening file dialog on hidden element when click() is called
+ on the element. The behaviour is similar to Firefox and IE.
+
+ Added manual test as the file dialog opens only on user gesture.
+
+ * html/FileInputType.cpp:
+ (WebCore::FileInputType::handleDOMActivateEvent):
+ Removed renderer check as input can be hidden.
+
+2012-02-01 Philippe Normand <pnormand@igalia.com>
+
+ [GStreamer] FFTFrame implementation
+ https://bugs.webkit.org/show_bug.cgi?id=73545
+
+ Reviewed by Chris Rogers.
+
+ FFTFrame implementation based on GStreamer's FFT processing
+ library.
+
+ No new tests, existing WebAudio tests cover this.
+
+ * GNUmakefile.am:
+ * GNUmakefile.list.am:
+ * platform/audio/FFTFrame.h:
+ * platform/audio/FFTFrameStub.cpp:
+ * platform/audio/gstreamer/FFTFrameGStreamer.cpp: Added.
+ (WebCore::FFTFrame::FFTFrame):
+ (WebCore::FFTFrame::initialize):
+ (WebCore::FFTFrame::cleanup):
+ (WebCore::FFTFrame::~FFTFrame):
+ (WebCore::FFTFrame::multiply):
+ (WebCore::FFTFrame::doFFT):
+ (WebCore::FFTFrame::doInverseFFT):
+ (WebCore::FFTFrame::realData):
+ (WebCore::FFTFrame::imagData):
+
+2012-02-02 Kentaro Hara <haraken@chromium.org>
+
+ Rename [CheckFrameSecurity] and [SVGCheckSecurity] to [CheckAccessToNode]
+ https://bugs.webkit.org/show_bug.cgi?id=77601
+
+ Reviewed by Adam Barth.
+
+ [CheckFrameSecurity] and [SVGCheckSecurity] have the same meaning; i.e. generate code
+ to check allowAccessToNode() for a given attribute or method. This patch renames them
+ to [CheckAccessToNode].
+
+ No tests. No changes in behavior.
+
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (GenerateImplementation):
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (GenerateNormalAttrGetter):
+ (GenerateFunctionCallback):
+ * html/HTMLEmbedElement.idl:
+ * html/HTMLFrameElement.idl:
+ * html/HTMLIFrameElement.idl:
+ * html/HTMLObjectElement.idl:
+
+ * bindings/scripts/test/TestObj.idl:
+ * bindings/scripts/test/V8/V8TestObj.cpp: Updated the test results.
+ (WebCore):
+
+2012-02-01 Kentaro Hara <haraken@chromium.org>
+
+ Rename [V8DisallowShadowing] to [V8Unforgeable]
+ https://bugs.webkit.org/show_bug.cgi?id=77599
+
+ Reviewed by Adam Barth.
+
+ This patch renames [V8DisallowShadowing] to [V8Unforgeable], following the Web IDL
+ spec (http://dev.w3.org/2006/webapi/WebIDL/#Unforgeable).
+
+ No tests. No change in behavior.
+
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (GenerateNormalAttrGetter):
+ (GenerateSingleBatchedAttribute):
+ (GenerateImplementation):
+ * page/DOMWindow.idl:
+ * page/Location.idl:
+
+2012-02-01 Kentaro Hara <haraken@chromium.org>
+
+ Rename [ConvertNullToNullString] to [TreatNullAs=EmptyString]
+ https://bugs.webkit.org/show_bug.cgi?id=77602
+
+ Reviewed by Adam Barth.
+
+ This patch renames [ConvertNullToNullString] to [TreatNullAs=EmptyString],
+ according to the spec (http://dev.w3.org/2006/webapi/WebIDL/#TreatNullAs).
+
+ No tests. No changes in behavior.
+
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (JSValueToNative):
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (GetNativeTypeFromSignature):
+
+ * css/CSSCharsetRule.idl:
+ * css/CSSPageRule.idl:
+ * css/CSSRule.idl:
+ * css/CSSStyleDeclaration.idl:
+ * css/CSSStyleRule.idl:
+ * css/CSSValue.idl:
+ * css/MediaList.idl:
+ * css/WebKitCSSKeyframesRule.idl:
+ * dom/Attr.idl:
+ * dom/CharacterData.idl:
+ * dom/DOMImplementation.idl:
+ * dom/Document.idl:
+ * dom/Element.idl:
+ * dom/NamedNodeMap.idl:
+ * dom/Node.idl:
+ * dom/ProcessingInstruction.idl:
+ * dom/ShadowRoot.idl:
+ * html/HTMLAnchorElement.idl:
+ * html/HTMLButtonElement.idl:
+ * html/HTMLDocument.idl:
+ * html/HTMLElement.idl:
+ * html/HTMLFormElement.idl:
+ * html/HTMLFrameElement.idl:
+ * html/HTMLInputElement.idl:
+ * html/HTMLMediaElement.idl:
+ * html/HTMLOutputElement.idl:
+ * html/HTMLScriptElement.idl:
+ * html/HTMLSelectElement.idl:
+ * html/HTMLTextAreaElement.idl:
+ * html/HTMLTitleElement.idl:
+ * html/canvas/CanvasRenderingContext2D.idl:
+ * page/DOMWindow.idl:
+ * storage/StorageEvent.idl:
+ * svg/SVGAngle.idl:
+ * svg/SVGElement.idl:
+ * svg/SVGLength.idl:
+ * svg/SVGScriptElement.idl:
+
+ * bindings/scripts/test/TestObj.idl: No change in the run-bindings-tests results.
+
+2012-02-01 Kentaro Hara <haraken@chromium.org>
+
+ Remove [GenerateConstructor] from IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=77598
+
+ Reviewed by Adam Barth.
+
+ [GenerateConstructor] is used in some IDL files but it is not implemented
+ in code generators. I could not find any history about [GenerateConstructor]
+ in code generators. This patch removes them from IDL files.
+
+ Maybe we want to replace [GenerateConstructor] with [Constructor] eventually,
+ but currently no tests are written for their constructors (e.g. "new DOMTokenList()").
+ So simply removing them would make sense.
+
+ No new tests. No change in behavior.
+
+ * dom/DOMStringList.idl:
+ * html/DOMSettableTokenList.idl:
+ * html/DOMTokenList.idl:
+ * svg/SVGFEConvolveMatrixElement.idl:
+ * webaudio/AudioPannerNode.idl:
+
+2012-02-01 Eric Carlson <eric.carlson@apple.com>
+
+ Consider user's preferred language when choosing text tracks
+ https://bugs.webkit.org/show_bug.cgi?id=74121
+
+ Reviewed by Alexey Proskuryakov.
+
+ Tests: media/track/track-language-preference.html
+ media/track/track-prefer-captions.html
+
+ * html/HTMLMediaElement.cpp:
+ (WebCore::HTMLMediaElement::loadTimerFired): configureTextTracks -> configureNewTextTracks.
+ (WebCore::HTMLMediaElement::textTracksAreReady): Add more comments.
+ (WebCore::HTMLMediaElement::textTrackModeChanged): Ditto.
+ (WebCore::HTMLMediaElement::showingTrackWithSameKind): Minor restructuring.
+ (WebCore::HTMLMediaElement::userIsInterestedInThisTrackKind): Renamed from userIsInterestedInThisTrack,
+ don't consider user's language preference.
+ (WebCore::HTMLMediaElement::configureTextTrackGroup): New, configure all tracks in a group,
+ considering user's kind and language preferences.
+ (WebCore::HTMLMediaElement::configureNewTextTracks): New, configure all newly added tracks.
+ * html/HTMLMediaElement.h:
+ (WebCore::HTMLMediaElement::TrackGroup::TrackGroup):
+ (TrackGroup):
+
+ * platform/Language.cpp:
+ (WebCore::canonicalLanguageIdentifier): New, create a canonicalized version of a language string.
+ (WebCore::bestMatchingLanguage): New, return the language from the list that best matches the
+ specified language.
+ (WebCore::preferredLanguageFromList): New, return the language in the specified list that best
+ matches the user's language preference.
+ * platform/Language.h:
+
+ * testing/Internals.cpp:
+ (WebCore::Internals::setShouldDisplayTrackType): New, allow DRT to set the track type preference.
+ (WebCore::Internals::shouldDisplayTrackType): New, allow DRT to read the track type preference.
+ * testing/Internals.h:
+ * testing/Internals.idl:
+
+2012-02-01 Hayato Ito <hayato@chromium.org>
+
+ Change class hierarycy so that ShadowRoot can inherit DocumentFragment.
+ https://bugs.webkit.org/show_bug.cgi?id=76693
+
+ Reviewed by Darin Adler.
+
+ Make ShadowRoot inherit DocumentFragment so that it matches the class hierarchy of IDL in the spec.
+ TreeScope becomes a separated class, which is now inherited by Document and ShadowRoot using multiple-inheritance.
+ This patch is pre-requirement for coming IDL change.
+
+ No tests. No change in behavior.
+
+ * dom/Document.cpp:
+ (WebCore::Document::Document):
+ (WebCore::Document::~Document):
+ (WebCore::Document::buildAccessKeyMap):
+ (WebCore::Document::childrenChanged):
+ (WebCore::Document::attach):
+ (WebCore::Document::detach):
+ * dom/Document.h:
+ (Document):
+ * dom/DocumentFragment.cpp:
+ (WebCore::DocumentFragment::DocumentFragment):
+ * dom/DocumentFragment.h:
+ (DocumentFragment):
+ * dom/DocumentOrderedMap.cpp:
+ (WebCore::DocumentOrderedMap::get):
+ * dom/ShadowRoot.cpp:
+ (WebCore::ShadowRoot::ShadowRoot):
+ (WebCore::ShadowRoot::~ShadowRoot):
+ (WebCore::ShadowRoot::attach):
+ * dom/ShadowRoot.h:
+ * dom/TreeScope.cpp:
+ (WebCore::TreeScope::TreeScope):
+ (WebCore::TreeScope::~TreeScope):
+ (WebCore::TreeScope::setParentTreeScope):
+ (WebCore::TreeScope::getImageMap):
+ (WebCore::TreeScope::findAnchor):
+ * dom/TreeScope.h:
+ (WebCore):
+ (WebCore::TreeScope::rootNode):
+ (TreeScope):
+ * dom/TreeScopeAdopter.cpp:
+ (WebCore::TreeScopeAdopter::moveTreeToNewScope):
+ * page/DragController.cpp:
+ (WebCore::asFileInput):
+ * page/FocusController.cpp:
+ (WebCore::ownerOfTreeScope):
+ (WebCore::FocusController::nextFocusableNode):
+ (WebCore::FocusController::previousFocusableNode):
+
+2012-02-01 Benjamin Poulain <bpoulain@apple.com>
+
+ WorkerScriptController::evaluate() should not return anything
+ https://bugs.webkit.org/show_bug.cgi?id=77587
+
+ Reviewed by Adam Barth.
+
+ Remove the return value from WorkerScriptController as it is
+ (and should be) unused.
+
+ * bindings/js/WorkerScriptController.cpp:
+ (WebCore::WorkerScriptController::evaluate):
+ * bindings/js/WorkerScriptController.h:
+ (WorkerScriptController):
+ * bindings/v8/WorkerScriptController.cpp:
+ (WebCore::WorkerScriptController::evaluate):
+ * bindings/v8/WorkerScriptController.h:
+ (WorkerScriptController):
+
+2012-02-01 Kentaro Hara <haraken@chromium.org>
+
+ Remove [DelegatingPrototypeGetOwnPropertySlot], [HasCustomIndexGetter],
+ [HasIndexSetter], [JSConstructorTemplate] and [NonEmpty] from code generators
+ https://bugs.webkit.org/show_bug.cgi?id=77585
+
+ Reviewed by Adam Barth.
+
+ This patch removes [DelegatingPrototypeGetOwnPropertySlot], [HasCustomIndexGetter],
+ [HasIndexSetter], [JSConstructorTemplate] and [NonEmpty] from code generators,
+ since they are not used by no IDL files.
+
+ No tests. No change in behavior.
+
+ * bindings/scripts/CodeGenerator.pm:
+ (GetterExpression):
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (GenerateGetOwnPropertySlotBody):
+ (GenerateGetOwnPropertyDescriptorBody):
+ (GenerateHeader):
+ (GenerateImplementation):
+ (IsConstructable):
+ (IsConstructorTemplate):
+
+ * bindings/scripts/test/TestObj.idl:
+ * bindings/scripts/test/CPP/WebDOMTestObj.cpp: Updated the run-bindings-tests results.
+ * bindings/scripts/test/CPP/WebDOMTestObj.h: Ditto.
+ * bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: Ditto.
+ (webkit_dom_test_obj_set_property):
+ (webkit_dom_test_obj_get_property):
+ (webkit_dom_test_obj_class_init):
+ * bindings/scripts/test/GObject/WebKitDOMTestObj.h: Ditto.
+ * bindings/scripts/test/JS/JSTestObj.cpp: Ditto.
+ (WebCore):
+ * bindings/scripts/test/JS/JSTestObj.h: Ditto.
+ (WebCore):
+ * bindings/scripts/test/ObjC/DOMTestObj.h: Ditto.
+ * bindings/scripts/test/ObjC/DOMTestObj.mm: Ditto.
+ * bindings/scripts/test/V8/V8TestObj.cpp: Ditto.
+ (WebCore):
+
+2012-02-01 Shinya Kawanaka <shinyak@google.com>
+
+ Select attribute of HTMLContentElement should be able be changed dynamically.
+ https://bugs.webkit.org/show_bug.cgi?id=76261
+
+ Reviewed by Hajime Morita.
+
+ When select attribute is changed, the flag to recalc style is set.
+
+ Test: fast/dom/shadow/content-element-select-dynamic.html
+
+ * html/shadow/HTMLContentElement.cpp:
+ (WebCore::HTMLContentElement::parseMappedAttribute):
+ Sets recalc style when select is changed.
+ * html/shadow/HTMLContentElement.h:
+ (HTMLContentElement):
+
+2012-02-01 Kentaro Hara <haraken@chromium.org>
+
+ Remove [ImplementationUUID] and [InterfaceUUID] from html/canvas/*.idl
+ https://bugs.webkit.org/show_bug.cgi?id=77589
+
+ Reviewed by Adam Barth.
+
+ [ImplementationUUID] and [InterfaceUUID] are used in html/canvas/*.idl,
+ but they are not implemented in code generators and thus have no meaning.
+ This patch removes them.
+
+ No tests. No change in behavior.
+
+ * html/canvas/CanvasGradient.idl:
+ * html/canvas/CanvasPattern.idl: The file is now empty though.
+ * html/canvas/CanvasRenderingContext.idl:
+ * html/canvas/CanvasRenderingContext2D.idl:
+ * html/canvas/WebGLRenderingContext.idl:
+
+2012-02-01 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r106408.
+ http://trac.webkit.org/changeset/106408
+ https://bugs.webkit.org/show_bug.cgi?id=77592
+
+ crashes in chromium mac release tests (Requested by japhet on
+ #webkit).
+
+ * platform/graphics/Region.cpp:
+ * platform/graphics/Region.h:
+ (Region):
+ (Shape):
+
+2012-02-01 No'am Rosenthal <noam.rosenthal@nokia.com>
+
+ [Texmap] Use glScissors for clipping in TextureMapperGL when possible
+ https://bugs.webkit.org/show_bug.cgi?id=77575
+
+ Reviewed by Martin Robinson.
+
+ Maintain a clipping stack, that helps us use stencils in conjunction with scissors.
+ We apply scissors when the clip region is rectalinear, and stencil when it's not.
+
+ No behavior changes so no new tests.
+
+ * platform/graphics/opengl/TextureMapperGL.cpp:
+ (SharedGLData):
+ (WebCore::TextureMapperGLData::SharedGLData::SharedGLData):
+ (WebCore::TextureMapperGL::drawTexture):
+ (WebCore::TextureMapperGL::bindSurface):
+ (WebCore):
+ (WebCore::scissorClip):
+ (WebCore::TextureMapperGL::beginScissorClip):
+ (WebCore::TextureMapperGL::endScissorClip):
+ (WebCore::TextureMapperGL::beginClip):
+ (WebCore::TextureMapperGL::endClip):
+ * platform/graphics/opengl/TextureMapperGL.h:
+ (TextureMapperGL):
+
+2012-02-01 Anders Carlsson <andersca@apple.com>
+
+ Move the scrolling coordinator to page/scrolling
+ https://bugs.webkit.org/show_bug.cgi?id=77590
+
+ Reviewed by Dan Bernstein.
+
+ Put scrolling related files in page/scrolling to avoid cluttering the page directory.
+
+ * WebCore.xcodeproj/project.pbxproj:
+ * page/scrolling/ScrollingCoordinator.cpp: Renamed from Source/WebCore/page/ScrollingCoordinator.cpp.
+ * page/scrolling/ScrollingCoordinator.h: Renamed from Source/WebCore/page/ScrollingCoordinator.h.
+ * page/scrolling/mac/ScrollingCoordinatorMac.mm: Renamed from Source/WebCore/page/mac/ScrollingCoordinatorMac.mm.
+
+2012-02-01 Kentaro Hara <haraken@chromium.org>
+
+ Remove [ObjCPrefix], [V8ConstructorTemplate], [allowAccessToNode],
+ [v8implname] and [v8referenceattr] from code generators
+ https://bugs.webkit.org/show_bug.cgi?id=77588
+
+ Reviewed by Adam Barth.
+
+ This patch removes [ObjCPrefix], [V8ConstructorTemplate], [allowAccessToNode],
+ [v8implname] and [v8referenceattr] from code generators, since they are not used
+ by any IDL files.
+
+ No tests. No changes in behavior.
+
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (GenerateImplementation):
+ * bindings/scripts/CodeGeneratorObjC.pm:
+ (GenerateHeader):
+ (GenerateImplementation):
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (IsConstructable):
+ (IsConstructorTemplate):
+ (GenerateNormalAttrGetter):
+ (GenerateFunctionCallString):
+
+2012-02-01 Pablo Flouret <pablof@motorola.com>
+
+ Support targetOrigin = "/" in postMessage for sending messages to same origin as source document.
+ https://bugs.webkit.org/show_bug.cgi?id=77580
+
+ Reviewed by Adam Barth.
+
+ No new tests. Modified http/tests/security/postMessage/target-origin.html
+ to test this case as well.
+
+ * page/DOMWindow.cpp:
+ (WebCore::DOMWindow::postMessage):
+
+2012-02-01 Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
+
+ Avoid creating NamedNodeMap unnecessarily
+ https://bugs.webkit.org/show_bug.cgi?id=77574
+
+ Reviewed by Ryosuke Niwa.
+
+ The method Element::attributes() was being used for multiple things in our
+ codebase: (1) as the getter for NamedNodeMap exposed to DOM, (2) as a way to other WebCore
+ code get the "attribute storage" (currently inside NamedNodeMap), and (3) as a way to
+ get the attribute storage creating one if necessary.
+
+ This commit separate the jobs in different functions:
+
+ 1) attributes() keeps being the DOM getter, and loses its boolean parameter.
+
+ 2) updatedAttributes() updates the invalid attributes and returns the attribute
+ storage. If we don't have one, return 0.
+
+ 3) ensureUpdatedAttributes() updates the invalid attributes and forces the
+ creation of attribute storage to return.
+
+ There is also another way to get to the attribute storage currently, via
+ attributeMap(), which doesn't update the attributes for possible changes in Style
+ or SVG attributes.
+
+ Note that the new functions are not available in Node class, so C++ code manipulating
+ attributes should cast to Element.
+
+ This separation also made easier to spot and fix some places where we do not
+ need to create the attribute storage if it doesn't exist.
+
+ No new tests, this commit shouldn't change the behavior of existing code.
+
+ * css/SelectorChecker.cpp:
+ (WebCore::SelectorChecker::checkOneSelector):
+ * dom/DatasetDOMStringMap.cpp:
+ (WebCore::DatasetDOMStringMap::getNames):
+ (WebCore::DatasetDOMStringMap::item):
+ (WebCore::DatasetDOMStringMap::contains):
+ * dom/Document.cpp:
+ (WebCore::Document::importNode):
+ * dom/Element.cpp:
+ (WebCore::Element::setAttribute):
+ (WebCore::Element::hasAttributes):
+ (WebCore::Element::setAttributeNode):
+ (WebCore::Element::setAttributeNodeNS):
+ (WebCore::Element::removeAttributeNode):
+ (WebCore::Element::getAttributeNode):
+ (WebCore::Element::getAttributeNodeNS):
+ (WebCore::Element::hasAttribute):
+ (WebCore::Element::hasAttributeNS):
+ (WebCore::Element::normalizeAttributes):
+ * dom/Element.h:
+ (Element):
+ (WebCore::Element::attributes):
+ (WebCore::Element::ensureAttributeData):
+ (WebCore::Element::ensureUpdatedAttributes):
+ (WebCore::Element::updatedAttributes):
+ (WebCore::Element::setAttributesFromElement):
+ (WebCore::Element::ensureAttributeMap): Made const to be reused by ensureUpdatedAttributes().
+ (WebCore::Element::updateInvalidAttributes):
+ (WebCore):
+ * dom/NamedNodeMap.cpp:
+ (WebCore::NamedNodeMap::mapsEquivalent): Having no attributes is equivalent to
+ not having an attribute storage because the attribute storage is lazily created.
+ * dom/Node.cpp:
+ (WebCore::Node::isEqualNode): Do not force the creation of attribute storage to
+ compare two nodes.
+ (WebCore::Node::isDefaultNamespace): Use updatedAttributes(). Since we iterate
+ using length, it's OK if the attribute storage is empty.
+ (WebCore::Node::lookupNamespaceURI): Ditto.
+ (WebCore::Node::lookupNamespacePrefix): Ditto.
+ (WebCore::Node::compareDocumentPosition): Ditto.
+ * editing/ApplyStyleCommand.cpp:
+ (WebCore::hasNoAttributeOrOnlyStyleAttribute):
+ (WebCore::isEmptyFontTag):
+ * editing/CompositeEditCommand.cpp:
+ (WebCore::CompositeEditCommand::isRemovableBlock): Use isElementNode() explicitly
+ to identify non-Element nodes, then use hasAttributes() if is Element.
+ * editing/InsertParagraphSeparatorCommand.cpp:
+ (WebCore::highestVisuallyEquivalentDivBelowRoot):
+ * editing/MarkupAccumulator.cpp:
+ (WebCore::MarkupAccumulator::appendElement): Do not create the attribute storage
+ unnecessarily.
+ * editing/htmlediting.cpp:
+ (WebCore::areIdenticalElements): Do not create the attribute storage
+ unnecessarily. Use mapsEquivalent() for comparing the attributes.
+ * editing/markup.cpp:
+ (WebCore::completeURLs): Do not create the attribute storage unnecessarily.
+ (WebCore::StyledMarkupAccumulator::appendElement): Ditto.
+ (WebCore::isPlainTextMarkup): hasAttributes() will avoid creating the attribute
+ storage unnecessarily.
+ * html/HTMLEmbedElement.cpp:
+ (WebCore::HTMLEmbedElement::parametersForPlugin):
+ * html/HTMLObjectElement.cpp:
+ (WebCore::HTMLObjectElement::parametersForPlugin):
+ * html/HTMLParamElement.cpp:
+ (WebCore::HTMLParamElement::isURLAttribute): Do not create the attribute storage
+ unnecessarily.
+ * html/parser/HTMLConstructionSite.cpp:
+ (WebCore::HTMLConstructionSite::mergeAttributesFromTokenIntoElement): Use
+ ensureUpdatedAttributes() since we will add new attributes.
+ (WebCore):
+ * inspector/InspectorCSSAgent.cpp:
+ (WebCore::InspectorCSSAgent::buildArrayForAttributeStyles):
+ * inspector/InspectorDOMAgent.cpp:
+ (WebCore::InspectorDOMAgent::setAttributesAsText):
+ (WebCore::InspectorDOMAgent::performSearch):
+ (WebCore::InspectorDOMAgent::buildArrayForElementAttributes):
+ * page/PageSerializer.cpp:
+ (WebCore::isCharsetSpecifyingNode): Do not assume attributeMap will exist.
+ * svg/properties/SVGAnimatedPropertySynchronizer.h: Use ensureUpdatedAttributes()
+ since we will add new attributes.
+ * xml/XPathFunctions.cpp:
+ (WebCore::XPath::FunLang::evaluate): Do not create the attribute storage
+ unnecessarily.
+ * xml/XPathNodeSet.cpp:
+ (WebCore::XPath::NodeSet::traversalSort):
+ * xml/XPathStep.cpp:
+ (WebCore::XPath::Step::nodesInAxis): Use isElementNode() instead of comparing
+ nodeType() manually. Do not create the attribute storage unnecessarily.
+ * xml/parser/XMLDocumentParserLibxml2.cpp:
+ (WebCore::XMLDocumentParser::XMLDocumentParser): Do not create the attribute
+ storage unnecessarily.
+ * xml/parser/XMLDocumentParserQt.cpp:
+ (WebCore::XMLDocumentParser::XMLDocumentParser): Ditto.
+ * xml/parser/XMLTreeBuilder.cpp:
+ (WebCore::XMLTreeBuilder::XMLTreeBuilder): Ditto.
+
+2012-02-01 Adam Barth <abarth@webkit.org>
+
+ contentDispositionType misparses the Content-Disposition header in some obscure corner cases
+ https://bugs.webkit.org/show_bug.cgi?id=77577
+
+ Reviewed by Eric Seidel.
+
+ The contentDispositionType extracts the disposition-type from the
+ Content-Disposition header. According to RFC 6266 (and previous RFCs),
+ the disposition-type must be an RFC 2616 token. Rather than enforce
+ this general rule, we had special-cased some examples (including
+ name=foo and filename=bar). This patch generalizes our check to
+ properly validate that the disposition-type is an RFC 2616 token.
+
+ In conjunction with some other work in the Chromium network stack, this
+ causes Chromium to pass the following tests:
+
+ http://greenbytes.de/tech/tc2231/#inlonlyquoted
+ http://greenbytes.de/tech/tc2231/#attonlyquoted
+
+ Without this patch, these test cases neither trigger a navigation nor a
+ download in Chromium. This patch does not appear to cause any visible
+ change in Safari. (Safari passes these tests both before and after
+ this patch.)
+
+ * platform/network/HTTPParsers.cpp:
+ (WebCore::isRFC2616Token):
+ (WebCore::contentDispositionType):
+ - This patch also adds a comment to
+ filenameFromHTTPContentDisposition, which explains some of the
+ was this function incorrectly implements the requirements in
+ RFC 6266. Resolving these issues is a subject for a future
+ patch.
+ * platform/network/HTTPParsers.h:
+
+2012-02-01 Dan Bernstein <mitz@apple.com>
+
+ WebCore part of <rdar://problem/10442663> Paginated display does not respect page-break-{before,after}
+ https://bugs.webkit.org/show_bug.cgi?id=77505
+
+ Reviewed by Darin Adler.
+
+ * page/Page.h:
+ (WebCore::Page::Pagination::Pagination): Added initializer for the behavesLikeColumns member
+ variable.
+ (WebCore::Page::Pagination::operator==): Added comparison of behavesLikeColumns values.
+ (Pagination): Added behavesLikeColumns member variable. When set to false (the default),
+ paginated display respects the page-break-{before,after} properties rather than the
+ column-break-{before,after} ones.
+ * rendering/ColumnInfo.h:
+ (WebCore::ColumnInfo::ColumnInfo): Added initializer for the m_paginationUnit member.
+ (WebCore::ColumnInfo::paginationUnit): Added this getter.
+ (WebCore::ColumnInfo::setPaginationUnit): Added this setter.
+ (ColumnInfo): Added m_paginationUnit member. It defaults to Column.
+ * rendering/LayoutState.h:
+ (WebCore::LayoutState::isPaginatingColumns): Changed to check the pagination unit.
+ * rendering/RenderBlock.cpp:
+ (WebCore::RenderBlock::layoutBlock): Added code to set the pagination unit in the ColumnInfo.
+ (WebCore::RenderBlock::paginationUnit): Added. The base class implementation returns Column.
+ * rendering/RenderBlock.h:
+ * rendering/RenderView.cpp:
+ (WebCore::RenderView::paginationUnit): Added this override that returns Page, unless
+ this is the RenderView for the main frame and pagination is set to behave like columns.
+ * rendering/RenderView.h:
+
+2012-02-01 Florin Malita <fmalita@google.com>
+
+ Backgrounds in HTML inside foreignObject don't draw
+ https://bugs.webkit.org/show_bug.cgi?id=23111
+
+ Reviewed by Eric Seidel.
+
+ Test: svg/foreignObject/body-background.svg
+
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::paintBackground):
+ Tweak the <body> background inhibiting logic to allow drawing when the element is embedded in FOs.
+
+
+2012-02-01 Gustavo Lima Chaves <glima@profusion.mobi>
+
+ Make one able to set the local storage (tracker) database dir's path
+ https://bugs.webkit.org/show_bug.cgi?id=77006
+
+ Reviewed by Darin Adler.
+
+ There are no behavior changes with the diff, so no need for new tests.
+
+ * storage/StorageTracker.cpp:
+ (WebCore::StorageTracker::setDatabaseDirectoryPath):
+ (WebCore):
+ (WebCore::StorageTracker::databaseDirectoryPath):
+ * storage/StorageTracker.h:
+ (StorageTracker):
+
+2012-02-01 Anders Carlsson <andersca@apple.com>
+
+ Fix Window build.
+
+ * platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.cpp:
+ (WebCore::LayerClient::platformCALayerDidCreateTiles):
+ * platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
+ (WebCore::MediaPlayerPrivateQuickTimeVisualContext::LayerClient::platformCALayerDidCreateTiles):
+
+2012-02-01 John Yani <vanuan@gmail.com>
+
+ GetMIMEDescription should return const char *
+ https://bugs.webkit.org/show_bug.cgi?id=77297
+
+ Reviewed by Alexey Proskuryakov.
+
+ No new tests. No change in behaviour.
+
+ * plugins/blackberry/PluginPackageBlackBerry.cpp:
+ (WebCore::PluginPackage::fetchInfo):
+ * plugins/efl/PluginPackageEfl.cpp:
+ (WebCore):
+ (WebCore::PluginPackage::fetchInfo):
+ * plugins/npapi.h:
+ * plugins/npfunctions.h:
+ * plugins/qt/PluginPackageQt.cpp:
+ (WebCore::PluginPackage::fetchInfo):
+
+2012-02-01 Timothy Hatcher <timothy@apple.com>
+
+ Consolidate duplicate "willHide" functions in DetailedHeapshotView.js to fix
+ a syntax error in JSC and make the Inspector open again in Release builds.
+
+ https://webkit.org/b/77424
+
+ Reviewed by Brian Weinstein.
+
+ * inspector/front-end/DetailedHeapshotView.js:
+ (WebInspector.DetailedHeapshotView.prototype.willHide): Consolidated.
+ (WebInspector.DetailedHeapshotView.prototype.willHide): Removed.
+
+2012-02-01 Justin Novosad <junov@chromium.org>
+
+ [Chromium] Enable deferred canvas rendering in the skia port
+ https://bugs.webkit.org/show_bug.cgi?id=76732
+
+ Reviewed by Stephen White.
+
+ No new tests: covered by existing canvas layout tests
+
+ Adding a new setting to enable deferred 2d canvas rendering.
+ Added support for deferred 2d canvas rendering in ImageBufferSkia
+ and Canvas2DLayerChromium, mostly plumbing. Deffered rendering
+ implementation is provided by skia (class SkDeferredCanvas).
+
+ * html/HTMLCanvasElement.cpp:
+ (WebCore::HTMLCanvasElement::shouldDefer):
+ (WebCore):
+ (WebCore::HTMLCanvasElement::createImageBuffer):
+ * html/HTMLCanvasElement.h:
+ (HTMLCanvasElement):
+ * page/Settings.cpp:
+ (WebCore::Settings::Settings):
+ (WebCore::Settings::setAccelerated2dCanvasEnabled):
+ (WebCore):
+ (WebCore::Settings::setDeferred2dCanvasEnabled):
+ * page/Settings.h:
+ (Settings):
+ (WebCore::Settings::deferred2dCanvasEnabled):
+ * platform/graphics/ImageBuffer.h:
+ (WebCore::ImageBuffer::create):
+ (ImageBuffer):
+ * platform/graphics/cairo/ImageBufferCairo.cpp:
+ (WebCore::ImageBuffer::ImageBuffer):
+ * platform/graphics/cg/ImageBufferCG.cpp:
+ (WebCore::ImageBuffer::ImageBuffer):
+ * platform/graphics/chromium/Canvas2DLayerChromium.cpp:
+ (WebCore::Canvas2DLayerChromium::Canvas2DLayerChromium):
+ (WebCore):
+ (WebCore::Canvas2DLayerChromium::setCanvas):
+ (WebCore::Canvas2DLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/Canvas2DLayerChromium.h:
+ (Canvas2DLayerChromium):
+ * platform/graphics/qt/ImageBufferQt.cpp:
+ (WebCore::ImageBuffer::ImageBuffer):
+ * platform/graphics/skia/ImageBufferSkia.cpp:
+ (AcceleratedDeviceContext):
+ (WebCore::AcceleratedDeviceContext::AcceleratedDeviceContext):
+ (WebCore::AcceleratedDeviceContext::prepareForDraw):
+ (WebCore::AcceleratedDeviceContext::flush):
+ (WebCore):
+ (WebCore::createAcceleratedCanvas):
+ (WebCore::ImageBuffer::ImageBuffer):
+ * platform/graphics/wince/ImageBufferWinCE.cpp:
+ (WebCore::ImageBuffer::ImageBuffer):
+ * platform/graphics/wx/ImageBufferWx.cpp:
+ (WebCore::ImageBuffer::ImageBuffer):
+
+2012-02-01 Ryosuke Niwa <rniwa@webkit.org>
+
+ Gcc build fix after r106482.
+
+ * platform/graphics/ca/GraphicsLayerCA.h:
+ (GraphicsLayerCA):
+
+2012-02-01 Beth Dakin <bdakin@apple.com>
+
+ https://bugs.webkit.org/show_bug.cgi?id=77383
+ Add a different didFirstVisuallNonEmptyLayout heuristic to experiment with
+ -and corresponding-
+ <rdar://problem/10709560>
+
+ Reviewed by Sam Weinig.
+
+ The goal is to re-vamp didFirstVisuallyNonEmptyLayout to be more accurate.
+ This patch adds a new heuristic called didNewFirstVisuallNonEmptyLayout and
+ leaves the old one for the time being. That is temporary.
+
+ The heuristic for didNewFirstVisuallNonEmptyLayout is to count relevant
+ painted RenderObjects on Page.
+ * page/Page.cpp:
+ (WebCore::Page::Page):
+ (WebCore::Page::setPaintedObjectsCounterThreshold):
+ (WebCore::Page::addRelevantRepaintedObject):
+ * page/Page.h:
+ (WebCore):
+ (Page):
+ (WebCore::Page::startCountingRepaintedObjects):
+ * WebCore.exp.in:
+
+ Start counting relevant painted RenderObjects on the page once the first
+ layout is complete.
+ * page/FrameView.cpp:
+ (WebCore::FrameView::performPostLayoutTasks):
+
+ Machinery for firing didNewFirstVisuallNonEmptyLayout.
+ * loader/EmptyClients.h:
+ (WebCore::EmptyFrameLoaderClient::dispatchDidNewFirstVisuallyNonEmptyLayout):
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::didNewFirstVisuallyNonEmptyLayout):
+ (WebCore):
+ * loader/FrameLoader.h:
+ (FrameLoader):
+ * loader/FrameLoaderClient.h:
+ (WebCore::FrameLoaderClient::dispatchDidNewFirstVisuallyNonEmptyLayout):
+
+ These RenderObjects are the ones that this api currently consider to be
+ relevant. If their repaint rects intersect with the viewRect, then they are
+ added to the relevant objects set on the Page.
+ * rendering/InlineBox.cpp:
+ (WebCore::InlineBox::paint):
+ * rendering/InlineTextBox.cpp:
+ (WebCore::InlineTextBox::paint):
+ * rendering/RenderEmbeddedObject.cpp:
+ (WebCore::RenderEmbeddedObject::paintReplaced):
+ * rendering/RenderHTMLCanvas.cpp:
+ (WebCore::RenderHTMLCanvas::paintReplaced):
+ * rendering/RenderImage.cpp:
+ (WebCore::RenderImage::paintReplaced):
+ * rendering/RenderRegion.cpp:
+ (WebCore::RenderRegion::paintReplaced):
+ * rendering/RenderReplaced.cpp:
+ (WebCore::RenderReplaced::paint):
+ * rendering/RenderVideo.cpp:
+ (WebCore::RenderVideo::paintReplaced):
+ * rendering/svg/RenderSVGRoot.cpp:
+ (WebCore::RenderSVGRoot::paintReplaced):
+
+2012-02-01 Alexis Menard <alexis.menard@openbossa.org>
+
+ CSSStyleDeclaration.getPropertyPriority() fails for CSS shorthand properties with 'important' priority
+ https://bugs.webkit.org/show_bug.cgi?id=49058
+
+ Reviewed by Andreas Kling.
+
+ CSSMutableStyleDeclaration::getPropertyPriority was not handling shorthands properly. Shorthands are
+ not part of the property list of the style so we need to query the longhands which are the one added
+ in the list. Only if the longhands have equal priority the shorthand priority is known. I also renamed
+ getPropertyPriority (not the CSSOM exposed method) to something more consistent with WebKit naming guidelines.
+
+ Test: fast/css/shorthand-priority.html
+
+ * css/CSSMutableStyleDeclaration.cpp:
+ (WebCore::CSSMutableStyleDeclaration::propertyIsImportant):
+ (WebCore::CSSMutableStyleDeclaration::addParsedProperty):
+ (WebCore::CSSMutableStyleDeclaration::getPropertyPriority):
+ * css/CSSMutableStyleDeclaration.h:
+ (CSSMutableStyleDeclaration):
+ * editing/EditingStyle.cpp:
+ (WebCore::EditingStyle::extractAndRemoveTextDirection):
+ (WebCore::EditingStyle::collapseTextDecorationProperties):
+ (WebCore::EditingStyle::conflictsWithInlineStyleOfElement):
+ (WebCore::setTextDecorationProperty):
+ * editing/RemoveCSSPropertyCommand.cpp:
+ (WebCore::RemoveCSSPropertyCommand::doApply):
+
+2012-02-01 Ryosuke Niwa <rniwa@webkit.org>
+
+ Crash in EventHandler::updateDragAndDrop
+ https://bugs.webkit.org/show_bug.cgi?id=77569
+
+ Reviewed by Alexey Proskuryakov.
+
+ Test: fast/events/remove-target-with-shadow-in-drag.html
+
+ * page/EventHandler.cpp:
+ (WebCore::EventHandler::updateDragAndDrop):
+
+2012-02-01 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r106382.
+ http://trac.webkit.org/changeset/106382
+ https://bugs.webkit.org/show_bug.cgi?id=77571
+
+ Causing chromium crashes in PNGImageDecoder (Requested by
+ japhet on #webkit).
+
+ * bindings/v8/V8Proxy.cpp:
+ (WebCore::V8Proxy::evaluate):
+ * page/Console.cpp:
+ (WebCore::Console::time):
+ (WebCore::Console::timeEnd):
+ * platform/chromium/PlatformSupport.h:
+ * platform/chromium/TraceEvent.h:
+ (internal):
+ (ScopeTracer):
+ (WebCore::internal::ScopeTracer::ScopeTracer):
+ (WebCore::internal::ScopeTracer::~ScopeTracer):
+
+2012-02-01 Anders Carlsson <andersca@apple.com>
+
+ Reviewed by Darin Adler.
+
+ Simplify the code that creates a new tile layer by getting a reference to the RetainPtr<WebTileLayer>&
+ slot in the hash map and assign directly into it.
+
+ * platform/graphics/ca/mac/TileCache.mm:
+ (WebCore::TileCache::revalidateTiles):
+
+2012-02-01 Anders Carlsson <andersca@apple.com>
+
+ Tile cache doesn't have an upper limit
+ https://bugs.webkit.org/show_bug.cgi?id=77564
+ <rdar://problem/10710744>
+
+ Reviewed by Darin Adler.
+
+ Cache enough tiles to cover 3x the visible height and 2x the visible width of the page,
+ and drop tiles that are outside that area.
+
+ * platform/graphics/ca/GraphicsLayerCA.cpp:
+ (WebCore::GraphicsLayerCA::platformCALayerDidCreateTiles):
+ Call GraphicsLayerClient::notifySyncRequired here, which will schedule a layer flush and ensure that
+ the page layout is up to date before the new tiles are painted.
+
+ * platform/graphics/ca/PlatformCALayerClient.h:
+ Add platformCALayerDidCreateTiles member function.
+
+ * platform/graphics/ca/mac/TileCache.h:
+ Update for new/removed member functions and member variables.
+
+ * platform/graphics/ca/mac/TileCache.mm:
+ (WebCore::TileCache::TileCache):
+ Initialize the tile revalidation timer.
+
+ (WebCore::TileCache::tileCacheLayerBoundsChanged):
+ If we don't have any tiles at all right now, revalidate the tiles immediately. Otherwise,
+ schedule the revalidation timer.
+
+ (WebCore::TileCache::setNeedsDisplayInRect):
+ Return early if we have no tiles.
+
+ (WebCore::TileCache::visibleRectChanged):
+ Schedule tile revalidation.
+
+ (WebCore::TileCache::rectForTileIndex):
+ New helper function that returns the bounds rect of a tile given its tile index.
+
+ (WebCore::TileCache::getTileIndexRangeForRect):
+ Clamp the rect to the bounds of the tile cache layer.
+
+ (WebCore::TileCache::scheduleTileRevalidation):
+ Schedule the revalidation timer if it hasn't already been scheduled.
+
+ (WebCore::TileCache::tileRevalidationTimerFired):
+ Call revalidateTiles.
+
+ (WebCore::TileCache::revalidateTiles):
+ Compute the tile coverage rect and remove all tiles that are outside. Create new tiles for any
+ parts of the tile coverage rect that don't have tiles already.
+
+ (WebCore::TileCache::tileLayerAtIndex):
+ Remove invalid assertions.
+
+2012-02-01 Max Vujovic <mvujovic@adobe.com>
+
+ Add support for fixed and percent min-width on the table element for table-layout: auto to
+ match Firefox and Opera's behavior.
+
+ In FixedTableLayout.cpp, the computePreferredLogicalWidths method looks like it has
+ issues based on the comment: "FIXME: This entire calculation is incorrect for both
+ minwidth and maxwidth." (minwidth and maxwidth refer to the preferred widths, not the
+ min-width and max-width styles). I have not implemented min-width for FixedTableLayout
+ in this patch since it requires some more research around that comment.
+
+ min-width and max-width on the table element was discussed on the www-style list:
+ http://lists.w3.org/Archives/Public/www-style/2012Jan/0684.html
+
+ min-width is not implemented on <table> for table-layout: auto
+ https://bugs.webkit.org/show_bug.cgi?id=76553
+
+ Reviewed by Julien Chaffraix.
+
+ Test: fast/table/min-width.html
+
+ * rendering/AutoTableLayout.cpp:
+ (WebCore::AutoTableLayout::computePreferredLogicalWidths):
+
+ If the min or max preferred logical width is less than a fixed min width style, it is
+ set to the fixed min width style. Like a percent width style, a percent min-width style
+ does not affect the min or max preferred logical widths computed by the table layout
+ algorithm. RenderTable's computeLogicalWidth method handles percent min-width styles.
+
+ min-width for the table-layout: fixed case has been split out into this bug:
+ https://bugs.webkit.org/show_bug.cgi?id=76948
+
+ * rendering/RenderTable.cpp:
+ (WebCore::RenderTable::computeLogicalWidth):
+
+ If the RenderStyle's logical min width is defined and greater than the logical width
+ calculation, this method sets the logical width to the logical min width.
+
+ (WebCore::RenderTable::convertStyleWidthToComputedWidth):
+
+ This new method generalizes and factors out logic from RenderTable::computeLogicalWidth
+ that converted the width style to a computed value in the fixed and percent case.
+ RenderTable::computeLogicalWidth now calls this method to determine the computed values
+ for both the width style and the min-width style. In the future, it can also be used for
+ the max-width style.
+
+ Note that this method handles the special CSS table case, which requires borders and
+ paddings to be included in the computed width calculation. This applies to all width
+ styles, including width, min-width, and max-width. Before, this special case was handled
+ in RenderTable::computeLogicalWidth.
+
+ * rendering/RenderTable.h:
+
+2012-02-01 Brian Salomon <bsalomon@google.com>
+
+ [SKIA/CHROMIUM] Perform getImageData format conversions using Skia
+ https://bugs.webkit.org/show_bug.cgi?id=77553
+
+ Reviewed by Stephen White.
+
+ Many existing canvas tests exercise this functionality.
+
+ * platform/graphics/skia/ImageBufferSkia.cpp:
+ (WebCore::getImageData):
+
+2012-02-01 Nate Chapin <japhet@chromium.org>
+
+ preventDefault() in a mousedown in a subframe should not
+ prevent the scrollbar from handling mouse movements if the
+ cursor leaves the subframe.
+ https://bugs.webkit.org/show_bug.cgi?id=73097
+
+ Reviewed by Darin Adler.
+
+ Test: fast/events/scroll-div-with-prevent-default-in-subframe.html
+
+ * page/EventHandler.cpp:
+ (WebCore::EventHandler::handleMousePressEvent):
+
+2012-02-01 Mario Sanchez Prada <msanchez@igalia.com>
+
+ [GTK] editing/inserting/4960120-2.html flaky crash
+ https://bugs.webkit.org/show_bug.cgi?id=76815
+
+ Reviewed by Martin Robinson.
+
+ Check if the node for the first parent object not ignoring
+ accessibility is null before using it. This might happen with
+ certain kind of accessibility objects, such as the root one (the
+ scroller containing the webArea object as its only child).
+
+ * accessibility/gtk/WebKitAccessibleWrapperAtk.cpp:
+ (objectFocusedAndCaretOffsetUnignored): Add missing null check.
+
+2012-02-01 Antti Koivisto <antti@apple.com>
+
+ Make CSSMappedAttributeDeclaration have CSSMutableStyleDeclaration instead of being one
+ https://bugs.webkit.org/show_bug.cgi?id=77545
+
+ Reviewed by Andreas Kling.
+
+ This is the easiest path for eliminating the last remaining subclass of CSSMutableStyleDeclaration.
+
+ On negative side this increases memory use of CSSMappedAttributeDeclaration by one ptr and refcount
+ (it loses the vptr) in total.
+
+ This is not meant to be the end state, just an intermediate refactoring step. CSSMappedAttributeDeclaration
+ should clearly be renamed too but this patch doesn't do that. It might not exist in its current form
+ much longer.
+
+ * css/CSSMutableStyleDeclaration.cpp:
+ (WebCore::CSSMutableStyleDeclaration::setProperty):
+ (WebCore::CSSMutableStyleDeclaration::merge):
+ * css/CSSMutableStyleDeclaration.h:
+
+ Remove protected section. No subclasses remain.
+ Rename setPropertyInternal() to setProperty(). All public methods here are internal.
+
+ (CSSMutableStyleDeclaration):
+ * css/CSSParser.cpp:
+ (WebCore::CSSParser::parseMappedAttributeValue):
+ * dom/Attribute.cpp:
+ (WebCore::Attribute::clone):
+ * dom/Attribute.h:
+ (Attribute):
+ (WebCore::Attribute::decl):
+ (WebCore::Attribute::mappedAttributeDeclaration):
+ (WebCore::Attribute::setMappedAttributeDeclaration):
+ (WebCore::Attribute::Attribute):
+ * dom/CSSMappedAttributeDeclaration.cpp:
+ (WebCore::CSSMappedAttributeDeclaration::setMappedImageProperty):
+ (WebCore::CSSMappedAttributeDeclaration::setMappedProperty):
+ (WebCore::CSSMappedAttributeDeclaration::removeMappedProperty):
+ * dom/CSSMappedAttributeDeclaration.h:
+ (CSSMappedAttributeDeclaration):
+ (WebCore::CSSMappedAttributeDeclaration::declaration):
+ (WebCore::CSSMappedAttributeDeclaration::CSSMappedAttributeDeclaration):
+
+ Make CSSMutableStyleDeclaration a member instead of the base class.
+
+ * dom/StyledElement.cpp:
+ (WebCore::StyledElement::attributeChanged):
+ (WebCore::StyledElement::removeCSSProperty):
+ (WebCore::StyledElement::addCSSProperty):
+ (WebCore::StyledElement::addCSSImageProperty):
+ (WebCore::StyledElement::addCSSLength):
+ (WebCore::StyledElement::addCSSColor):
+ (WebCore::StyledElement::createMappedDecl):
+ * svg/SVGStyledElement.cpp:
+ (WebCore::SVGStyledElement::getPresentationAttribute):
+
+2012-02-01 Allan Sandfeld Jensen <allan.jensen@nokia.com>
+
+ [Qt] Set all PlatformTouchPoint values possible from a QTouch event.
+ https://bugs.webkit.org/show_bug.cgi?id=77442
+
+ Reviewed by Kenneth Rohde Christiansen.
+
+ * platform/qt/PlatformTouchPointQt.cpp:
+ (WebCore::PlatformTouchPoint::PlatformTouchPoint):
+
+2012-02-01 Peter Rybin <peter.rybin@gmail.com>
+
+ Web Inspector: CodeGeneratorInspector.py: move type builder code to dedicated InspectorTypeBuilder .h/.cpp
+ https://bugs.webkit.org/show_bug.cgi?id=77471
+
+ Reviewed by Yury Semikhatsky.
+
+ Code is moved physically to other file -- generator is changed accrodingly.
+
+ * inspector/CodeGeneratorInspector.py:
+ (String):
+ (provides):
+ (typename):
+ (Array):
+
+2012-02-01 Yury Semikhatsky <yurys@chromium.org>
+
+ Web Inspector: debugger reports wrong sources when paused in inline script on page reload
+ https://bugs.webkit.org/show_bug.cgi?id=77548
+
+ V8 returns treats each script source as ending with \n, now we take
+ this into account when reporting script line count to the inspector
+ front-end.
+
+ Reviewed by Vsevolod Vlasov.
+
+ Test: inspector/debugger/pause-in-inline-script.html
+
+ * bindings/js/ScriptDebugServer.cpp:
+ (WebCore::ScriptDebugServer::dispatchDidParseSource):
+ * bindings/v8/DebuggerScript.js:
+
+2012-02-01 Shinya Kawanaka <shinyak@google.com>
+
+ Content element should be able to be dynamically added/removed/replaced in a shadow tree.
+ https://bugs.webkit.org/show_bug.cgi?id=76611
+
+ Reviewed by Hajime Morita
+
+ When a content element is added/removed/replaced in a shadow tree, we have to recreate
+ the shadow tree to recalculate inclusions of content elements. Currently we didn't recalculate it
+ when content element is removed. (When added, it is recalculated.)
+ This patch enables us to recalcurate the shadow tree when content element is removed.
+
+ Test: fast/dom/shadow/content-element-move.html
+
+ * dom/Element.cpp:
+ (WebCore::Element::attach):
+ If a shadow root exists, attaches shadow tree before attaching child elements.
+ * dom/ShadowRoot.cpp:
+ Added a flag to recalculate shadow tree.
+ (WebCore::ShadowRoot::ShadowRoot):
+ (WebCore::ShadowRoot::recalcShadowTreeStyle):
+ Recalculates light children and shadow tree.
+ (WebCore::ShadowRoot::setNeedsReattachHostChildrenAndShadow):
+ (WebCore::ShadowRoot::reattachHostChildrenAndShadow):
+ Detaches shadow tree and host light children, and attaches them again.
+ * dom/ShadowRoot.h:
+ (WebCore::ShadowRoot::clearNeedsReattachHostChildrenAndShadow):
+ (WebCore::ShadowRoot::needsReattachHostChildrenAndShadow):
+ * html/shadow/HTMLContentElement.cpp:
+ (WebCore::HTMLContentElement::attach):
+ Does not need to detach included elements, because they are not attached in ContainerNode anymore.
+ (WebCore::HTMLContentElement::detach):
+ When a content element detached, reattaches a shadow tree.
+
+2012-02-01 Peter Beverloo <peter@chromium.org>
+
+ getIntersectionList causes transforms to be recalculated in SVG
+ https://bugs.webkit.org/show_bug.cgi?id=77179
+
+ Reviewed by Nikolas Zimmermann.
+
+ Introduce a local variable to store the element's local-to-parent
+ transformation matrix in, removing the need for the const_cast and
+ stopping us from modifying any matrices elsewhere.
+
+ Test: svg/custom/intersection-list-transforms.svg
+
+ * rendering/svg/RenderSVGModelObject.cpp:
+ (WebCore::getElementCTM):
+
+2012-02-01 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: [InspectorIndexedDB] Add tests for database names and database structure requests.
+ https://bugs.webkit.org/show_bug.cgi?id=77439
+
+ Reviewed by Pavel Feldman.
+
+ Tests: http/tests/inspector/indexeddb/database-names.html
+ http/tests/inspector/indexeddb/database-structure.html
+
+ * inspector/front-end/IndexedDBModel.js:
+ (WebInspector.IndexedDBModel.prototype.refreshDatabase):
+
+2012-01-31 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: Scripts navigator becomes empty after "show folders" settings change.
+ https://bugs.webkit.org/show_bug.cgi?id=77441
+
+ Reviewed by Pavel Feldman.
+
+ * inspector/front-end/ScriptsNavigator.js:
+ (WebInspector.ScriptsNavigator.prototype._reset):
+ * inspector/front-end/utilities.js:
+
+2012-02-01 Shawn Singh <shawnsingh@chromium.org>
+
+ Fix the semantics of passing contentsVisible flag to GraphicsLayers
+ https://bugs.webkit.org/show_bug.cgi?id=76975
+
+ Reviewed by Simon Fraser.
+
+ This patch is covered by existing tests, in particular
+ compositing/visibility/layer-visible-content.html; its
+ expectations are rebaselined.
+
+ * rendering/RenderLayerBacking.cpp:
+ (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
+
+2012-02-01 Philippe Normand <pnormand@igalia.com>
+
+ Unreviewed, another GTK build fix after r106446.
+
+ * platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
+ Restore webKitWebSrcGetProtocols as it was before r106446.
+
+2012-02-01 Philippe Normand <pnormand@igalia.com>
+
+ Unreviewed, GTK build fix after r106446.
+
+ * platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
+
+2012-02-01 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r106432.
+ http://trac.webkit.org/changeset/106432
+ https://bugs.webkit.org/show_bug.cgi?id=77529
+
+ it breaks tests (Requested by shinyak on #webkit).
+
+ * dom/Element.cpp:
+ (WebCore::Element::attach):
+ * dom/ShadowRoot.cpp:
+ (WebCore::ShadowRoot::recalcShadowTreeStyle):
+ * dom/ShadowRoot.h:
+ (ShadowRoot):
+ * html/shadow/HTMLContentElement.cpp:
+ (WebCore::HTMLContentElement::attach):
+ (WebCore::HTMLContentElement::detach):
+
+2012-02-01 Hayato Ito <hayato@chromium.org>
+
+ Remove unnecessary [OldStyleObjC] from ShadowRoot.idl.
+ https://bugs.webkit.org/show_bug.cgi?id=77516
+
+ Reviewed by Kentaro Hara.
+
+ No new tests. No change in behavior.
+
+ * dom/ShadowRoot.idl:
+
+2012-01-26 Philippe Normand <pnormand@igalia.com>
+
+ [GStreamer] 0.11 webkitwebsrc
+ https://bugs.webkit.org/show_bug.cgi?id=77086
+
+ Port the webkitwebsrc element to GStreamer 0.11 APIs.
+
+ Reviewed by Gustavo Noronha Silva.
+
+ No new tests, existing http media layout tests should cover this.
+
+ * platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
+ (webkit_web_src_class_init):
+
+2012-01-31 Hans Wennborg <hans@chromium.org>
+
+ Speech Input: Report speech element rect relative to window rather than frame
+ https://bugs.webkit.org/show_bug.cgi?id=76443
+
+ Reviewed by Darin Fisher.
+
+ When requesting speech input, report the speech element rect relative
+ to the window rather than the frame. The embedder will typically use
+ this position to show a bubble indicating that speech recognition is
+ in progress.
+
+ Test: fast/speech/bubble-position.html
+
+ * html/shadow/TextControlInnerElements.cpp:
+ (WebCore::InputFieldSpeechButtonElement::startSpeechInput):
+
+2012-01-31 Andreas Kling <awesomekling@apple.com>
+
+ Make elements that don't have attributes smaller.
+ <http://webkit.org/b/76876>
+
+ Reviewed by Sam Weinig and Antti Koivisto.
+
+ Move the inline style declaration from StyledElement to ElementAttributeData, since having
+ an inline style declaration also implies having a style attribute on the element.
+ This saves one CPU word per element that has no attributes.
+
+ This reduces memory consumption by 412 kB (on 64-bit) when viewing the full
+ HTML5 spec at <http://whatwg.org/c>.
+
+ This was rolled out once because of a performance regression which has been averted this
+ time around by adding an Element::ensureAttributeMap() so we can force creation of the
+ NamedNodeMap without also serializing the inline style for the "style" attribute.
+
+ * dom/Element.h:
+ (Element):
+ (WebCore::Element::ensureAttributeMap):
+ (WebCore):
+ * dom/ElementAttributeData.h:
+ (ElementAttributeData):
+ * dom/NamedNodeMap.cpp:
+ (WebCore::NamedNodeMap::ensureInlineStyleDecl):
+ (WebCore):
+ (WebCore::NamedNodeMap::destroyInlineStyleDecl):
+ * dom/NamedNodeMap.h:
+ (WebCore::NamedNodeMap::inlineStyleDecl):
+ (NamedNodeMap):
+ * dom/StyledElement.cpp:
+ (WebCore::StyledElement::addSubresourceAttributeURLs):
+ * dom/StyledElement.h:
+ (WebCore::StyledElement::inlineStyleDecl):
+ (WebCore::StyledElement::ensureInlineStyleDecl):
+ (StyledElement):
+ (WebCore::StyledElement::destroyInlineStyleDecl):
+
+2012-01-31 Hayato Ito <hayato@chromium.org>
+
+ Add APIs, getElementsByXXX family, to ShadowRoot IDL.
+ https://bugs.webkit.org/show_bug.cgi?id=77323
+
+ Reviewed by Dimitri Glazkov.
+
+ Add APIs (getElementById, getElemesntByTagName, getElementsByClassName and getElementsByTagNameNS)
+ to ShadowRoot IDL.
+
+ * dom/ShadowRoot.idl:
+
+2012-01-31 Shinya Kawanaka <shinyak@google.com>
+
+ Content element should be able to be dynamically added/removed/replaced in a shadow tree.
+ https://bugs.webkit.org/show_bug.cgi?id=76611
+
+ Reviewed by Hajime Morita.
+
+ When a content element is added/removed/replaced in a shadow tree, we have to recreate
+ the shadow tree to recalculate inclusions of content elements. Currently we didn't recalculate it
+ when content element is removed. (When added, it is recalculated.)
+ This patch enables us to recalcurate the shadow tree when content element is removed.
+
+ Test: fast/dom/shadow/content-element-move.html
+
+ * dom/Element.cpp:
+ (WebCore::Element::attach):
+ If a shadow root exists, attaches shadow tree before attaching child elements.
+ * dom/ShadowRoot.cpp:
+ (WebCore::ShadowRoot::recalcShadowTreeStyle):
+ Recalculates light children and shadow tree.
+ (WebCore::ShadowRoot::reattachHostChildrenAndShadow):
+ Detaches shadow tree and host light children, and attaches them again.
+ * dom/ShadowRoot.h:
+ Added a flag to recalculate shadow tree.
+ (WebCore::ShadowRoot::setNeedsShadowTreeStyleRecalc):
+ (WebCore::ShadowRoot::clearNeedsShadowTreeStyleRecalc):
+ (WebCore::ShadowRoot::needsShadowTreeStyleRecalc):
+ * html/shadow/HTMLContentElement.cpp:
+ (WebCore::HTMLContentElement::attach):
+ Does not need to detach included elements, because they are not attached in ContainerNode anymore.
+ (WebCore::HTMLContentElement::detach):
+ When a content element detached, reattaches a shadow tree.
+
+2012-01-31 Joe Thomas <joethomas@motorola.com>
+
+ https://bugs.webkit.org/show_bug.cgi?id=76801
+ Listboxes incorrectly display contents when cleared and then re-populated.
+
+ Whenever the number of items in the listbox is less than the size of listbox (number of visible items the listbox can accomodate),
+ we set the listbox scroll-offset to zero. The scroll-offset of the Scrollbar should also be set to 0 so that when the listbox is re-populated,
+ scrollbar position and the content inside the listbox are in sync.
+
+ Reviewed by Andreas Kling.
+
+ Tests: fast/forms/listbox-clear-restore.html
+
+ * rendering/RenderListBox.cpp:
+ (WebCore::RenderListBox::computeLogicalHeight): Setting the scroll-offset of the Scrollbar to 0 when scrollbar is not needed.
+
+2012-01-31 Gyuyoung Kim <gyuyoung.kim@samsung.com>
+
+ Unreviewed. Fix build break after r106373.
+
+ * CMakeLists.txt:
+
+2012-01-31 Adam Barth <abarth@webkit.org>
+
+ HTMLPreloadScanner should understand the <base> element
+ https://bugs.webkit.org/show_bug.cgi?id=77231
+
+ Reviewed by Eric Seidel.
+
+ Previously, the HTMLPreloadScanner would ignore the <base> element when
+ preloading resources. If there was a <base> tag, this could cause the
+ preload scanner to make a bunch of useless requests.
+
+ This patch teaches the preload scanner to use <base> tags to better
+ predict which URLs will be used by the document.
+
+ Tests: fast/preloader/first-base-tag-scanned-wins.html
+ fast/preloader/first-base-tag-wins.html
+ fast/preloader/understands-base-tag.html
+
+ * dom/Document.cpp:
+ (WebCore::Document::completeURL):
+ (WebCore):
+ * dom/Document.h:
+ (Document):
+ (WebCore::Document::baseElementURL):
+ * html/parser/HTMLPreloadScanner.cpp:
+ (WebCore::HTMLNames::PreloadTask::PreloadTask):
+ (WebCore::HTMLNames::PreloadTask::processAttributes):
+ (WebCore::HTMLNames::PreloadTask::preload):
+ (WebCore::HTMLNames::PreloadTask::baseElementHref):
+ (PreloadTask):
+ (WebCore::HTMLPreloadScanner::scan):
+ (WebCore::HTMLPreloadScanner::processToken):
+ (WebCore::HTMLPreloadScanner::updatePredictedBaseElementURL):
+ (WebCore):
+ * html/parser/HTMLPreloadScanner.h:
+ (HTMLPreloadScanner):
+
+2012-01-31 Raymond Liu <raymond.liu@intel.com>
+
+ Dynamic allocate AudioBus with required number of channels for AudioNodeInput
+ https://bugs.webkit.org/show_bug.cgi?id=76516
+
+ Reviewed by Kenneth Russell.
+
+ No new tests required.
+
+ * webaudio/AudioBasicProcessorNode.cpp:
+ (WebCore::AudioBasicProcessorNode::checkNumberOfChannelsForInput):
+ * webaudio/AudioChannelMerger.cpp:
+ (WebCore::AudioChannelMerger::checkNumberOfChannelsForInput):
+ * webaudio/AudioGainNode.cpp:
+ (WebCore::AudioGainNode::checkNumberOfChannelsForInput):
+ * webaudio/AudioNode.cpp:
+ (WebCore::AudioNode::checkNumberOfChannelsForInput):
+ * webaudio/AudioNode.h:
+ * webaudio/AudioNodeInput.cpp:
+ (WebCore::AudioNodeInput::AudioNodeInput):
+ (WebCore::AudioNodeInput::updateInternalBus):
+ (WebCore::AudioNodeInput::internalSummingBus):
+ * webaudio/AudioNodeInput.h:
+
+2012-01-31 Alexey Proskuryakov <ap@apple.com>
+
+ REGRESSION (WebKit2): event.keyCode is always zero when typing in Russian
+ https://bugs.webkit.org/show_bug.cgi?id=77473
+ <rdar://problem/10751357>
+
+ Reviewed by Darin Adler.
+
+ Test: platform/mac/fast/events/non-roman-key-code.html
+
+ * WebCore.exp.in:
+ * platform/mac/PlatformEventFactoryMac.h:
+ * platform/mac/PlatformEventFactoryMac.mm:
+ (WebCore::keyIdentifierForKeyEvent):
+ (WebCore::windowsKeyCodeForKeyEvent):
+ Export functions for reuse in WebKit2. I did not attempt any larger refactoring at this time.
+
+2012-01-31 Gregg Tavares <gman@google.com>
+
+ Make WebGL put synthesized errors in the JS console
+ https://bugs.webkit.org/show_bug.cgi?id=77267
+
+ Reviewed by Kenneth Russell.
+
+ No new functionality so no new tests.
+
+ * html/canvas/WebGLCompressedTextures.cpp:
+ (WebCore::WebGLCompressedTextures::validateCompressedTexFuncData):
+ (WebCore::WebGLCompressedTextures::compressedTexImage2D):
+ (WebCore::WebGLCompressedTextures::compressedTexSubImage2D):
+ * html/canvas/WebGLCompressedTextures.h:
+ (WebGLCompressedTextures):
+ * html/canvas/WebGLDebugShaders.cpp:
+ (WebCore::WebGLDebugShaders::getTranslatedShaderSource):
+ * html/canvas/WebGLRenderingContext.cpp:
+ (WebCore::WebGLRenderingContext::activeTexture):
+ (WebCore::WebGLRenderingContext::attachShader):
+ (WebCore::WebGLRenderingContext::bindAttribLocation):
+ (WebCore::WebGLRenderingContext::checkObjectToBeBound):
+ (WebCore::WebGLRenderingContext::bindBuffer):
+ (WebCore::WebGLRenderingContext::bindFramebuffer):
+ (WebCore::WebGLRenderingContext::bindRenderbuffer):
+ (WebCore::WebGLRenderingContext::bindTexture):
+ (WebCore::WebGLRenderingContext::blendEquation):
+ (WebCore::WebGLRenderingContext::blendEquationSeparate):
+ (WebCore::WebGLRenderingContext::blendFunc):
+ (WebCore::WebGLRenderingContext::blendFuncSeparate):
+ (WebCore::WebGLRenderingContext::bufferData):
+ (WebCore::WebGLRenderingContext::bufferSubData):
+ (WebCore::WebGLRenderingContext::checkFramebufferStatus):
+ (WebCore::WebGLRenderingContext::clear):
+ (WebCore::WebGLRenderingContext::compileShader):
+ (WebCore::WebGLRenderingContext::compressedTexImage2D):
+ (WebCore::WebGLRenderingContext::compressedTexSubImage2D):
+ (WebCore::WebGLRenderingContext::copyTexImage2D):
+ (WebCore::WebGLRenderingContext::copyTexSubImage2D):
+ (WebCore::WebGLRenderingContext::createShader):
+ (WebCore::WebGLRenderingContext::deleteObject):
+ (WebCore::WebGLRenderingContext::depthRange):
+ (WebCore::WebGLRenderingContext::detachShader):
+ (WebCore::WebGLRenderingContext::disable):
+ (WebCore::WebGLRenderingContext::disableVertexAttribArray):
+ (WebCore::WebGLRenderingContext::validateWebGLObject):
+ (WebCore::WebGLRenderingContext::drawArrays):
+ (WebCore::WebGLRenderingContext::drawElements):
+ (WebCore::WebGLRenderingContext::enable):
+ (WebCore::WebGLRenderingContext::enableVertexAttribArray):
+ (WebCore::WebGLRenderingContext::framebufferRenderbuffer):
+ (WebCore::WebGLRenderingContext::framebufferTexture2D):
+ (WebCore::WebGLRenderingContext::generateMipmap):
+ (WebCore::WebGLRenderingContext::getActiveAttrib):
+ (WebCore::WebGLRenderingContext::getActiveUniform):
+ (WebCore::WebGLRenderingContext::getAttachedShaders):
+ (WebCore::WebGLRenderingContext::getAttribLocation):
+ (WebCore::WebGLRenderingContext::getBufferParameter):
+ (WebCore::WebGLRenderingContext::getFramebufferAttachmentParameter):
+ (WebCore::WebGLRenderingContext::getParameter):
+ (WebCore::WebGLRenderingContext::getProgramParameter):
+ (WebCore::WebGLRenderingContext::getProgramInfoLog):
+ (WebCore::WebGLRenderingContext::getRenderbufferParameter):
+ (WebCore::WebGLRenderingContext::getShaderParameter):
+ (WebCore::WebGLRenderingContext::getShaderInfoLog):
+ (WebCore::WebGLRenderingContext::getShaderSource):
+ (WebCore::WebGLRenderingContext::getTexParameter):
+ (WebCore::WebGLRenderingContext::getUniform):
+ (WebCore::WebGLRenderingContext::getUniformLocation):
+ (WebCore::WebGLRenderingContext::getVertexAttrib):
+ (WebCore::WebGLRenderingContext::hint):
+ (WebCore::WebGLRenderingContext::isEnabled):
+ (WebCore::WebGLRenderingContext::linkProgram):
+ (WebCore::WebGLRenderingContext::pixelStorei):
+ (WebCore::WebGLRenderingContext::readPixels):
+ (WebCore::WebGLRenderingContext::renderbufferStorage):
+ (WebCore::WebGLRenderingContext::scissor):
+ (WebCore::WebGLRenderingContext::shaderSource):
+ (WebCore::WebGLRenderingContext::stencilFunc):
+ (WebCore::WebGLRenderingContext::stencilFuncSeparate):
+ (WebCore::WebGLRenderingContext::stencilMaskSeparate):
+ (WebCore::WebGLRenderingContext::texImage2DBase):
+ (WebCore::WebGLRenderingContext::texImage2DImpl):
+ (WebCore::WebGLRenderingContext::texImage2D):
+ (WebCore::WebGLRenderingContext::videoFrameToImage):
+ (WebCore::WebGLRenderingContext::texParameter):
+ (WebCore::WebGLRenderingContext::texSubImage2DBase):
+ (WebCore::WebGLRenderingContext::texSubImage2DImpl):
+ (WebCore::WebGLRenderingContext::texSubImage2D):
+ (WebCore::WebGLRenderingContext::uniform1f):
+ (WebCore::WebGLRenderingContext::uniform1fv):
+ (WebCore::WebGLRenderingContext::uniform1i):
+ (WebCore::WebGLRenderingContext::uniform1iv):
+ (WebCore::WebGLRenderingContext::uniform2f):
+ (WebCore::WebGLRenderingContext::uniform2fv):
+ (WebCore::WebGLRenderingContext::uniform2i):
+ (WebCore::WebGLRenderingContext::uniform2iv):
+ (WebCore::WebGLRenderingContext::uniform3f):
+ (WebCore::WebGLRenderingContext::uniform3fv):
+ (WebCore::WebGLRenderingContext::uniform3i):
+ (WebCore::WebGLRenderingContext::uniform3iv):
+ (WebCore::WebGLRenderingContext::uniform4f):
+ (WebCore::WebGLRenderingContext::uniform4fv):
+ (WebCore::WebGLRenderingContext::uniform4i):
+ (WebCore::WebGLRenderingContext::uniform4iv):
+ (WebCore::WebGLRenderingContext::uniformMatrix2fv):
+ (WebCore::WebGLRenderingContext::uniformMatrix3fv):
+ (WebCore::WebGLRenderingContext::uniformMatrix4fv):
+ (WebCore::WebGLRenderingContext::useProgram):
+ (WebCore::WebGLRenderingContext::validateProgram):
+ (WebCore::WebGLRenderingContext::vertexAttrib1f):
+ (WebCore::WebGLRenderingContext::vertexAttrib1fv):
+ (WebCore::WebGLRenderingContext::vertexAttrib2f):
+ (WebCore::WebGLRenderingContext::vertexAttrib2fv):
+ (WebCore::WebGLRenderingContext::vertexAttrib3f):
+ (WebCore::WebGLRenderingContext::vertexAttrib3fv):
+ (WebCore::WebGLRenderingContext::vertexAttrib4f):
+ (WebCore::WebGLRenderingContext::vertexAttrib4fv):
+ (WebCore::WebGLRenderingContext::vertexAttribPointer):
+ (WebCore::WebGLRenderingContext::viewport):
+ (WebCore::WebGLRenderingContext::forceLostContext):
+ (WebCore::WebGLRenderingContext::loseContextImpl):
+ (WebCore::WebGLRenderingContext::forceRestoreContext):
+ (WebCore::WebGLRenderingContext::validateTextureBinding):
+ (WebCore::WebGLRenderingContext::validateLocationLength):
+ (WebCore::WebGLRenderingContext::validateSize):
+ (WebCore::WebGLRenderingContext::validateString):
+ (WebCore::WebGLRenderingContext::validateTexFuncFormatAndType):
+ (WebCore::WebGLRenderingContext::validateTexFuncLevel):
+ (WebCore::WebGLRenderingContext::validateTexFuncParameters):
+ (WebCore::WebGLRenderingContext::validateTexFuncData):
+ (WebCore::WebGLRenderingContext::validateDrawMode):
+ (WebCore::WebGLRenderingContext::validateStencilSettings):
+ (WebCore::WebGLRenderingContext::validateStencilFunc):
+ (WebCore::WebGLRenderingContext::validateFramebufferFuncParameters):
+ (WebCore::WebGLRenderingContext::validateBlendEquation):
+ (WebCore::WebGLRenderingContext::validateBlendFuncFactors):
+ (WebCore::WebGLRenderingContext::validateCapability):
+ (WebCore::WebGLRenderingContext::validateUniformParameters):
+ (WebCore::WebGLRenderingContext::validateUniformMatrixParameters):
+ (WebCore::WebGLRenderingContext::validateBufferDataParameters):
+ (WebCore::WebGLRenderingContext::validateHTMLImageElement):
+ (WebCore::WebGLRenderingContext::vertexAttribfImpl):
+ (WebCore::WebGLRenderingContext::vertexAttribfvImpl):
+ (WebCore::WebGLRenderingContext::maybeRestoreContext):
+ (WebCore):
+ (WebCore::WebGLRenderingContext::synthesizeGLError):
+ * html/canvas/WebGLRenderingContext.h:
+ (WebGLRenderingContext):
+
+2012-01-31 Raymond Liu <raymond.liu@intel.com>
+
+ Clean up m_processLock logic in AudioBasicProcessorNode and AudioGainNode
+ https://bugs.webkit.org/show_bug.cgi?id=76772
+
+ Reviewed by Kenneth Russell.
+
+ No new tests required.
+
+ * webaudio/AudioBasicProcessorNode.cpp:
+ (WebCore::AudioBasicProcessorNode::process):
+ (WebCore::AudioBasicProcessorNode::checkNumberOfChannelsForInput):
+ * webaudio/AudioBasicProcessorNode.h:
+ * webaudio/AudioGainNode.cpp:
+ (WebCore::AudioGainNode::process):
+ (WebCore::AudioGainNode::checkNumberOfChannelsForInput):
+ * webaudio/AudioGainNode.h:
+ (AudioGainNode):
+
+2012-01-31 Adam Klein <adamk@chromium.org>
+
+ ProcessingInstruction should not be a ContainerNode
+ https://bugs.webkit.org/show_bug.cgi?id=75141
+
+ Reviewed by Darin Adler.
+
+ Per the DOM spec, ProcessingInstruction can't have any children.
+ And the WebCore behavior already matches the spec by always returning
+ false for childTypeAllowed(). This change simplifies
+ ProcessingInstruction's implementation by making it subclass Node
+ instead of ContainerNode.
+
+ Test: fast/dom/processing-instruction-appendChild-exceptions.xhtml
+
+ * dom/ContainerNode.cpp: Moved dispatchBeforeLoadEvent up to Node.
+ * dom/ContainerNode.h:
+ * dom/Node.cpp:
+ (WebCore::Node::dispatchBeforeLoadEvent): Moved up from ContainerNode
+ since it's used both by ProcessingInstruction and various Element
+ subclasses.
+ * dom/Node.h:
+ * dom/ProcessingInstruction.cpp:
+ (WebCore::ProcessingInstruction::ProcessingInstruction): Call Node constructor.
+ (WebCore::ProcessingInstruction::insertedIntoDocument): Call Node impl.
+ (WebCore::ProcessingInstruction::removedFromDocument): ditto.
+ (WebCore::ProcessingInstruction::finishParsingChildren): ditto.
+ * dom/ProcessingInstruction.h:
+
+2012-01-31 Matthew Delaney <mdelaney@apple.com>
+
+ Failing 2d.shadow.enable.off.2.html on Lion
+ https://bugs.webkit.org/show_bug.cgi?id=77489
+
+ Reviewed by Dan Bernstein.
+
+ The canvas spec requires that shadows not be drawn under certain
+ circumstances outlined here: http://www.whatwg.org/specs/web-apps/current-work/#shadows
+ This patch adds in those checks which allows us to pass now (on Lion)
+ the philip canvas test that was checking that constraint.
+
+ No new tests. Unskipping the test on Lion that this patch fixes.
+
+ * html/canvas/CanvasRenderingContext2D.cpp:
+ (WebCore::CanvasRenderingContext2D::setAllAttributesToDefault):
+ (WebCore::CanvasRenderingContext2D::setShadow):
+ (WebCore::CanvasRenderingContext2D::applyShadow):
+ (WebCore::CanvasRenderingContext2D::shouldDrawShadows):
+ * html/canvas/CanvasRenderingContext2D.h:
+ (CanvasRenderingContext2D):
+
+2012-01-31 Anders Carlsson <andersca@apple.com>
+
+ TileCache::setNeedsDisplayInRect cleanup
+ https://bugs.webkit.org/show_bug.cgi?id=77486
+
+ Reviewed by Andreas Kling.
+
+ * platform/graphics/ca/mac/TileCache.h:
+ * platform/graphics/ca/mac/TileCache.mm:
+ (WebCore::TileCache::setNeedsDisplayInRect):
+ TileCache::tileLayerAtIndex can in the future return nil, so cope with that. Also, replace
+ nested if statements with continue statements.
+
+ (WebCore::TileCache::getTileIndexRangeForRect):
+ Rename this to better indicate that it returns a range of indices.
+
+2012-01-31 Dana Jansens <danakj@chromium.org>
+
+ Add contains() test to Region
+ https://bugs.webkit.org/show_bug.cgi?id=72294
+
+ Reviewed by Anders Carlsson.
+
+ * platform/graphics/Region.cpp:
+ (WebCore::Region::contains):
+ (WebCore):
+ * platform/graphics/Region.h:
+ (Region):
+ (Shape):
+ (WebCore::operator==):
+ (WebCore):
+
+2012-01-31 Sami Kyostila <skyostil@chromium.org>
+
+ [chromium] Compositor debug borders are not scaled correctly
+ https://bugs.webkit.org/show_bug.cgi?id=77468
+
+ Use CCLayerImpl::contentBounds() instead of CCLayerImpl::bounds() to calculate debug
+ border geometry so that the borders are properly scaled.
+
+ Reviewed by James Robinson.
+
+ * platform/graphics/chromium/cc/CCLayerImpl.cpp:
+ (WebCore::CCLayerImpl::appendDebugBorderQuad):
+
+2012-01-31 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r106376.
+ http://trac.webkit.org/changeset/106376
+ https://bugs.webkit.org/show_bug.cgi?id=77481
+
+ Broke WebGLLayerChromiumTest in webkit_unit_tests (Requested
+ by kbr_google on #webkit).
+
+ * platform/graphics/gpu/DrawingBuffer.cpp:
+ (WebCore):
+ (WebCore::DrawingBuffer::reset):
+
+2012-01-31 Mihnea Ovidenie <mihnea@adobe.com>
+
+ [CSSRegions]Reduce the cases when the box style in region is computed
+ https://bugs.webkit.org/show_bug.cgi?id=77446
+
+ Reviewed by David Hyatt.
+
+ Covered by existing region style tests.
+
+ * rendering/RenderRegion.cpp:
+ (WebCore::RenderRegion::setRegionBoxesRegionStyle):
+ (WebCore::RenderRegion::restoreRegionBoxesOriginalStyle):
+
+2012-01-31 Anders Carlsson <andersca@apple.com>
+
+ Put tiles in a HashMap
+ https://bugs.webkit.org/show_bug.cgi?id=77480
+
+ Reviewed by Antti Koivisto.
+
+ Put tiles in a hash map keyed off the tile index.
+
+ * platform/graphics/ca/mac/TileCache.h:
+ Shuffle member variables around so the order makes more sense.
+ Add the tile map.
+
+ * platform/graphics/ca/mac/TileCache.mm:
+ (WebCore::TileCache::TileCache):
+ Update member initializers.
+
+ (WebCore::TileCache::setNeedsDisplayInRect):
+ Call tileLayerAtIndex instead of tileLayerAtPosition.
+
+ (WebCore::TileCache::setAcceleratesDrawing):
+ (WebCore::TileCache::setTileDebugBorderWidth):
+ (WebCore::TileCache::setTileDebugBorderColor):
+ Iterate over the hash map instead of the sublayers.
+
+ (WebCore::TileCache::resizeTileGrid):
+ Add the created layers to the map.
+
+ (WebCore::TileCache::tileLayerAtIndex):
+ Rename from tileLayerAtPoint to better reflect that this member function
+ returns a tile layer at the given index and not the given point.
+
+2012-01-31 Antti Koivisto <antti@apple.com>
+
+ Make CSSOM style() return CSSStyleDeclaration*
+ https://bugs.webkit.org/show_bug.cgi?id=77475
+
+ Reviewed by Anders Carlsson
+
+ CSSStyleRule::style() and some other places return CSSMutableStyleDeclaration.
+ They should return the plain CSSOM type instead.
+
+ CSSMutableStyleDeclaration* should be available through non-CSSOM function.
+
+ * css/CSSFontFaceRule.h:
+ (WebCore::CSSFontFaceRule::style):
+ (WebCore::CSSFontFaceRule::declaration):
+ * css/CSSFontSelector.cpp:
+ (WebCore::CSSFontSelector::addFontFaceRule):
+ * css/CSSStyleRule.h:
+ (WebCore::CSSStyleRule::style):
+ (WebCore::CSSStyleRule::declaration):
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::styleForKeyframe):
+ * css/WebKitCSSKeyframeRule.h:
+ (WebCore):
+ (WebCore::WebKitCSSKeyframeRule::style):
+ (WebCore::WebKitCSSKeyframeRule::declaration):
+ (WebKitCSSKeyframeRule):
+ * editing/EditingStyle.cpp:
+ (WebCore::styleFromMatchedRulesForElement):
+ * inspector/InspectorStyleSheet.cpp:
+ (WebCore::InspectorStyleSheet::revalidateStyle):
+ * page/PageSerializer.cpp:
+ (WebCore::PageSerializer::retrieveResourcesForCSSRule):
+ * svg/SVGFontFaceElement.cpp:
+ (WebCore::SVGFontFaceElement::parseMappedAttribute):
+ (WebCore::SVGFontFaceElement::fontFamily):
+ (WebCore::SVGFontFaceElement::rebuildFontFace):
+
+2012-01-31 Scott Graham <scottmg@chromium.org>
+
+ [Chromium] Remove references to gyp cygwin build target
+ https://bugs.webkit.org/show_bug.cgi?id=77253
+
+ Reviewed by Julien Chaffraix.
+
+ Target dependency is no longer required, it's done earlier in the
+ build process.
+
+ * WebCore.gyp/WebCore.gyp:
+
+2012-01-31 Jon Lee <jonlee@apple.com>
+
+ Hidden form elements do not save their state prior to form submission
+ https://bugs.webkit.org/show_bug.cgi?id=77391
+ <rdar://problem/10563108>
+
+ Reviewed by Brady Eidson.
+
+ Test: fast/forms/state-restore-hidden.html
+
+ * html/HiddenInputType.cpp: Teach hidden inputs to save and restore their state.
+ (WebCore::HiddenInputType::saveFormControlState):
+ (WebCore::HiddenInputType::restoreFormControlState):
+ * html/HiddenInputType.h:
+ (HiddenInputType):
+
+2012-01-31 Joshua Bell <jsbell@chromium.org>
+
+ IndexedDB: IDBCursor.update() should raise exception if key changed
+ https://bugs.webkit.org/show_bug.cgi?id=76952
+
+ Move the test from the async task to the synchronous call, per spec. Also re-ordered the tests
+ done during the synchronous call and the asynchronous task to follow the spec order.
+
+ Reviewed by Tony Chang.
+
+ Tests: storage/indexeddb/cursor-update.html
+
+ * storage/IDBObjectStoreBackendImpl.cpp:
+ (WebCore::IDBObjectStoreBackendImpl::put): Added check during update() call, order checks per spec.
+ (WebCore::IDBObjectStoreBackendImpl::putInternal): Move effective key calculation inline.
+ * storage/IDBObjectStoreBackendImpl.h: Removed selectKeyForPut method.
+
+2012-01-31 Anders Carlsson <andersca@apple.com>
+
+ Inform the tile cache whenever the visible rect changes
+ https://bugs.webkit.org/show_bug.cgi?id=77470
+
+ Reviewed by Andreas Kling.
+
+ * platform/graphics/GraphicsLayer.h:
+ (WebCore::GraphicsLayer::visibleRectChanged):
+ Add empty function.
+
+ * platform/graphics/ca/GraphicsLayerCA.cpp:
+ (WebCore::GraphicsLayerCA::visibleRectChanged):
+ Call through to the PlatformCALayer.
+
+ * platform/graphics/ca/mac/PlatformCALayerMac.mm:
+ (PlatformCALayer::visibleRectChanged):
+ Call through to the underlying WebTileCacheLayer.
+
+ * platform/graphics/ca/mac/TileCache.mm:
+ (WebCore::TileCache::visibleRectChanged):
+ Add empty stub.
+
+ (WebCore::TileCache::visibleRect):
+ Add new (currently unused) helper function that returns the visible rect of the
+ tile cache layer.
+
+ * platform/graphics/ca/mac/WebTileCacheLayer.h:
+ * platform/graphics/ca/mac/WebTileCacheLayer.mm:
+ (-[WebTileCacheLayer visibleRectChanged]):
+ Call through to the TielCache object.
+
+ * platform/graphics/ca/win/PlatformCALayerWin.cpp:
+ (PlatformCALayer::visibleRectChanged):
+ Add stub.
+
+ * rendering/RenderLayerCompositor.cpp:
+ (WebCore::RenderLayerCompositor::frameViewDidScroll):
+ Call GraphicsLayer::visibleRectChanged.
+
+2012-01-31 Antti Koivisto <antti@apple.com>
+
+ Remove CSSStyleDeclaration isElementStyleDeclaration bit
+ https://bugs.webkit.org/show_bug.cgi?id=77460
+
+ Reviewed by Andreas Kling.
+
+ Inline style declaration is now the only type of style declaration with element parent.
+ We can remove the bit and the associated logic.
+
+ * bindings/js/JSDOMBinding.h:
+ (WebCore::root):
+ * css/CSSMutableStyleDeclaration.cpp:
+ (WebCore::CSSMutableStyleDeclaration::CSSMutableStyleDeclaration):
+ (WebCore::CSSMutableStyleDeclaration::setNeedsStyleRecalc):
+ * css/CSSMutableStyleDeclaration.h:
+ (WebCore::CSSMutableStyleDeclaration::createInline):
+ (CSSMutableStyleDeclaration):
+ * css/CSSStyleDeclaration.cpp:
+ (WebCore::CSSStyleDeclaration::CSSStyleDeclaration):
+ (WebCore):
+ (WebCore::CSSStyleDeclaration::parentStyleSheet):
+ * css/CSSStyleDeclaration.h:
+ (WebCore::CSSStyleDeclaration::parentRule):
+ (WebCore::CSSStyleDeclaration::clearParentRule):
+ (WebCore::CSSStyleDeclaration::parentElement):
+ (WebCore::CSSStyleDeclaration::clearParentElement):
+ (CSSStyleDeclaration):
+
+2012-01-31 Dana Jansens <danakj@chromium.org>
+
+ [chromium] Compute occlusion during paint loop
+ https://bugs.webkit.org/show_bug.cgi?id=76858
+
+ Reviewed by James Robinson.
+
+ New unit tests in TiledLayerChromiumTest.cpp, CCLayerTreeHostCommonTest.cpp, CCLayerTreeHostTest.cpp
+
+ * platform/graphics/FloatRect.cpp:
+ (WebCore::enclosedIntRect):
+ (WebCore):
+ * platform/graphics/FloatRect.h:
+ (WebCore):
+ * platform/graphics/chromium/Canvas2DLayerChromium.cpp:
+ (WebCore::Canvas2DLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/Canvas2DLayerChromium.h:
+ (Canvas2DLayerChromium):
+ * platform/graphics/chromium/ContentLayerChromium.cpp:
+ (WebCore::ContentLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/ContentLayerChromium.h:
+ (ContentLayerChromium):
+ * platform/graphics/chromium/ImageLayerChromium.cpp:
+ (WebCore::ImageLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/ImageLayerChromium.h:
+ (ImageLayerChromium):
+ * platform/graphics/chromium/LayerChromium.cpp:
+ (WebCore::LayerChromium::contentToScreenSpaceTransform):
+ (WebCore):
+ (WebCore::LayerChromium::addSelfToOccludedScreenSpace):
+ (WebCore::LayerChromium::isPaintedAxisAlignedInScreen):
+ * platform/graphics/chromium/LayerChromium.h:
+ (WebCore):
+ (WebCore::LayerChromium::paintContentsIfDirty):
+ (LayerChromium):
+ * platform/graphics/chromium/RenderSurfaceChromium.h:
+ (RenderSurfaceChromium):
+ * platform/graphics/chromium/TiledLayerChromium.cpp:
+ (WebCore::TiledLayerChromium::addSelfToOccludedScreenSpace):
+ * platform/graphics/chromium/TiledLayerChromium.h:
+ (WebCore):
+ ():
+ * platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
+ (WebCore::CCLayerTreeHost::paintContentsIfDirty):
+ (WebCore::CCLayerTreeHost::paintMaskAndReplicaForRenderSurface):
+ (RenderSurfaceRegion):
+ (WebCore):
+ (WebCore::pushTargetRenderSurfaceRegion):
+ (WebCore::popAndPushTargetRenderSurfaceRegion):
+ (WebCore::CCLayerTreeHost::paintLayerContents):
+ * platform/graphics/chromium/cc/CCLayerTreeHost.h:
+ (WebCore):
+ ():
+ * platform/graphics/chromium/cc/CCQuadCuller.cpp:
+
+2012-01-31 John Bates <jbates@google.com>
+
+ [Chromium] Add chromium-style tracing support
+ https://bugs.webkit.org/show_bug.cgi?id=76885
+
+ Reviewed by Darin Fisher.
+
+ This code enables WebKit trace events to pass through more data to the
+ chromium platform tracing API and generally to use the full tracing
+ API provided by chromium.
+
+ * bindings/v8/V8Proxy.cpp:
+ (WebCore::V8Proxy::evaluate): Replace old tracing API.
+ * page/Console.cpp:
+ (WebCore::Console::time): Replace old tracing API.
+ (WebCore::Console::timeEnd): Replace old tracing API.
+ * platform/chromium/PlatformSupport.h:
+ * platform/chromium/TraceEvent.h:
+ (WebCore::TraceEvent::TraceID::TraceID):
+ (WebCore::TraceEvent::TraceID::data):
+ (WebCore::TraceEvent::TraceStringWithCopy::TraceStringWithCopy):
+ (WebCore::TraceEvent::TraceStringWithCopy::operator const char* ):
+ (WebCore::TraceEvent::setTraceValue):
+ (WebCore::TraceEvent::addTraceEvent):
+ (WebCore::TraceEvent::TraceEndOnScopeClose::TraceEndOnScopeClose):
+ (WebCore::TraceEvent::TraceEndOnScopeClose::~TraceEndOnScopeClose):
+ (WebCore::TraceEvent::TraceEndOnScopeClose::initialize):
+ (WebCore::TraceEvent::TraceEndOnScopeClose::addEventIfEnabled):
+ (WebCore::TraceEvent::TraceEndOnScopeCloseThreshold::TraceEndOnScopeCloseThreshold):
+ (WebCore::TraceEvent::TraceEndOnScopeCloseThreshold::~TraceEndOnScopeCloseThreshold):
+ (WebCore::TraceEvent::TraceEndOnScopeCloseThreshold::initialize):
+ (WebCore::TraceEvent::TraceEndOnScopeCloseThreshold::addEventIfEnabled):
+
+2012-01-31 Ryosuke Niwa <rniwa@webkit.org>
+
+ Crash in DeleteSelectionCommand::handleGeneralDelete when attempting to delete the start block
+ https://bugs.webkit.org/show_bug.cgi?id=77077
+
+ Reviewed by Enrica Casucci.
+
+ The crash was caused by a missing null check after removing the position out of the start block.
+ Fixed the bug by adding an early return.
+
+ Tests: editing/deleting/delete-start-block.html
+ editing/selection/move-into-empty-root-inline-box.html
+
+ * editing/DeleteSelectionCommand.cpp:
+ (WebCore::DeleteSelectionCommand::handleGeneralDelete):
+
+2012-01-31 Rafael Brandao <rafael.lobo@openbossa.org>
+
+ HTMLIsIndexElement should not expose HTMLInputElement properties
+ https://bugs.webkit.org/show_bug.cgi?id=76095
+
+ Reviewed by Eric Seidel.
+
+ This is a buildfix for r106373.
+
+ * DerivedSources.pri: Removed reference to HTMLIsIndexElement.idl
+
+2012-01-31 Yongsheng Zhu <yongsheng.zhu@intel.com>
+
+ WebGL must allocate smaller drawing buffer when the allocation fails.
+ https://bugs.webkit.org/show_bug.cgi?id=76654
+
+ Reviewed by Kenneth Russell.
+
+ Test: fast/canvas/webgl/drawingbuffer-test.html
+
+ * platform/graphics/gpu/DrawingBuffer.cpp:
+ (WebCore):
+ (WebCore::DrawingBuffer::create):
+ (WebCore::DrawingBuffer::reset):
+
+2012-01-25 Eric Seidel <eric@webkit.org>
+
+ HTMLIsIndexElement should not expose HTMLInputElement properties
+ https://bugs.webkit.org/show_bug.cgi?id=76095
+
+ Reviewed by Adam Barth.
+
+ document.createElement("isindex") should produce an HTMLUnknownElement
+ per the HTML5 spec. The parser automagically translates <isindex> into
+ a whole dom tree roughly representing what <isindex> used to do 15 years ago. :)
+
+ This patch just removes our support for HTMLIsIndexElement. The parser
+ support was already in. Having support for HTMLIsIndexElement was causing
+ one of the IE TestCenter tests to fail.
+
+ Test: fast/dom/HTMLIsIndexElement/prototype-chain.html
+
+ * DerivedSources.cpp:
+ * DerivedSources.make:
+ * DerivedSources.pri:
+ * GNUmakefile.list.am:
+ * Target.pri:
+ * WebCore.exp.in:
+ * WebCore.gypi:
+ * WebCore.order:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * bindings/gobject/GNUmakefile.am:
+ * bindings/gobject/WebKitHTMLElementWrapperFactory.cpp:
+ (WebKit::createHTMLElementWrapper):
+ * bindings/objc/DOM.mm:
+ (WebCore::createElementClassMap):
+ * bindings/objc/DOMHTML.h:
+ * bindings/objc/PublicDOMInterfaces.h:
+ * html/HTMLElementsAllInOne.cpp:
+ * html/HTMLIsIndexElement.cpp: Removed.
+ * html/HTMLIsIndexElement.h: Removed.
+ * html/HTMLIsIndexElement.idl: Removed.
+ * html/HTMLTagNames.in:
+ * page/DOMWindow.idl:
+
+2012-01-31 Alexis Menard <alexis.menard@openbossa.org>
+
+ Unreviewed include cleanup.
+
+ Tested locally on Qt and Chromium port.
+
+ * dom/Node.cpp:
+
+2012-01-31 Arko Saha <arko@motorola.com>
+
+ The spec renamed addTrack() to addTextTrack().
+ https://bugs.webkit.org/show_bug.cgi?id=77381
+
+ Reviewed by Eric Carlson.
+
+ Renamed addTrack() to addTextTrack().
+ Spec: http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#media-elements
+
+ * html/HTMLMediaElement.cpp:
+ (WebCore::HTMLMediaElement::addTextTrack):
+ * html/HTMLMediaElement.h:
+ (WebCore::HTMLMediaElement::addTextTrack):
+ * html/HTMLMediaElement.idl:
+
+2012-01-31 Peter Rybin <peter.rybin@gmail.com>
+
+ Web Inspector: CodeGeneratorInspector.py: reimplement generated array types
+ https://bugs.webkit.org/show_bug.cgi?id=77289
+
+ Reviewed by Vsevolod Vlasov.
+
+ Arrays are now rendered as a single template class. Its type-specific
+ details are provided by an additional ArrayItemHelper class that is
+ defined using C++ specialization technique.
+
+ * inspector/CodeGeneratorInspector.py:
+ (RawTypes.String):
+ (RawTypes.String.get_array_item_raw_c_type_text):
+ (RawTypes.Int):
+ (RawTypes.Int.get_array_item_raw_c_type_text):
+ (RawTypes.Number):
+ (RawTypes.Number.get_array_item_raw_c_type_text):
+ (RawTypes.Bool):
+ (RawTypes.Bool.get_array_item_raw_c_type_text):
+ (RawTypes.Object):
+ (RawTypes.Object.get_array_item_raw_c_type_text):
+ (RawTypes.Any):
+ (RawTypes.Any.get_array_item_raw_c_type_text):
+ (RawTypes.Array):
+ (RawTypes.Array.get_array_item_raw_c_type_text):
+ (RawTypes):
+ (TypeBindings.create_type_declaration_.EnumBinding):
+ (TypeBindings.create_type_declaration_.EnumBinding.get_array_item_c_type_text):
+ (TypeBindings.create_type_declaration_.PlainString):
+ (TypeBindings.create_type_declaration_.PlainString.get_array_item_c_type_text):
+ (TypeBindings.create_type_declaration_.TypedefString):
+ (TypeBindings.create_type_declaration_.TypedefString.get_array_item_c_type_text):
+ (StructItemTraits):
+ (get_array_item_c_type_text):
+ (PlainObjectBinding):
+ (PlainObjectBinding.get_array_item_c_type_text):
+ (AdHocTypeContext):
+ (AdHocTypeContext.get_type_name_fix):
+ (AdHocTypeContext.add_type):
+ (ArrayBinding):
+ (ArrayBinding.resolve_inner):
+ (ArrayBinding.resolve_inner.ResolveData):
+ (ArrayBinding.request_user_runtime_cast):
+ (ArrayBinding.request_internal_runtime_cast):
+ (ArrayBinding.get_code_generator):
+ (ArrayBinding.get_code_generator.CodeGenerator):
+ (ArrayBinding.get_code_generator.CodeGenerator.generate_type_builder):
+ (ArrayBinding.get_code_generator.CodeGenerator.generate_forward_declaration):
+ (ArrayBinding.get_code_generator.CodeGenerator.register_use):
+ (ArrayBinding.get_code_generator.CodeGenerator.get_generate_pass_id):
+ (ArrayBinding.get_validator_call_text):
+ (ArrayBinding.get_in_c_type_text):
+ (ArrayBinding.get_array_item_c_type_text):
+ (ArrayBinding.get_setter_value_expression_pattern):
+ (ArrayBinding.reduce_to_raw_type):
+ (RawTypeBinding.get_array_item_c_type_text):
+ (ArrayItemHelper):
+ (typename):
+ (ArrayOf):
+ (Traits):
+
+2012-01-31 Antti Koivisto <antti@apple.com>
+
+ Parent SVGFontFaceElements style declaration to the rule
+ https://bugs.webkit.org/show_bug.cgi?id=77421
+
+ Reviewed by Adam Roben.
+
+ For some reason the declaration is parented to the element which adds a bunch of unnecessary special case code.
+ The invalidation on mutation is done explicitly by SVGFontFaceElement so that is not affected. The declaration
+ is not exposed so the change is not observable with a test.
+
+ * css/CSSFontFaceRule.cpp:
+ (WebCore::CSSFontFaceRule::~CSSFontFaceRule):
+ * css/CSSMutableStyleDeclaration.h:
+ (WebCore::CSSMutableStyleDeclaration::createInline):
+ * svg/SVGFontFaceElement.cpp:
+ (WebCore::SVGFontFaceElement::SVGFontFaceElement):
+ (WebCore::SVGFontFaceElement::parseMappedAttribute):
+ (WebCore::SVGFontFaceElement::fontFamily):
+ (WebCore::SVGFontFaceElement::rebuildFontFace):
+ * svg/SVGFontFaceElement.h:
+
+ Remove the unnecessary m_styleDeclaration field, access through m_fontFaceRule instead.
+
+2012-01-31 Kenneth Rohde Christiansen <kenneth@webkit.org>
+
+ Tap highlighting: Support better outlines for multiline inlines
+ https://bugs.webkit.org/show_bug.cgi?id=77428
+
+ Reviewed by Simon Hausmann.
+
+ Covered by manual tests.
+
+ Do not use the linesBoundingBox anymore but draw a custom path
+ with rounded corners. Inlines are drawn as max 3 rects, first
+ line rect, joined middle rect and the rect for the last line.
+
+ * page/GestureTapHighlighter.cpp:
+ * platform/graphics/Path.h: Make addBeziersForRoundedRect public.
+
+2012-01-31 Alexei Filippov <alexeif@chromium.org>
+
+ Web Inspector: show sizes in bytes instead of KB, MB in heap profiler.
+ https://bugs.webkit.org/show_bug.cgi?id=77199
+
+ Reviewed by Pavel Feldman.
+
+ * inspector/front-end/DetailedHeapshotGridNodes.js:
+ (WebInspector.HeapSnapshotGenericObjectNode.prototype.get data):
+ (WebInspector.HeapSnapshotInstanceNode.prototype._enhanceData):
+ (WebInspector.HeapSnapshotConstructorNode.prototype.get data):
+ (WebInspector.HeapSnapshotDiffNode.prototype.get data):
+ * inspector/front-end/UIUtils.js:
+ (Number.withThousandsSeparator):
+
+2012-01-26 Hans Wennborg <hans@chromium.org>
+
+ Speech Input: move MockSpeechInputClient into Chromium DumpRenderTree implementation
+ https://bugs.webkit.org/show_bug.cgi?id=77083
+
+ Reviewed by Darin Fisher.
+
+ Remove SpeechInputClientMock. The mock is moving to the DumpRenderTree
+ implementation.
+
+ No new tests, just refactoring.
+
+ * GNUmakefile.list.am:
+ * Target.pri:
+ * WebCore.gypi:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * platform/mock/SpeechInputClientMock.cpp: Removed.
+ * platform/mock/SpeechInputClientMock.h: Removed.
+
+2012-01-31 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: boost protocol version to 1.0
+ https://bugs.webkit.org/show_bug.cgi?id=77408
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/Inspector-1.0.json: Copied from Source/WebCore/inspector/Inspector.json.
+ * inspector/Inspector.json:
+
+2012-01-31 Roland Steiner <rolandsteiner@chromium.org>
+
+ <style scoped>: Improve shortcut code for cases where <style scoped> isn't used
+ https://bugs.webkit.org/show_bug.cgi?id=77410
+
+ Move shortcut from setupScopingElementStack(), do it at the calling sites instead
+ (where a larger chunk of work can be skipped).
+
+ Reviewed by Antti Koivisto.
+
+ No new tests. (refactoring)
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::setupScopingElementStack): remove shortcut code
+ (WebCore::CSSStyleSelector::pushParent): add shortcut code
+ (WebCore::CSSStyleSelector::matchScopedAuthorRules): factor matching scoped rules out from matchAuthorRules
+ (WebCore::CSSStyleSelector::matchAuthorRules): add shortcut code
+ * css/CSSStyleSelector.h:
+ (CSSStyleSelector): add matchScopedAuthorRules
+
+2012-01-31 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: DOMDebugger.setEventListenerBreakpoint should accept regular DOM event names.
+ https://bugs.webkit.org/show_bug.cgi?id=77409
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/Inspector.json:
+ * inspector/InspectorDOMDebuggerAgent.cpp:
+ (WebCore::InspectorDOMDebuggerAgent::setEventListenerBreakpoint):
+ (WebCore):
+ (WebCore::InspectorDOMDebuggerAgent::setInstrumentationBreakpoint):
+ (WebCore::InspectorDOMDebuggerAgent::setBreakpoint):
+ (WebCore::InspectorDOMDebuggerAgent::removeEventListenerBreakpoint):
+ (WebCore::InspectorDOMDebuggerAgent::removeInstrumentationBreakpoint):
+ (WebCore::InspectorDOMDebuggerAgent::removeBreakpoint):
+ (WebCore::InspectorDOMDebuggerAgent::pauseOnNativeEventIfNeeded):
+ * inspector/InspectorDOMDebuggerAgent.h:
+ (InspectorDOMDebuggerAgent):
+ * inspector/InspectorInstrumentation.cpp:
+ (WebCore::InspectorInstrumentation::didInstallTimerImpl):
+ (WebCore::InspectorInstrumentation::didRemoveTimerImpl):
+ (WebCore::InspectorInstrumentation::willHandleEventImpl):
+ (WebCore::InspectorInstrumentation::willFireTimerImpl):
+ (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
+ * inspector/InspectorInstrumentation.h:
+ (InspectorInstrumentation):
+ * inspector/front-end/BreakpointsSidebarPane.js:
+ (WebInspector.EventListenerBreakpointsSidebarPane.prototype._setBreakpoint):
+ (WebInspector.EventListenerBreakpointsSidebarPane.prototype._removeBreakpoint):
+
+2012-01-31 Pablo Flouret <pablof@motorola.com>
+
+ Fix compilation errors on build-webkit --debug --no-workers on mac.
+ https://bugs.webkit.org/show_bug.cgi?id=75869
+
+ Reviewed by Adam Barth.
+
+ * WebCore.exp.in:
+
+2012-01-30 Konrad Piascik <kpiascik@rim.com>
+
+ Web Inspector: [BlackBerry] Add BlackBerry UA Strings to the Inspector's UA switcher
+ https://bugs.webkit.org/show_bug.cgi?id=77343
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/front-end/SettingsScreen.js:
+ (WebInspector.SettingsScreen.prototype._createUserAgentSelectRowElement.get const):
+
+2012-01-30 Yury Semikhatsky <yurys@chromium.org>
+
+ Web Inspector: should be possible to step through all event listeners when event listener breakpoint is hit
+ https://bugs.webkit.org/show_bug.cgi?id=77331
+
+ Inspector instrumentation is called before and after each event handler invokation.
+ In case inspector front-end is closed it is no-op, otherwise it may stop execution
+ on an event listener breakpoint.
+
+ Reviewed by Pavel Feldman.
+
+ Test: inspector/debugger/step-through-event-listeners.html
+
+ * dom/EventTarget.cpp:
+ (WebCore::EventTarget::fireEventListeners):
+ * inspector/InspectorInstrumentation.cpp:
+ (WebCore::InspectorInstrumentation::willDispatchEventImpl):
+ (WebCore::InspectorInstrumentation::willHandleEventImpl):
+ (WebCore):
+ (WebCore::InspectorInstrumentation::didHandleEventImpl):
+ (WebCore::InspectorInstrumentation::didDispatchEventImpl):
+ (WebCore::InspectorInstrumentation::willDispatchEventOnWindowImpl):
+ (WebCore::InspectorInstrumentation::didDispatchEventOnWindowImpl):
+ * inspector/InspectorInstrumentation.h:
+ (InspectorInstrumentation):
+ (WebCore::InspectorInstrumentation::willHandleEvent):
+ (WebCore):
+ (WebCore::InspectorInstrumentation::didHandleEvent):
+
+2011-01-30 Hayato Ito <hayato@chromium.org>
+
+ Attach light children after removing a shadow root.
+ https://bugs.webkit.org/show_bug.cgi?id=74267
+
+ 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::removeShadowRoot):
+
+2012-01-30 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r106324.
+ http://trac.webkit.org/changeset/106324
+ https://bugs.webkit.org/show_bug.cgi?id=77406
+
+ Broke CCLayerTreeHostTestLayerOcclusion.runMultiThread and
+ runSingleThread (Requested by yuzo1 on #webkit).
+
+ * platform/graphics/FloatRect.cpp:
+ * platform/graphics/FloatRect.h:
+ (WebCore):
+ * platform/graphics/chromium/Canvas2DLayerChromium.cpp:
+ (WebCore::Canvas2DLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/Canvas2DLayerChromium.h:
+ (WebCore):
+ (Canvas2DLayerChromium):
+ * platform/graphics/chromium/ContentLayerChromium.cpp:
+ (WebCore::ContentLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/ContentLayerChromium.h:
+ (WebCore):
+ (ContentLayerChromium):
+ * platform/graphics/chromium/ImageLayerChromium.cpp:
+ (WebCore::ImageLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/ImageLayerChromium.h:
+ (WebCore):
+ (ImageLayerChromium):
+ * platform/graphics/chromium/LayerChromium.cpp:
+ * platform/graphics/chromium/LayerChromium.h:
+ (WebCore):
+ (WebCore::LayerChromium::paintContentsIfDirty):
+ (LayerChromium):
+ * platform/graphics/chromium/RenderSurfaceChromium.h:
+ * platform/graphics/chromium/TiledLayerChromium.cpp:
+ * platform/graphics/chromium/TiledLayerChromium.h:
+ (WebCore):
+ * platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
+ (WebCore::CCLayerTreeHost::paintContentsIfDirty):
+ (WebCore::CCLayerTreeHost::paintMaskAndReplicaForRenderSurface):
+ (WebCore::CCLayerTreeHost::paintLayerContents):
+ * platform/graphics/chromium/cc/CCLayerTreeHost.h:
+ (WebCore):
+ * platform/graphics/chromium/cc/CCQuadCuller.cpp:
+ (WebCore::enclosedIntRect):
+ (WebCore):
+
+2012-01-30 Raymond Liu <raymond.liu@intel.com>
+
+ Fix ASSERT fail within AudioBus::processWithGainFrom()
+ https://bugs.webkit.org/show_bug.cgi?id=76685
+
+ Reviewed by Daniel Bates.
+
+ Test: webaudio/audionode-connect-order.html
+
+ * webaudio/AudioBasicProcessorNode.cpp:
+ (WebCore::AudioBasicProcessorNode::AudioBasicProcessorNode):
+
+2012-01-30 Roland Steiner <rolandsteiner@chromium.org>
+
+ <style scoped>: Implement scoped stylesheets and basic application
+ https://bugs.webkit.org/show_bug.cgi?id=73190
+
+ Implementing support for <style scoped>:
+ Add a vector stack to CSSStyleSelector that keeps track of encountered scoping elements. This is
+ used for O(1) access to all relevant style sheets for a given element.
+ Adapt matching of author style sheets to also consult appropriate scoped sheets.
+ Finally, prevent style sharing from crossing into/out of scoped style boundaries.
+
+ Reviewed by Antti Koivisto.
+
+ Tests: fast/css/style-scoped/style-scoped-attach.html
+ fast/css/style-scoped/style-scoped-basic.html
+ fast/css/style-scoped/style-scoped-detach.html
+ fast/css/style-scoped/style-scoped-remove-scoped.html
+ fast/css/style-scoped/style-scoped-set-scoped.html
+
+ * css/CSSStyleSelector.cpp:
+ (RuleSet):
+ (WebCore::CSSStyleSelector::CSSStyleSelector): add code for scoped style sheets
+ (WebCore::CSSStyleSelector::collectFeatures): ditto
+ (WebCore):
+ (WebCore::CSSStyleSelector::determineScopingElement): determine whether an author sheet is scoped (and to which scope), or global
+ (WebCore::CSSStyleSelector::scopedRuleSetForElement): returns the RuleSet for the <style scoped> contained by the passed-in element (if any), or 0
+ (WebCore::CSSStyleSelector::appendAuthorStylesheets): add code for scoped style sheets
+ (WebCore::CSSStyleSelector::setupScopingElementStack): determine scoping element ancestors of the given element
+ (WebCore::CSSStyleSelector::pushParent): simplify and refactor SelectorChecker::pushParent, as code in CSStyleSelector needs partial access
+ (WebCore::CSSStyleSelector::popParent): ditto
+ (WebCore::CSSStyleSelector::sortAndTransferMatchedRules): helper function
+ (WebCore::CSSStyleSelector::matchAuthorRules): use AuthorRuleSetIterator to iterate over all relevant RuleSets
+ (WebCore::CSSStyleSelector::matchRules): adapt for scoped style rules
+ (WebCore::CSSStyleSelector::matchAllRules): ditto
+ (WebCore::CSSStyleSelector::locateCousinList): prevent style sharing across scope boundaries
+ (WebCore::CSSStyleSelector::canShareStyleWithElement): ditto
+ (WebCore::CSSStyleSelector::locateSharedStyle): ditto
+ (WebCore::CSSStyleSelector::pseudoStyleForElement): changed call to matchAuthorRules
+ (WebCore::CSSStyleSelector::styleForPage): add comment
+ (WebCore::CSSStyleSelector::checkRegionStyle): add global scope parameter
+ (WebCore::CSSStyleSelector::pseudoStyleRulesForElement): changed call to matchAuthorRules
+ (WebCore::RuleSet::addRulesFromSheet): adapt for scoped style rules
+ * css/CSSStyleSelector.h:
+ (CSSStyleSelector):
+ (WebCore::CSSStyleSelector::ScopeStackFrame::ScopeStackFrame): struct holding an Element pointer and a RuleSet pointer, to be used in a Vector
+ (WebCore::CSSStyleSelector::scopingElementStackIsConsistent): returns if the last seen parent matches the passed-in element
+ * css/SelectorChecker.cpp:
+ (WebCore::SelectorChecker::setupParentStack): set up the parent stack (refactoring)
+ (WebCore::SelectorChecker::pushParent): simplify and refactor
+ * css/SelectorChecker.h:
+ (WebCore::SelectorChecker::popParent): ditto
+ (WebCore::SelectorChecker::parentStackIsEmpty): ditto
+ (WebCore::SelectorChecker::parentStackIsConsistent): make parameter const
+
+2012-01-30 Dana Jansens <danakj@chromium.org>
+
+ [chromium] Compute occlusion during paint loop
+ https://bugs.webkit.org/show_bug.cgi?id=76858
+
+ Reviewed by James Robinson.
+
+ New unit tests in TiledLayerChromiumTest.cpp, CCLayerTreeHostCommonTest.cpp, CCLayerTreeHostTest.cpp
+
+ * platform/graphics/FloatRect.cpp:
+ (WebCore::enclosedIntRect):
+ (WebCore):
+ * platform/graphics/FloatRect.h:
+ (WebCore):
+ * platform/graphics/chromium/Canvas2DLayerChromium.cpp:
+ (WebCore::Canvas2DLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/Canvas2DLayerChromium.h:
+ (Canvas2DLayerChromium):
+ * platform/graphics/chromium/ContentLayerChromium.cpp:
+ (WebCore::ContentLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/ContentLayerChromium.h:
+ (ContentLayerChromium):
+ * platform/graphics/chromium/ImageLayerChromium.cpp:
+ (WebCore::ImageLayerChromium::paintContentsIfDirty):
+ * platform/graphics/chromium/ImageLayerChromium.h:
+ (ImageLayerChromium):
+ * platform/graphics/chromium/LayerChromium.cpp:
+ (WebCore::LayerChromium::contentToScreenSpaceTransform):
+ (WebCore):
+ (WebCore::LayerChromium::addSelfToOccludedScreenSpace):
+ (WebCore::LayerChromium::isPaintedAxisAlignedInScreen):
+ * platform/graphics/chromium/LayerChromium.h:
+ (WebCore):
+ (WebCore::LayerChromium::paintContentsIfDirty):
+ (LayerChromium):
+ * platform/graphics/chromium/RenderSurfaceChromium.h:
+ (RenderSurfaceChromium):
+ * platform/graphics/chromium/TiledLayerChromium.cpp:
+ (WebCore::TiledLayerChromium::addSelfToOccludedScreenSpace):
+ * platform/graphics/chromium/TiledLayerChromium.h:
+ (WebCore):
+ ():
+ * platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
+ (WebCore::CCLayerTreeHost::paintContentsIfDirty):
+ (WebCore::CCLayerTreeHost::paintMaskAndReplicaForRenderSurface):
+ (RenderSurfaceRegion):
+ (WebCore):
+ (WebCore::pushTargetRenderSurfaceRegion):
+ (WebCore::popAndPushTargetRenderSurfaceRegion):
+ (WebCore::CCLayerTreeHost::paintLayerContents):
+ * platform/graphics/chromium/cc/CCLayerTreeHost.h:
+ (WebCore):
+ ():
+ * platform/graphics/chromium/cc/CCQuadCuller.cpp:
+
+2012-01-30 Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
+
+ Add ElementAttributeData class to replace internal uses of NamedNodeMap
+ https://bugs.webkit.org/show_bug.cgi?id=77233
+
+ Reviewed by Andreas Kling.
+
+ Move part of non-DOM functionality of NamedNodeMap into a separate class. This is
+ the first step toward the goal of separating NamedNodeMap from internal attribute
+ storage, as described in https://bugs.webkit.org/show_bug.cgi?id=75069.
+
+ The internal attribute storage is exposed as attributeData() in Element, and when
+ necessary (because it has no back pointer to Element) via methods in Element.
+
+ No new tests. Except from setClass() change this is just moving the code, no new
+ feature was added.
+
+ * CMakeLists.txt:
+ * GNUmakefile.list.am:
+ * Target.pri:
+ * WebCore.gypi:
+ * WebCore.xcodeproj/project.pbxproj:
+ * dom/DOMAllInOne.cpp:
+ * dom/Element.cpp:
+ (WebCore::Element::idAttributeChanged):
+ * dom/Element.h:
+ (WebCore::Element::attributeData):
+ (WebCore::Element::ensureAttributeData):
+ (Element):
+ (WebCore::Element::idForStyleResolution):
+ * dom/ElementAttributeData.cpp: Added.
+ (WebCore):
+ (WebCore::ElementAttributeData::setClass): the only caller of this function
+ already deal with the case when the element has no class, so don't do it here.
+ * dom/ElementAttributeData.h: Added.
+ (WebCore):
+ (ElementAttributeData):
+ (WebCore::ElementAttributeData::clearClass):
+ (WebCore::ElementAttributeData::classNames):
+ (WebCore::ElementAttributeData::idForStyleResolution):
+ (WebCore::ElementAttributeData::setIdForStyleResolution):
+ (WebCore::ElementAttributeData::ElementAttributeData):
+ * dom/NamedNodeMap.cpp:
+ (WebCore::NamedNodeMap::clearAttributes):
+ * dom/NamedNodeMap.h:
+ (WebCore::NamedNodeMap::attributeData):
+ (NamedNodeMap):
+ * dom/StyledElement.cpp:
+ (WebCore::StyledElement::classAttributeChanged):
+ * dom/StyledElement.h:
+ (WebCore::StyledElement::classNames):
+ * html/ClassList.cpp:
+ (WebCore::ClassList::classNames):
+
+2012-01-30 Gregg Tavares <gman@chromium.org>
+
+ Add Plumming to get graphics error messages to JS Console
+ https://bugs.webkit.org/show_bug.cgi?id=77238
+
+ Reviewed by Kenneth Russell.
+
+ No new tests. No change in behavior.
+
+ * html/canvas/WebGLRenderingContext.cpp:
+ (WebGLRenderingContextErrorMessageCallback):
+ (WebCore::WebGLRenderingContextErrorMessageCallback::WebGLRenderingContextErrorMessageCallback):
+ (WebCore::WebGLRenderingContextErrorMessageCallback::onErrorMessage):
+ (WebCore::WebGLRenderingContextErrorMessageCallback::~WebGLRenderingContextErrorMessageCallback):
+ (WebCore):
+ (WebCore::WebGLRenderingContext::create):
+ (WebCore::WebGLRenderingContext::initializeNewContext):
+ (WebCore::WebGLRenderingContext::~WebGLRenderingContext):
+ * html/canvas/WebGLRenderingContext.h:
+ (WebGLRenderingContext):
+ * platform/graphics/GraphicsContext3D.h:
+ (ErrorMessageCallback):
+ (WebCore::GraphicsContext3D::ErrorMessageCallback::~ErrorMessageCallback):
+ (GraphicsContext3D):
+ * platform/graphics/cairo/GraphicsContext3DCairo.cpp:
+ (WebCore::GraphicsContext3D::setErrorMessageCallback):
+ (WebCore):
+ * platform/graphics/efl/GraphicsContext3DEfl.cpp:
+ (WebCore::GraphicsContext3D::setErrorMessageCallback):
+ (WebCore):
+ * platform/graphics/qt/GraphicsContext3DQt.cpp:
+ (WebCore::GraphicsContext3D::setErrorMessageCallback):
+ (WebCore):
+
2012-01-30 Anders Carlsson <andersca@apple.com>
+ Simplify RenderLayerCompositor::frameViewDidScroll
+ https://bugs.webkit.org/show_bug.cgi?id=77398
+
+ Reviewed by Sam Weinig.
+
+ Remove the scrollPosition parameter from RenderLayerCompositor::frameViewDidScroll; we can just get it
+ from the associated FrameView object.
+
+ * page/FrameView.cpp:
+ (WebCore::FrameView::scrollPositionChanged):
+ * rendering/RenderLayerCompositor.cpp:
+ (WebCore::RenderLayerCompositor::frameViewDidChangeSize):
+ Call frameViewDidScroll() here instead of setting the scroll layer position explicitly.
+
+ (WebCore::RenderLayerCompositor::frameViewDidScroll):
+ (WebCore::RenderLayerCompositor::ensureRootLayer):
+ * rendering/RenderLayerCompositor.h:
+ (RenderLayerCompositor):
+
+2012-01-30 Pablo Flouret <pablof@motorola.com>
+
+ Don't select the next selectable index when deselecting an option in select elements with size set to a value greater than one.
+ https://bugs.webkit.org/show_bug.cgi?id=76389
+
+ Reviewed by Kent Tamura.
+
+ This behavior matches the rest of the browsers.
+
+ Test: fast/forms/select/option-selecting.html
+
+ * html/HTMLSelectElement.cpp:
+ (WebCore::HTMLSelectElement::optionSelectionStateChanged):
+
+2012-01-30 Anders Carlsson <andersca@apple.com>
+
+ Scrollbars don't show when scrolling on the scrolling thread
+ https://bugs.webkit.org/show_bug.cgi?id=77396
+ <rdar://problem/10710736>
+
+ Reviewed by Sam Weinig.
+
+ Use ScrollAnimator::scrollToOffsetWithoutAnimation when updating the frame view scroll offset,
+ since that function will end up invalidating the scrollbars correctly.
+
+ * page/ScrollingCoordinator.cpp:
+ (WebCore::ScrollingCoordinator::didUpdateMainFrameScrollPosition):
+
+2012-01-30 Anders Carlsson <andersca@apple.com>
+
Show repaint counters in individual tiles
https://bugs.webkit.org/show_bug.cgi?id=77390
<rdar://problem/10767967>
« no previous file with comments | « no previous file | Source/WebCore/platform/graphics/chromium/cc/CCVideoLayerImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698