Index: Source/WebCore/ChangeLog |
=================================================================== |
--- Source/WebCore/ChangeLog (revision 93254) |
+++ Source/WebCore/ChangeLog (working copy) |
@@ -1,5 +1,836 @@ |
+2011-08-05 James Robinson <jamesr@chromium.org> |
+ |
+ [chromium] Accelerated canvas breaks when moving canvases or resources between Pages |
+ https://bugs.webkit.org/show_bug.cgi?id=65402 |
+ |
+ Reviewed by Stephen White. |
+ |
+ Use one shared GraphicsContext3D for the whole process instead of one per Page as canvases can move between |
+ pages and directly draw into contexts in different pages. Also switches DrawingBufferChromium over to use a |
+ directly shared the color attachment instead of copying it to a separate texture and removes the now-unnecessary |
+ DrawingBuffer::didReset() call and WillPublishCallback mechanism. |
+ |
+ * page/Page.cpp: |
+ (WebCore::Page::sharedGraphicsContext3D): |
+ * page/Page.h: |
+ * platform/graphics/chromium/Canvas2DLayerChromium.cpp: |
+ (WebCore::Canvas2DLayerChromium::~Canvas2DLayerChromium): |
+ (WebCore::Canvas2DLayerChromium::updateCompositorResources): |
+ (WebCore::Canvas2DLayerChromium::textureId): |
+ (WebCore::Canvas2DLayerChromium::setDrawingBuffer): |
+ * platform/graphics/chromium/CanvasLayerChromium.cpp: |
+ (WebCore::CanvasLayerChromium::CanvasLayerChromium): |
+ * platform/graphics/chromium/CanvasLayerChromium.h: |
+ * platform/graphics/chromium/DrawingBufferChromium.cpp: |
+ (WebCore::DrawingBuffer::DrawingBuffer): |
+ (WebCore::DrawingBuffer::publishToPlatformLayer): |
+ * platform/graphics/chromium/Extensions3DChromium.h: |
+ * platform/graphics/chromium/WebGLLayerChromium.cpp: |
+ (WebCore::WebGLLayerChromium::WebGLLayerChromium): |
+ * platform/graphics/chromium/WebGLLayerChromium.h: |
+ |
+2011-08-05 Anders Carlsson <andersca@apple.com> |
+ |
+ Fix Chromium build. |
+ |
+ * WebCore.gypi: |
+ |
+2011-08-05 Anders Carlsson <andersca@apple.com> |
+ |
+ Remove PluginHalter |
+ https://bugs.webkit.org/show_bug.cgi?id=65729 |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Remove plug-in halter and associated classes. |
+ |
+ * CMakeLists.txt: |
+ * GNUmakefile.list.am: |
+ * WebCore.exp.in: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * loader/EmptyClients.h: |
+ * page/HaltablePlugin.h: Removed. |
+ * page/Page.cpp: |
+ (WebCore::Page::Page): |
+ * page/Page.h: |
+ * page/PluginHalter.cpp: Removed. |
+ * page/PluginHalter.h: Removed. |
+ * page/PluginHalterClient.h: Removed. |
+ * page/Settings.cpp: |
+ (WebCore::Settings::Settings): |
+ * page/Settings.h: |
+ * plugins/PluginView.cpp: |
+ (WebCore::PluginView::start): |
+ (WebCore::PluginView::stop): |
+ (WebCore::PluginView::PluginView): |
+ * plugins/PluginView.h: |
+ * plugins/PluginViewNone.cpp: |
+ * plugins/gtk/PluginViewGtk.cpp: |
+ * plugins/qt/PluginViewQt.cpp: |
+ * plugins/win/PluginViewWin.cpp: |
+ * rendering/RenderWidget.cpp: |
+ (WebCore::RenderWidget::paint): |
+ * rendering/RenderWidget.h: |
+ |
+2011-08-05 Jochen Eisinger <jochen@chromium.org> |
+ |
+ Introduce a new ResourceRequest::TargetType for XHRs |
+ https://bugs.webkit.org/show_bug.cgi?id=65544 |
+ |
+ Reviewed by Darin Fisher. |
+ |
+ * platform/network/chromium/ResourceRequest.h: |
+ * xml/XMLHttpRequest.cpp: |
+ (WebCore::XMLHttpRequest::createRequest): |
+ |
+2011-08-05 Yael Aharon <yael.aharon@nokia.com> |
+ |
+ dir=auto needs to work on value of input and textarea elements |
+ https://bugs.webkit.org/show_bug.cgi?id=65428 |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Changed the directionality algorithm to evaluate the value of input elements and textarea elements when |
+ they have the attribute dir-auto Set. |
+ Also skip these elements when evaluating the directionality of their ancestors. |
+ |
+ HTMLTextAreaElement::childrenChanged is not called when a user types into the textarea, so call |
+ calculateAndAdjustDirectionality() explicitly from HTMLTextAreaElement::subtreeHasChanged(). |
+ |
+ HTMLInputElement::childrenChanged is not called when a user types into the textarea, so call |
+ calculateAndAdjustDirectionality() explicitly from HTMLTextAreaElement::subtreeHasChanged(). |
+ |
+ Tests: fast/dom/HTMLElement/attr-dir-auto-changed-text-form-control.html |
+ fast/dom/HTMLElement/attr-dir-auto-text-form-control-child.html |
+ fast/dom/HTMLElement/attr-dir-auto-text-form-control.html |
+ |
+ * html/HTMLElement.cpp: |
+ (WebCore::HTMLElement::directionality): |
+ * html/HTMLElement.h: |
+ * html/HTMLTextAreaElement.cpp: |
+ (WebCore::HTMLTextAreaElement::subtreeHasChanged): |
+ |
+2011-08-05 Keishi Hattori <keishi@webkit.org> |
+ |
+ Implement <input type=color> UI behavior WebCore part |
+ https://bugs.webkit.org/show_bug.cgi?id=62619 |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ * WebCore.exp.in: Added __ZN7WebCore12ColorChooser7chooserEv and |
+ __ZNK7WebCore12ColorChooser13colorSelectedERKNS_5ColorE |
+ * WebCore.xcodeproj/project.pbxproj: Added ColorChooser.{h,cpp} |
+ * html/ColorInputType.cpp: |
+ (WebCore::ColorInputType::~ColorInputType): Added. Close color chooser. ex. when type attribute changes. |
+ (WebCore::ColorInputType::valueAsColor): Added. |
+ (WebCore::ColorInputType::setValueAsColor): Added. |
+ (WebCore::ColorInputType::valueChanged): Update selected color in color chooser too. |
+ (WebCore::ColorInputType::handleClickEvent): Opens color chooser. |
+ (WebCore::ColorInputType::handleDOMActivateEvent): Called when element.click(). Open color chooser only |
+ when it was initiated by a user interaction. |
+ (WebCore::ColorInputType::detach): Close color chooser. Called when input element or its ancestors have "display:none" |
+ or is removed from DOM. |
+ (WebCore::ColorInputType::colorSelected): Callback from color chooser. |
+ (WebCore::ColorInputType::closeColorChooserIfClientIsInDocument): Close color chooser if element is in |
+ document. Called when the page navigates away. |
+ (WebCore::ColorInputType::closeColorChooserIfCurrentClient): Close color chooser if this input type is the current client of ColorChooser. |
+ * html/ColorInputType.h: |
+ * html/FileInputType.cpp: |
+ * html/FileInputType.h: |
+ * html/HTMLInputElement.cpp: |
+ (WebCore::HTMLInputElement::detach): Calls InputType::detach |
+ * html/InputType.cpp: |
+ (WebCore::InputType::chrome): Added. Used in FileInputType and ColorInputType. |
+ (WebCore::InputType::detach): ColorInputType overrides this. |
+ * html/InputType.h: |
+ * loader/EmptyClients.h: |
+ (WebCore::EmptyChromeClient::openColorChooser): Added. |
+ (WebCore::EmptyChromeClient::closeColorChooser): Added. |
+ (WebCore::EmptyChromeClient::setSelectedColorInColorChooser): Added. |
+ * loader/FrameLoader.cpp: |
+ (WebCore::FrameLoader::transitionToCommitted): Close color chooser when |
+ navigating away from the page. |
+ * page/Chrome.cpp: |
+ (WebCore::Chrome::openColorChooser): Added. Opens the color chooser. |
+ (WebCore::Chrome::closeColorChooser): Added. Tries to close the color chooser. Might not close if the |
+ listener of the color chooser is another part of the browser or another render process. |
+ (WebCore::Chrome::setSelectedColorInColorChooser): Added. Sets the selected color in the color chooser. |
+ Again, might not be executed if the listener of the color chooser is another part of the browser or another render process. |
+ * page/Chrome.h: |
+ * page/ChromeClient.h: |
+ * platform/ColorChooser.cpp: Added. |
+ (WebCore::ColorChooserClient::~ColorChooserClient): Disconnects itself from the ColorChooser. |
+ (WebCore::ColorChooser::chooser): Get shared instance of ColorChooser. |
+ (WebCore::ColorChooser::chooser): Get shared instance of ColorChooser. |
+ (WebCore::ColorChooser::connectClient): Connects a ColorChooserClient that receives the colorSelected callbacks. |
+ (WebCore::ColorChooser::disconnectClient): Disconnects the connectClient. |
+ (WebCore::ColorChooser::closeColorChooserIfClientIsInDocument): Close the color chooser if the client is |
+ inside the document. |
+ (WebCore::ColorChooser::colorSelected): Called from color chooser listener. |
+ * platform/ColorChooser.h: Added. |
+ (WebCore::ColorChooser::client): Added. Returns the current connected client. |
+ (WebCore::ColorChooser::ColorChooser): Added. |
+ |
+2011-08-05 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Web Inspector: check that detaching frame has been attached before removing it from the console selector |
+ https://bugs.webkit.org/show_bug.cgi?id=65686 |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * inspector/front-end/JavaScriptContextManager.js: |
+ (WebInspector.JavaScriptContextManager.prototype._frameDetached): |
+ |
+2011-08-05 Keishi Hattori <keishi@webkit.org> |
+ |
+ Sort WebCore.xcodeproj |
+ Accomplished using sort-Xcode-project-file. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ |
+2011-08-05 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Web Inspector: constrain maximum depth for returnByValue objects |
+ https://bugs.webkit.org/show_bug.cgi?id=65761 |
+ |
+ Set maximum depth to 20 for objects returned by value as a result of evaluations. |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ * bindings/js/ScriptValue.cpp: |
+ (WebCore::jsToInspectorValue): |
+ (WebCore::ScriptValue::toInspectorValue): |
+ * bindings/v8/ScriptValue.cpp: |
+ (WebCore::v8ToInspectorValue): |
+ (WebCore::ScriptValue::toInspectorValue): |
+ * inspector/InjectedScript.cpp: |
+ (WebCore::InjectedScript::makeCall): |
+ * inspector/InspectorValues.h: |
+ |
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from appcache IDL files |
+ https://bugs.webkit.org/show_bug.cgi?id=65752 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * loader/appcache/DOMApplicationCache.idl: |
+ |
+2011-08-04 Hans Wennborg <hans@chromium.org> |
+ |
+ IndexedDB: Stop using free-lists for database/object store/index ids. |
+ https://bugs.webkit.org/show_bug.cgi?id=65678 |
+ |
+ Reviewed by Tony Chang. |
+ |
+ Don't use free-lists for database/object store/index ids, |
+ just assign increasing numbers. |
+ |
+ It turns out that deleting an object store and creating a new one with |
+ the same id would cause the delete markers from the old object store to |
+ slow down lookups into the new one. Therefore we should generate |
+ a new id every time. Running out of ids (64 bits for databases and |
+ object stores, 32 bits for indices) is not realistic. |
+ |
+ Also make functions that generate new ids report errors, and make the |
+ callers of those functions check the return values. |
+ |
+ We must still delete free-lists when deleting an object store, and we |
+ must keep the code for encoding/decoding/comparison of free-list keys |
+ since users might have them in their databases. |
+ |
+ This is just a performance optimization, so no new tests. |
+ |
+ * storage/IDBLevelDBBackingStore.cpp: |
+ (WebCore::getNewDatabaseId): |
+ (WebCore::IDBLevelDBBackingStore::setIDBDatabaseMetaData): |
+ (WebCore::getNewObjectStoreId): |
+ (WebCore::IDBLevelDBBackingStore::createObjectStore): |
+ (WebCore::IDBLevelDBBackingStore::deleteObjectStore): |
+ (WebCore::getNewIndexId): |
+ (WebCore::IDBLevelDBBackingStore::createIndex): |
+ (WebCore::IDBLevelDBBackingStore::deleteIndex): |
+ |
+2011-08-05 Roland Steiner <rolandsteiner@chromium.org> |
+ |
+ Unreviewed: change an instance of isImportRule() that was overlooked in commit 92448. |
+ |
+ No new tests. (No functional change) |
+ |
+ * xml/XSLImportRule.h: |
+ (WebCore::XSLImportRule::isImportRule): |
+ |
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from remaining SVG IDL files |
+ https://bugs.webkit.org/show_bug.cgi?id=65751 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * svg/SVGElementInstance.idl: |
+ |
+2011-08-05 Pavel Feldman <pfeldman@google.com> |
+ |
+ Web Inspector: do not delay scroll event handling - scroll is already async. |
+ https://bugs.webkit.org/show_bug.cgi?id=65693 |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ * inspector/front-end/TextViewer.js: |
+ (WebInspector.TextViewer.prototype._syncScroll): |
+ |
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from web audio API |
+ https://bugs.webkit.org/show_bug.cgi?id=65750 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * webaudio/AudioBufferSourceNode.idl: |
+ |
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from Web Workers |
+ https://bugs.webkit.org/show_bug.cgi?id=65746 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * workers/AbstractWorker.idl: |
+ * workers/DedicatedWorkerContext.idl: |
+ * workers/Worker.idl: |
+ * workers/WorkerContext.idl: |
+ |
+2011-08-05 Kenichi Ishibashi <bashi@chromium.org> |
+ |
+ Parsing CSS3 font-feature-settings property |
+ https://bugs.webkit.org/show_bug.cgi?id=63618 |
+ |
+ Introduces CSS3 font-feature-settings property as -webkit-font-feature-settings. This change only contains parsing part. Parsed information are stored in FontDescription class. |
+ |
+ Reviewed by Shinichiro Hamaji. |
+ |
+ Test: css3/font-feature-settings-parsing.html |
+ |
+ * CMakeLists.txt: Added FontFeatureValue.{h,cpp} and FontFeatureSettings.{h,cpp}. |
+ * GNUmakefile.list.am: Ditto. |
+ * WebCore.gypi: Ditto. |
+ * WebCore.pro: Ditto. |
+ * WebCore.vcproj/WebCore.vcproj: Ditto. |
+ * WebCore.xcodeproj/project.pbxproj: Ditto. |
+ * css/CSSComputedStyleDeclaration.cpp: |
+ (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Added CSSPropertyWebkitFontFeatureSettings. |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::parseValue): Added parsing code for -webkit-font-feature-settings. |
+ (WebCore::CSSParser::parseFontFeatureTag): Added. |
+ (WebCore::CSSParser::parseFontFeatureSettings): Added. |
+ * css/CSSParser.h: Added parseFontFeatureSettings(). |
+ * css/CSSPropertyNames.in: Added -webkit-font-feature-settings. |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyDeclarations): Modified a compile assert to follow adding -webkit-font-feature-settings. |
+ (WebCore::CSSStyleSelector::applyProperty): Added a case clause for CSSPropertyWebkitFontFeatureSettings. |
+ * css/CSSValue.h: |
+ (WebCore::CSSValue::isFontFeatureValue): Added. |
+ * css/CSSValueKeywords.in: Added 'on' and 'off' keywords. |
+ * css/FontFeatureValue.cpp: Added. |
+ (WebCore::FontFeatureValue::FontFeatureValue): |
+ (WebCore::FontFeatureValue::value): |
+ (WebCore::FontFeatureValue::cssText): |
+ * css/FontFeatureValue.h: Added. |
+ (WebCore::FontFeatureValue::create): |
+ (WebCore::FontFeatureValue::tag): |
+ (WebCore::FontFeatureValue::isFontFeatureValue): |
+ * platform/graphics/FontDescription.cpp: |
+ (WebCore::FontDescription::makeNormalFeatureSettings): Added. |
+ * platform/graphics/FontDescription.h: |
+ (WebCore::FontDescription::FontDescription): Added m_featureSettings member variable. |
+ (WebCore::FontDescription::featureSettings): Added. |
+ (WebCore::FontDescription::setFeatureSettings): Added. |
+ (WebCore::FontDescription::operator==): Modified to take into account m_featureSettings |
+ * platform/graphics/FontFeatureSettings.cpp: Added. |
+ (WebCore::FontFeature::FontFeature): |
+ (WebCore::FontFeature::operator=): |
+ (WebCore::FontFeature::operator==): |
+ (WebCore::FontFeatureSettings::FontFeatureSettings): |
+ * platform/graphics/FontFeatureSettings.h: Added. |
+ (WebCore::FontFeature::tag): |
+ (WebCore::FontFeature::value): |
+ (WebCore::FontFeatureSettings::create): |
+ (WebCore::FontFeatureSettings::append): |
+ (WebCore::FontFeatureSettings::size): |
+ (WebCore::FontFeatureSettings::operator[]): |
+ (WebCore::FontFeatureSettings::at): |
+ |
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from websockets |
+ https://bugs.webkit.org/show_bug.cgi?id=65749 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * websockets/WebSocket.idl: |
+ |
+2011-08-05 Adam Barth <abarth@webkit.org> |
+ |
+ Attempt to heal media/video-can-play-type.html and media/video-src-change.html. |
+ |
+ * html/HTMLMediaElement.idl: |
+ |
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from notifications IDL files |
+ https://bugs.webkit.org/show_bug.cgi?id=65747 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * notifications/Notification.idl: |
+ * notifications/NotificationCenter.idl: |
+ |
+2011-08-05 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r92439. |
+ http://trac.webkit.org/changeset/92439 |
+ https://bugs.webkit.org/show_bug.cgi?id=65753 |
+ |
+ Caused 9 tests to fail on Qt (Requested by abarth on #webkit). |
+ |
+ * dom/Document.cpp: |
+ (WebCore::Document::implicitOpen): |
+ * loader/FrameLoader.cpp: |
+ (WebCore::FrameLoader::transitionToCommitted): |
+ (WebCore::FrameLoader::detachChildren): |
+ |
+2011-08-04 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ Use RenderedPosition instead of getInlineBoxAndOffset in Editor and AccessibilityObject |
+ https://bugs.webkit.org/show_bug.cgi?id=65647 |
+ |
+ Reviewed by Hajime Morita. |
+ |
+ Encapsulated the use of getInlineBoxAndOffset in Editor.cpp and AccessibilityObject.cpp by RenderedPosition. |
+ |
+ Also added rendererFromPosition that returns the renderer of deprecatedNode but using a proper Position interface. |
+ |
+ * accessibility/AccessibilityObject.cpp: |
+ (WebCore::updateAXLineStartForVisiblePosition): |
+ * editing/Editor.cpp: |
+ (WebCore::Editor::firstRectForRange): |
+ * editing/RenderedPosition.cpp: |
+ (WebCore::rendererFromPosition): |
+ (WebCore::RenderedPosition::RenderedPosition): |
+ (WebCore::RenderedPosition::absoluteRect): |
+ * editing/RenderedPosition.h: |
+ (WebCore::RenderedPosition::absoluteRect): |
+ |
+2011-08-04 Hayato Ito <hayato@chromium.org> |
+ |
+ Make ScopedEventQueue enqueue an EventDispatchMediator, instead of an Event. |
+ https://bugs.webkit.org/show_bug.cgi?id=65613 |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ No changes to functionality so no new tests. |
+ |
+ * dom/Event.h: |
+ * dom/EventDispatcher.cpp: |
+ (WebCore::EventDispatcher::dispatchScopedEvent): |
+ * dom/EventDispatcher.h: |
+ * dom/Node.cpp: |
+ (WebCore::Node::dispatchScopedEvent): |
+ (WebCore::Node::dispatchScopedEventDispatchMediator): |
+ * dom/Node.h: |
+ * dom/ScopedEventQueue.cpp: |
+ (WebCore::ScopedEventQueue::~ScopedEventQueue): |
+ (WebCore::ScopedEventQueue::enqueueEventDispatchMediator): |
+ (WebCore::ScopedEventQueue::dispatchAllEvents): |
+ (WebCore::ScopedEventQueue::dispatchEvent): |
+ * dom/ScopedEventQueue.h: |
+ |
2011-08-04 James Robinson <jamesr@chromium.org> |
+ Unreviewed build fix. gcc 4.5 can't figure out that the 'data' variables are always initialized in these functions. |
+ |
+ * bindings/v8/custom/V8WebGLRenderingContextCustom.cpp: |
+ (WebCore::jsArrayToFloatArray): |
+ (WebCore::jsArrayToIntArray): |
+ |
+2011-08-05 Roland Steiner <rolandsteiner@chromium.org> |
+ |
+ Improve const-correctness in CSS code: type information methods |
+ https://bugs.webkit.org/show_bug.cgi?id=65506 |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ Make "is...()" functions (e.g., "isStyleRule()") const. |
+ Also, change CSSRule's type() function to return CSSRuleType. |
+ |
+ No new tests. (no change in semantics) |
+ |
+ * css/CSSCharsetRule.h: |
+ (WebCore::CSSCharsetRule::isCharsetRule): |
+ (WebCore::CSSCharsetRule::type): |
+ * css/CSSFontFaceRule.h: |
+ (WebCore::CSSFontFaceRule::isFontFaceRule): |
+ (WebCore::CSSFontFaceRule::type): |
+ * css/CSSImportRule.h: |
+ (WebCore::CSSImportRule::isImportRule): |
+ (WebCore::CSSImportRule::type): |
+ * css/CSSMediaRule.h: |
+ (WebCore::CSSMediaRule::isMediaRule): |
+ (WebCore::CSSMediaRule::type): |
+ * css/CSSPageRule.h: |
+ (WebCore::CSSPageRule::isPageRule): |
+ (WebCore::CSSPageRule::type): |
+ * css/CSSRule.h: |
+ (WebCore::CSSRule::isRule): |
+ * css/CSSStyleRule.h: |
+ (WebCore::CSSStyleRule::isStyleRule): |
+ (WebCore::CSSStyleRule::type): |
+ * css/CSSUnknownRule.h: |
+ (WebCore::CSSUnknownRule::type): |
+ * css/StyleBase.h: |
+ (WebCore::StyleBase::isCharsetRule): |
+ (WebCore::StyleBase::isFontFaceRule): |
+ (WebCore::StyleBase::isImportRule): |
+ (WebCore::StyleBase::isKeyframeRule): |
+ (WebCore::StyleBase::isKeyframesRule): |
+ (WebCore::StyleBase::isMediaRule): |
+ (WebCore::StyleBase::isPageRule): |
+ (WebCore::StyleBase::isRule): |
+ (WebCore::StyleBase::isStyleRule): |
+ * css/WebKitCSSKeyframeRule.h: |
+ (WebCore::WebKitCSSKeyframeRule::isKeyframeRule): |
+ (WebCore::WebKitCSSKeyframeRule::type): |
+ * css/WebKitCSSKeyframesRule.h: |
+ (WebCore::WebKitCSSKeyframesRule::isKeyframesRule): |
+ (WebCore::WebKitCSSKeyframesRule::type): |
+ |
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from plugin IDL files |
+ https://bugs.webkit.org/show_bug.cgi?id=65745 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * plugins/DOMMimeTypeArray.idl: |
+ * plugins/DOMPlugin.idl: |
+ * plugins/DOMPluginArray.idl: |
+ |
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from XML/XSL IDL files |
+ https://bugs.webkit.org/show_bug.cgi?id=65740 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * xml/DOMParser.idl: |
+ * xml/XMLSerializer.idl: |
+ * xml/XPathEvaluator.idl: |
+ * xml/XPathExpression.idl: |
+ * xml/XPathNSResolver.idl: |
+ * xml/XPathResult.idl: |
+ * xml/XSLTProcessor.idl: |
+ |
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from canvas IDL files |
+ https://bugs.webkit.org/show_bug.cgi?id=65737 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * html/canvas/CanvasGradient.idl: |
+ * html/canvas/Float32Array.idl: |
+ * html/canvas/Float64Array.idl: |
+ * html/canvas/Int16Array.idl: |
+ * html/canvas/Int32Array.idl: |
+ * html/canvas/Int8Array.idl: |
+ * html/canvas/OESVertexArrayObject.idl: |
+ * html/canvas/Uint16Array.idl: |
+ * html/canvas/Uint32Array.idl: |
+ * html/canvas/Uint8Array.idl: |
+ |
+2011-08-04 MORITA Hajime <morrita@google.com> |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ DocumentMarker: Type specific details should be separately held by other object. |
+ https://bugs.webkit.org/show_bug.cgi?id=59855 |
+ |
+ - Introduced DocumentMarkerDetails abstract class and two its subclasses |
+ DocumentMarkerDescription and DocumentMarkerActiveMatch, |
+ |
+ - Replacing DocumentMarker::m_description and |
+ DocumentMarker::m_activeMatch with these new details classes. |
+ |
+ No new tests. No behavior change. |
+ |
+ * CMakeLists.txt: |
+ * GNUmakefile.list.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * dom/DOMAllInOne.cpp: |
+ * dom/DocumentMarker.cpp: Added. |
+ (WebCore::emptyDescription): |
+ (WebCore::DocumentMarkerDetails::DocumentMarkerDetails): |
+ (WebCore::DocumentMarkerDetails::~DocumentMarkerDetails): |
+ (WebCore::DocumentMarkerDescription::description): |
+ (WebCore::DocumentMarkerDescription::isDescription): |
+ (WebCore::DocumentMarkerDescription::DocumentMarkerDescription): |
+ (WebCore::DocumentMarkerDescription::createUnlessEmpty): |
+ (WebCore::DocumentMarkerDescription::compatibleTypes): |
+ (WebCore::DocumentMarkerTextMatch::activeMatch): |
+ (WebCore::DocumentMarkerTextMatch::isTextMatch): |
+ (WebCore::DocumentMarkerTextMatch::DocumentMarkerTextMatch): |
+ (WebCore::DocumentMarkerTextMatch::instanceFor): |
+ (WebCore::DocumentMarkerTextMatch::compatibleTypes): |
+ (WebCore::DocumentMarker::DocumentMarker): |
+ (WebCore::DocumentMarker::shiftOffsets): |
+ (WebCore::DocumentMarker::setActiveMatch): |
+ (WebCore::DocumentMarker::description): |
+ (WebCore::DocumentMarker::activeMatch): |
+ * dom/DocumentMarker.h: |
+ (WebCore::DocumentMarker::details): |
+ (WebCore::DocumentMarker::clearDetails): |
+ (WebCore::DocumentMarkerDetails::isDescription): |
+ (WebCore::DocumentMarkerDetails::isTextMatch): |
+ (WebCore::DocumentMarkerDetails::isAllowedFor): |
+ * dom/DocumentMarkerController.cpp: |
+ (WebCore::DocumentMarkerController::clearDescriptionOnMarkersIntersectingRange): |
+ * editing/CompositeEditCommand.cpp: |
+ (WebCore::CompositeEditCommand::replaceTextInNodePreservingMarkers): |
+ |
+2011-08-04 Scott Graham <scottmg@chromium.org> |
+ |
+ Bad interaction between document destruction and unload events |
+ https://bugs.webkit.org/show_bug.cgi?id=64741 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Three different errors triggered by this test case. The case to |
+ consider is a subdocument with an onunload on an element, that |
+ destroys the parent document during the onunload. One fix was a |
+ lifetime issue fixed by a protecting RefPtr, and another was an |
+ additional cancel of event triggers. The main fix was that during the |
+ transition to commited state, the documentLoader is being replaced by |
+ the provisionalDocumentLoader. But, because during firing events in |
+ the subdocument the parent is destroyed, that subevent caused the |
+ provisionalDocumentLoader to be detached from its frame. By marking |
+ the page as being in committed state before the parent documentLoader |
+ is set, this is avoided. |
+ |
+ Test: loader/document-destruction-within-unload.html |
+ |
+ * dom/Document.cpp: |
+ (WebCore::Document::implicitOpen): |
+ * loader/FrameLoader.cpp: |
+ (WebCore::FrameLoader::transitionToCommitted): |
+ (WebCore::FrameLoader::detachChildren): |
+ |
+2011-08-04 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Add code to determine whether a Range in inside fixed position content |
+ https://bugs.webkit.org/show_bug.cgi?id=65720 |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Propagate the 'wasFixed' flag out from various methods that |
+ are used when computing the collection of rects or quads |
+ for a range. Use the flag in Range to computer whether |
+ the Range is entirely within, partially within, or outside |
+ of fixed-position content. |
+ |
+ * WebCore.exp.in: |
+ * accessibility/AccessibilityRenderObject.cpp: |
+ (WebCore::AccessibilityRenderObject::boundingBoxRect): |
+ * dom/Range.cpp: |
+ (WebCore::Range::textRects): |
+ (WebCore::Range::textQuads): |
+ * dom/Range.h: |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::absoluteQuads): |
+ * rendering/RenderBlock.h: |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::absoluteQuads): |
+ * rendering/RenderBox.h: |
+ * rendering/RenderInline.cpp: |
+ (WebCore::RenderInline::absoluteQuads): |
+ * rendering/RenderInline.h: |
+ * rendering/RenderObject.h: |
+ (WebCore::RenderObject::absoluteQuads): |
+ * rendering/RenderText.cpp: |
+ (WebCore::RenderText::absoluteRectsForRange): |
+ (WebCore::RenderText::absoluteQuads): |
+ (WebCore::RenderText::absoluteQuadsForRange): |
+ * rendering/RenderText.h: |
+ * rendering/RenderView.cpp: |
+ (WebCore::RenderView::absoluteQuads): |
+ * rendering/RenderView.h: |
+ * rendering/svg/RenderSVGHiddenContainer.cpp: |
+ (WebCore::RenderSVGHiddenContainer::absoluteQuads): |
+ * rendering/svg/RenderSVGHiddenContainer.h: |
+ * rendering/svg/RenderSVGInline.cpp: |
+ (WebCore::RenderSVGInline::absoluteQuads): |
+ * rendering/svg/RenderSVGInline.h: |
+ * rendering/svg/RenderSVGModelObject.cpp: |
+ (WebCore::RenderSVGModelObject::absoluteQuads): |
+ * rendering/svg/RenderSVGModelObject.h: |
+ * rendering/svg/RenderSVGText.cpp: |
+ (WebCore::RenderSVGText::absoluteQuads): |
+ * rendering/svg/RenderSVGText.h: |
+ |
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from Document.idl |
+ https://bugs.webkit.org/show_bug.cgi?id=65717 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * dom/Document.idl: |
+ |
+2011-08-04 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Fix release build failure with last commit. |
+ |
+ * rendering/RenderView.cpp: |
+ (WebCore::RenderView::mapLocalToContainer): |
+ |
+2011-08-04 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Add code to determine whether a Range in inside fixed position content |
+ https://bugs.webkit.org/show_bug.cgi?id=65720 |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ Add an out param for various coordinate mapping functions |
+ that tells us whether the point or rectangle being |
+ mapped is inside fixed-position content. |
+ |
+ This brings back some of the changes from r89754. |
+ |
+ No behavior changes, so no tests. |
+ |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::mapLocalToContainer): |
+ * rendering/RenderBox.h: |
+ * rendering/RenderInline.cpp: |
+ (WebCore::RenderInline::mapLocalToContainer): |
+ * rendering/RenderInline.h: |
+ * rendering/RenderObject.cpp: |
+ (WebCore::RenderObject::mapLocalToContainer): |
+ (WebCore::RenderObject::localToContainerQuad): |
+ * rendering/RenderObject.h: |
+ (WebCore::RenderObject::localToAbsoluteQuad): |
+ * rendering/RenderView.cpp: |
+ (WebCore::RenderView::mapLocalToContainer): |
+ * rendering/RenderView.h: |
+ * rendering/svg/RenderSVGForeignObject.cpp: |
+ (WebCore::RenderSVGForeignObject::mapLocalToContainer): |
+ * rendering/svg/RenderSVGForeignObject.h: |
+ * rendering/svg/RenderSVGInline.cpp: |
+ (WebCore::RenderSVGInline::mapLocalToContainer): |
+ * rendering/svg/RenderSVGInline.h: |
+ * rendering/svg/RenderSVGModelObject.cpp: |
+ (WebCore::RenderSVGModelObject::mapLocalToContainer): |
+ * rendering/svg/RenderSVGModelObject.h: |
+ * rendering/svg/RenderSVGRoot.cpp: |
+ (WebCore::RenderSVGRoot::mapLocalToContainer): |
+ * rendering/svg/RenderSVGRoot.h: |
+ * rendering/svg/RenderSVGText.cpp: |
+ (WebCore::RenderSVGText::mapLocalToContainer): |
+ * rendering/svg/RenderSVGText.h: |
+ * rendering/svg/SVGRenderSupport.cpp: |
+ (WebCore::SVGRenderSupport::mapLocalToContainer): |
+ * rendering/svg/SVGRenderSupport.h: |
+ |
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org> |
+ |
+ Remove LegacyDefaultOptionalArguments flag from DOM-related files except Document.idl |
+ https://bugs.webkit.org/show_bug.cgi?id=65715 |
+ |
+ Reviewed by Adam Barth. |
+ |
+ No new tests, all existing tests pass. |
+ |
+ * dom/CharacterData.idl: |
+ * dom/ClientRectList.idl: |
+ * dom/Clipboard.idl: |
+ * dom/DOMImplementation.idl: |
+ * dom/DOMStringList.idl: |
+ * dom/DataTransferItem.idl: |
+ * dom/DataTransferItems.idl: |
+ * dom/DocumentFragment.idl: |
+ * dom/Element.idl: |
+ * dom/EventListener.idl: |
+ * dom/EventTarget.idl: |
+ * dom/MediaStream.idl: |
+ * dom/MediaStreamList.idl: |
+ * dom/MediaStreamTrackList.idl: |
+ * dom/MessagePort.idl: |
+ * dom/NamedNodeMap.idl: |
+ * dom/Node.idl: |
+ * dom/NodeFilter.idl: |
+ * dom/NodeList.idl: |
+ * dom/Range.idl: |
+ * dom/RequestAnimationFrameCallback.idl: |
+ * dom/StringCallback.idl: |
+ * dom/Text.idl: |
+ * dom/TouchList.idl: |
+ |
+2011-08-04 Mark Rowe <mrowe@apple.com> |
+ |
+ Future-proof Xcode configuration settings. |
+ |
+ * Configurations/Base.xcconfig: |
+ * Configurations/DebugRelease.xcconfig: |
+ * Configurations/Version.xcconfig: |
+ * Configurations/WebCore.xcconfig: |
+ |
+2011-08-04 Van Lam <vanlam@google.com> |
+ |
+ A few purely stylistic modifications to visible_units.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=65723 |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ Renamed greatestValueUnder to greatestOffsetUnder, positionIsInsideBox |
+ to positionIsInBoxButNotOnBoundary (to avoid confusion with |
+ positionIsInBox, which is just a getInlineBoxAndOffset check). |
+ Removed use of invalidOffset as an error value in greatestOffsetUnder |
+ and smallestOffsetAbove since semantically it should only be used to |
+ check if it makes sense to compare offsets in a single box. |
+ |
+ * editing/visible_units.cpp: |
+ (WebCore::greatestOffsetUnder): |
+ (WebCore::smallestOffsetAbove): |
+ (WebCore::positionIsInBoxButNotOnBoundary): |
+ (WebCore::leftWordPositionAcrossBoundary): |
+ (WebCore::rightWordPositionAcrossBoundary): |
+ |
+2011-08-04 James Robinson <jamesr@chromium.org> |
+ |
[chromium] Implement a global resource limit for DrawingBuffer to limit the amount of GPU memory used by 2d canvas backing stores |
https://bugs.webkit.org/show_bug.cgi?id=65655 |