Index: Source/WebCore/ChangeLog |
=================================================================== |
--- Source/WebCore/ChangeLog (revision 85485) |
+++ Source/WebCore/ChangeLog (working copy) |
@@ -1,3 +1,6530 @@ |
+2011-04-28 Steve Lacey <sjl@chromium.org> |
+ |
+ Reviewed by Eric Carlson. |
+ |
+ Change chromium media controls to use absolute positioning until bug 59644 is fixed. |
+ https://bugs.webkit.org/show_bug.cgi?id=59732 |
+ |
+ * css/mediaControlsChromium.css: |
+ (audio::-webkit-media-controls-current-time-display, video::-webkit-media-controls-current-time-display): |
+ (audio::-webkit-media-controls-timeline, video::-webkit-media-controls-timeline): |
+ |
+2011-04-28 Luke Macpherson <macpherson@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Implement CSS border width and related properties in CSSStyleApplyProperty. |
+ https://bugs.webkit.org/show_bug.cgi?id=59414 |
+ |
+ No new tests as no functionality added. |
+ |
+ * css/CSSStyleApplyProperty.cpp: |
+ Separated ApplyPropertyDefault into parent and child so that other classes could inherit |
+ the applyInheritValue and applyInitialValue methods only. |
+ This was necessary to avoid casting from CSSPrimitiveValue to inappropriate types in ApplyPropertyDefault::applyValue(). |
+ |
+ Added ApplyPropertyWidth class for handling width based CSS properties. |
+ * css/CSSStyleSelector.cpp: |
+ Removed no-longer-required code for handling width based CSS properties. |
+ * css/CSSStyleSelector.h: |
+ Added a getter for m_rootElementStyle. |
+ |
+2011-04-28 Kent Tamura <tkent@chromium.org> |
+ |
+ Run sort-Xcode-project-file. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ |
+2011-04-28 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Dirk Schulze. |
+ |
+ [Cairo] Text underline is not shadowed when text-shadow is enabled |
+ https://bugs.webkit.org/show_bug.cgi?id=48074 |
+ |
+ Use ContextShadow to enable shadows for text underlines. Also remove quite a bit |
+ of duplicate code in GraphicsContext used for stroking lines. This code looks like it was |
+ originally copied from the CG GraphicsContext. |
+ |
+ * platform/graphics/GraphicsContext.h: Make adjustLineToPixelBoundaries so that we |
+ can call it from a static helper function. |
+ * platform/graphics/cairo/GraphicsContextCairo.cpp: |
+ (WebCore::calculateStrokePatternOffset): Abstracted this code into a helper. |
+ (WebCore::drawLineOnCairoContext): Abstracted this code into a helper. We need to |
+ call it multiple times when shadowing text underlines. |
+ (WebCore::GraphicsContext::drawLine): Use drawLineOnCairoContext. |
+ (WebCore::GraphicsContext::strokeArc): Use calculateStrokePatternOffset. |
+ (WebCore::GraphicsContext::drawLineForText): Use drawLineOnCairoContext. |
+ |
+2011-04-28 Sam Weinig <sam@webkit.org> |
+ |
+ Rubber-stamped by Mark Rowe. |
+ |
+ Cleanup WebCore.xcodeproj a bit. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ Added CompilerVersion.xcconfig and moved some JS bindings files into the |
+ correct sub groups. |
+ |
+2011-04-28 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Remove IMAGE_RESIZER related code. |
+ https://bugs.webkit.org/show_bug.cgi?id=59735 |
+ |
+ * CMakeLists.txt: |
+ * Configurations/FeatureDefines.xcconfig: |
+ * GNUmakefile.am: |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * features.pri: |
+ * html/AsyncImageResizer.cpp: Removed. |
+ * html/AsyncImageResizer.h: Removed. |
+ * html/ImageResizerThread.cpp: Removed. |
+ * html/ImageResizerThread.h: Removed. |
+ |
+2011-04-28 Ilya Tikhonovsky <loislo@chromium.org> |
+ |
+ Unreviewed trivial one-line fix inspired by wrong patch for https://bugs.webkit.org/show_bug.cgi?id=59630. |
+ |
+ * inspector/front-end/Settings.js: |
+ (WebInspector.Settings): |
+ |
+2011-04-28 Beth Dakin <bdakin@apple.com> |
+ |
+ Rubber-stamped by Sam Weinig. |
+ |
+ Rolling out revision 85258! I want to fix that bug a different way. |
+ * platform/mac/ScrollbarThemeMac.mm: |
+ (WebCore::ScrollbarThemeMac::paint): |
+ |
+2011-04-28 Chang Shu <cshu@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ isContentEditable is not working properly with document.designMode |
+ https://bugs.webkit.org/show_bug.cgi?id=53031 |
+ |
+ The specification (http://dev.w3.org/html5/spec/Overview.html#attr-contenteditable) |
+ indicates that the element.isContentEditable relies on document.designMode if its |
+ and its ancestors' contenteditable attributes are set to inherit. |
+ |
+ 1. Sync document render style after document.designMode is changed. |
+ 2. Removed the code that does unnecessary and incorrect handling for contenteditable |
+ attribute with "inherit" value. |
+ 3. Ryosuke Niwa also helped to fix the code in ApplyStyleCommand that causes an assertion. |
+ |
+ Tests: fast/dom/HTMLElement/iscontenteditable-designmodeon-allinherit-subframe.html |
+ fast/dom/HTMLElement/iscontenteditable-designmodeon-allinherit.html |
+ fast/dom/HTMLElement/iscontenteditable-designmodeon-ancestor.html |
+ fast/dom/HTMLElement/iscontenteditable-designmodeon.html |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::styleForDocument): |
+ * dom/Document.cpp: |
+ (WebCore::Document::setDesignMode): |
+ * dom/Node.cpp: |
+ (WebCore::Node::rendererIsEditable): |
+ * editing/ApplyStyleCommand.cpp: |
+ (WebCore::ApplyStyleCommand::surroundNodeRangeWithElement): |
+ * html/HTMLElement.cpp: |
+ (WebCore::HTMLElement::setContentEditable): |
+ |
+2011-04-28 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r85101. |
+ http://trac.webkit.org/changeset/85101 |
+ https://bugs.webkit.org/show_bug.cgi?id=59757 |
+ |
+ it'd be better to rollout the patch and do another one with |
+ another property name. Current one will bring us to a problem |
+ at first run after upgrade because it's type was changed from |
+ int to string. (Requested by loislo on #webkit). |
+ |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._debuggerWasEnabled): |
+ (WebInspector.ScriptsPanel.prototype._setPauseOnExceptions.callback): |
+ (WebInspector.ScriptsPanel.prototype._setPauseOnExceptions): |
+ |
+2011-04-28 Mike Reed <reed@google.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ fix shadw behavior for both CSS and Canvas -- skia only |
+ https://bugs.webkit.org/show_bug.cgi?id=59700 |
+ |
+ Test: |
+ canvas/philip/tests/2d.shadow.alpha.3.html |
+ canvas/philip/tests/2d.shadow.alpha.4.html |
+ canvas/philip/tests/2d.shadow.alpha.5.html |
+ fast/canvas/canvas-fillPath-alpha-shadow.html |
+ fast/canvas/canvas-strokePath-alpha-shadow.html |
+ fast/canvas/canvas-strokeRect-alpha-shadow.html |
+ fast/css/text-input-with-webkit-border-radius.html |
+ fast/repaint/shadow-multiple-horizontal.html |
+ fast/repaint/shadow-multiple-strict-horizontal.html |
+ fast/repaint/shadow-multiple-strict-vertical.html |
+ fast/repaint/shadow-multiple-vertical.html |
+ fast/canvas/canvas-getImageData.html |
+ |
+ * platform/graphics/chromium/FontChromiumWin.cpp: |
+ (WebCore::Font::drawGlyphs): |
+ * platform/graphics/skia/GraphicsContextSkia.cpp: |
+ (WebCore::GraphicsContext::setPlatformShadow): |
+ |
+2011-04-28 David Kilzer <ddkilzer@apple.com> |
+ |
+ <http://webkit.org/b/59755> AXObjectCache::attachWrapper() over-retains an AccessibilityObjectWrapper |
+ |
+ Reviewed by Chris Fleizach. |
+ |
+ Previously, AXObjectCache::attachWrapper over-retained each |
+ AccessibilityObjectWrapper it created, which is why |
+ AXObjectCache::detachWrapper() contained an extra -release. |
+ |
+ * accessibility/mac/AXObjectCacheMac.mm: |
+ (WebCore::AXObjectCache::detachWrapper): Removed extra -release. |
+ (WebCore::AXObjectCache::attachWrapper): Because setWrapper() |
+ assigns to a RetainPtr<> instance variable, it doesn't need a +1 |
+ retained object passed in. Fix the leak by using a RetainPtr<> |
+ for the AccessibilityObjectWrapper. |
+ |
+2011-04-28 Nat Duca <nduca@chromium.org> |
+ |
+ Reviewed by Darin Fisher. |
+ |
+ [chromium] Compute compositor fps value and history using the frameBegin timestamp |
+ https://bugs.webkit.org/show_bug.cgi?id=59668 |
+ |
+ This change switches the compositor FPS meter to use the timestamp |
+ of the beginning of the frame rather than the time we call |
+ swapBuffers. When scheduling is correctly tied to vsync, we can |
+ have highly variable times when we finish vsync and still have |
+ stable frame rates. |
+ |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::updateAndDrawLayers): |
+ * platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp: |
+ (WebCore::CCHeadsUpDisplay::CCHeadsUpDisplay): |
+ (WebCore::CCHeadsUpDisplay::drawFPSCounter): |
+ (WebCore::CCHeadsUpDisplay::onFrameBegin): |
+ (WebCore::CCHeadsUpDisplay::onPresent): |
+ * platform/graphics/chromium/cc/CCHeadsUpDisplay.h: |
+ |
+2011-04-28 Beth Dakin <bdakin@apple.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=59759 |
+ Overflow:scroll regions with overlay scrollbars show full thumb instead of just a |
+ track when there is nothing to scroll to |
+ <rdar://problem/9282066> |
+ |
+ Scrollbar::totalSize() and Scrollbar::visibleSize() are the same when there is |
+ nothing to scroll to. Just dividing them to get a knob proportion yields 1, which |
+ is not right. Set the proportion to 0 in this case instead. |
+ * platform/mac/ScrollbarThemeMac.mm: |
+ (WebCore::ScrollbarThemeMac::paint): |
+ |
+2011-04-28 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ <rdar://problem/9291132> Add support for -epub-prefixed properties |
+ |
+ Test: fast/css/epub-properties.html |
+ |
+ * bindings/js/JSCSSStyleDeclarationCustom.cpp: |
+ (WebCore::cssPropertyName): Map "epub"-prefixed properties to "-epub"-prefixed ones. |
+ * css/CSSPropertyNames.in: Added -epub- aliases to caption-side, -webkit-hyphens, -webkit-text-combine, |
+ -webkit-text-emphasis, -webkit-text-emphasis-color, -webkit-text-emphasis-style, -webkit-text-orientation, |
+ text-transform, text-transform, word-break, and -webkit-writing-mode. |
+ |
+2011-04-28 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Remove WML |
+ https://bugs.webkit.org/show_bug.cgi?id=59678 |
+ |
+ This patch removes WML from WebCore. After removing WML, there's a |
+ bunch of twisted code that could be untwisted, but this patch contains |
+ only the purely mechanical part of removing the feature. |
+ |
+ There's been a lot of discussion on webkit-dev about whether we should |
+ remove WML. In addition to those threads, we've had an informal poll |
+ of the reviewers as well as an in-person discussion at the WebKit |
+ contributor's meeting. Removing any feature is likely to make some |
+ folks unhappy, but, on balance, removing WML appears to be the right |
+ thing for the project to do at this time. |
+ |
+ * CMakeLists.txt: |
+ * CodeGenerators.pri: |
+ * Configurations/FeatureDefines.xcconfig: |
+ * DerivedSources.cpp: |
+ * DerivedSources.make: |
+ * GNUmakefile.am: |
+ * GNUmakefile.list.am: |
+ * WebCore.gyp/WebCore.gyp: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::linkAttribute): |
+ (WebCore::CSSStyleSelector::styleForElement): |
+ (WebCore::CSSStyleSelector::adjustRenderStyle): |
+ * css/wml.css: Removed. |
+ * dom/DOMImplementation.cpp: |
+ (WebCore::DOMImplementation::createDocument): |
+ * dom/Document.cpp: |
+ (WebCore::Document::Document): |
+ (WebCore::Document::createElement): |
+ * dom/Document.h: |
+ (WebCore::Document::isMediaDocument): |
+ * dom/Document.idl: |
+ * dom/InputElement.h: |
+ * dom/Node.cpp: |
+ (WebCore::Node::toInputElement): |
+ * dom/Node.h: |
+ * dom/OptionElement.cpp: |
+ (WebCore::toOptionElement): |
+ (WebCore::isOptionElement): |
+ * dom/OptionElement.h: |
+ * dom/OptionGroupElement.cpp: |
+ (WebCore::toOptionGroupElement): |
+ (WebCore::isOptionGroupElement): |
+ * dom/SelectElement.cpp: |
+ (WebCore::toSelectElement): |
+ * dom/SelectElement.h: |
+ * dom/Text.cpp: |
+ (WebCore::Text::attach): |
+ * dom/XMLDocumentParser.cpp: |
+ (WebCore::XMLDocumentParser::insertErrorMessageBlock): |
+ * dom/XMLDocumentParser.h: |
+ * dom/XMLDocumentParserLibxml2.cpp: |
+ (WebCore::XMLDocumentParser::internalSubset): |
+ (WebCore::getEntityHandler): |
+ * dom/XMLDocumentParserQt.cpp: |
+ (WebCore::XMLDocumentParser::parse): |
+ (WebCore::XMLDocumentParser::parseDtd): |
+ * editing/htmlediting.cpp: |
+ (WebCore::canHaveChildrenForEditing): |
+ * features.pri: |
+ * history/BackForwardList.h: |
+ * history/BackForwardListImpl.cpp: |
+ * history/BackForwardListImpl.h: |
+ * history/PageCache.cpp: |
+ (WebCore::PageCache::canCachePageContainingThisFrame): |
+ * html/HTMLAnchorElement.h: |
+ * inspector/ConsoleMessage.cpp: |
+ (WebCore::messageSourceValue): |
+ * inspector/front-end/ConsoleView.js: |
+ (WebInspector.ConsoleMessage.prototype.toString): |
+ * loader/FrameLoader.cpp: |
+ (WebCore::isBackForwardLoadType): |
+ (WebCore::FrameLoader::transitionToCommitted): |
+ (WebCore::FrameLoader::shouldReload): |
+ (WebCore::FrameLoader::loadDifferentDocumentItem): |
+ (WebCore::FrameLoader::loadItem): |
+ * loader/FrameLoaderTypes.h: |
+ * loader/HistoryController.cpp: |
+ (WebCore::HistoryController::restoreDocumentState): |
+ * manual-tests/wml: Removed. |
+ * manual-tests/wml/StartTests.wml: Removed. |
+ * manual-tests/wml/a-br-element.wml: Removed. |
+ * manual-tests/wml/a-element.wml: Removed. |
+ * manual-tests/wml/a-img-element.wml: Removed. |
+ * manual-tests/wml/access-target.wml: Removed. |
+ * manual-tests/wml/anchor-br-element.wml: Removed. |
+ * manual-tests/wml/anchor-element.wml: Removed. |
+ * manual-tests/wml/anchor-img-element.wml: Removed. |
+ * manual-tests/wml/card-newcontext-attr.wml: Removed. |
+ * manual-tests/wml/card-onenterbackward.wml: Removed. |
+ * manual-tests/wml/card-onenterforward.wml: Removed. |
+ * manual-tests/wml/card-ontimer.wml: Removed. |
+ * manual-tests/wml/deck-access-control.wml: Removed. |
+ * manual-tests/wml/input-format.wml: Removed. |
+ * manual-tests/wml/onevent-go.wml: Removed. |
+ * manual-tests/wml/onevent-noop.wml: Removed. |
+ * manual-tests/wml/onevent-prev.wml: Removed. |
+ * manual-tests/wml/onevent-refresh.wml: Removed. |
+ * manual-tests/wml/onevent-shadow.wml: Removed. |
+ * manual-tests/wml/resources: Removed. |
+ * manual-tests/wml/resources/smiley.png: Removed. |
+ * manual-tests/wml/select-element.wml: Removed. |
+ * manual-tests/wml/select-onpick-event-crash.wml: Removed. |
+ * manual-tests/wml/select-onpick-event.wml: Removed. |
+ * manual-tests/wml/setvar-element.wml: Removed. |
+ * manual-tests/wml/targetdeck.wml: Removed. |
+ * manual-tests/wml/task-go-in-anchor.wml: Removed. |
+ * manual-tests/wml/task-prev-in-anchor.wml: Removed. |
+ * manual-tests/wml/task-refresh-in-anchor.wml: Removed. |
+ * manual-tests/wml/template-go.wml: Removed. |
+ * manual-tests/wml/template-ontimer.wml: Removed. |
+ * manual-tests/wml/timer.wml: Removed. |
+ * manual-tests/wml/variable-substitution.wml: Removed. |
+ * page/Console.cpp: |
+ (WebCore::printMessageSourceAndLevelPrefix): |
+ * page/Console.h: |
+ * page/Frame.cpp: |
+ (WebCore::Frame::Frame): |
+ * page/Page.cpp: |
+ * page/Page.h: |
+ * platform/MIMETypeRegistry.cpp: |
+ (WebCore::initializeSupportedNonImageMimeTypes): |
+ * platform/network/FormDataBuilder.h: |
+ * platform/network/mac/ResourceHandleMac.mm: |
+ (-[WebCoreResourceHandleAsDelegate connection:didReceiveResponse:]): |
+ * platform/wince/MIMETypeRegistryWinCE.cpp: |
+ (WebCore::initMIMETypeEntensionMap): |
+ * rendering/HitTestResult.cpp: |
+ (WebCore::HitTestResult::altDisplayString): |
+ (WebCore::HitTestResult::absoluteImageURL): |
+ (WebCore::HitTestResult::absoluteLinkURL): |
+ (WebCore::HitTestResult::isLiveLink): |
+ * rendering/RenderBox.cpp: |
+ * rendering/RenderButton.cpp: |
+ (WebCore::RenderButton::updateFromElement): |
+ * rendering/RenderFieldset.cpp: |
+ (WebCore::RenderFieldset::findLegend): |
+ * rendering/RenderImage.cpp: |
+ (WebCore::RenderImage::updateAltText): |
+ * rendering/RenderObject.cpp: |
+ (WebCore::RenderObject::isLegend): |
+ (WebCore::RenderObject::offsetParent): |
+ * wml: Removed. |
+ * wml/WMLAElement.cpp: Removed. |
+ * wml/WMLAElement.h: Removed. |
+ * wml/WMLAccessElement.cpp: Removed. |
+ * wml/WMLAccessElement.h: Removed. |
+ * wml/WMLAnchorElement.cpp: Removed. |
+ * wml/WMLAnchorElement.h: Removed. |
+ * wml/WMLAttributeNames.in: Removed. |
+ * wml/WMLBRElement.cpp: Removed. |
+ * wml/WMLBRElement.h: Removed. |
+ * wml/WMLCardElement.cpp: Removed. |
+ * wml/WMLCardElement.h: Removed. |
+ * wml/WMLDoElement.cpp: Removed. |
+ * wml/WMLDoElement.h: Removed. |
+ * wml/WMLDocument.cpp: Removed. |
+ * wml/WMLDocument.h: Removed. |
+ * wml/WMLElement.cpp: Removed. |
+ * wml/WMLElement.h: Removed. |
+ * wml/WMLErrorHandling.cpp: Removed. |
+ * wml/WMLErrorHandling.h: Removed. |
+ * wml/WMLEventHandlingElement.cpp: Removed. |
+ * wml/WMLEventHandlingElement.h: Removed. |
+ * wml/WMLFieldSetElement.cpp: Removed. |
+ * wml/WMLFieldSetElement.h: Removed. |
+ * wml/WMLFormControlElement.cpp: Removed. |
+ * wml/WMLFormControlElement.h: Removed. |
+ * wml/WMLGoElement.cpp: Removed. |
+ * wml/WMLGoElement.h: Removed. |
+ * wml/WMLImageElement.cpp: Removed. |
+ * wml/WMLImageElement.h: Removed. |
+ * wml/WMLImageLoader.cpp: Removed. |
+ * wml/WMLImageLoader.h: Removed. |
+ * wml/WMLInputElement.cpp: Removed. |
+ * wml/WMLInputElement.h: Removed. |
+ * wml/WMLInsertedLegendElement.cpp: Removed. |
+ * wml/WMLInsertedLegendElement.h: Removed. |
+ * wml/WMLIntrinsicEvent.cpp: Removed. |
+ * wml/WMLIntrinsicEvent.h: Removed. |
+ * wml/WMLIntrinsicEventHandler.cpp: Removed. |
+ * wml/WMLIntrinsicEventHandler.h: Removed. |
+ * wml/WMLMetaElement.cpp: Removed. |
+ * wml/WMLMetaElement.h: Removed. |
+ * wml/WMLNoopElement.cpp: Removed. |
+ * wml/WMLNoopElement.h: Removed. |
+ * wml/WMLOnEventElement.cpp: Removed. |
+ * wml/WMLOnEventElement.h: Removed. |
+ * wml/WMLOptGroupElement.cpp: Removed. |
+ * wml/WMLOptGroupElement.h: Removed. |
+ * wml/WMLOptionElement.cpp: Removed. |
+ * wml/WMLOptionElement.h: Removed. |
+ * wml/WMLPElement.cpp: Removed. |
+ * wml/WMLPElement.h: Removed. |
+ * wml/WMLPageState.cpp: Removed. |
+ * wml/WMLPageState.h: Removed. |
+ * wml/WMLPostfieldElement.cpp: Removed. |
+ * wml/WMLPostfieldElement.h: Removed. |
+ * wml/WMLPrevElement.cpp: Removed. |
+ * wml/WMLPrevElement.h: Removed. |
+ * wml/WMLRefreshElement.cpp: Removed. |
+ * wml/WMLRefreshElement.h: Removed. |
+ * wml/WMLSelectElement.cpp: Removed. |
+ * wml/WMLSelectElement.h: Removed. |
+ * wml/WMLSetvarElement.cpp: Removed. |
+ * wml/WMLSetvarElement.h: Removed. |
+ * wml/WMLTableElement.cpp: Removed. |
+ * wml/WMLTableElement.h: Removed. |
+ * wml/WMLTagNames.in: Removed. |
+ * wml/WMLTaskElement.cpp: Removed. |
+ * wml/WMLTaskElement.h: Removed. |
+ * wml/WMLTemplateElement.cpp: Removed. |
+ * wml/WMLTemplateElement.h: Removed. |
+ * wml/WMLTimerElement.cpp: Removed. |
+ * wml/WMLTimerElement.h: Removed. |
+ * wml/WMLVariables.cpp: Removed. |
+ * wml/WMLVariables.h: Removed. |
+ |
+2011-04-28 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r85233 and r85235. |
+ http://trac.webkit.org/changeset/85233 |
+ http://trac.webkit.org/changeset/85235 |
+ https://bugs.webkit.org/show_bug.cgi?id=59754 |
+ |
+ Causes issues with jsc. (Requested by dave_levin on #webkit). |
+ |
+ * ForwardingHeaders/wtf/ThreadRestrictionVerifier.h: Removed. |
+ * loader/icon/IconDatabase.cpp: |
+ (WebCore::IconDatabase::defaultIcon): |
+ (WebCore::IconDatabase::setIconDataForIconURL): |
+ (WebCore::IconDatabase::getOrCreateIconRecord): |
+ (WebCore::IconDatabase::readFromDatabase): |
+ |
+2011-04-28 Jay Civelli <jcivelli@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Introducing a new class that allows serialization of a page back |
+ to HTML/XML with all its sub-frames and also retrieves the page's |
+ resources. |
+ Tests: unit-tests in chromium/tests/WebPageNewSerializerTest.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=58947 |
+ |
+ * Android.mk: |
+ * CMakeLists.txt: |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * dom/Document.cpp: |
+ (WebCore::Document::suggestedMIMEType): |
+ * dom/Document.h: |
+ * editing/MarkupAccumulator.cpp: |
+ (WebCore::MarkupAccumulator::appendCustomAttributes): |
+ (WebCore::MarkupAccumulator::appendElement): |
+ * editing/MarkupAccumulator.h: |
+ * html/parser/HTMLMetaCharsetParser.cpp: |
+ (WebCore::HTMLMetaCharsetParser::processMeta): |
+ (WebCore::HTMLMetaCharsetParser::encodingFromMetaAttributes): |
+ * html/parser/HTMLMetaCharsetParser.h: |
+ * page/PageSerializer.cpp: Added. |
+ * page/PageSerializer.h: Added. |
+ |
+2011-04-28 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ IndexedDB createIndex should fail if name arg is null |
+ https://bugs.webkit.org/show_bug.cgi?id=58365 |
+ |
+ Test: storage/indexeddb/mozilla/create-index-null-name.html |
+ |
+ * storage/IDBObjectStore.idl: |
+ * storage/IDBObjectStoreBackendImpl.cpp: |
+ (WebCore::IDBObjectStoreBackendImpl::createIndex): |
+ |
+2011-04-28 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ IndexedDB put() should fail if second (key) parameter is null |
+ https://bugs.webkit.org/show_bug.cgi?id=58613 |
+ |
+ If key arg is not specified, prpKey ends up as a null pointer in ::put(). |
+ However, if the key arg is specified but is null, prpKey ends up as a |
+ valid IDBKey which has a null key type. As it happens, we need to be able |
+ to detect the difference between these cases (the key arg is optional but |
+ if specified must not be null). |
+ |
+ Test: storage/indexeddb/mozilla/key-requirements-put-null-key.html |
+ |
+ * storage/IDBObjectStoreBackendImpl.cpp: |
+ (WebCore::IDBObjectStoreBackendImpl::put): |
+ |
+2011-04-07 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Add asserts to RefCounted to make sure ref/deref happens on the right thread. |
+ https://bugs.webkit.org/show_bug.cgi?id=31639 |
+ |
+ No new functionality exposed so no new tests. (The change is basically adding |
+ more testing.) |
+ |
+ * ForwardingHeaders/wtf/ThreadRestrictionVerifier.h: Added. |
+ * loader/icon/IconDatabase.cpp: |
+ (WebCore::IconDatabase::defaultIcon): Set the mutex which does the guarding of the variable. |
+ (WebCore::IconDatabase::getOrCreateIconRecord): Ditto. |
+ (WebCore::IconDatabase::setIconDataForIconURL): Ditto. |
+ (WebCore::IconDatabase::readFromDatabase): Ditto. |
+ |
+2011-04-28 Kenneth Russell <kbr@google.com> |
+ |
+ Unreviewed build fix for Web Audio with strict OwnPtr. |
+ |
+ * platform/audio/HRTFDatabaseLoader.cpp: |
+ (WebCore::HRTFDatabaseLoader::HRTFDatabaseLoader): |
+ |
+2011-04-28 Dominic Cooney <dominicc@chromium.org> |
+ |
+ Reviewed by Geoffrey Garen. |
+ |
+ Walk shadow hosts as well as parent nodes when finding opaque roots. |
+ https://bugs.webkit.org/show_bug.cgi?id=59571 |
+ |
+ Test: fast/dom/shadow/gc-shadow.html |
+ |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::root): |
+ |
+2011-04-28 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Xan Lopez. |
+ |
+ [GTK] Click counting logic should be shared between WebKit1 and WebKit2 |
+ https://bugs.webkit.org/show_bug.cgi?id=59715 |
+ |
+ Added a GtkClickCounter class which is just the logic from WebKit1 pushed down |
+ into WebCore so that WebKit1 and WebKit2 can share the code. |
+ |
+ No new tests. This is covered by fast/events/click-count.html. |
+ |
+ * GNUmakefile.list.am: Added new files. |
+ * platform/gtk/GtkClickCounter.cpp: Added. |
+ (WebCore::GtkClickCounter::GtkClickCounter): Added. |
+ (WebCore::GtkClickCounter::reset): Added. |
+ (WebCore::GtkClickCounter::shouldProcessButtonEvent): Added. |
+ (WebCore::getEventTime): Added. |
+ (WebCore::GtkClickCounter::clickCountForGdkButtonEvent): Added. |
+ * platform/gtk/GtkClickCounter.h: Added. |
+ |
+2011-04-28 Dan Bernstein <mitz@apple.com> |
+ |
+ Build fix. |
+ |
+ * css/CSSParser.cpp: |
+ (WebCore::cssPropertyID): |
+ |
+2011-04-28 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ Allow specifying CSS property aliases in CSSPropertyNames.in files |
+ https://bugs.webkit.org/show_bug.cgi?id=59712 |
+ |
+ * css/CSSParser.cpp: |
+ (WebCore::cssPropertyID): Removed code to handle some aliases |
+ * css/CSSPropertyNames.in: Added aliases no longer handled in code. |
+ * css/makeprop.pl: Interpret lines of the form "alias-property-name = property-name" as |
+ defining an alias, and add an appropriate entry to the gperf file. |
+ |
+2011-04-28 Arno Renevier <arno@renevier.net> |
+ |
+ Reviewed by Gustavo Noronha Silva. |
+ |
+ [Soup] does not load css stylesheets without content-type |
+ https://bugs.webkit.org/show_bug.cgi?id=59697 |
+ |
+ When sniffing content type, do not replace http Content-Type header in |
+ soup message anymore, but keep sniffed content type in a separate |
+ variable. |
+ |
+ * platform/network/soup/ResourceHandleSoup.cpp: |
+ (WebCore::contentSniffedCallback): |
+ * platform/network/soup/ResourceResponse.h: |
+ (WebCore::ResourceResponse::sniffedContentType): |
+ (WebCore::ResourceResponse::setSniffedContentType): |
+ * platform/network/soup/ResourceResponseSoup.cpp: |
+ (WebCore::ResourceResponse::updateFromSoupMessage): |
+ |
+2011-04-28 Yael Aharon <yael.aharon@nokia.com> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ Remove flag ENABLE_SYMBIAN_DIALOG_PROVIDER |
+ https://bugs.webkit.org/show_bug.cgi?id=59704 |
+ |
+ No new tests, just code cleanup. |
+ |
+ * WebCore.pri: |
+ |
+2011-04-28 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: remove "Pretty print" source frame context menu item. |
+ https://bugs.webkit.org/show_bug.cgi?id=59709 |
+ |
+ * inspector/front-end/DebuggerPresentationModel.js: |
+ (WebInspector.DebuggerPresentationModel.prototype.setFormatSourceFiles): |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._toggleFormatSourceFiles): |
+ * inspector/front-end/SourceFrame.js: |
+ (WebInspector.SourceFrame.prototype.populateTextAreaContextMenu): |
+ |
+2011-04-28 Satish Sampath <satish@chromium.org> |
+ |
+ Reviewed by Tony Gentilcore. |
+ |
+ Speech input button layout issues with padding and border. |
+ https://bugs.webkit.org/show_bug.cgi?id=59613 |
+ |
+ * rendering/RenderTextControlSingleLine.cpp: |
+ (WebCore::RenderTextControlSingleLine::layout): |
+ (WebCore::RenderTextControlSingleLine::nodeAtPoint): |
+ (WebCore::RenderTextControlSingleLine::forwardEvent): |
+ |
+2011-04-28 Carlos Garcia Campos <cgarcia@igalia.com> |
+ |
+ Reviewed by Martin Robinson. |
+ |
+ [GTK] Check whether to use native scrollbars at run time |
+ https://bugs.webkit.org/show_bug.cgi?id=59694 |
+ |
+ It's currently decided at build time depending on whether webkit2 |
+ build is enabled or not. That makes imposible to build both |
+ webkit1 and webkit2 using the same libWebCore. |
+ |
+ * GNUmakefile.am: |
+ * platform/ScrollView.cpp: |
+ (WebCore::ScrollView::wheelEvent): |
+ * platform/gtk/ScrollViewGtk.cpp: |
+ (WebCore::ScrollView::createScrollbar): |
+ |
+2011-04-28 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: add status bar button for pretty print. |
+ https://bugs.webkit.org/show_bug.cgi?id=59467 |
+ |
+ * WebCore.gypi: |
+ * inspector/front-end/Images/prettyPrintButtonGlyph.png: Added. |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel): |
+ (WebInspector.ScriptsPanel.prototype.get statusBarItems): |
+ (WebInspector.ScriptsPanel.prototype._toggleFormatSourceFiles): |
+ * inspector/front-end/WebKit.qrc: |
+ * inspector/front-end/inspector.css: |
+ (.scripts-toggle-pretty-print-status-bar-item .glyph): |
+ (.scripts-toggle-pretty-print-status-bar-item.toggled .glyph): |
+ |
+2011-04-28 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: add ResourceTreeModel tests. |
+ https://bugs.webkit.org/show_bug.cgi?id=59616 |
+ |
+ Tests: http/tests/inspector/resource-tree/resource-tree-document-url.html |
+ http/tests/inspector/resource-tree/resource-tree-frame-navigate.html |
+ http/tests/inspector/resource-tree/resource-tree-reload.html |
+ |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.ResourceTreeModel.prototype.frontendReused): |
+ (WebInspector.ResourceTreeModel.prototype._addFrame): |
+ (WebInspector.ResourceTreeModel.prototype._frameNavigated): |
+ (WebInspector.ResourceTreeModel.prototype._frameDetached): |
+ (WebInspector.ResourceTreeModel.prototype._onResourceUpdated): |
+ (WebInspector.ResourceTreeModel.prototype.forAllResources): |
+ (WebInspector.ResourceTreeModel.prototype._clearChildFramesAndResources): |
+ (WebInspector.ResourceTreeModel.prototype._callForFrameResources): |
+ (WebInspector.ResourceTreeModel.prototype._addFramesRecursively): |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype._populateResourceTree): |
+ (WebInspector.ResourcesPanel.prototype._frameAdded): |
+ (WebInspector.ResourcesPanel.prototype._frameDetached): |
+ (WebInspector.ResourcesPanel.prototype._frameNavigated): |
+ (WebInspector.FrameTreeElement): |
+ (WebInspector.FrameTreeElement.prototype.frameNavigated): |
+ |
+2011-04-28 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ WebInspector: [Chromium] Implement retaining paths following in detailed heap snapshots. |
+ https://bugs.webkit.org/show_bug.cgi?id=59592 |
+ |
+ * inspector/front-end/DetailedHeapshotGridNodes.js: |
+ (WebInspector.HeapSnapshotGridNode.prototype._populate): |
+ (WebInspector.HeapSnapshotGridNode.prototype.populateChildren.childrenRetrieved): |
+ (WebInspector.HeapSnapshotGridNode.prototype.populateChildren.callSerialize): |
+ (WebInspector.HeapSnapshotGridNode.prototype.populateChildren): |
+ * inspector/front-end/DetailedHeapshotView.js: |
+ (WebInspector.HeapSnapshotContainmentDataGrid.prototype._defaultPopulateCount.100.expandRoute.nextStep.else.afterExpand): |
+ (WebInspector.HeapSnapshotContainmentDataGrid.prototype._defaultPopulateCount.100.expandRoute): |
+ (WebInspector.HeapSnapshotRetainingPathsList.prototype.showNext.pathFound): |
+ (WebInspector.DetailedHeapshotView.prototype._mouseClickInRetainmentGrid.expandRoute): |
+ (WebInspector.DetailedHeapshotView.prototype._mouseClickInRetainmentGrid): |
+ (WebInspector.DetailedHeapshotView.prototype.changeView.sortingComplete): |
+ (WebInspector.DetailedHeapshotView.prototype.changeView): |
+ * inspector/front-end/HeapSnapshot.js: |
+ (WebInspector.HeapSnapshotPathFinder.prototype.findNext): |
+ (WebInspector.HeapSnapshotPathFinder.prototype._nextEdgeIter): |
+ (WebInspector.HeapSnapshotPathFinder.prototype._buildNextPath): |
+ (WebInspector.HeapSnapshotPathFinder.prototype._pathToString): |
+ (WebInspector.HeapSnapshotPathFinder.prototype._pathToRoute): |
+ * inspector/front-end/heapProfiler.css: |
+ (div.retaining-paths-view td.path-column div:hover): |
+ |
+2011-04-28 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: [Chromium] Optimize detailed heap snapshots loading / parsing. |
+ https://bugs.webkit.org/show_bug.cgi?id=59454 |
+ |
+ Test: inspector/profiler/heap-snapshot-loader.html |
+ |
+ * inspector/front-end/HeapSnapshot.js: |
+ (WebInspector.HeapSnapshotLoader): |
+ (WebInspector.HeapSnapshotLoader.prototype._findBalancedCurlyBrackets): |
+ (WebInspector.HeapSnapshotLoader.prototype.finishLoading): |
+ (WebInspector.HeapSnapshotLoader.prototype._parseNodes): |
+ (WebInspector.HeapSnapshotLoader.prototype._parseStringsArray): |
+ (WebInspector.HeapSnapshotLoader.prototype.pushJSONChunk): |
+ * inspector/front-end/HeapSnapshotProxy.js: |
+ (WebInspector.HeapSnapshotFakeWorker.prototype.postMessage): |
+ |
+2011-04-28 ojab <ojab@ojab.ru> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix build with libpng-1.5.1 |
+ https://bugs.webkit.org/show_bug.cgi?id=59607 |
+ |
+ * platform/image-decoders/png/PNGImageDecoder.cpp: |
+ (WebCore::readColorProfile): |
+ |
+2011-04-28 Zoltan Herczeg <zherczeg@inf.u-szeged.hu> |
+ |
+ Reviewed by Nikolas Zimmermann. |
+ |
+ Optimizing gaussian blur filter to ARM-neon SIMD instruction set |
+ https://bugs.webkit.org/show_bug.cgi?id=59447 |
+ |
+ This patch contains two sub-routines, one for speeding up |
+ alpha channel only gaussian blur (by 2.5 times) and one |
+ for speeding up all channel blur (by 4 times). |
+ |
+ The common code is also redesigned to better fit for platform |
+ specific code. This means large chunks of code was moved to |
+ different files, and all platform specific files are added to |
+ all build systems. |
+ |
+ * CMakeLists.txt: |
+ * GNUmakefile.am: |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.vcproj/WebCoreCommon.vsprops: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * platform/graphics/filters/FEGaussianBlur.cpp: |
+ (WebCore::FEGaussianBlur::platformApplyGeneric): |
+ (WebCore::FEGaussianBlur::apply): |
+ * platform/graphics/filters/FEGaussianBlur.h: |
+ (WebCore::FEGaussianBlur::kernelPosition): |
+ (WebCore::FEGaussianBlur::platformApply): |
+ * platform/graphics/filters/FELighting.cpp: |
+ (WebCore::FELighting::platformApplyGeneric): |
+ (WebCore::FELighting::drawLighting): |
+ * platform/graphics/filters/FELighting.h: |
+ (WebCore::FELighting::platformApply): |
+ * platform/graphics/filters/arm/FEGaussianBlurNEON.cpp: Added. |
+ (WebCore::WTF_ALIGNED): |
+ (WebCore::feGaussianBlurConstantsForNeon): |
+ * platform/graphics/filters/arm/FEGaussianBlurNEON.h: Added. |
+ (WebCore::FEGaussianBlur::platformApplyNeon): |
+ * platform/graphics/filters/arm/FELightingNEON.cpp: |
+ (WebCore::WTF_ALIGNED): |
+ (WebCore::FELighting::getPowerCoefficients): |
+ * platform/graphics/filters/arm/FELightingNEON.h: |
+ (WebCore::FELighting::platformApplyNeon): |
+ |
+2011-04-28 Alexander Pavlov <apavlov@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: provide a hotkey for script pause |
+ https://bugs.webkit.org/show_bug.cgi?id=59593 |
+ |
+ In fact, F8 works as a hotkey, but this fact was not reflected |
+ in the shortcut help popup. |
+ |
+ * English.lproj/localizedStrings.js: |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._registerShortcuts): |
+ |
+2011-04-28 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ PluginStream should play nice with strict OwnPtr |
+ https://bugs.webkit.org/show_bug.cgi?id=59675 |
+ |
+ These failures appear when enabling strict OwnPtr on Qt. |
+ |
+ * plugins/PluginStream.cpp: |
+ (WebCore::PluginStream::PluginStream): |
+ (WebCore::PluginStream::didReceiveData): |
+ |
+2011-04-26 Alexander Pavlov <apavlov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: Use CachedResource to retrieve charset-decoded stylesheet text |
+ https://bugs.webkit.org/show_bug.cgi?id=59326 |
+ |
+ Concrete CachedResource successors for stylesheets and scripts |
+ can decode their content better than the generic approach we use. |
+ |
+ * inspector/InspectorPageAgent.cpp: |
+ (WebCore::decodeSharedBuffer): |
+ (WebCore::prepareCachedResourceBuffer): |
+ (WebCore::cachedResourceDecoded): |
+ (WebCore::InspectorPageAgent::resourceContent): |
+ (WebCore::InspectorPageAgent::resourceData): |
+ * inspector/InspectorStyleSheet.cpp: |
+ (WebCore::InspectorStyleSheet::originalStyleSheetText): |
+ |
+2011-04-28 Jon Lee <jonlee@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ REGRESSION: white overlay scrollbars on apple.com/startpage |
+ https://bugs.webkit.org/show_bug.cgi?id=59540 |
+ <rdar://problem/9338653> |
+ |
+ Now we look at the document background in addition to the <body> element, |
+ and blend those colors in with the base background of the frame view to |
+ arrive at our aggregate color. This provides a better result to determine |
+ overlay scrollbar style. |
+ |
+ * page/Frame.cpp: |
+ (WebCore::Frame::getDocumentBackgroundColor): look up the colors on the html and body element, and properly composite them. |
+ * platform/graphics/Color.h: a short comment to note that blend() uses the Porter-Duff source-over equation |
+ |
+2011-04-27 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ Fix OwnPtr issues in IndexedDB |
+ https://bugs.webkit.org/show_bug.cgi?id=59656 |
+ |
+ This patch is an attempt to fix the clang build. Clang can't seem to |
+ cope with OwnPtrs to classes declared in an anonymous namespace because |
+ OwnPtr's copy constructor isn't defined (but would need to be defined |
+ in this translation unit). |
+ |
+ * platform/leveldb/LevelDBDatabase.cpp: |
+ |
+2011-04-27 Ryuan Choi <ryuan.choi@samsung.com> |
+ |
+ Reviewed by Antonio Gomes. |
+ |
+ [EFL] Change cursor to LAZY_NATIVE_CURSOR |
+ https://bugs.webkit.org/show_bug.cgi?id=59411 |
+ |
+ Enable LAZY_NATIVE_CURSOR on EFL build. |
+ |
+ No test added because functionality is unchanged. |
+ |
+ * CMakeListsEfl.txt: Add Cursor.cpp |
+ * platform/Cursor.h: |
+ (WebCore::Cursor::Cursor): |
+ * platform/efl/CursorEfl.cpp: |
+ (WebCore::Cursor::Cursor): |
+ (WebCore::Cursor::~Cursor): |
+ (WebCore::getCursorString): |
+ (WebCore::Cursor::ensurePlatformCursor): |
+ * platform/efl/WidgetEfl.cpp: |
+ (WebCore::Widget::setCursor): |
+ |
+2011-04-27 Adam Barth <abarth@webkit.org> |
+ |
+ Build fix from the future. Another trival strict OwnPtr fixes. This |
+ is the last futuristic build fix I can detect with my time traveling |
+ abilities. |
+ |
+ * platform/graphics/chromium/TransparencyWin.cpp: |
+ (WebCore::TransparencyWin::initializeNewContext): |
+ |
+2011-04-27 Dmitry Lomov <dslomov@google.com> |
+ |
+ Reviewed by David Levin. |
+ |
+ CrossThreadCopier should not have a default specialization for raw pointers |
+ https://bugs.webkit.org/show_bug.cgi?id=59234 |
+ Removed the ablity to pass raw pointers cross-thread |
+ Added and applied annotations for doing that |
+ |
+ * fileapi/FileReader.cpp: |
+ (WebCore::FileReader::readInternal): |
+ (WebCore::FileReader::abort): |
+ * fileapi/FileStreamProxy.cpp: |
+ (WebCore::FileStreamProxy::startOnFileThread): |
+ (WebCore::FileStreamProxy::stopOnFileThread): |
+ (WebCore::FileStreamProxy::getSize): |
+ (WebCore::FileStreamProxy::getSizeOnFileThread): |
+ (WebCore::FileStreamProxy::openForRead): |
+ (WebCore::FileStreamProxy::openForReadOnFileThread): |
+ (WebCore::FileStreamProxy::openForWrite): |
+ (WebCore::FileStreamProxy::openForWriteOnFileThread): |
+ (WebCore::FileStreamProxy::close): |
+ (WebCore::FileStreamProxy::read): |
+ (WebCore::FileStreamProxy::readOnFileThread): |
+ (WebCore::FileStreamProxy::write): |
+ (WebCore::FileStreamProxy::writeOnFileThread): |
+ (WebCore::FileStreamProxy::truncate): |
+ (WebCore::FileStreamProxy::truncateOnFileThread): |
+ * loader/WorkerThreadableLoader.cpp: |
+ (WebCore::WorkerThreadableLoader::MainThreadBridge::MainThreadBridge): |
+ (WebCore::WorkerThreadableLoader::MainThreadBridge::destroy): |
+ (WebCore::WorkerThreadableLoader::MainThreadBridge::cancel): |
+ * page/GeolocationPositionCache.cpp: |
+ (WebCore::GeolocationPositionCache::triggerReadFromDatabase): |
+ (WebCore::GeolocationPositionCache::triggerWriteToDatabase): |
+ * platform/CrossThreadCopier.h: |
+ (WebCore::AllowCrossThreadAccessWrapper::AllowCrossThreadAccessWrapper): |
+ (WebCore::AllowCrossThreadAccessWrapper::value): |
+ (WebCore::AllowCrossThreadAccess): |
+ (WebCore::AllowExtendedLifetimeWrapper::AllowExtendedLifetimeWrapper): |
+ (WebCore::AllowExtendedLifetimeWrapper::value): |
+ (WebCore::AllowExtendedLifetime): |
+ * platform/graphics/chromium/cc/CCCompletionEvent.h: |
+ * storage/IDBObjectStoreBackendImpl.cpp: |
+ (WebCore::IDBObjectStoreBackendImpl::get): |
+ (WebCore::IDBObjectStoreBackendImpl::put): |
+ (WebCore::IDBObjectStoreBackendImpl::deleteFunction): |
+ (WebCore::IDBObjectStoreBackendImpl::clear): |
+ (WebCore::IDBObjectStoreBackendImpl::createIndex): |
+ (WebCore::IDBObjectStoreBackendImpl::deleteIndex): |
+ (WebCore::IDBObjectStoreBackendImpl::openCursor): |
+ * storage/SQLCallbackWrapper.h: |
+ (WebCore::SQLCallbackWrapper::clear): |
+ * websockets/WorkerThreadableWebSocketChannel.cpp: |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::mainThreadCreateWebSocketChannel): |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::Bridge): |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::connect): |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::send): |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::bufferedAmount): |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::close): |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::disconnect): |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::suspend): |
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::resume): |
+ * workers/WorkerMessagingProxy.cpp: |
+ (WebCore::WorkerMessagingProxy::postConsoleMessageToWorkerObject): |
+ |
+2011-04-27 Adam Barth <abarth@webkit.org> |
+ |
+ Two build fixes from the future. (Trivial strict OwnPtr fixes.) |
+ |
+ * page/Frame.cpp: |
+ (WebCore::Frame::setTiledBackingStoreEnabled): |
+ * platform/audio/ReverbConvolver.cpp: |
+ (WebCore::ReverbConvolver::ReverbConvolver): |
+ |
+2011-04-27 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix five strict PassOwnPtr violations in WebCore |
+ https://bugs.webkit.org/show_bug.cgi?id=59640 |
+ |
+ * css/MediaList.cpp: |
+ (WebCore::MediaList::deleteMedium): |
+ (WebCore::MediaList::setMediaText): |
+ (WebCore::MediaList::appendMedium): |
+ * css/MediaQuery.h: |
+ * dom/MessagePort.cpp: |
+ (WebCore::MessagePort::disentanglePorts): |
+ (WebCore::MessagePort::entanglePorts): |
+ * inspector/InspectorStyleSheet.cpp: |
+ (ParsedStyleSheet::setText): |
+ * rendering/RenderTheme.cpp: |
+ (WebCore::RenderTheme::adjustStyle): |
+ (WebCore::RenderTheme::adjustMeterStyle): |
+ * rendering/RenderThemeMac.mm: |
+ (WebCore::RenderThemeMac::adjustMenuListStyle): |
+ (WebCore::RenderThemeMac::adjustSliderTrackStyle): |
+ (WebCore::RenderThemeMac::adjustSliderThumbStyle): |
+ (WebCore::RenderThemeMac::adjustSearchFieldStyle): |
+ (WebCore::RenderThemeMac::adjustSearchFieldCancelButtonStyle): |
+ (WebCore::RenderThemeMac::adjustSearchFieldDecorationStyle): |
+ (WebCore::RenderThemeMac::adjustSearchFieldResultsDecorationStyle): |
+ (WebCore::RenderThemeMac::adjustSearchFieldResultsButtonStyle): |
+ |
+2011-04-27 Chris Fleizach <cfleizach@apple.com> |
+ |
+ Reviewed by Beth Dakin. |
+ |
+ <rdar://problem/9315254> ARIA role attribute implemented incorrectly; does not support token list with fallbacks |
+ https://bugs.webkit.org/show_bug.cgi?id=59648 |
+ |
+ Test: accessibility/aria-fallback-roles.html |
+ |
+ * accessibility/AccessibilityObject.cpp: |
+ (WebCore::AccessibilityObject::ariaRoleToWebCoreRole): |
+ |
+2011-04-27 Beth Dakin <bdakin@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=59671 |
+ Race condition with scrollbar animations and closing a page can cause a crash in |
+ WebCore::FrameView::setVisibleScrollerThumbRect + 15 |
+ -and corresponding- |
+ <rdar://problem/9329253> |
+ |
+ It is definitely possible for Page to be null here. So we should null-check it! |
+ And we should not ASSERT. |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::didCompleteRubberBand): |
+ (WebCore::FrameView::scrollbarStyleChanged): |
+ (WebCore::FrameView::setVisibleScrollerThumbRect): |
+ |
+2011-04-27 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ WinCE build fix after r85143. |
+ |
+ * platform/graphics/wince/FontWinCE.cpp: |
+ (WebCore::TextRunComponent::TextRunComponent): |
+ |
+2011-04-27 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ IndexedDB object store delete should fail if key is null |
+ https://bugs.webkit.org/show_bug.cgi?id=58614 |
+ |
+ Test: storage/indexeddb/mozilla/key-requirements-delete-null-key.html |
+ |
+ * storage/IDBObjectStoreBackendImpl.cpp: |
+ (WebCore::IDBObjectStoreBackendImpl::deleteFunction): |
+ |
+2011-04-27 Eric Seidel <eric@webkit.org> |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ Should have an easy way to construct starting BidiStatus for a paragraph root |
+ https://bugs.webkit.org/show_bug.cgi?id=59226 |
+ |
+ Two places try to construct BidiStatuses for a paragraph root using copy/paste code. |
+ I've made this a constructor for BidiStatus instead. |
+ |
+ As part of this effort I added a direction() accessor for TextRun and got |
+ rid of the old m_rtl bool. This is part of the generic effort in the |
+ Bidi code to replace old bool usage with the superior TextDirection enum |
+ (this generally makes the code cleaner). |
+ |
+ As part of this replacement effort I found several places which were |
+ assuming LTR (by passing rtl=false) when they probably want to use |
+ the current text direction. I suspect that LTR vs. RTL may affect |
+ string width in the case of ligatures. It's unclear. |
+ |
+ This is almost entirely a mechanical change. |
+ |
+ * html/canvas/CanvasRenderingContext2D.cpp: |
+ (WebCore::CanvasRenderingContext2D::drawTextInternal): |
+ * platform/graphics/GraphicsContext.cpp: |
+ (WebCore::GraphicsContext::drawBidiText): |
+ * platform/graphics/TextRun.h: |
+ (WebCore::TextRun::TextRun): |
+ (WebCore::TextRun::direction): |
+ (WebCore::TextRun::rtl): |
+ (WebCore::TextRun::ltr): |
+ (WebCore::TextRun::setDirection): |
+ * platform/text/BidiResolver.h: |
+ (WebCore::BidiStatus::BidiStatus): |
+ * rendering/EllipsisBox.cpp: |
+ (WebCore::EllipsisBox::paint): |
+ (WebCore::EllipsisBox::selectionRect): |
+ (WebCore::EllipsisBox::paintSelection): |
+ * rendering/InlineTextBox.cpp: |
+ (WebCore::InlineTextBox::selectionRect): |
+ (WebCore::InlineTextBox::paint): |
+ (WebCore::InlineTextBox::paintSelection): |
+ (WebCore::InlineTextBox::paintCompositionBackground): |
+ (WebCore::InlineTextBox::paintSpellingOrGrammarMarker): |
+ (WebCore::InlineTextBox::paintTextMatchMarker): |
+ (WebCore::InlineTextBox::computeRectForReplacementMarker): |
+ (WebCore::InlineTextBox::offsetForPosition): |
+ (WebCore::InlineTextBox::positionForOffset): |
+ * rendering/RenderBlockLineLayout.cpp: |
+ (WebCore::RenderBlock::determineStartPosition): |
+ * rendering/RenderFileUploadControl.cpp: |
+ (WebCore::RenderFileUploadControl::paintObject): |
+ (WebCore::RenderFileUploadControl::computePreferredLogicalWidths): |
+ * rendering/RenderListBox.cpp: |
+ (WebCore::RenderListBox::updateFromElement): |
+ (WebCore::RenderListBox::paintItemForeground): |
+ * rendering/RenderTextControl.cpp: |
+ (WebCore::RenderTextControl::getAvgCharWidth): |
+ (WebCore::RenderTextControl::paintPlaceholder): |
+ * rendering/svg/SVGInlineTextBox.cpp: |
+ (WebCore::SVGInlineTextBox::constructTextRun): |
+ * rendering/svg/SVGTextMetrics.cpp: |
+ (WebCore::constructTextRun): |
+ |
+2011-04-27 Robert Hogan <robert@webkit.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Allow shadowing of history object |
+ https://bugs.webkit.org/show_bug.cgi?id=55965 |
+ |
+ Tests: http/tests/history/cross-origin-replace-history-object-child.html |
+ http/tests/history/cross-origin-replace-history-object.html |
+ |
+ * page/DOMWindow.idl: |
+ |
+2011-04-27 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [chromium] Ensure compositing layers are up to date before entering doComposite |
+ https://bugs.webkit.org/show_bug.cgi?id=59159 |
+ |
+ Adds some assertions to try to catch GraphicsLayer mutations at bad times (such as during |
+ GraphicsLayerClient::paintContents() implementations). |
+ |
+ * platform/graphics/chromium/GraphicsLayerChromium.cpp: |
+ (WebCore::GraphicsLayerChromium::updateLayerPreserves3D): |
+ * platform/graphics/chromium/LayerChromium.cpp: |
+ (WebCore::LayerChromium::LayerChromium): |
+ (WebCore::LayerChromium::~LayerChromium): |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::updateLayers): |
+ * platform/graphics/chromium/LayerRendererChromium.h: |
+ Add a set of assertions that we aren't creating or destroying LayerChromiums during paintContents |
+ |
+2011-04-27 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix OwnPtr strict issues in chromium linux build |
+ https://bugs.webkit.org/show_bug.cgi?id=59664 |
+ |
+ * platform/graphics/chromium/ComplexTextControllerLinux.cpp: |
+ (WebCore::ComplexTextController::getNormalizedTextRun): |
+ * platform/graphics/skia/ImageBufferSkia.cpp: |
+ (WebCore::ImageBuffer::ImageBuffer): |
+ * platform/graphics/skia/PlatformContextSkia.cpp: |
+ (WebCore::PlatformContextSkia::PlatformContextSkia): |
+ |
+2011-04-27 Chris Rogers <crogers@google.com> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ Make sure to set sample-rate of created AudioBus in AudioBus::createBySampleRateConverting() |
+ https://bugs.webkit.org/show_bug.cgi?id=59641 |
+ |
+ No new tests since audio API is not yet implemented. |
+ |
+ * platform/audio/AudioBus.cpp: |
+ (WebCore::AudioBus::createBySampleRateConverting): |
+ (WebCore::AudioBus::createByMixingToMono): |
+ |
+2011-04-27 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Motivated by Alexey Proskuryakov. |
+ |
+ * bindings/scripts/CodeGeneratorJS.pm: Tried to make this comment more |
+ informative, since it confused Alexey and me. |
+ |
+2011-04-27 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Fixed a small leak related to CSSValues when deallocating a DOMWrapperWorld |
+ https://bugs.webkit.org/show_bug.cgi?id=59646 |
+ |
+ Made the cssValueRoots map per-world, instead of a global. When we deallocate |
+ a world, we need to clear all entries in the map for that world, since |
+ the finalizers that would otherwise clear those entries don't run. The |
+ simplest way to do this is just to make the world own the map. |
+ |
+ * bindings/js/DOMWrapperWorld.h: |
+ * bindings/js/JSCSSStyleDeclarationCustom.cpp: |
+ (WebCore::JSCSSStyleDeclaration::getPropertyCSSValue): |
+ * bindings/js/JSCSSValueCustom.cpp: |
+ (WebCore::JSCSSValueOwner::isReachableFromOpaqueRoots): |
+ (WebCore::JSCSSValueOwner::finalize): |
+ * bindings/js/JSDOMBinding.h: |
+ |
+2011-04-27 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix OwnPtr issues in IndexedDB |
+ https://bugs.webkit.org/show_bug.cgi?id=59656 |
+ |
+ I didn't do an exhaustive review of this code, but I fixed the problems |
+ caught by turning on strict OwnPtr and all their antecedents. This |
+ patch is entirely tighter bookkeeping. There shouldn't be any actual |
+ behavior change. |
+ |
+ * platform/leveldb/LevelDBDatabase.cpp: |
+ (WebCore::LevelDBDatabase::LevelDBDatabase): |
+ (WebCore::LevelDBDatabase::open): |
+ (WebCore::LevelDBDatabase::createIterator): |
+ * platform/leveldb/LevelDBDatabase.h: |
+ * platform/leveldb/LevelDBIterator.cpp: |
+ (WebCore::LevelDBIterator::LevelDBIterator): |
+ * platform/leveldb/LevelDBIterator.h: |
+ * storage/IDBLevelDBBackingStore.cpp: |
+ (WebCore::IDBLevelDBBackingStore::IDBLevelDBBackingStore): |
+ (WebCore::IDBLevelDBBackingStore::open): |
+ (WebCore::getNewDatabaseId): |
+ (WebCore::IDBLevelDBBackingStore::getObjectStores): |
+ (WebCore::getNewObjectStoreId): |
+ (WebCore::deleteRange): |
+ (WebCore::IDBLevelDBBackingStore::nextAutoIncrementNumber): |
+ (WebCore::IDBLevelDBBackingStore::forEachObjectStoreRecord): |
+ (WebCore::IDBLevelDBBackingStore::getIndexes): |
+ (WebCore::getNewIndexId): |
+ (WebCore::findGreatestKeyLessThan): |
+ (WebCore::IDBLevelDBBackingStore::getPrimaryKeyViaIndex): |
+ (WebCore::IDBLevelDBBackingStore::keyExistsInIndex): |
+ (WebCore::findLastIndexKeyEqualTo): |
+ * storage/IDBLevelDBBackingStore.h: |
+ |
+2011-04-19 MORITA Hajime <morrita@google.com> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ [Refactoring] DocumentMarkerController::MarkerMapVectorPair should be replaced with a list of some class. |
+ https://bugs.webkit.org/show_bug.cgi?id=58113 |
+ |
+ * Introduced RenderedDocumentMarker, a subclass of DocumentMarker. |
+ * Eliminated Vector<IntRect> and move the IntRect into RenderedDocumentMarker |
+ * Now MarkerMapVectorPair is no longer used. |
+ |
+ No new tests, No behavior change. |
+ |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * dom/DocumentMarker.h: |
+ * dom/DocumentMarkerController.cpp: |
+ (WebCore::DocumentMarkerController::addMarker): |
+ (WebCore::DocumentMarkerController::copyMarkers): |
+ (WebCore::DocumentMarkerController::removeMarkers): |
+ (WebCore::DocumentMarkerController::markerContainingPoint): |
+ (WebCore::DocumentMarkerController::markersForNode): |
+ (WebCore::DocumentMarkerController::renderedRectsForMarkers): |
+ (WebCore::DocumentMarkerController::removeMarkersFromList): |
+ (WebCore::DocumentMarkerController::repaintMarkers): |
+ (WebCore::DocumentMarkerController::setRenderedRectForMarker): |
+ (WebCore::DocumentMarkerController::invalidateRenderedRectsForMarkersInRect): |
+ (WebCore::DocumentMarkerController::shiftMarkers): |
+ (WebCore::DocumentMarkerController::setMarkersActive): |
+ (WebCore::DocumentMarkerController::clearDescriptionOnMarkersIntersectingRange): |
+ (WebCore::DocumentMarkerController::showMarkers): |
+ * dom/DocumentMarkerController.h: |
+ * dom/RenderedDocumentMarker.h: Added. |
+ (WebCore::RenderedDocumentMarker::RenderedDocumentMarker): |
+ (WebCore::RenderedDocumentMarker::isRendered): |
+ (WebCore::RenderedDocumentMarker::contains): |
+ (WebCore::RenderedDocumentMarker::setRenderedRect): |
+ (WebCore::RenderedDocumentMarker::renderedRect): |
+ (WebCore::RenderedDocumentMarker::invalidate): |
+ (WebCore::RenderedDocumentMarker::invalidMarkerRect): |
+ |
+2011-04-27 James Robinson <jamesr@chromium.org> |
+ |
+ Unreviewed, rolling out r85112. |
+ http://trac.webkit.org/changeset/85112 |
+ https://bugs.webkit.org/show_bug.cgi?id=59159 |
+ |
+ Broke mac compile |
+ |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::updateCompositingLayers): |
+ * page/FrameView.h: |
+ * platform/graphics/chromium/GraphicsLayerChromium.cpp: |
+ (WebCore::GraphicsLayerChromium::updateLayerPreserves3D): |
+ * platform/graphics/chromium/LayerChromium.cpp: |
+ (WebCore::LayerChromium::LayerChromium): |
+ (WebCore::LayerChromium::~LayerChromium): |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::updateLayers): |
+ * platform/graphics/chromium/LayerRendererChromium.h: |
+ |
+2011-04-27 Mark Rowe <mrowe@apple.com> |
+ |
+ Build fix. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: Make RenderLayerCompositor.h accessbile to WebKit and WebKit2. |
+ |
+2011-04-27 Pratik Solanki <psolanki@apple.com> |
+ |
+ Unreviewed. Fix WinCE build. The include should be guarded inside USE(CFNETWORK). |
+ |
+ * platform/network/ResourceHandleClient.h: |
+ |
+2011-04-27 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [chromium] Ensure compositing layers are up to date before entering doComposite |
+ https://bugs.webkit.org/show_bug.cgi?id=59159 |
+ |
+ Adds some assertions to try to catch GraphicsLayer mutations at bad times (such as during |
+ GraphicsLayerClient::paintContents() implementations). |
+ |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::updateCompositingLayers): |
+ Add an optional CompositingUpdateType parameter. |
+ * page/FrameView.h: |
+ * platform/graphics/chromium/GraphicsLayerChromium.cpp: |
+ (WebCore::GraphicsLayerChromium::updateLayerPreserves3D): |
+ * platform/graphics/chromium/LayerChromium.cpp: |
+ (WebCore::LayerChromium::LayerChromium): |
+ (WebCore::LayerChromium::~LayerChromium): |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::updateLayers): |
+ * platform/graphics/chromium/LayerRendererChromium.h: |
+ Add a set of assertions that we aren't creating or destroying LayerChromiums during paintContents |
+ * rendering/RenderLayer.cpp: |
+ (WebCore::RenderLayer::dirtyZOrderLists): |
+ (WebCore::RenderLayer::updateCompositingAndLayerListsIfNeeded): |
+ * rendering/RenderLayerCompositor.cpp: |
+ (WebCore::RenderLayerCompositor::updateCompositingLayers): |
+ * rendering/RenderLayerCompositor.h: |
+ Fix a typo in CompositingUpdateType Pait->Paint |
+ |
+2011-04-27 Pratik Solanki <psolanki@apple.com> |
+ |
+ Reviewed by Antti Koivisto. |
+ |
+ Part of WebCore should use CFNetwork-based loader on Mac |
+ https://bugs.webkit.org/show_bug.cgi?id=51836 |
+ |
+ Add willCacheResponse method when using CFNetwork on Mac. Also put appropriate ifdefs around |
+ willCacheResponse/shouldCacheResponse. |
+ |
+ * loader/EmptyClients.h: |
+ * loader/FrameLoaderClient.h: |
+ * loader/ResourceLoader.h: |
+ * loader/cf/ResourceLoaderCFNet.cpp: |
+ * loader/mac/ResourceLoaderMac.mm: |
+ (WebCore::ResourceLoader::willCacheResponse): |
+ * platform/network/ResourceHandleClient.h: |
+ (WebCore::ResourceHandleClient::willCacheResponse): |
+ (WebCore::ResourceHandleClient::shouldCacheResponse): |
+ * platform/network/cf/ResourceHandleCFNet.cpp: |
+ (WebCore::willCacheResponse): |
+ |
+2011-04-27 Ilya Tikhonovsky <loislo@chromium.org> |
+ |
+ Reviewed by Brian Weinstein. |
+ |
+ Web Inspector: Stop on Exception state does not persist. |
+ |
+ A wrong property name was used when we persist PauseOnExceptionState property to Local Storage. |
+ https://bugs.webkit.org/show_bug.cgi?id=59630 |
+ |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._debuggerWasEnabled): |
+ (WebInspector.ScriptsPanel.prototype._setPauseOnExceptions.callback): |
+ (WebInspector.ScriptsPanel.prototype._setPauseOnExceptions): |
+ |
+2011-04-27 Adrienne Walker <enne@google.com> |
+ |
+ [chromium] Unreviewed, rollout r85075 (scissor rect changes) |
+ https://bugs.webkit.org/show_bug.cgi?id=59020 |
+ |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::updatePropertiesAndRenderSurfaces): |
+ |
+2011-04-27 Darin Adler <darin@apple.com> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix strict OwnPtr issues seen in about 30 more files |
+ https://bugs.webkit.org/show_bug.cgi?id=59615 |
+ |
+ * bindings/js/JSCallbackData.h: |
+ (WebCore::DeleteCallbackDataTask::create): Use adoptPtr. |
+ * bindings/js/JSMessageEventCustom.cpp: |
+ (WebCore::JSMessageEvent::initMessageEvent): Use adoptPtr. |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::updateSpecifiersWithElementName): Use OwnPtr, |
+ adoptPtr, and release. |
+ * dom/MessageEvent.cpp: |
+ (WebCore::MessageEvent::initMessageEvent): Use OwnPtr, adoptPtr, |
+ and release. |
+ * dom/MessagePortChannel.cpp: |
+ (WebCore::MessagePortChannel::EventData::create): Use adoptPtr. |
+ * dom/ScriptExecutionContext.cpp: |
+ (WebCore::ProcessMessagesSoonTask::create): Use adoptPtr. |
+ * dom/XMLDocumentParserLibxml2.cpp: |
+ (WebCore::XMLDocumentParser::doEnd): Use adoptPtr. |
+ * dom/default/PlatformMessagePortChannel.cpp: |
+ (WebCore::MessagePortChannel::create): Use adoptPtr. |
+ * html/HTMLFormControlElement.cpp: |
+ (WebCore::HTMLFormControlElement::detach): Use nullptr. |
+ * inspector/InspectorAgent.cpp: |
+ (WebCore::PostWorkerNotificationToFrontendTask::create): Use adoptPtr. |
+ * inspector/InspectorConsoleAgent.cpp: |
+ (WebCore::InspectorConsoleAgent::addMessageToConsole): Use adoptPtr. |
+ (WebCore::InspectorConsoleAgent::didReceiveResponse): Use adoptPtr. |
+ (WebCore::InspectorConsoleAgent::didFailLoading): Use adoptPtr. |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::connectFrontend): Use adoptPtr. |
+ * inspector/InspectorDOMAgent.cpp: |
+ (WebCore::InspectorDOMAgent::didInvalidateStyleAttr): Use adoptPtr. |
+ * inspector/InspectorResourceAgent.cpp: |
+ (WebCore::InspectorResourceAgent::InspectorResourceAgent): Use adoptPtr. |
+ * page/FrameActionScheduler.cpp: |
+ (WebCore::FrameActionScheduler::scheduleEvent): Use adoptPtr. |
+ * platform/graphics/ShadowBlur.cpp: |
+ (WebCore::ScratchBuffer::clearScratchBuffer): Use nullptr. |
+ * platform/graphics/ca/GraphicsLayerCA.cpp: |
+ (WebCore::GraphicsLayer::create): Use adoptPtr. |
+ (WebCore::GraphicsLayerCA::ensureCloneLayers): Use adoptPtr. |
+ (WebCore::GraphicsLayerCA::removeCloneLayers): Use nullptr. |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::setMaxMarginBeforeValues): Use adoptPtr. |
+ (WebCore::RenderBlock::setMaxMarginAfterValues): Use adoptPtr. |
+ (WebCore::RenderBlock::setPaginationStrut): Use adoptPtr. |
+ (WebCore::RenderBlock::setPageLogicalOffset): Use adoptPtr. |
+ * rendering/RenderLayerBacking.cpp: |
+ (WebCore::RenderLayerBacking::destroyGraphicsLayer): Use nullptr. |
+ (WebCore::RenderLayerBacking::updateClippingLayers): Use nullptr. |
+ (WebCore::RenderLayerBacking::updateForegroundLayer): Use nullptr. |
+ (WebCore::RenderLayerBacking::updateMaskLayer): Use nullptr. |
+ * rendering/RenderLayerCompositor.cpp: |
+ (WebCore::RenderLayerCompositor::updateOverflowControlsLayers): Use nullptr. |
+ (WebCore::RenderLayerCompositor::ensureRootPlatformLayer): Use nullptr. |
+ (WebCore::RenderLayerCompositor::destroyRootPlatformLayer): Use nullptr. |
+ * rendering/style/StyleRareNonInheritedData.cpp: |
+ (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): Removed |
+ explicit initializations of OwnPtr members to 0. Use adoptPtr. |
+ * workers/SharedWorkerContext.cpp: |
+ (WebCore::createConnectEvent): Use adoptPtr. |
+ * workers/Worker.cpp: |
+ (WebCore::Worker::notifyFinished): Use nullptr. |
+ |
+2011-04-27 Enrica Casucci <enrica@apple.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ REGRESSION (r84311): Copy should preserve background color if specified in the body only if the entire content is selected. |
+ https://bugs.webkit.org/show_bug.cgi?id=59251 |
+ <rdar://problem/9327044> |
+ |
+ When we are looking for wrapping elements that are presentational, we should |
+ include elements that have a non transparent background color only if they |
+ are not blocks. |
+ |
+ Test: editing/pasteboard/do-not-copy-body-color.html |
+ |
+ * editing/markup.cpp: |
+ (WebCore::isElementPresentational): |
+ |
+2011-04-20 Dimitri Glazkov <dglazkov@chromium.org> |
+ |
+ Reviewed by Antti Koivisto. |
+ |
+ Teach sub-selector chains about shadow descendants |
+ https://bugs.webkit.org/show_bug.cgi?id=58342 |
+ |
+ The primary change is to the logic of parsing specifiers: |
+ 1) The shadow descendant selectors (those specifiers that are unknown |
+ pseudo element selectors) are always kept at the top of the chain. |
+ 2) The sub-selectors after shadow descendant selectors are stashed right |
+ behind the sub-selector, but not at the end of the chain. |
+ 3) Other sub-selectors are appended at the end of the chain. |
+ |
+ * css/CSSGrammar.y: Changed specifier_list collection to use new |
+ CSSParser::updateSpecifier helper. |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::updateSpecifiersWithElementName): Added logic to |
+ look for the last ShadowDescendant relation in the chain of selectors, |
+ because the next selector after it is the one that should get the |
+ element name. |
+ (WebCore::CSSParser::updateSpecifiers): Moved and modified the logic from |
+ CSSGrammar.y. The new logic adjusts the selector chain to allow |
+ shadow descendant selectors have sub-selectors (and have multiple shadow |
+ descendants in the chain). |
+ * css/CSSParser.h: Added decl. |
+ * css/CSSParserValues.cpp: |
+ (WebCore::CSSParserSelector::insertTagHistory): Added. |
+ (WebCore::CSSParserSelector::appendTagHistory): Aded. |
+ * css/CSSParserValues.h: Added decls. |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): Added |
+ shadow descendant selector match check, since now there could be many |
+ of them in the selector chain. |
+ |
+2011-04-20 Adrienne Walker <enne@google.com> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [chromium] Fix incorrect scissor rect for layers that render into a rendersurface |
+ https://bugs.webkit.org/show_bug.cgi?id=59020 |
+ |
+ mapRect is the incorrect transform here. The parent scissor rect |
+ needs to be projected into layer space instead. |
+ |
+ Test: compositing/flat-with-transformed-child.html |
+ |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::updatePropertiesAndRenderSurfaces): |
+ |
+2011-04-27 Pratik Solanki <psolanki@apple.com> |
+ |
+ Reviewed by Antti Koivisto. |
+ |
+ Part of WebCore should use CFNetwork-based loader on Mac |
+ https://bugs.webkit.org/show_bug.cgi?id=51836 |
+ |
+ * Configurations/WebCore.xcconfig: Add location of CFNetwork.framework. |
+ * platform/mac/SoftLinking.h: |
+ * platform/network/cf/CookieJarCFNet.cpp: Soft-link in CFNetwork functions on Mac. |
+ |
+2011-04-27 Nikolas Zimmermann <nzimmermann@rim.com> |
+ |
+ Reviewed by Antti Koivisto. |
+ |
+ Cleanup CSSStyleApplyProperty.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59623 |
+ |
+ As first step remove ApplyPropertyColorBase, merge it with ApplyPropertyColor, there's no need for two seperated classes. |
+ Use typedefs instead of repating the function pointer declarations in a few places. |
+ |
+ * css/CSSStyleApplyProperty.cpp: |
+ (WebCore::ApplyPropertyDefault::ApplyPropertyDefault): |
+ (WebCore::ApplyPropertyColor::ApplyPropertyColor): |
+ (WebCore::ApplyPropertyColor::applyInheritValue): |
+ (WebCore::ApplyPropertyColor::applyInitialValue): |
+ (WebCore::ApplyPropertyColor::applyValue): |
+ (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): |
+ |
+2011-04-27 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Anders Carlsson. |
+ |
+ <rdar://problem/9335973> REGRESSION (r84341): Buttons in the Mac App store are rendered incorrectly |
+ https://bugs.webkit.org/show_bug.cgi?id=59622 |
+ |
+ Test: fast/borders/border-image-trumps-radius.html |
+ |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::determineBackgroundBleedAvoidance): If there is a renderable, loaded border |
+ image, then border radius is ignored, so no bleed avoidance is needed. |
+ * rendering/RenderObject.cpp: |
+ (WebCore::RenderObject::borderImageIsLoadedAndCanBeRendered): Added. Moved some logic here from... |
+ (WebCore::RenderObject::mustRepaintBackgroundOrBorder): ...here. |
+ * rendering/RenderObject.h: |
+ |
+2011-02-03 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Xan Lopez. |
+ |
+ [GTK] editing/pasteboard/copy-standalone-image.html fails |
+ https://bugs.webkit.org/show_bug.cgi?id=53645 |
+ |
+ When writing an image to the clipboard, also write the URL and title |
+ into the markup and URL portion. This change also abstracts a helper |
+ imageToMarkup into markup.{cpp/h} from several different locations |
+ throughout the platform layer. |
+ |
+ * editing/markup.h: Added new function definition. |
+ * editing/markup.cpp: |
+ (WebCore::imageToMarkup): Add this function which turns an image URL |
+ and element into some markup representing that image, while preserving |
+ non-src attributes. |
+ * platform/chromium/ClipboardChromium.cpp: Move the imageToMarkup helper |
+ to markup.cpp. |
+ * platform/gtk/PasteboardGtk.cpp: |
+ (WebCore::getURLForImageNode): Add this helper which gets the appropriate |
+ image URL for a variety of image tag types. |
+ (WebCore::Pasteboard::writeImage): Write the image URL and title to the |
+ markup and URL portions of the clipboard. This will ensure that images |
+ paste properly back into WebKit. |
+ * platform/win/ClipboardWin.cpp: Remove the imageToMarkup helper and the |
+ one from markup.h |
+ |
+2011-04-27 Steve Block <steveblock@google.com> |
+ |
+ Reviewed by David Levin. |
+ |
+ Remove Android build system |
+ https://bugs.webkit.org/show_bug.cgi?id=48111 |
+ |
+ This is to avoid the maintenance burden until the Android port is |
+ fully upstreamed. |
+ |
+ No new tests, build change only. |
+ |
+ * Android.derived.jscbindings.mk: Removed. |
+ * Android.derived.mk: Removed. |
+ * Android.derived.v8bindings.mk: Removed. |
+ * Android.jscbindings.mk: Removed. |
+ * Android.mk: Removed. |
+ * Android.v8bindings.mk: Removed. |
+ |
+2011-04-27 Darin Adler <darin@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Somehow about:blank gets into the icon database |
+ https://bugs.webkit.org/show_bug.cgi?id=58067 |
+ rdar://problem/6751446 |
+ |
+ Implemented the policy Brady suggested, where only http and https URLs can |
+ be associated with and icon, guarding both on the way in to the database, |
+ and on the way out. |
+ |
+ * loader/icon/IconDatabase.cpp: |
+ (WebCore::pageCanHaveIcon): Added. |
+ (WebCore::IconDatabase::synchronousIconForPageURL): Call pageCanHaveIcon |
+ instead of just checking for an empty URL. |
+ (WebCore::IconDatabase::synchronousIconURLForPageURL): Ditto. |
+ (WebCore::IconDatabase::retainIconForPageURL): Ditto. |
+ (WebCore::IconDatabase::releaseIconForPageURL): Ditto. |
+ (WebCore::IconDatabase::setIconURLForPageURL): Ditto. |
+ (WebCore::IconDatabase::getOrCreatePageURLRecord): Ditto. |
+ (WebCore::IconDatabase::importIconURLForPageURL): Broke assertion into |
+ two separate assertions and added a third. |
+ (WebCore::IconDatabase::performURLImport): Call pageCanHaveIcon. |
+ |
+ * platform/KURL.cpp: Sorted includes. Moved a FIXME to the top of the file. |
+ (WebCore::isLetterMatchIgnoringCase): Renamed matchLetter to this name, |
+ overloaded it to work on both UChar and char, and added an assertion to it. |
+ Also moved this to the top of the file and made the UChar version usable |
+ in the shared code used even by Google. |
+ (WebCore::KURL::invalidate): Updated for name change. |
+ (WebCore::KURL::protocolIs): Use isLetterMatchIgnoringCase instead of |
+ toASCIILower since it's faster. |
+ (WebCore::KURL::parse): Updated for name changes. |
+ (WebCore::protocolIs):Use isLetterMatchIgnoringCase instead of toASCIILower |
+ since it's faster. |
+ (WebCore::protocolIsInHTTPFamily): Added. |
+ |
+ * platform/KURL.h: Tweaked formatting. Renamed protocolInHTTPFamily |
+ to protocolIsInHTTPFamily and added a version that works on a string. |
+ Kept the old name as an inline function so we don't have to rename |
+ all the cal sites now. |
+ |
+ * platform/KURLGoogle.cpp: Renamed as above. |
+ (WebCore::KURLGooglePrivate::KURLGooglePrivate): Updated for rename. |
+ (WebCore::KURLGooglePrivate::setUtf8): Ditto. |
+ (WebCore::KURLGooglePrivate::setAscii): Ditto. |
+ (WebCore::KURLGooglePrivate::initProtocolIsInHTTPFamily): Ditto. |
+ (WebCore::KURLGooglePrivate::copyTo): Ditto. |
+ (WebCore::KURL::KURL): Ditto. |
+ (WebCore::KURL::protocolIsInHTTPFamily): Ditto. |
+ (WebCore::KURL::invalidate): Ditto. |
+ * platform/KURLGooglePrivate.h: Ditto. |
+ |
+ * WebCore.exp.in: Updated for above changes. |
+ |
+2011-04-27 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: browser crash on evaluation of 'throw undefined' |
+ https://bugs.webkit.org/show_bug.cgi?id=59611 |
+ |
+ Test: inspector/console/console-eval-throw-undefined.html |
+ |
+ * inspector/InjectedScriptSource.js: use try/catch to protect from |
+ exception during conversion of another exception to string. |
+ |
+2011-04-27 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: clicking bookmarklet multiplies resources panel entries. |
+ https://bugs.webkit.org/show_bug.cgi?id=59590 |
+ |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.ResourceTreeModel): |
+ (WebInspector.ResourceTreeModel.prototype._processCachedResources): |
+ (WebInspector.ResourceTreeModel.prototype._frameNavigated): |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype._populateResourceTree): |
+ (WebInspector.FrameTreeElement): |
+ (WebInspector.FrameTreeElement.prototype.appendResource): |
+ * inspector/front-end/inspector.js: |
+ (WebInspector.frontendReused): |
+ |
+2011-04-27 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: opening bookmarklet disconnects DOM agent. |
+ https://bugs.webkit.org/show_bug.cgi?id=59588 |
+ |
+ * inspector/InspectorDOMAgent.cpp: |
+ (WebCore::InspectorDOMAgent::setDocument): |
+ |
+2011-04-27 Hans Wennborg <hans@chromium.org> |
+ |
+ Reviewed by Tony Gentilcore. |
+ |
+ IndexedDB: Fix compiler warnings about uninitialized variables |
+ https://bugs.webkit.org/show_bug.cgi?id=59599 |
+ |
+ Build fix. No new tests. |
+ |
+ * storage/IDBLevelDBBackingStore.cpp: |
+ (WebCore::getNewObjectStoreId): |
+ (WebCore::getNewIndexId): |
+ |
+2011-04-26 Hans Wennborg <hans@chromium.org> |
+ |
+ Reviewed by Tony Gentilcore. |
+ |
+ IndexedDB: Move LevelDB key coding routines to separate file |
+ https://bugs.webkit.org/show_bug.cgi?id=59452 |
+ |
+ Move all routines concerned with encoding, decoding and comparison of |
+ LevelDB keys from IDBLevelDBBackingStore.cpp to a separate file: |
+ IDBLevelDBCoding.cpp. |
+ |
+ This makes IDBLevelDBBackingStore.cpp easier to work with, and will |
+ allow for unit testing of the coding routines. |
+ |
+ No new functionality, no new tests. |
+ |
+ * WebCore.gypi: |
+ * storage/IDBLevelDBBackingStore.cpp: |
+ (WebCore::getInt): |
+ (WebCore::putInt): |
+ (WebCore::getString): |
+ (WebCore::putString): |
+ (WebCore::compareKeys): |
+ (WebCore::compareIndexKeys): |
+ (WebCore::Comparator::compare): |
+ (WebCore::setUpMetadata): |
+ (WebCore::IDBLevelDBBackingStore::extractIDBDatabaseMetaData): |
+ (WebCore::getNewDatabaseId): |
+ (WebCore::IDBLevelDBBackingStore::setIDBDatabaseMetaData): |
+ (WebCore::IDBLevelDBBackingStore::getObjectStores): |
+ (WebCore::getNewObjectStoreId): |
+ (WebCore::IDBLevelDBBackingStore::createObjectStore): |
+ (WebCore::IDBLevelDBBackingStore::deleteObjectStore): |
+ (WebCore::IDBLevelDBBackingStore::getObjectStoreRecord): |
+ (WebCore::getNewVersionNumber): |
+ (WebCore::IDBLevelDBBackingStore::putObjectStoreRecord): |
+ (WebCore::IDBLevelDBBackingStore::clearObjectStore): |
+ (WebCore::IDBLevelDBBackingStore::deleteObjectStoreRecord): |
+ (WebCore::IDBLevelDBBackingStore::nextAutoIncrementNumber): |
+ (WebCore::IDBLevelDBBackingStore::keyExistsInObjectStore): |
+ (WebCore::IDBLevelDBBackingStore::forEachObjectStoreRecord): |
+ (WebCore::IDBLevelDBBackingStore::getIndexes): |
+ (WebCore::getNewIndexId): |
+ (WebCore::IDBLevelDBBackingStore::createIndex): |
+ (WebCore::IDBLevelDBBackingStore::putIndexDataForRecord): |
+ (WebCore::versionExists): |
+ (WebCore::IDBLevelDBBackingStore::getPrimaryKeyViaIndex): |
+ (WebCore::IDBLevelDBBackingStore::keyExistsInIndex): |
+ (WebCore::IDBLevelDBBackingStore::openObjectStoreCursor): |
+ (WebCore::IDBLevelDBBackingStore::openIndexKeyCursor): |
+ (WebCore::IDBLevelDBBackingStore::openIndexCursor): |
+ * storage/IDBLevelDBCoding.cpp: Added. |
+ (WebCore::IDBLevelDBCoding::encodeByte): |
+ (WebCore::IDBLevelDBCoding::maxIDBKey): |
+ (WebCore::IDBLevelDBCoding::minIDBKey): |
+ (WebCore::IDBLevelDBCoding::encodeInt): |
+ (WebCore::IDBLevelDBCoding::decodeInt): |
+ (WebCore::IDBLevelDBCoding::encodeVarInt): |
+ (WebCore::IDBLevelDBCoding::decodeVarInt): |
+ (WebCore::IDBLevelDBCoding::encodeString): |
+ (WebCore::IDBLevelDBCoding::decodeString): |
+ (WebCore::IDBLevelDBCoding::encodeStringWithLength): |
+ (WebCore::IDBLevelDBCoding::decodeStringWithLength): |
+ (WebCore::IDBLevelDBCoding::encodeDouble): |
+ (WebCore::IDBLevelDBCoding::decodeDouble): |
+ (WebCore::IDBLevelDBCoding::encodeIDBKey): |
+ (WebCore::IDBLevelDBCoding::decodeIDBKey): |
+ (WebCore::IDBLevelDBCoding::extractEncodedIDBKey): |
+ (WebCore::IDBLevelDBCoding::compareEncodedIDBKeys): |
+ (WebCore::IDBLevelDBCoding::compare): |
+ (WebCore::IDBLevelDBCoding::KeyPrefix::KeyPrefix): |
+ (WebCore::IDBLevelDBCoding::KeyPrefix::decode): |
+ (WebCore::IDBLevelDBCoding::KeyPrefix::encode): |
+ (WebCore::IDBLevelDBCoding::KeyPrefix::compare): |
+ (WebCore::IDBLevelDBCoding::KeyPrefix::type): |
+ (WebCore::IDBLevelDBCoding::SchemaVersionKey::encode): |
+ (WebCore::IDBLevelDBCoding::MaxDatabaseIdKey::encode): |
+ (WebCore::IDBLevelDBCoding::DatabaseFreeListKey::DatabaseFreeListKey): |
+ (WebCore::IDBLevelDBCoding::DatabaseFreeListKey::decode): |
+ (WebCore::IDBLevelDBCoding::DatabaseFreeListKey::encode): |
+ (WebCore::IDBLevelDBCoding::DatabaseFreeListKey::databaseId): |
+ (WebCore::IDBLevelDBCoding::DatabaseFreeListKey::compare): |
+ (WebCore::IDBLevelDBCoding::DatabaseNameKey::decode): |
+ (WebCore::IDBLevelDBCoding::DatabaseNameKey::encode): |
+ (WebCore::IDBLevelDBCoding::DatabaseNameKey::compare): |
+ (WebCore::IDBLevelDBCoding::DatabaseMetaDataKey::encode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreMetaDataKey::ObjectStoreMetaDataKey): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreMetaDataKey::decode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreMetaDataKey::encode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreMetaDataKey::objectStoreId): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreMetaDataKey::metaDataType): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreMetaDataKey::compare): |
+ (WebCore::IDBLevelDBCoding::IndexMetaDataKey::IndexMetaDataKey): |
+ (WebCore::IDBLevelDBCoding::IndexMetaDataKey::decode): |
+ (WebCore::IDBLevelDBCoding::IndexMetaDataKey::encode): |
+ (WebCore::IDBLevelDBCoding::IndexMetaDataKey::compare): |
+ (WebCore::IDBLevelDBCoding::IndexMetaDataKey::indexId): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreFreeListKey::ObjectStoreFreeListKey): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreFreeListKey::decode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreFreeListKey::encode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreFreeListKey::objectStoreId): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreFreeListKey::compare): |
+ (WebCore::IDBLevelDBCoding::IndexFreeListKey::IndexFreeListKey): |
+ (WebCore::IDBLevelDBCoding::IndexFreeListKey::decode): |
+ (WebCore::IDBLevelDBCoding::IndexFreeListKey::encode): |
+ (WebCore::IDBLevelDBCoding::IndexFreeListKey::compare): |
+ (WebCore::IDBLevelDBCoding::IndexFreeListKey::objectStoreId): |
+ (WebCore::IDBLevelDBCoding::IndexFreeListKey::indexId): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreNamesKey::decode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreNamesKey::encode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreNamesKey::compare): |
+ (WebCore::IDBLevelDBCoding::IndexNamesKey::IndexNamesKey): |
+ (WebCore::IDBLevelDBCoding::IndexNamesKey::decode): |
+ (WebCore::IDBLevelDBCoding::IndexNamesKey::encode): |
+ (WebCore::IDBLevelDBCoding::IndexNamesKey::compare): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreDataKey::decode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreDataKey::encode): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreDataKey::compare): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreDataKey::userKey): |
+ (WebCore::IDBLevelDBCoding::ExistsEntryKey::decode): |
+ (WebCore::IDBLevelDBCoding::ExistsEntryKey::encode): |
+ (WebCore::IDBLevelDBCoding::ExistsEntryKey::compare): |
+ (WebCore::IDBLevelDBCoding::ExistsEntryKey::userKey): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::IndexDataKey): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::decode): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::encode): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::encodeMaxKey): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::compare): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::databaseId): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::objectStoreId): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::indexId): |
+ (WebCore::IDBLevelDBCoding::IndexDataKey::userKey): |
+ * storage/IDBLevelDBCoding.h: Added. |
+ (WebCore::IDBLevelDBCoding::DatabaseNameKey::origin): |
+ (WebCore::IDBLevelDBCoding::DatabaseNameKey::databaseName): |
+ (WebCore::IDBLevelDBCoding::IndexMetaDataKey::metaDataType): |
+ (WebCore::IDBLevelDBCoding::ObjectStoreNamesKey::objectStoreName): |
+ (WebCore::IDBLevelDBCoding::IndexNamesKey::indexName): |
+ |
+2011-04-27 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: expose exception details when script is paused on exception |
+ https://bugs.webkit.org/show_bug.cgi?id=59591 |
+ |
+ When script is paused on an exception, the exception value is added as <exception> |
+ property in the local scope and the breakpoint details message says |
+ "Paused on excepion: '<exception to strin value>'." |
+ |
+ * English.lproj/localizedStrings.js: |
+ * inspector/front-end/ScopeChainSidebarPane.js: |
+ (WebInspector.ScopeChainSidebarPane.prototype.update): |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._debuggerPaused.else.didGetSourceLocation): |
+ (WebInspector.ScriptsPanel.prototype._debuggerPaused): |
+ |
+2011-04-26 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [protocol] Paused event should expose exception value that caused it |
+ https://bugs.webkit.org/show_bug.cgi?id=58996 |
+ |
+ Debug.pause event now contains an optional reference to the exception |
+ object in case script execution is paused on a JavaScript exception. |
+ |
+ Also JavaScript call frames are passed directly into the injected script when |
+ we need to wrap them for passing to the front-end. This change breaks cyclic |
+ dependency ScriptDebugServer->InspectorDebuggerAgent->InjectedScript->InjectedScriptHost->ScriptDebugServer |
+ |
+ * bindings/js/JSInjectedScriptHostCustom.cpp: |
+ * bindings/js/ScriptDebugServer.cpp: |
+ (WebCore::ScriptDebugServer::dispatchDidPause): |
+ * bindings/js/ScriptDebugServer.h: |
+ * bindings/v8/ScriptDebugServer.cpp: |
+ (WebCore::ScriptDebugServer::breakProgram): |
+ (WebCore::ScriptDebugServer::editScriptSource): |
+ (WebCore::ScriptDebugServer::breakProgramCallback): |
+ (WebCore::ScriptDebugServer::handleV8DebugEvent): |
+ * bindings/v8/ScriptDebugServer.h: |
+ * bindings/v8/custom/V8InjectedScriptHostCustom.cpp: |
+ * inspector/InjectedScript.cpp: |
+ (WebCore::InjectedScript::evaluateOnCallFrame): |
+ (WebCore::InjectedScript::wrapCallFrames): |
+ * inspector/InjectedScript.h: |
+ * inspector/InjectedScriptHost.cpp: |
+ * inspector/InjectedScriptHost.h: |
+ (WebCore::InjectedScriptHost::init): |
+ * inspector/InjectedScriptHost.idl: |
+ * inspector/InjectedScriptSource.js: |
+ (.): |
+ * inspector/Inspector.json: |
+ * inspector/InspectorAgent.cpp: |
+ (WebCore::InspectorAgent::InspectorAgent): |
+ * inspector/InspectorDebuggerAgent.cpp: |
+ (WebCore::InspectorDebuggerAgent::evaluateOnCallFrame): |
+ (WebCore::InspectorDebuggerAgent::currentCallFrames): |
+ (WebCore::InspectorDebuggerAgent::wrapCallFrames): |
+ (WebCore::InspectorDebuggerAgent::didPause): |
+ (WebCore::InspectorDebuggerAgent::didContinue): |
+ (WebCore::InspectorDebuggerAgent::clear): |
+ * inspector/InspectorDebuggerAgent.h: |
+ * inspector/ScriptDebugListener.h: |
+ * inspector/WorkerInspectorController.cpp: |
+ (WebCore::WorkerInspectorController::WorkerInspectorController): |
+ |
+2011-04-27 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: implement incremental CSS editing in the structure view. |
+ https://bugs.webkit.org/show_bug.cgi?id=59455 |
+ |
+ * inspector/front-end/StylesSidebarPane.js: |
+ (WebInspector.StylePropertyTreeElement.prototype): |
+ |
+2011-04-27 Mihai Parparita <mihaip@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Frame.h shouldn't include Document.h |
+ https://bugs.webkit.org/show_bug.cgi?id=59560 |
+ |
+ Frame can use a forward-declared Document if we move |
+ Frame::displayStringModifiedByEncoding's implementation out of the |
+ header. |
+ Size #includes Size * #includes |
+ Before: page/Frame.h: 0.746 653 487.138 |
+ dom/Document.h: 0.220 1306 287.320 |
+ |
+ After: page/Frame.h: 0.646 653 421.838 |
+ dom/Document.h: 0.220 1189 261.580 |
+ |
+ * WebCore.exp.in: |
+ * bindings/ScriptControllerBase.cpp: |
+ * bindings/generic/BindingSecurityBase.cpp: |
+ * bindings/v8/V8Proxy.cpp: |
+ * bindings/v8/custom/V8XMLHttpRequestCustom.cpp: |
+ * dom/CharacterData.cpp: |
+ * dom/UserTypingGestureIndicator.cpp: |
+ * editing/chromium/SelectionControllerChromium.cpp: |
+ * inspector/InspectorDOMStorageResource.cpp: |
+ * inspector/InspectorInstrumentation.h: |
+ * loader/PingLoader.cpp: |
+ * loader/SubresourceLoader.cpp: |
+ * loader/appcache/DOMApplicationCache.cpp: |
+ * loader/cache/CachedResourceRequest.cpp: |
+ * page/Frame.cpp: |
+ (WebCore::Frame::displayStringModifiedByEncoding): |
+ * page/Frame.h: |
+ * page/Geolocation.cpp: |
+ * page/Location.cpp: |
+ * page/Navigator.cpp: |
+ * page/PageGroupLoadDeferrer.cpp: |
+ * page/PerformanceTiming.cpp: |
+ * xml/XSLTProcessorLibxslt.cpp: |
+ |
+2011-04-27 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Fix OwnPtr strict errors in RenderStyle and make StyleRareInheritedData::textShadow an OwnPtr |
+ https://bugs.webkit.org/show_bug.cgi?id=59377 |
+ |
+ This cleans up some strict OwnPtr<> violations around text and box shadow data. ShadowData's linked list data |
+ structure now uses OwnPtr<>s to manage memory - each entry in the list has ownership of the next ShadowData. |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ * page/animation/AnimationBase.cpp: |
+ (WebCore::blendFunc): |
+ (WebCore::PropertyWrapperShadow::PropertyWrapperShadow): |
+ (WebCore::PropertyWrapperShadow::blend): |
+ * rendering/style/RenderStyle.cpp: |
+ (WebCore::RenderStyle::setTextShadow): |
+ (WebCore::RenderStyle::setBoxShadow): |
+ * rendering/style/RenderStyle.h: |
+ (WebCore::InheritedFlags::textShadow): |
+ * rendering/style/ShadowData.cpp: |
+ (WebCore::ShadowData::ShadowData): |
+ * rendering/style/ShadowData.h: |
+ (WebCore::ShadowData::ShadowData): |
+ (WebCore::ShadowData::next): |
+ (WebCore::ShadowData::setNext): |
+ * rendering/style/StyleRareInheritedData.cpp: |
+ (WebCore::StyleRareInheritedData::StyleRareInheritedData): |
+ (WebCore::StyleRareInheritedData::~StyleRareInheritedData): |
+ * rendering/style/StyleRareInheritedData.h: |
+ * rendering/style/StyleRareNonInheritedData.cpp: |
+ (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): |
+ |
+2011-04-26 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Fix some strict PassOwnPtr issues in WebCore. |
+ https://bugs.webkit.org/show_bug.cgi?id=59563 |
+ |
+ * css/SVGCSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applySVGProperty): |
+ * html/canvas/WebGLRenderingContext.cpp: |
+ (WebCore::WebGLRenderingContext::create): |
+ (WebCore::WebGLRenderingContext::~WebGLRenderingContext): |
+ * platform/text/TextCodecLatin1.cpp: |
+ (WebCore::newStreamingTextDecoderWindowsLatin1): |
+ * platform/text/TextCodecUTF16.cpp: |
+ (WebCore::newStreamingTextDecoderUTF16LE): |
+ (WebCore::newStreamingTextDecoderUTF16BE): |
+ * platform/text/TextCodecUserDefined.cpp: |
+ (WebCore::newStreamingTextDecoderUserDefined): |
+ * platform/text/mac/TextCodecMac.cpp: |
+ (WebCore::newTextCodecMac): |
+ * workers/Worker.cpp: |
+ (WebCore::Worker::notifyFinished): |
+ |
+2011-04-26 Justin Novosad <junov@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [Chromium] Expose skia gpu canvas rendering as a runtime flag |
+ https://bugs.webkit.org/show_bug.cgi?id=58683 |
+ Replace the SKIA_GPU compile flag by the pre-existing |
+ acceleratedDrawingEnabled flag. Most code changes are trivial: |
+ compile-time conditionals on SKIA_GPU were either replaced with |
+ a run-time check, or a compile time check on USE(SKIA) |
+ |
+ No new tests. Covered by existing tests. |
+ |
+ * html/canvas/CanvasRenderingContext2D.cpp: |
+ (WebCore::CanvasRenderingContext2D::setGlobalCompositeOperation): |
+ * page/Page.cpp: |
+ (WebCore::Page::sharedGraphicsContext3D): |
+ Added check for the acceleratedDrawing flag. |
+ * platform/graphics/chromium/DrawingBufferChromium.cpp: |
+ (WebCore::DrawingBuffer::DrawingBuffer): |
+ (WebCore::DrawingBuffer::publishToPlatformLayer): |
+ * platform/graphics/gpu/DrawingBuffer.h: |
+ * platform/graphics/gpu/SharedGraphicsContext3D.cpp: |
+ (WebCore::SharedGraphicsContext3D::create): |
+ On Skia builds, this methos now receives a bool arg to turn on |
+ GPU-accelerated skia rendering |
+ (WebCore::SharedGraphicsContext3D::SharedGraphicsContext3D): |
+ (WebCore::SharedGraphicsContext3D::~SharedGraphicsContext3D): |
+ (WebCore::SharedGraphicsContext3D::grContext): |
+ * platform/graphics/gpu/SharedGraphicsContext3D.h: |
+ * platform/graphics/skia/ImageSkia.cpp: |
+ (WebCore::computeResamplingMode): |
+ (WebCore::paintSkBitmap): |
+ (WebCore::Image::drawPattern): |
+ * platform/graphics/skia/PlatformContextSkia.cpp: |
+ (WebCore::PlatformContextSkia::PlatformContextSkia): |
+ (WebCore::PlatformContextSkia::~PlatformContextSkia): |
+ (WebCore::PlatformContextSkia::isNativeFontRenderingAllowed): |
+ (WebCore::PlatformContextSkia::setSharedGraphicsContext3D): |
+ (WebCore::PlatformContextSkia::prepareForSoftwareDraw): |
+ (WebCore::PlatformContextSkia::prepareForHardwareDraw): |
+ (WebCore::PlatformContextSkia::syncSoftwareCanvas): |
+ (WebCore::PlatformContextSkia::markDirtyRect): |
+ * platform/graphics/skia/PlatformContextSkia.h: |
+ (WebCore::PlatformContextSkia::accelerationMode): |
+ (WebCore::PlatformContextSkia::useGPU): |
+ (WebCore::PlatformContextSkia::useSkiaGPU): |
+ |
+2011-04-26 Pavel Feldman <pfeldman@chromium.org> |
+ |
+ Not reviewed: restore inspector test disabled in r84913. |
+ |
+ Test: http/tests/inspector-enabled/dom-storage-open.html |
+ |
+ * inspector/InspectorDOMStorageAgent.cpp: |
+ (WebCore::InspectorDOMStorageAgent::clearFrontend): |
+ |
+2011-04-26 Kenichi Ishibashi <bashi@chromium.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ [Chromium] Vertical positions are off for some Arabic glyphs on Linux |
+ https://bugs.webkit.org/show_bug.cgi?id=59182 |
+ |
+ Use vertical offsets of the shaping results. |
+ |
+ Test: platform/chromium-linux/fast/text/international/arabic-vertical-offset.html |
+ |
+ * platform/graphics/chromium/ComplexTextControllerLinux.cpp: |
+ (WebCore::ComplexTextController::ComplexTextController): |
+ Added initialization of m_startingY. |
+ (WebCore::ComplexTextController::nextScriptRun): |
+ Followed the change in handling positions. |
+ (WebCore::ComplexTextController::deleteGlyphArrays): Ditto. |
+ (WebCore::ComplexTextController::createGlyphArrays): Ditto. |
+ (WebCore::ComplexTextController::resetGlyphArrays): Ditto. |
+ (WebCore::ComplexTextController::setGlyphPositions): |
+ Changed to use vertical offsets as same as horizontal offsets. |
+ * platform/graphics/chromium/ComplexTextControllerLinux.h: |
+ Removed m_xPositions and Added m_positions and m_startingY. |
+ (WebCore::ComplexTextController::positions): Added. |
+ * platform/graphics/chromium/FontLinux.cpp: |
+ Followed the change in ComplexTextController. |
+ (WebCore::Font::drawComplexText): Ditto. |
+ (WebCore::Font::floatWidthForComplexText): Ditto. |
+ (WebCore::glyphIndexForXPositionInScriptRun): Ditto. |
+ (WebCore::Font::offsetForPositionForComplexText): Ditto. |
+ (WebCore::Font::selectionRectForComplexText): Ditto. |
+ |
+2011-04-26 Levi Weintraub <leviw@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Root element should establish a new block formatting context |
+ https://bugs.webkit.org/show_bug.cgi?id=54573 |
+ |
+ Always expanding the root renderer to include overhanging floats. |
+ |
+ Test: fast/block/float/float-overhangs-root.html |
+ |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::expandsToEncloseOverhangingFloats): |
+ |
+2011-04-26 Dawit Alemayehu <adawit@kde.org> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ [Qt] Improper rendering of <button> tag when it contains a <br> |
+ https://bugs.webkit.org/show_bug.cgi?id=50521 |
+ |
+ Test: platform/qt/fast/forms/button-line-break.html |
+ |
+ * platform/qt/RenderThemeQt.cpp: |
+ (WebCore::RenderThemeQt::computeSizeBasedOnStyle): |
+ (WebCore::RenderThemeQt::setButtonPadding): |
+ |
+2011-04-26 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [chromium] Only reset tiles if the LayerRendererChromium is actually changing |
+ https://bugs.webkit.org/show_bug.cgi?id=59572 |
+ |
+ http://trac.webkit.org/changeset/84981/ caused us to reset the tiles |
+ when LayerRendererChromium was set. We set this pointer every frame |
+ and should only reset the tiles if the new LayerRendererChromium is |
+ different from the old one (indicating a context loss), not just reset |
+ on every frame. |
+ |
+ * platform/graphics/chromium/LayerTilerChromium.cpp: |
+ (WebCore::LayerTilerChromium::setLayerRenderer): |
+ * platform/graphics/chromium/LayerTilerChromium.h: |
+ |
+2011-04-26 Dinesh K Garg <dineshg@codeaurora.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ beginElement broken by setAttribute |
+ https://bugs.webkit.org/show_bug.cgi?id=26019 |
+ |
+ Test: svg/animations/animate-beginElementAt.svg |
+ |
+ * svg/SVGAnimationElement.cpp: |
+ (WebCore::SVGAnimationElement::attributeChanged): |
+ Reset the animation state here as the rest of the code reads it and would still |
+ think we are animating when we are not. |
+ |
+ * svg/animation/SVGSMILElement.h: |
+ (WebCore::SVGSMILElement::setInactive): Helper function to reset the state. |
+ |
+2011-04-26 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r84989. |
+ http://trac.webkit.org/changeset/84989 |
+ https://bugs.webkit.org/show_bug.cgi?id=59566 |
+ |
+ REGRESSION (r84989): Lots of vertical text tests failing on |
+ windows (Requested by weinig on #webkit). |
+ |
+ * platform/graphics/FontPlatformData.h: |
+ * platform/graphics/win/FontCacheWin.cpp: |
+ (WebCore::FontCache::getFontDataForCharacters): |
+ (WebCore::FontCache::createFontPlatformData): |
+ * platform/graphics/win/FontCustomPlatformData.cpp: |
+ (WebCore::FontCustomPlatformData::fontPlatformData): |
+ * platform/graphics/win/FontPlatformDataCGWin.cpp: |
+ (WebCore::FontPlatformData::FontPlatformData): |
+ * platform/graphics/win/FontPlatformDataWin.cpp: |
+ (WebCore::FontPlatformData::FontPlatformData): |
+ |
+2011-04-26 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Try again: |
+ Remove JSDOMWrapperWithGlobalPointer now that all JSDOMWrappers have global objects |
+ https://bugs.webkit.org/show_bug.cgi?id=59310 |
+ |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::DOMConstructorObject::DOMConstructorObject): |
+ * bindings/js/JSDOMWrapper.h: |
+ (WebCore::JSDOMWrapper::globalObject): |
+ (WebCore::JSDOMWrapper::scriptExecutionContext): |
+ (WebCore::JSDOMWrapper::createStructure): |
+ (WebCore::JSDOMWrapper::JSDOMWrapper): |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ |
+2011-04-26 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by David Hyatt. |
+ |
+ Remove Datagrid from the tree |
+ https://bugs.webkit.org/show_bug.cgi?id=59543 |
+ |
+ * Android.derived.jscbindings.mk: |
+ * Android.derived.v8bindings.mk: |
+ * Android.jscbindings.mk: |
+ * Android.v8bindings.mk: |
+ * CMakeLists.txt: |
+ * CodeGenerators.pri: |
+ * Configurations/FeatureDefines.xcconfig: |
+ * DerivedSources.cpp: |
+ * DerivedSources.make: |
+ * GNUmakefile.am: |
+ * GNUmakefile.list.am: |
+ * UseJSC.cmake: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * bindings/js/JSBindingsAllInOne.cpp: |
+ * bindings/js/JSDataGridColumnListCustom.cpp: Removed. |
+ * bindings/js/JSDataGridDataSource.cpp: Removed. |
+ * bindings/js/JSDataGridDataSource.h: Removed. |
+ * bindings/js/JSHTMLDataGridElementCustom.cpp: Removed. |
+ * bindings/v8/V8DataGridDataSource.cpp: Removed. |
+ * bindings/v8/V8DataGridDataSource.h: Removed. |
+ * bindings/v8/custom/V8DataGridColumnListCustom.cpp: Removed. |
+ * bindings/v8/custom/V8HTMLDataGridElementCustom.cpp: Removed. |
+ * css/CSSStyleSelector.cpp: |
+ * css/CSSStyleSelector.h: |
+ * css/html.css: |
+ (input, textarea, keygen, select, button, isindex): |
+ (select:disabled, keygen:disabled, optgroup:disabled, option:disabled): |
+ * editing/htmlediting.cpp: |
+ (WebCore::canHaveChildrenForEditing): |
+ * features.pri: |
+ * html/DOMDataGridDataSource.cpp: Removed. |
+ * html/DOMDataGridDataSource.h: Removed. |
+ * html/DataGridColumn.cpp: Removed. |
+ * html/DataGridColumn.h: Removed. |
+ * html/DataGridColumn.idl: Removed. |
+ * html/DataGridColumnList.cpp: Removed. |
+ * html/DataGridColumnList.h: Removed. |
+ * html/DataGridColumnList.idl: Removed. |
+ * html/DataGridDataSource.h: Removed. |
+ * html/HTMLDataGridCellElement.cpp: Removed. |
+ * html/HTMLDataGridCellElement.h: Removed. |
+ * html/HTMLDataGridCellElement.idl: Removed. |
+ * html/HTMLDataGridColElement.cpp: Removed. |
+ * html/HTMLDataGridColElement.h: Removed. |
+ * html/HTMLDataGridColElement.idl: Removed. |
+ * html/HTMLDataGridElement.cpp: Removed. |
+ * html/HTMLDataGridElement.h: Removed. |
+ * html/HTMLDataGridElement.idl: Removed. |
+ * html/HTMLDataGridRowElement.cpp: Removed. |
+ * html/HTMLDataGridRowElement.h: Removed. |
+ * html/HTMLDataGridRowElement.idl: Removed. |
+ * html/HTMLElement.cpp: |
+ (WebCore::HTMLElement::ieForbidsInsertHTML): |
+ * html/HTMLElementsAllInOne.cpp: |
+ * html/HTMLTagNames.in: |
+ * page/DOMWindow.idl: |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::sizesToIntrinsicLogicalWidth): |
+ * rendering/RenderDataGrid.cpp: Removed. |
+ * rendering/RenderDataGrid.h: Removed. |
+ * rendering/RenderingAllInOne.cpp: |
+ |
+2011-04-26 Chun-Lung Huang <alvincl.huang@gmail.com> |
+ |
+ Reviewed by Adele Peterson. |
+ |
+ On WebKit (Windows), glyphs in vertical text tests are rotated 90 |
+ degrees clockwise. https://bugs.webkit.org/show_bug.cgi?id=48459 |
+ |
+ This platform dependent patch makes WebKit (Windows) show the |
+ vertical writing text correctly. Job was done by adding a prefix '@' |
+ in front of the font family name (Windows Only). No new tests added. |
+ Some layout tests images: |
+ http://www.flickr.com/photos/burorly/sets/72157625585506341/ |
+ |
+ * platform/graphics/FontPlatformData.h: |
+ * platform/graphics/win/FontCacheWin.cpp: |
+ (WebCore::FontCache::getFontDataForCharacters): |
+ (WebCore::FontCache::createFontPlatformData): |
+ * platform/graphics/win/FontCustomPlatformData.cpp: |
+ (WebCore::FontCustomPlatformData::fontPlatformData): |
+ * platform/graphics/win/FontPlatformDataCGWin.cpp: |
+ (WebCore::FontPlatformData::FontPlatformData): |
+ * platform/graphics/win/FontPlatformDataWin.cpp: |
+ (WebCore::FontPlatformData::FontPlatformData): |
+ |
+2011-04-26 Jer Noble <jer.noble@apple.com> |
+ |
+ Reviewed by Brady Eidson. |
+ |
+ HTML5 video fullscreen transition causes an audio stutter |
+ https://bugs.webkit.org/show_bug.cgi?id=59544 |
+ |
+ AVFoundation will momentarily stutter when you disconnect a AVPlayerLayer |
+ from its AVPlayer. Since you shouldn't necessarily have to destroy the |
+ layer to render it invisible, instead just set the layer as hidden. |
+ |
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp: |
+ (WebCore::MediaPlayerPrivateAVFoundation::setVisible): Call through to platformSetVisible(). |
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.h: |
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.h: |
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.mm: |
+ (WebCore::MediaPlayerPrivateAVFoundationObjC::platformSetVisible): Added. |
+ |
+2011-04-26 Christian Dywan <christian@lanedo.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Extra separator after Select All in context menu |
+ https://bugs.webkit.org/show_bug.cgi?id=54092 |
+ |
+ * page/ContextMenuController.cpp: Move separator to non-GTK+ |
+ port guards since it is for the spelling options menu |
+ which the GTK+ port doesn't use. |
+ |
+2011-04-26 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [chromium] Clear out LayerTilerChromium's tiles on lost context |
+ https://bugs.webkit.org/show_bug.cgi?id=59233 |
+ |
+ Clears out the LayerTilerChromium's tile set when the LayerRendererChromium is reset. In this case the tiles are |
+ no longer valid since they exist in the previous LayerRendererChromium. |
+ |
+ Will be tested by a chrome test since we don't have any way to exercise the context lost case in layout tests. |
+ |
+ * platform/graphics/chromium/LayerTilerChromium.h: |
+ (WebCore::LayerTilerChromium::setLayerRenderer): |
+ |
+2011-04-26 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Dirk Schulze. |
+ |
+ [GTK] Flash in divs with overflow: auto is not positioned and clipped properly |
+ https://bugs.webkit.org/show_bug.cgi?id=57644 |
+ |
+ Manual test: plugins/windowed-in-iframe-2.html |
+ |
+ * manual-tests/plugins/windowed-in-iframe-2.html: Added. |
+ * platform/gtk/WidgetGtk.cpp: |
+ (WebCore::Widget::setFrameRect): Like the Qt port, we notify the widget via frameRectsChanged. |
+ * plugins/PluginView.h: Added helper method to set the allocation and clip. |
+ * plugins/gtk/PluginViewGtk.cpp: |
+ (WebCore::PluginView::setNPWindowIfNeeded):Call the helper method instead of setting |
+ the allocation immediately. Bring this method into sync with the implementation in |
+ the Qt port. |
+ (WebCore::PluginView::updateWidgetAllocationAndClip): Added. |
+ (WebCore::PluginView::plugAddedCallback): Call the new helper method. |
+ |
+2011-04-26 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ [GTK] fast/block/float/overhanging-tall-block.html crashes in the bots |
+ https://bugs.webkit.org/show_bug.cgi?id=58818 |
+ |
+ Prevent allocating scratch buffers larger than the target GdkDrawable |
+ when creating a WidgetRenderingContext. This prevents incredibly large |
+ widgets from crashing the X Server. This change also allowed greatly |
+ simplifying the way that the painting offset is calculated. |
+ |
+ * platform/gtk/WidgetRenderingContext.cpp: |
+ (WebCore::WidgetRenderingContext::WidgetRenderingContext): Never allocate a pixmap |
+ larger than the target GdkDrawable. |
+ (WebCore::WidgetRenderingContext::~WidgetRenderingContext): Calculate the offset of the |
+ blit by looking at the location of the target rect itself, rather than delaying the |
+ calculation up to this point. |
+ (WebCore::WidgetRenderingContext::calculateClipRect): Added. |
+ (WebCore::WidgetRenderingContext::gtkPaintBox): Use the new calculateClipRectHelper. |
+ (WebCore::WidgetRenderingContext::gtkPaintFlatBox): Ditto. |
+ (WebCore::WidgetRenderingContext::gtkPaintFocus): Ditto. |
+ (WebCore::WidgetRenderingContext::gtkPaintSlider): Ditto. |
+ (WebCore::WidgetRenderingContext::gtkPaintCheck): Ditto. |
+ (WebCore::WidgetRenderingContext::gtkPaintOption): Ditto. |
+ (WebCore::WidgetRenderingContext::gtkPaintShadow): Ditto. |
+ (WebCore::WidgetRenderingContext::gtkPaintArrow): Ditto. |
+ (WebCore::WidgetRenderingContext::gtkPaintVLine): Ditto. |
+ * platform/gtk/WidgetRenderingContext.h: Remove some now unused members. Add |
+ a member to store the IntSize mapping from the coordinates of the target to the coordinates |
+ of the scratch buffer. |
+ |
+2011-04-26 Patrick Gansterer <paroga@webkit.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Respect charset in handleDataURL |
+ https://bugs.webkit.org/show_bug.cgi?id=47746 |
+ |
+ Original patch by Kwang Yul Seo <skyul@company100.net> |
+ |
+ We must use TextEncoding(charset) to encode data URL again. Otherwise TextEncoding::encode() |
+ returns an empty string because encoding name is not specified. |
+ |
+ Also use "US-ASCII" as fallback charset as extractCharsetFromMediaType can be empty. |
+ |
+ This change fixes over 130 layout tests when running GTK port with cURL network backend. |
+ |
+ * platform/network/DataURL.cpp: |
+ (WebCore::handleDataURL): |
+ |
+2011-04-26 Anders Carlsson <andersca@apple.com> |
+ |
+ Fix clang++ build. |
+ |
+ You can never have too many casts! |
+ |
+ * accessibility/mac/AccessibilityObjectWrapper.mm: |
+ (-[AccessibilityObjectWrapper position]): |
+ * editing/mac/SelectionControllerMac.mm: |
+ (WebCore::accessibilityConvertScreenRect): |
+ |
+2011-04-26 Chris Rogers <crogers@google.com> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ Fix web audio build on mac port |
+ https://bugs.webkit.org/show_bug.cgi?id=59355 |
+ |
+ No new tests since audio API is not yet implemented. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * bindings/js/JSAudioContextCustom.cpp: |
+ (WebCore::JSAudioContext::visitChildren): |
+ * bindings/js/JSJavaScriptAudioNodeCustom.cpp: |
+ (WebCore::JSJavaScriptAudioNode::visitChildren): |
+ * webaudio/AudioContext.idl: |
+ |
+2011-04-26 Chris Rogers <crogers@google.com> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ Add FFTFrame implementation for FFmpeg |
+ https://bugs.webkit.org/show_bug.cgi?id=59408 |
+ |
+ No new tests since audio API is not yet implemented. |
+ |
+ * WebCore.gyp/WebCore.gyp: |
+ * WebCore.gypi: |
+ * platform/audio/FFTFrame.h: |
+ * platform/audio/FFTFrameStub.cpp: |
+ * platform/audio/ffmpeg: Added. |
+ * platform/audio/ffmpeg/FFTFrameFFMPEG.cpp: Added. |
+ (WebCore::FFTFrame::FFTFrame): |
+ (WebCore::FFTFrame::initialize): |
+ (WebCore::FFTFrame::cleanup): |
+ (WebCore::FFTFrame::~FFTFrame): |
+ (WebCore::FFTFrame::multiply): |
+ (WebCore::FFTFrame::doFFT): |
+ (WebCore::FFTFrame::doInverseFFT): |
+ (WebCore::FFTFrame::realData): |
+ (WebCore::FFTFrame::imagData): |
+ (WebCore::FFTFrame::getUpToDateComplexData): |
+ (WebCore::FFTFrame::contextForSize): |
+ * platform/audio/mac/FFTFrameMac.cpp: |
+ * webaudio/ConvolverNode.cpp: |
+ |
+2011-04-26 Jeff Miller <jeffm@apple.com> |
+ |
+ Add a newline at the end of SVGFEDropShadowElement.idl. |
+ |
+ * svg/SVGFEDropShadowElement.idl: |
+ |
+2011-04-26 Mihai Parparita <mihaip@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ InlineBox.h shouldn't include RenderBR.h |
+ https://bugs.webkit.org/show_bug.cgi?id=59480 |
+ |
+ Per bug 59348 RenderBR.h is a "top header" by include * size, because |
+ InlineBox.h includes it. There's no reason why it has to. |
+ |
+ * rendering/InlineBox.h: |
+ * rendering/InlineTextBox.cpp: |
+ * rendering/RenderBlock.h: |
+ |
+2011-04-26 Sam Weinig <sam@webkit.org> |
+ |
+ Nope, roll out r84950 and r84952. Working on a laptop is clearly not something |
+ I should do. |
+ |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::JSDOMWrapperWithGlobalPointer::globalObject): |
+ (WebCore::JSDOMWrapperWithGlobalPointer::scriptExecutionContext): |
+ (WebCore::JSDOMWrapperWithGlobalPointer::createStructure): |
+ (WebCore::JSDOMWrapperWithGlobalPointer::JSDOMWrapperWithGlobalPointer): |
+ (WebCore::DOMConstructorObject::DOMConstructorObject): |
+ * bindings/js/JSDOMWrapper.h: |
+ (WebCore::JSDOMWrapper::JSDOMWrapper): |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ |
+2011-04-26 Sam Weinig <sam@webkit.org> |
+ |
+ Fix build. |
+ |
+ * bindings/js/JSDOMWrapper.h: |
+ (WebCore::JSDOMWrapper::globalObject): |
+ |
+2011-04-26 Sam Weinig <sam@webkit.org> |
+ |
+ Roll r84945 back in with infinite recursion removed. |
+ |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::DOMConstructorObject::DOMConstructorObject): |
+ * bindings/js/JSDOMWrapper.h: |
+ (WebCore::JSDOMWrapper::globalObject): |
+ (WebCore::JSDOMWrapper::scriptExecutionContext): |
+ (WebCore::JSDOMWrapper::createStructure): |
+ (WebCore::JSDOMWrapper::JSDOMWrapper): |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ |
+2011-04-26 Sam Weinig <sam@webkit.org> |
+ |
+ Rollout previous patch. It broke stuff. |
+ |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::JSDOMWrapperWithGlobalPointer::globalObject): |
+ (WebCore::JSDOMWrapperWithGlobalPointer::scriptExecutionContext): |
+ (WebCore::JSDOMWrapperWithGlobalPointer::createStructure): |
+ (WebCore::JSDOMWrapperWithGlobalPointer::JSDOMWrapperWithGlobalPointer): |
+ (WebCore::DOMConstructorObject::DOMConstructorObject): |
+ * bindings/js/JSDOMWrapper.h: |
+ (WebCore::JSDOMWrapper::JSDOMWrapper): |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ |
+2011-04-26 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ Remove JSDOMWrapperWithGlobalPointer now that all JSDOMWrappers have global objects |
+ https://bugs.webkit.org/show_bug.cgi?id=59310 |
+ |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::DOMConstructorObject::DOMConstructorObject): |
+ * bindings/js/JSDOMWrapper.h: |
+ (WebCore::JSDOMWrapper::globalObject): |
+ (WebCore::JSDOMWrapper::scriptExecutionContext): |
+ (WebCore::JSDOMWrapper::createStructure): |
+ (WebCore::JSDOMWrapper::JSDOMWrapper): |
+ Merge JSDOMWrapperWithGlobalPointer with JSDOMWrapper. |
+ |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ Make JSDOMWrapper the base class. |
+ |
+2011-04-26 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Mark Rowe. |
+ |
+ Choose the compiler based on the Xcode version for Snow Leopard debug builds. |
+ |
+ * Configurations/Base.xcconfig: |
+ * Configurations/CompilerVersion.xcconfig: Added. |
+ |
+2011-04-26 Adrienne Walker <enne@google.com> |
+ |
+ Reviewed by Geoffrey Garen. |
+ |
+ Use OwnArrayPtr<T> instead of OwnPtr<T*> in ScopeTracer |
+ https://bugs.webkit.org/show_bug.cgi?id=59469 |
+ |
+ * platform/chromium/TraceEvent.h: |
+ (WebCore::internal::ScopeTracer::ScopeTracer): |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Nixed special finalizer handling for WebCore strings |
+ https://bugs.webkit.org/show_bug.cgi?id=59425 |
+ |
+ Not needed anymore, since weak handles have finalizers. |
+ |
+ * WebCore.exp.in: Exports! |
+ |
+ * bindings/js/DOMWrapperWorld.cpp: |
+ (WebCore::JSStringOwner::finalize): |
+ (WebCore::DOMWrapperWorld::DOMWrapperWorld): Use a weak handle finalizer, |
+ so we don't need special treatment anymore. |
+ |
+ * bindings/js/DOMWrapperWorld.h: |
+ (WebCore::JSStringOwner::JSStringOwner): |
+ (WebCore::DOMWrapperWorld::stringWrapperOwner): Use a HashMap of Weak<T> |
+ instead of a WeakGCMap, so we can specify a custom finalizer. |
+ |
+ * bindings/js/JSDOMBinding.cpp: |
+ (WebCore::jsStringSlowCase): |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::jsString): Updated for string map change. |
+ |
+2011-04-26 David Kilzer <ddkilzer@apple.com> |
+ |
+ <http://webkit.org/b/59372> Restore user default for enabling HTTP pipelining |
+ |
+ Reviewed by Antti Koivisto. |
+ |
+ To enable HTTP pipelining post-SnowLeopard, use this command: |
+ |
+ defaults write BUNDLE.ID WebKitEnableHTTPPipelining -bool YES |
+ |
+ * platform/network/cf/ResourceRequestCFNet.cpp: |
+ (WebCore::readBooleanPreference): Re-added. Originally added in |
+ r76756 and removed in r84120. |
+ (WebCore::initializeMaximumHTTPConnectionCountPerHost): Enable |
+ HTTP pipelining if use default is set to true. Added macro |
+ guards to match those in ResourceRequestMac.mm. |
+ |
+2011-04-26 Antti Koivisto <antti@apple.com> |
+ |
+ Reviewed by Adele Peterson. |
+ |
+ Don't strip fragments from file: urls in memory cache |
+ https://bugs.webkit.org/show_bug.cgi?id=59463 |
+ <rdar://problem/9231130> |
+ |
+ Some clients expect file: resources to be unique based on fragment identifier. |
+ Limit the changes in http://trac.webkit.org/changeset/74107 to the HTTP protocol only |
+ |
+ Test: fast/loader/file-protocol-fragment.html |
+ |
+ * loader/cache/MemoryCache.cpp: |
+ (WebCore::MemoryCache::removeFragmentIdentifierIfNeeded): |
+ |
+2011-04-26 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Fix more strict OwnPtr in WebCore (round N). |
+ https://bugs.webkit.org/show_bug.cgi?id=59464 |
+ |
+ * dom/Document.cpp: |
+ (WebCore::Document::Document): |
+ (WebCore::Document::attach): |
+ * fileapi/FileReader.cpp: |
+ (WebCore::FileReader::terminate): |
+ * loader/DocumentThreadableLoader.cpp: |
+ (WebCore::DocumentThreadableLoader::preflightFailure): |
+ * workers/DefaultSharedWorkerRepository.cpp: |
+ (WebCore::SharedWorkerConnectTask::create): |
+ (WebCore::SharedWorkerScriptLoader::load): |
+ |
+2011-04-26 Ruben <chromium@hybridsource.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ Change linux ifdefs for Chromium to Unix instead, allowing |
+ new Unix ports to be enabled by default |
+ https://bugs.webkit.org/show_bug.cgi?id=59297 |
+ |
+ No new tests, just changing preprocessor defines. |
+ |
+ * bindings/js/JSInspectorFrontendHostCustom.cpp: |
+ * bindings/v8/custom/V8InspectorFrontendHostCustom.cpp: |
+ * dom/SelectElement.cpp: |
+ * loader/cache/CachedFont.cpp: |
+ * page/EventHandler.cpp: |
+ * page/chromium/EventHandlerChromium.cpp: |
+ * platform/Scrollbar.cpp: |
+ * platform/chromium/PlatformBridge.h: |
+ * platform/graphics/chromium/FontPlatformData.h: |
+ * platform/graphics/skia/FontCustomPlatformData.cpp: |
+ * platform/graphics/skia/FontCustomPlatformData.h: |
+ |
+2011-04-26 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ [RTL] Arabic/AB - after typing a date, cursors doesn't go back |
+ https://bugs.webkit.org/show_bug.cgi?id=49111 |
+ |
+ Even when the offset corresponds to a position visually left of the box |
+ and there aren't any inline boxes on the left, the previous/next position |
+ may still correspond to some position in the same line. |
+ |
+ The bug was caused by our ignoring such cases. Fixed the bug by comparing |
+ previous/next position's inline box to the current box. If they match, |
+ then we stay on the same position because moving to the left visually at |
+ the left edge should not result in a position on the same line. |
+ |
+ Also fixed a bug that WebKit uses offsets that are not extrema when moved to |
+ the left edge or to the right edge, and a bug that WebKit could not move to |
+ the left from 12^3 CBA abc to 123 C^BA abc (there is no offset between 3 and C). |
+ |
+ Test cases are added to editing/selection/move-left-right.html |
+ |
+ * editing/VisiblePosition.cpp: |
+ (WebCore::VisiblePosition::leftVisuallyDistinctCandidate): |
+ (WebCore::VisiblePosition::rightVisuallyDistinctCandidate): |
+ |
+2011-04-26 Gabor Loki <loki@webkit.org> |
+ |
+ Reviewed by Csaba Osztrogonác. |
+ |
+ Speeding up SVG filters with multicore (SMP) support |
+ https://bugs.webkit.org/show_bug.cgi?id=43903 |
+ |
+ Some SVG filters execute a huge number of pixel manipulations, which |
+ cannot be sped up by graphics accelerators, since their algorithm is |
+ too complex. Using the power of Symmetric Multi Processing (SMP) we |
+ can split up a task to smaller (data independent) tasks, which can be |
+ executed independently. |
+ |
+ The ParallelJobs framework provides a simple way for distributed |
+ programming. The framework is based on WebKit's threading infrastructure, |
+ Open Multi-Processing's (OpenMP) API, and libdispatch API. |
+ |
+ * ForwardingHeaders/wtf/ParallelJobs.h: Added. |
+ * platform/graphics/filters/FETurbulence.cpp: |
+ (WebCore::FETurbulence::PaintingData::PaintingData): |
+ (WebCore::FETurbulence::noise2D): |
+ (WebCore::FETurbulence::calculateTurbulenceValueForPoint): |
+ (WebCore::FETurbulence::fillRegion): |
+ (WebCore::FETurbulence::fillRegionWorker): |
+ (WebCore::FETurbulence::apply): |
+ * platform/graphics/filters/FETurbulence.h: |
+ |
+2011-04-26 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: close button is off in docked mode on a mac. |
+ https://bugs.webkit.org/show_bug.cgi?id=59453 |
+ |
+ * inspector/front-end/inspector.css: |
+ (.toolbar-item.close-left): |
+ |
+2011-04-26 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: New Style is not working (UI glitch) |
+ https://bugs.webkit.org/show_bug.cgi?id=59451 |
+ |
+ * inspector/InspectorDOMStorageAgent.cpp: |
+ (WebCore::InspectorDOMStorageAgent::setFrontend): |
+ * inspector/front-end/AuditRules.js: |
+ (WebInspector.AuditRules.ImageDimensionsRule.prototype.doRun.getStyles): |
+ (WebInspector.AuditRules.CssInHeadRule.prototype.doRun.externalStylesheetsReceived): |
+ (WebInspector.AuditRules.CssInHeadRule.prototype.doRun.inlineStylesReceived): |
+ (WebInspector.AuditRules.StylesScriptsOrderRule.prototype.doRun.cssBeforeInlineReceived): |
+ (WebInspector.AuditRules.StylesScriptsOrderRule.prototype.doRun.lateStylesReceived): |
+ * inspector/front-end/CSSStyleModel.js: |
+ (WebInspector.CSSStyleModel.prototype.setRuleSelector): |
+ (WebInspector.CSSStyleModel.prototype.setRuleSelector.callback): |
+ (WebInspector.CSSStyleModel.prototype.addRule): |
+ (WebInspector.CSSStyleModel.prototype.addRule.callback): |
+ (WebInspector.CSSStyleModel.prototype._documentElementId): |
+ * inspector/front-end/DOMAgent.js: |
+ (WebInspector.DOMNode.prototype._addAttribute): |
+ (WebInspector.DOMNode.prototype.documentElement): |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.PageDispatcher.prototype.frameDetached): |
+ * inspector/front-end/StylesSidebarPane.js: |
+ (WebInspector.StylesSidebarPane.prototype._createNewRule): |
+ |
+2011-04-26 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: introduce DOMStorage::enable and disable. |
+ https://bugs.webkit.org/show_bug.cgi?id=59445 |
+ |
+ * inspector/Inspector.json: |
+ * inspector/InspectorAgent.cpp: |
+ (WebCore::InspectorAgent::InspectorAgent): |
+ (WebCore::InspectorAgent::restoreInspectorStateFromCookie): |
+ * inspector/InspectorDOMStorageAgent.cpp: |
+ (WebCore::InspectorDOMStorageAgent::InspectorDOMStorageAgent): |
+ (WebCore::InspectorDOMStorageAgent::restore): |
+ (WebCore::InspectorDOMStorageAgent::enable): |
+ (WebCore::InspectorDOMStorageAgent::disable): |
+ * inspector/InspectorDOMStorageAgent.h: |
+ (WebCore::InspectorDOMStorageAgent::create): |
+ * inspector/front-end/DOMStorage.js: |
+ (WebInspector.DOMStorageDispatcher.prototype.addDOMStorage): |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype._reset): |
+ * inspector/front-end/inspector.js: |
+ |
+2011-04-26 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: extract Page agent dispatcher into the ResourceTreeModel. |
+ https://bugs.webkit.org/show_bug.cgi?id=59440 |
+ |
+ * inspector/Inspector.json: |
+ * inspector/InspectorAgent.cpp: |
+ (WebCore::InspectorAgent::restoreInspectorStateFromCookie): |
+ * inspector/InspectorPageAgent.cpp: |
+ (WebCore::InspectorPageAgent::setFrontend): |
+ (WebCore::InspectorPageAgent::frameNavigated): |
+ * inspector/InspectorPageAgent.h: |
+ * inspector/front-end/AuditsPanel.js: |
+ (WebInspector.AuditsPanel): |
+ (WebInspector.AuditsPanel.prototype._onLoadEventFired): |
+ (WebInspector.AuditsPanel.prototype._domContentLoadedEventFired): |
+ * inspector/front-end/CSSStyleModel.js: |
+ (WebInspector.CSSStyleModelResourceBinding): |
+ (WebInspector.CSSStyleModelResourceBinding.prototype._inspectedURLChanged): |
+ * inspector/front-end/DOMBreakpointsSidebarPane.js: |
+ (WebInspector.DOMBreakpointsSidebarPane): |
+ (WebInspector.DOMBreakpointsSidebarPane.prototype._inspectedURLChanged): |
+ * inspector/front-end/DOMStorage.js: |
+ (WebInspector.DOMStorageDispatcher.prototype.addDOMStorage): |
+ * inspector/front-end/ExtensionServer.js: |
+ (WebInspector.ExtensionServer.prototype._inspectedURLChanged): |
+ (WebInspector.ExtensionServer.prototype.initExtensions): |
+ * inspector/front-end/NetworkPanel.js: |
+ (WebInspector.NetworkPanel): |
+ (WebInspector.NetworkPanel.prototype._onLoadEventFired): |
+ (WebInspector.NetworkPanel.prototype._domContentLoadedEventFired): |
+ (WebInspector.NetworkPanel.prototype._frameNavigated): |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.ResourceTreeModel): |
+ (WebInspector.ResourceTreeModel.prototype._processCachedResources): |
+ (WebInspector.ResourceTreeModel.prototype._dispatchInspectedURLChanged): |
+ (WebInspector.ResourceTreeModel.prototype._frameNavigated): |
+ (WebInspector.ResourceTreeModel.prototype._frameDetached): |
+ (WebInspector.ResourceTreeModel.prototype._clearChildFramesAndResources): |
+ (WebInspector.PageDispatcher): |
+ (WebInspector.PageDispatcher.prototype.domContentEventFired): |
+ (WebInspector.PageDispatcher.prototype.loadEventFired): |
+ (WebInspector.PageDispatcher.prototype.frameNavigated): |
+ (WebInspector.PageDispatcher.prototype.frameDetached): |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel): |
+ (WebInspector.ResourcesPanel.prototype._onLoadEventFired): |
+ (WebInspector.ResourcesPanel.prototype._frameNavigated): |
+ * inspector/front-end/inspector.js: |
+ |
+2011-04-26 Csaba Osztrogonác <ossy@webkit.org> |
+ |
+ Unreviewed rollout r84877 and StyleRareInheritedData.cpp changes of r84892, |
+ because it broke transitions/multiple-text-shadow-transition.html |
+ |
+ Fix OwnPtr strict errors in RenderStyle and make StyleRareInheritedData::textShadow an OwnPtr |
+ https://bugs.webkit.org/show_bug.cgi?id=59377 |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ * page/animation/AnimationBase.cpp: |
+ (WebCore::PropertyWrapperShadow::PropertyWrapperShadow): |
+ (WebCore::PropertyWrapperShadow::blend): |
+ * rendering/style/RenderStyle.cpp: |
+ (WebCore::RenderStyle::setTextShadow): |
+ (WebCore::RenderStyle::setBoxShadow): |
+ * rendering/style/RenderStyle.h: |
+ (WebCore::InheritedFlags::textShadow): |
+ * rendering/style/StyleRareInheritedData.cpp: |
+ (WebCore::StyleRareInheritedData::StyleRareInheritedData): |
+ (WebCore::StyleRareInheritedData::~StyleRareInheritedData): |
+ * rendering/style/StyleRareInheritedData.h: |
+ * rendering/style/StyleRareNonInheritedData.cpp: |
+ (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): |
+ |
+2011-04-26 Mihai Parparita <mihaip@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Turn off make built-in implicit rules for derived sources makefile |
+ https://bugs.webkit.org/show_bug.cgi?id=59418 |
+ |
+ We don't use any of make's built-in implicit rules, turning them off |
+ speeds up parsing of the makefile. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * gyp/generate-derived-sources.sh: |
+ |
+2011-04-26 Leo Yang <leo.yang@torchmobile.com.cn> |
+ |
+ Reviewed by Nikolas Zimmermann. |
+ |
+ ASSERT failure in SVGUseElement |
+ https://bugs.webkit.org/show_bug.cgi?id=59313 |
+ |
+ In SVGUseElement::insertedIntoDocument(), ASSERT(!m_isPendingResource) |
+ was wrong because the document may not be well-formed. |
+ |
+ This patch asserts the element is not pending on resource or the |
+ document is not well-formed. |
+ |
+ Test: svg/custom/use-crash-in-non-wellformed-document.svg |
+ |
+ * svg/SVGUseElement.cpp: |
+ (WebCore::isWellFormedDocument): |
+ (WebCore::SVGUseElement::insertedIntoDocument): |
+ |
+2011-04-26 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [Chromium] Fix columns resizing in the detailed heap snapshot grids. |
+ https://bugs.webkit.org/show_bug.cgi?id=59438 |
+ |
+ * inspector/front-end/DetailedHeapshotView.js: |
+ (WebInspector.DetailedHeapshotView.prototype._changeView): |
+ |
+2011-04-26 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix a few OwnPtr strict errors in WebCore headers. |
+ https://bugs.webkit.org/show_bug.cgi?id=59431 |
+ |
+ * bindings/js/ScheduledAction.cpp: |
+ (WebCore::ScheduledAction::create): |
+ * css/MediaQueryMatcher.cpp: |
+ (WebCore::MediaQueryMatcher::prepareEvaluator): |
+ * dom/NodeRareData.h: |
+ (WebCore::NodeListsNodeData::create): |
+ * dom/ScriptRunner.h: |
+ (WebCore::ScriptRunner::create): |
+ * inspector/WorkerInspectorController.cpp: |
+ (WebCore::WorkerInspectorController::connectFrontend): |
+ * loader/icon/IconDatabase.h: |
+ (WebCore::IconDatabase::create): |
+ * platform/graphics/SimpleFontData.cpp: |
+ (WebCore::SimpleFontData::verticalRightOrientationFontData): |
+ (WebCore::SimpleFontData::uprightOrientationFontData): |
+ (WebCore::SimpleFontData::brokenIdeographFontData): |
+ * rendering/style/StyleRareInheritedData.cpp: |
+ (WebCore::StyleRareInheritedData::StyleRareInheritedData): |
+ * rendering/svg/SVGResources.h: |
+ (WebCore::SVGResources::ClipperFilterMaskerData::create): |
+ (WebCore::SVGResources::MarkerData::create): |
+ (WebCore::SVGResources::FillStrokeData::create): |
+ * storage/StorageSyncManager.cpp: |
+ (WebCore::StorageSyncManager::close): |
+ * workers/WorkerContext.cpp: |
+ (WebCore::CloseWorkerContextTask::create): |
+ * workers/WorkerMessagingProxy.cpp: |
+ (WebCore::MessageWorkerContextTask::create): |
+ (WebCore::MessageWorkerTask::create): |
+ (WebCore::WorkerExceptionTask::create): |
+ (WebCore::WorkerContextDestroyedTask::create): |
+ (WebCore::WorkerTerminateTask::create): |
+ (WebCore::WorkerThreadActivityReportTask::create): |
+ * workers/WorkerRunLoop.cpp: |
+ (WebCore::WorkerRunLoop::Task::create): |
+ * workers/WorkerThread.cpp: |
+ (WebCore::WorkerThreadStartupData::create): |
+ (WebCore::WorkerThreadShutdownFinishTask::create): |
+ (WebCore::WorkerThreadShutdownStartTask::create): |
+ |
+2011-04-26 Adam Klein <adamk@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix last strict OwnPtr violation under svg/... |
+ https://bugs.webkit.org/show_bug.cgi?id=59429 |
+ |
+ * svg/SVGDocumentExtensions.cpp: |
+ (WebCore::SVGDocumentExtensions::removePendingResource): |
+ |
+2011-04-26 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix more strict OwnPtr violations in WebCore |
+ https://bugs.webkit.org/show_bug.cgi?id=59433 |
+ |
+ These manifest in the Chromium build. |
+ |
+ * bindings/v8/ScriptController.cpp: |
+ (WebCore::ScriptController::ScriptController): |
+ * bindings/v8/ScriptDebugServer.cpp: |
+ (WebCore::ScriptDebugServer::editScriptSource): |
+ * bindings/v8/V8DOMMap.cpp: |
+ (WebCore::DOMDataStoreHandle::DOMDataStoreHandle): |
+ * bindings/v8/V8Proxy.cpp: |
+ (WebCore::V8Proxy::precompileScript): |
+ * bindings/v8/WorkerScriptController.cpp: |
+ (WebCore::WorkerScriptController::WorkerScriptController): |
+ * page/Page.cpp: |
+ (WebCore::Page::Page): |
+ (WebCore::Page::speechInput): |
+ * platform/PlatformGestureRecognizer.cpp: |
+ (WebCore::PlatformGestureRecognizer::create): |
+ * platform/PlatformGestureRecognizer.h: |
+ * platform/graphics/chromium/DrawingBufferChromium.cpp: |
+ (WebCore::DrawingBuffer::DrawingBuffer): |
+ * platform/graphics/chromium/cc/CCLayerImpl.cpp: |
+ (WebCore::CCLayerImpl::CCLayerImpl): |
+ * platform/graphics/gpu/Texture.cpp: |
+ (WebCore::Texture::create): |
+ * platform/image-decoders/bmp/BMPImageDecoder.cpp: |
+ (WebCore::BMPImageDecoder::decodeHelper): |
+ * platform/image-decoders/gif/GIFImageDecoder.cpp: |
+ (WebCore::GIFImageDecoder::decode): |
+ * platform/image-decoders/ico/ICOImageDecoder.cpp: |
+ (WebCore::ICOImageDecoder::decodeAtIndex): |
+ * platform/image-decoders/jpeg/JPEGImageDecoder.cpp: |
+ (WebCore::JPEGImageDecoder::decode): |
+ * platform/image-decoders/png/PNGImageDecoder.cpp: |
+ (WebCore::PNGImageDecoder::decode): |
+ * storage/chromium/DatabaseTrackerChromium.cpp: |
+ (WebCore::DatabaseTracker::addOpenDatabase): |
+ |
+2011-04-25 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix OwnPtr strict errors in CrossThreadTask.h |
+ https://bugs.webkit.org/show_bug.cgi?id=59427 |
+ |
+ * dom/CrossThreadTask.h: |
+ (WebCore::CrossThreadTask1::create): |
+ (WebCore::CrossThreadTask2::create): |
+ (WebCore::CrossThreadTask3::create): |
+ (WebCore::CrossThreadTask4::create): |
+ (WebCore::CrossThreadTask5::create): |
+ (WebCore::CrossThreadTask6::create): |
+ (WebCore::CrossThreadTask7::create): |
+ (WebCore::CrossThreadTask8::create): |
+ |
+2011-04-25 Mark Rowe <mrowe@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ <rdar://problem/9330337> Leak of 'WebCore::ApplyPropertyFillLayer<WebCore::FillSize>' |
+ |
+ * css/CSSStyleApplyProperty.cpp: |
+ (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): Don't initialize CSSPropertyWebkitMaskSize twice. |
+ * css/CSSStyleApplyProperty.h: |
+ (WebCore::CSSStyleApplyProperty::setPropertyValue): Add some asserts to catch this class of leak. |
+ |
+2011-04-25 Levi Weintraub <leviw@chromium.org> |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ REGRESSION: a character appears after tab span when typing immediately before the tab span |
+ https://bugs.webkit.org/show_bug.cgi?id=58132 |
+ |
+ Correcting the order of operations we use to avoid inserting into invalid positions in |
+ ReplaceSelectionCommand to avoid inserting into tab-spans. |
+ |
+ Test: editing/pasteboard/paste-before-tab-span.html |
+ |
+ * editing/ReplaceSelectionCommand.cpp: |
+ (WebCore::ReplaceSelectionCommand::doApply): Moving positionOutsideTabSpan to after |
+ positionAvoidingPrecedingNodes, since that function could once again put our position |
+ inside a tab span. |
+ |
+2011-04-25 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix OwnPtr strict error in ImageBuffer.h |
+ https://bugs.webkit.org/show_bug.cgi?id=59422 |
+ |
+ * platform/graphics/ImageBuffer.h: |
+ (WebCore::ImageBuffer::create): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix strict OwnPtr violations and whitelist known-tricky files |
+ https://bugs.webkit.org/show_bug.cgi?id=59421 |
+ |
+ With this patch, WebCore builds cleanly with strict OwnPtrs. |
+ |
+ * dom/MessagePort.cpp: |
+ (WebCore::MessagePort::MessagePort): |
+ * dom/XMLDocumentParserLibxml2.cpp: |
+ * page/ContextMenuController.cpp: |
+ (WebCore::ContextMenuController::ContextMenuController): |
+ (WebCore::ContextMenuController::clearContextMenu): |
+ * page/PluginHalter.cpp: |
+ * platform/graphics/MediaPlayer.cpp: |
+ * platform/network/ResourceRequestBase.cpp: |
+ (WebCore::ResourceRequestBase::adopt): |
+ (WebCore::ResourceRequestBase::copyData): |
+ * svg/SVGDocumentExtensions.cpp: |
+ |
+2011-04-25 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix OwnPtr strict errors in DatasetDOMStringMap.h. |
+ https://bugs.webkit.org/show_bug.cgi?id=59419 |
+ |
+ * dom/DatasetDOMStringMap.h: |
+ (WebCore::DatasetDOMStringMap::create): |
+ |
+2011-04-25 Jocelyn Turcotte <jocelyn.turcotte@nokia.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ KeyboardEvent.cpp should work with strict OwnPtrs. |
+ https://bugs.webkit.org/show_bug.cgi?id=59403 |
+ |
+ * dom/KeyboardEvent.cpp: |
+ (WebCore::KeyboardEvent::KeyboardEvent): |
+ |
+2011-04-25 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Fix OwnPtr strict errors in RenderStyle and make StyleRareInheritedData::textShadow an OwnPtr |
+ https://bugs.webkit.org/show_bug.cgi?id=59377 |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ * page/animation/AnimationBase.cpp: |
+ (WebCore::PropertyWrapperShadow::PropertyWrapperShadow): |
+ (WebCore::PropertyWrapperShadow::blend): |
+ * rendering/style/RenderStyle.cpp: |
+ (WebCore::RenderStyle::setTextShadow): |
+ (WebCore::RenderStyle::setBoxShadow): |
+ * rendering/style/RenderStyle.h: |
+ (WebCore::InheritedFlags::textShadow): |
+ * rendering/style/StyleRareInheritedData.cpp: |
+ (WebCore::StyleRareInheritedData::StyleRareInheritedData): |
+ (WebCore::StyleRareInheritedData::~StyleRareInheritedData): |
+ * rendering/style/StyleRareInheritedData.h: |
+ * rendering/style/StyleRareNonInheritedData.cpp: |
+ (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): |
+ |
+2011-04-25 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r84864. |
+ http://trac.webkit.org/changeset/84864 |
+ https://bugs.webkit.org/show_bug.cgi?id=59413 |
+ |
+ Expect result of use-crash-in-non-wellformed-document.svg is |
+ platform dependent (Requested by leoyang on #webkit). |
+ |
+ * svg/SVGUseElement.cpp: |
+ (WebCore::SVGUseElement::insertedIntoDocument): |
+ |
+2011-04-25 Leo Yang <leo.yang@torchmobile.com.cn> |
+ |
+ Reviewed by Nikolas Zimmermann. |
+ |
+ ASSERT failure in SVGUseElement |
+ https://bugs.webkit.org/show_bug.cgi?id=59313 |
+ |
+ In SVGUseElement::insertedIntoDocument(), ASSERT(!m_isPendingResource) |
+ was wrong because the document may not be well-formed. |
+ |
+ This patch asserts the element is not pending on resource or the |
+ document is not well-formed. |
+ |
+ Test: svg/custom/use-crash-in-non-wellformed-document.svg |
+ |
+ * svg/SVGUseElement.cpp: |
+ (WebCore::isWellFormedDocument): |
+ (WebCore::SVGUseElement::insertedIntoDocument): |
+ |
+2011-04-25 Igor Oliveira <igor.oliveira@openbossa.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ Move complexity from DataTransferItemsChromium and DataTransferItemChromium for base class |
+ https://bugs.webkit.org/show_bug.cgi?id=59028 |
+ |
+ The DataTransferItemsChromium and DataTransferItemChromium code can be reused by other platforms. |
+ Moving the code for base class we can reduce the efforts to support DataTransferItems |
+ element in different platforms. |
+ |
+ * Android.mk: |
+ * CMakeLists.txt: |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * dom/DataTransferItem.cpp: |
+ (WebCore::DataTransferItem::DataTransferItem): |
+ (WebCore::DataTransferItem::owner): |
+ (WebCore::DataTransferItem::kind): |
+ (WebCore::DataTransferItem::type): |
+ * dom/DataTransferItem.h: |
+ * dom/DataTransferItems.cpp: Copied from Source/WebCore/platform/chromium/DataTransferItemsChromium.cpp. |
+ (WebCore::DataTransferItems::DataTransferItems): |
+ (WebCore::DataTransferItems::length): |
+ (WebCore::DataTransferItems::item): |
+ (WebCore::DataTransferItems::deleteItem): |
+ (WebCore::DataTransferItems::clear): |
+ (WebCore::DataTransferItems::add): |
+ * dom/DataTransferItems.h: |
+ * platform/chromium/DataTransferItemChromium.cpp: |
+ (WebCore::DataTransferItem::create): |
+ (WebCore::DataTransferItemChromium::DataTransferItemChromium): |
+ (WebCore::DataTransferItemChromium::getAsString): |
+ (WebCore::DataTransferItemChromium::getAsFile): |
+ * platform/chromium/DataTransferItemChromium.h: |
+ * platform/chromium/DataTransferItemsChromium.cpp: |
+ (WebCore::DataTransferItemsChromium::DataTransferItemsChromium): |
+ * platform/chromium/DataTransferItemsChromium.h: |
+ |
+2011-04-25 Chris Marrin <cmarrin@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ REGRESSION(75137): directly composited images are double-drawn sometimes |
+ https://bugs.webkit.org/show_bug.cgi?id=58632 |
+ |
+ Depending on the order of operations (layout, creating compositing |
+ layers and creating simple image layers) a layer can have both image |
+ content and have m_drawsContent=true. In this case 2 copies of the image |
+ are drawn in the layer. If the image has alpha, it looks wrong. |
+ |
+ I added an updateDrawsContent() call after setting the image contents |
+ to make sure it's correct. |
+ |
+ I couldn't create a test case that shows the problem because layerTreeAsText() |
+ doesn't emit enough info to show this level of detail. I've opened |
+ https://bugs.webkit.org/show_bug.cgi?id=59352 to fix that. |
+ |
+ * rendering/RenderLayerBacking.cpp: |
+ (WebCore::RenderLayerBacking::updateImageContents): |
+ |
+2011-04-25 Kenneth Russell <kbr@google.com> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix OwnPtr strict errors in InlineFlowBox.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59392 |
+ |
+ * rendering/InlineFlowBox.cpp: |
+ (WebCore::InlineFlowBox::setLayoutOverflow): |
+ (WebCore::InlineFlowBox::setVisualOverflow): |
+ |
+2011-04-25 Kenneth Russell <kbr@google.com> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix OwnPtr strict errors in TransformState.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59380 |
+ |
+ * rendering/TransformState.cpp: |
+ (WebCore::TransformState::applyTransform): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ DOMWindow.cpp should work with strict OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59356 |
+ |
+ * page/DOMWindow.cpp: |
+ (WebCore::PostMessageTimer::fired): |
+ (WebCore::DOMWindow::postMessageTimerFired): |
+ * page/DOMWindow.h: |
+ |
+2011-04-25 Adam Klein <adamk@chromium.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ Fix OwnPtr strict errors in some SVG classes. |
+ https://bugs.webkit.org/show_bug.cgi?id=59395 |
+ |
+ * rendering/style/SVGRenderStyleDefs.cpp: |
+ (WebCore::StyleShadowSVGData::StyleShadowSVGData): |
+ * svg/SVGSVGElement.cpp: |
+ (WebCore::SVGSVGElement::SVGSVGElement): |
+ * svg/SVGStyledTransformableElement.cpp: |
+ (WebCore::SVGStyledTransformableElement::supplementalTransform): |
+ * svg/SVGTextElement.cpp: |
+ (WebCore::SVGTextElement::supplementalTransform): |
+ * svg/graphics/SVGImage.cpp: |
+ (WebCore::SVGImage::dataChanged): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Re-land changes to ValidationMessage. |
+ |
+ * html/ValidationMessage.cpp: |
+ (WebCore::ValidationMessage::setMessage): |
+ (WebCore::ValidationMessage::setMessageDOMAndStartTimer): |
+ (WebCore::ValidationMessage::requestToHideMessage): |
+ |
+2011-04-25 Ojan Vafai <ojan@chromium.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ fix OwnPtr strict error in FrameView.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59402 |
+ |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::FrameView): |
+ (WebCore::FrameView::addWidgetToUpdate): |
+ |
+2011-04-25 Adam Klein <adamk@chromium.org> |
+ |
+ Unreviewed, rolling out r84844. |
+ http://trac.webkit.org/changeset/84844 |
+ https://bugs.webkit.org/show_bug.cgi?id=59395 |
+ |
+ Checked in too much. |
+ |
+ * rendering/style/SVGRenderStyleDefs.cpp: |
+ (WebCore::StyleShadowSVGData::StyleShadowSVGData): |
+ * svg/SVGDocumentExtensions.cpp: |
+ (WebCore::SVGDocumentExtensions::removePendingResource): |
+ * svg/SVGSVGElement.cpp: |
+ (WebCore::SVGSVGElement::SVGSVGElement): |
+ * svg/SVGStyledTransformableElement.cpp: |
+ (WebCore::SVGStyledTransformableElement::supplementalTransform): |
+ * svg/SVGTextElement.cpp: |
+ (WebCore::SVGTextElement::supplementalTransform): |
+ * svg/graphics/SVGImage.cpp: |
+ (WebCore::SVGImage::dataChanged): |
+ |
+2011-04-25 Adrienne Walker <enne@google.com> |
+ |
+ Reviewed by David Levin. |
+ |
+ Make ActiveDOMCallback play nice with OwnPtr strict mode |
+ https://bugs.webkit.org/show_bug.cgi?id=59398 |
+ |
+ * bindings/generic/ActiveDOMCallback.cpp: |
+ (WebCore::ActiveDOMCallback::ActiveDOMCallback): |
+ |
+2011-04-25 Adam Klein <adamk@chromium.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ Fix OwnPtr strict errors in some SVG classes. |
+ https://bugs.webkit.org/show_bug.cgi?id=59395 |
+ |
+ * rendering/style/SVGRenderStyleDefs.cpp: |
+ (WebCore::StyleShadowSVGData::StyleShadowSVGData): |
+ * svg/SVGSVGElement.cpp: |
+ (WebCore::SVGSVGElement::SVGSVGElement): |
+ * svg/SVGStyledTransformableElement.cpp: |
+ (WebCore::SVGStyledTransformableElement::supplementalTransform): |
+ * svg/SVGTextElement.cpp: |
+ (WebCore::SVGTextElement::supplementalTransform): |
+ * svg/graphics/SVGImage.cpp: |
+ (WebCore::SVGImage::dataChanged): |
+ |
+2011-04-25 James Robinson <jamesr@chromium.org> |
+ |
+ Unreviewed, rolling out r84835. |
+ http://trac.webkit.org/changeset/84835 |
+ https://bugs.webkit.org/show_bug.cgi?id=59394 |
+ |
+ Breaks compile because Deque<OwnPtr> doesn't work |
+ |
+ * dom/XMLDocumentParserLibxml2.cpp: |
+ (WebCore::PendingCallbacks::~PendingCallbacks): |
+ (WebCore::PendingCallbacks::appendStartElementNSCallback): |
+ (WebCore::PendingCallbacks::appendEndElementNSCallback): |
+ (WebCore::PendingCallbacks::appendCharactersCallback): |
+ (WebCore::PendingCallbacks::appendProcessingInstructionCallback): |
+ (WebCore::PendingCallbacks::appendCDATABlockCallback): |
+ (WebCore::PendingCallbacks::appendCommentCallback): |
+ (WebCore::PendingCallbacks::appendInternalSubsetCallback): |
+ (WebCore::PendingCallbacks::appendErrorCallback): |
+ (WebCore::XMLDocumentParser::XMLDocumentParser): |
+ * html/ValidationMessage.cpp: |
+ (WebCore::ValidationMessage::setMessage): |
+ (WebCore::ValidationMessage::setMessageDOMAndStartTimer): |
+ (WebCore::ValidationMessage::requestToHideMessage): |
+ |
+2011-04-25 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Fix OwnPtr strict mode violation in ImageBufferCG |
+ https://bugs.webkit.org/show_bug.cgi?id=59396 |
+ |
+ * platform/graphics/cg/ImageBufferCG.cpp: |
+ (WebCore::ImageBuffer::ImageBuffer): |
+ |
+2011-04-25 Huang Dongsung <luxtella@company100.net> |
+ |
+ Reviewed by Dirk Schulze. |
+ |
+ [Skia] Remove an unused local variable in PlatformContextSkia::readbackHardwareToSoftware(). |
+ https://bugs.webkit.org/show_bug.cgi?id=59308 |
+ |
+ An unused local variable is created and even allocates heap memory. |
+ |
+ * platform/graphics/skia/PlatformContextSkia.cpp: |
+ (WebCore::PlatformContextSkia::readbackHardwareToSoftware): |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Qt build fix: Updated files not used by other ports for rename. |
+ |
+ * bindings/js/JSTouchCustom.cpp: |
+ (WebCore::toJSNewlyCreated): |
+ * bindings/js/JSTouchListCustom.cpp: |
+ (WebCore::toJSNewlyCreated): |
+ |
+2011-04-25 Adrienne Walker <enne@google.com> |
+ |
+ Reviewed by David Levin. |
+ |
+ WebCore/css/CSS* files should play nice with strict OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59382 |
+ |
+ * css/CSSFontFaceSource.cpp: |
+ (WebCore::CSSFontFaceSource::getFontData): |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::addProperty): |
+ (WebCore::CSSParser::parseValue): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ ValidationMessage and XMLDocumentParserLibxml2 should play nice with strict OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59394 |
+ |
+ * dom/XMLDocumentParserLibxml2.cpp: |
+ (WebCore::PendingCallbacks::PendingCallbacks): |
+ (WebCore::PendingCallbacks::appendStartElementNSCallback): |
+ (WebCore::PendingCallbacks::appendEndElementNSCallback): |
+ (WebCore::PendingCallbacks::appendCharactersCallback): |
+ (WebCore::PendingCallbacks::appendProcessingInstructionCallback): |
+ (WebCore::PendingCallbacks::appendCDATABlockCallback): |
+ (WebCore::PendingCallbacks::appendCommentCallback): |
+ (WebCore::PendingCallbacks::appendInternalSubsetCallback): |
+ (WebCore::PendingCallbacks::appendErrorCallback): |
+ (WebCore::XMLDocumentParser::XMLDocumentParser): |
+ * html/ValidationMessage.cpp: |
+ (WebCore::ValidationMessage::setMessage): |
+ (WebCore::ValidationMessage::setMessageDOMAndStartTimer): |
+ (WebCore::ValidationMessage::requestToHideMessage): |
+ |
+2011-04-25 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix OwnPtr strictness issues in WebCore/inspector/ |
+ https://bugs.webkit.org/show_bug.cgi?id=59387 |
+ |
+ * inspector/InspectorAgent.cpp: |
+ (WebCore::InspectorAgent::InspectorAgent): |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::InspectorController): |
+ * inspector/InspectorStyleSheet.cpp: |
+ (WebCore::InspectorStyleSheet::ensureSourceData): |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Beth Dakin. |
+ |
+ Removed XPathNamespace because it was unused |
+ https://bugs.webkit.org/show_bug.cgi?id=59381 |
+ |
+ This allows us to make Node::ownerDocument() non-virtual, though it has |
+ the unhappy side-effect of rebuilding the world. |
+ |
+ * CMakeLists.txt: |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: Bye bye dead code. |
+ |
+ * dom/Node.h: Hello fast code! |
+ |
+ * xml/XPathNamespace.cpp: Removed. |
+ * xml/XPathNamespace.h: Removed. |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ HistoryItemMac should play nice with OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59388 |
+ |
+ * history/mac/HistoryItemMac.mm: |
+ (WebCore::HistoryItem::setTransientProperty): |
+ |
+2011-04-25 David Levin <levin@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ Fix strict OwnPtrs in ContextMenuController.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59375 |
+ |
+ * page/ContextMenuController.cpp: |
+ (WebCore::ContextMenuController::createContextMenu): |
+ (WebCore::separatorItem): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ CheckedRadioButton should play nice with strict OwnPtr |
+ https://bugs.webkit.org/show_bug.cgi?id=59386 |
+ |
+ * dom/CheckedRadioButtons.cpp: |
+ (WebCore::CheckedRadioButtons::addButton): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Resource*.cpp should play nice with strict OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59383 |
+ |
+ * bridge/runtime_method.cpp: |
+ (JSC::RuntimeMethod::RuntimeMethod): |
+ * platform/network/ResourceHandle.cpp: |
+ (WebCore::ResourceHandle::ResourceHandle): |
+ * platform/network/ResourceResponseBase.cpp: |
+ (WebCore::ResourceResponseBase::adopt): |
+ (WebCore::ResourceResponseBase::copyData): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Page*.cpp and objc_runtime.mm should play nice with strict OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59374 |
+ |
+ * bridge/objc/objc_runtime.mm: |
+ (JSC::Bindings::callObjCFallbackObject): |
+ * page/Page.cpp: |
+ (WebCore::Page::Page): |
+ (WebCore::Page::initGroup): |
+ * page/PageGroup.cpp: |
+ (WebCore::PageGroup::addUserScriptToWorld): |
+ (WebCore::PageGroup::addUserStyleSheetToWorld): |
+ * page/mac/PageMac.cpp: |
+ (WebCore::Page::addSchedulePair): |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Build fix: This time for sure! |
+ |
+ * dom/make_names.pl: |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Build fix: Let's edit the script instead of the autogenrated file. That |
+ way, our change will stand the test of time. |
+ |
+ * dom/make_names.pl: |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix strict OwnPtrs in HTMLMediaElement and HTTPHeaderMap |
+ https://bugs.webkit.org/show_bug.cgi?id=59368 |
+ |
+ * html/HTMLMediaElement.cpp: |
+ (WebCore::HTMLMediaElement::HTMLMediaElement): |
+ * platform/network/HTTPHeaderMap.cpp: |
+ (WebCore::HTTPHeaderMap::copyData): |
+ |
+2011-04-25 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix strict OwnPtr violations in Render*.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59361 |
+ |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::RenderBlock): |
+ (WebCore::RenderBlock::finishDelayUpdateScrollInfo): |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::addLayoutOverflow): |
+ (WebCore::RenderBox::addVisualOverflow): |
+ * rendering/RenderButton.cpp: |
+ (WebCore::RenderButton::styleDidChange): |
+ * rendering/RenderLayer.cpp: |
+ (WebCore::RenderLayer::updateTransform): |
+ (WebCore::RenderLayer::paintChildLayerIntoColumns): |
+ (WebCore::RenderLayer::hitTestChildLayerColumns): |
+ (WebCore::RenderLayer::ensureBacking): |
+ * rendering/RenderTable.cpp: |
+ (WebCore::RenderTable::styleDidChange): |
+ * rendering/RenderView.cpp: |
+ (WebCore::RenderView::compositor): |
+ * rendering/style/RenderStyle.cpp: |
+ (WebCore::RenderStyle::addCachedPseudoStyle): |
+ (WebCore::RenderStyle::accessCounterDirectives): |
+ (WebCore::RenderStyle::accessAnimations): |
+ (WebCore::RenderStyle::accessTransitions): |
+ * rendering/svg/RenderSVGResourceFilter.cpp: |
+ (WebCore::RenderSVGResourceFilter::applyResource): |
+ * rendering/svg/SVGResources.cpp: |
+ (WebCore::SVGResources::SVGResources): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Attempt to fix the compile. |
+ |
+ * platform/graphics/MediaPlayer.cpp: |
+ (WebCore::MediaPlayer::loadWithNextMediaEngine): |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Rubber-stamped by Beth Dakin. |
+ |
+ Merged CREATE_DOM_NODE_WRAPPER and CREATE_DOM_OBJECT_WRAPPER into |
+ CREATE_DOM_WRAPPER because there's no meaningful difference between |
+ them anymore. |
+ |
+ * bindings/js/JSCDATASectionCustom.cpp: |
+ (WebCore::toJSNewlyCreated): |
+ * bindings/js/JSCSSRuleCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSCSSValueCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSDOMBinding.h: |
+ * bindings/js/JSDocumentCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSElementCustom.cpp: |
+ (WebCore::toJSNewlyCreated): |
+ * bindings/js/JSEventCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSHTMLCollectionCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSImageDataCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSNodeCustom.cpp: |
+ (WebCore::createWrapperInline): |
+ * bindings/js/JSSVGPathSegCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSStyleSheetCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSTextCustom.cpp: |
+ (WebCore::toJSNewlyCreated): |
+ * bindings/js/JSWebKitCSSMatrixCustom.cpp: |
+ (WebCore::JSWebKitCSSMatrixConstructor::constructJSWebKitCSSMatrix): |
+ * bindings/js/JSWebSocketCustom.cpp: |
+ (WebCore::JSWebSocketConstructor::constructJSWebSocket): |
+ * bindings/js/JSXMLHttpRequestCustom.cpp: |
+ (WebCore::JSXMLHttpRequestConstructor::constructJSXMLHttpRequest): |
+ * bindings/js/JSXSLTProcessorCustom.cpp: |
+ (WebCore::JSXSLTProcessorConstructor::constructJSXSLTProcessor): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ HTMLDocumentParser should play nice with OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59363 |
+ |
+ * html/parser/HTMLDocumentParser.cpp: |
+ (WebCore::HTMLDocumentParser::pumpTokenizer): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ Frame.cpp should work with strict OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59360 |
+ |
+ * page/Frame.cpp: |
+ (WebCore::createRegExpForLabels): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix strict OwnPtr in Element.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59357 |
+ |
+ * dom/NodeRareData.h: |
+ (WebCore::NodeRareData::ensureEventTargetData): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix strict OwnPtr issues in Media*.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59354 |
+ |
+ There's still a tricky case in MediaPlayer.cpp, but we'll worry about |
+ that later. |
+ |
+ * css/MediaQueryMatcher.cpp: |
+ (WebCore::MediaQueryMatcher::addListener): |
+ * dom/Document.cpp: |
+ * platform/graphics/MediaPlayer.cpp: |
+ (WebCore::createNullMediaPlayer): |
+ |
+2011-04-25 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Fix strict OwnPtr violations in ListHashSet and RenderLayerCompositor |
+ https://bugs.webkit.org/show_bug.cgi?id=59353 |
+ |
+ * rendering/RenderLayerCompositor.cpp: |
+ (WebCore::RenderLayerCompositor::RenderLayerCompositor): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ Worker*.cpp should work with strict OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59346 |
+ |
+ * inspector/WorkerInspectorController.cpp: |
+ (WebCore::WorkerInspectorController::WorkerInspectorController): |
+ * workers/WorkerContext.cpp: |
+ (WebCore::WorkerContext::WorkerContext): |
+ * workers/WorkerRunLoop.cpp: |
+ (WebCore::WorkerRunLoop::WorkerRunLoop): |
+ * workers/WorkerScriptLoader.cpp: |
+ (WebCore::WorkerScriptLoader::createResourceRequest): |
+ |
+2011-04-25 James Robinson <jamesr@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Remove bad use of OwnPtr::set() in IconDatabase.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=59344 |
+ |
+ Needed for strict OwnPtr compliance. |
+ |
+ * loader/icon/IconDatabase.cpp: |
+ (WebCore::readySQLiteStatement): |
+ |
+2011-04-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ ImageBuffer.h should work with strict OwnPtrs |
+ https://bugs.webkit.org/show_bug.cgi?id=59341 |
+ |
+ This patch is to prepare for the strict OwnPtr hack-a-thon. |
+ |
+ * platform/graphics/ImageBuffer.h: |
+ (WebCore::ImageBuffer::create): |
+ |
+2011-04-25 Steve Falkenburg <sfalken@apple.com> |
+ |
+ Reviewed by Brian Weinstein. |
+ |
+ WebCore build on Windows should include windows.h in its precompiled header to speed builds |
+ https://bugs.webkit.org/show_bug.cgi?id=59339 |
+ |
+ windows.h is included by several of the underlying wtf headers, including Atomics.h and |
+ ThreadingPrimitives.h anyway, so include it in the WebCorePrefix.h to speed up compilation. |
+ |
+ On my 8 core Mac Pro, this reduces a clean rebuild of debug WebCore from 8 minutes to 7 minutes |
+ (a 12% speedup). |
+ |
+ * WebCorePrefix.h: Include windows.h |
+ |
+2011-04-25 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Xan Lopez. |
+ |
+ [GTK] Crash in WebCore::FrameView::notifyPageThatContentAreaWillPaint() |
+ https://bugs.webkit.org/show_bug.cgi?id=59311 |
+ |
+ * platform/gtk/MainFrameScrollbarGtk.cpp: |
+ (MainFrameScrollbarGtk::attachAdjustment): Before connecting an adjustment to a scrollbar |
+ disconnect any lingering signal handlers. This prevents an adjustment from controlling the |
+ active ScrollView and some zombie ScrollView. |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Another shot at fixing the EFL build. |
+ |
+ * UseJSC.cmake: |
+ * bindings/js/JSDOMBinding.cpp: |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Try to fix EFL build. |
+ |
+ * UseJSC.cmake: Added a missing file. |
+ |
+2011-04-25 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Removed a use of markDOMObjectWrapper: ActiveDOMObjects, Workers, and MessagePorts |
+ https://bugs.webkit.org/show_bug.cgi?id=59333 |
+ |
+ * WebCore.xcodeproj/project.pbxproj: Added .idl files for easier editing. |
+ |
+ * bindings/js/JSDOMBinding.cpp: |
+ * bindings/js/JSDOMBinding.h: |
+ * bindings/js/JSDocumentCustom.cpp: |
+ * bindings/js/JSWorkerContextCustom.cpp: |
+ (WebCore::JSWorkerContext::visitChildren): Removed now-unused functions. |
+ |
+ * bindings/scripts/CodeGeneratorJS.pm: Added support for ActiveDOMObject |
+ lifetime management. |
+ |
+ * dom/Document.idl: No need for a custom mark function anymore, since |
+ ActiveDOMObjets now manage their own lifetimes. |
+ |
+ * dom/MessagePort.cpp: |
+ (WebCore::MessagePort::hasPendingActivity): Correctly report that we |
+ have pending activity when we're remotely entangled, instead of relying |
+ on our clients to know this about us and do the check for us. |
+ |
+ * dom/MessagePort.h: FIXME! |
+ |
+ * dom/MessagePort.idl: |
+ * fileapi/DOMFileSystem.idl: |
+ * fileapi/FileReader.idl: |
+ * fileapi/FileWriter.idl: |
+ * notifications/Notification.idl: |
+ * notifications/NotificationCenter.idl: |
+ * page/EventSource.idl: |
+ * storage/IDBDatabase.idl: |
+ * storage/IDBRequest.idl: |
+ * storage/IDBTransaction.idl: |
+ * webaudio/AudioContext.idl: |
+ * websockets/WebSocket.idl: |
+ * workers/AbstractWorker.idl: |
+ * workers/SharedWorker.idl: |
+ * workers/Worker.idl: Opt in to ActiveDOMObject lifetime management. |
+ |
+ * workers/WorkerContext.cpp: |
+ (WebCore::WorkerContext::hasPendingActivity): No need to make up for |
+ MessagePort::hasPendingActivity being wrong anymore, since it's now right. |
+ |
+ * xml/XMLHttpRequest.idl: Opt in to ActiveDOMObject lifetime management. |
+ |
+2011-04-25 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: worker messages should be routed to corresponding worker front-end |
+ https://bugs.webkit.org/show_bug.cgi?id=59323 |
+ |
+ Added WorkerManager which is responsible for routing messages between |
+ Page inspector front-end and Worker inspector front-ends. |
+ |
+ * WebCore.gypi: |
+ * inspector/front-end/WebKit.qrc: |
+ * inspector/front-end/WorkerManager.js: Added. |
+ (WebInspector.WorkerManager): |
+ (WebInspector.WorkerManager.loaded.InspectorFrontendHost.sendMessageToBackend): |
+ (WebInspector.WorkerManager.loaded.InspectorFrontendHost.loaded): |
+ (WebInspector.WorkerManager.loaded): |
+ (WebInspector.WorkerFrontendStub): |
+ (WebInspector.WorkerFrontendStub.prototype._receiveMessage): |
+ (WebInspector.WorkerDispatcher): |
+ (WebInspector.WorkerDispatcher.prototype._receiveMessage): |
+ (WebInspector.WorkerDispatcher.prototype.workerCreated): |
+ (WebInspector.WorkerDispatcher.prototype.dispatchMessageFromWorker): |
+ * inspector/front-end/inspector.html: |
+ * inspector/front-end/inspector.js: |
+ (WebInspector.loaded): |
+ |
+2011-04-25 Annie Sullivan <sullivan@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [Resources panel] Should be easy to copy data. |
+ https://bugs.webkit.org/show_bug.cgi?id=45662 |
+ |
+ Adds context menu items to copy resource location, resource request headers, and resource response headers. |
+ |
+ * inspector/front-end/NetworkPanel.js: |
+ (WebInspector.NetworkPanel.prototype._contextMenu): Add new items to context menu. |
+ (WebInspector.NetworkPanel.prototype._exportLocation): Copy resource location to clipboard. |
+ (WebInspector.NetworkPanel.prototype._exportRequestHeaders): Copy resource request headers to clipboard. |
+ (WebInspector.NetworkPanel.prototype._exportResponseHeaders): Copy resource response headers to clipboard. |
+ * inspector/front-end/Resource.js: |
+ (WebInspector.Resource.prototype.get requestHeadersText): If _requestHeadersText is undefined, generate it from requestHeaders. |
+ (WebInspector.Resource.prototype.get responseHeadersText): If _responseHeadersText is undefined, generate it form responseHeaders. |
+ |
+2011-04-25 Alexander Pavlov <apavlov@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: CSS is parsed improperly when saved in UTF-8 with a BOM |
+ https://bugs.webkit.org/show_bug.cgi?id=59322 |
+ |
+ Use TextResourceDecoder to determine external stylesheet charsets correctly. |
+ |
+ Test: inspector/styles/parse-utf8-bom.html |
+ |
+ * inspector/InspectorStyleSheet.cpp: |
+ (WebCore::InspectorStyleSheet::originalStyleSheetText): |
+ |
+2011-04-25 Pavel Feldman <pfeldman@google.com> |
+ |
+ Not reviewed: Inspector.json cleanup. |
+ |
+ * inspector/Inspector.json: |
+ |
+2011-04-25 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: introduce InspectorWorkerAgent |
+ https://bugs.webkit.org/show_bug.cgi?id=59320 |
+ |
+ InspectorWorkerAgent is responsible for routing inspector messages |
+ between worker context inspector agents and corresponding worker |
+ inspector front-end. |
+ |
+ * CMakeLists.txt: |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * inspector/CodeGeneratorInspector.pm: |
+ * inspector/Inspector.json: |
+ * inspector/InspectorAgent.cpp: |
+ (WebCore::InspectorAgent::InspectorAgent): |
+ (WebCore::InspectorAgent::setFrontend): |
+ (WebCore::InspectorAgent::disconnectFrontend): |
+ * inspector/InspectorAgent.h: |
+ (WebCore::InspectorAgent::workerAgent): |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::connectFrontend): |
+ (WebCore::InspectorController::disconnectFrontend): |
+ * inspector/InspectorInstrumentation.cpp: |
+ (WebCore::InspectorInstrumentation::didStartWorkerContextImpl): |
+ * inspector/InspectorInstrumentation.h: |
+ (WebCore::InspectorInstrumentation::willStartWorkerContext): |
+ (WebCore::InspectorInstrumentation::didStartWorkerContext): |
+ * inspector/InspectorWorkerAgent.cpp: Added. |
+ (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::WorkerFrontendChannel): |
+ (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::~WorkerFrontendChannel): |
+ (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::id): |
+ (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::inspectorProxy): |
+ (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::sendMessageToFrontend): |
+ (WebCore::InspectorWorkerAgent::create): |
+ (WebCore::InspectorWorkerAgent::InspectorWorkerAgent): |
+ (WebCore::InspectorWorkerAgent::~InspectorWorkerAgent): |
+ (WebCore::InspectorWorkerAgent::setFrontend): |
+ (WebCore::InspectorWorkerAgent::clearFrontend): |
+ (WebCore::InspectorWorkerAgent::dispatchMessageOnWorkerInspector): |
+ (WebCore::InspectorWorkerAgent::didStartWorkerContext): |
+ * inspector/InspectorWorkerAgent.h: Added. |
+ * inspector/InstrumentingAgents.h: |
+ (WebCore::InstrumentingAgents::InstrumentingAgents): |
+ (WebCore::InstrumentingAgents::inspectorWorkerAgent): |
+ (WebCore::InstrumentingAgents::setInspectorWorkerAgent): |
+ * inspector/WorkerInspectorController.cpp: |
+ (WebCore::WorkerInspectorController::connectFrontend): |
+ * inspector/generate-inspector-idl: |
+ * workers/Worker.cpp: |
+ (WebCore::Worker::notifyFinished): |
+ * workers/WorkerContextInspectorProxy.h: Added. Platform-specific transport for inspector |
+ messages sent from the inspected page worker agent to the worker context inspector agents. |
+ (WebCore::WorkerContextInspectorProxy::connectFrontend): |
+ (WebCore::WorkerContextInspectorProxy::disconnectFrontend): |
+ (WebCore::WorkerContextInspectorProxy::sendMessageToWorkerContextInspector): |
+ (WebCore::WorkerContextInspectorProxy::~WorkerContextInspectorProxy): |
+ * workers/WorkerContextProxy.h: |
+ (WebCore::WorkerContextProxy::inspectorProxy): |
+ |
+2011-04-25 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: move Frame and Resource Tree management into the Page agent. |
+ https://bugs.webkit.org/show_bug.cgi?id=59321 |
+ |
+ Network agent should only handle network-related activities. |
+ Frame structure should be managed by the Page agent instead. |
+ |
+ * inspector/Inspector.json: |
+ * inspector/InspectorAgent.cpp: |
+ (WebCore::InspectorAgent::InspectorAgent): |
+ * inspector/InspectorDOMAgent.cpp: |
+ (WebCore::InspectorDOMAgent::InspectorDOMAgent): |
+ (WebCore::InspectorDOMAgent::setFrontend): |
+ (WebCore::InspectorDOMAgent::restore): |
+ (WebCore::InspectorDOMAgent::highlightFrame): |
+ * inspector/InspectorDOMAgent.h: |
+ (WebCore::InspectorDOMAgent::create): |
+ * inspector/InspectorInstrumentation.cpp: |
+ (WebCore::InspectorInstrumentation::domContentLoadedEventFiredImpl): |
+ (WebCore::InspectorInstrumentation::loadEventFiredImpl): |
+ (WebCore::InspectorInstrumentation::frameDetachedFromParentImpl): |
+ (WebCore::InspectorInstrumentation::didCommitLoadImpl): |
+ * inspector/InspectorPageAgent.cpp: |
+ (WebCore::InspectorPageAgent::create): |
+ (WebCore::InspectorPageAgent::resourceContent): |
+ (WebCore::InspectorPageAgent::resourceContentBase64): |
+ (WebCore::InspectorPageAgent::resourceData): |
+ (WebCore::InspectorPageAgent::cachedResource): |
+ (WebCore::InspectorPageAgent::resourceTypeString): |
+ (WebCore::InspectorPageAgent::cachedResourceType): |
+ (WebCore::InspectorPageAgent::cachedResourceTypeString): |
+ (WebCore::InspectorPageAgent::InspectorPageAgent): |
+ (WebCore::InspectorPageAgent::setFrontend): |
+ (WebCore::InspectorPageAgent::reload): |
+ (WebCore::InspectorPageAgent::open): |
+ (WebCore::InspectorPageAgent::getCookies): |
+ (WebCore::InspectorPageAgent::deleteCookie): |
+ (WebCore::InspectorPageAgent::getResourceTree): |
+ (WebCore::InspectorPageAgent::getResourceContent): |
+ (WebCore::InspectorPageAgent::restore): |
+ (WebCore::InspectorPageAgent::domContentEventFired): |
+ (WebCore::InspectorPageAgent::loadEventFired): |
+ (WebCore::InspectorPageAgent::frameNavigated): |
+ (WebCore::InspectorPageAgent::frameDetached): |
+ (WebCore::InspectorPageAgent::didClearWindowObjectInWorld): |
+ (WebCore::InspectorPageAgent::mainFrame): |
+ (WebCore::pointerAsId): |
+ (WebCore::InspectorPageAgent::frameForId): |
+ (WebCore::InspectorPageAgent::frameId): |
+ (WebCore::InspectorPageAgent::loaderId): |
+ (WebCore::InspectorPageAgent::buildObjectForFrame): |
+ (WebCore::InspectorPageAgent::buildObjectForFrameTree): |
+ * inspector/InspectorPageAgent.h: |
+ * inspector/InspectorResourceAgent.cpp: |
+ (WebCore::buildObjectForCachedResource): |
+ (WebCore::InspectorResourceAgent::willSendRequest): |
+ (WebCore::InspectorResourceAgent::didReceiveResponse): |
+ (WebCore::InspectorResourceAgent::didLoadResourceFromMemoryCache): |
+ (WebCore::InspectorResourceAgent::setInitialScriptContent): |
+ (WebCore::InspectorResourceAgent::setInitialXHRContent): |
+ (WebCore::InspectorResourceAgent::InspectorResourceAgent): |
+ * inspector/InspectorResourceAgent.h: |
+ (WebCore::InspectorResourceAgent::create): |
+ * inspector/InspectorStyleSheet.cpp: |
+ (WebCore::InspectorStyleSheet::resourceStyleSheetText): |
+ * inspector/front-end/NetworkManager.js: |
+ (WebInspector.NetworkManager.prototype.requestContent): |
+ * inspector/front-end/NetworkPanel.js: |
+ (WebInspector.NetworkPanel): |
+ (WebInspector.NetworkPanel.prototype.frameNavigated): |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.ResourceTreeModel): |
+ (WebInspector.ResourceTreeModel.prototype.frontendReused): |
+ (WebInspector.ResourceTreeModel.prototype.frameNavigated): |
+ (WebInspector.ResourceTreeModel.prototype.frameDetached): |
+ * inspector/front-end/inspector.js: |
+ (WebInspector.domContentEventFired): |
+ (WebInspector.loadEventFired): |
+ (WebInspector.frameNavigated): |
+ (WebInspector.frameDetached): |
+ |
+2011-04-25 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: few protocol improvements. |
+ https://bugs.webkit.org/show_bug.cgi?id=59319 |
+ |
+ - Page domain is documented |
+ - setUserAgentOverride is moved to the Network agent |
+ - setSearchingForNode -> setInspectModeEnabled |
+ - highlightDOMNode -> highlightNode |
+ - reloadPage -> reload |
+ - openInNewWindow -> open with optional parameter |
+ |
+ * inspector/Inspector.json: |
+ * inspector/InspectorDOMAgent.cpp: |
+ (WebCore::InspectorDOMAgent::setInspectModeEnabled): |
+ (WebCore::InspectorDOMAgent::highlightNode): |
+ * inspector/InspectorDOMAgent.h: |
+ (WebCore::InspectorDOMAgent::hideNodeHighlight): |
+ * inspector/InspectorInstrumentation.cpp: |
+ (WebCore::InspectorInstrumentation::applyUserAgentOverrideImpl): |
+ * inspector/InspectorPageAgent.cpp: |
+ (WebCore::InspectorPageAgent::clearFrontend): |
+ (WebCore::InspectorPageAgent::reload): |
+ (WebCore::InspectorPageAgent::open): |
+ * inspector/InspectorPageAgent.h: |
+ * inspector/InspectorResourceAgent.cpp: |
+ (WebCore::InspectorResourceAgent::clearFrontend): |
+ (WebCore::InspectorResourceAgent::applyUserAgentOverride): |
+ (WebCore::InspectorResourceAgent::setUserAgentOverride): |
+ * inspector/InspectorResourceAgent.h: |
+ * inspector/front-end/AuditsPanel.js: |
+ (WebInspector.AuditsPanel.prototype._reloadResources): |
+ * inspector/front-end/ElementsPanel.js: |
+ (WebInspector.ElementsPanel.prototype.setSearchingForNode): |
+ * inspector/front-end/ExtensionServer.js: |
+ (WebInspector.ExtensionServer.prototype._onReload): |
+ * inspector/front-end/NetworkPanel.js: |
+ (WebInspector.NetworkDataGridNode.prototype._openInNewTab): |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.FrameResourceTreeElement.prototype.ondblclick): |
+ * inspector/front-end/inspector.js: |
+ (WebInspector.highlightDOMNode): |
+ (WebInspector.openResource): |
+ (WebInspector.documentKeyDown): |
+ |
+2011-04-25 Jon Lee <jonlee@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ Overlay scroller hard to see on pages with dark background (59183) |
+ https://bugs.webkit.org/show_bug.cgi?id=59183 |
+ <rdar://problem/8975367> |
+ |
+ Switch the scrollbar's overlay style depending on its frame's background color. |
+ This refactors the getDocumentBackgroundColor method needed for gestures. The style |
+ is determined and set on every paint() call to the Mac scrollbar theme. |
+ |
+ * WebCore.exp.in: adding method to allow changing style |
+ * page/Frame.cpp: |
+ (WebCore::Frame::getDocumentBackgroundColor): moving code from WebFrame for reuse by FrameView |
+ * page/Frame.h: |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::recommendedScrollbarOverlayStyle): overridden to suggest a style based on CSS background color |
+ * page/FrameView.h: |
+ * platform/ScrollTypes.h: new enum to represent different overlay scrollbar styles |
+ * platform/ScrollableArea.h: |
+ (WebCore::ScrollableArea::recommendedScrollbarOverlayStyle): new virtual function to return a suggested overlay style |
+ * platform/mac/ScrollbarThemeMac.mm: |
+ (WebCore::ScrollbarThemeMac::paint): |
+ * platform/mac/WebCoreSystemInterface.h: |
+ * platform/mac/WebCoreSystemInterface.mm: |
+ |
+2011-04-25 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ <rdar://problem/9113516> REGRESSION (WK2): Holding down Option while dragging scrollbar thumb has no effect (should make scroll distance equal mouse translation) |
+ https://bugs.webkit.org/show_bug.cgi?id=59315 |
+ |
+ * platform/Scrollbar.cpp: |
+ (WebCore::Scrollbar::Scrollbar): Initialize new member variables. |
+ (WebCore::Scrollbar::moveThumb): Added a boolean draggingDocument parameter. When true, the document |
+ is scrolled a distance equal to the change in the mouse position. |
+ (WebCore::Scrollbar::mouseMoved): Check with the theme whether the mouse move event should drag |
+ the document rather than the thumb, and pass the result to moveThumb. |
+ (WebCore::Scrollbar::mouseUp): Reset m_draggingDocument. |
+ * platform/Scrollbar.h: |
+ * platform/ScrollbarTheme.h: |
+ (WebCore::ScrollbarTheme::shouldDragDocumentInsteadOfThumb): Added. The base class implementation |
+ returns false. |
+ * platform/mac/ScrollbarThemeMac.h: |
+ * platform/mac/ScrollbarThemeMac.mm: |
+ (WebCore::ScrollbarThemeMac::shouldDragDocumentInsteadOfThumb): Overridden to return true if the |
+ Option key is down. |
+ |
+2011-04-24 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Removed a use of markDOMObjectWrapper: WebGL, XHR, workers; plus, more autogeneration |
+ https://bugs.webkit.org/show_bug.cgi?id=59307 |
+ |
+ * WebCore.xcodeproj/project.pbxproj: Added Blob.idl, so it's easier to edit. |
+ |
+ * bindings/js/JSCSSRuleCustom.cpp: |
+ * bindings/js/JSCSSStyleDeclarationCustom.cpp: |
+ * bindings/js/JSCanvasRenderingContextCustom.cpp: |
+ * bindings/js/JSDOMApplicationCacheCustom.cpp: Autogenerate these instead |
+ of hand-coding them. |
+ |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::root): Moved some GC helpers here from JSNodeCustom.h, because |
+ they're reasonably generic. |
+ |
+ * bindings/js/JSDOMImplementationCustom.cpp: |
+ * bindings/js/JSDOMStringMapCustom.cpp: |
+ * bindings/js/JSDOMTokenListCustom.cpp: |
+ * bindings/js/JSHTMLCollectionCustom.cpp: |
+ * bindings/js/JSMediaListCustom.cpp: Autogenerate these instead of |
+ hand-coding them. |
+ |
+ * bindings/js/JSMessageChannelCustom.cpp: |
+ (WebCore::JSMessageChannel::visitChildren): |
+ * bindings/js/JSMessagePortCustom.cpp: |
+ (WebCore::JSMessagePort::visitChildren): Use addOpaqueRoot instead of |
+ markDOMObjectWrapper. This is one of the few cases where a DOM object is |
+ considered a stand-alone root. It's not as efficient as the shared root |
+ case, but workers and message ports are very rare objects, so it's no |
+ big deal. |
+ |
+ * bindings/js/JSNamedNodeMapCustom.cpp: Autogenerate! |
+ |
+ * bindings/js/JSNodeCustom.h: Moved to JSDOMBinding.h. |
+ |
+ * bindings/js/JSSharedWorkerCustom.cpp: |
+ (WebCore::JSSharedWorker::visitChildren): Use addOpaqueRoot instead of |
+ markDOMObjectWrapper, as above. Once again, a stand-alone root. |
+ |
+ * bindings/js/JSStyleSheetCustom.cpp: |
+ * bindings/js/JSStyleSheetListCustom.cpp: Autogenerate! |
+ |
+ * bindings/js/JSWebGLRenderingContextCustom.cpp: |
+ (WebCore::JSWebGLRenderingContext::visitChildren): Use the opaque roots |
+ system for marking WebGL contexts and their associated satellite objects. |
+ |
+ * bindings/js/JSWorkerContextCustom.cpp: |
+ (WebCore::JSWorkerContext::visitChildren): |
+ * bindings/js/JSXMLHttpRequestCustom.cpp: Use addOpaqueRoot instead of |
+ markDOMObjectWrapper, as above. Once again, stand-alone roots. |
+ |
+ (WebCore::JSXMLHttpRequest::visitChildren): |
+ * bindings/js/JSXMLHttpRequestUploadCustom.cpp: No need to mark our owner |
+ XHR because it is not reachable from us. |
+ |
+ * bindings/scripts/CodeGeneratorJS.pm: Added autogeneration support for a |
+ bunch of repetitive cases of isReachableFromOpaqueRoots callbacks. |
+ |
+ * css/CSSRule.idl: |
+ * css/CSSStyleDeclaration.idl: |
+ * css/MediaList.idl: |
+ * css/StyleMedia.idl: |
+ * css/StyleSheet.idl: |
+ * css/StyleSheetList.idl: |
+ * dom/DOMImplementation.idl: |
+ * dom/DOMStringMap.idl: |
+ * dom/MessagePort.idl: |
+ * dom/NamedNodeMap.idl: |
+ * fileapi/Blob.idl: |
+ * html/DOMTokenList.idl: Opt in to autogeneration. |
+ |
+ * html/HTMLCanvasElement.cpp: |
+ (WebCore::HTMLCanvasElement::getContext): Standardized on PassOwnPtr/OwnPtr |
+ usage, to reduce human error and make ownership rules more obvious. |
+ |
+ * html/HTMLCollection.idl: |
+ * html/canvas/ArrayBuffer.idl: Opt in to autogeneration. |
+ |
+ * html/canvas/CanvasRenderingContext.cpp: |
+ * html/canvas/CanvasRenderingContext.h: |
+ (WebCore::CanvasRenderingContext::ref): |
+ (WebCore::CanvasRenderingContext::deref): |
+ * html/canvas/CanvasRenderingContext2D.h: |
+ (WebCore::CanvasRenderingContext2D::create): |
+ (WebCore::CanvasRenderingContext2D::state): Standardized on PassOwnPtr/OwnPtr |
+ usage, to reduce human error and make ownership rules more obvious. |
+ |
+ Inlined some functions to match our standard idiom and to make the code |
+ in the header document itself. |
+ |
+ * html/canvas/CanvasRenderingContext.idl: Opt in to autogeneration. |
+ |
+ * html/canvas/OESStandardDerivatives.cpp: |
+ (WebCore::OESStandardDerivatives::OESStandardDerivatives): |
+ (WebCore::OESStandardDerivatives::create): |
+ * html/canvas/OESStandardDerivatives.h: |
+ * html/canvas/OESStandardDerivatives.idl: |
+ * html/canvas/OESTextureFloat.cpp: |
+ (WebCore::OESTextureFloat::OESTextureFloat): |
+ (WebCore::OESTextureFloat::create): |
+ * html/canvas/OESTextureFloat.h: |
+ * html/canvas/OESTextureFloat.idl: |
+ * html/canvas/OESVertexArrayObject.cpp: |
+ (WebCore::OESVertexArrayObject::OESVertexArrayObject): |
+ (WebCore::OESVertexArrayObject::create): |
+ * html/canvas/OESVertexArrayObject.h: |
+ * html/canvas/OESVertexArrayObject.idl: |
+ * html/canvas/WebGLExtension.cpp: |
+ (WebCore::WebGLExtension::WebGLExtension): |
+ * html/canvas/WebGLExtension.h: |
+ (WebCore::WebGLExtension::ref): |
+ (WebCore::WebGLExtension::deref): |
+ (WebCore::WebGLExtension::context): |
+ * html/canvas/WebGLRenderingContext.cpp: |
+ (WebCore::WebGLRenderingContext::~WebGLRenderingContext): |
+ (WebCore::WebGLRenderingContext::getExtension): |
+ * html/canvas/WebGLRenderingContext.h: |
+ * html/canvas/WebKitLoseContext.cpp: |
+ (WebCore::WebKitLoseContext::WebKitLoseContext): |
+ (WebCore::WebKitLoseContext::create): |
+ (WebCore::WebKitLoseContext::loseContext): |
+ * html/canvas/WebKitLoseContext.h: |
+ * html/canvas/WebKitLoseContext.idl: Updated the ownership model for WebGL |
+ canavs rendering contexts to match the model for 2D canvas rendering |
+ contexts. This makes garbage collection a lot more straightforward, but |
+ it also makes the behavior of these objects more reliable. (Previously, |
+ satellite objects would magically stop working when the last reference |
+ to their owner object was dropped. Now, satellite objects keep their owner |
+ alive through reference counting.) |
+ |
+ * loader/appcache/DOMApplicationCache.idl: |
+ * page/BarInfo.idl: |
+ * page/Console.idl: |
+ * page/DOMSelection.idl: |
+ * page/Geolocation.idl: |
+ * page/History.idl: |
+ * page/Location.idl: |
+ * page/Navigator.idl: |
+ * page/Screen.idl: |
+ * page/WorkerNavigator.idl: |
+ * plugins/DOMMimeTypeArray.h: |
+ (WebCore::DOMMimeTypeArray::frame): |
+ * plugins/DOMMimeTypeArray.idl: |
+ * plugins/DOMPluginArray.h: |
+ (WebCore::DOMPluginArray::frame): |
+ * plugins/DOMPluginArray.idl: |
+ * storage/Storage.idl: |
+ * workers/WorkerLocation.idl: Opt in to autogeneration. |
+ |
+ * xml/XMLHttpRequest.cpp: |
+ (WebCore::XMLHttpRequest::~XMLHttpRequest): |
+ * xml/XMLHttpRequest.h: |
+ * xml/XMLHttpRequestUpload.cpp: |
+ (WebCore::XMLHttpRequestUpload::scriptExecutionContext): |
+ * xml/XMLHttpRequestUpload.h: |
+ (WebCore::XMLHttpRequestUpload::create): |
+ (WebCore::XMLHttpRequestUpload::ref): |
+ (WebCore::XMLHttpRequestUpload::deref): |
+ (WebCore::XMLHttpRequestUpload::xmlHttpRequest): |
+ (WebCore::XMLHttpRequestUpload::toXMLHttpRequestUpload): |
+ * xml/XMLHttpRequestUpload.idl: Updated the ownership model for XHR and |
+ its associated upload object to match the model for canvas. This makes |
+ garbage collection a lot more straightforward, but it also makes the |
+ behavior of these objects more reliable. |
+ |
+2011-04-24 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r84759. |
+ http://trac.webkit.org/changeset/84759 |
+ https://bugs.webkit.org/show_bug.cgi?id=59306 |
+ |
+ Caused gc-shadow.html to start crashing in V8. (Requested by |
+ dglazkov|away on #webkit). |
+ |
+ * dom/Node.cpp: |
+ (WebCore::NodeRendererFactory::createRendererAndStyle): |
+ (WebCore::Node::styleForRenderer): |
+ |
+2011-04-24 Maciej Stachowiak <mjs@apple.com> |
+ |
+ Reviewed by George Staikos. |
+ |
+ Handling of URLs like http:/example.com/ is incorrect |
+ https://bugs.webkit.org/show_bug.cgi?id=59300 |
+ <rdar://problem/9231956> |
+ |
+ URLs like http:/example.com/ or http:example.com/ are now correctly |
+ canonicalized as http://example.com/ |
+ |
+ The code still doesn't quite match other browsers - at least some |
+ other browsers seem to base parsing behavior on whether they |
+ recongize a scheme from a fixed list, and ignore whether // is |
+ present in the URL or not. |
+ |
+ * platform/KURL.cpp: |
+ (WebCore::isNonFileHierarchicalScheme): New helper function. |
+ (WebCore::KURL::parse): For a particular list of whitelisted schemes, |
+ assume they are hierarchical and need an authority even if there is no // |
+ after the :/ |
+ |
+2011-04-24 Dan Bernstein <mitz@apple.com> |
+ |
+ LLVM Compiler build fix. |
+ |
+ * page/ContentSecurityPolicy.cpp: Removed an unused function. |
+ |
+2011-04-24 Dominic Cooney <dominicc@chromium.org> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ Crash when adding a text node to a shadow root |
+ https://bugs.webkit.org/show_bug.cgi?id=59304 |
+ |
+ Text nodes need to consult their host element for style. |
+ |
+ Test: fast/dom/shadow/append-child-text.html |
+ |
+ * dom/Node.cpp: |
+ (WebCore::NodeRendererFactory::styleForRenderer): |
+ (WebCore::NodeRendererFactory::createRendererAndStyle): |
+ (WebCore::Node::styleForRenderer): forward to NodeRendererFactory |
+ |
+2011-04-24 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Update Content-Security-Policy syntax to match new version of spec |
+ https://bugs.webkit.org/show_bug.cgi?id=59291 |
+ |
+ Brandon removed disable-xss-protection in favor of unsafe-inline and |
+ allow-eval in favor of unsafe-eval. This change in syntax also means |
+ the options directive no longer exists. |
+ |
+ * page/ContentSecurityPolicy.cpp: |
+ (WebCore::CSPSourceList::allowInline): |
+ (WebCore::CSPSourceList::allowEval): |
+ (WebCore::CSPSourceList::CSPSourceList): |
+ (WebCore::CSPSourceList::parseSource): |
+ (WebCore::CSPSourceList::addSourceUnsafeInline): |
+ (WebCore::CSPSourceList::addSourceUnsafeEval): |
+ (WebCore::CSPDirective::allowInline): |
+ (WebCore::CSPDirective::allowEval): |
+ (WebCore::ContentSecurityPolicy::allowJavaScriptURLs): |
+ (WebCore::ContentSecurityPolicy::allowInlineEventHandlers): |
+ (WebCore::ContentSecurityPolicy::allowInlineScript): |
+ (WebCore::ContentSecurityPolicy::allowEval): |
+ (WebCore::ContentSecurityPolicy::addDirective): |
+ * page/ContentSecurityPolicy.h: |
+ |
+2011-04-24 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ Manual test for <rdar://problem/9329741> Reproducible crash in WebChromeClient::invalidateContentsAndWindow() on simonscat.com |
+ https://bugs.webkit.org/show_bug.cgi?id=59299 |
+ |
+ * manual-tests/back-from-document-with-scrollbar.html: Added. |
+ |
+2011-04-24 Rik Cabanier <cabanier@adobe.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ Tables are not rendered correctly |
+ https://bugs.webkit.org/show_bug.cgi?id=59138 |
+ |
+ Test: fast/table/auto-100-percent-width.html |
+ |
+ * rendering/AutoTableLayout.cpp: |
+ (WebCore::AutoTableLayout::computePreferredLogicalWidths): |
+ |
+2011-04-23 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Fix Windows build, which complains about unreachable code. |
+ |
+ * platform/ScrollableArea.cpp: |
+ (WebCore::ScrollableArea::hasLayerForHorizontalScrollbar): |
+ (WebCore::ScrollableArea::hasLayerForVerticalScrollbar): |
+ (WebCore::ScrollableArea::hasLayerForScrollCorner): |
+ |
+2011-04-23 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ Area under composited scrollbars not repainted in WebKit2 on scrolling |
+ https://bugs.webkit.org/show_bug.cgi?id=59294 |
+ <rdar://problem/9299062> |
+ |
+ When computing the rect that can be copied on scrolling, we normally |
+ exclude the areas occupied by overlay scrollbars to avoid smeared scrollbars; |
+ the assumption is that other code will repaint these areas. However, when |
+ scrollbars are in their own compositing layers, we can, and should |
+ copy the entire area. |
+ |
+ Not testable in DRT because it does not allow tests to use overlay scrollbars. |
+ |
+ * platform/ScrollView.h: |
+ * platform/ScrollView.cpp: |
+ (WebCore::ScrollView::rectToCopyOnScroll): |
+ New method, with code moved from scrollContents and fixed to look |
+ for scrollbars in layers. |
+ (WebCore::ScrollView::scrollContents): |
+ Call rectToCopyOnScroll(). |
+ |
+ * platform/ScrollableArea.h: |
+ * platform/ScrollableArea.cpp: |
+ (WebCore::ScrollableArea::hasLayerForHorizontalScrollbar): |
+ (WebCore::ScrollableArea::hasLayerForVerticalScrollbar): |
+ (WebCore::ScrollableArea::hasLayerForScrollCorner): |
+ Methods we can call outside of the ACCELERATED_COMPOSITING #ifdef. |
+ |
+2011-04-23 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r84740. |
+ http://trac.webkit.org/changeset/84740 |
+ https://bugs.webkit.org/show_bug.cgi?id=59290 |
+ |
+ change breaks apple internal builds and is incorrect |
+ (Requested by smfr on #webkit). |
+ |
+ * WebCore.exp.in: |
+ * page/FrameView.cpp: |
+ * page/FrameView.h: |
+ * platform/ScrollTypes.h: |
+ * platform/ScrollableArea.h: |
+ * platform/mac/ScrollbarThemeMac.mm: |
+ (WebCore::ScrollbarThemeMac::paint): |
+ * platform/mac/WebCoreSystemInterface.h: |
+ * platform/mac/WebCoreSystemInterface.mm: |
+ |
+2011-04-23 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ <rdar://problem/8970549> WebFindOptionsAtWordStarts fails with Japanese words |
+ https://bugs.webkit.org/show_bug.cgi?id=59288 |
+ |
+ * platform/text/mac/TextBoundaries.mm: |
+ (WebCore::wordStringTokenizer): Added this helper function. |
+ (WebCore::findNextWordFromIndex): Changed to use a CFStringTokenizer with kCFStringTokenizerUnitWord, |
+ whose behavior is not locale-dependent. |
+ |
+2011-04-23 Dominic Cooney <dominicc@chromium.org> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ [V8] Nodes in shadow DOM should not be GCed while their hosts are alive |
+ https://bugs.webkit.org/show_bug.cgi?id=59284 |
+ |
+ Test: fast/dom/shadow/gc-shadow.html |
+ |
+ * bindings/v8/V8GCController.cpp: |
+ (WebCore::calculateGroupId): group shadow nodes with their hosts |
+ |
+2011-04-23 MORITA Hajime <morrita@google.com> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ [Refactoring] NodeRenderParentDetector should be NodeRenderFactory |
+ https://bugs.webkit.org/show_bug.cgi?id=59280 |
+ |
+ - Rename NodeRenderParentDetector to NodeRendererFactory |
+ - move createRendererAndStyle() and createRendererIfNeeded to |
+ NodeRendererFactory |
+ |
+ No new test, no behavioral change. |
+ |
+ * dom/Node.cpp: |
+ (WebCore::NodeRendererFactory::NodeRendererFactory): |
+ (WebCore::NodeRendererFactory::document): |
+ (WebCore::NodeRendererFactory::nextRenderer): |
+ (WebCore::NodeRendererFactory::findVisualParent): |
+ (WebCore::NodeRendererFactory::shouldCreateRenderer): |
+ (WebCore::NodeRendererFactory::createRendererAndStyle): |
+ (WebCore::NodeRendererFactory::createRendererIfNeeded): |
+ (WebCore::Node::parentNodeForRenderingAndStyle): |
+ (WebCore::Node::createRendererIfNeeded): |
+ * dom/Node.h: |
+ * html/HTMLDetailsElement.cpp: |
+ (WebCore::DetailsSummaryElement::detailsElement): Added const_cast due to the chage on parentNodeForRenderingAndStyle() |
+ |
+2011-04-23 MORITA Hajime <morrita@google.com> |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ REGRESSION: (non-Mac) <meter>'s bar part isn't rendered for with -webkit-writing-mode: vertical-lr from r82899 |
+ https://bugs.webkit.org/show_bug.cgi?id=59281 |
+ |
+ Styles for -webkit-meter-bar and -webkit-progress-bar missed width property, which caused unexpected layout results |
+ for vertical writing modes. This fix added a width property for each of them. |
+ |
+ Tests: fast/dom/HTMLMeterElement/meter-writing-mode.html |
+ fast/dom/HTMLProgressElement/progress-writing-mode.html |
+ |
+ * css/html.css: |
+ (meter::-webkit-meter-bar): |
+ (progress::-webkit-progress-bar): |
+ |
+2011-04-23 Kevin Ollivier <kevino@theolliviers.com> |
+ |
+ [wx] Unreviewed build fix. Added missing header. |
+ |
+ * platform/graphics/filters/FEDropShadow.h: |
+ |
+2011-04-23 Alexey Proskuryakov <ap@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ REGRESSION (r80812): window.print();window.close() doesn't work in WebKit2 |
+ https://bugs.webkit.org/show_bug.cgi?id=59241 |
+ <rdar://problem/9150861> |
+ |
+ Covered by manual-tests/print-after-window-close.html. |
+ |
+ Re-fixing <https://bugs.webkit.org/show_bug.cgi?id=51357> in a way that doesn't cause the |
+ regression. |
+ |
+ * loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::didCancel): We shouldn't be |
+ doing complicated work while the loader is half-canceled. |
+ |
+ * manual-tests/print-after-window-close.html: Extended to actually print a non-empty page, |
+ thus being more practical. |
+ |
+ * page/Chrome.cpp: (WebCore::Chrome::print): Added a FIXME about a need for PageGroupLoadDeferrer. |
+ It's too scary for me to try now. |
+ |
+ * page/DOMWindow.cpp: |
+ (WebCore::DOMWindow::DOMWindow): |
+ (WebCore::DOMWindow::finishedLoading): |
+ * page/DOMWindow.h: |
+ Reverted the previous fix for <https://bugs.webkit.org/show_bug.cgi?id=51357>. |
+ |
+2011-04-19 Vitaly Repeshko <vitalyr@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ [V8] Use implicit references for V8 listeners on DOM nodes. |
+ https://bugs.webkit.org/show_bug.cgi?id=58953 |
+ |
+ Instead of allocating an auxiliary V8 array referencing V8 |
+ listener objects associated with a DOM node and using an extra |
+ pointer in every DOM node wrapper, we can register implicit |
+ references between nodes and their listeners during GC. This also |
+ makes V8 bindings more aligned with JSC bindings. |
+ |
+ No new tests because this is a refactoring. |
+ |
+ * bindings/scripts/CodeGeneratorV8.pm: Stopped generating the |
+ listener cache internal field for DOM nodes. |
+ * bindings/v8/V8AbstractEventListener.h: Exposed the lister handle |
+ to the GC. |
+ (WebCore::V8AbstractEventListener::existingListenerObjectPeristentHandle): |
+ * bindings/v8/V8GCController.cpp: Started using implicit references. |
+ (WebCore::GrouperVisitor::visitDOMWrapper): |
+ * dom/EventTarget.h: Implemented an iterator over all listeners. |
+ (WebCore::EventListenerIterator): |
+ |
+2011-04-22 Jon Lee <jonlee@apple.com> |
+ |
+ Reviewed by Beth Dakin. |
+ |
+ Overlay scroller hard to see on pages with dark background (59183) |
+ https://bugs.webkit.org/show_bug.cgi?id=59183 |
+ <rdar://problem/8975367> |
+ |
+ * WebCore.exp.in: adding method to allow changing style |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::recommendedScrollbarOverlayStyle): overridden to suggest a style based on CSS background color |
+ * page/FrameView.h: |
+ * platform/ScrollTypes.h: new enum to represent different overlay scrollbar styles |
+ * platform/ScrollableArea.cpp: |
+ (WebCore::ScrollableArea::recommendedScrollbarOverlayStyle): new virtual function to return a suggested overlay style |
+ * platform/ScrollableArea.h: |
+ * platform/mac/ScrollbarThemeMac.mm: |
+ (WebCore::ScrollbarThemeMac::paint): |
+ * platform/mac/WebCoreSystemInterface.h: |
+ * platform/mac/WebCoreSystemInterface.mm: |
+ |
+2011-04-22 Chris Evans <cevans@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Upgrade CSS loads from mixed content warning (displayed) to mixed |
+ content error (ran) |
+ https://bugs.webkit.org/show_bug.cgi?id=59056 |
+ |
+ Test: http/tests/security/mixedContent/insecure-css-in-iframe.html |
+ Test: http/tests/security/mixedContent/insecure-css-in-main-frame.html |
+ |
+ * loader/cache/CachedResourceLoader.cpp: |
+ (WebCore::CachedResourceLoader::canRequest): CSS loads are running rather than displaying content. |
+ |
+2011-04-22 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Fixed an upside-down conditional in my last check-in. |
+ |
+ * bindings/js/JSDOMApplicationCacheCustom.cpp: |
+ (WebCore::isObservable): |
+ |
+2011-04-22 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Rolled back in r84725 and r84728 with appcache crash fixed. |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Removed a use of markDOMObjectWrapper: DOMWindow |
+ https://bugs.webkit.org/show_bug.cgi?id=59260 |
+ |
+ * bindings/js/JSDOMWindowCustom.cpp: |
+ (WebCore::JSDOMWindow::visitChildren): No need to mark explicitly, since |
+ we use the opaque roots system now. |
+ |
+ * bindings/js/JSNavigatorCustom.cpp: Ditto. |
+ |
+ * bindings/scripts/CodeGeneratorJS.pm: Added autogeneration for objects |
+ whose opaque roots are frames. |
+ |
+ * css/StyleMedia.h: |
+ (WebCore::StyleMedia::frame): Added an accessor for the sake of GC. |
+ |
+ * css/StyleMedia.idl: |
+ * loader/appcache/DOMApplicationCache.idl: |
+ * page/BarInfo.idl: |
+ * page/Console.idl: |
+ * page/DOMSelection.idl: |
+ * page/Geolocation.idl: |
+ * page/History.idl: |
+ * page/Location.idl: |
+ * page/Navigator.idl: |
+ * page/Screen.idl: |
+ * storage/Storage.idl: Updated to use the opaque roots system. |
+ |
+2011-04-22 Nate Chapin <japhet@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Don't use XMLTreeViewer when script is disabled (it uses Javascript |
+ to build a prettified version of the xml document). |
+ https://bugs.webkit.org/show_bug.cgi?id=59113 |
+ |
+ * xml/XMLTreeViewer.cpp: |
+ (WebCore::XMLTreeViewer::transformDocumentToTreeView): |
+ |
+2011-04-22 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r84725 and r84728. |
+ http://trac.webkit.org/changeset/84725 |
+ http://trac.webkit.org/changeset/84728 |
+ https://bugs.webkit.org/show_bug.cgi?id=59270 |
+ |
+ It broke http/tests/appcache tests (Requested by Ossy on |
+ #webkit). |
+ |
+ * bindings/js/JSDOMWindowCustom.cpp: |
+ (WebCore::JSDOMWindow::visitChildren): |
+ * bindings/js/JSNavigatorCustom.cpp: |
+ (WebCore::JSNavigator::visitChildren): |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ * css/StyleMedia.h: |
+ * css/StyleMedia.idl: |
+ * loader/appcache/DOMApplicationCache.idl: |
+ * page/BarInfo.idl: |
+ * page/Console.idl: |
+ * page/DOMSelection.idl: |
+ * page/DOMWindow.h: |
+ (WebCore::DOMWindow::optionalScreen): |
+ (WebCore::DOMWindow::optionalSelection): |
+ (WebCore::DOMWindow::optionalHistory): |
+ (WebCore::DOMWindow::optionalCrypto): |
+ (WebCore::DOMWindow::optionalLocationbar): |
+ (WebCore::DOMWindow::optionalMenubar): |
+ (WebCore::DOMWindow::optionalPersonalbar): |
+ (WebCore::DOMWindow::optionalScrollbars): |
+ (WebCore::DOMWindow::optionalStatusbar): |
+ (WebCore::DOMWindow::optionalToolbar): |
+ (WebCore::DOMWindow::optionalConsole): |
+ (WebCore::DOMWindow::optionalNavigator): |
+ (WebCore::DOMWindow::optionalLocation): |
+ (WebCore::DOMWindow::optionalMedia): |
+ (WebCore::DOMWindow::optionalSessionStorage): |
+ (WebCore::DOMWindow::optionalLocalStorage): |
+ (WebCore::DOMWindow::optionalPerformance): |
+ * page/Geolocation.idl: |
+ * page/History.idl: |
+ * page/Location.idl: |
+ * page/Navigator.h: |
+ (WebCore::Navigator::optionalGeolocation): |
+ * page/Navigator.idl: |
+ * page/Screen.idl: |
+ * storage/Storage.idl: |
+ |
+2011-04-22 Dimitri Glazkov <dglazkov@chromium.org> |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ Change SVG-related shadow root checks to use isSVGShadowRoot. |
+ https://bugs.webkit.org/show_bug.cgi?id=59237 |
+ |
+ Covered by existing tests, no behavior change. |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::SelectorChecker::checkSelector): Use isSVGShadowRoot. |
+ * svg/SVGUseElement.cpp: |
+ (WebCore::ShadowTreeUpdateBlocker::while): Ditto. |
+ * svg/animation/SVGSMILElement.cpp: |
+ (WebCore::SVGSMILElement::insertedIntoDocument): Ditto. |
+ |
+2011-04-22 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Mac build fix. |
+ |
+ * page/DOMWindow.h: |
+ (WebCore::DOMWindow::optionalApplicationCache): Restored this function, |
+ which is used after all. |
+ |
+2011-04-22 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Mac build fix. |
+ |
+ * page/DOMWindow.h: |
+ (WebCore::DOMWindow::optionalApplicationCache): Restored this function, |
+ which is used after all. |
+ |
+2011-04-22 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Removed a use of markDOMObjectWrapper: DOMWindow |
+ https://bugs.webkit.org/show_bug.cgi?id=59260 |
+ |
+ * bindings/js/JSDOMWindowCustom.cpp: |
+ (WebCore::JSDOMWindow::visitChildren): No need to mark explicitly, since |
+ we use the opaque roots system now. |
+ |
+ * bindings/js/JSNavigatorCustom.cpp: Ditto. |
+ |
+ * bindings/scripts/CodeGeneratorJS.pm: Added autogeneration for objects |
+ whose opaque roots are frames. |
+ |
+ * css/StyleMedia.h: |
+ (WebCore::StyleMedia::frame): Added an accessor for the sake of GC. |
+ |
+ * css/StyleMedia.idl: |
+ * loader/appcache/DOMApplicationCache.idl: |
+ * page/BarInfo.idl: |
+ * page/Console.idl: |
+ * page/DOMSelection.idl: |
+ * page/Geolocation.idl: |
+ * page/History.idl: |
+ * page/Location.idl: |
+ * page/Navigator.idl: |
+ * page/Screen.idl: |
+ * storage/Storage.idl: Updated to use the opaque roots system. |
+ |
+2011-04-22 Xiaomei Ji <xji@chromium.org> |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ move caret by word in visual order returns wrong result when caret itself is at word boundary. |
+ https://bugs.webkit.org/show_bug.cgi?id=58504. |
+ |
+ positionBeforeNextWord should take care that the current position is after current word. |
+ positionAfterPreviousWord should take care that the current positin is before current word. |
+ |
+ * editing/visible_units.cpp: |
+ (WebCore::positionBeforeNextWord): |
+ (WebCore::positionAfterPreviousWord): |
+ |
+2011-04-19 Jer Noble <jer.noble@apple.com> |
+ |
+ Reviewed by Daniel Bates. |
+ |
+ Full Screen from within an <iframe> does not cause <iframe> to resize. |
+ https://bugs.webkit.org/show_bug.cgi?id=58638 |
+ |
+ Tests: fullscreen/full-screen-frameset.html |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): Add the |
+ -webkit-full-screen pseudo class to frame elements which contain |
+ full screen elements. |
+ * css/fullscreen.css: |
+ (iframe:-webkit-full-screen): Add iframe-specific styling rules. |
+ * dom/Document.cpp: |
+ (WebCore::Document::setContainsFullScreenElementRecursively): Added. Walk up each |
+ successive document owner element, calling setContainsFullScreenElement() if |
+ owner is a frame element. |
+ (WebCore::Document::fullScreenIsAllowedForElement): Use the new Element::isFrameElementBase |
+ call instead of checking the element's tag name. |
+ (WebCore::Document::webkitWillEnterFullScreenForElement): Mark any containing |
+ iframes as containing a full screen element. |
+ (WebCore::Document::webkitWillExitFullScreenForElement): Ditto. |
+ * dom/Element.h: |
+ (WebCore::Element::isFrameElementBase): Added. |
+ * html/HTMLFrameElement.h: |
+ (WebCore::HTMLFrameElement::allowFullScreen): Disable full screen from within |
+ HTMLFrameElements. |
+ * html/HTMLFrameElementBase.cpp: |
+ (WebCore::HTMLFrameElementBase::setContainsFullScreenElement): Added. |
+ * html/HTMLFrameElementBase.h: |
+ (WebCore::HTMLFrameElementBase::containsFullScreenElement): Added. |
+ (WebCore::HTMLFrameElementBase::isFrameElementBase): Added. |
+ |
+2011-04-22 Michael Saboff <msaboff@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ Creating copy of ContainerNode's when inserting or removing is inefficient |
+ https://bugs.webkit.org/show_bug.cgi?id=58695 |
+ |
+ Eliminated node copying in willRemove() and insertedIntoDocument(). |
+ |
+ No new tests as this is a more efficient implementation of |
+ existing code that is covered by existing tests. |
+ |
+ * dom/ContainerNode.cpp: |
+ (WebCore::ContainerNode::willRemove): Changed method to use |
+ RefPtr<> to protect against modification during removal. |
+ (WebCore::ContainerNode::insertedIntoDocument): Changed method to use |
+ RefPtr<> and two other deletion checks to protect against |
+ modification during insertion. |
+ |
+2011-04-22 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ We have an autogenerator, so use it |
+ https://bugs.webkit.org/show_bug.cgi?id=59247 |
+ |
+ * bindings/js/JSCSSRuleCustom.cpp: |
+ * bindings/js/JSCSSRuleListCustom.cpp: |
+ * bindings/js/JSCSSStyleDeclarationCustom.cpp: |
+ * bindings/js/JSCSSValueCustom.cpp: |
+ * bindings/js/JSCanvasRenderingContextCustom.cpp: |
+ * bindings/js/JSDOMImplementationCustom.cpp: |
+ * bindings/js/JSDOMStringMapCustom.cpp: |
+ * bindings/js/JSHTMLCollectionCustom.cpp: |
+ * bindings/js/JSNamedNodeMapCustom.cpp: |
+ * bindings/js/JSNodeCustom.h: |
+ * bindings/js/JSNodeListCustom.cpp: |
+ * bindings/js/JSStyleSheetCustom.cpp: |
+ * bindings/js/JSStyleSheetListCustom.cpp: Removed repetitive hand-written code. |
+ |
+ * bindings/scripts/CodeGeneratorJS.pm: Augmented autogenerator to do the job. |
+ |
+ * css/CSSRule.idl: |
+ * css/CSSRuleList.idl: |
+ * css/CSSStyleDeclaration.idl: |
+ * css/CSSValue.idl: |
+ * css/MediaList.idl: |
+ * css/StyleSheet.idl: |
+ * css/StyleSheetList.idl: |
+ * dom/DOMImplementation.idl: |
+ * dom/DOMStringMap.idl: |
+ * dom/NamedNodeMap.idl: |
+ * dom/Node.idl: |
+ * dom/NodeList.idl: |
+ * html/DOMTokenList.idl: |
+ * html/HTMLCollection.idl: |
+ * html/canvas/CanvasRenderingContext.idl: Updated attributes to plug into |
+ the autogenerator. |
+ |
+2011-04-22 Beth Dakin <bdakin@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=59223 |
+ REGRESSION (r82185): iframe not displayed after navigation back to cached page |
+ |
+ Manual test added. |
+ * manual-tests/frame-layout-on-back.html: Added. |
+ * manual-tests/resources/empty-document-goes-back.html: Added. |
+ * manual-tests/resources/simple-iframe.html: Added. |
+ |
+2011-04-22 Yael Aharon <yael.aharon@nokia.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ HTMLFormControlElement::labels should allow custom attributes. |
+ https://bugs.webkit.org/show_bug.cgi?id=59128 |
+ |
+ Cache the labels list when it is created and keep using it for future |
+ labels calls. |
+ |
+ Test: fast/forms/labels-custom-property.html |
+ |
+ * html/HTMLFormControlElement.cpp: |
+ (WebCore::HTMLFormControlElement::labels): |
+ |
+2011-04-22 Adrienne Walker <enne@google.com> |
+ |
+ Reviewed by James Robinson. |
+ |
+ [chromium] Remove minimum texture size for compositor tiling |
+ https://bugs.webkit.org/show_bug.cgi?id=59236 |
+ |
+ This was mistakenly committed in r83915 and causes bad filtering |
+ on very small textures. |
+ |
+ Tests: compositing/images/content-image-change.html |
+ |
+ * platform/graphics/chromium/LayerTilerChromium.cpp: |
+ (WebCore::LayerTilerChromium::setTileSize): |
+ |
+2011-04-22 Beth Dakin <bdakin@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=59223 |
+ REGRESSION (r82185): iframe not displayed after navigation back to cached page |
+ -and corresponding- |
+ <rdar://problem/9323763> |
+ |
+ Cache the scroll position in Document::setInPageCache() instead of |
+ FrameView::resetScrollbarsAndClearContentsSize(), and only clear the contents size |
+ for the mainFrame(). Still resetScrollbars() for other frames, which matches pre- |
+ r82185 code. |
+ * dom/Document.cpp: |
+ (WebCore::Document::setInPageCache): |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::resetScrollbarsAndClearContentsSize): |
+ |
+2011-04-22 Joseph Pecoraro <joepeck@webkit.org> |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ Let the RenderTheme decide if an input type should have a spinner |
+ https://bugs.webkit.org/show_bug.cgi?id=59246 |
+ |
+ Whether an input should have a spinner or not should |
+ be a UI decision made by the RenderTheme. Remove |
+ WebCore's default hasSpinButton values and defer |
+ to the RenderTheme. |
+ |
+ No change in functionality, covered by existing tests. |
+ |
+ * html/HTMLInputElement.cpp: |
+ (WebCore::HTMLInputElement::isRangeControl): expose isRangeControl |
+ for possible RenderTheme queries. |
+ * html/HTMLInputElement.h: |
+ |
+ * rendering/RenderTextControlSingleLine.cpp: |
+ (WebCore::RenderTextControlSingleLine::createSubtreeIfNeeded): |
+ (WebCore::RenderTextControlSingleLine::createInnerTextStyle): |
+ For both of these ask the render theme if the input should |
+ have a spinner. |
+ |
+ * rendering/RenderTheme.cpp: |
+ (WebCore::RenderTheme::shouldHaveSpinButtonForInputElement): |
+ Steppable inputs (number, date, and range) but not range |
+ should have spinners. This does not change existing behavior. |
+ * rendering/RenderTheme.h: |
+ |
+ * dom/InputElement.h: |
+ * html/BaseDateAndTimeInputType.cpp: remove hasSpinButton. |
+ * html/BaseDateAndTimeInputType.h: remove hasSpinButton. |
+ * html/InputType.cpp: remove hasSpinButton. |
+ * html/InputType.h: remove hasSpinButton. |
+ * html/NumberInputType.cpp: remove hasSpinButton. |
+ * html/NumberInputType.h: remove hasSpinButton. |
+ |
+2011-04-22 Joseph Pecoraro <joepeck@webkit.org> |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ Don't rely on hasSpinButton to check if an input type can be stepped |
+ https://bugs.webkit.org/show_bug.cgi?id=59224 |
+ |
+ Add isSteppable to know for certain if an input type can |
+ be stepped or not. Previously we were checking if the input |
+ has a spin control or is the range input type. These are UI |
+ details, but the step functionality does not require there |
+ to be a UI. |
+ |
+ No change in behavior, covered by existing tests. |
+ |
+ * html/BaseDateAndTimeInputType.cpp: |
+ (WebCore::BaseDateAndTimeInputType::isSteppable): |
+ * html/BaseDateAndTimeInputType.h: |
+ * html/HTMLInputElement.cpp: |
+ (WebCore::HTMLInputElement::isSteppable): |
+ (WebCore::HTMLInputElement::stepUpFromRenderer): remove the |
+ check for a spinner and range control and instead ASSERT |
+ that the type is steppable. |
+ * html/HTMLInputElement.h: |
+ * html/InputType.cpp: |
+ (WebCore::InputType::isSteppable): |
+ * html/InputType.h: |
+ * html/NumberInputType.cpp: |
+ (WebCore::NumberInputType::isSteppable): |
+ * html/NumberInputType.h: |
+ * html/RangeInputType.cpp: |
+ (WebCore::RangeInputType::isSteppable): |
+ * html/RangeInputType.h: |
+ |
+2011-04-22 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ CSP frame-src is missing |
+ https://bugs.webkit.org/show_bug.cgi?id=58643 |
+ |
+ In this patch, I've audited all the callers to |
+ SecurityOrigin::canDisplay to make sure they all the appropriate |
+ Content-Security-Policy method as well. I'm not sure how to test all |
+ these cases, but making this changes fixed the frame-src test. |
+ |
+ * loader/SubframeLoader.cpp: |
+ (WebCore::SubframeLoader::loadMediaPlayerProxyPlugin): |
+ (WebCore::SubframeLoader::createJavaAppletWidget): |
+ (WebCore::SubframeLoader::loadSubframe): |
+ (WebCore::SubframeLoader::loadPlugin): |
+ * loader/SubresourceLoader.cpp: |
+ (WebCore::SubresourceLoader::create): |
+ * loader/cache/CachedResourceLoader.cpp: |
+ (WebCore::CachedResourceLoader::canRequest): |
+ (WebCore::CachedResourceLoader::requestResource): |
+ - While I was understanding this code, I fixed the FIXME here. |
+ * loader/cache/CachedResourceLoader.h: |
+ |
+2011-04-22 Adrienne Walker <enne@google.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ Clear shared graphics context when destroying CanvasRenderingContext2D. |
+ https://bugs.webkit.org/show_bug.cgi?id=59212 |
+ |
+ * html/canvas/CanvasRenderingContext2D.cpp: |
+ (WebCore::CanvasRenderingContext2D::~CanvasRenderingContext2D): |
+ |
+2011-04-22 Levi Weintraub <leviw@chromium.org> |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ End/Home keys do not work in a contentEditable element |
+ https://bugs.webkit.org/show_bug.cgi?id=50564 |
+ |
+ Changing logical[Start/End]OfLine to return a valid VisiblePosition for for editable contexts that are |
+ inline and share a line with non-editable content. |
+ |
+ Test: editing/selection/modify-by-lineboundary-in-inline-editable-contexts.html |
+ |
+ * editing/visible_units.cpp: |
+ (WebCore::logicalStartOfLine): |
+ (WebCore::logicalEndOfLine): |
+ |
+2011-04-22 Dimitri Glazkov <dglazkov@chromium.org> |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ Make showTree/showTreeForThis aware of the shadow DOM |
+ https://bugs.webkit.org/show_bug.cgi?id=52965 |
+ |
+ No new tests, because this code is only compiled in DEBUG. |
+ |
+ * dom/Node.cpp: |
+ (WebCore::traverseTreeAndMark): Moved tree traversal logic here, adding |
+ recursive sub-traversal of the shadow tree. |
+ (WebCore::Node::showTreeAndMark): Changed to use the new tree traversal function. |
+ |
+2011-04-22 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by Gavin Barraclough and Oliver Hunt. |
+ |
+ Arrays should participate in global object forwarding fun |
+ https://bugs.webkit.org/show_bug.cgi?id=59215 |
+ |
+ Change callers of constructArray to use the new variant that takes |
+ a global object. |
+ |
+ * bindings/js/JSClipboardCustom.cpp: |
+ (WebCore::JSClipboard::types): |
+ * bindings/js/JSConsoleCustom.cpp: |
+ (WebCore::JSConsole::profiles): |
+ * bindings/js/JSJavaScriptCallFrameCustom.cpp: |
+ (WebCore::JSJavaScriptCallFrame::scopeChain): |
+ * bindings/js/JSMessageEventCustom.cpp: |
+ (WebCore::JSMessageEvent::ports): |
+ * bindings/js/JSScriptProfileNodeCustom.cpp: |
+ (WebCore::JSScriptProfileNode::children): |
+ * bindings/js/JSWebGLRenderingContextCustom.cpp: |
+ (WebCore::toJS): |
+ (WebCore::JSWebGLRenderingContext::getAttachedShaders): |
+ (WebCore::JSWebGLRenderingContext::getSupportedExtensions): |
+ |
+2011-04-22 Csaba Osztrogonác <ossy@webkit.org> |
+ |
+ [Qt] Unreviewed typo fix after r84522. |
+ |
+ * WebCore.pro: |
+ |
+2011-04-22 Daniel Bates <dbates@rim.com> |
+ |
+ Fix misspelled word in comment; tagets => targets. |
+ |
+ * page/EventHandler.cpp: |
+ (WebCore::EventHandler::handleTouchEvent): |
+ |
+2011-04-21 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Removed a use of markDOMObjectWrapper: Element and Document |
+ https://bugs.webkit.org/show_bug.cgi?id=59178 |
+ |
+ * GNUmakefile.list.am: |
+ * UseJSC.cmake: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * bindings/js/JSBindingsAllInOne.cpp: Build! |
+ |
+ * bindings/js/JSDOMImplementationCustom.cpp: |
+ (WebCore::JSDOMImplementationOwner::isReachableFromOpaqueRoots): |
+ (WebCore::JSDOMImplementationOwner::finalize): |
+ (WebCore::wrapperOwner): |
+ (WebCore::wrapperContext): |
+ (WebCore::toJS): Use the opaque roots system for DOMImplementation. Also, |
+ DOMImplementation does not need to keep its document wrapper alive, since |
+ there's no DOM API for getting a DOMImplementation's document. |
+ |
+ * bindings/js/JSDOMStringMapCustom.cpp: |
+ (WebCore::JSDOMStringMapOwner::isReachableFromOpaqueRoots): |
+ (WebCore::JSDOMStringMapOwner::finalize): |
+ (WebCore::wrapperOwner): |
+ (WebCore::wrapperContext): |
+ (WebCore::toJS): Use the opaque roots system for DOMStringMap. |
+ |
+ * bindings/js/JSDOMTokenListCustom.cpp: Added. |
+ (WebCore::JSDOMTokenListOwner::isReachableFromOpaqueRoots): |
+ (WebCore::JSDOMTokenListOwner::finalize): |
+ (WebCore::wrapperOwner): |
+ (WebCore::wrapperContext): |
+ (WebCore::toJS): Use the opaque roots system for DOMTokenList. |
+ |
+ * bindings/js/JSDocumentCustom.cpp: |
+ (WebCore::JSDocument::visitChildren): |
+ * bindings/js/JSElementCustom.cpp: No more need for explicit marking. |
+ |
+ * bindings/scripts/CodeGeneratorV8.pm: Boo on whoever made this design decision. |
+ |
+ * dom/DOMImplementation.cpp: |
+ (WebCore::DOMImplementation::DOMImplementation): |
+ (WebCore::DOMImplementation::createDocument): |
+ (WebCore::DOMImplementation::createHTMLDocument): |
+ * dom/DOMImplementation.h: |
+ (WebCore::DOMImplementation::create): |
+ (WebCore::DOMImplementation::ref): |
+ (WebCore::DOMImplementation::deref): |
+ (WebCore::DOMImplementation::document): Fixed the DOMImplementation |
+ ownership model to match other satellite objects in the DOM. This was |
+ necessary to fix some crashes due to incorrect reference counting. |
+ |
+ * dom/DOMImplementation.idl: |
+ * dom/DOMStringMap.idl: |
+ * dom/Document.cpp: |
+ (WebCore::Document::~Document): |
+ * dom/Document.h: |
+ * dom/Element.cpp: |
+ * dom/Element.h: |
+ * dom/Element.idl: |
+ * html/DOMTokenList.idl: Updated to match changes above. |
+ |
+2011-04-22 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ Rounded corners don't looks smooth when background-image is a gradient |
+ https://bugs.webkit.org/show_bug.cgi?id=59213 |
+ |
+ Add the "shrink background" behavior to the slow background |
+ drawing code path. |
+ |
+ Test: fast/backgrounds/gradient-background-leakage.html |
+ |
+ * rendering/RenderBoxModelObject.cpp: |
+ (WebCore::backgroundRectAdjustedForBleedAvoidance): |
+ Utility function that returns a shrunk border rect based |
+ on the context scale. |
+ (WebCore::RenderBoxModelObject::paintFillLayerExtended): |
+ Use backgroundRectAdjustedForBleedAvoidance() in both the fast |
+ and slow paths. |
+ |
+2011-04-22 Xiaomei Ji <xji@chromium.org> |
+ |
+ Reviewed by David Hyatt. |
+ |
+ Local caret rectangle calculation should be relative to its containing block. |
+ https://bugs.webkit.org/show_bug.cgi?id=49508. |
+ |
+ InlineBox and caret rectangle are positioned relative to its containing block. |
+ So its left and right edge value should be relative to its containing block as well. |
+ |
+ Test: fast/forms/cursor-at-editable-content-boundary.html |
+ |
+ * rendering/RenderText.cpp: |
+ (WebCore::RenderText::localCaretRect): |
+ |
+2011-04-22 Levi Weintraub <leviw@chromium.org> |
+ |
+ Reviewed by Darin Fisher. |
+ |
+ REGRESSION: left property broken with position:fixed elements in RTL documents |
+ https://bugs.webkit.org/show_bug.cgi?id=59204 |
+ http://code.google.com/p/chromium/issues/detail?id=80216 |
+ |
+ Test: fast/block/positioning/rtl-fixed-positioning.html |
+ fast/block/positioning/vertical-rl/fixed-positioning.html |
+ |
+ Returning the proper scroll offsets for fixed position content in RTL documents. |
+ We regressed to X and Y offsets not updating while scrolling when we clamped to zero, |
+ but RTL documents scroll in negative space. |
+ |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::scrollXForFixedPosition): Properly handling RTL documents where |
+ the scroll origin and offsets are negative. |
+ (WebCore::FrameView::scrollYForFixedPosition): Properly handling vertical writing- |
+ mode RTL documents, like above. |
+ * platform/ScrollView.h: |
+ (WebCore::ScrollView::scrollOrigin): Changing this accessor to const. |
+ |
+2011-04-22 Justin Schuh <jschuh@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Should assert on invalid type in FEComponentTransfer::apply |
+ https://bugs.webkit.org/show_bug.cgi?id=57959 |
+ |
+ No functional change. No new tests needed. |
+ |
+ * platform/graphics/filters/FEComponentTransfer.cpp: |
+ (WebCore::FEComponentTransfer::apply): |
+ |
+2011-04-22 MORITA Hajime <morrita@google.com> |
+ |
+ Unreviewed crash fix. |
+ |
+ * html/HTMLDetailsElement.cpp: |
+ (WebCore::DetailsSummaryElement::shouldInclude): |
+ |
+2011-04-22 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Exclude scrollbars from the dirty rect passed to the document |
+ https://bugs.webkit.org/show_bug.cgi?id=59206 |
+ |
+ No test added because functionality is unchanged. |
+ |
+ * platform/ScrollView.cpp: |
+ (WebCore::ScrollView::paint): Compute a clip rectangle (analogous to the frame of the clip view |
+ in an NSScrollView) by subtracting non-overlay scrollbars, and intersect the document dirty rect |
+ with it. If the result is empty, avoid calling paintContents(). |
+ |
+2011-04-22 Jer Noble <jer.noble@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ Disable MediaPlayerPrivateAVFoundation when using old full screen mode. |
+ https://bugs.webkit.org/show_bug.cgi?id=59147 |
+ |
+ Add a new isAVFoundationEnabled setting and check it before adding |
+ MediaPlayerPrivateAVFoundationObjC to the media engine registry. |
+ |
+ * WebCore.exp.in: |
+ * page/Settings.cpp: |
+ * page/Settings.h: |
+ (WebCore::Settings::setAVFoundationEnabled): Added. |
+ (WebCore::Settings::isAVFoundationEnabled): Added. |
+ * platform/graphics/MediaPlayer.cpp: |
+ (WebCore::installedMediaEngines): Check the settings added above. |
+ |
+2011-04-22 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by Gavin Barraclough. |
+ |
+ Remove remaining uses of deprecated version of toJS |
+ https://bugs.webkit.org/show_bug.cgi?id=59170 |
+ |
+ * bindings/js/JSCanvasRenderingContext2DCustom.cpp: |
+ (WebCore::toJS): |
+ (WebCore::JSCanvasRenderingContext2D::strokeStyle): |
+ (WebCore::JSCanvasRenderingContext2D::fillStyle): |
+ (WebCore::JSCanvasRenderingContext2D::createPattern): |
+ * bindings/js/JSConsoleCustom.cpp: |
+ (WebCore::JSConsole::profiles): |
+ * bindings/js/JSCustomSQLStatementErrorCallback.cpp: |
+ (WebCore::JSSQLStatementErrorCallback::handleEvent): |
+ * bindings/js/JSDOMBinding.h: |
+ * bindings/js/JSDOMMimeTypeArrayCustom.cpp: |
+ (WebCore::JSDOMMimeTypeArray::nameGetter): |
+ * bindings/js/JSDOMPluginArrayCustom.cpp: |
+ (WebCore::JSDOMPluginArray::nameGetter): |
+ * bindings/js/JSDOMPluginCustom.cpp: |
+ (WebCore::JSDOMPlugin::nameGetter): |
+ * bindings/js/JSDOMWindowCustom.cpp: |
+ (WebCore::namedItemGetter): |
+ (WebCore::JSDOMWindow::event): |
+ * bindings/js/JSEventTarget.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSInjectedScriptHostCustom.cpp: |
+ Propagate stored global object down access chain. |
+ |
+ (WebCore::InjectedScriptHost::nodeAsScriptValue): |
+ Replace use of deprecated toJS overload with explicit call to deprecatedGlobalObjectForPrototype. |
+ |
+ (WebCore::JSInjectedScriptHost::currentCallFrame): |
+ (WebCore::JSInjectedScriptHost::inspectedNode): |
+ * bindings/js/JSNamedNodeMapCustom.cpp: |
+ (WebCore::JSNamedNodeMap::nameGetter): |
+ * bindings/js/JSNodeListCustom.cpp: |
+ (WebCore::callNodeList): |
+ (WebCore::JSNodeList::nameGetter): |
+ * bindings/js/JSScriptProfileNodeCustom.cpp: |
+ (WebCore::JSScriptProfileNode::children): |
+ * bindings/js/JSStyleSheetListCustom.cpp: |
+ (WebCore::JSStyleSheetList::nameGetter): |
+ * bindings/js/JSXSLTProcessorCustom.cpp: |
+ (WebCore::JSXSLTProcessor::transformToFragment): |
+ (WebCore::JSXSLTProcessor::transformToDocument): |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ Propagate stored global object down access chain. |
+ |
+ * css/MediaQueryListListener.cpp: |
+ (WebCore::MediaQueryListListener::queryChanged): |
+ Replace use of deprecated toJS overload with explicit call to deprecatedGlobalObjectForPrototype, |
+ we will want refactor this to match other callbacks so we can remove this. |
+ |
+2011-04-22 Naoki Takano <takano.naoki@gmail.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ [Chromium]Mac UI polish to add left/right padding space for autofill popup window. |
+ https://bugs.webkit.org/show_bug.cgi?id=58960 |
+ http://code.google.com/p/chromium/issues/detail?id=51077 |
+ |
+ No new test because Chromium Autofill popup window doesn't have any test framework. |
+ This assumes only AutofillPopupMenuClient gives TexFieldPart appearance. |
+ Only if Autofill popup window has right/left padding. |
+ The code should be removed after Autofill popup window logic is separated from WebKit to Chromium. |
+ |
+ * rendering/RenderThemeChromiumMac.h: Add popupInternalPaddingRight() and popupInternalPaddingLeft(). |
+ * rendering/RenderThemeChromiumMac.mm: |
+ (WebCore::RenderThemeChromiumMac::popupInternalPaddingLeft): Return padding size only if TextFieldPart is specified. |
+ (WebCore::RenderThemeChromiumMac::popupInternalPaddingRight): Return padding size only if TextFieldPart is specified. |
+ |
+2011-04-21 Adam Roben <aroben@apple.com> |
+ |
+ Add code to enumerate all the functions imported by a particular Windows binary |
+ |
+ PEImage is where all the logic starts. It represents a single Portable Executable (PE) |
+ binary. The various enumerator classes are used in conjunction with PEImage to enumerate the |
+ modules and functions imported by that image. |
+ |
+ I couldn't figure out how to write a test for this. :-( |
+ |
+ Part of <http://webkit.org/b/51063> <rdar://problem/8769281> REGRESSION (WebKit2): No |
+ context menu appears when right-clicking on windowless Flash plugin |
+ |
+ Reviewed by Sam Weinig, and given a good once-over by Jeff Miller. |
+ |
+ * WebCore.vcproj/WebCore.vcproj: Added the new files. |
+ |
+ * platform/win/PEImage.cpp: Added. |
+ (WebCore::PEImage::PEImage): Checks that the module contains the expected magic numbers, |
+ then initializes m_ntHeaders using the module. |
+ (WebCore::PEImage::convertRVAToAddress): Converts a relative virtual address (RVA) to a |
+ non-relative address, which can then be dereferenced. (The PE format uses RVAs in lots of |
+ places.) |
+ (WebCore::PEImage::dataDirectoryEntryAddress): Returns the address of the data associated |
+ with the given entry in the image's DataDirectory. |
+ |
+ * platform/win/PEImage.h: Added. |
+ (WebCore::PEImage::isValid): We're a valid image if the validation code in our constructor |
+ succeeded, in which case m_ntHeaders will be set. |
+ |
+ * platform/win/ImportedFunctionsEnumerator.cpp: Added. This class enumerates all the |
+ functions imported by the image from a single module by walking the Import Name Table (INT) |
+ and Import Address Table (IAT). |
+ (WebCore::ImportedFunctionsEnumerator::ImportedFunctionsEnumerator): Initialize our members. |
+ (WebCore::ImportedFunctionsEnumerator::isAtEnd): The INT and IAT are both 0-terminated |
+ arrays. |
+ (WebCore::ImportedFunctionsEnumerator::next): Step to the next entries in the INT and IAT. |
+ (WebCore::ImportedFunctionsEnumerator::currentFunctionName): Gets the address of the |
+ function's name, if possible, using the INT. |
+ (WebCore::ImportedFunctionsEnumerator::addressOfCurrentFunctionPointer): Returns the address |
+ of the function pointer for the current function. This function pointer is used whenever |
+ code in m_image calls the current function. (A future patch will take advantage of this by |
+ overwriting this function pointer with another, thereby redirecting calls to that function.) |
+ |
+ * platform/win/ImportedFunctionsEnumerator.h: Added. |
+ |
+ * platform/win/ImportedModulesEnumeratorBase.h: Added. This base class is used by |
+ ImportedModulesEnumerator and DelayLoadedModulesEnumerator, and allows code to be written |
+ that works with either one. |
+ |
+ * platform/win/ImportedModulesEnumerator.cpp: Added. This class enumerates all the |
+ non-delay-loaded modules imported by the image by walking the image's import array. |
+ (WebCore::ImportedModulesEnumerator::ImportedModulesEnumerator): Module import information |
+ is stored in the IMAGE_DIRECTORY_ENTRY_IMPORT entry of the image's DataDirectory. |
+ (WebCore::ImportedModulesEnumerator::isAtEnd): The import array is 0-terminated. |
+ (WebCore::ImportedModulesEnumerator::next): Steps along the import array. |
+ (WebCore::ImportedModulesEnumerator::currentModuleName): Gets the address of the module's |
+ name. |
+ (WebCore::ImportedModulesEnumerator::functionsEnumerator): Get's the addresses of the INT |
+ and IAT for this module and wraps them in an ImportedFunctionsEnumerator. |
+ |
+ * platform/win/ImportedModulesEnumerator.h: Added. |
+ |
+ * platform/win/DelayLoadedModulesEnumerator.cpp: Added. This class enumerates all the |
+ delay-loaded modules that are imported by the given PEImage by walking the image's |
+ ImageDelayDescr array. |
+ (WebCore::DelayLoadedModulesEnumerator::DelayLoadedModulesEnumerator): Delay-load |
+ information is stored in the IMAGE_DIRECTORY_ENTRY_DELAY_IMPORT entry of the image's |
+ DataDirectory. |
+ (WebCore::DelayLoadedModulesEnumerator::isAtEnd): The delay-loaded modules array is |
+ 0-terminated. |
+ (WebCore::DelayLoadedModulesEnumerator::next): Steps along the delay-loaded modules array. |
+ (WebCore::DelayLoadedModulesEnumerator::currentModuleName): Gets the address of the module's |
+ name out of the descriptor. |
+ (WebCore::DelayLoadedModulesEnumerator::functionsEnumerator): Gets the INT and IAT for the |
+ current module out of the descriptor and makes an ImportedFunctionsEnumerator from them. |
+ (WebCore::DelayLoadedModulesEnumerator::convertPotentialRVAToAddress): Pre-VC7.0 images |
+ stored non-relative addresses in the ImageDelayDescr structures, while VC7.0 and later |
+ images store RVAs. This function abstracts away the difference. |
+ |
+ * platform/win/DelayLoadedModulesEnumerator.h: Added. |
+ |
+2011-04-21 Adam Roben <aroben@apple.com> |
+ |
+ Add a shared do-nothing window class that anyone can use |
+ |
+ Fixes <http://webkit.org/b/59102> WebKit has too many dummy window classes |
+ |
+ Reviewed by Anders Carlsson. |
+ |
+ * WebCore.vcproj/WebCore.vcproj: Added DefWndProcWindowClass.{cpp,h}. Moved Arena.cpp out of |
+ platform/win and into platform. Let VS reorder other files. |
+ |
+ * platform/graphics/ca/win/CACFLayerTreeHost.cpp: |
+ (WebCore::CACFLayerTreeHost::acceleratedCompositingAvailable): Changed to use the shared |
+ dummy window class instead of creating our own. |
+ |
+ * platform/win/DefWndProcWindowClass.cpp: Added. |
+ (WebCore::registerClass): Register it! |
+ (WebCore::defWndProcWindowClassName): Register it and return it! |
+ |
+ * platform/win/DefWndProcWindowClass.h: Added. |
+ |
+2011-04-21 MORITA Hajime <morrita@google.com> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ Content of <details> should be forwarded through the shadow DOM |
+ https://bugs.webkit.org/show_bug.cgi?id=59055 |
+ |
+ - Made ShadowContentElement abstract and added shouldInclude() pure virtual method. |
+ - Added SummaryContentElement, DetailsContentElement, DetailsSummaryElement as |
+ subclasses of ShadowContentElement. |
+ - Gave two shadow children for HTMLDetailsElement. One is DetailsContentElement, that |
+ handles non-summary contents of <details>. Another is SummaryContentElement, that |
+ handles the main summary element. |
+ - Removed most of RenderDetails custom layout mechanism, which is replaced by |
+ ShadowContentElement subclasses. |
+ |
+ Tests: fast/html/details-add-child-1.html |
+ fast/html/details-add-child-2.html |
+ fast/html/details-remove-child-1.html |
+ fast/html/details-remove-child-2.html |
+ |
+ * dom/ShadowRoot.cpp: |
+ (WebCore::ShadowRoot::contentContainerFor): |
+ (WebCore::ShadowRoot::hostChildrenChanged): |
+ (WebCore::ShadowRoot::hasContentElement): |
+ * dom/ShadowRoot.h: |
+ * html/HTMLDetailsElement.cpp: |
+ (WebCore::DetailsContentElement::DetailsContentElement): |
+ (WebCore::DetailsContentElement::create): |
+ (WebCore::DetailsContentElement::shouldInclude): |
+ (WebCore::DetailsSummaryElement::DetailsSummaryElement): |
+ (WebCore::DetailsSummaryElement::create): |
+ (WebCore::DetailsSummaryElement::shouldInclude): |
+ (WebCore::HTMLDetailsElement::create): |
+ (WebCore::HTMLDetailsElement::HTMLDetailsElement): |
+ (WebCore::HTMLDetailsElement::ensureShadowSubtreeOf): |
+ (WebCore::findSummaryFor): |
+ (WebCore::HTMLDetailsElement::ensureMainSummary): |
+ (WebCore::HTMLDetailsElement::refreshMainSummary): |
+ (WebCore::HTMLDetailsElement::createShadowSubtree): |
+ * html/HTMLDetailsElement.h: |
+ * html/HTMLSummaryElement.cpp: |
+ (WebCore::SummaryContentElement::SummaryContentElement): |
+ (WebCore::SummaryContentElement::shouldInclude): |
+ (WebCore::SummaryContentElement::create): |
+ (WebCore::HTMLSummaryElement::createShadowSubtree): |
+ * html/shadow/ShadowContentElement.h: |
+ * rendering/RenderDetails.cpp: |
+ (WebCore::RenderDetails::RenderDetails): |
+ (WebCore::RenderDetails::addChild): |
+ * rendering/RenderDetails.h: |
+ * rendering/RenderSummary.cpp: |
+ * rendering/RenderSummary.h: |
+ |
+2011-04-21 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ Extra GraphicsContext::save() in InlineTextBox::paintDecoration() |
+ https://bugs.webkit.org/show_bug.cgi?id=59109 |
+ |
+ Remove an extraneous GraphicsContext::save() in InlineTextBox::paintDecoration(). |
+ |
+ Add an assertion that fires when a GraphicsContext() is destroyed with a non- |
+ empty state stack. Fix canvas to avoid this assertion, since bad content might |
+ trigger it. |
+ |
+ Also add an assertion that fires for mismatched begin/endTransparencyLayers. |
+ |
+ Test: fast/text/multiple-text-shadow-alpha.html |
+ |
+ * html/HTMLCanvasElement.cpp: |
+ (WebCore::HTMLCanvasElement::~HTMLCanvasElement): |
+ Make sure that the CanvasRenderingContext goes away before the ImageBuffer, |
+ so that CanvasRenderingContext2D has a chance to unwind the state stack in |
+ the ImageBuffer's GraphicsContext. |
+ |
+ * html/canvas/CanvasRenderingContext2D.cpp: |
+ (WebCore::CanvasRenderingContext2D::~CanvasRenderingContext2D): |
+ Balance unmatched state saves in the canvas context. |
+ |
+ * platform/graphics/GraphicsContext.cpp: |
+ (WebCore::GraphicsContext::~GraphicsContext): |
+ Assert that the stack is empty. |
+ |
+ * platform/graphics/cg/GraphicsContextPlatformPrivateCG.h: |
+ (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): |
+ (WebCore::GraphicsContextPlatformPrivate::~GraphicsContextPlatformPrivate): |
+ (WebCore::GraphicsContextPlatformPrivate::beginTransparencyLayer): |
+ (WebCore::GraphicsContextPlatformPrivate::endTransparencyLayer): |
+ Changes to allow for asserting for unmatched transparency layer begin/ends. |
+ |
+ * rendering/InlineTextBox.cpp: |
+ (WebCore::InlineTextBox::paintDecoration): |
+ Remove an unbalanced save(). |
+ |
+2011-04-22 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: add Save As context menu item to the TextViewer. |
+ https://bugs.webkit.org/show_bug.cgi?id=59196 |
+ |
+ * inspector/front-end/ResourceView.js: |
+ (WebInspector.ResourceSourceFrame.prototype.requestContent): |
+ (WebInspector.ResourceSourceFrame.prototype.suggestedFileName): |
+ (WebInspector.RevisionSourceFrame.prototype.requestContent): |
+ (WebInspector.RevisionSourceFrame.prototype.suggestedFileName): |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._createSourceFrame): |
+ (WebInspector.SourceFrameDelegateForScriptsPanel): |
+ (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.formatSourceFilesToggled): |
+ (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.suggestedFileName): |
+ * inspector/front-end/SourceFrame.js: |
+ (WebInspector.SourceFrame.prototype._initializeTextViewer): |
+ (WebInspector.SourceFrame.prototype.populateTextAreaContextMenu): |
+ (WebInspector.SourceFrame.prototype.suggestedFileName): |
+ (WebInspector.SourceFrameDelegate.prototype.formatSourceFilesToggled): |
+ (WebInspector.SourceFrameDelegate.prototype.suggestedFileName): |
+ * inspector/front-end/TextViewer.js: |
+ (WebInspector.TextViewer.prototype._contextMenu): |
+ (WebInspector.TextViewerDelegate.prototype.cancelEditing): |
+ (WebInspector.TextViewerDelegate.prototype.populateLineGutterContextMenu): |
+ (WebInspector.TextViewerDelegate.prototype.populateTextAreaContextMenu): |
+ (WebInspector.TextViewerDelegate.prototype.suggestedFileName): |
+ |
+2011-04-22 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [Chromium] Move detailed heap snapshots storage and processing into workers. |
+ https://bugs.webkit.org/show_bug.cgi?id=59094 |
+ |
+ * WebCore.gypi: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * gyp/streamline-inspector-source.sh: |
+ * inspector/front-end/DetailedHeapshotGridNodes.js: |
+ (WebInspector.HeapSnapshotGridNode.prototype.dispose): |
+ (WebInspector.HeapSnapshotObjectNode.prototype._createProvider): |
+ (WebInspector.HeapSnapshotInstanceNode.prototype._createProvider): |
+ (WebInspector.HeapSnapshotConstructorNode.prototype._createNodesProvider): |
+ (WebInspector.HeapSnapshotIteratorsTuple.prototype.dispose): |
+ (WebInspector.HeapSnapshotDiffNode.prototype.calculateDiff.diffCalculated): |
+ (WebInspector.HeapSnapshotDiffNode.prototype._createNodesProvider.createProvider): |
+ (WebInspector.HeapSnapshotDiffNode.prototype._createNodesProvider): |
+ (WebInspector.HeapSnapshotDominatorObjectNode.prototype._createProvider): |
+ (MixInSnapshotNodeFunctions): |
+ * inspector/front-end/DetailedHeapshotView.js: |
+ (WebInspector.HeapSnapshotSortableDataGrid.prototype.dispose): |
+ (WebInspector.HeapSnapshotSortableDataGrid.prototype.resetSortingCache): |
+ (WebInspector.HeapSnapshotDiffDataGrid.prototype.setBaseDataSource): |
+ (WebInspector.HeapSnapshotRetainingPathsList.prototype.dispose): |
+ (WebInspector.HeapSnapshotRetainingPathsList.prototype._resetPaths): |
+ (WebInspector.HeapSnapshotRetainingPathsList.prototype.setDataSource): |
+ (WebInspector.HeapSnapshotRetainingPathsList.prototype.showNext): |
+ (WebInspector.HeapSnapshotRetainingPathsList.prototype.searchCancelled): |
+ (WebInspector.HeapSnapshotRetainingPathsList.prototype._setRootChildrenForFinder): |
+ (WebInspector.DetailedHeapshotView.prototype.dispose): |
+ (WebInspector.DetailedHeapshotView.prototype.get profileWrapper): |
+ (WebInspector.DetailedHeapshotView.prototype.get baseProfileWrapper): |
+ * inspector/front-end/HeapSnapshot.js: |
+ (WebInspector.HeapSnapshotLoader): |
+ (WebInspector.HeapSnapshotLoader.prototype.finishLoading): |
+ (WebInspector.HeapSnapshotLoader.prototype.pushJSONChunk): |
+ (WebInspector.HeapSnapshot.prototype.pushBaseIds): |
+ (WebInspector.HeapSnapshot.prototype.createDiff): |
+ (WebInspector.HeapSnapshot.prototype._parseFilter): |
+ (WebInspector.HeapSnapshot.prototype.createEdgesProvider): |
+ (WebInspector.HeapSnapshot.prototype.createNodesProvider): |
+ (WebInspector.HeapSnapshot.prototype.createPathFinder): |
+ (WebInspector.HeapSnapshot.prototype.updateStaticData): |
+ (WebInspector.HeapSnapshotPathFinder): |
+ (WebInspector.HeapSnapshotPathFinder.prototype.updateRoots): |
+ (WebInspector.HeapSnapshotPathFinder.prototype._reset): |
+ * inspector/front-end/HeapSnapshotProxy.js: |
+ (WebInspector.HeapSnapshotRealWorker): |
+ (WebInspector.HeapSnapshotRealWorker.prototype._messageReceived): |
+ (WebInspector.HeapSnapshotRealWorker.prototype.postMessage): |
+ (WebInspector.HeapSnapshotRealWorker.prototype.terminate): |
+ (WebInspector.HeapSnapshotFakeWorker): |
+ (WebInspector.HeapSnapshotFakeWorker.prototype.postMessage): |
+ (WebInspector.HeapSnapshotFakeWorker.prototype.terminate): |
+ (WebInspector.HeapSnapshotFakeWorker.prototype._postMessageFromWorker): |
+ (WebInspector.HeapSnapshotWorker): |
+ (WebInspector.HeapSnapshotWorker.prototype.createObject): |
+ (WebInspector.HeapSnapshotWorker.prototype.dispose): |
+ (WebInspector.HeapSnapshotWorker.prototype.disposeObject): |
+ (WebInspector.HeapSnapshotWorker.prototype.callGetter): |
+ (WebInspector.HeapSnapshotWorker.prototype.callFactoryMethod.wrapCallback): |
+ (WebInspector.HeapSnapshotWorker.prototype.callFactoryMethod): |
+ (WebInspector.HeapSnapshotWorker.prototype.callMethod): |
+ (WebInspector.HeapSnapshotWorker.prototype._findFunction): |
+ (WebInspector.HeapSnapshotWorker.prototype._messageReceived): |
+ (WebInspector.HeapSnapshotWorker.prototype._postMessage): |
+ (WebInspector.HeapSnapshotProxyObject): |
+ (WebInspector.HeapSnapshotProxyObject.prototype._callWorker): |
+ (WebInspector.HeapSnapshotProxyObject.prototype.dispose): |
+ (WebInspector.HeapSnapshotProxyObject.prototype.disposeWorker): |
+ (WebInspector.HeapSnapshotProxyObject.prototype.callFactoryMethod): |
+ (WebInspector.HeapSnapshotProxyObject.prototype.callGetter): |
+ (WebInspector.HeapSnapshotProxyObject.prototype.callMethod): |
+ (WebInspector.HeapSnapshotLoaderProxy): |
+ (WebInspector.HeapSnapshotLoaderProxy.prototype.finishLoading.callLoadCallbacks): |
+ (WebInspector.HeapSnapshotLoaderProxy.prototype.finishLoading.updateStaticData): |
+ (WebInspector.HeapSnapshotLoaderProxy.prototype.finishLoading): |
+ (WebInspector.HeapSnapshotLoaderProxy.prototype.get loaded): |
+ (WebInspector.HeapSnapshotLoaderProxy.prototype.startLoading): |
+ (WebInspector.HeapSnapshotLoaderProxy.prototype.pushJSONChunk): |
+ (WebInspector.HeapSnapshotProxy): |
+ (WebInspector.HeapSnapshotProxy.prototype.aggregates): |
+ (WebInspector.HeapSnapshotProxy.prototype.createDiff): |
+ (WebInspector.HeapSnapshotProxy.prototype.createEdgesProvider): |
+ (WebInspector.HeapSnapshotProxy.prototype.createNodesProvider): |
+ (WebInspector.HeapSnapshotProxy.prototype.createPathFinder): |
+ (WebInspector.HeapSnapshotProxy.prototype.dispose): |
+ (WebInspector.HeapSnapshotProxy.prototype.finishLoading): |
+ (WebInspector.HeapSnapshotProxy.prototype.get loaded): |
+ (WebInspector.HeapSnapshotProxy.prototype.get nodeCount): |
+ (WebInspector.HeapSnapshotProxy.prototype.nodeFieldValuesByIndex): |
+ (WebInspector.HeapSnapshotProxy.prototype.pushBaseIds): |
+ (WebInspector.HeapSnapshotProxy.prototype.get rootNodeIndex): |
+ (WebInspector.HeapSnapshotProxy.prototype.updateStaticData): |
+ (WebInspector.HeapSnapshotProxy.prototype.startLoading): |
+ (WebInspector.HeapSnapshotProxy.prototype.get totalSize): |
+ (WebInspector.HeapSnapshotProxy.prototype.get uid): |
+ (WebInspector.HeapSnapshotProviderProxy): |
+ (WebInspector.HeapSnapshotProviderProxy.prototype.isEmpty): |
+ (WebInspector.HeapSnapshotProviderProxy.prototype.serializeNextItems): |
+ (WebInspector.HeapSnapshotProviderProxy.prototype.sortAndRewind): |
+ (WebInspector.HeapSnapshotPathFinderProxy): |
+ (WebInspector.HeapSnapshotPathFinderProxy.prototype.findNext): |
+ (WebInspector.HeapSnapshotPathFinderProxy.prototype.updateRoots): |
+ (WebInspector.HeapSnapshotsDiffProxy): |
+ (WebInspector.HeapSnapshotsDiffProxy.prototype.calculate): |
+ (WebInspector.HeapSnapshotsDiffProxy.prototype.pushBaseIds): |
+ (WebInspector.HeapSnapshotsDiffProxy.prototype.pushBaseSelfSizes): |
+ * inspector/front-end/HeapSnapshotWorker.js: Added. |
+ (WebInspector.UIString): |
+ (postMessageWrapper): |
+ * inspector/front-end/HeapSnapshotWorkerDispatcher.js: Added. |
+ (WebInspector.HeapSnapshotWorkerDispatcher): |
+ (WebInspector.HeapSnapshotWorkerDispatcher.prototype._findFunction): |
+ (WebInspector.HeapSnapshotWorkerDispatcher.prototype.dispatchMessage): |
+ * inspector/front-end/ProfilesPanel.js: |
+ (WebInspector.ProfilesPanel.prototype.loadHeapSnapshot): |
+ (WebInspector.ProfilesPanel.prototype._finishHeapSnapshot.else.parsed): |
+ (WebInspector.ProfilesPanel.prototype._finishHeapSnapshot): |
+ * inspector/front-end/WebKit.qrc: |
+ * inspector/front-end/inspector.html: |
+ |
+2011-04-22 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: fix de-obfuscation of scripts inlined in html. |
+ https://bugs.webkit.org/show_bug.cgi?id=58538 |
+ |
+ Use html tokenizer to find script boundaries instead of relying on scriptParsed events. |
+ |
+ Test: inspector/debugger/script-formatter.html |
+ |
+ * WebCore.gypi: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * inspector/front-end/JavaScriptFormatter.js: Renamed from Source/WebCore/inspector/front-end/UglifyJS/JavaScriptFormatter.js. |
+ * inspector/front-end/ScriptFormatter.js: |
+ (WebInspector.ScriptFormatter): |
+ (WebInspector.ScriptFormatter.prototype.formatContent): |
+ (WebInspector.ScriptFormatter.prototype._didFormatContent): |
+ * inspector/front-end/ScriptFormatterWorker.js: |
+ (onmessage): |
+ (formatScript): |
+ (HTMLScriptFormatter): |
+ (HTMLScriptFormatter.prototype.format): |
+ (HTMLScriptFormatter.prototype.scriptStarted): |
+ (HTMLScriptFormatter.prototype.scriptEnded): |
+ (HTMLScriptFormatter.prototype.styleSheetStarted): |
+ (HTMLScriptFormatter.prototype.styleSheetEnded): |
+ * inspector/front-end/SourceFile.js: |
+ (WebInspector.FormattedSourceFile.prototype._didRequestContent): |
+ * inspector/front-end/SourceHTMLTokenizer.js: |
+ (WebInspector.SourceHTMLTokenizer.prototype.scriptStarted): |
+ (WebInspector.SourceHTMLTokenizer.prototype.scriptEnded): |
+ (WebInspector.SourceHTMLTokenizer.prototype.styleSheetStarted): |
+ (WebInspector.SourceHTMLTokenizer.prototype.styleSheetEnded): |
+ (WebInspector.SourceHTMLTokenizer.prototype.nextToken): |
+ * inspector/front-end/SourceHTMLTokenizer.re2js: |
+ * inspector/front-end/WebKit.qrc: |
+ |
+2011-04-22 Andrey Kosyakov <caseq@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [Extensions API] misc API cleanup |
+ https://bugs.webkit.org/show_bug.cgi?id=59194 |
+ |
+ * inspector/front-end/ExtensionAPISchema.json: |
+ * inspector/front-end/ExtensionPanel.js: |
+ (WebInspector.ExtensionSidebarPane.prototype._setObject): |
+ |
+2011-04-21 Pavel Feldman <pfeldman@google.com> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: group content scripts in the scripts panel drop down. |
+ https://bugs.webkit.org/show_bug.cgi?id=59108 |
+ |
+ * English.lproj/localizedStrings.js: |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect.optionCompare): |
+ (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect): |
+ |
+2011-04-22 Mike West <mkwst@google.com> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: Tiny improvement to UI for adding an XHR breakpoint |
+ https://bugs.webkit.org/show_bug.cgi?id=59100 |
+ |
+ Improve web inspector UI around adding XHR breakpoints by adding |
+ a textual label to the input-like `span` used to specify which |
+ XHR requests the breakpoint ought to target. |
+ |
+ * inspector/front-end/BreakpointsSidebarPane.js: |
+ (WebInspector.XHRBreakpointsSidebarPane.prototype._addButtonClicked.finishEditing): |
+ (WebInspector.XHRBreakpointsSidebarPane.prototype._addButtonClicked): |
+ * inspector/front-end/inspector.css: |
+ (#breakpoint-condition-input): |
+ |
+2011-04-22 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: js formatter should never insert new tokens or remove existing tokens and comments. |
+ https://bugs.webkit.org/show_bug.cgi?id=59191 |
+ |
+ * WebCore.gypi: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * inspector/front-end/ScriptFormatter.js: |
+ (WebInspector.ScriptFormatter.prototype.formatContent.didFormatChunks): |
+ (WebInspector.ScriptFormatter.prototype.formatContent): |
+ * inspector/front-end/ScriptFormatterWorker.js: |
+ (onmessage): |
+ * inspector/front-end/SourceFile.js: |
+ (WebInspector.SourceFile.prototype._loadResourceContent): |
+ (WebInspector.FormattedSourceFile.prototype._didRequestContent): |
+ (WebInspector.FormattedSourceMapping): |
+ (WebInspector.FormattedSourceMapping.prototype.scriptLocationToSourceLine): |
+ (WebInspector.FormattedSourceMapping.prototype.sourceLineToScriptLocation): |
+ (WebInspector.FormattedSourceMapping.prototype._convertPosition): |
+ * inspector/front-end/UglifyJS/JavaScriptFormatter.js: Added. |
+ (FormattedContentBuilder): Helper class for building formatted script content. |
+ (FormattedContentBuilder.prototype.addToken): |
+ (FormattedContentBuilder.prototype.addSpace): |
+ (FormattedContentBuilder.prototype.addNewLine): |
+ (FormattedContentBuilder.prototype.increaseNestingLevel): |
+ (FormattedContentBuilder.prototype.decreaseNestingLevel): |
+ (FormattedContentBuilder.prototype.content): |
+ (FormattedContentBuilder.prototype.mapping): |
+ (Tokenizer): Wrapper class for UglifyJS tokenizer. |
+ (Tokenizer.prototype.content): |
+ (Tokenizer.prototype.next): |
+ (Tokenizer.prototype._convertUglifyToken): |
+ (JavaScriptFormatter): New formatter based on UglifyJS tokenizer. |
+ (JavaScriptFormatter.prototype.format): |
+ * inspector/front-end/UglifyJS/process.js: Removed. |
+ * inspector/front-end/WebKit.qrc: |
+ |
+2011-04-22 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [Chromium] Refactor code prior to moving detailed heap snapshots into workers. |
+ https://bugs.webkit.org/show_bug.cgi?id=59189 |
+ |
+ * WebCore.gypi: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * inspector/front-end/BinarySearch.js: Added. |
+ * inspector/front-end/DetailedHeapshotGridNodes.js: |
+ (WebInspector.HeapSnapshotGridNode.prototype._populate.sorted): |
+ (WebInspector.HeapSnapshotGridNode.prototype._populate): |
+ (WebInspector.HeapSnapshotGridNode.prototype.populateChildren): |
+ (WebInspector.HeapSnapshotGridNode.prototype.sort.afterSort.afterPopulate): |
+ (WebInspector.HeapSnapshotGridNode.prototype.sort.afterSort): |
+ (WebInspector.HeapSnapshotGridNode.prototype.sort): |
+ (WebInspector.HeapSnapshotObjectNode): |
+ (WebInspector.HeapSnapshotObjectNode.prototype._createChildNode): |
+ (WebInspector.HeapSnapshotInstanceNode.prototype._createChildNode): |
+ (WebInspector.HeapSnapshotInstanceNode.prototype._enhanceData): |
+ (WebInspector.HeapSnapshotInstanceNode.prototype.get isDeletedNode): |
+ (WebInspector.HeapSnapshotDiffNode.prototype.calculateDiff.diffCalculated): |
+ (WebInspector.HeapSnapshotDiffNode.prototype.calculateDiff.baseIdsReceived): |
+ (WebInspector.HeapSnapshotDiffNode.prototype.get data): |
+ * inspector/front-end/DetailedHeapshotView.js: |
+ (WebInspector.HeapSnapshotSortableDataGrid.prototype.sortingChanged.sort): |
+ (WebInspector.HeapSnapshotSortableDataGrid.prototype.sortingChanged): |
+ (WebInspector.HeapSnapshotSortableDataGrid.prototype._performSorting): |
+ (WebInspector.HeapSnapshotSortableDataGrid.prototype.recursiveSortingEnter): |
+ (WebInspector.HeapSnapshotSortableDataGrid.prototype.recursiveSortingLeave): |
+ (WebInspector.HeapSnapshotContainmentDataGrid): |
+ (WebInspector.HeapSnapshotContainmentDataGrid.prototype._defaultPopulateCount.100.setDataSource): |
+ (WebInspector.HeapSnapshotContainmentDataGrid.prototype.sortingChanged): |
+ (WebInspector.HeapSnapshotDiffDataGrid.prototype.setBaseDataSource): |
+ (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren.baseAggregatesReceived.aggregatesReceived.addNodeIfNonZeroDiff): |
+ (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren.baseAggregatesReceived.aggregatesReceived): |
+ (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren): |
+ (WebInspector.HeapSnapshotDominatorsDataGrid): |
+ (WebInspector.HeapSnapshotDominatorsDataGrid.prototype._defaultPopulateCount.25.setDataSource): |
+ (WebInspector.HeapSnapshotDominatorsDataGrid.prototype.sortingChanged): |
+ (WebInspector.HeapSnapshotRetainingPathsList.prototype._performSorting): |
+ (WebInspector.DetailedHeapshotView.prototype._changeBase.baseProfileLoaded): |
+ (WebInspector.DetailedHeapshotView.prototype._changeBase): |
+ (WebInspector.DetailedHeapshotView.prototype._changeView): |
+ (WebInspector.DetailedHeapshotView.prototype._showStringContentPopup.displayString): |
+ (WebInspector.DetailedHeapshotView.prototype._showStringContentPopup): |
+ * inspector/front-end/HeapSnapshot.js: |
+ (WebInspector.HeapSnapshot.prototype.dispose): |
+ (WebInspector.HeapSnapshot.prototype.pushBaseNodeIds): |
+ (WebInspector.HeapSnapshotFilteredOrderedIterator.prototype.serializeNextItems): |
+ (WebInspector.HeapSnapshotFilteredOrderedIterator.prototype.sortAndRewind): |
+ (WebInspector.HeapSnapshotEdgesProvider.prototype._serialize): |
+ (WebInspector.HeapSnapshotNodesProvider.prototype._serialize): |
+ (WebInspector.HeapSnapshotsDiff.prototype.calculate): |
+ (WebInspector.HeapSnapshotsDiff.prototype.pushBaseIds): |
+ (WebInspector.HeapSnapshotsDiff.prototype.pushBaseSelfSizes): |
+ * inspector/front-end/HeapSnapshotProxy.js: |
+ (WebInspector.HeapSnapshotProxy.prototype.createEdgesProvider): |
+ (WebInspector.HeapSnapshotProxy.prototype.createNodesProvider): |
+ (WebInspector.HeapSnapshotProxy.prototype.pushBaseIds): |
+ (WebInspector.HeapSnapshotProviderProxy): |
+ (WebInspector.HeapSnapshotProviderProxy.prototype.serializeNextItems): |
+ (WebInspector.HeapSnapshotProviderProxy.prototype.sortAndRewind): |
+ (WebInspector.HeapSnapshotsDiffProxy.prototype.pushBaseIds): |
+ (WebInspector.HeapSnapshotsDiffProxy.prototype.pushBaseSelfSizes): |
+ * inspector/front-end/PleaseWaitMessage.js: |
+ (WebInspector.PleaseWaitMessage.prototype.showAndWaitFor): |
+ (WebInspector.PleaseWaitMessage.prototype.unlistenAndHide): |
+ * inspector/front-end/WebKit.qrc: |
+ * inspector/front-end/inspector.html: |
+ * inspector/front-end/utilities.js: |
+ |
+2011-04-22 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Unreviewed fix after r84484 -- conditional statement was inverted incorrectly. |
+ |
+ * inspector/front-end/ProfilesPanel.js: |
+ (WebInspector.ProfilesPanel.prototype.handleShortcut): |
+ |
+2011-04-22 Annie Sullivan <sullivan@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: Timeline is too narrow |
+ https://bugs.webkit.org/show_bug.cgi?id=58399 |
+ |
+ Changed Network Panel widths so that Timeline is 50%. |
+ Added titles to the values in the other columns, so that a tooltip will show the value if the column is too narrow. |
+ |
+ * inspector/front-end/NetworkPanel.js: |
+ (WebInspector.NetworkPanel.prototype._createTable): Changed column widths. |
+ (WebInspector.NetworkPanel.prototype._toggleGridMode): Changed column widths. |
+ (WebInspector.NetworkDataGridNode.prototype.refreshResource): Added titles to column values. |
+ (WebInspector.NetworkDataGridNode.prototype._refreshStatusCell): Ditto |
+ (WebInspector.NetworkDataGridNode.prototype._refreshSizeCell): Ditto |
+ (WebInspector.NetworkDataGridNode.prototype._refreshTimeCell): Ditto |
+ * inspector/front-end/utilities.js: |
+ (Element.prototype.setTextAndTitle): Added new method to set the textContent and title of an Element at the same time, so that it can have a tooltip. |
+ |
+2011-04-22 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ <rdar://problem/9322486> REGRESSION (r83397): Default controls fade out when playing audio-only media |
+ https://bugs.webkit.org/show_bug.cgi?id=59180 |
+ |
+ Test: media/audio-controls-do-not-fade-out.html |
+ |
+ * html/HTMLMediaElement.cpp: |
+ (WebCore::HTMLMediaElement::playbackProgressTimerFired): Only start fading out the controls if there is |
+ a video track. |
+ |
+2011-04-22 Luke Macpherson <macpherson@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Add FontItalics and FontSmallCaps enums to be used instead of boolean values. |
+ https://bugs.webkit.org/show_bug.cgi?id=59080 |
+ |
+ No new tests as no new functionality added. |
+ |
+ * css/CSSPrimitiveValueMappings.h: |
+ Add casts to/from FontItalics and FontSmallCaps |
+ * css/CSSStyleSelector.cpp: |
+ Use new casts defined in PrimitiveValueMappings.h |
+ * platform/graphics/Font.h: |
+ Use FontItalics enum. |
+ * platform/graphics/FontDescription.h: |
+ Define and use FontItalics and FontSmallCaps enums. |
+ |
+2011-04-21 Andrey Kosyakov <caseq@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: exception in front-end when user selects resource info for resource that does not have response headers yet |
+ https://bugs.webkit.org/show_bug.cgi?id=59123 |
+ |
+ * inspector/front-end/ResourceCookiesView.js: |
+ (WebInspector.ResourceCookiesView): |
+ (WebInspector.ResourceCookiesView.prototype.show): |
+ (WebInspector.ResourceCookiesView.prototype.get _gotCookies): |
+ (WebInspector.ResourceCookiesView.prototype._buildCookiesTable): |
+ |
+2011-04-21 Beth Dakin <bdakin@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=57898 |
+ REGRESSION (r82185): Scroll position not restored on navigation back to a page in |
+ the page cache |
+ -and corresponding- |
+ <rdar://problem/9226652> |
+ |
+ The original fix for this change broke scroll-position restore for pages that are |
+ *not* in the page cache. This patch fixes both cached and non-cached pages by |
+ using the cachedScrollPosition() only for cached pages. |
+ * loader/HistoryController.cpp: |
+ (WebCore::HistoryController::saveScrollPositionAndViewStateToItem): |
+ |
+2011-04-21 Luke Macpherson <macpherson@chromium.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Remove redundant calls to RenderStyle::setDisplay() |
+ https://bugs.webkit.org/show_bug.cgi?id=59060 |
+ |
+ No new tests / no new functionality added. |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ Remove redundant calls to RenderStyle::setDisplay() |
+ |
+2011-04-21 John Bauman <jbauman@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ WebGL context drawing buffer may not be cleared if FBO bound |
+ https://bugs.webkit.org/show_bug.cgi?id=58956 |
+ |
+ Clear the drawing buffer even if it's not currently bound, by |
+ switching between FBOs around the clear. Also, make sure that we |
+ handle depth and stencil masks properly. |
+ |
+ No new tests, because testing this under DRT doesn't work. |
+ |
+ * html/canvas/WebGLRenderingContext.cpp: |
+ (WebCore::WebGLRenderingContext::initializeNewContext): |
+ (WebCore::WebGLRenderingContext::clearIfComposited): |
+ (WebCore::WebGLRenderingContext::depthMask): |
+ * html/canvas/WebGLRenderingContext.h: |
+ * |
+2011-04-21 Chris Fleizach <cfleizach@apple.com> |
+ |
+ Reviewed by Beth Dakin. |
+ |
+ WebKit is erroneously modifying the DOM; adding 'aria-selected' to tables |
+ https://bugs.webkit.org/show_bug.cgi?id=59140 |
+ |
+ WebKit should not modify the DOM when the screen reader requests certain things be set. There is no |
+ specification that allows this behavior yet, but we are working on something for the future. |
+ |
+ * accessibility/AccessibilityRenderObject.cpp: |
+ (WebCore::AccessibilityRenderObject::setARIAGrabbed): |
+ (WebCore::AccessibilityRenderObject::setIsExpanded): |
+ (WebCore::AccessibilityRenderObject::setSelected): |
+ (WebCore::AccessibilityRenderObject::setValue): |
+ |
+2011-04-21 Chris Fleizach <cfleizach@apple.com> |
+ |
+ Reviewed by Beth Dakin. |
+ |
+ AX: order AccessibilityRole alphabetically |
+ https://bugs.webkit.org/show_bug.cgi?id=31524 |
+ |
+ * accessibility/AccessibilityObject.h: |
+ |
+2011-04-21 Jia Pu <jpu@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ [Mac] correctionPanelTimerFired() crashes due to rangeToBeReplaced being cleared. |
+ https://bugs.webkit.org/show_bug.cgi?id=59155 |
+ <rdar://problem/9261698> |
+ |
+ Make sure the range isn't cleared before calling windowRectForRange() on it. Also removed unused functions in Editor.h, |
+ and fixed a typo in function name. |
+ |
+ * editing/Editor.h: |
+ * editing/SpellingCorrectionController.cpp: |
+ (WebCore::SpellingCorrectionController::correctionPanelTimerFired): |
+ (WebCore::SpellingCorrectionController::respondToChangedSelection): |
+ * editing/SpellingCorrectionController.h: |
+ (WebCore::SpellingCorrectionController::shouldStartTimerFor): |
+ |
+2011-04-21 Maciej Stachowiak <mjs@apple.com> |
+ |
+ Reviewed by Adam Roben. |
+ |
+ Add a feature define to allow <details> and <summary> to be disabled |
+ https://bugs.webkit.org/show_bug.cgi?id=59118 |
+ <rdar://problem/9257045> |
+ |
+ No new tests; no behavior change. |
+ |
+ * Configurations/FeatureDefines.xcconfig: |
+ * DerivedSources.make: |
+ * GNUmakefile.am: |
+ * features.pri: |
+ * html/HTMLDetailsElement.cpp: |
+ * html/HTMLSummaryElement.cpp: |
+ * html/HTMLTagNames.in: |
+ * html/shadow/DetailsMarkerControl.cpp: |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::canMergeContiguousAnonymousBlocks): |
+ * rendering/RenderDetails.cpp: |
+ * rendering/RenderDetails.h: |
+ * rendering/RenderDetailsMarker.cpp: |
+ * rendering/RenderDetailsMarker.h: |
+ * rendering/RenderObject.h: |
+ * rendering/RenderSummary.cpp: |
+ * rendering/RenderSummary.h: |
+ * rendering/RenderTreeAsText.cpp: |
+ (WebCore::RenderTreeAsText::writeRenderObject): |
+ |
+2011-04-21 Dimitri Glazkov <dglazkov@chromium.org> |
+ |
+ Reviewed by Adele Peterson. |
+ |
+ REGRESSION (r83386): Typing in WSJ.com search field doesn't clear placeholder text |
+ https://bugs.webkit.org/show_bug.cgi?id=59053 |
+ |
+ Turns out, the relatedTarget of an event could be also an ancestor of an event, |
+ not just its parent. Duh. |
+ |
+ * dom/EventDispatcher.cpp: |
+ (WebCore::EventDispatcher::adjustToShadowBoundaries): Changed to use shadowHost |
+ explicitly. |
+ |
+2011-04-21 Michael Saboff <msaboff@apple.com> |
+ |
+ Reviewed by Maciej Stachowiak. |
+ |
+ Qualified names used for all TagName access, yet namespace usage is rare |
+ https://bugs.webkit.org/show_bug.cgi?id=58997 |
+ |
+ The methods getElementsByTagName and getElementsByTagNameNS where |
+ always creating and using QualifiedNames. QualifiedName::init |
+ was consistently in the top 3 routines when running the Dromaeo |
+ DOM-query benchmark. Split out the functionality so that |
+ getElementsByTagName uses just the local name, an implied "*" |
+ namespace and a separate TagNodeListCache keyed by an atomic name |
+ instead of a QualifiedName. Access to elements via |
+ getElementsByTagNameNS that have "*" namespace are forwarded to |
+ getElementsByTagName as well. This provides ~10% speed up in that |
+ Dromaeo test. |
+ |
+ No new tests added, existing tests have coverage. The changes are |
+ an optimization of existing functionality. |
+ |
+ * dom/Node.cpp: |
+ (WebCore::Node::removeCachedTagNodeList): |
+ (WebCore::Node::getElementsByTagName): |
+ (WebCore::Node::getElementsByTagNameNS): |
+ (WebCore::NodeListsNodeData::invalidateCaches): |
+ (WebCore::NodeListsNodeData::isEmpty): |
+ * dom/Node.h: |
+ * dom/NodeRareData.h: |
+ * dom/TagNodeList.cpp: |
+ (WebCore::TagNodeList::~TagNodeList): |
+ |
+2011-04-21 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Removed a use of markDOMObjectWrapper: more style- and node-related stuff |
+ https://bugs.webkit.org/show_bug.cgi?id=59127 |
+ |
+ * bindings/js/JSCanvasRenderingContextCustom.cpp: |
+ * bindings/js/JSCanvasRenderingContextCustom.cpp: |
+ (WebCore::JSCanvasRenderingContextOwner::isReachableFromOpaqueRoots): |
+ (WebCore::JSCanvasRenderingContextOwner::finalize): |
+ (WebCore::wrapperOwner): |
+ (WebCore::wrapperContext): Use opaque roots for marking canvas rendering |
+ contexts, so JSNode isn't responsible for it. |
+ |
+ * bindings/js/JSElementCustom.cpp: |
+ (WebCore::JSElement::markChildren): |
+ * bindings/js/JSHTMLCanvasElementCustom.cpp: |
+ * bindings/js/JSHTMLLinkElementCustom.cpp: |
+ * bindings/js/JSHTMLStyleElementCustom.cpp: Removed obsolete marking code. |
+ |
+ * bindings/js/JSNodeCustom.cpp: |
+ (WebCore::isObservable): |
+ (WebCore::isReachableFromDOM): |
+ (WebCore::JSNodeOwner::isReachableFromOpaqueRoots): Removed a bunch of |
+ CSS object model related special cases, since the CSS object model |
+ now tracks its own lifetime through the opaque roots system. Removed |
+ canvas special case, covered above. |
+ |
+ * bindings/js/JSProcessingInstructionCustom.cpp: |
+ * bindings/js/JSWebKitAnimationListCustom.cpp: |
+ * dom/ProcessingInstruction.idl: |
+ * html/HTMLCanvasElement.idl: |
+ * html/HTMLLinkElement.idl: |
+ * html/HTMLStyleElement.idl: |
+ * page/WebKitAnimationList.idl: Removed obsolete marking code. |
+ |
+2011-04-21 Vangelis Kokkevis <vangelis@chromium.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ [chromium] Only update the contents of layers that are going to |
+ be drawn. Remove individual texture unreserves that happen |
+ after the layer draw calls with a single unreserve for all the |
+ textures in the texture manager. |
+ https://bugs.webkit.org/show_bug.cgi?id=59129 |
+ |
+ Tests: No new functionality. Covered by existing layout tests. |
+ |
+ * platform/graphics/chromium/ContentLayerChromium.cpp: |
+ (WebCore::ContentLayerChromium::draw): |
+ * platform/graphics/chromium/ContentLayerChromium.h: |
+ * platform/graphics/chromium/ImageLayerChromium.cpp: |
+ (WebCore::ImageLayerChromium::paintContentsIfDirty): |
+ (WebCore::ImageLayerChromium::updateCompositorResources): |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::drawRootLayer): |
+ (WebCore::LayerRendererChromium::updateAndDrawLayers): |
+ (WebCore::LayerRendererChromium::updateLayers): |
+ (WebCore::LayerRendererChromium::paintLayerContents): |
+ * platform/graphics/chromium/LayerRendererChromium.h: |
+ * platform/graphics/chromium/LayerTexture.cpp: |
+ * platform/graphics/chromium/LayerTexture.h: |
+ * platform/graphics/chromium/LayerTilerChromium.cpp: |
+ * platform/graphics/chromium/LayerTilerChromium.h: |
+ * platform/graphics/chromium/RenderSurfaceChromium.cpp: |
+ (WebCore::RenderSurfaceChromium::drawSurface): |
+ * platform/graphics/chromium/TextureManager.cpp: |
+ (WebCore::TextureManager::unprotectAllTextures): |
+ * platform/graphics/chromium/TextureManager.h: |
+ * platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp: |
+ (WebCore::CCHeadsUpDisplay::draw): |
+ * platform/graphics/chromium/cc/CCLayerImpl.h: |
+ (WebCore::CCLayerImpl::owner): |
+ |
+2011-04-21 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Add Frame* to the argument lists of canCopyCut and canPaste |
+ https://bugs.webkit.org/show_bug.cgi?id=59153 |
+ |
+ canCopyCut and canPaste now takes Frame* for which supportedCopyCut and supportedPaste are called. |
+ This allows editor client to grant copy/cut and paste permissions per origin. |
+ |
+ * editing/EditorCommand.cpp: |
+ (WebCore::supportedCopyCut): |
+ (WebCore::supportedPaste): |
+ * loader/EmptyClients.h: |
+ (WebCore::EmptyEditorClient::canCopyCut): |
+ (WebCore::EmptyEditorClient::canPaste): |
+ * page/EditorClient.h: |
+ |
+2011-04-21 Oliver Hunt <oliver@apple.com> |
+ |
+ Whoops, change lost during merge |
+ |
+ * bindings/js/JSEventListener.h: |
+ |
+2011-04-21 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Fixed a failure seen in fast/dom/prototype-inheritance-2.html due to |
+ wrong global object use. |
+ |
+ * bindings/js/JSHTMLDocumentCustom.cpp: |
+ (WebCore::JSHTMLDocument::nameGetter): |
+ (WebCore::JSHTMLDocument::all): |
+ * bindings/js/JSHTMLFormElementCustom.cpp: |
+ (WebCore::JSHTMLFormElement::nameGetter): Use the right global object. |
+ |
+2011-04-21 Oliver Hunt <oliver@apple.com> |
+ |
+ Reviewed by Geoffrey Garen. |
+ |
+ Start moving to a general visitor pattern for GC traversal |
+ https://bugs.webkit.org/show_bug.cgi?id=59141 |
+ |
+ This is just a rename: |
+ markChildren -> visitChildren |
+ markAggregate -> visitAggregate |
+ markStack -> visitor |
+ MarkStack -> typedef'd to SlotVisitor |
+ |
+ * bindings/js/JSAttrCustom.cpp: |
+ (WebCore::JSAttr::visitChildren): |
+ * bindings/js/JSAudioConstructor.cpp: |
+ (WebCore::constructAudio): |
+ * bindings/js/JSCSSRuleCustom.cpp: |
+ (WebCore::JSCSSRule::visitChildren): |
+ * bindings/js/JSCSSStyleDeclarationCustom.cpp: |
+ (WebCore::JSCSSStyleDeclaration::visitChildren): |
+ * bindings/js/JSDOMBinding.cpp: |
+ (WebCore::visitActiveObjectsForContext): |
+ (WebCore::markDOMObjectWrapper): |
+ * bindings/js/JSDOMBinding.h: |
+ * bindings/js/JSDOMGlobalObject.cpp: |
+ (WebCore::JSDOMGlobalObject::visitChildren): |
+ * bindings/js/JSDOMGlobalObject.h: |
+ * bindings/js/JSDOMImplementationCustom.cpp: |
+ (WebCore::JSDOMImplementation::visitChildren): |
+ * bindings/js/JSDOMWindowCustom.cpp: |
+ (WebCore::JSDOMWindow::visitChildren): |
+ * bindings/js/JSDOMWindowShell.cpp: |
+ (WebCore::JSDOMWindowShell::visitChildren): |
+ * bindings/js/JSDOMWindowShell.h: |
+ * bindings/js/JSDocumentCustom.cpp: |
+ (WebCore::JSDocument::visitChildren): |
+ * bindings/js/JSElementCustom.cpp: |
+ (WebCore::JSElement::visitChildren): |
+ * bindings/js/JSEventListener.cpp: |
+ (WebCore::JSEventListener::markJSFunction): |
+ * bindings/js/JSEventListener.h: |
+ * bindings/js/JSHTMLCanvasElementCustom.cpp: |
+ (WebCore::JSHTMLCanvasElement::visitChildren): |
+ * bindings/js/JSHTMLLinkElementCustom.cpp: |
+ (WebCore::JSHTMLLinkElement::visitChildren): |
+ * bindings/js/JSHTMLStyleElementCustom.cpp: |
+ (WebCore::JSHTMLStyleElement::visitChildren): |
+ * bindings/js/JSImageConstructor.cpp: |
+ (WebCore::constructImage): |
+ * bindings/js/JSJavaScriptAudioNodeCustom.cpp: |
+ (WebCore::JSJavaScriptAudioNode::visitChildren): |
+ * bindings/js/JSMessageChannelCustom.cpp: |
+ (WebCore::JSMessageChannel::visitChildren): |
+ * bindings/js/JSMessagePortCustom.cpp: |
+ (WebCore::JSMessagePort::visitChildren): |
+ * bindings/js/JSNamedNodeMapCustom.cpp: |
+ (WebCore::JSNamedNodeMapOwner::isReachableFromOpaqueRoots): |
+ (WebCore::JSNamedNodeMap::visitChildren): |
+ * bindings/js/JSNavigatorCustom.cpp: |
+ (WebCore::JSNavigator::visitChildren): |
+ * bindings/js/JSNodeCustom.cpp: |
+ (WebCore::isObservable): |
+ (WebCore::isReachableFromDOM): |
+ (WebCore::JSNodeOwner::isReachableFromOpaqueRoots): |
+ (WebCore::JSNode::visitChildren): |
+ * bindings/js/JSNodeCustom.h: |
+ * bindings/js/JSNodeFilterCondition.cpp: |
+ (WebCore::JSNodeFilterCondition::WeakOwner::isReachableFromOpaqueRoots): |
+ * bindings/js/JSNodeFilterCondition.h: |
+ * bindings/js/JSNodeFilterCustom.cpp: |
+ (WebCore::JSNodeFilter::visitChildren): |
+ * bindings/js/JSNodeIteratorCustom.cpp: |
+ (WebCore::JSNodeIterator::visitChildren): |
+ * bindings/js/JSNodeListCustom.cpp: |
+ (WebCore::JSNodeListOwner::isReachableFromOpaqueRoots): |
+ * bindings/js/JSProcessingInstructionCustom.cpp: |
+ (WebCore::JSProcessingInstruction::visitChildren): |
+ * bindings/js/JSSVGElementInstanceCustom.cpp: |
+ (WebCore::JSSVGElementInstance::visitChildren): |
+ * bindings/js/JSSharedWorkerCustom.cpp: |
+ (WebCore::JSSharedWorker::visitChildren): |
+ * bindings/js/JSStyleSheetCustom.cpp: |
+ (WebCore::JSStyleSheet::visitChildren): |
+ * bindings/js/JSTreeWalkerCustom.cpp: |
+ (WebCore::JSTreeWalker::visitChildren): |
+ * bindings/js/JSWebGLRenderingContextCustom.cpp: |
+ (WebCore::JSWebGLRenderingContext::visitChildren): |
+ * bindings/js/JSWebKitAnimationListCustom.cpp: |
+ (WebCore::JSWebKitAnimationList::visitChildren): |
+ * bindings/js/JSWorkerContextCustom.cpp: |
+ (WebCore::JSWorkerContext::visitChildren): |
+ * bindings/js/JSXMLHttpRequestCustom.cpp: |
+ (WebCore::JSXMLHttpRequest::visitChildren): |
+ * bindings/js/JSXMLHttpRequestUploadCustom.cpp: |
+ (WebCore::JSXMLHttpRequestUpload::visitChildren): |
+ * bindings/scripts/CodeGeneratorJS.pm: |
+ * bridge/qt/qt_instance.cpp: |
+ (JSC::Bindings::QtRuntimeObject::visitChildren): |
+ (JSC::Bindings::QtInstance::markAggregate): |
+ * bridge/qt/qt_instance.h: |
+ * bridge/qt/qt_pixmapruntime.cpp: |
+ * bridge/qt/qt_runtime.cpp: |
+ (JSC::Bindings::QtRuntimeMetaMethod::visitChildren): |
+ * bridge/qt/qt_runtime.h: |
+ * dom/EventListener.h: |
+ (WebCore::EventListener::visitJSFunction): |
+ * dom/EventTarget.h: |
+ (WebCore::EventTarget::visitJSEventListeners): |
+ * dom/Node.h: |
+ * dom/NodeFilterCondition.h: |
+ (WebCore::NodeFilterCondition::visitAggregate): |
+ * page/DOMWindow.h: |
+ * workers/WorkerContext.h: |
+ |
+2011-04-21 Beth Dakin <bdakin@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=59048 |
+ Need to track whether overlay scrollbar is currently visible and in lower- |
+ righthand corner |
+ -and corresponding- |
+ <rdar://problem/9211232> |
+ |
+ Whenever we start a new ScrollbarPartAnimation, check if the scrollbar is vertical |
+ and showing (alpha of 1), and if so, call setVisibleScrollerThumbRect with the |
+ knob rect, otherwise with an empty rect. ScrollAnimatorMac has a local rect to |
+ track this value so that it can avoid sending messages across the ChromeClient to |
+ the UIProcess if the rect has not changed. |
+ * platform/mac/ScrollAnimatorMac.h: |
+ * platform/mac/ScrollAnimatorMac.mm: |
+ (-[ScrollbarPainterDelegate setUpAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]): |
+ (WebCore::ScrollAnimatorMac::ScrollAnimatorMac): |
+ (WebCore::ScrollAnimatorMac::setVisibleScrollerThumbRect): |
+ |
+ Plumbing for the chrome client. |
+ * page/ChromeClient.h: |
+ (WebCore::ChromeClient::notifyScrollerThumbIsVisibleInRect): |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::setVisibleScrollerThumbRect): |
+ * page/FrameView.h: |
+ * platform/ScrollableArea.h: |
+ (WebCore::ScrollableArea::setVisibleScrollerThumbRect): |
+ |
+ New WebCoreSystemInterface function to get the knob rect. |
+ * WebCore.exp.in: |
+ * platform/mac/WebCoreSystemInterface.h: |
+ * platform/mac/WebCoreSystemInterface.mm: |
+ |
2011-04-21 Nat Duca <nduca@chromium.org> |
Reviewed by James Robinson. |