Index: Source/WebCore/ChangeLog |
=================================================================== |
--- Source/WebCore/ChangeLog (revision 92875) |
+++ Source/WebCore/ChangeLog (working copy) |
@@ -1,3 +1,1397 @@ |
+2011-08-11 David Reveman <reveman@chromium.org> |
+ |
+ [Chromium] Temporarily disable layer anti-aliasing on ChromeOS. |
+ https://bugs.webkit.org/show_bug.cgi?id=65922 |
+ |
+ Reviewed by James Robinson. |
+ |
+ No new tests. |
+ |
+ * platform/graphics/chromium/LayerTilerChromium.cpp: |
+ (WebCore::LayerTilerChromium::draw): |
+ |
+2011-08-11 Benjamin Poulain <benjamin@webkit.org> |
+ |
+ [Qt] PageSerializer is specific to MHTML, we should not compile it |
+ https://bugs.webkit.org/show_bug.cgi?id=66054 |
+ |
+ Reviewed by Noam Rosenthal. |
+ |
+ PageSerializer is specific to MHTML. Do not build PageSerializer unless MHTML is enabled. |
+ |
+ * WebCore.pro: |
+ |
+2011-08-11 Andrey Kosyakov <caseq@chromium.org> |
+ |
+ Web Inspector: [refactoring] make tabbed pane a view |
+ https://bugs.webkit.org/show_bug.cgi?id=66060 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * inspector/front-end/NetworkItemView.js: |
+ (WebInspector.NetworkItemView): |
+ (WebInspector.NetworkItemView.prototype.show): |
+ (WebInspector.NetworkItemView.prototype._selectTab.get if): |
+ (WebInspector.NetworkItemView.prototype._selectTab): |
+ * inspector/front-end/TabbedPane.js: |
+ (WebInspector.TabbedPane): |
+ * inspector/front-end/inspector.html: |
+ |
+2011-08-11 Andrey Kosyakov <caseq@chromium.org> |
+ |
+ Web Inspector: source frame popover simetimes pops up after user switches from Scripts panel |
+ https://bugs.webkit.org/show_bug.cgi?id=66057 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * inspector/front-end/SourceFrame.js: remove popover timer when removing popover. |
+ (WebInspector.SourceFrame.prototype._mouseDown): |
+ (WebInspector.SourceFrame.prototype._hidePopup): |
+ |
+2011-08-11 Vsevolod Vlasov <vsevik@chromium.org> |
+ |
+ Web Inspector: Scripts panel: display the current search match index in the toolbar. |
+ https://bugs.webkit.org/show_bug.cgi?id=66048 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * English.lproj/localizedStrings.js: |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype.performSearch.finishedCallback): |
+ (WebInspector.ScriptsPanel.prototype.performSearch): |
+ (WebInspector.ScriptsPanel.prototype.jumpToNextSearchResult): |
+ (WebInspector.ScriptsPanel.prototype.jumpToPreviousSearchResult): |
+ * inspector/front-end/SearchController.js: |
+ (WebInspector.SearchController.prototype.updateSearchMatchesCount): |
+ (WebInspector.SearchController.prototype.updateCurrentMatchIndex): |
+ (WebInspector.SearchController.prototype.activePanelChanged.performPanelSearch): |
+ (WebInspector.SearchController.prototype.activePanelChanged): |
+ (WebInspector.SearchController.prototype._updateSearchMatchesCountAndCurrentMatchIndex): |
+ (WebInspector.SearchController.prototype._performSearch): |
+ * inspector/front-end/SourceFrame.js: |
+ (WebInspector.SourceFrame.prototype.get currentSearchResultIndex): |
+ |
+2011-08-11 Xan Lopez <xlopez@igalia.com> |
+ |
+ [GTK] Add another missing file to GNUmakefile |
+ |
+ Reviewed by Gustavo Noronha. |
+ |
+ * GNUmakefile.list.am: |
+ |
+2011-08-11 Xan Lopez <xlopez@igalia.com> |
+ |
+ [GTK] Add missing files to GNUMakefile |
+ |
+ Reviewed by Gustavo Noronha. |
+ |
+ * GNUmakefile.am: |
+ * GNUmakefile.list.am: |
+ |
+2011-08-11 Yuta Kitamura <yutak@chromium.org> |
+ |
+ WebSocket: Use PassRefPtr<> in function arguments in WorkerThreadableWebSocketChannel.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=66047 |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ It's safe to use PassRefPtr<> in these functions because: |
+ - CrossThreadTask knows how to pass a RefPtr<> across threads, and |
+ - Call sites do not pass the pointers to anywhere else. |
+ |
+ No change in functionality, thus no new tests. |
+ |
+ * websockets/WorkerThreadableWebSocketChannel.cpp: |
+ (WebCore::workerContextDidSend): |
+ (WebCore::workerContextDidGetBufferedAmount): |
+ (WebCore::workerContextDidConnect): |
+ (WebCore::workerContextDidReceiveMessage): |
+ (WebCore::workerContextDidStartClosingHandshake): |
+ (WebCore::workerContextDidClose): |
+ |
+2011-08-11 Pavel Feldman <pfeldman@google.com> |
+ |
+ Web Inspector: [Mac] search as you type does not work with the new indent in the script selector. |
+ https://bugs.webkit.org/show_bug.cgi?id=66039 |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect): |
+ * inspector/front-end/inspector.css: |
+ (select.status-bar-item): |
+ |
+2011-08-11 Alexis Menard <alexis.menard@openbossa.org> |
+ |
+ Unreviewed build fix for Qt. |
+ |
+ Make sure we build when Qt is namespaced. |
+ |
+ * platform/graphics/gstreamer/PlatformVideoWindowPrivate.h: |
+ |
+2011-08-11 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Web Inspector: use source-file-replaced instead of source-file-changed event. |
+ https://bugs.webkit.org/show_bug.cgi?id=65551 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Test: inspector/debugger/source-frame-count.html |
+ |
+ * inspector/front-end/DebuggerPresentationModel.js: |
+ (WebInspector.DebuggerPresentationModel.prototype._addScript): |
+ (WebInspector.DebuggerPresentationModel.prototype._uiSourceCodeReplaced): |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._removeSourceFrame): |
+ (WebInspector.ScriptsPanel.prototype._sourceFileReplaced): |
+ * inspector/front-end/SourceFile.js: |
+ (WebInspector.RawSourceCode): |
+ (WebInspector.RawSourceCode.prototype.get uiSourceCode): |
+ (WebInspector.RawSourceCode.prototype.reload): |
+ * inspector/front-end/inspector.js: |
+ (WebInspector._createPanels): |
+ |
+2011-08-11 Vsevolod Vlasov <vsevik@chromium.org> |
+ |
+ Web Inspector: [REGRESSION] Resources panel search fails when search result is found in a resource used in several frames. |
+ https://bugs.webkit.org/show_bug.cgi?id=66007 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype.performSearch.callback): |
+ (WebInspector.ResourcesPanel.prototype.performSearch): |
+ (WebInspector.FrameTreeElement.prototype.resourceByURL): |
+ |
+2011-08-11 Andras Becsi <abecsi@webkit.org> |
+ |
+ [GTK] Fix the build after r92792. |
+ |
+ Rubber-stamped by Csaba Osztrogonác. |
+ |
+ No new tests needed. |
+ |
+ * GNUmakefile.list.am: add missing |
+ RenderFlowThread.{cpp|h} |
+ RenderRegion.{cpp|h} |
+ |
+2011-08-11 Renata Hodovan <reni@webkit.org> |
+ |
+ [Qt] Fix WebGL on Windows after r92805 |
+ https://bugs.webkit.org/show_bug.cgi?id=66042 |
+ |
+ WEBGL is temporarly disabled, because it broked the bots. |
+ |
+ * features.pri: |
+ |
+2011-08-11 Pavel Feldman <pfeldman@google.com> |
+ |
+ Web Inspector: do not evaluate watch expressions on load. |
+ https://bugs.webkit.org/show_bug.cgi?id=66002 |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype.show): |
+ (WebInspector.ScriptsPanel.prototype.hide): |
+ (WebInspector.ScriptsPanel.prototype.reset): |
+ * inspector/front-end/WatchExpressionsSidebarPane.js: |
+ (WebInspector.WatchExpressionsSidebarPane): |
+ (WebInspector.WatchExpressionsSidebarPane.prototype.hide): |
+ (WebInspector.WatchExpressionsSidebarPane.prototype.reset): |
+ (WebInspector.WatchExpressionsSidebarPane.prototype.refreshExpressions): |
+ (WebInspector.WatchExpressionsSidebarPane.prototype._refreshExpressionsIfNeeded): |
+ (WebInspector.WatchExpressionsSidebarPane.prototype._refreshButtonClicked): |
+ |
+2011-08-10 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ Copying can result in span around block elements on the clipboard |
+ https://bugs.webkit.org/show_bug.cgi?id=34564 |
+ |
+ Reviewed by Tony Chang. |
+ |
+ Completely overhauled the way WebKit preserves style in copy and paste. Instead of wrapping the entire |
+ serialized contents by a Apple style span, WebKit now adds inline style to the top level elements, |
+ wrap top level text nodes by a style span. |
+ |
+ * editing/EditingStyle.cpp: |
+ (WebCore::EditingStyle::collapseTextDecorationProperties): Remove text-decoration property when the value |
+ of -webkit-text-decorations-in-effect is none. |
+ (WebCore::EditingStyle::removeStyleFromRulesAndContext): Since display: inline and float: none are now |
+ added on copy, remove these properties on paste. |
+ (WebCore::EditingStyle::removePropertiesInElementDefaultStyle): Takes Element* instead of StyledElement*. |
+ (WebCore::EditingStyle::forceInline): Added. |
+ (WebCore::getPropertiesNotIn): Remove properties only when the base style has them. |
+ * editing/EditingStyle.h: |
+ * editing/markup.cpp: |
+ (WebCore::StyledMarkupAccumulator::shouldApplyWrappingStyle): Added. |
+ (WebCore::StyledMarkupAccumulator::StyledMarkupAccumulator): Takes highestNodeToBeSerialized. |
+ (WebCore::StyledMarkupAccumulator::wrapWithStyleNode): Calls appendStyleNodeOpenTag and styleNodeCloseTag. |
+ (WebCore::StyledMarkupAccumulator::appendStyleNodeOpenTag): Extracted from wrapWithStyleNode. |
+ (WebCore::StyledMarkupAccumulator::styleNodeCloseTag): Ditto. |
+ (WebCore::StyledMarkupAccumulator::appendText): Wraps text node with a style span if needed. |
+ Set display: inline and float: none so that it won't be converted to a block on paste side. |
+ (WebCore::StyledMarkupAccumulator::appendElement): Add wrapping style if appropriate; Remove any properties |
+ that are overridden by default style and any style that may conflict with the computed style of node to |
+ avoid modifying the appearance of the serialized nodes. |
+ (WebCore::StyledMarkupAccumulator::serializeNodes): Compute wrapping style; copies of this style are |
+ modified as needed when serializing top-level elements or text nodes. We call traverseNodesForSerialization |
+ with NodeTraversalMode set to DoNotEmitString first to compute the highest node to be serialized. The second |
+ call to the function actually serialize the nodes. |
+ (WebCore::StyledMarkupAccumulator::traverseNodesForSerialization): Extracted from serializeNodes. |
+ Outputs string only if NodeTraversalMode is set to EmitString. |
+ (WebCore::createMarkup): No longer adds wrapping spans. |
+ |
+2011-08-10 Adam Barth <abarth@webkit.org> |
+ |
+ Add tests of optional arguments for Geolocation |
+ https://bugs.webkit.org/show_bug.cgi?id=65810 |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Geolocation's treatment of not-enough-arguments almost matches the |
+ spec. getCurrentPosition and watchPosition work properly, but, because |
+ they are custom, we should use the prettier form of the [Optional] |
+ attribute. |
+ |
+ Before this series of patches, clearWatch argument was optional, but |
+ that doesn't match the spec or other browsers. Calling this function |
+ with zero arguments is pretty non-sensical, so there shouldn't be much |
+ compat risk to tightening up our behavior here. Matching other |
+ browsers and the spec seems like the bigger win. If we run into compat |
+ problems, we can re-evaluate this decision. |
+ |
+ Test: fast/dom/Geolocation/not-enough-arguments.html |
+ |
+ * page/Geolocation.idl: |
+ |
+2011-08-10 Tim Horton <timothy_horton@apple.com> |
+ |
+ SVGAElement check for internal non-view anchors is inside ENABLE(SVG_ANIMATION) |
+ https://bugs.webkit.org/show_bug.cgi?id=66026 |
+ |
+ Reviewed by Rob Buis. |
+ |
+ Move only the SMIL-related code inside the ENABLE(SVG_ANIMATION) block. |
+ |
+ No new tests, as it depends on the enabled features. |
+ |
+ * svg/SVGAElement.cpp: |
+ (WebCore::SVGAElement::defaultEventHandler): |
+ |
+2011-08-10 Tim Horton <timothy_horton@apple.com> |
+ |
+ Crash when clicking an SVG <a> link to the local document |
+ https://bugs.webkit.org/show_bug.cgi?id=66019 |
+ <rdar://problem/9933311> |
+ |
+ Reviewed by Rob Buis. |
+ |
+ Check if the target element of an internal link is valid before |
+ making use of it. |
+ |
+ Test: svg/custom/click-internal-anchor-with-use-crash.xhtml |
+ |
+ * svg/SVGAElement.cpp: |
+ (WebCore::SVGAElement::defaultEventHandler): |
+ |
+2011-08-10 No'am Rosenthal <noam.rosenthal@nokia.com> |
+ |
+ [Qt] Enable WebGL by default for QtWebKit |
+ https://bugs.webkit.org/show_bug.cgi?id=65998 |
+ |
+ Reviewed by Benjamin Poulain. |
+ |
+ Make ENABLE_WEBGL true by default when Qt is configured with OpenGL. |
+ |
+ No new tests. Build change only. |
+ |
+ * features.pri: |
+ |
+2011-08-10 Abhishek Arya <inferno@chromium.org> |
+ |
+ Check that we do not need layout before trying to dirty |
+ m_originatingLine for our floats. |
+ https://bugs.webkit.org/show_bug.cgi?id=65938 |
+ |
+ Reviewed by Dave Hyatt. |
+ |
+ Test: fast/block/float/float-originating-line-deleted-crash.html |
+ |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::removeFloatingObject): |
+ (WebCore::RenderBlock::clearFloats): |
+ |
+2011-08-10 David Hyatt <hyatt@apple.com> |
+ |
+ Add missing regions/exclusions-related files to EFL. |
+ |
+ * CMakeLists.txt: |
+ |
+2011-08-10 David Hyatt <hyatt@apple.com> |
+ |
+ Add missing regions/exclusions-related files to Chromium. |
+ |
+ * WebCore.gypi: |
+ |
+2011-08-10 David Hyatt <hyatt@apple.com> |
+ |
+ Fix Qt build bustage. Add missing regions/exclusions-related files to their project. |
+ |
+ * WebCore.pro: |
+ |
+2011-08-10 David Hyatt <hyatt@apple.com> |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=66004 |
+ |
+ Remove ifdefs for CSS Exclusions (since the feature doesn't exist any longer and has morphed into positioned |
+ floats instead). Go ahead and yank the ifdefs for CSS Regions also and just turn them on. |
+ |
+ Reviewed by Adam Roben. |
+ |
+ * Configurations/FeatureDefines.xcconfig: |
+ * css/CSSComputedStyleDeclaration.cpp: |
+ (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::parseValue): |
+ (WebCore::CSSParser::parseContent): |
+ (WebCore::CSSParser::parseWrapShape): |
+ (WebCore::CSSParser::parseFromFlowContent): |
+ * css/CSSParser.h: |
+ * css/CSSPrimitiveValue.cpp: |
+ (WebCore::CSSPrimitiveValue::init): |
+ (WebCore::CSSPrimitiveValue::cleanup): |
+ (WebCore::CSSPrimitiveValue::getStringValue): |
+ (WebCore::CSSPrimitiveValue::cssText): |
+ * css/CSSPrimitiveValue.h: |
+ (WebCore::CSSPrimitiveValue::getShapeValue): |
+ * css/CSSPrimitiveValueMappings.h: |
+ (WebCore::CSSPrimitiveValue::operator RegionOverflow): |
+ * css/CSSPropertyNames.in: |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ * css/CSSValueKeywords.in: |
+ * css/CSSWrapShapes.cpp: |
+ * css/CSSWrapShapes.h: |
+ * css/SVGCSSValueKeywords.in: |
+ * dom/Node.cpp: |
+ (WebCore::Node::diff): |
+ * dom/NodeRenderingContext.cpp: |
+ (WebCore::NodeRenderingContext::NodeRenderingContext): |
+ (WebCore::NodeRenderingContext::setStyle): |
+ (WebCore::NodeRenderingContext::nextRenderer): |
+ (WebCore::NodeRenderingContext::previousRenderer): |
+ (WebCore::NodeRenderingContext::parentRenderer): |
+ (WebCore::NodeRenderingContext::moveToFlowThreadIfNeeded): |
+ (WebCore::NodeRendererFactory::createRendererIfNeeded): |
+ * dom/NodeRenderingContext.h: |
+ * rendering/RenderFlowThread.cpp: |
+ * rendering/RenderFlowThread.h: |
+ * rendering/RenderLayer.cpp: |
+ (WebCore::RenderLayer::collectLayers): |
+ * rendering/RenderObject.cpp: |
+ (WebCore::RenderObject::createObject): |
+ * rendering/RenderObject.h: |
+ (WebCore::RenderObject::isRenderRegion): |
+ (WebCore::RenderObject::isRenderFlowThread): |
+ * rendering/RenderRegion.cpp: |
+ * rendering/RenderRegion.h: |
+ * rendering/RenderTreeAsText.cpp: |
+ (WebCore::writeLayers): |
+ * rendering/RenderView.cpp: |
+ (WebCore::RenderView::renderFlowThreadWithName): |
+ * rendering/RenderView.h: |
+ * rendering/style/RenderStyle.cpp: |
+ (WebCore::RenderStyle::diff): |
+ * rendering/style/RenderStyle.h: |
+ (WebCore::InheritedFlags::regionOverflow): |
+ (WebCore::InheritedFlags::setRegionOverflow): |
+ (WebCore::InheritedFlags::initialWrapShape): |
+ (WebCore::InheritedFlags::initialRegionOverflow): |
+ * rendering/style/RenderStyleConstants.h: |
+ * rendering/style/StyleRareNonInheritedData.cpp: |
+ (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): |
+ (WebCore::StyleRareNonInheritedData::operator==): |
+ * rendering/style/StyleRareNonInheritedData.h: |
+ |
+2011-08-10 Abhishek Arya <inferno@chromium.org> |
+ |
+ Don't create html renderer for SVG <glyphRef> |
+ https://bugs.webkit.org/show_bug.cgi?id=66000 |
+ |
+ Reviewed by David Hyatt. |
+ |
+ Test: svg/text/glyphref-renderer-create-crash.html |
+ |
+ * svg/SVGGlyphRefElement.h: |
+ |
+2011-08-10 Scott Graham <scottmg@google.com> |
+ |
+ OpenTypeSanitizer in chromium gyp files is linked twice |
+ https://bugs.webkit.org/show_bug.cgi?id=65927 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, should just build and link properly on all platforms. |
+ |
+ * WebCore.gyp/WebCore.gyp: |
+ |
+2011-08-10 Oliver Hunt <oliver@apple.com> |
+ |
+ Make GC checks more aggressive in release builds |
+ https://bugs.webkit.org/show_bug.cgi?id=66001 |
+ |
+ Reviewed by Gavin Barraclough. |
+ |
+ Fix GC bugs found while testing increased validation logic |
+ |
+ * bindings/js/JSDOMWindowShell.cpp: |
+ (WebCore::JSDOMWindowShell::JSDOMWindowShell): |
+ * bindings/js/JSDOMWindowShell.h: |
+ * bindings/js/ScriptController.cpp: |
+ (WebCore::ScriptController::createWindowShell): |
+ * bridge/objc/ObjCRuntimeObject.h: |
+ (JSC::Bindings::ObjCRuntimeObject::create): |
+ * bridge/objc/ObjCRuntimeObject.mm: |
+ (JSC::Bindings::ObjCRuntimeObject::ObjCRuntimeObject): |
+ * bridge/objc/objc_instance.mm: |
+ |
+2011-08-10 Ben Wells <benwells@chromium.org> |
+ |
+ [skia] Move calls to makeGrContextCurrent into clipPathAntiAliased from callers |
+ https://bugs.webkit.org/show_bug.cgi?id=65733 |
+ |
+ This protects against callers of clipPathAntiAliased forgetting to call makeGrContextCurrent. |
+ |
+ Reviewed by James Robinson. |
+ |
+ No change in functionality, no new tests. |
+ |
+ * platform/graphics/skia/GraphicsContextSkia.cpp: |
+ (WebCore::GraphicsContext::addInnerRoundedRectClip): |
+ (WebCore::GraphicsContext::clip): |
+ * platform/graphics/skia/PlatformContextSkia.cpp: |
+ (WebCore::PlatformContextSkia::clipPathAntiAliased): |
+ |
+2011-08-10 Benjamin Poulain <ikipou@gmail.com> |
+ |
+ MarkupAccumulator: make resolution of URLs implicit to appendQuotedURLAttributeValue() |
+ https://bugs.webkit.org/show_bug.cgi?id=65990 |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ Instead of resolving the URLs in the call site of appendQuotedURLAttributeValue(), |
+ the URL is now resolved if necessary in appendQuotedURLAttributeValue(). |
+ |
+ * editing/MarkupAccumulator.cpp: |
+ (WebCore::MarkupAccumulator::appendQuotedURLAttributeValue): |
+ (WebCore::MarkupAccumulator::appendAttribute): |
+ * editing/MarkupAccumulator.h: |
+ |
+2011-08-10 Kenichi Ishibashi <bashi@chromium.org> |
+ |
+ [Chromium] Implement text shaping with font-feature-settings on Linux |
+ https://bugs.webkit.org/show_bug.cgi?id=65903 |
+ |
+ - Modifies Font::codePath() to return Complex when there is font feature settings CSS property. |
+ - Implements OpenType advanced typography feature using Harfbuzz on Chromium Linux port. |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Test: css3/font-feature-settings-rendering.html |
+ |
+ * platform/graphics/Font.cpp: |
+ (WebCore::Font::codePath): Returns Complex when fontFeatureSettings exists. |
+ * platform/graphics/chromium/ComplexTextControllerLinux.cpp: |
+ (WebCore::setupFontFeatures): Added. |
+ (WebCore::ComplexTextController::setupFontForScriptRun): Calls setupFontFeatures(). |
+ |
+2011-08-10 Tony Gentilcore <tonyg@chromium.org> |
+ |
+ [chromium] Notify MemoryCache of decoded data access |
+ https://bugs.webkit.org/show_bug.cgi?id=65859 |
+ |
+ Reviewed by James Robinson. |
+ |
+ This causes us to call CachedImage::didDraw() which calls CachedResource::didAccessDecodedData() |
+ which updates the LRU for decoded image data and potentially prunes the cache. This is already done |
+ by almost all of the other ports. |
+ |
+ The decoded bitmaps obviously use quite a bit of memory, so pruning them when necessary is important. |
+ In my tests, this saves several MB on many popular web pages. |
+ |
+ No new tests because no observable difference in functionality. |
+ |
+ * platform/graphics/skia/ImageSkia.cpp: |
+ (WebCore::BitmapImage::draw): |
+ (WebCore::BitmapImageSingleFrameSkia::draw): |
+ |
+2011-08-10 Vsevolod Vlasov <vsevik@chromium.org> |
+ |
+ Web Inspector: Remove Network.initialContentSet from protocol, store workers content on backend. |
+ https://bugs.webkit.org/show_bug.cgi?id=65929 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Tests: http/tests/inspector/network/network-shared-worker.html |
+ http/tests/inspector/network/network-worker.html |
+ |
+ * inspector/Inspector.json: |
+ * inspector/InspectorInstrumentation.cpp: |
+ (WebCore::InspectorInstrumentation::didReceiveScriptResponseImpl): |
+ * inspector/InspectorInstrumentation.h: |
+ (WebCore::InspectorInstrumentation::didReceiveScriptResponse): |
+ * inspector/InspectorResourceAgent.cpp: |
+ (WebCore::InspectorResourceAgent::didReceiveResponse): |
+ (WebCore::InspectorResourceAgent::setInitialScriptContent): |
+ (WebCore::InspectorResourceAgent::didReceiveScriptResponse): |
+ * inspector/InspectorResourceAgent.h: |
+ * inspector/front-end/NetworkManager.js: |
+ * inspector/front-end/Resource.js: |
+ * workers/DefaultSharedWorkerRepository.cpp: |
+ (WebCore::SharedWorkerScriptLoader::didReceiveResponse): |
+ * workers/Worker.cpp: |
+ (WebCore::Worker::didReceiveResponse): |
+ * workers/Worker.h: |
+ * workers/WorkerScriptLoader.cpp: |
+ (WebCore::WorkerScriptLoader::didReceiveResponse): |
+ * workers/WorkerScriptLoaderClient.h: |
+ (WebCore::WorkerScriptLoaderClient::didReceiveResponse): |
+ |
+2011-08-10 Vsevolod Vlasov <vsevik@chromium.org> |
+ |
+ Web Inspector: [Network] Filtering by stylesheet does not update the # reqs, filesizes. |
+ https://bugs.webkit.org/show_bug.cgi?id=65515 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * English.lproj/localizedStrings.js: |
+ * inspector/front-end/NetworkPanel.js: |
+ (WebInspector.NetworkLogView.prototype._updateSummaryBar): |
+ (WebInspector.NetworkLogView.prototype._updateFilter): |
+ |
+2011-08-10 Vsevolod Vlasov <vsevik@chromium.org> |
+ |
+ Web Inspector: console messages markers are lost in Resources panel after page reload |
+ https://bugs.webkit.org/show_bug.cgi?id=60701 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Test: http/tests/inspector/resource-tree/resource-tree-errors-reload.html |
+ |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.ResourceTreeModel): |
+ (WebInspector.ResourceTreeModel.prototype._consoleMessageAdded): |
+ (WebInspector.ResourceTreeModel.prototype._addPendingConsoleMessage): |
+ (WebInspector.ResourceTreeModel.prototype._addPendingConsoleMessagesToResource): |
+ (WebInspector.ResourceTreeModel.prototype._addConsoleMessageToResource): |
+ (WebInspector.ResourceTreeModel.prototype._consoleCleared): |
+ (WebInspector.ResourceTreeModel.prototype._bindResourceURL): |
+ |
+2011-08-10 Lars Knudsen <lars.knudsen@nokia.com> |
+ |
+ Add device specific settings to support viewport computing |
+ https://bugs.webkit.org/show_bug.cgi?id=65984 |
+ |
+ Reviewed by Kenneth Rohde Christiansen. |
+ |
+ Based on patch by Zalan Bujtas. |
+ |
+ Added settings for device size, dpi and layout fallback width |
+ to support viewport computing. |
+ |
+ * page/Settings.cpp: |
+ (WebCore::Settings::Settings): |
+ * page/Settings.h: |
+ (WebCore::Settings::setLayoutFallbackWidth): |
+ (WebCore::Settings::layoutFallbackWidth): |
+ (WebCore::Settings::setDeviceWidth): |
+ (WebCore::Settings::deviceWidth): |
+ (WebCore::Settings::setDeviceHeight): |
+ (WebCore::Settings::deviceHeight): |
+ (WebCore::Settings::setDeviceDPI): |
+ (WebCore::Settings::deviceDPI): |
+ |
+2011-08-10 Benjamin Poulain <benjamin@webkit.org> |
+ |
+ Unify the way we generate HTML for an image in the Clipboard |
+ https://bugs.webkit.org/show_bug.cgi?id=58043 |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ Unify the way we generate the markup when dragging and image. |
+ |
+ Previously, the code was using the special function imageToMarkup() that was |
+ spread from the Chromium port. |
+ That function ignores some edge cases so this patch replace it by the |
+ more generic MarkupAccumulator (which is also used by the other pasteboard |
+ actions). |
+ |
+ The MarkupAccumulator was modified because following r61178, the local filepath |
+ are treated differently (the enum was changed to make this behavior |
+ explicit). |
+ |
+ Tests: fast/drag-and-drop/drag-and-drop-image-contenteditable.html |
+ fast/drag-and-drop/drag-and-drop-inputimage-contenteditable.html |
+ fast/drag-and-drop/drag-and-drop-objectimage-contenteditable.html |
+ |
+ * editing/MarkupAccumulator.cpp: |
+ (WebCore::MarkupAccumulator::MarkupAccumulator): |
+ (WebCore::MarkupAccumulator::resolveURLIfNeeded): |
+ (WebCore::MarkupAccumulator::appendAttribute): |
+ * editing/MarkupAccumulator.h: |
+ * editing/markup.cpp: |
+ * editing/markup.h: |
+ * page/PageSerializer.cpp: |
+ (WebCore::SerializerMarkupAccumulator::SerializerMarkupAccumulator): |
+ * platform/chromium/ClipboardChromium.cpp: |
+ (WebCore::ClipboardChromium::declareAndWriteDragImage): |
+ (WebCore::ClipboardChromium::writeRange): |
+ * platform/chromium/PasteboardChromium.cpp: |
+ (WebCore::Pasteboard::writeSelection): |
+ * platform/gtk/ClipboardGtk.cpp: |
+ (WebCore::ClipboardGtk::declareAndWriteDragImage): |
+ (WebCore::ClipboardGtk::writeRange): |
+ * platform/gtk/DataObjectGtk.cpp: |
+ (WebCore::DataObjectGtk::markup): |
+ * platform/gtk/PasteboardGtk.cpp: |
+ (WebCore::Pasteboard::writeSelection): |
+ (WebCore::Pasteboard::writeImage): |
+ * platform/haiku/PasteboardHaiku.cpp: |
+ (WebCore::Pasteboard::writeSelection): |
+ * platform/qt/ClipboardQt.cpp: |
+ (WebCore::ClipboardQt::declareAndWriteDragImage): |
+ (WebCore::ClipboardQt::writeRange): |
+ * platform/qt/PasteboardQt.cpp: |
+ (WebCore::Pasteboard::writeSelection): |
+ * platform/win/ClipboardWin.cpp: |
+ (WebCore::ClipboardWin::declareAndWriteDragImage): |
+ |
+2011-08-10 Pavel Feldman <pfeldman@google.com> |
+ |
+ Web Inspector: auto-completion for string literals doesn't work |
+ https://bugs.webkit.org/show_bug.cgi?id=65701 |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ * inspector/InjectedScript.cpp: |
+ (WebCore::InjectedScript::evaluateOnCallFrame): |
+ * inspector/InjectedScript.h: |
+ * inspector/InjectedScriptSource.js: |
+ (.): |
+ (): |
+ * inspector/Inspector.json: |
+ * inspector/InspectorDebuggerAgent.cpp: |
+ (WebCore::InspectorDebuggerAgent::evaluateOnCallFrame): |
+ * inspector/InspectorDebuggerAgent.h: |
+ * inspector/front-end/ConsoleView.js: |
+ (WebInspector.ConsoleView.prototype._completions.evaluated.getCompletions): |
+ (WebInspector.ConsoleView.prototype._completions.evaluated): |
+ (WebInspector.ConsoleView.prototype._completions.receivedPropertyNamesFromEval): |
+ (WebInspector.ConsoleView.prototype._completions.receivedPropertyNames): |
+ (WebInspector.ConsoleView.prototype._completions): |
+ (WebInspector.ConsoleView.prototype.evalInInspectedWindow): |
+ * inspector/front-end/DebuggerPresentationModel.js: |
+ (WebInspector.PresenationCallFrame.prototype.evaluate): |
+ * inspector/front-end/RemoteObject.js: |
+ (WebInspector.RemoteObject): |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype.evaluateInSelectedCallFrame): |
+ (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.evaluateInSelectedCallFrame): |
+ |
+2011-08-10 Pavel Feldman <pfeldman@google.com> |
+ |
+ Web Inspector: CSS editing file containing tabs leads to the whole file change in revision history. |
+ https://bugs.webkit.org/show_bug.cgi?id=65959 |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ * inspector/front-end/SourceFrame.js: |
+ (WebInspector.SourceFrame): |
+ * inspector/front-end/TextEditorModel.js: |
+ (WebInspector.TextEditorModel.prototype._innerSetText): |
+ |
+2011-08-10 Pavel Feldman <pfeldman@chromium.org> |
+ |
+ Web Inspector: there should be a way to tell what properties are non-enumerable when expanding objects. |
+ https://bugs.webkit.org/show_bug.cgi?id=65518 |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ * inspector/InjectedScript.cpp: |
+ (WebCore::InjectedScript::getProperties): |
+ * inspector/InjectedScript.h: |
+ * inspector/InjectedScriptSource.js: |
+ (.): |
+ * inspector/Inspector.json: |
+ * inspector/InspectorRuntimeAgent.cpp: |
+ (WebCore::InspectorRuntimeAgent::getProperties): |
+ * inspector/InspectorRuntimeAgent.h: |
+ * inspector/front-end/ObjectPropertiesSection.js: |
+ (WebInspector.ObjectPropertyTreeElement.prototype.ondblclick): |
+ (WebInspector.ObjectPropertyTreeElement.prototype.update): |
+ * inspector/front-end/RemoteObject.js: |
+ (WebInspector.RemoteObject.prototype.getOwnProperties): |
+ (WebInspector.RemoteObject.prototype.getAllProperties): |
+ (WebInspector.RemoteObjectProperty): |
+ * inspector/front-end/inspector.css: |
+ (.section .properties .dimmed): |
+ |
+2011-08-10 Alexander Pavlov <apavlov@chromium.org> |
+ |
+ Web Inspector: [REGRESSION] Editor lost after committing a CSS property value for inline style |
+ https://bugs.webkit.org/show_bug.cgi?id=65918 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * inspector/front-end/StylesSidebarPane.js: |
+ (WebInspector.StylesSidebarPane.prototype._innerUpdate.stylesCallback): |
+ (WebInspector.StylesSidebarPane.prototype._innerUpdate.computedStyleCallback): |
+ (WebInspector.StylesSidebarPane.prototype._innerUpdate): |
+ (WebInspector.StylePropertyTreeElement.prototype): |
+ (WebInspector.StylePropertyTreeElement.prototype.event): |
+ (WebInspector.StylePropertyTreeElement.prototype.styleText.updateInterface.majorChange.isRevert.parentPane): |
+ (WebInspector.StylePropertyTreeElement.prototype.styleText.updateInterface.majorChange.isRevert): |
+ |
+2011-08-10 Yuta Kitamura <yutak@chromium.org> |
+ |
+ WebSocket: Add binaryType attribute |
+ https://bugs.webkit.org/show_bug.cgi?id=65967 |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ Add a new attribute "binaryType" to WebSocket. It controls the data type of MessageEvent |
+ for WebSocket's binary frames. |
+ |
+ Tests: http/tests/websocket/tests/hixie76/undefined-attributes.html |
+ http/tests/websocket/tests/hybi/binary-type.html |
+ |
+ * websockets/WebSocket.cpp: |
+ (WebCore::WebSocket::WebSocket): |
+ (WebCore::WebSocket::binaryType): |
+ (WebCore::WebSocket::setBinaryType): |
+ * websockets/WebSocket.h: |
+ * websockets/WebSocket.idl: |
+ |
+2011-08-10 Yuta Kitamura <yutak@chromium.org> |
+ |
+ WebSocket: Accept multiple subprotocols |
+ https://bugs.webkit.org/show_bug.cgi?id=65247 |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ WebSocket constructor should accept an array of subprotocols if hybi-10 protocol is chosen. |
+ |
+ Tests: http/tests/websocket/tests/hybi/invalid-subprotocol-characters.html |
+ http/tests/websocket/tests/hybi/invalid-subprotocols.html |
+ http/tests/websocket/tests/hybi/multiple-subprotocols.html |
+ http/tests/websocket/tests/hybi/workers/multiple-subprotocols.html |
+ |
+ * bindings/js/JSWebSocketCustom.cpp: |
+ (WebCore::JSWebSocketConstructor::constructJSWebSocket): |
+ If the second argument of WebSocket constructor is an array, build a Vector<String> and |
+ pass it to WebSocket::connect(). |
+ * bindings/v8/custom/V8WebSocketCustom.cpp: |
+ (WebCore::V8WebSocket::constructorCallback): |
+ Ditto. |
+ * websockets/WebSocket.cpp: |
+ (WebCore::isValidProtocolCharacter): Added. |
+ (WebCore::isValidProtocolString): Added. |
+ (WebCore::isValidProtocolStringHixie76): Renamed from isValidProtocolString. |
+ (WebCore::joinStrings): |
+ (WebCore::WebSocket::connect): |
+ If an array is provided as a subprotocol, and if hixie-76 is used for this connection, it |
+ must be converted to a string in the same manner as JavaScript's Array.toString() in order |
+ to keep compatibility. |
+ * websockets/WebSocket.h: |
+ Removed m_protocol because it was not used at all. |
+ * websockets/WebSocketChannel.cpp: |
+ (WebCore::WebSocketChannel::disconnect): |
+ WebSocketChannel may be released before connect() is called, thus m_handshake may be null. |
+ |
+2011-08-10 Andrey Kosyakov <caseq@chromium.org> |
+ |
+ Web Inspector: Network panel always displays file:// resources as "(pending)" |
+ https://bugs.webkit.org/show_bug.cgi?id=65784 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Test: inspector/network-status-non-http.html |
+ |
+ * English.lproj/localizedStrings.js: |
+ * inspector/front-end/NetworkPanel.js: |
+ (WebInspector.NetworkDataGridNode.prototype._refreshStatusCell): |
+ * inspector/front-end/Resource.js: |
+ (WebInspector.Resource.prototype.isHttpFamily): |
+ |
+2011-08-09 Luke Macpherson <macpherson@chromium.org> |
+ |
+ Use setFontDescription helper to set m_fontDirty. |
+ https://bugs.webkit.org/show_bug.cgi?id=65956 |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ No new tests / trivial cleanup. |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ |
+2011-08-09 Dominic Cooney <dominicc@chromium.org> |
+ |
+ [V8] The prototype property of DOM constructors should be read-only. |
+ https://bugs.webkit.org/show_bug.cgi?id=39655 |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ Test: fast/dom/prototype-property.html |
+ |
+ * bindings/scripts/CodeGeneratorV8.pm: Make prototypes read-only. |
+ * bindings/scripts/test/V8/V8TestInterface.cpp: Update expectations. |
+ (WebCore::ConfigureV8TestInterfaceTemplate): |
+ * bindings/scripts/test/V8/V8TestMediaQueryListListener.cpp: |
+ (WebCore::ConfigureV8TestMediaQueryListListenerTemplate): |
+ * bindings/scripts/test/V8/V8TestObj.cpp: |
+ (WebCore::ConfigureV8TestObjTemplate): |
+ * bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp: |
+ (WebCore::ConfigureV8TestSerializedScriptValueInterfaceTemplate): |
+ |
+2011-08-09 Abhishek Arya <inferno@chromium.org> |
+ |
+ Style change not propagating for before, after content |
+ for table parts. |
+ https://bugs.webkit.org/show_bug.cgi?id=51862 |
+ |
+ Fix by Chris Evans (cevans@chromium.org). |
+ In the table case, make sure to return the before / |
+ after parent rather then the before / after node |
+ itself. |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ * rendering/RenderObjectChildList.cpp: |
+ (WebCore::findBeforeAfterParent): |
+ |
+2011-08-09 Luke Macpherson <macpherson@chromium.org> |
+ |
+ Implement string based properties in CSSStyleApplyProperty. |
+ https://bugs.webkit.org/show_bug.cgi?id=65662 |
+ |
+ Reviewed by Darin Adler. |
+ |
+ No new tests / refactoring only. |
+ |
+ * css/CSSStyleApplyProperty.cpp: |
+ (WebCore::ApplyPropertyString::ApplyPropertyString): |
+ Added class to handle string based properties. |
+ (WebCore::ApplyPropertyString::applyValue): |
+ (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): |
+ Add initializers for CSSPropertyWebkitHighlight and CSSPropertyWebkitHyphenateCharacter. |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ Remove existing implementations for CSSPropertyWebkitHighlight and CSSPropertyWebkitHyphenateCharacter. |
+ |
+2011-08-09 Emil A Eklund <eae@chromium.org> |
+ |
+ Switch RenderBlock to to new layout types |
+ https://bugs.webkit.org/show_bug.cgi?id=65396 |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ No new tests, no new functionality. |
+ |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::MarginInfo::MarginInfo): |
+ (WebCore::RenderBlock::layoutBlock): |
+ (WebCore::RenderBlock::addOverflowFromChildren): |
+ (WebCore::RenderBlock::computeOverflow): |
+ (WebCore::RenderBlock::adjustPositionedBlock): |
+ (WebCore::RenderBlock::adjustFloatingBlock): |
+ (WebCore::RenderBlock::collapseMargins): |
+ (WebCore::RenderBlock::clearFloatsIfNeeded): |
+ (WebCore::RenderBlock::estimateLogicalTopPosition): |
+ (WebCore::RenderBlock::determineLogicalLeftPositionForChild): |
+ (WebCore::RenderBlock::setLogicalLeftForChild): |
+ (WebCore::RenderBlock::setLogicalTopForChild): |
+ (WebCore::RenderBlock::layoutBlockChildren): |
+ (WebCore::RenderBlock::layoutBlockChild): |
+ (WebCore::RenderBlock::flipFloatForWritingMode): |
+ (WebCore::RenderBlock::isPointInOverflowControl): |
+ (WebCore::RenderBlock::hitTestFloats): |
+ * rendering/RenderBlock.h: |
+ (WebCore::RenderBlock::logicalTopForFloat): |
+ (WebCore::RenderBlock::logicalBottomForFloat): |
+ (WebCore::RenderBlock::logicalLeftForFloat): |
+ (WebCore::RenderBlock::logicalRightForFloat): |
+ (WebCore::RenderBlock::logicalWidthForFloat): |
+ (WebCore::RenderBlock::setLogicalTopForFloat): |
+ (WebCore::RenderBlock::setLogicalLeftForFloat): |
+ (WebCore::RenderBlock::setLogicalHeightForFloat): |
+ (WebCore::RenderBlock::setLogicalWidthForFloat): |
+ |
+2011-08-09 Jeffrey Pfau <jpfau@apple.com> |
+ |
+ Initial pass at a new XML tree builder |
+ https://bugs.webkit.org/show_bug.cgi?id=65803 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * xml/parser/NewXMLDocumentParser.cpp: |
+ (WebCore::NewXMLDocumentParser::NewXMLDocumentParser): |
+ (WebCore::NewXMLDocumentParser::~NewXMLDocumentParser): |
+ (WebCore::NewXMLDocumentParser::append): |
+ (WebCore::NewXMLDocumentParser::finish): |
+ (WebCore::NewXMLDocumentParser::finishWasCalled): |
+ * xml/parser/NewXMLDocumentParser.h: |
+ * xml/parser/XMLTreeBuilder.cpp: Added. |
+ (WebCore::XMLTreeBuilder::XMLTreeBuilder): |
+ (WebCore::XMLTreeBuilder::processToken): |
+ (WebCore::XMLTreeBuilder::pushCurrentNode): |
+ (WebCore::XMLTreeBuilder::popCurrentNode): |
+ (WebCore::XMLTreeBuilder::processProcessingInstruction): |
+ (WebCore::XMLTreeBuilder::processXMLDeclaration): |
+ (WebCore::XMLTreeBuilder::processDOCTYPE): |
+ (WebCore::XMLTreeBuilder::processStartTag): |
+ (WebCore::XMLTreeBuilder::processEndTag): |
+ (WebCore::XMLTreeBuilder::processCharacter): |
+ (WebCore::XMLTreeBuilder::processCDATA): |
+ (WebCore::XMLTreeBuilder::processComment): |
+ (WebCore::XMLTreeBuilder::processEntity): |
+ (WebCore::XMLTreeBuilder::processNamespaces): |
+ (WebCore::XMLTreeBuilder::processAttributes): |
+ (WebCore::XMLTreeBuilder::processXMLEntity): |
+ (WebCore::XMLTreeBuilder::processHTMLEntity): |
+ (WebCore::XMLTreeBuilder::add): |
+ (WebCore::XMLTreeBuilder::appendToText): |
+ (WebCore::XMLTreeBuilder::enterText): |
+ (WebCore::XMLTreeBuilder::exitText): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::NodeStackItem): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::hasNamespaceURI): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::namespaceURI): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::setNamespaceURI): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::namespaceForPrefix): |
+ * xml/parser/XMLTreeBuilder.h: Added. |
+ (WebCore::XMLTreeBuilder::create): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::namespaceURI): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::setNamespaceURI): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::node): |
+ (WebCore::XMLTreeBuilder::NodeStackItem::setNode): |
+ |
+2011-08-09 Mark Hahnenberg <mhahnenberg@apple.com> |
+ |
+ Add ParentClass typedef in all JSC classes |
+ https://bugs.webkit.org/show_bug.cgi?id=65731 |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ No new tests. |
+ |
+ Just added the Base typedefs in all the classes that are a subclass of JSCell |
+ to point at their parent classes. This is a change to support future changes to the way |
+ constructors and destructors are implemented in JS objects, among other things. |
+ |
+ * bindings/js/JSAudioConstructor.h: |
+ * bindings/js/JSImageConstructor.h: |
+ * bindings/js/JSOptionConstructor.h: |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ (GenerateHeader): |
+ (GenerateConstructorDeclaration): |
+ * bindings/scripts/test/JS/JSTestInterface.cpp: |
+ * bindings/scripts/test/JS/JSTestInterface.h: |
+ * bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp: |
+ * bindings/scripts/test/JS/JSTestMediaQueryListListener.h: |
+ * bindings/scripts/test/JS/JSTestObj.cpp: |
+ * bindings/scripts/test/JS/JSTestObj.h: |
+ * bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp: |
+ * bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.h: |
+ * bridge/c/CRuntimeObject.h: |
+ * bridge/c/c_instance.cpp: |
+ * bridge/jni/jsc/JavaInstanceJSC.cpp: |
+ * bridge/jni/jsc/JavaRuntimeObject.h: |
+ * bridge/objc/ObjCRuntimeObject.h: |
+ * bridge/objc/objc_runtime.h: |
+ * bridge/qt/qt_instance.cpp: |
+ * bridge/qt/qt_pixmapruntime.cpp: |
+ * bridge/qt/qt_runtime.h: |
+ * bridge/runtime_array.h: |
+ * bridge/runtime_method.h: |
+ * bridge/runtime_object.h: |
+ * bridge/testqtbindings.cpp: |
+ (Global::className): |
+ |
+2011-08-09 Alexei Svitkine <asvitkine@chromium.org> |
+ |
+ [Chromium] Enable rubber banding when scrolling. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=65707 |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ No new tests since this is just syncing changes to chromium platform. |
+ |
+ * platform/PlatformWheelEvent.h: |
+ (WebCore::PlatformWheelEvent::PlatformWheelEvent): |
+ * platform/chromium/ScrollAnimatorChromiumMac.mm: |
+ (WebCore::ScrollAnimatorChromiumMac::handleGestureEvent): |
+ |
+2011-08-09 Vsevolod Vlasov <vsevik@chromium.org> |
+ |
+ Web Inspector: XHRs end up in the resources panel. |
+ https://bugs.webkit.org/show_bug.cgi?id=60321 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Test: http/tests/inspector/resource-tree/resource-tree-no-xhrs.html |
+ |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.ResourceTreeModel.prototype._onResourceUpdated): |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel): |
+ |
+2011-08-08 Adrienne Walker <enne@google.com> |
+ |
+ Add testing for --force-compositing-mode to windows.internal |
+ https://bugs.webkit.org/show_bug.cgi?id=65777 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Test: platform/chromium/compositing/force-compositing-mode/force-composite-empty.html |
+ |
+ Move initial initialization to settings caching function so that RLC |
+ picks up changes to the force compositing mode flag. |
+ |
+ * WebCore.exp.in: |
+ * rendering/RenderLayerCompositor.cpp: |
+ (WebCore::RenderLayerCompositor::RenderLayerCompositor): |
+ (WebCore::RenderLayerCompositor::cacheAcceleratedCompositingFlags): |
+ (WebCore::RenderLayerCompositor::updateCompositingLayers): |
+ * testing/Internals.cpp: |
+ (WebCore::Internals::setForceCompositingMode): |
+ * testing/Internals.h: |
+ * testing/Internals.idl: |
+ |
+2011-08-09 Alexandru Chiculita <achicu@adobe.com> |
+ |
+ Fixing 65868 REGRESSION(r92610) caused by 65668 - Optimize floating elements lookup |
+ https://bugs.webkit.org/show_bug.cgi?id=65871 |
+ |
+ Added an interval tree in the FloatingObjects structure. Also added new mechanisms to make |
+ sure the tree is updated correctly when a float is repositioned. |
+ |
+ Changed the PODIntervalTree to support giving a search adapter that can be implemented by the |
+ client. I'm not adding a different bug for that because PODIntervalTree is not used anywhere else |
+ and would be hard to test that the change is not breaking anything. |
+ |
+ Reviewed by Dave Hyatt. |
+ |
+ No new tests, just a refactor on the floating objects data structure. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * platform/PODIntervalTree.h: |
+ (WebCore::PODIntervalSearchAdapter::PODIntervalSearchAdapter): |
+ (WebCore::PODIntervalSearchAdapter::lowValue): |
+ (WebCore::PODIntervalSearchAdapter::highValue): |
+ (WebCore::PODIntervalSearchAdapter::collectIfNeeded): |
+ (WebCore::PODIntervalTree::PODIntervalTree): |
+ (WebCore::PODIntervalTree::allOverlaps): |
+ (WebCore::PODIntervalTree::allOverlapsWithAdapter): |
+ (WebCore::PODIntervalTree::searchForOverlapsFrom): |
+ * platform/PODRedBlackTree.h: |
+ (WebCore::PODRedBlackTree::PODRedBlackTree): |
+ (WebCore::PODRedBlackTree::clear): |
+ (WebCore::PODRedBlackTree::isInitialized): |
+ (WebCore::PODRedBlackTree::initIfNeeded): |
+ (WebCore::PODRedBlackTree::add): |
+ (WebCore::PODRedBlackTree::remove): |
+ (WebCore::PODRedBlackTree::contains): |
+ (WebCore::PODRedBlackTree::visitInorder): |
+ (WebCore::PODRedBlackTree::size): |
+ (WebCore::PODRedBlackTree::checkInvariants): |
+ (WebCore::PODRedBlackTree::dump): |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::styleDidChange): |
+ (WebCore::RenderBlock::addOverflowFromFloats): |
+ (WebCore::RenderBlock::repaintOverhangingFloats): |
+ (WebCore::RenderBlock::paintFloats): |
+ (WebCore::RenderBlock::selectionGaps): |
+ (WebCore::RenderBlock::insertFloatingObject): |
+ (WebCore::RenderBlock::removeFloatingObject): |
+ (WebCore::RenderBlock::removeFloatingObjectsBelow): |
+ (WebCore::RenderBlock::positionNewFloats): |
+ (WebCore::::collectIfNeeded): |
+ (WebCore::RenderBlock::logicalLeftOffsetForLine): |
+ (WebCore::RenderBlock::logicalRightOffsetForLine): |
+ (WebCore::RenderBlock::nextFloatLogicalBottomBelow): |
+ (WebCore::RenderBlock::lowestFloatLogicalBottom): |
+ (WebCore::RenderBlock::addPositionedFloats): |
+ (WebCore::RenderBlock::clearFloats): |
+ (WebCore::RenderBlock::addOverhangingFloats): |
+ (WebCore::RenderBlock::hasOverhangingFloat): |
+ (WebCore::RenderBlock::addIntrudingFloats): |
+ (WebCore::RenderBlock::markSiblingsWithFloatsForLayout): |
+ (WebCore::RenderBlock::hitTestFloats): |
+ (WebCore::RenderBlock::adjustForBorderFit): |
+ (WebCore::RenderBlock::FloatingObjects::clear): |
+ (WebCore::RenderBlock::FloatingObjects::intervalForFloatingObject): |
+ (WebCore::RenderBlock::FloatingObjects::addPlacedObject): |
+ (WebCore::RenderBlock::FloatingObjects::removePlacedObject): |
+ (WebCore::RenderBlock::FloatingObjects::add): |
+ (WebCore::RenderBlock::FloatingObjects::remove): |
+ (WebCore::RenderBlock::FloatingObjects::computePlacedFloatsTree): |
+ (WebCore::::string): |
+ * rendering/RenderBlock.h: |
+ (WebCore::RenderBlock::FloatingObject::FloatingObject): |
+ (WebCore::RenderBlock::FloatingObject::setX): |
+ (WebCore::RenderBlock::FloatingObject::setY): |
+ (WebCore::RenderBlock::FloatingObject::setWidth): |
+ (WebCore::RenderBlock::FloatingObject::setHeight): |
+ (WebCore::RenderBlock::FloatingObject::setFrameRect): |
+ (WebCore::RenderBlock::FloatingObject::isInPlacedTree): |
+ (WebCore::RenderBlock::FloatingObject::setIsInPlacedTree): |
+ (WebCore::RenderBlock::FloatIntervalSearchAdapter::FloatIntervalSearchAdapter): |
+ (WebCore::RenderBlock::FloatIntervalSearchAdapter::lowValue): |
+ (WebCore::RenderBlock::FloatIntervalSearchAdapter::highValue): |
+ (WebCore::RenderBlock::FloatingObjects::FloatingObjects): |
+ (WebCore::RenderBlock::FloatingObjects::setHorizontalWritingMode): |
+ (WebCore::RenderBlock::FloatingObjects::set): |
+ (WebCore::RenderBlock::FloatingObjects::placedFloatsTree): |
+ (WebCore::RenderBlock::FloatingObjects::computePlacedFloatsTreeIfNeeded): |
+ * rendering/RenderBlockLineLayout.cpp: |
+ (WebCore::RenderBlock::layoutRunsAndFloatsInRange): |
+ (WebCore::RenderBlock::linkToEndLineIfNeeded): |
+ (WebCore::RenderBlock::matchedEndLine): |
+ (WebCore::RenderBlock::positionNewFloatOnLine): |
+ |
+2011-08-08 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ Repeated copy and paste result in nested font elements |
+ https://bugs.webkit.org/show_bug.cgi?id=65824 |
+ |
+ Reviewed by Tony Chang. |
+ |
+ Modified isInlineNodeWithStyle to match font element. Also made the function match strictly with those nodes that |
+ ApplyStyleCommand may add to avoid removing non-styling elements. And fixed a regression from r81887 that |
+ ReplaceSelectionCommand may remove non-editing styles such as border in the attempt to avoid nesting styling elements. |
+ |
+ This patch paves the way to fix the bug 34564. |
+ |
+ Tests: editing/pasteboard/paste-text-with-style-2.html |
+ editing/pasteboard/paste-text-with-style-3.html |
+ editing/pasteboard/paste-text-with-style-4.html |
+ |
+ * editing/EditingStyle.cpp: |
+ (WebCore::HTMLElementEquivalent::matches): Takes const Element* instead of Element*. |
+ (WebCore::HTMLAttributeEquivalent::matches): Ditto. |
+ (WebCore::htmlElementEquivalents): Extracted from conflictsWithImplicitStyleOfElement. |
+ (WebCore::EditingStyle::conflictsWithImplicitStyleOfElement): Calls htmlElementEquivalents. |
+ (WebCore::EditingStyle::elementIsStyledSpanOrHTMLEquivalent): Added; determines whether an element is style span |
+ or a styling element (e.g. b, i, font) possibly with editing style. |
+ * editing/EditingStyle.h: |
+ * editing/ReplaceSelectionCommand.cpp: |
+ (WebCore::isInlineNodeWithStyle): Calls EditingStyle::elementIsStyledSpanOrHTMLEquivalent. |
+ (WebCore::ReplaceSelectionCommand::doApply): More aggressively avoid nesting styling elements. |
+ * editing/htmlediting.cpp: |
+ (WebCore::highestEnclosingNodeOfType): |
+ * editing/htmlediting.h: |
+ |
+2011-08-09 Dmitry Lomov <dslomov@google.com> |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=65778 |
+ [WebWorkers][chromium] Make statics thread-safe and make sure V8 API accesses correct isolates. |
+ |
+ Reviewed by Dmitry Titov. |
+ |
+ Covered by existing tests. |
+ |
+ * bindings/v8/V8Binding.cpp: |
+ (WebCore::V8BindingPerIsolateData::V8BindingPerIsolateData): |
+ * bindings/v8/V8Binding.h: |
+ (WebCore::V8BindingPerIsolateData::lazyEventListenerToStringTemplate): |
+ (WebCore::V8BindingPerIsolateData::hiddenPropertyName): |
+ (WebCore::V8BindingPerIsolateData::globalHandleMap): |
+ (WebCore::AllowAllocation::AllowAllocation):Moving to V8Binding.h from V8Utilities.h to resolve header dependency. |
+ (WebCore::AllowAllocation::~AllowAllocation): |
+ (WebCore::AllowAllocation::current): |
+ (WebCore::SafeAllocation::newInstance): |
+ * bindings/v8/V8GCController.cpp: |
+ (WebCore::currentGlobalHandleMap): |
+ (WebCore::enumerateGlobalHandles): |
+ (WebCore::V8GCController::registerGlobalHandle): |
+ (WebCore::V8GCController::unregisterGlobalHandle): |
+ * bindings/v8/V8HiddenPropertyName.cpp: |
+ (WebCore::V8HiddenPropertyName::createString): |
+ * bindings/v8/V8HiddenPropertyName.h: |
+ (WebCore::V8HiddenPropertyName::V8HiddenPropertyName): |
+ * bindings/v8/V8LazyEventListener.cpp: |
+ (WebCore::V8LazyEventListener::prepareListenerObject): |
+ * bindings/v8/V8NPObject.cpp: |
+ * bindings/v8/V8Proxy.cpp: |
+ (WebCore::V8Proxy::checkNewLegal): |
+ * bindings/v8/V8Utilities.h: |
+ * bindings/v8/WorkerContextExecutionProxy.cpp: |
+ (WebCore::WorkerContextExecutionProxy::WorkerContextExecutionProxy): |
+ (WebCore::WorkerContextExecutionProxy::initIsolate): |
+ * bindings/v8/WorkerContextExecutionProxy.h: |
+ * bindings/v8/WorkerScriptController.cpp: |
+ (WebCore::WorkerScriptController::scheduleExecutionTermination): |
+ |
+2011-08-09 Steve Block <steveblock@google.com> |
+ |
+ JavaString is not needed by V8 |
+ https://bugs.webkit.org/show_bug.cgi?id=65909 |
+ |
+ Removes the JavaString wrapper class, moves the JSC implementation |
+ to JavaStringJSC.h and removes the V8 implementation. |
+ |
+ Reviewed by Alexey Proskuryakov. |
+ |
+ No new tests, refactoring only. |
+ |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * bridge/jni/JavaString.h: Removed. |
+ * bridge/jni/jni_jsobject.mm: |
+ * bridge/jni/jsc/JavaFieldJSC.h: |
+ * bridge/jni/jsc/JavaInstanceJSC.cpp: |
+ * bridge/jni/jsc/JavaMethodJSC.h: |
+ * bridge/jni/jsc/JavaStringJSC.h: |
+ (JSC::Bindings::JavaString::JavaString): |
+ (JSC::Bindings::JavaString::~JavaString): |
+ (JSC::Bindings::JavaString::init): |
+ * bridge/jni/v8/JavaStringV8.h: Removed. |
+ |
+2011-08-09 Abhishek Arya <inferno@chromium.org> |
+ |
+ Regression(83075): Fix updateAlwaysCreateLineBoxes on information |
+ about full layout which is needed when dirtying lineboxes. |
+ https://bugs.webkit.org/show_bug.cgi?id=65924 |
+ |
+ Reviewed by Dave Hyatt. |
+ |
+ Test: fast/inline/update-always-create-line-boxes-full-layout-crash.html |
+ |
+ * rendering/RenderBlockLineLayout.cpp: |
+ (WebCore::RenderBlock::layoutInlineChildren): |
+ * rendering/RenderInline.cpp: |
+ (WebCore::RenderInline::updateAlwaysCreateLineBoxes): |
+ * rendering/RenderInline.h: |
+ |
+2011-08-09 Nate Chapin <japhet@chromium.org> |
+ |
+ Remove receivedCancellation() from SubresourceLoaderClient. |
+ |
+ http://bugs.webkit.org/show_bug.cgi?id=65330 |
+ |
+ Reviewed by Alexey Proskuryakov. |
+ |
+ No new tests, no change in functionality intended. |
+ |
+ * loader/DocumentThreadableLoader.cpp: |
+ * loader/DocumentThreadableLoader.h: |
+ * loader/SubresourceLoader.cpp: |
+ * loader/SubresourceLoader.h: |
+ * loader/SubresourceLoaderClient.h: |
+ * loader/ThreadableLoaderClient.h: |
+ * loader/WorkerThreadableLoader.cpp: |
+ * loader/WorkerThreadableLoader.h: |
+ * notifications/Notification.cpp: |
+ * notifications/Notification.h: Remove receivedCancellation() since it's |
+ identical to didFail(), which is always called immediately after. |
+ * workers/WorkerScriptLoader.cpp: |
+ * workers/WorkerScriptLoader.h: Remove receivedCancellation() since it's |
+ identical to didFail(), which is always called immediately after. |
+ * xml/XMLHttpRequest.cpp: |
+ * xml/XMLHttpRequest.h: Remove receivedCancellation(). All it does is |
+ set m_response, which is cleared in didFail() before it can be used. |
+ |
+2011-08-09 Steve Block <steveblock@google.com> |
+ |
+ Java Bridge leaks local references |
+ https://bugs.webkit.org/show_bug.cgi?id=64320 |
+ |
+ Reviewed by Alexey Proskuryakov. |
+ |
+ No new tests as this is just an optimization. |
+ |
+ * bridge/jni/jsc/JavaFieldJSC.cpp: |
+ (JavaField::JavaField): |
+ * bridge/jni/jsc/JavaMethodJSC.cpp: |
+ (JavaMethod::JavaMethod): |
+ |
+2011-08-09 Pavel Feldman <pfeldman@google.com> |
+ |
+ Web Inspector: files from different domains are mixed in the file selector in Scripts panel |
+ https://bugs.webkit.org/show_bug.cgi?id=65601 |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect): |
+ (WebInspector.ScriptsPanel.prototype._folderAndDisplayNameForScriptURL): |
+ (WebInspector.ScriptsPanel.prototype.reset): |
+ * inspector/front-end/inspector.css: |
+ (select.status-bar-item): |
+ |
+2011-08-09 Pavel Feldman <pfeldman@google.com> |
+ |
+ Web Inspector: provide a way to hide user agent styles |
+ https://bugs.webkit.org/show_bug.cgi?id=37766 |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Test: inspector/styles/show-user-agent-styles.html |
+ |
+ * English.lproj/localizedStrings.js: |
+ * inspector/front-end/Settings.js: |
+ (WebInspector.Settings): |
+ * inspector/front-end/SettingsScreen.js: |
+ (WebInspector.SettingsScreen): |
+ * inspector/front-end/StylesSidebarPane.js: |
+ (WebInspector.StylesSidebarPane): |
+ (WebInspector.StylesSidebarPane.prototype._rebuildStyleRules.get continue): |
+ (WebInspector.StylesSidebarPane.prototype._rebuildStyleRules): |
+ (WebInspector.StylesSidebarPane.prototype._createElementStatePane): |
+ (WebInspector.StylesSidebarPane.prototype._showUserAgentStylesSettingChanged): |
+ |
+2011-08-09 Steve Block <steveblock@google.com> |
+ |
+ JavaMethodJobject is no longer needed by V8 |
+ https://bugs.webkit.org/show_bug.cgi?id=65910 |
+ |
+ Reviewed by Tony Gentilcore. |
+ |
+ Removes the JavaMethod interface and moves the JavaMethodJobject |
+ implementation to bridge/jni/jsc/JavaMethodJSC.[cpp|h]. |
+ |
+ No new tests, refactoring only. |
+ |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * bridge/jni/JavaMethod.h: Removed. |
+ * bridge/jni/jsc/JavaClassJSC.cpp: |
+ (JavaClass::JavaClass): |
+ * bridge/jni/jsc/JavaFieldJSC.h: |
+ * bridge/jni/jsc/JavaInstanceJSC.cpp: |
+ * bridge/jni/jsc/JavaMethodJSC.cpp: Renamed from Source/WebCore/bridge/jni/JavaMethodJobject.cpp. |
+ (JavaMethod::JavaMethod): |
+ (JavaMethod::~JavaMethod): |
+ (appendClassName): |
+ (JavaMethod::signature): |
+ * bridge/jni/jsc/JavaMethodJSC.h: Renamed from Source/WebCore/bridge/jni/JavaMethodJobject.h. |
+ (JSC::Bindings::JavaMethod::name): |
+ (JSC::Bindings::JavaMethod::returnTypeClassName): |
+ (JSC::Bindings::JavaMethod::parameterAt): |
+ (JSC::Bindings::JavaMethod::returnType): |
+ (JSC::Bindings::JavaMethod::isStatic): |
+ (JSC::Bindings::JavaMethod::numParameters): |
+ * bridge/jni/v8/JavaNPObjectV8.cpp: |
+ |
2011-08-09 Andrey Kosyakov <caseq@chromium.org> |
Web Inspector: assertion failure in InspectorDOMStorageResource::bind when opening inspector |
@@ -38,6 +1432,27 @@ |
2011-08-09 Vsevolod Vlasov <vsevik@chromium.org> |
+ Web Inspector: Add "Refresh" to context menu of resources panel cookies view. |
+ https://bugs.webkit.org/show_bug.cgi?id=65854 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * inspector/front-end/CookieItemsView.js: |
+ (WebInspector.CookieItemsView): |
+ (WebInspector.CookieItemsView.prototype._updateWithCookies): |
+ (WebInspector.CookieItemsView.prototype._refreshButtonClicked): |
+ (WebInspector.CookieItemsView.prototype._contextMenu): |
+ * inspector/front-end/CookiesTable.js: |
+ (WebInspector.CookiesTable): |
+ * inspector/front-end/DataGrid.js: |
+ (WebInspector.DataGrid.prototype.get refreshCallback): |
+ (WebInspector.DataGrid.prototype.set refreshCallback): |
+ (WebInspector.DataGrid.prototype._contextMenuInDataTable): |
+ * inspector/front-end/ResourceCookiesView.js: |
+ (WebInspector.ResourceCookiesView.prototype._buildCookiesTable): |
+ |
+2011-08-09 Vsevolod Vlasov <vsevik@chromium.org> |
+ |
Web Inspector: Resources panel does not show main resource cookies. |
https://bugs.webkit.org/show_bug.cgi?id=65770 |