Index: Source/WebCore/ChangeLog |
=================================================================== |
--- Source/WebCore/ChangeLog (revision 98584) |
+++ Source/WebCore/ChangeLog (working copy) |
@@ -1,3724 +1,3 @@ |
-2011-10-26 Dimitri Glazkov <dglazkov@chromium.org> |
- |
- REGRESSION (r94887): Scrolling the HTML spec is more jerky now than it was (regression) |
- https://bugs.webkit.org/show_bug.cgi?id=70857 |
- |
- Revert r94887, because it regressed performance. |
- |
- Rubber-stamped by Antti Koivisto. |
- |
- * css/CSSStyleSelector.cpp: |
- (WebCore::CSSStyleSelector::canShareStyleWithElement): |
- (WebCore::parentStylePreventsSharing): |
- * css/SelectorChecker.cpp: |
- (WebCore::SelectorChecker::checkSelector): |
- * dom/Element.cpp: |
- (WebCore::Element::recalcStyle): |
- (WebCore::checkForSiblingStyleChanges): |
- * rendering/style/RenderStyle.cpp: |
- (WebCore::RenderStyle::RenderStyle): |
- * rendering/style/RenderStyle.h: |
- (WebCore::InheritedFlags::childrenAffectedByDirectAdjacentRules): |
- (WebCore::InheritedFlags::setChildrenAffectedByDirectAdjacentRules): |
- |
-2011-10-26 Alexander Pavlov <apavlov@chromium.org> |
- |
- Web Inspector: Need workaround for the red crossed circle in the status bar not bringing up the console when clicked |
- https://bugs.webkit.org/show_bug.cgi?id=70928 |
- |
- Reviewed by Pavel Feldman. |
- |
- * inspector/front-end/inspector.css: |
- (#error-count-img): |
- (#error-count + #warning-count-img): |
- (#warning-count-img): |
- * inspector/front-end/inspector.js: |
- (WebInspector._updateErrorAndWarningCounts): |
- |
-2011-10-26 Mike Reed <reed@google.com> |
- |
- [skia] replace offscreen technique with native support for antialiased clipping |
- https://bugs.webkit.org/show_bug.cgi?id=70748 |
- |
- Reviewed by Stephen White. |
- |
- ~100 layout tests need to be rebaselined, as the native aa-clipping differs sometimes in the low |
- bits of the result. Other than that, the results should be the same, except that now the drawing |
- can go directly to the canvas, rather than being direct offscreen and then have that result |
- "clipped" during the restore. This has the effect of allowing LCD text to be drawing inside a |
- antialiased clip area. |
- |
- * platform/graphics/skia/PlatformContextSkia.cpp: |
- (WebCore::PlatformContextSkia::State::State): |
- (WebCore::PlatformContextSkia::State::cloneInheritedProperties): |
- (WebCore::PlatformContextSkia::clipPathAntiAliased): |
- (WebCore::PlatformContextSkia::restore): |
- |
-2011-10-26 Nikolas Zimmermann <nzimmermann@rim.com> |
- |
- CSS 2.1 failure: background-intrinsic-* |
- https://bugs.webkit.org/show_bug.cgi?id=47156 |
- |
- SVGs do not work as tiled background images |
- https://bugs.webkit.org/show_bug.cgi?id=16281 |
- |
- Apply preserveAspectRatio and synthesize viewboxes in <img> |
- https://bugs.webkit.org/show_bug.cgi?id=34521 |
- |
- SVG background doesn't resize properly when dimensions are changed |
- https://bugs.webkit.org/show_bug.cgi?id=42944 |
- |
- Images with percent height inside a floated div should use intrinsic height. |
- https://bugs.webkit.org/show_bug.cgi?id=45439 |
- |
- SVG image in HTML changes size as the window is resized |
- https://bugs.webkit.org/show_bug.cgi?id=52045 |
- |
- Reviewed by Antti Koivisto. |
- |
- Implement intrinsic sizing support for SVGImage (svg embedded through <html:img>/<svg:image>/background-image/border-image/...). |
- This is demanded by CSS 2.1, and covered by new layout tests in LayoutTests/css2.1 and several new custom testcases. |
- |
- Tests: css2.1/20110323/background-intrinsic-001.htm |
- css2.1/20110323/background-intrinsic-002.htm |
- css2.1/20110323/background-intrinsic-003.htm |
- css2.1/20110323/background-intrinsic-004.htm |
- css2.1/20110323/background-intrinsic-005.htm |
- css2.1/20110323/background-intrinsic-006.htm |
- css2.1/20110323/background-intrinsic-007.htm |
- css2.1/20110323/background-intrinsic-008.htm |
- css2.1/20110323/background-intrinsic-009.htm |
- svg/as-background-image/background-image-preserveaspectRatio-support.html (adapted from testcase from bug 34521) |
- svg/as-background-image/background-image-tiled.html (reduction from bug 16281) |
- svg/as-background-image/same-image-two-instances-background-image.html |
- svg/as-image/img-preserveAspectRatio-support-1.html (reduction from bug 34521) |
- svg/as-image/same-image-two-instances.html |
- svg/as-image/svg-as-relative-image-with-explicit-size.html |
- svg/as-image/svg-image-change-content-size.xhtml (reduction from bug 42944) |
- svg/zoom/page/zoom-img-preserveAspectRatio-support-1.html |
- |
- * loader/cache/CachedImage.cpp: Enable SVGImage <-> IntSize cache. |
- (WebCore::CachedImage::lookupImageForSize): Use recently introduced ImageBySizeCache, to lookup an image for a certain size. |
- (WebCore::CachedImage::lookupImageForRenderer): Lookup image by renderer, which first looks up a size for a renderer, then uses lookupImageForSize(). |
- (WebCore::createSVGImage): Refactored from createImage(), contains the SVGImage creation part only. |
- (WebCore::CachedImage::lookupOrCreateImageForRenderer): Use recently introduced ImageBySizeCache, to dynamically create copies of m_image if needed. |
- (WebCore::CachedImage::setContainerSizeForRenderer): For SVGImages, pass on container size handling to ImageBySizeCache. |
- (WebCore::CachedImage::imageSizeForRenderer): Figure out the image size, respecting per-renderer overrides, for a certain renderer. |
- (WebCore::CachedImage::computeIntrinsicDimensions): Remove unnecessary RenderObject parameter. |
- (WebCore::CachedImage::addClientForRenderer): Special variant of addClient(), overriding the existing in CachedResource. |
- (WebCore::CachedImage::removeClientForRenderer): Special variant of removeClient(), that also clears the image in the ImageBySizeCache. |
- (WebCore::CachedImage::createImage): Refactor SVGImage creation into createSVGImage() free function, to be useable from lookupOrCreateImageForRenderer(). |
- * loader/cache/CachedImage.h: Expose removeClientForRenderer(). |
- * page/ChromeClient.h: |
- (WebCore::ChromeClient::isSVGImageChromeClient): Used to identify whether a RenderSVGRoot is embedded through a SVGImage. Returns false, by default. |
- * rendering/ImageBySizeCache.cpp: |
- (WebCore::ImageBySizeCache::addClient): Assert the passed renderer is valid. |
- (WebCore::ImageBySizeCache::removeClient): Ditto. Allow removeClient() to be called w/o prio addClient() usage. |
- (WebCore::ImageBySizeCache::setClient): New helper function, that combines the usage of addClient/removeClient, for the use in CachedImage. |
- (WebCore::ImageBySizeCache::imageForSize): Respect empty sizes, just return 0, instead of asserting. |
- (WebCore::ImageBySizeCache::imageForRenderer): Added a helper that retrieves an image for a renderer, by lookup up its size and using imageForSize(). |
- * rendering/ImageBySizeCache.h: Expose setClient() & imageForRenderer(). |
- * rendering/RenderBoxModelObject.cpp: Implement CSS 2.1 intrinsic size negotiation for images. |
- (WebCore::resolveWidthForRatio): New inline helper function used bz calculateImageIntrinsicDimensions. |
- (WebCore::resolveHeightForRatio): Ditto. |
- (WebCore::resolveAgainstIntrinsicWidthOrHeightAndRatio): Ditto. |
- (WebCore::resolveAgainstIntrinsicRatio): Ditto. |
- (WebCore::RenderBoxModelObject::calculateImageIntrinsicDimensions): New helper function, containing the main algorithm, which is a pure transcription of the spec. |
- (WebCore::RenderBoxModelObject::calculateFillTileSize): Use new calculateImageIntrinsicDimensions() helper to figure out the intrinsic size. |
- (WebCore::RenderBoxModelObject::calculateBackgroundImageGeometry): Propagate calculateFillTileSize() result to the image resource, via setContainerSizeForRenderer(). |
- (WebCore::RenderBoxModelObject::paintNinePieceImage): Use new calculateImageIntrinsicDimensions() helper to figure out the intrinsic size. |
- * rendering/RenderBoxModelObject.h: Clarify some variable names, added calculateImageIntrinsicDimensions(). |
- * rendering/RenderImage.cpp: |
- (WebCore::RenderImage::RenderImage): Use IntSize(), instead of IntSize(0, 0). |
- (WebCore::RenderImage::updateIntrinsicSizeIfNeeded): Refactored from imageDimensionsChanged(). |
- (WebCore::RenderImage::imageDimensionsChanged): Use updateIntrinsicSizeIfNeeded(). |
- (WebCore::RenderImage::computeReplacedLogicalWidth): Use RenderReplaced::computeReplacedLogicalWidth() exclusively. For this to work, the intrinsic size must be correct. |
- (WebCore::RenderImage::computeIntrinsicRatioInformation): Default implementation for non-SVGImages. |
- (WebCore::RenderImage::needsPreferredWidthsRecalculation): Return true, just like RenderPart, if embeddedContentBox is not null. |
- (WebCore::RenderImage::embeddedContentBox): Returns the RenderSVGRoot* renderer of the embedded SVG, if possible. |
- * rendering/RenderImage.h: Remove isLogicalWidth/HeightSpecified() / computeReplacedLogicalHeight() / calcAspectRatioLogicalWidth/Height(). |
- * rendering/RenderImageResource.cpp: |
- (WebCore::RenderImageResource::setContainerSizeForRenderer): Pass around new "float containerZoomFactor" parameter. |
- * rendering/RenderImageResourceStyleImage.cpp: |
- (WebCore::RenderImageResourceStyleImage::image): embeddedContentBox() is now calling image() earlier than before. We now have to handle the case that the image is pending. |
- (WebCore::RenderImageResourceStyleImage::setContainerSizeForRenderer): Pass zoom factor. |
- * rendering/RenderImageResourceStyleImage.h: |
- * rendering/RenderListMarker.cpp: |
- (WebCore::RenderListMarker::computePreferredLogicalWidths): Pass effective zoom to setContainerSizeForRenderer(). |
- * rendering/RenderReplaced.cpp: |
- (WebCore::RenderReplaced::computeIntrinsicLogicalWidth): Generalized this code, as RenderImage is using it as well now. Marginal changes needed. |
- (WebCore::RenderReplaced::computeIntrinsicLogicalHeight): Ditto. |
- (WebCore::RenderReplaced::computeReplacedLogicalWidth): Ditto. |
- * rendering/style/StyleCachedImage.cpp: |
- (WebCore::StyleCachedImage::computeIntrinsicDimensions): Stop passing m_renderer to CachedImage, it's no longer needed. |
- (WebCore::StyleCachedImage::setContainerSizeForRenderer): Add "float containerZoomFactor" parameter. |
- (WebCore::StyleCachedImage::addClient): Forward to new addClientForRenderer(). |
- (WebCore::StyleCachedImage::removeClient): Call new removeClientForRenderer() instead of removeClient(), so the ImageBySizeCache is also updated. |
- * rendering/style/StyleCachedImage.h: Add "float containerZoomFactor" parameter to setContainerSizeForRenderer. |
- * rendering/style/StyleGeneratedImage.h: |
- (WebCore::StyleGeneratedImage::setContainerSizeForRenderer): Ditto. |
- * rendering/style/StyleImage.h: Ditto. |
- * rendering/style/StylePendingImage.h: |
- (WebCore::StylePendingImage::setContainerSizeForRenderer): Ditto. |
- * rendering/svg/RenderSVGImage.cpp: |
- (WebCore::RenderSVGImage::layout): Always supply a container size when embedding SVGs in <svg:image>. |
- * rendering/svg/RenderSVGRoot.cpp: Move "override container size" from SVGSVGElement into RenderSVGRoot, where it belongs. |
- (WebCore::RenderSVGRoot::isEmbeddedThroughImageElement): Add helper method to determine whether we're loaded through SVGImage. |
- (WebCore::RenderSVGRoot::computeReplacedLogicalWidth): If we have a valid container size, it has precendence (only supplied via external SVGImages). |
- (WebCore::RenderSVGRoot::computeReplacedLogicalHeight): Ditto. |
- (WebCore::RenderSVGRoot::layout): Remove calcViewport() usage, no need to track/override the viewport size anymore, all done in coputeReplacedLogical* now. |
- (WebCore::RenderSVGRoot::paint): Use borderBoxRect() which now always matches the previously computed m_viewportSize. |
- (WebCore::RenderSVGRoot::computeRectForRepaint): Ditto. |
- * rendering/svg/RenderSVGRoot.h: Move "override container size" from SVGSVGElement into RenderSVGRoot, where it belongs. |
- (WebCore::RenderSVGRoot::containerSize): |
- (WebCore::RenderSVGRoot::setContainerSize): |
- * svg/SVGLength.cpp: |
- (WebCore::SVGLength::determineViewport): Resolve lengths against override container size in documents embedded through SVGImage. |
- * svg/SVGSVGElement.cpp: Remove "override container size" handling from SVGSVGElement. |
- (WebCore::SVGSVGElement::SVGSVGElement): Ditto. |
- (WebCore::SVGSVGElement::currentViewBoxRect): Always synthesize a viewBox, if we're embedded through SVGImage, as demanded by SVG 1.1 2nd Edition. |
- * svg/SVGSVGElement.h: |
- * svg/graphics/SVGImage.cpp: |
- (WebCore::SVGImageChromeClient::setObserver): Helper function. |
- (WebCore::SVGImageChromeClient::isSVGImageChromeClient): Return true, used to identify whether RenderSVGRoot is embedded through SVGImage. |
- (WebCore::SVGImage::setContainerZoom): Forwarded to Page::setPageZoomFactor. |
- (WebCore::SVGImage::setContainerSize): Assert that container size is not empty. We should never receive an empty container size. |
- (WebCore::SVGImage::usesContainerSize): Adapt to override container size changes, it now lives in RenderSVGRoot instead of SVGSVGElement. |
- (WebCore::SVGImage::size): New algorithm to figure out the size of an embedded SVG, as demanded by the CSS/SVG specs. |
- (WebCore::SVGImage::draw): Stop calling layout() while painting! |
- (WebCore::SVGImage::embeddedContentBox): Add helper which returns the RenderSVGRoot of the document. |
- (WebCore::SVGImage::computeIntrinsicDimensions): Implement intrinsic ratio calculation. |
- (WebCore::SVGImage::dataChanged): Force calling FrameView::setCanHaveScrollbars(false), as SVG images now always synthesize a viewBox, and thus never receive scrollbars. |
- * svg/graphics/SVGImage.h: |
- |
-2011-10-26 Kenneth Rohde Christiansen <kenneth@webkit.org> |
- |
- Properly suspend/resume Geolocation/DeviceMotion/DeviceOrientation objects |
- https://bugs.webkit.org/show_bug.cgi?id=70328 |
- |
- Reviewed by Simon Hausmann. |
- |
- Also remove checks for m_client in DeviceMotionController as it can never be null. |
- |
- No new tests, as the suspend/resume functionality is not fully working yet. |
- |
- * dom/DeviceMotionController.cpp: |
- (WebCore::DeviceMotionController::timerFired): |
- (WebCore::DeviceMotionController::addListener): |
- (WebCore::DeviceMotionController::removeListener): |
- (WebCore::DeviceMotionController::removeAllListeners): |
- (WebCore::DeviceMotionController::suspend): |
- (WebCore::DeviceMotionController::resume): |
- * dom/DeviceMotionController.h: |
- * dom/DeviceOrientationController.cpp: |
- (WebCore::DeviceOrientationController::suspend): |
- (WebCore::DeviceOrientationController::resume): |
- * dom/DeviceOrientationController.h: |
- * dom/Document.cpp: |
- (WebCore::Document::suspendActiveDOMObjects): |
- (WebCore::Document::resumeActiveDOMObjects): |
- (WebCore::Document::stopActiveDOMObjects): |
- * dom/Document.h: |
- * dom/ScriptExecutionContext.h: |
- * page/GeolocationController.cpp: |
- (WebCore::GeolocationController::GeolocationController): |
- (WebCore::GeolocationController::removeObserver): |
- (WebCore::GeolocationController::suspend): |
- (WebCore::GeolocationController::resume): |
- * page/GeolocationController.h: |
- |
-2011-10-26 Pavel Feldman <pfeldman@google.com> |
- |
- Not reviewed: follow up to 98236 - moved inspector settings initialization earlier to unbreak settings panel. |
- |
- * inspector/front-end/TextEditorModel.js: |
- (WebInspector.TextEditorModel): |
- |
-2011-10-26 Zalan Bujtas <zbujtas@gmail.com> |
- |
- Name viewport change event consistently. |
- https://bugs.webkit.org/show_bug.cgi?id=70901 |
- |
- Reviewed by Kenneth Rohde Christiansen. |
- |
- Use ViewportPropertiesDidChange term consistently throughout WebKit. |
- |
- No tests needed as the change is only method renaming. |
- |
- * page/Chrome.cpp: |
- (WebCore::Chrome::dispatchViewportPropertiesDidChange): |
- * page/Chrome.h: |
- * page/ChromeClient.h: |
- (WebCore::ChromeClient::dispatchViewportPropertiesDidChange): |
- * page/Page.cpp: |
- (WebCore::Page::updateViewportArguments): |
- |
-2011-10-25 Stephen White <senorblanco@chromium.org> |
- |
- [chromium] Canvas2D should rate-limit drawing to prevent swamping the GPU process. |
- https://bugs.webkit.org/show_bug.cgi?id=70367 |
- |
- Reviewed by James Robinson. |
- |
- Sadly, we don't have infrastructure to test this kind of GPU swamping |
- yet. |
- |
- * WebCore.gypi: |
- Add RateLimiter.* to the Chromium build. |
- * platform/graphics/chromium/Canvas2DLayerChromium.cpp: |
- (WebCore::Canvas2DLayerChromium::contentChanged): |
- When the layer is notified that the contents have changed, ping the |
- rate limiter. |
- * platform/graphics/chromium/Canvas2DLayerChromium.h: |
- * platform/graphics/chromium/GraphicsLayerChromium.cpp: |
- (WebCore::GraphicsLayerChromium::setContentsNeedsDisplay): |
- Call (new) virtual contentChanged() on a layer when its contents have |
- been changed (e.g., by a drawing call). |
- * platform/graphics/chromium/LayerChromium.h: |
- (WebCore::LayerChromium::contentChanged): |
- New virtual contentChanged(). |
- * platform/graphics/chromium/RateLimiter.cpp: Added. |
- (WebCore::RateLimiter::create): |
- Rate limiter factory function. |
- (WebCore::RateLimiter::RateLimiter): |
- (WebCore::RateLimiter::start): |
- Public API to start rate limiting a context. |
- (WebCore::RateLimiter::stop): |
- Public API to stop rate limiting a context. |
- (WebCore::RateLimiter::rateLimitContext): |
- Internal timer callback when a context should be rate limited. |
- * platform/graphics/chromium/RateLimiter.h: Added. |
- * platform/graphics/chromium/WebGLLayerChromium.cpp: |
- (WebCore::WebGLLayerChromium::WebGLLayerChromium): |
- Remove rate limiting timer and extension check (moved to RateLimiter). |
- (WebCore::WebGLLayerChromium::contentChanged): |
- The function formerly known as setTextureUpdated(), now renamed to |
- match the base class virtual contentChanged(). Call rate limiter in |
- CCLayerTreeHost (local implementation removed). |
- (WebCore::WebGLLayerChromium::setContext): |
- When the context is changed, stop the pending rate limiter on the old |
- context. Remove extension check (moved to RateLimiter). |
- * platform/graphics/chromium/WebGLLayerChromium.h: |
- Remove rate limiting timer and extension check (moved to RateLimiter). |
- * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: |
- (WebCore::CCLayerTreeHost::startRateLimiter): |
- Public API for starting per-context rate limiter. |
- (WebCore::CCLayerTreeHost::stopRateLimiter): |
- Public API for stopping per-context rate limiter. |
- * platform/graphics/chromium/cc/CCLayerTreeHost.h: |
- Implementation of per-GraphicsContext3D RateLimiter. |
- |
- |
-2011-10-26 Pavel Feldman <pfeldman@chromium.org> |
- |
- Not reviewed: fixing inspector extensions tests. |
- |
- * inspector/front-end/ConsoleMessage.js: |
- (WebInspector.ConsoleMessageImpl): |
- * inspector/front-end/ExtensionServer.js: |
- (WebInspector.ExtensionServer.prototype._onAddConsoleMessage): |
- |
-2011-10-26 Balazs Kelemen <kbalazs@webkit.org> |
- |
- ParallelJobs path of FEConvolveMatrix is erroneous |
- https://bugs.webkit.org/show_bug.cgi?id=70409 |
- |
- Reviewed by Zoltan Herczeg. |
- |
- Covered by existing tests. |
- |
- * platform/graphics/filters/FEConvolveMatrix.cpp: |
- (WebCore::FEConvolveMatrix::fastSetInteriorPixels): |
- Reject the silly idea that bytes per pixel ratio is 3 in preserveAlphaValues mode. |
- The bug was hidden so far because that computation has no effect when we are going |
- with the full inferior area in one round (in that case clipBottom and yEnd are equals |
- and the value of the expression is always 0). With ParallelJobs we process sub-areas |
- on each thread which has triggered the issue. |
- |
-2011-10-26 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r98393. |
- http://trac.webkit.org/changeset/98393 |
- https://bugs.webkit.org/show_bug.cgi?id=70892 |
- |
- "Compilation is broken on Chromium Webkit Mac Builder (dbg) " |
- (Requested by yurys on #webkit). |
- |
- * platform/graphics/chromium/LayerChromium.h: |
- (WebCore::LayerChromium::setMasksToBounds): |
- (WebCore::LayerChromium::setMaskLayer): |
- |
-2011-10-26 Pavel Feldman <pfeldman@google.com> |
- |
- Web Inspector: order console message parameters for better optional parameters handling. |
- https://bugs.webkit.org/show_bug.cgi?id=70809 |
- |
- Reviewed by Yury Semikhatsky. |
- |
- * inspector/Inspector.json: |
- * inspector/front-end/ConsoleMessage.js: |
- (WebInspector.ConsoleMessageImpl): |
- (WebInspector.ConsoleMessageImpl.prototype._formatMessage): |
- * inspector/front-end/ConsoleModel.js: |
- (WebInspector.ConsoleModel.prototype._messageRepeatCountUpdated): |
- (WebInspector.ConsoleMessage.create): |
- (WebInspector.ConsoleDispatcher.prototype.messageAdded): |
- * inspector/front-end/ConsoleView.js: |
- (WebInspector.ConsoleCommandResult): |
- (WebInspector.ConsoleMessage.create): |
- * inspector/front-end/ExtensionServer.js: |
- (WebInspector.ExtensionServer.prototype._onAddConsoleMessage): |
- * inspector/front-end/NetworkManager.js: |
- (WebInspector.NetworkDispatcher.prototype._updateResourceWithResponse): |
- * inspector/front-end/inspector.js: |
- (WebInspector.log.logMessage): |
- (WebInspector.log): |
- |
-2011-10-26 Vsevolod Vlasov <vsevik@chromium.org> |
- |
- Web Inspector: Debugger fails when there is an invalid watch expression. |
- https://bugs.webkit.org/show_bug.cgi?id=70718 |
- |
- Reviewed by Pavel Feldman. |
- |
- Test: inspector/debugger/watch-expressions-panel-switch.html |
- |
- * inspector/front-end/ScriptsPanel.js: |
- (WebInspector.ScriptsPanel.prototype.evaluateInSelectedCallFrame): |
- |
-2011-10-26 Fady Samuel <fsamuel@chromium.org> |
- |
- Page Scale Factor broken when navigating history on pages with child frames |
- https://bugs.webkit.org/show_bug.cgi?id=70459 |
- |
- Reviewed by Darin Fisher. |
- |
- With frameScaleFactor now always returning 1.0 for subframes and pageScaleFactor for the mainFrame, |
- and there being only a single pageScaleFactor, history for scaling is broken. Scaling history is |
- saved on a per frame basis but restored, overriding the per-page pageScaleFactor multiple times. |
- As a result, sometimes pages that have subframes end up getting a scale factor of 1.0 instead |
- of the correct scale factor that was assigned to the main frame. |
- |
- No new tests because I don't know how to test this. |
- |
- * loader/HistoryController.cpp: |
- (WebCore::HistoryController::restoreScrollPositionAndViewState): |
- |
-2011-10-26 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r98429. |
- http://trac.webkit.org/changeset/98429 |
- https://bugs.webkit.org/show_bug.cgi?id=70881 |
- |
- "webkit_gpu_tests fail in Chromium" (Requested by yurys on |
- #webkit). |
- |
- * platform/graphics/chromium/LayerRendererChromium.cpp: |
- (WebCore::LayerRendererChromium::initialize): |
- * platform/graphics/chromium/WebGLLayerChromium.cpp: |
- (WebCore::WebGLLayerChromium::layerRendererContext): |
- * platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp: |
- (WebCore::CCHeadsUpDisplay::enabled): |
- * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: |
- (WebCore::CCLayerTreeHost::initialize): |
- (WebCore::CCLayerTreeHost::context): |
- (WebCore::CCLayerTreeHost::setNeedsAnimate): |
- (WebCore::CCLayerTreeHost::setNeedsCommitThenRedraw): |
- (WebCore::CCLayerTreeHost::setNeedsRedraw): |
- (WebCore::CCLayerTreeHost::composite): |
- * platform/graphics/chromium/cc/CCLayerTreeHost.h: |
- (WebCore::CCSettings::CCSettings): |
- * platform/graphics/chromium/cc/CCThreadProxy.cpp: |
- (WebCore::CCThreadProxy::setThread): |
- * platform/graphics/chromium/cc/CCThreadProxy.h: |
- |
-2011-10-26 Ben Wells <benwells@chromium.org> |
- |
- Canvas drawImage with SourceIn, DestinationIn, SourceOut, DestinationAtop and Copy have errors |
- https://bugs.webkit.org/show_bug.cgi?id=66920 |
- |
- Reviewed by James Robinson. |
- |
- Test: fast/canvas/canvas-composite-image.html |
- |
- Use the same method as fills to handle composited drawImage calls that will affect the whole |
- canvas. To do this code used when doing fills has been factored into seperate functions and |
- used by drawImage. Some functions used to do these types of operations have also been renamed. |
- |
- * html/canvas/CanvasRenderingContext2D.cpp: |
- (WebCore::isFullCanvasCompositeMode): |
- (WebCore::CanvasRenderingContext2D::fill): |
- (WebCore::CanvasRenderingContext2D::fillRect): |
- (WebCore::CanvasRenderingContext2D::drawImage): |
- (WebCore::CanvasRenderingContext2D::calculateCompositingBufferRect): |
- (WebCore::CanvasRenderingContext2D::createCompositingBuffer): |
- (WebCore::CanvasRenderingContext2D::compositeBuffer): |
- (WebCore::CanvasRenderingContext2D::fullCanvasCompositedDrawImage): |
- (WebCore::CanvasRenderingContext2D::fullCanvasCompositedFill): |
- * html/canvas/CanvasRenderingContext2D.h: |
- |
-2011-10-25 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r98379. |
- http://trac.webkit.org/changeset/98379 |
- https://bugs.webkit.org/show_bug.cgi?id=70875 |
- |
- Did not pass on JSC ports (Requested by abarth on #webkit). |
- |
- * bindings/scripts/CodeGeneratorJS.pm: |
- (GenerateHeader): |
- (GenerateImplementation): |
- * bindings/scripts/CodeGeneratorV8.pm: |
- (GenerateHeader): |
- (GenerateArgumentsCountCheck): |
- (GenerateImplementation): |
- * html/canvas/CanvasRenderingContext2D.idl: |
- * page/DOMWindow.idl: |
- |
-2011-10-25 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r98346. |
- http://trac.webkit.org/changeset/98346 |
- https://bugs.webkit.org/show_bug.cgi?id=70874 |
- |
- Breaks JSC-based bots. (Requested by pfeldman on #webkit). |
- |
- * inspector/front-end/ScriptsPanel.js: |
- (WebInspector.ScriptsPanel.prototype.evaluateInSelectedCallFrame): |
- |
-2011-10-25 Nat Duca <nduca@chromium.org> |
- |
- [chromium] Enable threaded compositing via CCThreadProxy::hasThread only |
- https://bugs.webkit.org/show_bug.cgi?id=70838 |
- |
- Reviewed by James Robinson. |
- |
- * platform/graphics/chromium/LayerRendererChromium.cpp: |
- (WebCore::LayerRendererChromium::initialize): |
- * platform/graphics/chromium/WebGLLayerChromium.cpp: |
- (WebCore::WebGLLayerChromium::layerRendererContext): |
- * platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp: |
- (WebCore::CCHeadsUpDisplay::enabled): |
- * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: |
- (WebCore::CCLayerTreeHost::initialize): |
- (WebCore::CCLayerTreeHost::context): |
- (WebCore::CCLayerTreeHost::setNeedsAnimate): |
- (WebCore::CCLayerTreeHost::setNeedsCommitThenRedraw): |
- (WebCore::CCLayerTreeHost::setNeedsRedraw): |
- (WebCore::CCLayerTreeHost::composite): |
- * platform/graphics/chromium/cc/CCLayerTreeHost.h: |
- (WebCore::CCSettings::CCSettings): |
- * platform/graphics/chromium/cc/CCThreadProxy.cpp: |
- (WebCore::CCThreadProxy::hasThread): |
- * platform/graphics/chromium/cc/CCThreadProxy.h: |
- |
-2011-10-25 Xiaomei Ji <xji@chromium.org> |
- |
- --webkit-visual-word does not work in multi-line |
- https://bugs.webkit.org/show_bug.cgi?id=61344 |
- |
- Reviewed by Ryosuke Niwa. |
- |
- Get next or previous root inline box (which is not in the same |
- render object as the current root inline box) by traversing DOM node. |
- |
- Test: editing/selection/move-by-word-visually-inline-block-positioned-element.html |
- |
- * editing/visible_units.cpp: |
- (WebCore::previousRootInlineBox): |
- (WebCore::nextRootInlineBox): |
- (WebCore::leftInlineBox): |
- (WebCore::rightInlineBox): |
- |
-2011-10-25 Adam Barth <abarth@webkit.org> |
- |
- JSEventTarget.cpp has a bunch of unnessary includes |
- https://bugs.webkit.org/show_bug.cgi?id=70865 |
- |
- Reviewed by Eric Seidel. |
- |
- In the process of removing these includes, I noticed we had a bunch of |
- non-autogenerated code in this file, which I've now autogenerated. |
- |
- * bindings/js/JSEventTarget.cpp: |
- (WebCore::toEventTarget): |
- - One subtly here is the outter static_cast, which is caused by an |
- inheritance infelicity in JavaScriptAudioNode, which I've noted |
- with a FIXME. In any case, the extra static_cast shouldn't cause |
- any trouble. |
- * webaudio/JavaScriptAudioNode.h: |
- |
-2011-10-25 Adam Barth <abarth@webkit.org> |
- |
- V8DOMWrapper.cpp has unneeded header includes |
- https://bugs.webkit.org/show_bug.cgi?id=70863 |
- |
- Reviewed by Eric Seidel. |
- |
- * bindings/v8/V8DOMWrapper.cpp: |
- |
-2011-10-25 Mark Hahnenberg <mhahnenberg@apple.com> |
- |
- Resetting baseline for test bindings |
- |
- Unreviewed preemptive build fix |
- |
- No new tests. |
- |
- * bindings/scripts/test/JS/JSTestObj.cpp: |
- * bindings/scripts/test/JS/JSTestObj.h: |
- |
-2011-10-25 Mark Hahnenberg <mhahnenberg@apple.com> |
- |
- Remove deletePropertyVirtual |
- https://bugs.webkit.org/show_bug.cgi?id=70738 |
- |
- Reviewed by Geoffrey Garen. |
- |
- No new tests. |
- |
- Removed all declarations and definitions of deletePropertyVirtual. |
- Also replaced all call sites to deletePropertyVirtual with a |
- corresponding lookup in the MethodTable. |
- |
- * WebCore.exp.in: |
- * bindings/js/JSDOMStringMapCustom.cpp: |
- (WebCore::JSDOMStringMap::deleteProperty): |
- * bindings/js/JSDOMWindowCustom.cpp: |
- (WebCore::JSDOMWindow::deleteProperty): |
- * bindings/js/JSDOMWindowShell.cpp: |
- (WebCore::JSDOMWindowShell::deleteProperty): |
- * bindings/js/JSDOMWindowShell.h: |
- * bindings/js/JSHistoryCustom.cpp: |
- (WebCore::JSHistory::deleteProperty): |
- * bindings/js/JSLocationCustom.cpp: |
- (WebCore::JSLocation::deleteProperty): |
- * bindings/js/JSStorageCustom.cpp: |
- (WebCore::JSStorage::deleteProperty): |
- * bindings/js/ScriptObject.cpp: |
- (WebCore::ScriptGlobalObject::remove): |
- * bindings/objc/WebScriptObject.mm: |
- (-[WebScriptObject removeWebScriptKey:]): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- (GenerateHeader): |
- * bridge/NP_jsobject.cpp: |
- (_NPN_RemoveProperty): |
- * bridge/jni/jni_jsobject.mm: |
- (JavaJSObject::removeMember): |
- * bridge/objc/objc_runtime.h: |
- * bridge/objc/objc_runtime.mm: |
- * bridge/runtime_array.cpp: |
- * bridge/runtime_array.h: |
- * bridge/runtime_object.cpp: |
- * bridge/runtime_object.h: |
- |
-2011-10-25 Chris Fleizach <cfleizach@apple.com> |
- |
- AX: WebKit does not expose HTML label for slider elements |
- https://bugs.webkit.org/show_bug.cgi?id=70856 |
- |
- Reviewed by Darin Adler. |
- |
- Test: platform/mac/accessibility/slider-allows-title-ui-element.html |
- |
- * accessibility/AccessibilitySlider.h: |
- (WebCore::AccessibilitySlider::isControl): |
- * accessibility/mac/WebAccessibilityObjectWrapper.mm: |
- (-[WebAccessibilityObjectWrapper accessibilityAttributeNames]): |
- |
-2011-10-25 Mark Hahnenberg <mhahnenberg@apple.com> |
- |
- Remove putVirtual |
- https://bugs.webkit.org/show_bug.cgi?id=70740 |
- |
- Reviewed by Geoffrey Garen. |
- |
- No new tests. |
- |
- Removed all declarations and definitions of putVirtual. |
- Also replaced all call sites to putVirtual with a |
- corresponding lookup in the MethodTable. |
- |
- * WebCore.exp.in: |
- * bindings/js/JSDOMWindowCustom.cpp: |
- * bindings/js/JSDOMWindowShell.cpp: |
- (WebCore::JSDOMWindowShell::put): |
- * bindings/js/JSDOMWindowShell.h: |
- * bindings/js/JSPluginElementFunctions.cpp: |
- (WebCore::runtimeObjectCustomPut): |
- * bindings/js/SerializedScriptValue.cpp: |
- (WebCore::CloneDeserializer::putProperty): |
- * bindings/objc/WebScriptObject.mm: |
- (-[WebScriptObject setValue:forKey:]): |
- (-[WebScriptObject setWebScriptValueAtIndex:value:]): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- (GenerateHeader): |
- (GenerateImplementation): |
- * bridge/NP_jsobject.cpp: |
- (_NPN_SetProperty): |
- * bridge/jni/jni_jsobject.mm: |
- (JavaJSObject::setMember): |
- (JavaJSObject::setSlot): |
- * bridge/objc/objc_runtime.h: |
- * bridge/objc/objc_runtime.mm: |
- * bridge/qt/qt_runtime.cpp: |
- (JSC::Bindings::convertQVariantToValue): |
- * bridge/runtime_array.cpp: |
- * bridge/runtime_array.h: |
- * bridge/runtime_object.cpp: |
- * bridge/runtime_object.h: |
- * bridge/testqtbindings.cpp: |
- (main): |
- |
-2011-10-25 Adam Barth <abarth@webkit.org> |
- |
- EventTargetFactory.in is not sorted |
- https://bugs.webkit.org/show_bug.cgi?id=70855 |
- |
- Reviewed by Eric Seidel. |
- |
- This patch sorts the file. I was not able to measure a performance |
- difference using the microbenchmark included with this patch. |
- |
- * dom/EventTargetFactory.in: |
- |
-2011-10-25 Adam Barth <abarth@webkit.org> |
- |
- Attempt to fix the Windows build. Windows mashes all these files into |
- one, which doesn't respect the compilation unit rules of C++. |
- |
- * bindings/js/JSEventCustom.cpp: |
- * bindings/js/JSEventTarget.cpp: |
- |
-2011-10-25 Ryosuke Niwa <rniwa@webkit.org> |
- |
- Moving to the start of line should not place the caret outside of the table |
- https://bugs.webkit.org/show_bug.cgi?id=70757 |
- |
- Reviewed by Chang Shu. |
- |
- The bug was caused by positionAvoidingFirstPositionInTable. Get rid of the function. |
- |
- Test: editing/selection/table-lineboundary.html |
- |
- * editing/visible_units.cpp: |
- (WebCore::startPositionForLine): |
- |
-2011-10-25 Beth Dakin <bdakin@apple.com> |
- |
- https://bugs.webkit.org/show_bug.cgi?id=70852 |
- Setting up a HiDPI base-level GraphicsContext should be more straightforward for |
- WebKit2 |
- |
- Reviewed by Dan Bernstein. |
- |
- This patch removes the old cg-only GraphicsContext::setBaseCTM() api, and adds |
- platform-independent GraphicsContext::applyDeviceScaleFactor(). |
- * WebCore.exp.in: |
- * platform/graphics/GraphicsContext.cpp: |
- (WebCore::GraphicsContext::platformApplyDeviceScaleFactor): |
- (WebCore::GraphicsContext::applyDeviceScaleFactor): |
- * platform/graphics/GraphicsContext.h: |
- * platform/graphics/cg/GraphicsContextCG.cpp: |
- (WebCore::GraphicsContext::platformApplyDeviceScaleFactor): |
- |
- Since this patch removes GraphicsContext::setBaseCTM(), this code has been |
- reverted to do what it used to do before that was added; it just calls into |
- WebCoreSystemInterface directly. |
- * platform/graphics/cg/ImageCG.cpp: |
- (WebCore::Image::drawPattern): |
- |
-2011-10-25 Anders Carlsson <andersca@apple.com> |
- |
- Plug-ins have to use JavaScript to find out the current device scale factor |
- https://bugs.webkit.org/show_bug.cgi?id=67225 |
- |
- Reviewed by Darin Adler. |
- |
- Test: platform/mac-wk2/plugins/contents-scale-factor.html |
- |
- Add NPNVcontentsScaleFactor, as per https://wiki.mozilla.org/NPAPI:ContentsScaleFactor. |
- |
- * plugins/npapi.h: |
- |
-2011-10-25 Adam Klein <adamk@chromium.org> |
- |
- Forward declare MutationObserverEntry as a struct to fix clang build |
- https://bugs.webkit.org/show_bug.cgi?id=70853 |
- |
- Reviewed by Ryosuke Niwa. |
- |
- Node.h forward declared MutationObserverEntry as a class, and clang |
- complains if the two declarations don't match. |
- |
- * dom/Node.h: |
- |
-2011-10-25 Dan Bernstein <mitz@apple.com> |
- |
- Caret can be positioned or repainted incorrectly in flipped-blocks multi-column blocks |
- https://bugs.webkit.org/show_bug.cgi?id=70851 |
- |
- Reviewed by Dave Hyatt. |
- |
- * manual-tests/caret-in-columns-flipped.html: Added. |
- * rendering/RenderBlock.cpp: |
- (WebCore::RenderBlock::adjustForColumns): Removed flipped-blocks writing mode considerations |
- from this function, since not all callers were expecting it to account for flippedness. This makes |
- it similar to adjustRectForColumns() in not adjusting for flippedness. |
- * rendering/RenderBox.cpp: |
- (WebCore::RenderBox::offsetFromContainer): In the columns case, account for flipped-blocks modes |
- by first mapping the point and the offset to non-flipped, non-columns space, then adjusting for |
- columns, then flipping. |
- |
-2011-10-25 Michal Mocny <mmocny@google.com> |
- |
- LayerRendererChromium::setVisible called needlessly without a visibility change |
- https://bugs.webkit.org/show_bug.cgi?id=70766 |
- |
- Reviewed by Kenneth Russell. |
- |
- Check that visibility has actually changed before calling LayerRendererChromium::setVisible |
- |
- * platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp: |
- (WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl): |
- (WebCore::CCLayerTreeHostImpl::setVisible): |
- * platform/graphics/chromium/cc/CCLayerTreeHostImpl.h: |
- |
-2011-10-25 Anders Carlsson <andersca@apple.com> |
- |
- REGRESSION(97821): HistoryController::itemsAreClones crashes a lot |
- https://bugs.webkit.org/show_bug.cgi?id=70827 |
- <rdar://problem/10342925> |
- |
- Reviewed by Adam Barth. |
- |
- Check that m_currentItem is non-null before passing it to itemsAreClones. While I wasn't |
- able to make a test case that would reproduce this crash, we do check m_currentItem for null |
- everywhere else, and the crash log indicates that this would fix the crash. |
- |
- * loader/HistoryController.cpp: |
- (WebCore::HistoryController::recursiveUpdateForCommit): |
- |
-2011-10-25 Adam Barth <abarth@webkit.org> |
- |
- EventTarget.h shouldn't need to know about every feature and ifdef |
- https://bugs.webkit.org/show_bug.cgi?id=70659 |
- |
- Reviewed by Darin Adler. |
- |
- This patch follows the approach of Event.h and introduces an |
- interfaceName virtual function that returns the name of the DOM |
- interface for the concrete type of the object. This function lets us |
- remove a large number of fake dynamic casts. |
- |
- * CMakeLists.txt: |
- * CodeGenerators.pri: |
- * DerivedSources.make: |
- * GNUmakefile.am: |
- * WebCore.gyp/WebCore.gyp: |
- * WebCore.gyp/scripts/action_makenames.py: |
- - Teach action_makenames how to handle the new "in" file. |
- * WebCore.gypi: |
- * WebCore.vcproj/WebCore.vcproj: |
- * WebCore.xcodeproj/project.pbxproj: |
- * bindings/js/JSEventTarget.cpp: |
- (WebCore::toJS): |
- - Use the new autogenerated macro to remove a large amount of |
- feature-specific code. |
- * bindings/js/WorkerScriptController.cpp: |
- (WebCore::WorkerScriptController::initScript): |
- * bindings/objc/DOM.mm: |
- (kit): |
- - The SVGElementInstance branch was never taken previously because |
- SVGElementInstance::toNode returns a non-NULL value. This code |
- was introduced in http://trac.webkit.org/changeset/42618 and |
- doesn't appear to have ever worked as the author intended. |
- * bindings/v8/V8DOMWrapper.cpp: |
- (WebCore::V8DOMWrapper::convertEventTargetToV8Object): |
- - Use the new autogenerated macro to remove a large amount of |
- feature-specific code. |
- * dom/EventNames.cpp: |
- (WebCore::EventNames::EventNames): |
- * dom/EventNames.h: |
- * dom/EventTarget.cpp: |
- * dom/EventTarget.h: |
- * dom/EventTargetFactory.in: Added. |
- - Add a new "in" file that lists all the EventTargets. |
- * dom/LocalMediaStream.cpp: |
- (WebCore::LocalMediaStream::interfaceName): |
- * dom/LocalMediaStream.h: |
- * dom/MediaStream.cpp: |
- (WebCore::MediaStream::interfaceName): |
- * dom/MediaStream.h: |
- * dom/MessagePort.cpp: |
- (WebCore::MessagePort::interfaceName): |
- * dom/MessagePort.h: |
- * dom/Node.cpp: |
- (WebCore::Node::toNode): |
- (WebCore::Node::interfaceName): |
- * dom/Node.h: |
- * dom/make_event_factory.pl: |
- - Generalize make_event_factory.pl to be able to generate |
- interfaces for different namespaces. |
- * fileapi/FileReader.cpp: |
- (WebCore::FileReader::interfaceName): |
- * fileapi/FileReader.h: |
- * fileapi/FileWriter.cpp: |
- (WebCore::FileWriter::interfaceName): |
- * fileapi/FileWriter.h: |
- * loader/appcache/DOMApplicationCache.cpp: |
- (WebCore::DOMApplicationCache::interfaceName): |
- * loader/appcache/DOMApplicationCache.h: |
- * notifications/Notification.cpp: |
- (WebCore::Notification::interfaceName): |
- * notifications/Notification.h: |
- (WebCore::Notification::scriptExecutionContext): |
- * p2p/PeerConnection.cpp: |
- (WebCore::PeerConnection::interfaceName): |
- * p2p/PeerConnection.h: |
- * page/DOMWindow.cpp: |
- (WebCore::DOMWindow::interfaceName): |
- (WebCore::DOMWindow::toDOMWindow): |
- * page/DOMWindow.h: |
- * page/EventSource.cpp: |
- (WebCore::EventSource::interfaceName): |
- * page/EventSource.h: |
- * storage/IDBDatabase.cpp: |
- (WebCore::IDBDatabase::interfaceName): |
- * storage/IDBDatabase.h: |
- * storage/IDBRequest.cpp: |
- (WebCore::IDBRequest::interfaceName): |
- * storage/IDBRequest.h: |
- * storage/IDBTransaction.cpp: |
- (WebCore::IDBTransaction::interfaceName): |
- * storage/IDBTransaction.h: |
- * storage/IDBVersionChangeRequest.cpp: |
- (WebCore::IDBVersionChangeRequest::interfaceName): |
- * storage/IDBVersionChangeRequest.h: |
- * svg/SVGElementInstance.cpp: |
- (WebCore::SVGElementInstance::interfaceName): |
- * svg/SVGElementInstance.h: |
- (WebCore::SVGElementInstance::toNode): |
- * webaudio/AudioContext.cpp: |
- (WebCore::AudioContext::interfaceName): |
- (WebCore::AudioContext::scriptExecutionContext): |
- * webaudio/AudioContext.h: |
- * webaudio/JavaScriptAudioNode.cpp: |
- (WebCore::JavaScriptAudioNode::interfaceName): |
- * webaudio/JavaScriptAudioNode.h: |
- * websockets/WebSocket.cpp: |
- (WebCore::WebSocket::interfaceName): |
- * websockets/WebSocket.h: |
- * workers/DedicatedWorkerContext.cpp: |
- (WebCore::DedicatedWorkerContext::interfaceName): |
- * workers/DedicatedWorkerContext.h: |
- * workers/DefaultSharedWorkerRepository.cpp: |
- (WebCore::SharedWorkerConnectTask::performTask): |
- * workers/SharedWorker.cpp: |
- (WebCore::SharedWorker::interfaceName): |
- * workers/SharedWorker.h: |
- * workers/SharedWorkerContext.cpp: |
- (WebCore::SharedWorkerContext::interfaceName): |
- * workers/SharedWorkerContext.h: |
- * workers/Worker.cpp: |
- (WebCore::Worker::interfaceName): |
- * workers/Worker.h: |
- * xml/XMLHttpRequest.cpp: |
- (WebCore::XMLHttpRequest::interfaceName): |
- * xml/XMLHttpRequest.h: |
- * xml/XMLHttpRequestUpload.cpp: |
- (WebCore::XMLHttpRequestUpload::interfaceName): |
- * xml/XMLHttpRequestUpload.h: |
- |
-2011-10-25 Pavel Podivilov <podivilov@chromium.org> |
- |
- Fix several minor problems in idls. |
- https://bugs.webkit.org/show_bug.cgi?id=70811 |
- |
- Reviewed by Adam Barth. |
- |
- This patch does not have any effect on generated bindings. |
- |
- * dom/DataTransferItems.idl: |
- * page/History.idl: |
- * storage/IDBIndex.idl: |
- * xml/XMLHttpRequest.idl: |
- |
-2011-10-25 Nate Chapin <japhet@chromium.org> |
- |
- Make DocumentThreadableLoader a CachedResourceClient. |
- This will allow us to cache ThreadableLoaderClients as |
- appropriate in a later patch. |
- https://bugs.webkit.org/show_bug.cgi?id=61225 |
- |
- Reviewed by Antti Koivisto. |
- |
- No new tests, no functionality change intended. |
- |
- * WebCore.gypi: |
- * loader/DocumentThreadableLoader.cpp: |
- (WebCore::DocumentThreadableLoader::notifyFinished): CachedResourceClients don't send separate callbacks |
- for didFail() and didFinishLoading(), so we need to differentiate here. |
- (WebCore::DocumentThreadableLoader::loadRequest): Call CachedResourceLoader::requestRawResource() |
- instead of ResourceLoadScheduler::scheduleSubresourceLoad(). |
- * loader/DocumentThreadableLoader.h: |
- * loader/cache/CachedRawResource.cpp: |
- (WebCore::CachedRawResource::data): Handle the complexity of some ThreadableLoaderClients setting |
- DoNotBufferData in ResoureLoaderOptions. If the DataBufferingPolicy is BufferData, then the |
- 'data' input parameter will contain all data received so far, and we need to determine the |
- incremental data to forward to the clients. If the policy is DoNotBufferData, 'data' is just |
- the incremental data. |
- * loader/cache/CachedRawResource.h: |
- * loader/cache/CachedResource.cpp: |
- * loader/cache/CachedResource.h: |
- (WebCore::CachedResource::preloadResult): |
- (WebCore::CachedResource::didSendData): |
- (WebCore::CachedResource::didDownloadData): |
- (WebCore::CachedResource::setLoadFinishTime): |
- (WebCore::CachedResource::loadFinishTime): |
- * loader/cache/CachedResourceClient.h: Add several new callbacks needed by DocumentThreadableLoader. |
- * loader/cache/CachedResourceRequest.cpp: |
- (WebCore::CachedResourceRequest::didReceiveResponse): CachedResource::setResponse() can now cancel |
- the request, so protect appropriately. |
- * loader/cache/CachedResourceRequest.h: |
- * loader/chromium/CachedRawResourceChromium.cpp: Added (plumb didDownloadData() to DocumentThreadableLoader). |
- * loader/chromium/CachedResourceRequestChromium.cpp: Added (plumb didDownloadData() to DocumentThreadableLoader). |
- * loader/chromium/DocumentThreadableLoaderChromium.cpp: |
- * platform/network/BlobResourceHandle.cpp: |
- (WebCore::BlobResourceHandle::notifyResponse): Protect before setting response, since it might kill us. |
- |
-2011-10-25 Pavel Podivilov <podivilov@chromium.org> |
- |
- Get rid of optional parameters in the middle in IDLs. |
- https://bugs.webkit.org/show_bug.cgi?id=70816 |
- |
- Reviewed by Adam Barth. |
- |
- Optional parameters in the middle are prohibited by WebIDL spec. |
- |
- * bindings/scripts/CodeGeneratorJS.pm: |
- (GenerateHeader): |
- (GenerateImplementation): |
- * bindings/scripts/CodeGeneratorV8.pm: |
- (GenerateHeader): |
- (GenerateArgumentsCountCheck): |
- (GenerateImplementation): |
- * html/canvas/CanvasRenderingContext2D.idl: |
- * page/DOMWindow.idl: |
- |
-2011-10-25 Chris Evans <cevans@google.com> |
- |
- Manage the CSS property array length correctly |
- https://bugs.webkit.org/show_bug.cgi?id=70783 |
- |
- Reviewed by Adam Barth. |
- |
- * css/CSSParser.cpp: |
- (WebCore::CSSParser::addProperty): don't allow max length to get out of sync with the buffer. |
- |
-2011-10-25 Tony Chang <tony@chromium.org> |
- |
- avoid unnecessary layouts of flex items during the flex pass |
- https://bugs.webkit.org/show_bug.cgi?id=70557 |
- |
- Reviewed by Ojan Vafai. |
- |
- If the preferred size of a flex item is provided, we don't need to |
- layout the flex item when computing the preferred size. This allows |
- us to only call layout on each flex item once in the common case. |
- |
- No new tests, covered by existing tests. |
- |
- * rendering/RenderFlexibleBox.cpp: |
- (WebCore::RenderFlexibleBox::flowAwareLogicalWidthLengthForChild): |
- (WebCore::RenderFlexibleBox::preferredLogicalContentWidthForFlexItem): |
- (WebCore::RenderFlexibleBox::computePreferredLogicalWidth): |
- (WebCore::RenderFlexibleBox::layoutAndPlaceChildrenInlineDirection): |
- * rendering/RenderFlexibleBox.h: |
- |
-2011-10-25 Fady Samuel <fsamuel@chromium.org> |
- |
- Crash in WebCore::RenderTableSection::addChild due to assert failure |
- https://bugs.webkit.org/show_bug.cgi?id=70678 |
- |
- Reviewed by David Hyatt. |
- |
- Tests: fast/table/table-anonymous-cell-bug.html |
- fast/table/table-anonymous-row-bug.html |
- fast/table/table-anonymous-section-bug.html |
- |
- If the child being added is not a Section/Row/Cell, and the previous sibling is not anonymous, |
- we need to create a new anonymous Section/Row/Cell respectively, instead of failing an |
- assert. |
- |
- * rendering/RenderTable.cpp: |
- (WebCore::RenderTable::addChild): |
- * rendering/RenderTableRow.cpp: |
- (WebCore::RenderTableRow::addChild): |
- * rendering/RenderTableSection.cpp: |
- (WebCore::RenderTableSection::addChild): |
- |
-2011-10-25 Mark Hahnenberg <mhahnenberg@apple.com> |
- |
- Add getOwnPropertySlot to MethodTable |
- https://bugs.webkit.org/show_bug.cgi?id=69807 |
- |
- Reviewed by Oliver Hunt. |
- |
- No new tests. |
- |
- * bridge/runtime_method.h: Changed getOwnPropertySlot to be protected so |
- subclasses can reference it in their MethodTables. |
- |
-2011-10-25 Simon Fraser <simon.fraser@apple.com> |
- |
- REGRESSION (r88580): Cursor fails to change to pointer on embedded Google maps popups |
- https://bugs.webkit.org/show_bug.cgi?id=62797 |
- |
- Reviewed by Chris Marrin. |
- |
- When hit testing through transformed layers, RenderLayer would simply use |
- the composited bounds of the layer as the localHitTestRect for hit testing sublayers. |
- However, this broke hit testing on pages that have a composited, non-transformed layer |
- which falls outside the bounds of its parent, composited-transformed layer, like Google |
- Maps. |
- |
- Fix by mapping the hitTestRect through transforms. However this is tricky, because |
- mapping a rect into the coordinate system of a layer can result in invalid rectangles |
- when point projection results in a negative w component. Fix TransformationMatrix::projectPoint() |
- to detect this case and replace X and Y with large values, and add boundsOfProjectedQuad(), which |
- maps rectangles with possibly-infinite location or bounds into rects which are representable |
- in an IntRect. |
- |
- Tests: transforms/3d/hit-testing/composited-hit-test.html |
- transforms/3d/hit-testing/rotated-hit-test-with-child.html |
- transforms/3d/hit-testing/rotated-hit-test2.html |
- |
- * platform/graphics/transforms/TransformationMatrix.cpp: |
- (WebCore::TransformationMatrix::projectPoint): |
- (WebCore::TransformationMatrix::projectQuad): |
- (WebCore::clampEdgeValue): |
- (WebCore::TransformationMatrix::boundsOfProjectedQuad): |
- * platform/graphics/transforms/TransformationMatrix.h: |
- * rendering/HitTestingTransformState.cpp: |
- (WebCore::HitTestingTransformState::boundsOfMappedQuad): |
- * rendering/HitTestingTransformState.h: |
- * rendering/RenderLayer.cpp: |
- (WebCore::RenderLayer::hitTestLayer): |
- |
-2011-10-25 Adrienne Walker <enne@google.com> |
- |
- [chromium] Batch up texture uploads so that they can be updated incrementally |
- https://bugs.webkit.org/show_bug.cgi?id=70454 |
- |
- Reviewed by James Robinson. |
- |
- Because texture uploads can sometimes take more than a frame to |
- complete, collect all the uploads that need to occur in a |
- CCTextureUpdater so that we can easily incrementally upload some of |
- them, interleaving input and drawing so that the compositor thread |
- continues to appear responsive during a commit. |
- |
- This change doesn't actually interleave the commits, it just pushes |
- the infrastructure for doing so up to the CCProxy level so that a |
- future scheduler can make this decision. |
- |
- VideoLayerChromium is the only layer type to not use this system. It |
- needs to be converted to use a texture updater first and it is less of |
- a priority than the tiled layers. |
- |
- * WebCore.gypi: |
- * platform/graphics/chromium/Canvas2DLayerChromium.cpp: |
- (WebCore::Canvas2DLayerChromium::updateCompositorResources): |
- * platform/graphics/chromium/Canvas2DLayerChromium.h: |
- * platform/graphics/chromium/LayerChromium.h: |
- (WebCore::LayerChromium::updateCompositorResources): |
- * platform/graphics/chromium/TiledLayerChromium.cpp: |
- (WebCore::TiledLayerChromium::updateCompositorResources): |
- * platform/graphics/chromium/TiledLayerChromium.h: |
- * platform/graphics/chromium/VideoLayerChromium.cpp: |
- (WebCore::VideoLayerChromium::updateCompositorResources): |
- * platform/graphics/chromium/VideoLayerChromium.h: |
- * platform/graphics/chromium/WebGLLayerChromium.cpp: |
- (WebCore::WebGLLayerChromium::updateCompositorResources): |
- * platform/graphics/chromium/WebGLLayerChromium.h: |
- * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: |
- (WebCore::CCLayerTreeHost::beginCommitOnImplThread): |
- (WebCore::CCLayerTreeHost::finishCommitOnImplThread): |
- (WebCore::CCLayerTreeHost::updateCompositorResources): |
- * platform/graphics/chromium/cc/CCLayerTreeHost.h: |
- * platform/graphics/chromium/cc/CCSingleThreadProxy.cpp: |
- (WebCore::CCSingleThreadProxy::setNeedsCommit): |
- (WebCore::CCSingleThreadProxy::doCommit): |
- (WebCore::CCSingleThreadProxy::commitIfNeeded): |
- * platform/graphics/chromium/cc/CCSingleThreadProxy.h: |
- * platform/graphics/chromium/cc/CCTextureUpdater.cpp: Added. |
- (WebCore::CCTextureUpdater::CCTextureUpdater): |
- (WebCore::CCTextureUpdater::~CCTextureUpdater): |
- (WebCore::CCTextureUpdater::append): |
- (WebCore::CCTextureUpdater::update): |
- (WebCore::CCTextureUpdater::clear): |
- * platform/graphics/chromium/cc/CCTextureUpdater.h: Added. |
- (WebCore::CCTextureUpdater::allocator): |
- * platform/graphics/chromium/cc/CCThreadProxy.cpp: |
- (WebCore::CCThreadProxy::commitOnImplThread): |
- |
-2011-10-24 Jer Noble <jer.noble@apple.com> |
- |
- compositing/video tests time out on Lion |
- https://bugs.webkit.org/show_bug.cgi?id=70448 |
- |
- Reviewed by Simon Fraser. |
- |
- AVFoundation will not begin loading or decoding media until a consumer for that media is present. |
- For video-only media, this means that an AVPlayerLayer must be created and attached to the |
- AVPlayer before the load state will ever reach "canPlayThrough". Once the metadata for a media |
- is loaded, create a AVPlayerLayer if the media has a video track. |
- |
- * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp: |
- (WebCore::MediaPlayerPrivateAVFoundation::updateStates): |
- |
-2011-10-24 Ryosuke Niwa <rniwa@webkit.org> |
- |
- Merge endOfLine with logicalEndOfLine and startOfLine with logicalStartOfLine |
- https://bugs.webkit.org/show_bug.cgi?id=70755 |
- |
- Reviewed by Chang Shu. |
- |
- Merged endOfLine with logicalEndOfLine, and startOfLine with logicalStartOfLine |
- to reduce the code duplication. |
- |
- * editing/visible_units.cpp: |
- (WebCore::startPositionForLine): |
- (WebCore::startOfLine): |
- (WebCore::logicalStartOfLine): |
- (WebCore::endPositionForLine): |
- (WebCore::inSameLogicalLine): |
- (WebCore::endOfLine): |
- (WebCore::logicalEndOfLine): |
- * editing/visible_units.h: |
- |
-2011-10-25 Arko Saha <arko@motorola.com> |
- |
- Microdata: itemtype attribute should be space-separated list to allow multiple types. |
- https://bugs.webkit.org/show_bug.cgi?id=70501 |
- |
- Reviewed by Ryosuke Niwa. |
- |
- Tests: fast/dom/MicroData/getitems-multiple-itemtypes.html |
- fast/dom/MicroData/itemtype-add-remove-tokens.html |
- fast/dom/MicroData/itemtype-attribute-test.html |
- |
- * dom/MicroDataItemList.cpp: |
- (WebCore::MicroDataItemList::nodeMatches): |
- * html/DOMSettableTokenList.h: |
- (WebCore::DOMSettableTokenList::tokens): |
- * html/HTMLElement.cpp: |
- (WebCore::HTMLElement::parseMappedAttribute): |
- (WebCore::HTMLElement::setItemValueText): |
- (WebCore::HTMLElement::itemType): |
- (WebCore::HTMLElement::setItemType): |
- * html/HTMLElement.h: |
- * html/HTMLElement.idl: |
- |
-2011-10-25 Dan Bernstein <mitz@apple.com> |
- |
- <rdar://problem/10337033> DOMRangeOfString:relativeTo:options has problems with -webkit-user-select: none |
- |
- Reviewed by Adam Roben. |
- |
- Test: added to TestWebKitAPI/Tests/mac/DOMRangeOfString.mm |
- |
- * page/Page.cpp: |
- (WebCore::Page::rangeOfString): This function was incorrectly adding the StartInSelection option to |
- the initial search. |
- |
-2011-10-25 Yury Semikhatsky <yurys@chromium.org> |
- |
- Web Inspector: redesign workers sidebar pane |
- https://bugs.webkit.org/show_bug.cgi?id=70815 |
- |
- Changed workers sidebar pane design. |
- |
- Now there is a link to worker inspector for each dedicated worker created |
- by inspected page. The sidebar display a link to a page with list of all |
- shared workers if there one is supported by the WebKit port. Also there |
- is a checkbox that allows pause all new workers on first statements. |
- |
- Reviewed by Pavel Feldman. |
- |
- * inspector/front-end/Settings.js: |
- * inspector/front-end/StylesSidebarPane.js: |
- (WebInspector.StylesSidebarPane.prototype._rebuildSectionsForStyleRules): |
- * inspector/front-end/WorkerManager.js: |
- (WebInspector.WorkerManager.prototype.openWorkerInspector): |
- (WebInspector.WorkerManager.prototype._workerInspectorClosing): |
- * inspector/front-end/WorkersSidebarPane.js: |
- (WebInspector.WorkerListSidebarPane): |
- (WebInspector.WorkerListSidebarPane.prototype._workerRemoved): |
- (WebInspector.WorkerListSidebarPane.prototype._workersCleared): |
- (WebInspector.WorkerListSidebarPane.prototype._addWorker): |
- (WebInspector.WorkerListSidebarPane.prototype._workerItemClicked): |
- (WebInspector.WorkerListSidebarPane.prototype._autoattachToWorkersClicked): |
- (WebInspector.WorkerListSidebarPane.prototype._createSharedWorkersLink.link.onclick): |
- (WebInspector.WorkerListSidebarPane.prototype._createSharedWorkersLink): |
- * inspector/front-end/elementsPanel.css: |
- * inspector/front-end/inspectorCommon.css: |
- (.sidebar-separator): |
- (.sidebar-label): |
- * inspector/front-end/scriptsPanel.css: |
- (.dedicated-worker-item): |
- (#shared-workers-list): |
- (#pause-workers-checkbox > input): |
- |
-2011-10-25 Nico Weber <thakis@chromium.org> |
- |
- [chromium/mac] Fix an ODR violation. |
- https://bugs.webkit.org/show_bug.cgi?id=70753 |
- |
- Reviewed by Anders Carlsson. |
- |
- Both ScrollAnimatorNone and ScrollAnimatorMac define ScrollAnimator::create() on OS X. |
- Remove the unused one. |
- |
- * WebCore.gyp/WebCore.gyp: |
- |
-2011-10-25 Vsevolod Vlasov <vsevik@chromium.org> |
- |
- Web Inspector: Debugger fails when there is an invalid watch expression. |
- https://bugs.webkit.org/show_bug.cgi?id=70718 |
- |
- Reviewed by Pavel Feldman. |
- |
- Test: inspector/debugger/watch-expressions-panel-switch.html |
- |
- * inspector/front-end/ScriptsPanel.js: |
- (WebInspector.ScriptsPanel.prototype.evaluateInSelectedCallFrame): |
- |
-2011-10-24 Pavel Podivilov <podivilov@chromium.org> |
- |
- Web Inspector: fix empty line handling in source maps. |
- https://bugs.webkit.org/show_bug.cgi?id=70726 |
- |
- Reviewed by Yury Semikhatsky. |
- |
- * inspector/front-end/CompilerSourceMapping.js: |
- (WebInspector.ClosureCompilerSourceMapping.prototype._parseMappings): |
- |
-2011-10-25 Justin Schuh <jschuh@chromium.org> |
- |
- Check for empty string in parseArcFlag |
- https://bugs.webkit.org/show_bug.cgi?id=70763 |
- |
- Reviewed by Dirk Schulze. |
- |
- Test: svg/path-invalid.html |
- |
- * svg/SVGParserUtilities.cpp: |
- (WebCore::parseArcFlag): |
- |
-2011-10-24 Pavel Podivilov <podivilov@chromium.org> |
- |
- Web Inspector: keep old source mapping when new one could not be loaded. |
- https://bugs.webkit.org/show_bug.cgi?id=70729 |
- |
- Reviewed by Yury Semikhatsky. |
- |
- * inspector/front-end/RawSourceCode.js: |
- (WebInspector.RawSourceCode.prototype._updateSourceMapping.didCreateSourceMapping): |
- (WebInspector.RawSourceCode.prototype._updateSourceMapping): |
- (WebInspector.RawSourceCode.prototype._createSourceMapping.didLoadSourceMapping): |
- |
-2011-10-25 Sachin Puranik <sachin.puranik@motorola.com> |
- |
- Double number step=.5 should be considered valid in <input type="number" step=".5"> |
- https://bugs.webkit.org/show_bug.cgi?id=70320 |
- |
- Reviewed by Kent Tamura. |
- |
- Test: fast/forms/input-step-as-double.html |
- |
- * html/parser/HTMLParserIdioms.cpp: |
- (WebCore::parseToDoubleForNumberType): Changed the function to consider |
- step as valid even if it starts with "." |
- |
-2011-10-25 Alexander Pavlov <apavlov@chromium.org> |
- |
- Web Inspector: [Chromium] Audits panel context menu for links not working properly |
- https://bugs.webkit.org/show_bug.cgi?id=70813 |
- |
- Reviewed by Pavel Feldman. |
- |
- * inspector/front-end/AuditResultView.js: |
- (WebInspector.AuditResultView): |
- (WebInspector.AuditResultView.prototype._contextMenuEventFired): |
- |
-2011-10-25 Vsevolod Vlasov <vsevik@chromium.org> |
- |
- Web Inspector: Resources panel: display the current search match index in the toolbar. |
- https://bugs.webkit.org/show_bug.cgi?id=66050 |
- |
- Reviewed by Pavel Feldman. |
- |
- Test: http/tests/inspector/search/resources-search-match-index.html |
- |
- * inspector/front-end/ResourcesPanel.js: |
- (WebInspector.ResourcesPanel.prototype.performSearch.callback): |
- (WebInspector.ResourcesPanel.prototype.performSearch): |
- (WebInspector.ResourcesPanel.prototype._showSearchResult.callback): |
- (WebInspector.ResourcesPanel.prototype._showSearchResult): |
- (WebInspector.BaseStorageTreeElement.prototype.get searchMatchesCount): |
- (WebInspector.ResourcesSearchController): |
- (WebInspector.ResourcesSearchController.prototype.nextSearchResult): |
- (WebInspector.ResourcesSearchController.prototype.previousSearchResult): |
- (WebInspector.ResourcesSearchController.prototype._searchResult): |
- (WebInspector.SearchResultsTreeElementsTraverser.prototype.next): |
- (WebInspector.SearchResultsTreeElementsTraverser.prototype.previous): |
- (WebInspector.SearchResultsTreeElementsTraverser.prototype.matchIndex): |
- (WebInspector.SearchResultsTreeElementsTraverser.prototype._elementSearchMatchesCount): |
- (WebInspector.SearchResultsTreeElementsTraverser.prototype._traverseNext): |
- (WebInspector.SearchResultsTreeElementsTraverser.prototype._traversePrevious): |
- * inspector/front-end/treeoutline.js: |
- |
-2011-10-25 Andrey Kosyakov <caseq@chromium.org> |
- |
- Web Inspector: JS exception in JavaScriptSourceFrame.onShowPopover/showObjectPopover() |
- https://bugs.webkit.org/show_bug.cgi?id=70812 |
- |
- Reviewed by Pavel Feldman. |
- |
- Check if highlighted element is present while displaying object popover. |
- |
- * inspector/front-end/JavaScriptSourceFrame.js: |
- (WebInspector.JavaScriptSourceFrame.prototype.onShowPopover.showObjectPopover): |
- (WebInspector.JavaScriptSourceFrame.prototype.onShowPopover): |
- |
-2011-10-21 Nat Duca <nduca@chromium.org> |
- |
- [chromium] Route Console::time and Console::timeEnd to trace_event |
- https://bugs.webkit.org/show_bug.cgi?id=70620 |
- |
- Reviewed by Pavel Feldman. |
- |
- * page/Console.cpp: |
- (WebCore::Console::time): |
- (WebCore::Console::timeEnd): |
- |
-2011-10-21 Andrey Kosyakov <caseq@chromium.org> |
- |
- Web Inspector: [Extensions API] allow extensions to specify script to be injected on reload |
- https://bugs.webkit.org/show_bug.cgi?id=70600 |
- |
- Reviewed by Pavel Feldman. |
- |
- - change webInspector.inspectedPage.reload() to accept multiple options in an object |
- - add an ability to inject a script upon a page reload |
- - return script id in addScriptToEvaluateOnLoad() |
- - provide removeScriptToEvaluateOnLoad() |
- - store scripts to evaluate on load in inspector state cookie |
- |
- Test: inspector/extensions/extensions-reload.html |
- |
- * inspector/Inspector.json: |
- * inspector/InspectorPageAgent.cpp: |
- (WebCore::InspectorPageAgent::InspectorPageAgent): |
- (WebCore::InspectorPageAgent::restore): |
- (WebCore::InspectorPageAgent::addScriptToEvaluateOnLoad): |
- (WebCore::InspectorPageAgent::removeScriptToEvaluateOnLoad): |
- (WebCore::InspectorPageAgent::removeAllScriptsToEvaluateOnLoad): |
- (WebCore::InspectorPageAgent::didClearWindowObjectInWorld): |
- (WebCore::InspectorPageAgent::frameNavigated): |
- * inspector/InspectorPageAgent.h: |
- * inspector/front-end/ExtensionAPI.js: |
- (injectedExtensionAPI.InspectedWindow.prototype.reload): |
- * inspector/front-end/ExtensionServer.js: |
- (WebInspector.ExtensionServer.prototype._onReload): |
- * inspector/front-end/WorkersSidebarPane.js: |
- (WebInspector.WorkersSidebarPane.prototype.setInstrumentation.callback): |
- (WebInspector.WorkersSidebarPane.prototype.setInstrumentation): |
- |
-2011-10-24 Pavel Feldman <pfeldman@google.com> |
- |
- Web Inspector: print expired message count before remaining messages. |
- https://bugs.webkit.org/show_bug.cgi?id=70736 |
- |
- Reviewed by Yury Semikhatsky. |
- |
- * inspector/Inspector-0.1.json: |
- * inspector/Inspector.json: |
- * inspector/InspectorConsoleAgent.cpp: |
- (WebCore::InspectorConsoleAgent::enable): |
- (WebCore::InspectorConsoleAgent::restore): |
- * inspector/InspectorConsoleAgent.h: |
- * inspector/front-end/ConsoleModel.js: |
- |
-2011-10-25 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r98309. |
- http://trac.webkit.org/changeset/98309 |
- https://bugs.webkit.org/show_bug.cgi?id=70801 |
- |
- Test fails on mac (Requested by benwells on #webkit). |
- |
- * html/canvas/CanvasRenderingContext2D.cpp: |
- (WebCore::CanvasRenderingContext2D::fill): |
- (WebCore::CanvasRenderingContext2D::fillRect): |
- (WebCore::CanvasRenderingContext2D::drawImage): |
- (WebCore::CanvasRenderingContext2D::shouldDisplayTransparencyElsewhere): |
- (WebCore::CanvasRenderingContext2D::fillAndDisplayTransparencyElsewhere): |
- * html/canvas/CanvasRenderingContext2D.h: |
- |
-2011-10-24 Geoffrey Garen <ggaren@apple.com> |
- |
- Chromium build fix: #include window.h for HANDLE. |
- |
- * platform/graphics/opentype/OpenTypeUtilities.h: |
- |
-2011-10-24 Ryosuke Niwa <rniwa@webkit.org> |
- |
- Include ThreadSafeRefCounted.h since the class declared in this header file |
- inherits from ThreadSafeRefCounted. |
- |
- * storage/IDBIndexBackendInterface.h: |
- |
-2011-10-24 Ryosuke Niwa <rniwa@webkit.org> |
- |
- Fix a typo. |
- |
- * bindings/v8/JavaScriptCallFrame.h: |
- |
-2011-10-24 Geoffrey Garen <ggaren@apple.com> |
- |
- Removed SharedUChar and tightened language around its previous uses |
- https://bugs.webkit.org/show_bug.cgi?id=70698 |
- |
- Reviewed by David Levin. |
- |
- - Renamed "threadSafeCopy" and "crossThreadCopy" to "isolatedCopy" because |
- the former names gave the dangerous misimpression that the resulting object |
- was thread-safe, which is not true. |
- |
- - Updated a bunch of files for header dependency changes, now that |
- CrossThreadRefCounted is gone. |
- |
- * ForwardingHeaders/wtf/CrossThreadRefCounted.h: Removed. |
- * WebCore.exp.in: |
- * bindings/js/ScriptHeapSnapshot.h: |
- * bindings/v8/SerializedScriptValue.cpp: |
- (WebCore::SerializedScriptValue::release): |
- (WebCore::SerializedScriptValue::SerializedScriptValue): |
- * css/CSSPropertySourceData.h: |
- * css/CSSValue.h: |
- * css/StyleMedia.h: |
- * css/WebKitCSSMatrix.h: |
- * dom/QualifiedName.h: |
- * fileapi/LocalFileSystem.h: |
- (WebCore::LocalFileSystem::SystemBasePath::operator String): |
- * history/CachedFrame.h: |
- * history/CachedPage.h: |
- * history/HistoryItem.h: |
- * html/canvas/CanvasStyle.h: |
- * inspector/InspectorResourceAgent.h: |
- * loader/FormState.h: |
- * loader/TextResourceDecoder.h: |
- * loader/WorkerThreadableLoader.cpp: |
- (WebCore::WorkerThreadableLoader::MainThreadBridge::MainThreadBridge): |
- * loader/icon/IconDatabase.cpp: |
- (WebCore::IconDatabase::open): |
- (WebCore::IconDatabase::synchronousIconForPageURL): |
- (WebCore::IconDatabase::synchronousIconURLForPageURL): |
- (WebCore::IconDatabase::retainIconForPageURL): |
- (WebCore::IconDatabase::releaseIconForPageURL): |
- (WebCore::IconDatabase::setIconDataForIconURL): |
- (WebCore::IconDatabase::setIconURLForPageURL): |
- (WebCore::IconDatabase::databasePath): |
- (WebCore::IconDatabase::defaultDatabaseFilename): |
- (WebCore::ImportedIconURLForPageURLWorkItem::ImportedIconURLForPageURLWorkItem): |
- (WebCore::ImportedIconDataForPageURLWorkItem::ImportedIconDataForPageURLWorkItem): |
- * page/ContentSecurityPolicy.h: |
- * page/Coordinates.h: |
- (WebCore::Coordinates::isolatedCopy): |
- * page/Geoposition.h: |
- (WebCore::Geoposition::isolatedCopy): |
- * page/SecurityOrigin.cpp: |
- (WebCore::SecurityOrigin::SecurityOrigin): |
- (WebCore::SecurityOrigin::isolatedCopy): |
- * page/SecurityOrigin.h: |
- * page/animation/AnimationBase.h: |
- * platform/CrossThreadCopier.cpp: |
- (WebCore::::copy): |
- * platform/FileChooser.h: |
- * platform/KURL.cpp: |
- (WebCore::KURL::copy): |
- * platform/cf/SchedulePair.h: |
- * platform/network/BlobData.cpp: |
- (WebCore::BlobDataItem::detachFromCurrentThread): |
- (WebCore::BlobData::detachFromCurrentThread): |
- * platform/network/HTTPHeaderMap.cpp: |
- (WebCore::HTTPHeaderMap::copyData): |
- * platform/network/MIMEHeader.h: |
- * platform/network/ResourceErrorBase.cpp: |
- (WebCore::ResourceErrorBase::copy): |
- * platform/network/ResourceLoadInfo.h: |
- * platform/network/ResourceRequestBase.cpp: |
- (WebCore::ResourceRequestBase::copyData): |
- * platform/network/ResourceResponseBase.cpp: |
- (WebCore::ResourceResponseBase::copyData): |
- * platform/sql/SQLValue.cpp: |
- (WebCore::SQLValue::SQLValue): |
- (WebCore::SQLValue::string): |
- * rendering/style/QuotesData.h: |
- * storage/AbstractDatabase.cpp: |
- (WebCore::updateGuidVersionMap): |
- (WebCore::AbstractDatabase::AbstractDatabase): |
- (WebCore::AbstractDatabase::performOpenAndVerify): |
- (WebCore::AbstractDatabase::stringIdentifier): |
- (WebCore::AbstractDatabase::displayName): |
- (WebCore::AbstractDatabase::fileName): |
- (WebCore::AbstractDatabase::setExpectedVersion): |
- (WebCore::AbstractDatabase::getCachedVersion): |
- * storage/ChangeVersionWrapper.cpp: |
- (WebCore::ChangeVersionWrapper::ChangeVersionWrapper): |
- * storage/Database.cpp: |
- (WebCore::Database::Database): |
- (WebCore::Database::tableNames): |
- * storage/DatabaseDetails.h: |
- * storage/DatabaseTracker.cpp: |
- (WebCore::DatabaseTracker::setDatabaseDirectoryPath): |
- (WebCore::DatabaseTracker::databaseDirectoryPath): |
- (WebCore::DatabaseTracker::trackerDatabasePath): |
- (WebCore::DatabaseTracker::canEstablishDatabase): |
- (WebCore::DatabaseTracker::originPath): |
- (WebCore::DatabaseTracker::fullPathForDatabase): |
- (WebCore::DatabaseTracker::populateOrigins): |
- (WebCore::DatabaseTracker::databaseNamesForOrigin): |
- (WebCore::DatabaseTracker::addOpenDatabase): |
- (WebCore::DatabaseTracker::setQuota): |
- (WebCore::DatabaseTracker::recordCreatingDatabase): |
- (WebCore::DatabaseTracker::recordDeletingDatabase): |
- (WebCore::DatabaseTracker::recordDeletingOrigin): |
- (WebCore::DatabaseTracker::scheduleNotifyDatabaseChanged): |
- * storage/OriginQuotaManager.cpp: |
- (WebCore::OriginQuotaManager::trackOrigin): |
- (WebCore::OriginQuotaManager::addDatabase): |
- * storage/SQLError.h: |
- (WebCore::SQLError::message): |
- (WebCore::SQLError::SQLError): |
- * storage/SQLResultSetRowList.h: |
- * storage/SQLStatement.cpp: |
- (WebCore::SQLStatement::SQLStatement): |
- * storage/StorageAreaSync.cpp: |
- (WebCore::StorageAreaSync::StorageAreaSync): |
- (WebCore::StorageAreaSync::syncTimerFired): |
- * storage/StorageAreaSync.h: |
- * storage/StorageMap.cpp: |
- (WebCore::StorageMap::importItem): |
- * storage/StorageNamespaceImpl.cpp: |
- (WebCore::StorageNamespaceImpl::StorageNamespaceImpl): |
- * storage/StorageSyncManager.cpp: |
- (WebCore::StorageSyncManager::StorageSyncManager): |
- * storage/StorageTracker.cpp: |
- (WebCore::StorageTracker::StorageTracker): |
- (WebCore::StorageTracker::syncImportOriginIdentifiers): |
- (WebCore::StorageTracker::syncFileSystemAndTrackerDatabase): |
- (WebCore::StorageTracker::setOriginDetails): |
- (WebCore::StorageTracker::willDeleteAllOrigins): |
- * svg/SVGPathSeg.h: |
- * workers/DefaultSharedWorkerRepository.cpp: |
- (WebCore::SharedWorkerProxy::url): |
- (WebCore::SharedWorkerProxy::name): |
- (WebCore::SharedWorkerProxy::SharedWorkerProxy): |
- (WebCore::DefaultSharedWorkerRepository::getProxy): |
- * workers/SharedWorkerThread.cpp: |
- (WebCore::SharedWorkerThread::SharedWorkerThread): |
- * workers/WorkerMessagingProxy.cpp: |
- (WebCore::WorkerExceptionTask::WorkerExceptionTask): |
- (WebCore::PostMessageToPageInspectorTask::PostMessageToPageInspectorTask): |
- * workers/WorkerRunLoop.cpp: |
- (WebCore::WorkerRunLoop::postTaskForMode): |
- (WebCore::WorkerRunLoop::Task::Task): |
- * workers/WorkerThread.cpp: |
- (WebCore::WorkerThreadStartupData::WorkerThreadStartupData): |
- |
-2011-10-24 Adam Barth <abarth@webkit.org> |
- |
- Remove empty directory. |
- |
- * p2p: Removed. |
- |
-2011-10-24 Shawn Singh <shawnsingh@chromium.org> |
- |
- [chromium] Refactor painting/updating/drawing code. |
- https://bugs.webkit.org/show_bug.cgi?id=70057 |
- |
- Reviewed by James Robinson. |
- |
- This refactoring separates the "early exits" from the actual loop |
- that draws layers. As part of this, it was also appropriate to |
- clean up the way that masks and replicas are dealt with when |
- painting and updating. The purpose of all this is to make things |
- easier to test, and for a scissoring optimization that needs to |
- know exatly what gets drawn. Existing layout tests were able to |
- verify the refactoring did not change behavior. |
- |
- * platform/graphics/chromium/LayerRendererChromium.cpp: |
- (WebCore::LayerRendererChromium::drawLayersOntoRenderSurfaces): |
- (WebCore::LayerRendererChromium::drawLayersInternal): |
- (WebCore::LayerRendererChromium::drawLayer): |
- * platform/graphics/chromium/LayerRendererChromium.h: |
- * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: |
- (WebCore::CCLayerTreeHost::paintMaskAndReplicaForRenderSurface): |
- (WebCore::CCLayerTreeHost::paintLayerContents): |
- (WebCore::CCLayerTreeHost::updateCompositorResources): |
- * platform/graphics/chromium/cc/CCLayerTreeHost.h: |
- * platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp: |
- (WebCore::layerShouldBeAppended): |
- (WebCore::calculateDrawTransformsAndVisibilityInternal): |
- * platform/graphics/chromium/cc/CCLayerTreeHostCommon.h: |
- (WebCore::CCLayerTreeHostCommon::calculateVisibleLayerRect): |
- |
-2011-10-24 Luke Macpherson <macpherson@chromium.org> |
- |
- REGRESSION(80893): HTML5 spec takes 2s longer to load due to time spent in CSSStyleSelector |
- https://bugs.webkit.org/show_bug.cgi?id=68944 |
- |
- Reviewed by Antti Koivisto. |
- |
- No new tests / Covered by existing tests. |
- |
- Removes all virtual dispatch from CSSStyleApplyProperty by converting to use static function pointers for dispatch. |
- Uses class templates instead of function templates to allow default parameters and also to share parameter declarations across static functions. |
- All constructor parameters become template parameters. This improves the level of compiler optimization that can be achieved. |
- Overall this achieved a ~2% overall speedup on run-safari PerformanceTests/Parser/html5-full-render.html, |
- and profiling showed a reduction in the time spent inside CSSStyleSelector::applyProperty(). |
- |
- * css/CSSStyleApplyProperty.cpp: |
- (WebCore::ApplyPropertyExpanding::applyInheritValue): |
- (WebCore::ApplyPropertyExpanding::applyInitialValue): |
- (WebCore::ApplyPropertyExpanding::applyValue): |
- (WebCore::ApplyPropertyExpanding::createHandler): |
- (WebCore::ApplyPropertyDefaultBase::setValue): |
- (WebCore::ApplyPropertyDefaultBase::value): |
- (WebCore::ApplyPropertyDefaultBase::initial): |
- (WebCore::ApplyPropertyDefaultBase::applyInheritValue): |
- (WebCore::ApplyPropertyDefaultBase::applyInitialValue): |
- (WebCore::ApplyPropertyDefaultBase::applyValue): |
- (WebCore::ApplyPropertyDefaultBase::createHandler): |
- (WebCore::ApplyPropertyDefault::setValue): |
- (WebCore::ApplyPropertyDefault::applyValue): |
- (WebCore::ApplyPropertyDefault::createHandler): |
- (WebCore::ApplyPropertyStyleImage::applyValue): |
- (WebCore::ApplyPropertyStyleImage::createHandler): |
- (WebCore::ApplyPropertyAuto::setValue): |
- (WebCore::ApplyPropertyAuto::value): |
- (WebCore::ApplyPropertyAuto::hasAuto): |
- (WebCore::ApplyPropertyAuto::setAuto): |
- (WebCore::ApplyPropertyAuto::applyInheritValue): |
- (WebCore::ApplyPropertyAuto::applyInitialValue): |
- (WebCore::ApplyPropertyAuto::applyValue): |
- (WebCore::ApplyPropertyAuto::createHandler): |
- (WebCore::defaultInitialColor): |
- (WebCore::ApplyPropertyColor::applyInheritValue): |
- (WebCore::ApplyPropertyColor::applyInitialValue): |
- (WebCore::ApplyPropertyColor::applyValue): |
- (WebCore::ApplyPropertyColor::createHandler): |
- (WebCore::ApplyPropertyDirection::applyValue): |
- (WebCore::ApplyPropertyDirection::createHandler): |
- (WebCore::ApplyPropertyLength::setValue): |
- (WebCore::ApplyPropertyLength::applyValue): |
- (WebCore::ApplyPropertyLength::createHandler): |
- (WebCore::ApplyPropertyString::setValue): |
- (WebCore::ApplyPropertyString::applyValue): |
- (WebCore::ApplyPropertyString::createHandler): |
- (WebCore::ApplyPropertyBorderRadius::setValue): |
- (WebCore::ApplyPropertyBorderRadius::applyValue): |
- (WebCore::ApplyPropertyBorderRadius::createHandler): |
- (WebCore::ApplyPropertyFillLayer::applyInheritValue): |
- (WebCore::ApplyPropertyFillLayer::applyInitialValue): |
- (WebCore::ApplyPropertyFillLayer::applyValue): |
- (WebCore::ApplyPropertyFillLayer::createHandler): |
- (WebCore::ApplyPropertyComputeLength::setValue): |
- (WebCore::ApplyPropertyComputeLength::applyValue): |
- (WebCore::ApplyPropertyComputeLength::createHandler): |
- (WebCore::ApplyPropertyFont::applyInheritValue): |
- (WebCore::ApplyPropertyFont::applyInitialValue): |
- (WebCore::ApplyPropertyFont::applyValue): |
- (WebCore::ApplyPropertyFont::createHandler): |
- (WebCore::ApplyPropertyFontWeight::applyValue): |
- (WebCore::ApplyPropertyFontWeight::createHandler): |
- (WebCore::ApplyPropertyCounter::emptyFunction): |
- (WebCore::ApplyPropertyCounter::applyValue): |
- (WebCore::ApplyPropertyCounter::createHandler): |
- (WebCore::ApplyPropertyCursor::applyInheritValue): |
- (WebCore::ApplyPropertyCursor::applyInitialValue): |
- (WebCore::ApplyPropertyCursor::applyValue): |
- (WebCore::ApplyPropertyCursor::createHandler): |
- (WebCore::ApplyPropertyTextEmphasisStyle::applyInheritValue): |
- (WebCore::ApplyPropertyTextEmphasisStyle::applyInitialValue): |
- (WebCore::ApplyPropertyTextEmphasisStyle::applyValue): |
- (WebCore::ApplyPropertyTextEmphasisStyle::createHandler): |
- (WebCore::ApplyPropertyAnimation::setValue): |
- (WebCore::ApplyPropertyAnimation::value): |
- (WebCore::ApplyPropertyAnimation::test): |
- (WebCore::ApplyPropertyAnimation::clear): |
- (WebCore::ApplyPropertyAnimation::initial): |
- (WebCore::ApplyPropertyAnimation::map): |
- (WebCore::ApplyPropertyAnimation::accessAnimations): |
- (WebCore::ApplyPropertyAnimation::animations): |
- (WebCore::ApplyPropertyAnimation::applyInheritValue): |
- (WebCore::ApplyPropertyAnimation::applyInitialValue): |
- (WebCore::ApplyPropertyAnimation::applyValue): |
- (WebCore::ApplyPropertyAnimation::createHandler): |
- (WebCore::ApplyPropertyOutlineStyle::applyInheritValue): |
- (WebCore::ApplyPropertyOutlineStyle::applyInitialValue): |
- (WebCore::ApplyPropertyOutlineStyle::applyValue): |
- (WebCore::ApplyPropertyOutlineStyle::createHandler): |
- (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): |
- * css/CSSStyleApplyProperty.h: |
- (WebCore::PropertyHandler::PropertyHandler): |
- (WebCore::PropertyHandler::applyInheritValue): |
- (WebCore::PropertyHandler::applyInitialValue): |
- (WebCore::PropertyHandler::applyValue): |
- (WebCore::PropertyHandler::isValid): |
- (WebCore::PropertyHandler::inheritFunction): |
- (WebCore::PropertyHandler::initialFunction): |
- (WebCore::PropertyHandler::applyFunction): |
- (WebCore::CSSStyleApplyProperty::propertyHandler): |
- (WebCore::CSSStyleApplyProperty::setPropertyHandler): |
- * css/CSSStyleSelector.cpp: |
- (WebCore::CSSStyleSelector::applyProperty): |
- * rendering/style/RenderStyle.h: |
- (WebCore::InheritedFlags::invalidColor): |
- |
-2011-10-24 Ben Wells <benwells@chromium.org> |
- |
- Canvas drawImage with SourceIn, DestinationIn, SourceOut, DestinationAtop and Copy have errors |
- https://bugs.webkit.org/show_bug.cgi?id=66920 |
- |
- Reviewed by James Robinson. |
- |
- Test: fast/canvas/canvas-composite-image.html |
- |
- Use the same method as fills to handle composited drawImage calls that will affect the whole |
- canvas. To do this code used when doing fills has been factored into seperate functions and |
- used by drawImage. Some functions used to do these types of operations have also been renamed. |
- |
- * html/canvas/CanvasRenderingContext2D.cpp: |
- (WebCore::isFullCanvasCompositeMode): |
- (WebCore::CanvasRenderingContext2D::fill): |
- (WebCore::CanvasRenderingContext2D::fillRect): |
- (WebCore::CanvasRenderingContext2D::drawImage): |
- (WebCore::CanvasRenderingContext2D::calculateCompositingBufferRect): |
- (WebCore::CanvasRenderingContext2D::createCompositingBuffer): |
- (WebCore::CanvasRenderingContext2D::compositeBuffer): |
- (WebCore::CanvasRenderingContext2D::fullCanvasCompositedDrawImage): |
- (WebCore::CanvasRenderingContext2D::fullCanvasCompositedFill): |
- * html/canvas/CanvasRenderingContext2D.h: |
- |
-2011-10-24 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r98294 and r98306. |
- http://trac.webkit.org/changeset/98294 |
- http://trac.webkit.org/changeset/98306 |
- https://bugs.webkit.org/show_bug.cgi?id=70790 |
- |
- the first patch broke chromium-mac and the second gclient |
- (Requested by jchaffraix on #webkit). |
- |
- * WebCore.gyp/WebCore.gyp: |
- |
-2011-10-24 Ryosuke Niwa <rniwa@webkit.org> |
- |
- Build fix after r98300. |
- |
- * WebCore.gypi: |
- * WebCore.pro: |
- |
-2011-10-24 Anders Carlsson <andersca@apple.com> |
- |
- It should be possible for Widget subclasses to control whether transforms affect the frame rect |
- https://bugs.webkit.org/show_bug.cgi?id=70787 |
- |
- Reviewed by Sam Weinig. |
- |
- Currently RenderWidget sets the frame rect of it's hosted widget to the bounding box of the transformed |
- content box rect (unless the Widget is a frame view). It should be possible for Widget subclasses to override |
- this behavior and have the frame rect set to the absolute content box without any transforms applied. |
- |
- * platform/Widget.h: |
- (WebCore::Widget::transformsAffectFrameRect): |
- Add new member function. |
- |
- * rendering/RenderWidget.cpp: |
- (WebCore::RenderWidget::updateWidgetGeometry): |
- Move shared code from setWidget and updateWidgetPosition out into a new function. If transformsAffectFrameRect returns |
- false, just set the frame rect to the absolute content box. |
- |
- (WebCore::RenderWidget::setWidget): |
- (WebCore::RenderWidget::updateWidgetPosition): |
- Call updateWidgetGeometry. |
- |
- * rendering/RenderWidget.h: |
- |
-2011-10-24 Arko Saha <arko@motorola.com> |
- |
- Microdata: Add itemprop, itemref, itemvalue attributes. |
- https://bugs.webkit.org/show_bug.cgi?id=69839 |
- |
- Reviewed by Ryosuke Niwa. |
- |
- Tests: fast/dom/MicroData/itemprop-add-remove-tokens.html |
- fast/dom/MicroData/itemprop-for-an-element-must-be-correct.html |
- fast/dom/MicroData/itemprop-must-be-read-only.html |
- fast/dom/MicroData/itemprop-reflected-by-itemProp-property.html |
- fast/dom/MicroData/itemref-add-remove-tokens.html |
- fast/dom/MicroData/itemref-attribute-reflected-by-itemRef-property.html |
- fast/dom/MicroData/itemref-for-an-element-must-be-correct.html |
- fast/dom/MicroData/itemref-must-be-read-only.html |
- fast/dom/MicroData/itemvalue-reflects-data-attr-on-object-element.html |
- fast/dom/MicroData/itemvalue-reflects-href-attr.html |
- fast/dom/MicroData/itemvalue-reflects-src-attribute-on-img-element.html |
- fast/dom/MicroData/itemvalue-reflects-the-content-attr-on-meta-element.html |
- fast/dom/MicroData/itemvalue-reflects-the-src-attr.html |
- fast/dom/MicroData/itemvalue-returns-element-itself.html |
- fast/dom/MicroData/itemvalue-returns-null.html |
- fast/dom/MicroData/itemvalue-throws-excpetion-onsetting-001.html |
- fast/dom/MicroData/itemvalue-throws-excpetion-onsetting-002.html |
- |
- * CMakeLists.txt: |
- * GNUmakefile.list.am: |
- * WebCore.gypi: |
- * WebCore.pro: |
- * WebCore.vcproj/WebCore.vcproj: |
- * bindings/js/JSHTMLElementCustom.cpp: |
- (WebCore::toJS): |
- (WebCore::JSHTMLElement::itemValue): |
- (WebCore::JSHTMLElement::setItemValue): |
- * bindings/v8/custom/V8HTMLElementCustom.cpp: |
- (WebCore::toV8Object): |
- (WebCore::V8HTMLElement::itemValueAccessorGetter): |
- (WebCore::V8HTMLElement::itemValueAccessorSetter): |
- * dom/MicroDataItemList.cpp: |
- * html/HTMLAnchorElement.cpp: |
- (WebCore::HTMLAnchorElement::itemValueText): |
- (WebCore::HTMLAnchorElement::setItemValueText): |
- * html/HTMLAnchorElement.h: |
- * html/HTMLAreaElement.cpp: |
- (WebCore::HTMLAreaElement::itemValueText): |
- (WebCore::HTMLAreaElement::setItemValueText): |
- * html/HTMLAreaElement.h: |
- * html/HTMLAttributeNames.in: |
- * html/HTMLElement.cpp: |
- (WebCore::HTMLElement::parseMappedAttribute): |
- (WebCore::HTMLElement::itemProp): Returns element's itemprop content attribute. |
- (WebCore::HTMLElement::setItemProp): Sets itemprop value on changing elemet's |
- itemprop attribute. |
- (WebCore::HTMLElement::itemRef): Returns element's itemref content attribute. |
- (WebCore::HTMLElement::setItemRef): Sets itemref value on changing elemet's |
- itemref attribute. |
- (WebCore::HTMLElement::setItemValue): Sets element's itemvalue. If the element has no |
- itemprop attribute or if element has an itemscope attribute, it throws INVALID_ACCESS_ERR |
- exception. |
- (WebCore::HTMLElement::itemValue): Returns the elements itemvalue. |
- |
- If the element has no itemprop attribute it returns null. |
- If the element has an itemscope attribute, it returns the element itself. |
- If the element is a meta element: It acts as it would if it was reflecting the element's |
- content content attribute. |
- If the element is an audio, embed, iframe, img, source, track, or video element: It acts |
- as it would if it was reflecting the element's src content attribute. |
- If the element is an a, area, or link element: It act as it would if it was reflecting the |
- href content attribute. |
- If the element is an object element: It acts as it would if it was reflecting the element's |
- data content attribute. |
- |
- (WebCore::HTMLElement::itemValueText): |
- (WebCore::HTMLElement::setItemValueText): |
- * html/HTMLElement.h: |
- * html/HTMLElement.idl: |
- * html/HTMLEmbedElement.cpp: |
- (WebCore::HTMLEmbedElement::itemValueText): |
- (WebCore::HTMLEmbedElement::setItemValueText): |
- * html/HTMLEmbedElement.h: |
- * html/HTMLIFrameElement.cpp: |
- (WebCore::HTMLIFrameElement::itemValueText): |
- (WebCore::HTMLIFrameElement::setItemValueText): |
- * html/HTMLIFrameElement.h: |
- * html/HTMLImageElement.cpp: |
- (WebCore::HTMLImageElement::itemValueText): |
- (WebCore::HTMLImageElement::setItemValueText): |
- * html/HTMLImageElement.h: |
- * html/HTMLLinkElement.cpp: |
- (WebCore::HTMLLinkElement::itemValueText): |
- (WebCore::HTMLLinkElement::setItemValueText): |
- * html/HTMLLinkElement.h: |
- * html/HTMLMediaElement.cpp: |
- (WebCore::HTMLMediaElement::itemValueText): |
- (WebCore::HTMLMediaElement::setItemValueText): |
- * html/HTMLMediaElement.h: |
- * html/HTMLMetaElement.cpp: |
- (WebCore::HTMLMetaElement::itemValueText): |
- (WebCore::HTMLMetaElement::setItemValueText): |
- * html/HTMLMetaElement.h: |
- * html/HTMLObjectElement.cpp: |
- (WebCore::HTMLObjectElement::itemValueText): |
- (WebCore::HTMLObjectElement::setItemValueText): |
- * html/HTMLObjectElement.h: |
- * html/HTMLSourceElement.cpp: |
- (WebCore::HTMLSourceElement::itemValueText): |
- (WebCore::HTMLSourceElement::setItemValueText): |
- * html/HTMLSourceElement.h: |
- * html/HTMLTrackElement.cpp: |
- (WebCore::HTMLTrackElement::itemValueText): |
- (WebCore::HTMLTrackElement::setItemValueText): |
- * html/HTMLTrackElement.h: |
- * html/MicroDataItemValue.cpp: Added. |
- (WebCore::MicroDataItemValue::MicroDataItemValue): |
- (WebCore::MicroDataItemValue::createFromString): |
- (WebCore::MicroDataItemValue::createFromNode): |
- * html/MicroDataItemValue.h: Added. |
- (WebCore::MicroDataItemValue::isNode): |
- (WebCore::MicroDataItemValue::getNode): |
- (WebCore::MicroDataItemValue::getString): |
- |
-2011-10-24 Joseph Pecoraro <joepeck@webkit.org> |
- |
- Remove unused instance variable |
- https://bugs.webkit.org/show_bug.cgi?id=70750 |
- |
- Reviewed by Dimitri Glazkov. |
- |
- * rendering/RenderTextControl.h: |
- |
-2011-10-24 Nico Weber <thakis@chromium.org> |
- |
- [chromium/mac] Fix an ODR violation. |
- https://bugs.webkit.org/show_bug.cgi?id=70753 |
- |
- Reviewed by Anders Carlsson. |
- |
- Both ScrollAnimatorNone and ScrollAnimatorMac define ScrollAnimator::create() on OS X. |
- Remove the unused one. |
- |
- * WebCore.gyp/WebCore.gyp: |
- |
-2011-10-24 Tim Horton <timothy_horton@apple.com> |
- |
- feColorMatrix doesn't use the correct default "matrix" type when no type attribute is specified |
- https://bugs.webkit.org/show_bug.cgi?id=70035 |
- <rdar://problem/10280177> |
- |
- Reviewed by Nikolas Zimmermann. |
- |
- Make feColorMatrix's type attribute be "matrix" by default. |
- |
- Test: svg/filters/feColorMatrix-default-type.svg |
- |
- * svg/SVGFEColorMatrixElement.cpp: |
- (WebCore::SVGFEColorMatrixElement::SVGFEColorMatrixElement): |
- |
-2011-10-24 Joshua Bell <jsbell@chromium.org> |
- |
- Unbreak binding generation tests |
- https://bugs.webkit.org/show_bug.cgi?id=70768 |
- |
- Reviewed by Darin Adler. |
- |
- Update the expected output of V8 binding tests to match |
- results c/o https://bugs.webkit.org/show_bug.cgi?id=70065 |
- |
- * bindings/scripts/test/V8/V8TestObj.cpp: |
- (WebCore::TestObjInternal::idbKeyCallback): |
- |
-2011-10-21 Nat Duca <nduca@chromium.org> |
- |
- [chromium] Add isTraceEventEnabled to PlatformSupport. |
- https://bugs.webkit.org/show_bug.cgi?id=70618 |
- |
- Reviewed by Darin Fisher. |
- |
- * platform/chromium/PlatformSupport.h: |
- |
-2011-10-24 Mihnea Ovidenie <mihnea@adobe.com> |
- |
- Use StringBuilder in CSSRuleList::rulesText()/CSSSelectorList::selectorsText() |
- https://bugs.webkit.org/show_bug.cgi?id=70746 |
- |
- Reviewed by Tony Chang. |
- |
- Covered by existing tests. |
- |
- * css/CSSRuleList.cpp: |
- (WebCore::CSSRuleList::rulesText): |
- * css/CSSSelectorList.cpp: |
- (WebCore::CSSSelectorList::selectorsText): |
- * css/WebKitCSSKeyframesRule.cpp: |
- (WebCore::WebKitCSSKeyframesRule::cssText): Use CSSRuleList::rulesText() method instead of explicit loop. |
- |
-2011-10-24 Michael Saboff <msaboff@apple.com> |
- |
- WebKit doesn't build with recent changes to libdispatch |
- https://bugs.webkit.org/show_bug.cgi?id=70737 |
- |
- Added initialization for new WebKitSystemInterface function |
- CreateVMPressureDispatchOnMainQueue. Changed MemoryPressureHandler::install |
- to use new function. |
- |
- Reviewed by Geoffrey Garen. |
- |
- No new tests. Moved functionality to WebKitSystemInterface. |
- |
- * WebCore.exp.in: |
- * platform/mac/MemoryPressureHandlerMac.mm: |
- (WebCore::MemoryPressureHandler::install): |
- * platform/mac/WebCoreSystemInterface.h: |
- * platform/mac/WebCoreSystemInterface.mm: |
- |
-2011-10-24 Joshua Bell <jsbell@chromium.org> |
- |
- IndexedDB: Throw DATA_ERR on invalid keys, remove null key support |
- https://bugs.webkit.org/show_bug.cgi?id=70065 |
- |
- Reviewed by Tony Chang. |
- |
- Part 1 of a two-sided commit. Renames IDBKey::NullType to |
- IDBKey::InvalidType. JS binding yields this type rather |
- than throwing immediately so the correct exception type can |
- be raised. Checks for invalid keys added to front-end classes. |
- |
- * bindings/js/JSIDBKeyCustom.cpp: |
- (WebCore::toJS): |
- * bindings/scripts/CodeGeneratorV8.pm: |
- (TypeCanFailConversion): |
- * bindings/v8/IDBBindingUtilities.cpp: |
- (WebCore::createIDBKeyFromValue): |
- * bindings/v8/custom/V8IDBKeyCustom.cpp: |
- (WebCore::toV8): |
- * storage/IDBCursor.cpp: |
- (WebCore::IDBCursor::continueFunction): |
- * storage/IDBFactory.cpp: |
- (WebCore::IDBFactory::cmp): |
- * storage/IDBIndex.cpp: |
- (WebCore::IDBIndex::get): |
- (WebCore::IDBIndex::getKey): |
- * storage/IDBKey.cpp: |
- (WebCore::IDBKey::IDBKey): |
- (WebCore::IDBKey::compare): |
- * storage/IDBKey.h: |
- (WebCore::IDBKey::createInvalid): |
- * storage/IDBKeyRange.cpp: |
- (WebCore::IDBKeyRange::only): |
- (WebCore::IDBKeyRange::lowerBound): |
- (WebCore::IDBKeyRange::upperBound): |
- (WebCore::IDBKeyRange::bound): |
- * storage/IDBKeyRange.h: |
- (WebCore::IDBKeyRange::lowerBound): |
- (WebCore::IDBKeyRange::upperBound): |
- (WebCore::IDBKeyRange::bound): |
- * storage/IDBKeyRange.idl: |
- * storage/IDBLevelDBCoding.cpp: |
- (WebCore::IDBLevelDBCoding::encodeIDBKey): |
- (WebCore::IDBLevelDBCoding::decodeIDBKey): |
- * storage/IDBObjectStore.cpp: |
- (WebCore::IDBObjectStore::get): |
- (WebCore::IDBObjectStore::add): |
- (WebCore::IDBObjectStore::put): |
- (WebCore::IDBObjectStore::deleteFunction): |
- * storage/IDBObjectStoreBackendImpl.cpp: |
- (WebCore::IDBObjectStoreBackendImpl::put): |
- (WebCore::IDBObjectStoreBackendImpl::putInternal): |
- (WebCore::IDBObjectStoreBackendImpl::deleteFunction): |
- * storage/IDBSQLiteBackingStore.cpp: |
- (WebCore::whereSyntaxForKey): |
- (WebCore::bindKeyToQuery): |
- (WebCore::lowerCursorWhereFragment): |
- (WebCore::upperCursorWhereFragment): |
- (WebCore::bindKeyToQueryWithNulls): |
- (WebCore::keyFromQuery): |
- |
-2011-10-24 Alok Priyadarshi <alokp@chromium.org> |
- |
- [chromium] Make accelerated drawing work with threaded compositing |
- https://bugs.webkit.org/show_bug.cgi?id=70656 |
- |
- Reviewed by James Robinson. |
- |
- Make sure that the compositor context is not used outside LayerTextureUpdater::updateTextureRect(). |
- Created a FrameBuffer class that wraps skia gpu canvas and manages GL resources. |
- |
- * platform/graphics/chromium/ContentLayerChromium.cpp: |
- (WebCore::ContentLayerChromium::createTextureUpdater): |
- * platform/graphics/chromium/LayerTextureUpdaterCanvas.cpp: |
- (WebCore::FrameBuffer::FrameBuffer::FrameBuffer): |
- (WebCore::FrameBuffer::FrameBuffer::~FrameBuffer): |
- (WebCore::FrameBuffer::FrameBuffer::initialize): |
- (WebCore::LayerTextureUpdaterSkPicture::LayerTextureUpdaterSkPicture): |
- (WebCore::LayerTextureUpdaterSkPicture::~LayerTextureUpdaterSkPicture): |
- (WebCore::LayerTextureUpdaterSkPicture::prepareToUpdate): |
- (WebCore::LayerTextureUpdaterSkPicture::updateTextureRect): |
- * platform/graphics/chromium/LayerTextureUpdaterCanvas.h: |
- * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: |
- (WebCore::CCLayerTreeHost::initialize): |
- |
-2011-10-24 Adam Klein <adamk@chromium.org> |
- |
- Optimize Node::isInShadowTree to execute in constant-time |
- https://bugs.webkit.org/show_bug.cgi?id=67686 |
- |
- Reviewed by Dimitri Glazkov. |
- |
- No expected change in behavior, so no new tests. |
- |
- * dom/Node.cpp: |
- (WebCore::Node::isInShadowTree): |
- |
-2011-10-24 Nat Duca <nduca@chromium.org> |
- |
- [chromium] Expose WebThread::postDelayedTask to CCThread |
- https://bugs.webkit.org/show_bug.cgi?id=70712 |
- |
- Reviewed by James Robinson. |
- |
- * platform/graphics/chromium/cc/CCThread.h: |
- |
-2011-10-24 Levi Weintraub <leviw@chromium.org> |
- |
- <svg> fails to use explicit width and height inside <html> inside IFRAME |
- https://bugs.webkit.org/show_bug.cgi?id=64823 |
- |
- Reviewed by Nikolas Zimmermann. |
- |
- Checking that embedded SVG is in an SVG document before negotiating size |
- with the host document. |
- |
- Test: svg/as-object/svg-embedded-in-html-in-iframe.html |
- |
- * rendering/svg/RenderSVGRoot.cpp: |
- (WebCore::isEmbeddedThroughFrameContainingSVGDocument): |
- (WebCore::RenderSVGRoot::computeReplacedLogicalWidth): |
- (WebCore::RenderSVGRoot::computeReplacedLogicalHeight): |
- |
-2011-10-24 Rafael Weinstein <rafaelw@chromium.org> |
- |
- Node::registeredMutationObserversOfType should be prefixed by "get" |
- https://bugs.webkit.org/show_bug.cgi?id=70359 |
- |
- Reviewed by Ojan Vafai. |
- |
- This change renames Node::registeredMutationObserversOfType to |
- Node::getRegisteredMutationObserversOfType per the WebKit style guide. |
- |
- No tests needed as the change is only method renaming. |
- |
- * dom/CharacterData.cpp: |
- (WebCore::CharacterData::dispatchModifiedEvent): |
- * dom/ChildListMutationScope.cpp: |
- (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::incrementScopingLevel): |
- * dom/Element.cpp: |
- (WebCore::enqueueAttributesMutationRecord): |
- * dom/Node.cpp: |
- (WebCore::Node::getRegisteredMutationObserversOfType): |
- * dom/Node.h: |
- |
-2011-10-24 Mark Hahnenberg <mhahnenberg@apple.com> |
- |
- Rename static getOwnPropertySlot to getOwnPropertySlotByIndex |
- https://bugs.webkit.org/show_bug.cgi?id=70271 |
- |
- Reviewed by Darin Adler. |
- |
- No new tests. |
- |
- Renaming versions of getOwnPropertySlot that use an unsigned as the property |
- name to "getOwnPropertySlotByIndex" in preparation for adding them to the |
- MethodTable, which requires unique names for each method. |
- |
- * bindings/js/SerializedScriptValue.cpp: |
- (WebCore::CloneSerializer::getSparseIndex): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- (GenerateHeader): |
- (GenerateImplementation): |
- * bridge/runtime_array.cpp: |
- (JSC::RuntimeArray::getOwnPropertySlotVirtual): |
- (JSC::RuntimeArray::getOwnPropertySlotByIndex): |
- * bridge/runtime_array.h: |
- |
-2011-10-24 Joshua Bell <jsbell@chromium.org> |
- |
- IndexedDB add should fail adding an inline and passed key simultaneously |
- https://bugs.webkit.org/show_bug.cgi?id=58598 |
- |
- Reviewed by Tony Chang. |
- |
- Test: storage/indexeddb/mozilla/key-requirements-inline-and-passed.html |
- |
- Implement the spec clause: passing a key into add() or put() when |
- the object store has a key path ("inline key") is an error. |
- |
- * storage/IDBObjectStoreBackendImpl.cpp: |
- (WebCore::IDBObjectStoreBackendImpl::put): |
- |
-2011-10-24 Simon Fraser <simon.fraser@apple.com> |
- |
- Improve debug compositing border colors |
- https://bugs.webkit.org/show_bug.cgi?id=70742 |
- |
- Reviewed by Dean Jackson. |
- |
- Improve the appearance of the debug borders used to indicate compositing: |
- 1. Use green for normal layers, orange for tiled layers |
- 2. Make the repaint counter translucent |
- 3. Change the color of the repaint counter to show when accelerated drawing is enabled. |
- |
- * platform/graphics/GraphicsLayer.cpp: |
- (WebCore::GraphicsLayer::updateDebugIndicators): |
- * platform/graphics/mac/WebLayer.mm: |
- (drawLayerContents): |
- |
-2011-10-24 Carlos Garcia Campos <cgarcia@igalia.com> |
- |
- [GTK] Fix make distcheck build |
- https://bugs.webkit.org/show_bug.cgi?id=69745 |
- |
- Reviewed by Martin Robinson. |
- |
- * GNUmakefile.am: |
- * GNUmakefile.list.am: |
- |
-2011-10-24 Alexander Pavlov <apavlov@chromium.org> |
- |
- Web Inspector: [Styles] Adding a new style rule breaks things |
- https://bugs.webkit.org/show_bug.cgi?id=70721 |
- |
- Reviewed by Pavel Feldman. |
- |
- We had used to swap BlankStylePropertiesSection prototype and recently got rid of this hack, |
- but still had dependencies on the old behavior. |
- |
- * inspector/front-end/StylesSidebarPane.js: |
- (WebInspector.StylesSidebarPane.prototype._refreshStyleRules): |
- (WebInspector.ComputedStylePropertiesSection.prototype.rebuildComputedTrace): |
- (WebInspector.BlankStylePropertiesSection.prototype.get isBlank): |
- (WebInspector.BlankStylePropertiesSection.prototype.expand): |
- (WebInspector.BlankStylePropertiesSection.prototype.editingSelectorCommitted.successCallback): |
- (WebInspector.BlankStylePropertiesSection.prototype.editingSelectorCommitted): |
- (WebInspector.BlankStylePropertiesSection.prototype.editingSelectorCancelled): |
- (WebInspector.BlankStylePropertiesSection.prototype.makeNormal): |
- |
-2011-10-24 Pavel Feldman <pfeldman@google.com> |
- |
- Web Inspector: do not force document loading on reload (should happen on demand). |
- https://bugs.webkit.org/show_bug.cgi?id=70724 |
- |
- Reviewed by Yury Semikhatsky. |
- |
- * inspector/front-end/ConsoleMessage.js: |
- (WebInspector.ConsoleMessageImpl.prototype._formatParameterAsNode): |
- * inspector/front-end/DOMAgent.js: |
- (WebInspector.DOMAgent.prototype._documentUpdated): |
- * inspector/front-end/ElementsPanel.js: |
- (WebInspector.ElementsPanel.prototype.wasShown): |
- * inspector/front-end/ElementsTreeOutline.js: |
- (WebInspector.ElementsTreeOutline.prototype.setVisible): |
- |
-2011-10-24 Eric Carlson <eric.carlson@apple.com> |
- |
- Drop MutableTextTrack and make TextTrack always mutable |
- https://bugs.webkit.org/show_bug.cgi?id=62890 |
- |
- The MutableTextTrack type was removed from the spec, all text tracks are now mutable. |
- |
- Reviewed by Sam Weinig. |
- |
- No new tests needed, the changes just refactor existing code. |
- |
- * CMakeLists.txt: Remove all traces of MutableTextTrack. |
- * CodeGenerators.pri: Ditto. |
- * DerivedSources.cpp: Ditto. |
- * DerivedSources.make: Ditto. |
- * GNUmakefile.list.am: Ditto. |
- * WebCore.gypi: Ditto. |
- * WebCore.pro: Ditto. |
- * WebCore.xcodeproj/project.pbxproj: Ditto. |
- * bindings/gobject/GNUmakefile.am: Ditto. |
- * bindings/gobject/WebKitHTMLElementWrapperFactory.cpp: Ditto. |
- * bindings/js/JSBindingsAllInOne.cpp: Ditto. |
- |
- * bindings/js/JSTextTrackCustom.cpp: Removed. |
- |
- * html/HTMLMediaElement.cpp: |
- (WebCore::HTMLMediaElement::addTrack): Return a TextTrack instead of a MutableTextTrack. |
- * html/HTMLMediaElement.h: |
- * html/HTMLMediaElement.idl: |
- |
- * html/LoadableTextTrack.cpp: |
- (WebCore::LoadableTextTrack::LoadableTextTrack): Return a TextTrack instead of a MutableTextTrack. |
- |
- * html/MutableTextTrack.cpp: Removed. |
- * html/MutableTextTrack.h: Removed. |
- * html/MutableTextTrack.idl: Removed. |
- |
- * html/TextTrack.cpp: |
- (WebCore::TextTrack::TextTrack): Remove Type parameter. |
- (WebCore::TextTrack::addCue): New, moved from MutableTextTrack. |
- (WebCore::TextTrack::removeCue): Ditto. |
- (WebCore::TextTrack::newCuesLoaded): Ditto. |
- (WebCore::TextTrack::fetchNewestCues): Ditto. |
- * html/TextTrack.h: |
- (WebCore::TextTrack::create): Remove Type parameter |
- * html/TextTrack.idl: Add addCue and removeCue. |
- |
-2011-10-24 Yury Semikhatsky <yurys@chromium.org> |
- |
- Web Inspector: allow debugging dedicated worker initialization |
- https://bugs.webkit.org/show_bug.cgi?id=70723 |
- |
- Now it is possible to pause each new dedicated worker on start. The user |
- needs to select a checkbox and then each newly created dedicated worker |
- will be paused on the first statement until the user resumes it explicitely. |
- |
- Reviewed by Pavel Feldman. |
- |
- * inspector/Inspector.json: resume command is added. It is similar to GDB's run |
- command, i.e. worker execution will be paused until explicit "resume" command is |
- received. |
- * inspector/InspectorInstrumentation.cpp: |
- (WebCore::InspectorInstrumentation::shouldPauseDedicatedWorkerOnStartImpl): |
- * inspector/InspectorInstrumentation.h: |
- (WebCore::InspectorInstrumentation::shouldPauseDedicatedWorkerOnStart): |
- * inspector/InspectorRuntimeAgent.cpp: |
- (WebCore::InspectorRuntimeAgent::resume): |
- (WebCore::InspectorRuntimeAgent::pauseWorkerContext): |
- * inspector/InspectorRuntimeAgent.h: |
- * inspector/InspectorWorkerAgent.cpp: |
- (WebCore::InspectorWorkerAgent::shouldPauseDedicatedWorkerOnStart): returns true |
- if the user chose to automatically pause all new workers. |
- * inspector/InspectorWorkerAgent.h: |
- * inspector/WorkerInspectorController.cpp: |
- (WebCore::WorkerInspectorController::resume): |
- * inspector/front-end/WorkerManager.js: |
- (WebInspector.WorkerManager.loaded): |
- (WebInspector.WorkerManager.loadCompleted): |
- (WebInspector.WorkerManager.prototype._workerCreated): |
- (WebInspector.WorkerManager.prototype.openWorkerInspector): |
- (WebInspector.WorkerManager.prototype._openInspectorWindow): |
- * inspector/front-end/inspector.js: |
- (WebInspector.loaded): |
- * workers/Worker.cpp: |
- (WebCore::Worker::notifyFinished): |
- * workers/WorkerContextProxy.h: |
- * workers/WorkerMessagingProxy.cpp: |
- (WebCore::WorkerMessagingProxy::startWorkerContext): additional flag specifying |
- worker initial state(paused/not paused) is added to the factory method. The flag |
- will be carried to the corresponding WorkerThread. |
- (WebCore::WorkerMessagingProxy::connectToInspector): |
- * workers/WorkerMessagingProxy.h: |
- |
-2011-10-21 Nikita Vasilyev <me@elv1s.ru> |
- |
- Web Inspector: Make indent configurable |
- https://bugs.webkit.org/show_bug.cgi?id=69986 |
- |
- Add a setting to choose an indent from "8 spaces", "4 spaces" (default), "2 spaces", or "Tab character". |
- |
- Reviewed by Pavel Feldman. |
- |
- * English.lproj/localizedStrings.js: |
- * inspector/front-end/SettingsScreen.js: |
- (WebInspector.SettingsScreen): |
- (WebInspector.SettingsScreen.prototype._createSelectSetting.get p): |
- (WebInspector.SettingsScreen.prototype._createSelectSetting.changeListener): |
- * inspector/front-end/TextEditorModel.js: |
- (WebInspector.TextEditorModel): |
- * inspector/front-end/TextViewer.js: |
- |
-2011-10-17 Andrey Kosyakov <caseq@chromium.org> |
- |
- Web Inspector: add "Add to Watch" option to context menu on selection in source frame |
- https://bugs.webkit.org/show_bug.cgi?id=69924 |
- |
- Reviewed by Pavel Feldman. |
- |
- - added 'Add to Watch' context menu item in scripts panel; |
- - include both default and custom items into a context menu if selection is present; |
- |
- * inspector/front-end/ScriptsPanel.js: |
- (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.suggestedFileName): |
- (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.addToWatch): |
- * inspector/front-end/SourceFrame.js: |
- (WebInspector.SourceFrame.prototype.populateSelectionContextMenu): |
- (WebInspector.TextViewerDelegateForSourceFrame.prototype.populateSelectionContextMenu): |
- (WebInspector.SourceFrameDelegate.prototype.suggestedFileName): |
- (WebInspector.SourceFrameDelegate.prototype.addToWatch): |
- * inspector/front-end/TextViewer.js: |
- (WebInspector.TextViewer.prototype._contextMenu): |
- (WebInspector.TextViewerDelegate.prototype.populateSelectionContextMenu): |
- * inspector/front-end/WatchExpressionsSidebarPane.js: |
- (WebInspector.WatchExpressionsSidebarPane.prototype.addExpression): |
- (WebInspector.WatchExpressionsSidebarPane.prototype._addButtonClicked): |
- (WebInspector.WatchExpressionsSection.prototype.addExpression): |
- (WebInspector.WatchExpressionsSection.prototype.addNewExpressionAndEdit): |
- * page/ContextMenuController.cpp: |
- (WebCore::ContextMenuController::showContextMenu): |
- |
-2011-10-21 Alexander Pavlov <apavlov@chromium.org> |
- |
- Web Inspector: Audits hang on pages without img[src] elements |
- https://bugs.webkit.org/show_bug.cgi?id=70604 |
- |
- Reviewed by Pavel Feldman. |
- |
- Test: inspector/audits/audits-panel-noimages-functional.html |
- |
- * inspector/front-end/AuditRules.js: |
- (WebInspector.AuditRules.ImageDimensionsRule.prototype.doRun.getStyles): |
- |
-2011-10-24 Deepak Sherveghar <bpwv64@motorola.com> |
- |
- [Qt] fast/events/media-focus-in-standalone-media-document.html fails |
- https://bugs.webkit.org/show_bug.cgi?id=70110 |
- |
- Reviewed by Simon Hausmann. |
- |
- ExtensionMap should also include media files mapping. |
- |
- * platform/qt/MIMETypeRegistryQt.cpp: Added media files mapping. |
- |
-2011-10-24 Pavel Feldman <pfeldman@google.com> |
- |
- Not reviewed: [chromium] follow up to 98228 - added missing css. |
- |
- * WebCore.gypi: |
- |
-2011-10-24 Pavel Feldman <pfeldman@chromium.org> |
- |
- Web Inspector: Make CSS files load upon View demand; split inspector.css into views. |
- https://bugs.webkit.org/show_bug.cgi?id=70694 |
- |
- Reviewed by Yury Semikhatsky. |
- |
- * WebCore.gypi: |
- * WebCore.vcproj/WebCore.vcproj: |
- * inspector/front-end/AuditsPanel.js: |
- (WebInspector.AuditsPanel): |
- * inspector/front-end/DataGrid.js: |
- (WebInspector.DataGrid): |
- * inspector/front-end/ElementsPanel.js: |
- * inspector/front-end/NetworkPanel.js: |
- (WebInspector.NetworkLogView): |
- * inspector/front-end/PanelEnablerView.js: |
- * inspector/front-end/ProfilesPanel.js: |
- * inspector/front-end/ResourceView.js: |
- (WebInspector.ResourceView): |
- * inspector/front-end/ResourcesPanel.js: |
- * inspector/front-end/ScriptsPanel.js: |
- * inspector/front-end/TextViewer.js: |
- * inspector/front-end/TimelinePanel.js: |
- (WebInspector.TimelinePanel): |
- * inspector/front-end/WebKit.qrc: |
- * inspector/front-end/WelcomeView.js: |
- (WebInspector.WelcomeView): |
- * inspector/front-end/auditsPanel.css: Renamed from Source/WebCore/inspector/front-end/audits.css. |
- |
-2011-10-24 Pavel Feldman <pfeldman@chromium.org> |
- |
- Web Inspector: make view load CSS files upon show. |
- https://bugs.webkit.org/show_bug.cgi?id=70696 |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Test: inspector/view-css.html |
- |
- * inspector/front-end/View.js: |
- (WebInspector.View): |
- (WebInspector.View.prototype._callOnVisibleChildren): |
- (WebInspector.View.prototype._processWillShow): |
- (WebInspector.View.prototype._processWasShown): |
- (WebInspector.View.prototype._processWillHide): |
- (WebInspector.View.prototype._processWasHidden): |
- (WebInspector.View.prototype._processOnResize): |
- (WebInspector.View.prototype.show): |
- (WebInspector.View.prototype.detach): |
- (WebInspector.View.prototype.registerRequiredCSS): |
- (WebInspector.View.prototype._loadCSSIfNeeded): |
- (WebInspector.View.prototype._doLoadCSS): |
- (WebInspector.View.prototype._disableCSSIfNeeded): |
- (WebInspector.View.prototype._doUnloadCSS): |
- |
-2011-10-24 Pavel Feldman <pfeldman@chromium.org> |
- |
- Web Inspector: make DataGrid a view |
- https://bugs.webkit.org/show_bug.cgi?id=70695 |
- |
- Reviewed by Yury Semikhatsky. |
- |
- * inspector/front-end/ApplicationCacheItemsView.js: |
- (WebInspector.ApplicationCacheItemsView.prototype._createDataGrid): |
- * inspector/front-end/CookieItemsView.js: |
- (WebInspector.CookieItemsView.prototype._updateWithCookies): |
- (WebInspector.SimpleCookiesTable): |
- (WebInspector.SimpleCookiesTable.prototype.setCookies): |
- * inspector/front-end/CookiesTable.js: |
- (WebInspector.CookiesTable): |
- * inspector/front-end/DOMStorageItemsView.js: |
- (WebInspector.DOMStorageItemsView.prototype._showDOMStorageEntries): |
- * inspector/front-end/DataGrid.js: |
- (WebInspector.DataGrid): |
- (WebInspector.DataGrid.prototype.onResize): |
- * inspector/front-end/DatabaseQueryView.js: |
- (WebInspector.DatabaseQueryView.prototype._queryFinished): |
- (WebInspector.DatabaseQueryView.prototype._queryError): |
- (WebInspector.DatabaseQueryView.prototype._appendViewQueryResult): |
- (WebInspector.DatabaseQueryView.prototype._appendErrorQueryResult): |
- (WebInspector.DatabaseQueryView.prototype._appendQueryResult): |
- * inspector/front-end/DatabaseTableView.js: |
- (WebInspector.DatabaseTableView.prototype._queryFinished): |
- * inspector/front-end/DetailedHeapshotView.js: |
- (WebInspector.DetailedHeapshotView.prototype.wasShown.profileCallback2): |
- (WebInspector.DetailedHeapshotView.prototype.wasShown): |
- (WebInspector.DetailedHeapshotView.prototype.onResize): |
- * inspector/front-end/NetworkPanel.js: |
- (WebInspector.NetworkLogView.prototype.onResize): |
- (WebInspector.NetworkLogView.prototype._createTable): |
- * inspector/front-end/ProfileView.js: |
- * inspector/front-end/ResourceCookiesView.js: |
- (WebInspector.ResourceCookiesView.prototype._buildCookiesTable): |
- * inspector/front-end/ResourceView.js: |
- (WebInspector.ResourceView): |
- |
-2011-10-24 Yosifumi Inoue <yosin@chromium.org> |
- |
- [Forms] Setting defaultValue should hide an input placeholder. |
- https://bugs.webkit.org/show_bug.cgi?id=70586 |
- |
- Reviewed by Kent Tamura. |
- |
- Tests: fast/forms/placeholder-and-default-value-expected.html |
- fast/forms/placeholder-and-default-value.html |
- |
- When input element has default value, we should not display |
- placeholder. Changing default value should tell rendering engine about |
- changing display of input element. |
- |
- * html/HTMLInputElement.cpp: |
- (WebCore::HTMLInputElement::parseMappedAttribute): Calls updatePlaceholderVisibility for valueAttr if it doesn't have dirty value. |
- |
-2011-10-23 Darin Adler <darin@apple.com> |
- |
- Eliminate OptionElement (second and final part) |
- https://bugs.webkit.org/show_bug.cgi?id=70710 |
- |
- Reviewed by Kent Tamura. |
- |
- * CMakeLists.txt: Removed OptionElement.h and OptionElement.cpp. |
- * GNUmakefile.list.am: Ditto. |
- * WebCore.gypi: Ditto. |
- * WebCore.pro: Ditto. |
- * WebCore.vcproj/WebCore.vcproj: Ditto. |
- * WebCore.xcodeproj/project.pbxproj: Ditto. |
- * dom/DOMAllInOne.cpp: Ditto. |
- |
- * dom/OptionElement.cpp: Removed. |
- * dom/OptionElement.h: Removed. |
- |
- * html/HTMLSelectElement.cpp: |
- (WebCore::HTMLSelectElement::saveLastSelection): Eliminate use of the |
- toOptionElement function. Use hasTagName and toHTMLOptionElement instead. |
- (WebCore::HTMLSelectElement::setActiveSelectionAnchorIndex): Ditto. |
- (WebCore::HTMLSelectElement::updateListBoxSelection): Ditto. |
- (WebCore::HTMLSelectElement::listBoxOnChange): Ditto. |
- (WebCore::HTMLSelectElement::recalcListItems): Ditto. |
- (WebCore::HTMLSelectElement::selectedIndex): Ditto. |
- (WebCore::HTMLSelectElement::setSelectedIndex): Ditto. |
- (WebCore::HTMLSelectElement::deselectItemsWithoutValidation): Ditto. |
- (WebCore::HTMLSelectElement::saveFormControlState): Ditto. |
- (WebCore::HTMLSelectElement::restoreFormControlState): Ditto. |
- (WebCore::HTMLSelectElement::appendFormData): Ditto. |
- (WebCore::HTMLSelectElement::reset): Ditto. |
- (WebCore::HTMLSelectElement::updateSelectedState): Ditto. |
- (WebCore::HTMLSelectElement::lastSelectedListIndex): Ditto. |
- (WebCore::HTMLSelectElement::typeAheadFind): Ditto. |
- (WebCore::HTMLSelectElement::accessKeySetSelectedIndex): Ditto. |
- |
- * html/HTMLSelectElement.h: Changed the argument type of the |
- deselectItemsWithoutValidation function to HTMLElement* since that's |
- the type of an item in our items vector now. |
- |
-2011-10-23 Yuzo Fujishima <yuzo@google.com> |
- |
- [chromium] Attempt to fix compilation error for Chromium OS. |
- |
- Unreviewed. |
- |
- * mediastream/MediaStreamTrack.cpp: |
- (WebCore::MediaStreamTrack::kind): |
- |
-2011-10-23 Adam Barth <abarth@webkit.org> |
- |
- <img crossorigin> should fail to load when CORS check fails |
- https://bugs.webkit.org/show_bug.cgi?id=69732 |
- |
- Reviewed by Darin Adler. |
- |
- When loading an image with the crossorigin attribute, the spec says |
- that we're not supposed to load the image if the CORS check fails. |
- This "fails fast" behavior is intended to help developers understand |
- whether they've configured CORS correctly (instead of only catching the |
- error later when trying to read back the canvas). |
- |
- Our new behavior matches the spec and Firefox. |
- |
- Test: http/tests/security/img-with-failed-cors-check-fails-to-load.html |
- |
- * loader/ImageLoader.cpp: |
- (WebCore::ImageLoader::notifyFinished): |
- |
-2011-10-23 Noel Gordon <noel.gordon@gmail.com> |
- |
- [chromium] Remove GeolocationServiceGtk.{h,cpp} from the gyp projects |
- https://bugs.webkit.org/show_bug.cgi?id=70702 |
- |
- Reviewed by Kent Tamura. |
- |
- platform/gtk/GeolocationServiceGtk.{h,cpp} were removed in r97746, so |
- remove references to these files from the gyp files. |
- |
- * WebCore.gypi: |
- |
-2011-10-23 Andreas Kling <kling@webkit.org> |
- |
- Move StyleSheet::addSubresourceStyleURLs() to CSSStyleSheet. |
- https://bugs.webkit.org/show_bug.cgi?id=70690 |
- |
- Reviewed by Antti Koivisto. |
- |
- This method isn't part of the StyleSheet DOM interface, nor is it used by |
- XSLStyleSheet, so move it down to CSSStyleSheet and make it non-virtual. |
- Also remove some nearby unneeded casts from CSSStyleSheet* to StyleSheet*. |
- |
- * css/CSSStyleSheet.h: |
- * css/StyleSheet.h: |
- * html/HTMLLinkElement.cpp: |
- (WebCore::HTMLLinkElement::isLoading): |
- (WebCore::HTMLLinkElement::addSubresourceAttributeURLs): |
- * html/HTMLStyleElement.cpp: |
- (WebCore::HTMLStyleElement::addSubresourceAttributeURLs): |
- (WebCore::HTMLStyleElement::disabled): |
- (WebCore::HTMLStyleElement::setDisabled): |
- |
-2011-10-23 Andreas Kling <kling@webkit.org> |
- |
- Remove style sheet constructors that take a style sheet parent. |
- https://bugs.webkit.org/show_bug.cgi?id=70692 |
- |
- Reviewed by Antti Koivisto. |
- |
- Style sheets are never directly parented by other style sheets, they always |
- go via a rule (such as @import.) |
- |
- * css/CSSStyleSheet.cpp: |
- * css/CSSStyleSheet.h: |
- (WebCore::CSSStyleSheet::create): |
- * css/StyleSheet.cpp: |
- * css/StyleSheet.h: |
- |
-2011-10-23 Mark Hahnenberg <mhahnenberg@apple.com> |
- |
- Add deleteProperty to the MethodTable |
- https://bugs.webkit.org/show_bug.cgi?id=70162 |
- |
- Reviewed by Sam Weinig. |
- |
- No new tests. |
- |
- * WebCore.exp.in: |
- |
-2011-10-23 Adam Barth <abarth@webkit.org> |
- |
- Update baselines for run-bindings-tests. |
- |
- * bindings/scripts/test/JS/JSTestInterface.cpp: |
- * bindings/scripts/test/JS/JSTestInterface.h: |
- * bindings/scripts/test/JS/JSTestObj.cpp: |
- |
-2011-10-23 Mark Hahnenberg <mhahnenberg@apple.com> |
- |
- Remove getConstructDataVirtual |
- https://bugs.webkit.org/show_bug.cgi?id=70638 |
- |
- Reviewed by Darin Adler. |
- |
- No new tests. |
- |
- Removed all declarations and definitions of getConstructDataVirtual. |
- Also replaced all call sites to getConstructDataVirtual with a |
- corresponding lookup in the MethodTable. |
- |
- * WebCore.exp.in: |
- * bindings/js/JSAudioConstructor.cpp: |
- * bindings/js/JSAudioConstructor.h: |
- * bindings/js/JSImageConstructor.cpp: |
- * bindings/js/JSImageConstructor.h: |
- * bindings/js/JSOptionConstructor.cpp: |
- * bindings/js/JSOptionConstructor.h: |
- * bindings/js/ScriptFunctionCall.cpp: |
- (WebCore::ScriptFunctionCall::construct): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- (GenerateConstructorDeclaration): |
- (GenerateConstructorDefinition): |
- * bridge/runtime_object.cpp: |
- * bridge/runtime_object.h: |
- |
-2011-10-22 Geoffrey Garen <ggaren@apple.com> |
- |
- Separated string lifetime bits from character buffer state bits |
- https://bugs.webkit.org/show_bug.cgi?id=70673 |
- |
- Reviewed by Anders Carlsson. |
- |
- * bindings/scripts/CodeGeneratorJS.pm: |
- (GenerateHashValue): Updated for string hashing changes in JavaScriptCore. |
- |
-2011-10-22 Pratik Solanki <psolanki@apple.com> |
- |
- HTTPBodyStream in NSURLRequest gets lost when using CFNetwork loader |
- https://bugs.webkit.org/show_bug.cgi?id=70583 |
- <rdar://problem/10281227> |
- |
- Reviewed by Darin Adler. |
- |
- Call WebCore::setHTTPBody() only if we actually have a FormData. Unconditionally calling it |
- results in us passing 0 to wkCFURLRequestSetHTTPRequestBodyParts() which resets any input |
- stream specified by the user on the NSURLRequest. This code matches the behavior in the |
- Foundation version of doUpdatePlatformRequest(). |
- |
- * platform/network/cf/ResourceRequestCFNet.cpp: |
- (WebCore::ResourceRequest::doUpdatePlatformRequest): |
- |
-2011-10-22 Adam Bergkvist <adam.bergkvist@ericsson.com> |
- |
- Move all MediaStream and PeerConnection related non-platform files to WebCore/mediastream |
- https://bugs.webkit.org/show_bug.cgi?id=70233 |
- |
- Reviewed by Adam Barth. |
- |
- Removed affected files from build systems that don't have a MEDIA_STREAM build flag. |
- |
- No new tests since we're just moving files. |
- |
- * CMakeLists.txt: |
- * CodeGenerators.pri: |
- * DerivedSources.cpp: |
- * DerivedSources.make: |
- * GNUmakefile.am: |
- * GNUmakefile.list.am: |
- * UseJSC.cmake: |
- * WebCore.gyp/WebCore.gyp: |
- * WebCore.gypi: |
- * WebCore.pro: |
- * WebCore.vcproj/WebCore.vcproj: |
- * WebCore.xcodeproj/project.pbxproj: |
- * bindings/js/JSBindingsAllInOne.cpp: |
- * dom/DOMAllInOne.cpp: |
- * dom/DispatchTask.h: Removed unused file. |
- * dom/ScriptExecutionContext.cpp: |
- * mediastream/LocalMediaStream.cpp: Renamed from Source/WebCore/dom/LocalMediaStream.cpp. |
- (WebCore::LocalMediaStream::create): |
- (WebCore::LocalMediaStream::LocalMediaStream): |
- (WebCore::LocalMediaStream::stop): |
- (WebCore::LocalMediaStream::stopTimerFired): |
- (WebCore::LocalMediaStream::~LocalMediaStream): |
- (WebCore::LocalMediaStream::toLocalMediaStream): |
- * mediastream/LocalMediaStream.h: Renamed from Source/WebCore/dom/LocalMediaStream.h. |
- * mediastream/LocalMediaStream.idl: Renamed from Source/WebCore/dom/LocalMediaStream.idl. |
- * mediastream/MediaStream.cpp: Renamed from Source/WebCore/dom/MediaStream.cpp. |
- (WebCore::MediaStream::create): |
- (WebCore::MediaStream::MediaStream): |
- (WebCore::MediaStream::~MediaStream): |
- (WebCore::MediaStream::readyState): |
- (WebCore::MediaStream::toMediaStream): |
- (WebCore::MediaStream::streamEnded): |
- (WebCore::MediaStream::scriptExecutionContext): |
- (WebCore::MediaStream::eventTargetData): |
- (WebCore::MediaStream::ensureEventTargetData): |
- * mediastream/MediaStream.h: Renamed from Source/WebCore/dom/MediaStream.h. |
- (WebCore::MediaStream::label): |
- (WebCore::MediaStream::tracks): |
- (WebCore::MediaStream::descriptor): |
- (WebCore::MediaStream::refEventTarget): |
- (WebCore::MediaStream::derefEventTarget): |
- * mediastream/MediaStream.idl: Renamed from Source/WebCore/dom/MediaStream.idl. |
- * mediastream/MediaStreamClient.h: Renamed from Source/WebCore/page/MediaStreamClient.h. |
- (WebCore::MediaStreamClient::~MediaStreamClient): |
- * mediastream/MediaStreamController.cpp: Renamed from Source/WebCore/page/MediaStreamController.cpp. |
- (WebCore::MediaStreamController::Request::Request): |
- (WebCore::MediaStreamController::Request::localId): |
- (WebCore::MediaStreamController::Request::frameController): |
- (WebCore::MediaStreamController::MediaStreamController): |
- (WebCore::MediaStreamController::~MediaStreamController): |
- (WebCore::MediaStreamController::isClientAvailable): |
- (WebCore::MediaStreamController::unregisterFrameController): |
- (WebCore::MediaStreamController::registerRequest): |
- (WebCore::MediaStreamController::generateStream): |
- (WebCore::MediaStreamController::streamGenerated): |
- (WebCore::MediaStreamController::streamGenerationFailed): |
- * mediastream/MediaStreamController.h: Renamed from Source/WebCore/page/MediaStreamController.h. |
- * mediastream/MediaStreamEvent.cpp: Renamed from Source/WebCore/p2p/MediaStreamEvent.cpp. |
- (WebCore::MediaStreamEvent::create): |
- (WebCore::MediaStreamEvent::MediaStreamEvent): |
- (WebCore::MediaStreamEvent::~MediaStreamEvent): |
- (WebCore::MediaStreamEvent::initMediaStreamEvent): |
- (WebCore::MediaStreamEvent::stream): |
- * mediastream/MediaStreamEvent.h: Renamed from Source/WebCore/p2p/MediaStreamEvent.h. |
- (WebCore::MediaStreamEvent::isMediaStreamEvent): |
- * mediastream/MediaStreamEvent.idl: Renamed from Source/WebCore/p2p/MediaStreamEvent.idl. |
- * mediastream/MediaStreamFrameController.cpp: Renamed from Source/WebCore/page/MediaStreamFrameController.cpp. |
- (WebCore::MediaStreamFrameController::Request::~Request): |
- (WebCore::MediaStreamFrameController::Request::scriptExecutionContext): |
- (WebCore::MediaStreamFrameController::Request::isGenerateStreamRequest): |
- (WebCore::MediaStreamFrameController::Request::Request): |
- (WebCore::MediaStreamFrameController::GenerateStreamRequest::create): |
- (WebCore::MediaStreamFrameController::GenerateStreamRequest::~GenerateStreamRequest): |
- (WebCore::MediaStreamFrameController::GenerateStreamRequest::isGenerateStreamRequest): |
- (WebCore::MediaStreamFrameController::GenerateStreamRequest::abort): |
- (WebCore::MediaStreamFrameController::GenerateStreamRequest::successCallback): |
- (WebCore::MediaStreamFrameController::GenerateStreamRequest::errorCallback): |
- (WebCore::MediaStreamFrameController::GenerateStreamRequest::GenerateStreamRequest): |
- (WebCore::MediaStreamFrameController::RequestMap::abort): |
- (WebCore::MediaStreamFrameController::RequestMap::abortAll): |
- (WebCore::MediaStreamFrameController::MediaStreamFrameController): |
- (WebCore::MediaStreamFrameController::~MediaStreamFrameController): |
- (WebCore::MediaStreamFrameController::securityOrigin): |
- (WebCore::MediaStreamFrameController::scriptExecutionContext): |
- (WebCore::MediaStreamFrameController::pageController): |
- (WebCore::MediaStreamFrameController::enterDetachedState): |
- (WebCore::MediaStreamFrameController::isClientAvailable): |
- (WebCore::MediaStreamFrameController::disconnectPage): |
- (WebCore::MediaStreamFrameController::disconnectFrame): |
- (WebCore::MediaStreamFrameController::transferToNewPage): |
- (WebCore::MediaStreamFrameController::parseGenerateStreamOptions): |
- (WebCore::MediaStreamFrameController::generateStream): |
- (WebCore::MediaStreamFrameController::streamGenerated): |
- (WebCore::MediaStreamFrameController::streamGenerationFailed): |
- * mediastream/MediaStreamFrameController.h: Renamed from Source/WebCore/page/MediaStreamFrameController.h. |
- (WebCore::MediaStreamFrameController::IdGenerator::IdGenerator): |
- (WebCore::MediaStreamFrameController::IdGenerator::getNextId): |
- * mediastream/MediaStreamList.cpp: Renamed from Source/WebCore/dom/MediaStreamList.cpp. |
- (WebCore::MediaStreamList::create): |
- (WebCore::MediaStreamList::MediaStreamList): |
- (WebCore::MediaStreamList::~MediaStreamList): |
- (WebCore::MediaStreamList::length): |
- (WebCore::MediaStreamList::item): |
- (WebCore::MediaStreamList::append): |
- (WebCore::MediaStreamList::remove): |
- (WebCore::MediaStreamList::contains): |
- * mediastream/MediaStreamList.h: Renamed from Source/WebCore/dom/MediaStreamList.h. |
- * mediastream/MediaStreamList.idl: Renamed from Source/WebCore/dom/MediaStreamList.idl. |
- * mediastream/MediaStreamRegistry.cpp: Renamed from Source/WebCore/platform/MediaStreamRegistry.cpp. |
- (WebCore::MediaStreamRegistry::registry): |
- (WebCore::MediaStreamRegistry::registerMediaStreamURL): |
- (WebCore::MediaStreamRegistry::unregisterMediaStreamURL): |
- (WebCore::MediaStreamRegistry::mediaStream): |
- * mediastream/MediaStreamRegistry.h: Renamed from Source/WebCore/platform/MediaStreamRegistry.h. |
- * mediastream/MediaStreamTrack.cpp: Renamed from Source/WebCore/dom/MediaStreamTrack.cpp. |
- (WebCore::MediaStreamTrack::create): |
- (WebCore::MediaStreamTrack::MediaStreamTrack): |
- (WebCore::MediaStreamTrack::~MediaStreamTrack): |
- (WebCore::MediaStreamTrack::kind): |
- (WebCore::MediaStreamTrack::label): |
- (WebCore::MediaStreamTrack::enabled): |
- (WebCore::MediaStreamTrack::setEnabled): |
- * mediastream/MediaStreamTrack.h: Renamed from Source/WebCore/dom/MediaStreamTrack.h. |
- * mediastream/MediaStreamTrack.idl: Renamed from Source/WebCore/dom/MediaStreamTrack.idl. |
- * mediastream/MediaStreamTrackList.cpp: Renamed from Source/WebCore/dom/MediaStreamTrackList.cpp. |
- (WebCore::MediaStreamTrackList::create): |
- (WebCore::MediaStreamTrackList::MediaStreamTrackList): |
- (WebCore::MediaStreamTrackList::~MediaStreamTrackList): |
- (WebCore::MediaStreamTrackList::length): |
- (WebCore::MediaStreamTrackList::item): |
- * mediastream/MediaStreamTrackList.h: Renamed from Source/WebCore/dom/MediaStreamTrackList.h. |
- * mediastream/MediaStreamTrackList.idl: Renamed from Source/WebCore/dom/MediaStreamTrackList.idl. |
- * mediastream/PeerConnection.cpp: Renamed from Source/WebCore/p2p/PeerConnection.cpp. |
- (WebCore::PeerConnection::create): |
- (WebCore::PeerConnection::PeerConnection): |
- (WebCore::PeerConnection::~PeerConnection): |
- (WebCore::PeerConnection::processSignalingMessage): |
- (WebCore::PeerConnection::readyState): |
- (WebCore::PeerConnection::send): |
- (WebCore::PeerConnection::addStream): |
- (WebCore::PeerConnection::removeStream): |
- (WebCore::PeerConnection::localStreams): |
- (WebCore::PeerConnection::remoteStreams): |
- (WebCore::PeerConnection::close): |
- (WebCore::PeerConnection::iceProcessingCompleted): |
- (WebCore::PeerConnection::sdpGenerated): |
- (WebCore::PeerConnection::dataStreamMessageReceived): |
- (WebCore::PeerConnection::remoteStreamAdded): |
- (WebCore::PeerConnection::remoteStreamRemoved): |
- (WebCore::PeerConnection::toPeerConnection): |
- (WebCore::PeerConnection::scriptExecutionContext): |
- (WebCore::PeerConnection::stop): |
- (WebCore::PeerConnection::eventTargetData): |
- (WebCore::PeerConnection::ensureEventTargetData): |
- (WebCore::PeerConnection::scheduleInitialNegotiation): |
- (WebCore::PeerConnection::initialNegotiationTimerFired): |
- (WebCore::PeerConnection::ensureStreamChangeScheduled): |
- (WebCore::PeerConnection::streamChangeTimerFired): |
- (WebCore::PeerConnection::scheduleReadyStateChange): |
- (WebCore::PeerConnection::readyStateChangeTimerFired): |
- (WebCore::PeerConnection::changeReadyState): |
- * mediastream/PeerConnection.h: Renamed from Source/WebCore/p2p/PeerConnection.h. |
- (WebCore::PeerConnection::refEventTarget): |
- (WebCore::PeerConnection::derefEventTarget): |
- * mediastream/PeerConnection.idl: Renamed from Source/WebCore/p2p/PeerConnection.idl. |
- * mediastream/SignalingCallback.h: Renamed from Source/WebCore/p2p/SignalingCallback.h. |
- (WebCore::SignalingCallback::~SignalingCallback): |
- * mediastream/SignalingCallback.idl: Renamed from Source/WebCore/p2p/SignalingCallback.idl. |
- * page/Frame.cpp: |
- * page/Navigator.cpp: |
- * page/Page.cpp: |
- |
-2011-10-20 Pavel Feldman <pfeldman@google.com> |
- |
- Web Inspector: get rid of View::hide, View::set visible, View::attach, |
- View::set visible. |
- https://bugs.webkit.org/show_bug.cgi?id=70523 |
- |
- This change also introduces assetions that forces view attach / detach operations |
- to be performed explicitly. |
- |
- Reviewed by Yury Semikhatsky. |
- |
- * inspector/front-end/ApplicationCacheItemsView.js: |
- (WebInspector.ApplicationCacheItemsView.prototype.wasShown): |
- (WebInspector.ApplicationCacheItemsView.prototype.willHide): |
- (WebInspector.ApplicationCacheItemsView.prototype._updateCallback): |
- * inspector/front-end/AuditsPanel.js: |
- (WebInspector.AuditsPanel.prototype.wasShown): |
- * inspector/front-end/ConsolePanel.js: |
- (WebInspector.ConsolePanel.prototype.wasShown): |
- (WebInspector.ConsolePanel.prototype.willHide): |
- * inspector/front-end/ConsoleView.js: |
- (WebInspector.ConsoleView.prototype.wasShown): |
- * inspector/front-end/CookieItemsView.js: |
- (WebInspector.CookieItemsView.prototype.wasShown): |
- (WebInspector.CookieItemsView.prototype.willHide): |
- (WebInspector.CookieItemsView.prototype._updateWithCookies): |
- * inspector/front-end/DOMStorageItemsView.js: |
- (WebInspector.DOMStorageItemsView.prototype.wasShown): |
- (WebInspector.DOMStorageItemsView.prototype.willHide): |
- * inspector/front-end/DatabaseQueryView.js: |
- * inspector/front-end/DatabaseTableView.js: |
- (WebInspector.DatabaseTableView.prototype.wasShown): |
- * inspector/front-end/DetailedHeapshotView.js: |
- (WebInspector.DetailedHeapshotView.prototype.willHide): |
- (WebInspector.DetailedHeapshotView.prototype._changeView): |
- * inspector/front-end/Drawer.js: |
- (WebInspector.Drawer.prototype.hide): |
- * inspector/front-end/ElementsPanel.js: |
- (WebInspector.ElementsPanel.prototype.wasShown): |
- (WebInspector.ElementsPanel.prototype.willHide): |
- * inspector/front-end/EmptyView.js: |
- (WebInspector.EmptyView.prototype.wasShown): |
- * inspector/front-end/FontView.js: |
- (WebInspector.FontView.prototype.wasShown): |
- * inspector/front-end/IFrameView.js: |
- (WebInspector.IFrameView): |
- * inspector/front-end/ImageView.js: |
- (WebInspector.ImageView.prototype.wasShown): |
- * inspector/front-end/NetworkItemView.js: |
- (WebInspector.NetworkItemView.prototype.wasShown): |
- (WebInspector.ResourceContentView.prototype.wasShown): |
- * inspector/front-end/NetworkPanel.js: |
- (WebInspector.NetworkLogView.prototype.wasShown): |
- (WebInspector.NetworkLogView.prototype.willHide): |
- (WebInspector.NetworkPanel.prototype.wasShown): |
- * inspector/front-end/Panel.js: |
- (WebInspector.Panel): |
- (WebInspector.Panel.prototype.show): |
- (WebInspector.Panel.prototype.wasShown): |
- (WebInspector.Panel.prototype.willHide): |
- * inspector/front-end/PanelEnablerView.js: |
- * inspector/front-end/ProfileView.js: |
- (WebInspector.CPUProfileView.prototype.willHide): |
- * inspector/front-end/ProfilesPanel.js: |
- (WebInspector.ProfilesPanel.prototype.wasShown): |
- (WebInspector.ProfilesPanel.prototype._addProfileHeader): |
- (WebInspector.ProfilesPanel.prototype._updateInterface): |
- (WebInspector.ProfilesPanel.prototype._enableDetailedHeapProfiles): |
- * inspector/front-end/ResourceCookiesView.js: |
- (WebInspector.ResourceCookiesView.prototype.wasShown): |
- * inspector/front-end/ResourceHTMLView.js: |
- (WebInspector.ResourceHTMLView.prototype.wasShown): |
- (WebInspector.ResourceHTMLView.prototype.willHide): |
- * inspector/front-end/ResourceJSONView.js: |
- (WebInspector.ResourceJSONView.parseJSON.WebInspector.ResourceJSONView.prototype.wasShown): |
- * inspector/front-end/ResourceTimingView.js: |
- (WebInspector.ResourceTimingView.prototype.wasShown): |
- * inspector/front-end/ResourcesPanel.js: |
- (WebInspector.ResourcesPanel.prototype.wasShown): |
- * inspector/front-end/ScriptsPanel.js: |
- (WebInspector.ScriptsPanel.prototype.wasShown): |
- (WebInspector.ScriptsPanel.prototype._updateDebuggerButtons): |
- * inspector/front-end/SourceFrame.js: |
- (WebInspector.SourceFrame.prototype.wasShown): |
- * inspector/front-end/TimelinePanel.js: |
- (WebInspector.TimelinePanel.prototype.wasShown): |
- (WebInspector.TimelinePanel.prototype.willHide): |
- * inspector/front-end/View.js: |
- (WebInspector.View): |
- (WebInspector.View.prototype.get visible): |
- (WebInspector.View.prototype.setHideOnDetach): |
- (WebInspector.View.prototype._wasShown): |
- (WebInspector.View.prototype._willHide): |
- (WebInspector.View.prototype._onResize): |
- (WebInspector.View.prototype.wasShown): |
- (WebInspector.View.prototype.willHide): |
- (WebInspector.View.prototype.onResize): |
- (WebInspector.View.prototype.show): |
- (WebInspector.View.prototype.detach): |
- (WebInspector.View.prototype.doResize): |
- (WebInspector.View.prototype.dispatchToSelfAndChildren): |
- (WebInspector.View.prototype.dispatchToChildren): |
- (WebInspector.View.prototype._handleInsertedIntoDocument): |
- |
-2011-10-22 Kent Tamura <tkent@chromium.org> |
- |
- REGRESSION(r92480): Passing uninitialized variable in HTMLElement::calculateAndAdjustDirectionality() |
- https://bugs.webkit.org/show_bug.cgi?id=70582 |
- |
- Reviewed by Darin Adler. |
- |
- No new tests. It's very hard to make a reliable test. |
- |
- * html/HTMLElement.cpp: |
- (WebCore::HTMLElement::directionality): |
- Clear *strongDirectionalityTextNode even if hasStrongDirectionality is false. |
- |
-2011-10-21 Dan Bernstein <mitz@apple.com> |
- |
- Caret is drawn in the wrong place in multi-column blocks |
- https://bugs.webkit.org/show_bug.cgi?id=70662 |
- |
- Reviewed by Gavin Barraclough. |
- |
- * manual-tests/caret-in-columns.html: Added. |
- * rendering/RenderBlock.cpp: |
- (WebCore::RenderBlock::paintCaret): Removed the call to offsetForContents(). That function takes |
- a point in local coordinates, whereas this function was applying it to a point in painting root |
- coordinates. The desired effect was only to undo the scroll adjustment done by the caller, |
- paintObject(). |
- (WebCore::RenderBlock::paintObject): Pass the original, rather than scroll-adjusted, paint offset |
- to paintCaret(). |
- |
-2011-10-21 Nat Duca <nduca@chromium.org> |
- |
- [chromium] Make setVisibility extension- and thread-correct |
- https://bugs.webkit.org/show_bug.cgi?id=70635 |
- |
- Reviewed by Kenneth Russell. |
- |
- * platform/graphics/chromium/Extensions3DChromium.h: |
- * platform/graphics/chromium/LayerRendererChromium.cpp: |
- (WebCore::LayerRendererChromium::initialize): |
- (WebCore::LayerRendererChromium::setVisible): |
- * platform/graphics/chromium/LayerRendererChromium.h: |
- * platform/graphics/chromium/cc/CCLayerTreeHost.h: |
- (WebCore::LayerRendererCapabilities::LayerRendererCapabilities): |
- * platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp: |
- (WebCore::CCLayerTreeHostImpl::setVisible): |
- |
-2011-10-21 Beth Dakin <bdakin@apple.com> |
- |
- https://bugs.webkit.org/show_bug.cgi?id=70647 |
- Common but unreproducible crash under [ScrollbarPartAnimation setCurrentProgress:] |
- -and corresponding- |
- <rdar://problem/9542018> |
- |
- Reviewed by Sam Weinig. |
- |
- This patch implements two speculative fixes for this crash. |
- |
- First, block exceptions around all of the code responsible for calling |
- stopAnimation. If that code throws any exceptions, we want to make sure the other |
- animations are still stopped. |
- * platform/mac/ScrollAnimatorMac.mm: |
- (-[WebScrollbarPartAnimation scrollAnimatorDestroyed]): |
- (-[WebScrollbarPainterDelegate scrollAnimatorDestroyed]): |
- (WebCore::ScrollAnimatorMac::~ScrollAnimatorMac): |
- |
- Only send AppKit these notifications for active pages. I originally made these |
- assertions, and I found that they were hit a surprising number of times. If we |
- only send notifications for active pages, then we should greatly reduce and |
- possibly eliminate our chances of hitting this crash. |
- (WebCore::ScrollAnimatorMac::notifyPositionChanged): |
- (WebCore::ScrollAnimatorMac::contentAreaWillPaint): |
- (WebCore::ScrollAnimatorMac::mouseEnteredContentArea): |
- (WebCore::ScrollAnimatorMac::mouseExitedContentArea): |
- (WebCore::ScrollAnimatorMac::mouseMovedInContentArea): |
- (WebCore::ScrollAnimatorMac::willStartLiveResize): |
- (WebCore::ScrollAnimatorMac::contentsResized): |
- (WebCore::ScrollAnimatorMac::willEndLiveResize): |
- (WebCore::ScrollAnimatorMac::contentAreaDidShow): |
- (WebCore::ScrollAnimatorMac::contentAreaDidHide): |
- (WebCore::ScrollAnimatorMac::didBeginScrollGesture): |
- (WebCore::ScrollAnimatorMac::didEndScrollGesture): |
- |
-2011-10-21 Mark Hahnenberg <mhahnenberg@apple.com> |
- |
- Add put to the MethodTable |
- https://bugs.webkit.org/show_bug.cgi?id=70439 |
- |
- Reviewed by Oliver Hunt. |
- |
- No new tests. |
- |
- * WebCore.exp.in: |
- * bindings/js/JSDOMWindowCustom.cpp: Added static put since it was overlooked in |
- previous patches. |
- (WebCore::JSDOMWindow::putVirtual): |
- (WebCore::JSDOMWindow::put): |
- * bindings/js/JSDOMWindowShell.cpp: Ditto. |
- (WebCore::JSDOMWindowShell::putVirtual): |
- (WebCore::JSDOMWindowShell::put): |
- * bindings/js/JSDOMWindowShell.h: |
- |
-2011-10-21 Dmitry Lomov <dslomov@google.com> |
- |
- https://bugs.webkit.org/show_bug.cgi?id=70580 |
- [Chromium] Implement MessagePort transfer in chromium port of webkit. |
- |
- Reviewed by David Levin. |
- |
- * bindings/v8/SerializedScriptValue.cpp: |
- (WebCore::V8ObjectMap::Writer::writeTransferredMessagePort): |
- (WebCore::V8ObjectMap::Serializer::Serializer): |
- (WebCore::V8ObjectMap::Serializer::doSerialize): |
- (WebCore::V8ObjectMap::Reader::read): |
- (WebCore::V8ObjectMap::Deserializer::Deserializer): |
- (WebCore::V8ObjectMap::Deserializer::tryGetTransferredMessagePort): |
- (WebCore::SerializedScriptValue::SerializedScriptValue): |
- (WebCore::SerializedScriptValue::deserialize): |
- |
-2011-10-21 Ojan Vafai <ojan@chromium.org> |
- |
- r97693 caused 42 failures on the chromium mac bots |
- https://bugs.webkit.org/show_bug.cgi?id=70300 |
- |
- Can't test this until https://bugs.webkit.org/show_bug.cgi?id=70660 |
- is fixed. The fix for that bug will test this codepath as well though. |
- |
- * rendering/RenderBlock.cpp: |
- (WebCore::RenderBlock::computeBlockPreferredLogicalWidths): |
- |
-2011-10-21 Vineet Chaudhary <vineet.chaudhary@motorola.com> |
- |
- Setting form.enctype reflected attribute behaves strangely. |
- https://bugs.webkit.org/show_bug.cgi?id=44879 |
- |
- Reviewed by Darin Adler. |
- |
- The spec says form.enctype should be application/x-www-form-urlencoded, |
- So as "text", "plain", "multipart" or "form-data" these are not a valid value so must be ignored. |
- |
- * loader/FormSubmission.cpp: |
- (WebCore::FormSubmission::Attributes::parseEncodingType): |
- |
-2011-10-21 Matthew Delaney <mdelaney@apple.com> |
- |
- Ensure periodic flushing of canvas drawing context |
- https://bugs.webkit.org/show_bug.cgi?id=70646 |
- |
- Reviewed by Simon Fraser. |
- |
- No new tests. No current way to track tests that cause hangs or |
- non-deterministic drops in performance. |
- |
- * platform/graphics/cg/ImageBufferDataCG.h: Adds a timestamp of last tracked flush. |
- * platform/graphics/cg/ImageBufferCG.cpp: Ensures periodic flushes on the drawing context. |
- (WebCore::ImageBuffer::ImageBuffer): |
- (WebCore::ImageBuffer::context): Flushes context if we're beyond flush interval. |
- (WebCore::ImageBuffer::copyNativeImage): Updates last flush timestamp. |
- (WebCore::ImageBuffer::getUnmultipliedImageData): Updates last flush timestamp. |
- (WebCore::ImageBuffer::getPremultipliedImageData): Updates last flush timestamp. |
- (WebCore::ImageBuffer::putUnmultipliedImageData): Updates last flush timestamp. |
- (WebCore::ImageBuffer::putPremultipliedImageData): Updates last flush timestamp. |
- |
-2011-10-21 Adam Barth <abarth@webkit.org> |
- |
- Introduce Event::hasInterface to make uses of interfaceName more readable |
- https://bugs.webkit.org/show_bug.cgi?id=70652 |
- |
- Reviewed by Eric Seidel. |
- |
- Eric felt that this approach would be more readable because it requires |
- less syntax at each callsite. |
- |
- * bindings/js/JSErrorHandler.cpp: |
- (WebCore::JSErrorHandler::handleEvent): |
- * bindings/objc/DOMEvents.mm: |
- (kitClass): |
- * bindings/v8/V8WindowErrorHandler.cpp: |
- (WebCore::V8WindowErrorHandler::callListenerFunction): |
- * bindings/v8/V8WorkerContextErrorHandler.cpp: |
- (WebCore::V8WorkerContextErrorHandler::callListenerFunction): |
- * dom/Event.cpp: |
- (WebCore::Event::hasInterface): |
- * dom/Event.h: |
- * dom/EventDispatcher.cpp: |
- (WebCore::EventDispatcher::determineDispatchBehavior): |
- * dom/Node.cpp: |
- (WebCore::Node::defaultEventHandler): |
- * html/HTMLInputElement.cpp: |
- (WebCore::HTMLInputElement::defaultEventHandler): |
- * html/HTMLTextAreaElement.cpp: |
- (WebCore::HTMLTextAreaElement::defaultEventHandler): |
- * html/TextFieldInputType.cpp: |
- (WebCore::TextFieldInputType::forwardEvent): |
- (WebCore::TextFieldInputType::shouldSubmitImplicitly): |
- * inspector/InspectorDOMStorageResource.cpp: |
- (WebCore::InspectorDOMStorageResource::handleEvent): |
- |
-2011-10-21 Adam Klein <adamk@chromium.org> |
- |
- [MutationObservers] Implement basic subtree observation |
- https://bugs.webkit.org/show_bug.cgi?id=70436 |
- |
- Reviewed by Ryosuke Niwa. |
- |
- Note that this patch only implements "basic" subtree semantics, |
- not the fully robust semantics described in |
- http://lists.w3.org/Archives/Public/public-webapps/2011JulSep/1622.html. |
- Most importantly, this change does not handle the case where mutations |
- occur in a temporarily detached subtree. |
- |
- The plan is to implement those semantics in a followup to avoid |
- blocking other parts of the MutationObserver spec that rely on |
- the existence of subtree observation but not its specific |
- implementation. |
- |
- Test: fast/mutation/observe-subtree.html |
- |
- * dom/Node.cpp: |
- (WebCore::addMatchingObservers): Static helper method for registeredMutationObserversOfType(). |
- (WebCore::Node::registeredMutationObserversOfType): Walk up the tree looking for observers. |
- * dom/NodeRareData.h: |
- (WebCore::MutationObserverEntry::hasAllOptions): A stricter, renamed from matches(). |
- |
-2011-10-21 Joshua Bell <jsbell@chromium.org> |
- |
- IndexedDB: objectStore.transaction property should be readonly |
- https://bugs.webkit.org/show_bug.cgi?id=62395 |
- |
- Reviewed by Tony Chang. |
- |
- Implement the IDBObjectStore.transaction property defined by the spec. |
- |
- Test: storage/indexeddb/readonly-properties.html |
- |
- * storage/IDBObjectStore.cpp: |
- (WebCore::IDBObjectStore::transaction): |
- * storage/IDBObjectStore.h: |
- * storage/IDBObjectStore.idl: |
- |
-2011-10-21 Adam Barth <abarth@webkit.org> |
- |
- Event.h has too many virtual isMumbleEvent() functions |
- https://bugs.webkit.org/show_bug.cgi?id=70636 |
- |
- Reviewed by Dimitri Glazkov. |
- |
- We should use the new interfaceName() way of doing run-time type inference. |
- |
- * bindings/js/JSErrorHandler.cpp: |
- (WebCore::JSErrorHandler::handleEvent): |
- * bindings/js/JSEventCustom.cpp: |
- (WebCore::JSEvent::clipboardData): |
- * bindings/objc/DOMEvents.mm: |
- (kitClass): |
- * bindings/v8/V8WindowErrorHandler.cpp: |
- (WebCore::V8WindowErrorHandler::callListenerFunction): |
- * bindings/v8/V8WorkerContextErrorHandler.cpp: |
- (WebCore::V8WorkerContextErrorHandler::callListenerFunction): |
- * bindings/v8/custom/V8EventCustom.cpp: |
- (WebCore::V8Event::clipboardDataAccessorGetter): |
- * dom/BeforeLoadEvent.h: |
- (WebCore::BeforeLoadEvent::interfaceName): |
- * dom/BeforeTextInsertedEvent.h: |
- * dom/ClipboardEvent.cpp: |
- * dom/ClipboardEvent.h: |
- * dom/CompositionEvent.cpp: |
- * dom/CompositionEvent.h: |
- * dom/ErrorEvent.cpp: |
- * dom/ErrorEvent.h: |
- * dom/Event.cpp: |
- (WebCore::Event::storesResultAsString): |
- (WebCore::Event::storeResult): |
- (WebCore::Event::clipboardData): |
- * dom/Event.h: |
- * dom/Event.idl: |
- - Remove this interface from the CPP bindings because this function |
- no longer exists. It's unclear to me how strong our API |
- commitments are in the CPP bindings. |
- * dom/EventDispatcher.cpp: |
- (WebCore::EventDispatcher::determineDispatchBehavior): |
- * dom/MessageEvent.cpp: |
- * dom/MessageEvent.h: |
- * dom/MutationEvent.cpp: |
- * dom/MutationEvent.h: |
- * dom/Node.cpp: |
- (WebCore::Node::defaultEventHandler): |
- * dom/OverflowEvent.cpp: |
- * dom/OverflowEvent.h: |
- * dom/PageTransitionEvent.h: |
- * dom/ProgressEvent.h: |
- * dom/TextEvent.cpp: |
- * dom/TextEvent.h: |
- * dom/WheelEvent.cpp: |
- * dom/WheelEvent.h: |
- * html/HTMLInputElement.cpp: |
- (WebCore::HTMLInputElement::defaultEventHandler): |
- * html/HTMLTextAreaElement.cpp: |
- (WebCore::HTMLTextAreaElement::defaultEventHandler): |
- * html/TextFieldInputType.cpp: |
- (WebCore::TextFieldInputType::forwardEvent): |
- (WebCore::TextFieldInputType::shouldSubmitImplicitly): |
- * html/shadow/TextControlInnerElements.cpp: |
- (WebCore::TextControlInnerTextElement::defaultEventHandler): |
- * svg/SVGZoomEvent.cpp: |
- * svg/SVGZoomEvent.h: |
- |
-2011-10-21 Raymond Toy <rtoy@google.com> |
- |
- Flush denormals in Biquad, ZeroPole, and DynamicsCompressor. |
- |
- We only flush when saving the state variables instead of in the |
- loops so that we don't impact performance too much when there are |
- no denormals. This will at least not propagate the denormals any |
- further within the class. |
- |
- https://bugs.webkit.org/show_bug.cgi?id=70626 |
- |
- Reviewed by Kenneth Russell. |
- |
- * platform/audio/Biquad.cpp: |
- (WebCore::Biquad::process): |
- Flush denormals when storing the filter state back in to class |
- filter state. |
- * platform/audio/DynamicsCompressorKernel.cpp: |
- (WebCore::DynamicsCompressorKernel::process): |
- Ditto. |
- * platform/audio/ZeroPole.cpp: |
- (WebCore::ZeroPole::process): |
- Ditto. |
- |
-2011-10-21 Vsevolod Vlasov <vsevik@chromium.org> |
- |
- Web Inspector: Advanced search results should be cleared on navigation. |
- https://bugs.webkit.org/show_bug.cgi?id=70627 |
- |
- Reviewed by Pavel Feldman. |
- |
- * inspector/front-end/AdvancedSearchController.js: |
- (WebInspector.AdvancedSearchController): |
- (WebInspector.AdvancedSearchController.prototype._frameNavigated): |
- |
-2011-10-21 Rafael Weinstein <rafaelw@chromium.org> |
- |
- [MutationObservers] Implement WebKitMutationObserver.observe for childList changes |
- https://bugs.webkit.org/show_bug.cgi?id=68955 |
- |
- Reviewed by Ryosuke Niwa. |
- |
- This patch adds a ChildListMutationScope object which manages the coalescing of |
- multiple child removals and additions within DOM operations into single childList |
- mutation records. |
- |
- Note that this patch doesn't cover all the cases which can be coalesced (it only |
- covers ContainerNode.*, and Node.innerHTML/innerText). A separate bug, |
- https://bugs.webkit.org/show_bug.cgi?id=70385, has been opened to track the |
- remaining cases. |
- |
- Test: fast/mutation/observe-childList.html |
- |
- * CMakeLists.txt: |
- * GNUmakefile.list.am: |
- * WebCore.gypi: |
- * WebCore.pro: |
- * WebCore.vcproj/WebCore.vcproj: |
- * WebCore.xcodeproj/project.pbxproj: |
- * dom/ChildListMutationScope.cpp: Added. |
- (WebCore::ChildListAccumulation::ChildListAccumulation::ChildListAccumulation): |
- (WebCore::ChildListAccumulation::ChildListAccumulation::~ChildListAccumulation): |
- (WebCore::ChildListAccumulation::ChildListAccumulation::childAdded): |
- (WebCore::ChildListAccumulation::ChildListAccumulation::willRemoveChild): |
- (WebCore::ChildListAccumulation::ChildListAccumulation::dispatch): |
- (WebCore::ChildListAccumulation::ChildListAccumulation::clear): |
- (WebCore::ChildListAccumulation::ChildListAccumulation::isEmpty): |
- (WebCore::ChildListAccumulation::ChildListAccumulator::ChildListAccumulator): |
- (WebCore::ChildListAccumulation::ChildListAccumulator::~ChildListAccumulator): |
- (WebCore::ChildListAccumulation::ChildListAccumulator::initialize): |
- (WebCore::ChildListAccumulation::ChildListAccumulator::instance): |
- (WebCore::ChildListAccumulation::ChildListAccumulator::childAdded): |
- (WebCore::ChildListAccumulation::ChildListAccumulator::willRemoveChild): |
- (WebCore::ChildListAccumulation::ChildListAccumulator::incrementScopingLevel): |
- (WebCore::ChildListAccumulation::ChildListAccumulator::decrementScopingLevel): |
- (WebCore::ChildListMutationScope::ChildListMutationScope): |
- (WebCore::ChildListMutationScope::~ChildListMutationScope): |
- (WebCore::ChildListMutationScope::childAdded): |
- (WebCore::ChildListMutationScope::willRemoveChild): |
- * dom/ChildListMutationScope.h: Added. |
- * dom/ContainerNode.cpp: |
- (WebCore::ContainerNode::insertBefore): |
- (WebCore::ContainerNode::replaceChild): |
- (WebCore::willRemoveChildren): |
- (WebCore::dispatchChildInsertionEvents): |
- (WebCore::dispatchChildRemovalEvents): |
- * dom/Element.cpp: |
- (WebCore::enqueueAttributesMutationRecord): |
- * dom/Node.cpp: |
- (WebCore::Node::setTextContent): |
- (WebCore::Node::registeredMutationObserversOfType): |
- * dom/WebKitMutationObserver.cpp: |
- (WebCore::WebKitMutationObserver::disconnect): |
- * html/HTMLElement.cpp: |
- (WebCore::replaceChildrenWithFragment): |
- (WebCore::replaceChildrenWithText): |
- |
-2011-10-21 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r98079. |
- http://trac.webkit.org/changeset/98079 |
- https://bugs.webkit.org/show_bug.cgi?id=70624 |
- |
- The extensions tests enabled are failing on the Chromium Win |
- Dbg bot (Requested by jchaffraix on #webkit). |
- |
- * inspector/front-end/ExtensionAPI.js: |
- (buildExtensionAPIInjectedScript): |
- |
-2011-10-21 Dan Bernstein <mitz@apple.com> |
- |
- Column rules positioned incorrectly in vertical-rl block with horizontal border or padding |
- https://bugs.webkit.org/show_bug.cgi?id=70622 |
- |
- Reviewed by Beth Dakin. |
- |
- Tests: fast/multicol/vertical-lr/rules-with-border-before.html |
- fast/multicol/vertical-rl/rules-with-border-before.html |
- |
- * rendering/RenderBlock.cpp: |
- (WebCore::RenderBlock::paintColumnRules): Changed to use the physical left border and padding for the |
- left edge of the horizontal column rule. |
- |
-2011-10-21 Simon Fraser <simon.fraser@apple.com> |
- |
- Fix Windows build. |
- |
- * rendering/RenderLayerCompositor.cpp: |
- (WebCore::RenderLayerCompositor::hasAnyAdditionalCompositedLayers): |
- |
-2011-10-21 Devdatta Deshpande <pwjd73@motorola.com> |
- |
- [Gtk] mousemove event always has metaKey == true |
- https://bugs.webkit.org/show_bug.cgi?id=35299 |
- |
- Reviewed by Martin Robinson. |
- |
- GDK_MOD2_MASK doesn't always mean meta so we can't use it to identify |
- the meta key state. Use GDK_META_MASK instead. |
- |
- Test: platform/gtk/fast/events/event-sender-metakey.html |
- |
- * platform/gtk/PlatformMouseEventGtk.cpp: |
- (WebCore::PlatformMouseEvent::PlatformMouseEvent): |
- |
-2011-10-21 Andreas Kling <kling@webkit.org> |
- |
- Style and Link elements' sheet() should return CSSStyleSheet. |
- https://bugs.webkit.org/show_bug.cgi?id=70608 |
- |
- Reviewed by Antti Koivisto. |
- |
- Return a CSSStyleSheet* instead of a StyleSheet* in these functions |
- since we know they are always CSSStyleSheets. |
- Also remove some now-unnecessary isCSSStyleSheet() checks. |
- |
- * dom/StyleElement.h: |
- (WebCore::StyleElement::sheet): |
- * html/HTMLLinkElement.cpp: |
- * html/HTMLLinkElement.h: |
- (WebCore::HTMLLinkElement::sheet): |
- * page/PageSerializer.cpp: |
- (WebCore::PageSerializer::serializeFrame): |
- |
-2011-10-21 Vsevolod Vlasov <vsevik@chromium.org> |
- |
- Web Inspector: Advanced search is working very slowly and does not show searching progress. |
- https://bugs.webkit.org/show_bug.cgi?id=70611 |
- |
- Search implementation changed so that we do not start searching in the next file unless |
- the previous one was already searched. This allows to interrupt search. |
- Not all search matches are added on UI by default now, only first 20 for each file. |
- Search progress information and search stop button were added to drawer status bar. |
- |
- Reviewed by Pavel Feldman. |
- |
- * English.lproj/localizedStrings.js: |
- * inspector/ContentSearchUtils.cpp: |
- (WebCore::ContentSearchUtils::getRegularExpressionMatchesByLines): |
- (WebCore::ContentSearchUtils::countRegularExpressionMatches): |
- * inspector/front-end/AdvancedSearchController.js: |
- (WebInspector.AdvancedSearchController.prototype._onSearchResult): |
- (WebInspector.AdvancedSearchController.prototype._onSearchFinished): |
- (WebInspector.AdvancedSearchController.prototype.startSearch): |
- (WebInspector.AdvancedSearchController.prototype.resetSearch): |
- (WebInspector.AdvancedSearchController.prototype.stopSearch): |
- (WebInspector.SearchView): |
- (WebInspector.SearchView.prototype.get statusBarItems): |
- (WebInspector.SearchView.prototype.get counterElement): |
- (WebInspector.SearchView.prototype.set resultsPane): |
- (WebInspector.SearchView.prototype.searchStarted): |
- (WebInspector.SearchView.prototype._updateSearchResultsMessage): |
- (WebInspector.SearchView.prototype._updateSearchProgress): |
- (WebInspector.SearchView.prototype.resetResults): |
- (WebInspector.SearchView.prototype._resetCounters): |
- (WebInspector.SearchView.prototype.nothingFound): |
- (WebInspector.SearchView.prototype.addSearchResult): |
- (WebInspector.SearchView.prototype.searchFinished): |
- (WebInspector.SearchView.prototype._searchStopButtonPressed): |
- (WebInspector.SearchResultsPane): |
- (WebInspector.FileBasedSearchResultsPane): |
- (WebInspector.FileBasedSearchResultsPane.prototype.addSearchResult): |
- (WebInspector.FileBasedSearchResultsPane.prototype._fileTreeElementExpanded): |
- (WebInspector.FileBasedSearchResultsPane.prototype._appendSearchMatches): |
- (WebInspector.FileBasedSearchResultsPane.prototype._appendShowMoreMatchesElement): |
- (WebInspector.FileBasedSearchResultsPane.prototype._showMoreMatchesElementSelected): |
- (WebInspector.FileBasedSearchResultsPane.prototype._addFileTreeElement): |
- (WebInspector.FileBasedSearchResultsPane.prototype._regexMatchRanges): |
- * inspector/front-end/Drawer.js: |
- (WebInspector.Drawer.prototype.hide): |
- * inspector/front-end/Images/statusbarButtonGlyphs.png: |
- * inspector/front-end/ScriptsSearchScope.js: |
- (WebInspector.ScriptsSearchScope): |
- (WebInspector.ScriptsSearchScope.prototype.performSearch.filterOutContentScripts): |
- (WebInspector.ScriptsSearchScope.prototype.performSearch.continueSearch): |
- (WebInspector.ScriptsSearchScope.prototype.performSearch.searchCallbackWrapper): |
- (WebInspector.ScriptsSearchScope.prototype.performSearch): |
- (WebInspector.ScriptsSearchScope.prototype.stopSearch): |
- * inspector/front-end/TimelinePanel.js: |
- (WebInspector.TimelinePanel.prototype._registerShortcuts): |
- * inspector/front-end/inspector.css: |
- (#drawer-status-bar .search-status-bar-item): |
- (#drawer-status-bar .search-status-bar-message): |
- (#drawer-status-bar .search-status-bar-progress): |
- (#drawer-status-bar .search-status-bar-stop-button-item): |
- (#drawer-status-bar .search-status-bar-stop-button .glyph): |
- (#drawer-status-bar .search-results-status-bar-message): |
- (.search-view .search-results): |
- (#search-results-pane-file-based .search-results-outline-disclosure): |
- (#search-results-pane-file-based .search-result): |
- (#search-results-pane-file-based .search-result:hover): |
- (#search-results-pane-file-based .show-more-matches): |
- (#search-results-pane-file-based .show-more-matches:hover): |
- (#search-results-pane-file-based .search-match:hover): |
- * inspector/front-end/utilities.js: |
- (): |
- |
-2011-10-20 Zhenyao Mo <zmo@google.com> |
- |
- Implement mechanism to enable privileged webgl extensions |
- https://bugs.webkit.org/show_bug.cgi?id=70538 |
- |
- Reviewed by Darin Fisher. |
- |
- * html/canvas/WebGLRenderingContext.cpp: |
- (WebCore::WebGLRenderingContext::allowPrivilegedExtensions): check page/Settings flag. |
- * page/Settings.cpp: Add the flag. |
- (WebCore::Settings::Settings): |
- (WebCore::Settings::setPrivilegedWebGLExtensionsEnabled): |
- * page/Settings.h: Ditto. |
- (WebCore::Settings::privilegedWebGLExtensionsEnabled): |
- |
-2011-10-20 Pavel Podivilov <podivilov@chromium.org> |
- |
- Web Inspector: first line in file is not highlighted. |
- https://bugs.webkit.org/show_bug.cgi?id=70504 |
- |
- Reviewed by Pavel Feldman. |
- |
- * inspector/front-end/SourceFrame.js: |
- (WebInspector.SourceFrame.prototype._initializeTextViewer): |
- |
-2011-10-21 Andreas Kling <kling@webkit.org> |
- |
- Simplify CSSStyleRule::setSelectorText(). |
- https://bugs.webkit.org/show_bug.cgi?id=70607 |
- |
- Reviewed by Antti Koivisto. |
- |
- Remove unnecessary isCSSStyleSheet() check and redundant Document finding |
- logic (CSSStyleSheet::document() will return the owner node's Document just |
- like this code would.) |
- |
- * css/CSSStyleRule.cpp: |
- (WebCore::CSSStyleRule::setSelectorText): |
- |
-2011-10-21 Alejandro G. Castro <alex@igalia.com> |
- |
- [cairo] Forward declaration of GraphicsContextState uses class instead of struct |
- https://bugs.webkit.org/show_bug.cgi?id=70522 |
- |
- The type is a struct not a class, replace the definition in the |
- forward declaration. |
- |
- Reviewed by Martin Robinson. |
- |
- * platform/graphics/cairo/PlatformContextCairo.h: |
- |
-2011-10-20 Andrey Kosyakov <caseq@chromium.org> |
- |
- Web Inspector: make extension tests pass on chromium |
- https://bugs.webkit.org/show_bug.cgi?id=70334 |
- |
- Reviewed by Pavel Feldman. |
- |
- * inspector/front-end/ExtensionAPI.js: |
- (buildExtensionAPIInjectedScript): |
- |
-2011-10-20 Peter Rybin <peter.rybin@gmail.com> |
- |
- Web Inspector: reimplement protocol backend/frontend source generator |
- https://bugs.webkit.org/show_bug.cgi?id=69295 |
- |
- Reviewed by Pavel Feldman. |
- |
- Old 2-stage python+perf generator that uses intermediate IDL output is |
- being replaced with a solid python script. This is for simplicity and |
- as a base for the future JSON validator. |
- |
- * CMakeLists.txt: |
- * CodeGenerators.pri: |
- * DerivedSources.make: |
- * GNUmakefile.am: |
- * WebCore.gyp/WebCore.gyp: |
- * inspector/CodeGeneratorInspector.pm: Removed. |
- * inspector/CodeGeneratorInspector.py: Added. |
- * inspector/Inspector.json: |
- * inspector/generate-inspector-idl: Removed. |
- |
-2011-10-20 Adam Barth <abarth@webkit.org> |
- |
- Attemp to fix a bunch of tests PLATFORM(MAC). We can't use a static |
- map because that's shared between threads (and events exist in worker |
- threads). It migh be better to add a thread-specific map, but we can |
- do that in another patch. |
- |
- * bindings/js/JSEventCustom.cpp: |
- (WebCore::toJS): |
- * bindings/v8/custom/V8EventCustom.cpp: |
- (WebCore::toV8): |
- |
2011-10-20 Simon Fraser <simon.fraser@apple.com> |
Hidden composited iframes cause infinite loop |