| 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
|
|
|
|
|