⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Aug 15, 2009:

11:49 PM Changeset in webkit [47335] by weinig@apple.com
  • 2 edits in trunk/LayoutTests

Make Mark Rowe a bit less grumpy.

  • http/tests/history/back-to-post.php:
11:43 PM Changeset in webkit [47334] by weinig@apple.com
  • 3 edits in trunk/LayoutTests

Make http/tests/history/back-to-post.php's output a bit more reliable.

  • http/tests/history/back-to-post-expected.txt:
  • http/tests/history/back-to-post.php:
9:40 PM Changeset in webkit [47333] by Simon Fraser
  • 2 edits in trunk/WebCore

2009-08-15 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

Optimize updateFillImages() to not add/remove clients every time
​https://bugs.webkit.org/show_bug.cgi?id=28334

Optimize the case where old and new just have a single layer pointing to the same (or no)
image.

  • rendering/RenderObject.cpp: (WebCore::RenderObject::updateFillImages):
6:32 PM Changeset in webkit [47332] by Simon Fraser
  • 4 edits
    2 adds in trunk

2009-08-15 Simon Fraser <Simon Fraser>

Reviewed by Anders Carlsson.

Implement animations of multiple text-shadows and -webkit-box-shadows.
​https://bugs.webkit.org/show_bug.cgi?id=28266

Test: transitions/multiple-shadow-transitions.html

  • page/animation/AnimationBase.cpp: (WebCore::PropertyWrapperShadow::equals): (WebCore::PropertyWrapperShadow::blend):
6:32 PM Changeset in webkit [47331] by Simon Fraser
  • 4 edits
    6 adds in trunk

2009-08-15 Simon Fraser <Simon Fraser>

Reviewed by Anders Carlsson.

Make transitions work for position and size of multiple backgrounds and masks.
​https://bugs.webkit.org/show_bug.cgi?id=28266

Make background-position, background-size, -webkit-mask-position and -webkit-mask-size
properties animatable with multiple backgrounds, rather than just animating the first one.

Tests: transitions/multiple-background-size-transitions.html

transitions/multiple-background-transitions.html
transitions/multiple-mask-transitions.html

  • page/animation/AnimationBase.cpp: (WebCore::FillLayerPropertyWrapperBase::FillLayerPropertyWrapperBase): (WebCore::FillLayerPropertyWrapperBase::~FillLayerPropertyWrapperBase): (WebCore::FillLayerPropertyWrapperGetter::FillLayerPropertyWrapperGetter): (WebCore::FillLayerPropertyWrapperGetter::equals): (WebCore::FillLayerPropertyWrapper::FillLayerPropertyWrapper): (WebCore::FillLayerPropertyWrapper::blend): (WebCore::FillLayersPropertyWrapper::FillLayersPropertyWrapper): (WebCore::FillLayersPropertyWrapper::equals): (WebCore::FillLayersPropertyWrapper::blend): (WebCore::ensurePropertyMap):
  • rendering/style/FillLayer.h: (WebCore::FillLayer::setXPosition): (WebCore::FillLayer::setYPosition): (WebCore::FillLayer::setSize):
5:23 PM Changeset in webkit [47330] by mrowe@apple.com
  • 3 edits in trunk/JavaScriptCore

Fix the build with JIT disabled.

  • runtime/Arguments.h: Only compile the jitCode method when the JIT is enabled.
  • runtime/Executable.h: Include PrototypeFunction.h so the compiler knows what

NativeFunctionWrapper is when the JIT is disabled.

5:16 PM Changeset in webkit [47329] by staikos@webkit.org
  • 3 edits
    1 add in trunk

2009-08-15 George Staikos <​george.staikos@torchmobile.com>

Reviewed by Jan Alonzo.

Don't crash in fieldset code when WML is enabled.
​https://bugs.webkit.org/show_bug.cgi?id=28012

Test: fast/wml/html-fieldset-crash.html

  • rendering/RenderFieldset.cpp: (WebCore::RenderFieldset::findLegend): ++brackets
4:01 PM Changeset in webkit [47328] by weinig@apple.com
  • 3 edits in trunk/WebCore

Try and enable EventSource on Windows.

  • WebCore.vcproj/WebCoreCommon.vsprops:
  • WebCore.vcproj/build-generated-files.sh:
3:54 PM Changeset in webkit [47327] by weinig@apple.com
  • 1 edit
    2 deletes in trunk/LayoutTests

Remove no longer correct and now failing test. document.body.onunload
has now been spec'ed in HTML5.

  • fast/events/onunload-body-property-expected.txt: Removed.
  • fast/events/onunload-body-property.html: Removed.
3:14 PM Changeset in webkit [47326] by weinig@apple.com
  • 4 edits
    6 adds in trunk

WebCore: Setting attribute event listeners to the body does not set them on the window
​https://bugs.webkit.org/show_bug.cgi?id=28343

Reviewed by Dan Bernstein.

Tests: fast/dom/Window/HTMLBodyElement-window-eventListener-attributes.html

fast/dom/Window/HTMLFrameSetElement-window-eventListener-attributes.html

  • html/HTMLBodyElement.cpp:

(WebCore::HTMLBodyElement::setOnblur): Use setWindowAttributeEventListener instead of setAttributeEventListener.
(WebCore::HTMLBodyElement::setOnerror): Ditto.
(WebCore::HTMLBodyElement::setOnfocus): Ditto.
(WebCore::HTMLBodyElement::setOnload): Ditto.
(WebCore::HTMLBodyElement::setOnbeforeunload): Ditto.
(WebCore::HTMLBodyElement::setOnmessage): Ditto.
(WebCore::HTMLBodyElement::setOnoffline): Ditto.
(WebCore::HTMLBodyElement::setOnonline): Ditto.
(WebCore::HTMLBodyElement::setOnresize): Ditto.
(WebCore::HTMLBodyElement::setOnstorage): Ditto.
(WebCore::HTMLBodyElement::setOnunload): Ditto.

  • html/HTMLFrameSetElement.cpp:

(WebCore::HTMLFrameSetElement::setOnblur): Ditto.
(WebCore::HTMLFrameSetElement::setOnerror): Ditto.
(WebCore::HTMLFrameSetElement::setOnfocus): Ditto.
(WebCore::HTMLFrameSetElement::setOnload): Ditto.
(WebCore::HTMLFrameSetElement::setOnbeforeunload): Ditto.
(WebCore::HTMLFrameSetElement::setOnmessage): Ditto.
(WebCore::HTMLFrameSetElement::setOnoffline): Ditto.
(WebCore::HTMLFrameSetElement::setOnonline): Ditto.
(WebCore::HTMLFrameSetElement::setOnresize): Ditto.
(WebCore::HTMLFrameSetElement::setOnstorage): Ditto.
(WebCore::HTMLFrameSetElement::setOnunload): Ditto.

LayoutTests: Setting attribute event listeners to the body does not set them on the window
​https://bugs.webkit.org/show_bug.cgi?id=28343

Reviewed by Dan Bernstein.

  • fast/dom/Window/HTMLBodyElement-window-eventListener-attributes-expected.txt: Added.
  • fast/dom/Window/HTMLBodyElement-window-eventListener-attributes.html: Added.
  • fast/dom/Window/HTMLFrameSetElement-window-eventListener-attributes-expected.txt: Added.
  • fast/dom/Window/HTMLFrameSetElement-window-eventListener-attributes.html: Added.
  • fast/dom/Window/resources/HTMLBodyElement-window-eventListener-attributes.js: Added.
  • fast/dom/Window/resources/HTMLFrameSetElement-window-eventListener-attributes.js: Added.
2:35 PM Changeset in webkit [47325] by weinig@apple.com
  • 3 edits in trunk/WebCore

Fix up some IDL comments.

Reviewed by Dan Bernstein.

  • html/HTMLBodyElement.idl:
  • html/HTMLFrameSetElement.idl:
2:25 PM Changeset in webkit [47324] by weinig@apple.com
  • 2 edits in trunk

[GTK] Added EventSource to the build (default on).
​https://bugs.webkit.org/show_bug.cgi?id=14997

Patch by Adam Bergkvist <​adam.bergkvist@ericsson.com> on 2009-08-14
Reviewed by Sam Weinig.

  • configure.ac:
2:04 PM Changeset in webkit [47323] by weinig@apple.com
  • 27 edits
    31 adds in trunk

JavaScriptCore: Added ENABLE_EVENTSOURCE flag.
​https://bugs.webkit.org/show_bug.cgi?id=14997

Patch by Adam Bergkvist <​adam.bergkvist@ericsson.com> on 2009-08-15
Reviewed by Sam Weinig.

  • Configurations/FeatureDefines.xcconfig:

WebCore: Added implementation of the EventSource object that enables
server-sent events from HTML5.
​http://dev.w3.org/html5/eventsource/
​https://bugs.webkit.org/show_bug.cgi?id=14997

Patch by Adam Bergkvist <​adam.bergkvist@ericsson.com> on 2009-08-15
Reviewed by Sam Weinig.

Tests: fast/eventsource/eventsource-attribute-listeners.html

fast/eventsource/eventsource-constructor.html
http/tests/eventsource/eventsource-bad-mime-type.html
http/tests/eventsource/eventsource-parse-event-stream.html
http/tests/eventsource/eventsource-reconnect.html
http/tests/eventsource/eventsource-status-code-states.html
http/tests/eventsource/workers/eventsource-simple.html

  • Configurations/FeatureDefines.xcconfig:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::JSDOMWindow::eventSource):

  • bindings/js/JSEventSourceConstructor.cpp: Added.

(WebCore::):
(WebCore::JSEventSourceConstructor::JSEventSourceConstructor):
(WebCore::constructEventSource):
(WebCore::JSEventSourceConstructor::getConstructData):

  • bindings/js/JSEventSourceConstructor.h: Added.

(WebCore::JSEventSourceConstructor::classInfo):

  • bindings/js/JSEventSourceCustom.cpp: Added.

(WebCore::JSEventSource::markChildren):
(WebCore::JSEventSource::addEventListener):
(WebCore::JSEventSource::removeEventListener):

  • bindings/js/JSEventTarget.cpp:

(WebCore::toJS):
(WebCore::toEventTarget):

  • bindings/js/JSWorkerContextCustom.cpp:

(WebCore::JSWorkerContext::eventSource):

  • dom/EventNames.h:
  • dom/EventTarget.cpp:

(WebCore::EventTarget::toEventSource):

  • dom/EventTarget.h:
  • page/DOMWindow.idl:
  • page/EventSource.cpp: Added.

(WebCore::EventSource::EventSource):
(WebCore::EventSource::~EventSource):
(WebCore::EventSource::connect):
(WebCore::EventSource::endRequest):
(WebCore::EventSource::scheduleReconnect):
(WebCore::EventSource::reconnectTimerFired):
(WebCore::EventSource::url):
(WebCore::EventSource::readyState):
(WebCore::EventSource::close):
(WebCore::EventSource::scriptExecutionContext):
(WebCore::EventSource::addEventListener):
(WebCore::EventSource::removeEventListener):
(WebCore::EventSource::dispatchEvent):
(WebCore::EventSource::didReceiveResponse):
(WebCore::EventSource::didReceiveData):
(WebCore::EventSource::didFinishLoading):
(WebCore::EventSource::didFail):
(WebCore::EventSource::didFailRedirectCheck):
(WebCore::EventSource::parseEventStream):
(WebCore::EventSource::parseEventStreamLine):
(WebCore::EventSource::dispatchGenericEvent):
(WebCore::EventSource::dispatchMessageEvent):
(WebCore::EventSource::stop):

  • page/EventSource.h: Added.

(WebCore::EventSource::create):
(WebCore::EventSource::):
(WebCore::EventSource::setOnopen):
(WebCore::EventSource::onopen):
(WebCore::EventSource::setOnmessage):
(WebCore::EventSource::onmessage):
(WebCore::EventSource::setOnerror):
(WebCore::EventSource::onerror):
(WebCore::EventSource::toEventSource):
(WebCore::EventSource::eventListeners):
(WebCore::EventSource::refEventTarget):
(WebCore::EventSource::derefEventTarget):

  • page/EventSource.idl: Added.
  • workers/WorkerContext.idl:

WebKit/mac: Added ENABLE_EVENTSOURCE flag.
​https://bugs.webkit.org/show_bug.cgi?id=14997

Patch by Adam Bergkvist <​adam.bergkvist@ericsson.com> on 2009-08-15
Reviewed by Sam Weinig.

  • Configurations/FeatureDefines.xcconfig:

WebKitTools: Added EventSource to the build script (default on).
​https://bugs.webkit.org/show_bug.cgi?id=14997

Patch by Adam Bergkvist <​adam.bergkvist@ericsson.com> on 2009-08-14
Reviewed by Sam Weinig.

  • Scripts/build-webkit:

LayoutTests: Tests for EventSource.
​https://bugs.webkit.org/show_bug.cgi?id=14997

Patch by Adam Bergkvist <​adam.bergkvist@ericsson.com> on 2009-08-14
Reviewed by Sam Weinig.

  • fast/eventsource: Added.
  • fast/eventsource/eventsource-attribute-listeners-expected.txt: Added.
  • fast/eventsource/eventsource-attribute-listeners.html: Added.
  • fast/eventsource/eventsource-constructor-expected.txt: Added.
  • fast/eventsource/eventsource-constructor.html: Added.
  • http/tests/eventsource: Added.
  • http/tests/eventsource/eventsource-bad-mime-type-expected.txt: Added.
  • http/tests/eventsource/eventsource-bad-mime-type.html: Added.
  • http/tests/eventsource/eventsource-parse-event-stream-expected.txt: Added.
  • http/tests/eventsource/eventsource-parse-event-stream.html: Added.
  • http/tests/eventsource/eventsource-reconnect-expected.txt: Added.
  • http/tests/eventsource/eventsource-reconnect.html: Added.
  • http/tests/eventsource/eventsource-status-code-states-expected.txt: Added.
  • http/tests/eventsource/eventsource-status-code-states.html: Added.
  • http/tests/eventsource/resources: Added.
  • http/tests/eventsource/resources/bad-mime-type.asis: Added.
  • http/tests/eventsource/resources/event-stream.php: Added.
  • http/tests/eventsource/resources/reconnect.php: Added.
  • http/tests/eventsource/resources/simple-event-stream.asis: Added.
  • http/tests/eventsource/resources/status-codes.php: Added.
  • http/tests/eventsource/workers: Added.
  • http/tests/eventsource/workers/eventsource-simple-expected.txt: Added.
  • http/tests/eventsource/workers/eventsource-simple.html: Added.
  • http/tests/eventsource/workers/resources: Added.
  • http/tests/eventsource/workers/resources/simple-worker.js: Added.

(es.onmessage):
(es.onerror):

1:53 PM Changeset in webkit [47322] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-08-15 Zan Dobersek <​zandobersek@gmail.com>

Reviewed by Xan Lopez.

Fix compiler warnings.

  • webkit/webkitwebview.cpp: (clipboard_contents_received): (webkit_web_view_drag_data_get):
11:30 AM Changeset in webkit [47321] by rniwa@webkit.org
  • 2 edits in trunk/WebKitTools

Unreviewed.

Patch by Ryosuke Niwa <​rniwa@webkit.org> on 2009-08-15
Add myself to list of committers.

  • Scripts/modules/committers.py:
4:25 AM Changeset in webkit [47320] by jhoneycutt@apple.com
  • 12 edits in trunk/WebKitTools

WebKitTools: Fix layout test failures after r47312.

Reviewed by Cameron Zwarich.

  • DumpRenderTree/AccessibilityController.h:

Replaced logFocusEvents() with setLogFocusEvents(), which takes a
boolean argument to turn logging of focus events on or off.
Added a function to reset the AccessibilityController to a consistent
state.

  • DumpRenderTree/AccessibilityController.cpp:

(logFocusEventsCallback):
Call setLogFocusEvents() to enable logging.
(AccessibilityController::resetToConsistentState):
Call setLogFocusEvents() to disable logging.

  • DumpRenderTree/gtk/AccessibilityControllerGtk.cpp:

(AccessibilityController::setLogFocusEvents):
Update stub.

  • DumpRenderTree/mac/AccessibilityControllerMac.mm:

(AccessibilityController::setLogFocusEvents):
Update stub.

  • DumpRenderTree/mac/DumpRenderTree.mm:

(resetWebViewToConsistentStateBeforeTesting):
Call the FrameLoadDelegate's resetToConsistentState method.

  • DumpRenderTree/mac/FrameLoadDelegate.h:

Declare the resetToConsistentState method.

  • DumpRenderTree/mac/FrameLoadDelegate.mm:

(-[FrameLoadDelegate resetToConsistentState]):
Call the AccessibilityController's resetToConsistentState() function.

  • DumpRenderTree/win/AccessibilityControllerWin.cpp:

(AccessibilityController::~AccessibilityController):
Turn off focus event logging when the controller is destroyed.
(AccessibilityController::setLogFocusEvents):
If the caller passes false, unhook the focus event, and clear
m_focusEventHook.

  • DumpRenderTree/win/DumpRenderTree.cpp:

(resetWebViewToConsistentStateBeforeTesting):
Call the FrameLoadDelegate's resetToConsistentState function.

  • DumpRenderTree/win/FrameLoadDelegate.h:

Declare the resetToConsistentState() function.

  • DumpRenderTree/win/FrameLoadDelegate.cpp:

(FrameLoadDelegate::resetToConsistentState):
Call the AccessibilityController's resetToConsistentState() function.

2:02 AM Changeset in webkit [47319] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2009-08-15 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Minor DOMAgent bugfixes.

​https://bugs.webkit.org/show_bug.cgi?id=28177

  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::nodeForId):
  • inspector/front-end/DOMAgent.js: (WebInspector.DOMAgent):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.this.treeOutline.focusedNodeChanged):
1:56 AM Changeset in webkit [47318] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2009-08-15 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Evaluating objects while on a breakpoint dumps string
representation instead of live object.

​https://bugs.webkit.org/show_bug.cgi?id=28280

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype.completions.this._evalInInspectedWindow): (WebInspector.ConsoleView.prototype.completions): (WebInspector.ConsoleView.prototype._reportCompletions):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.doEvalInCallFrame.delayedEvaluation): (WebInspector.ScriptsPanel.prototype.doEvalInCallFrame): (WebInspector.ScriptsPanel.prototype.variablesInSelectedCallFrame):
12:31 AM Changeset in webkit [47317] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/WebCore

2009-08-15 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

[Haiku] Adding two new files to WebCore/platform/haiku:
LocalizedStringsHaiku.cpp and LoggingHaiku.cpp
​https://bugs.webkit.org/show_bug.cgi?id=28129

  • platform/haiku/LocalizedStringsHaiku.cpp: Added. (WebCore::submitButtonDefaultLabel): (WebCore::inputElementAltText): (WebCore::resetButtonDefaultLabel): (WebCore::defaultLanguage): (WebCore::searchableIndexIntroduction): (WebCore::fileButtonChooseFileLabel): (WebCore::fileButtonNoFileSelectedLabel): (WebCore::contextMenuItemTagOpenLinkInNewWindow): (WebCore::contextMenuItemTagDownloadLinkToDisk): (WebCore::contextMenuItemTagCopyLinkToClipboard): (WebCore::contextMenuItemTagOpenImageInNewWindow): (WebCore::contextMenuItemTagDownloadImageToDisk): (WebCore::contextMenuItemTagCopyImageToClipboard): (WebCore::contextMenuItemTagOpenFrameInNewWindow): (WebCore::contextMenuItemTagCopy): (WebCore::contextMenuItemTagGoBack): (WebCore::contextMenuItemTagGoForward): (WebCore::contextMenuItemTagStop): (WebCore::contextMenuItemTagReload): (WebCore::contextMenuItemTagCut): (WebCore::contextMenuItemTagPaste): (WebCore::contextMenuItemTagNoGuessesFound): (WebCore::contextMenuItemTagIgnoreSpelling): (WebCore::contextMenuItemTagLearnSpelling): (WebCore::contextMenuItemTagSearchWeb): (WebCore::contextMenuItemTagLookUpInDictionary): (WebCore::contextMenuItemTagOpenLink): (WebCore::contextMenuItemTagIgnoreGrammar): (WebCore::contextMenuItemTagSpellingMenu): (WebCore::contextMenuItemTagShowSpellingPanel): (WebCore::contextMenuItemTagCheckSpelling): (WebCore::contextMenuItemTagCheckSpellingWhileTyping): (WebCore::contextMenuItemTagCheckGrammarWithSpelling): (WebCore::contextMenuItemTagFontMenu): (WebCore::contextMenuItemTagBold): (WebCore::contextMenuItemTagItalic): (WebCore::contextMenuItemTagUnderline): (WebCore::contextMenuItemTagOutline): (WebCore::contextMenuItemTagWritingDirectionMenu): (WebCore::contextMenuItemTagDefaultDirection): (WebCore::contextMenuItemTagLeftToRight): (WebCore::contextMenuItemTagRightToLeft): (WebCore::contextMenuItemTagInspectElement): (WebCore::searchMenuNoRecentSearchesText): (WebCore::searchMenuRecentSearchesText): (WebCore::searchMenuClearRecentSearchesText): (WebCore::unknownFileSizeText): (WebCore::AXWebAreaText): (WebCore::AXLinkText): (WebCore::AXListMarkerText): (WebCore::AXImageMapText): (WebCore::AXHeadingText): (WebCore::imageTitle): (WebCore::contextMenuItemTagTextDirectionMenu): (WebCore::AXButtonActionVerb): (WebCore::AXTextFieldActionVerb): (WebCore::AXRadioButtonActionVerb): (WebCore::AXCheckedCheckBoxActionVerb): (WebCore::AXUncheckedCheckBoxActionVerb): (WebCore::AXLinkActionVerb): (WebCore::AXDefinitionListTermText): (WebCore::AXDefinitionListDefinitionText):
  • platform/haiku/LoggingHaiku.cpp: Added. (WebCore::InitializeLoggingChannelsIfNecessary):
12:27 AM Changeset in webkit [47316] by Darin Adler
  • 2 edits in trunk/WebCore

Fix GTK build.

Patch by Darin Adler <Darin Adler> on 2009-08-15

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:

(fallbackObject): Pass 0 for document. Should be OK at least for now.

12:09 AM Changeset in webkit [47315] by Darin Adler
  • 2 edits in trunk/WebCore

Fix Qt build.

Patch by Darin Adler <Darin Adler> on 2009-08-15

  • dom/XMLTokenizerQt.cpp: Use create functions instead of calling new

directly to make CDATASection and Comment nodes.

Aug 14, 2009:

11:29 PM Changeset in webkit [47314] by jhoneycutt@apple.com
  • 2 edits in trunk/LayoutTests

Remove two tests from the skipped list that don't exist.

Rubber-stamped by Mark Rowe.

  • platform/win/Skipped:
11:25 PM Changeset in webkit [47313] by Darin Adler
  • 68 edits
    1 delete in trunk/WebCore

Make DOM classes start with a reference count of 1, like all other RefCounted
​https://bugs.webkit.org/show_bug.cgi?id=28068

Patch by Darin Adler <Darin Adler> on 2009-08-14
Reviewed by Sam Weinig.

First half, everything except for element classes.

  • GNUmakefile.am: Removed DocPtr.h.
  • WebCore.gypi: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • dom/Attr.cpp:

(WebCore::Attr::Attr): Added a call to createTextChild here so callers don't
have to call it explicitly.
(WebCore::Attr::create): Added.
(WebCore::Attr::setValue): Changed to take AtomicString.
(WebCore::Attr::cloneNode): Use create.

  • dom/Attr.h: Added a create function. Made the constructor private,

and a lot of other functions private as well.

  • dom/Attribute.cpp:

(WebCore::Attribute::createAttrIfNeeded): Use Attr::create.

  • dom/CDATASection.cpp:

(WebCore::CDATASection::create): Added.
(WebCore::CDATASection::cloneNode): Use create.
(WebCore::CDATASection::virtualCreate): Ditto.

  • dom/CDATASection.h: Added a create function. Made everything private.

Removed unneeded destructor declaration.

  • dom/CharacterData.cpp:

(WebCore::CharacterData::CharacterData): Replaced the multiple constructors
with a single one that takes ConstructionType.

  • dom/CharacterData.h: Made more functions be protected and private.

Made m_data be private.

  • dom/Comment.cpp:

(WebCore::Comment::Comment): Got rid of an extra constructor.
(WebCore::Comment::create): Added.
(WebCore::Comment::cloneNode): Call create.

  • dom/Comment.h: Added a create function. Made everything private.
  • dom/ContainerNode.cpp:

(WebCore::dispatchChildInsertionEvents): Use RefPtr instead of DocPtr.
(WebCore::dispatchChildRemovalEvents): Ditto.

  • dom/ContainerNode.h: Made the constructor protected and passed

ConstructionType instead of an isElement boolean.

  • dom/DocPtr.h: Removed.
  • dom/Document.cpp:

(WebCore::Document::Document): Simplified the code that sets m_document
since it's no longer a smart pointer.
(WebCore::Document::removedLastRef): Use explicit calls to selfOnlyRef
and selfOnlyDeref instead of a DocPtr in here.
(WebCore::Document::~Document): Simplified the code that sets m_document
since it's no longer a smart pointer.
(WebCore::Document::createDocumentFragment): Call create.
(WebCore::Document::createTextNode): Ditto.
(WebCore::Document::createComment): Ditto.
(WebCore::Document::createCDATASection): Ditto.
(WebCore::Document::createProcessingInstruction): Ditto.
(WebCore::Document::createEntityReference): Ditto.
(WebCore::Document::createEditingTextNode): Ditto.
(WebCore::Document::importNode): Call Attr::create.
(WebCore::Document::createAttributeNS): Ditto.

  • dom/Document.h: Call adoptRef. Made a lot of functions private and

protected and sorted them so public functions come first.

  • dom/DocumentFragment.cpp:

(WebCore::DocumentFragment::create): Added.
(WebCore::DocumentFragment::cloneNode): Call create.

  • dom/DocumentFragment.h: Added create.
  • dom/DocumentType.cpp:

(WebCore::DocumentType::DocumentType): Pass type to Node constructor.
(WebCore::DocumentType::cloneNode): Use create.

  • dom/DocumentType.h:

(WebCore::DocumentType::create): Call adoptRef.

  • dom/EditingText.cpp:

(WebCore::EditingText::create): Added.

  • dom/EditingText.h: Added a create function. Made everything private.
  • dom/Element.cpp:

(WebCore::Element::Element): Pass CreateElementZeroRefCount to preserve
the zero reference count behavior for classes derived from Element.

  • dom/EntityReference.cpp:

(WebCore::EntityReference::create): Added.
(WebCore::EntityReference::cloneNode): Call create.

  • dom/EntityReference.h: Added create. Made everything private.
  • dom/Node.cpp:

(WebCore::Node::initialRefCount): Added. Inline helper function for
the constructor.
(WebCore::Node::isContainer): Ditto.
(WebCore::Node::isElement): Ditto.
(WebCore::Node::isText): Ditto.
(WebCore::Node::Node): Changed to take a construction type argument.
Since m_document is now a normal pointer, added a call to selfOnlyRef.
(WebCore::Node::~Node): Ditto, but selfOnlyDeref.
(WebCore::Node::setDocument): Added selfOnlyRef/Deref calls.
(WebCore::Node::appendTextContent): Use the data function instead of
calling nodeValue functions, which do the same thing in a roundabout way.

  • dom/Node.h: Made the constructor protected and replaced the multiple

arguments iwth a single ConstructionType argument. Sorted the public
things first.

  • dom/Notation.h: Made most things private.
  • dom/Notation.cpp: Removed extra constructor.
  • dom/ProcessingInstruction.cpp:

(WebCore::ProcessingInstruction::create): Added.
(WebCore::ProcessingInstruction::cloneNode): Call create.

  • dom/ProcessingInstruction.h: Added create function. Made many other

members private.

  • dom/Range.cpp:

(WebCore::Range::processContents): Use DocumentFragment::create.

  • dom/Text.cpp:

(WebCore::Text::Text): Updated for base class change.
(WebCore::Text::create): Added.
(WebCore::Text::splitText): Changed to not require access to m_data.
(WebCore::Text::cloneNode): Call create.
(WebCore::Text::createRenderer): Call dataImpl.
(WebCore::Text::attach): Call data.
(WebCore::Text::recalcStyle): Call dataImpl.
(WebCore::Text::virtualCreate): Call create.
(WebCore::Text::createWithLengthLimit): Call create.
(WebCore::Text::formatForDebugger): Call data.

  • dom/Text.h: Added a create function. Made many other members private.

Renamed createNew to virtualCreate.

  • dom/XMLTokenizer.cpp:

(WebCore::XMLTokenizer::enterText): Call Text::create.

  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::XMLTokenizer::cdataBlock): Call CDATASection::create.
(WebCore::XMLTokenizer::comment): Call Comment::create.

  • editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::deleteInsignificantText):
Call data instead of string.

  • editing/CreateLinkCommand.cpp:

(WebCore::CreateLinkCommand::doApply): Call Text::create.

  • editing/EditorCommand.cpp:

(WebCore::executeInsertNode): Call DocumentFragment::create.

  • editing/SplitTextNodeCommand.cpp:

(WebCore::SplitTextNodeCommand::doApply): Call Text::create.

  • editing/markup.cpp:

(WebCore::appendStartMarkup): Call data instead of nodeValue.

  • html/HTMLDocument.h:

(WebCore::HTMLDocument::create): Call adoptRef.

  • html/HTMLElement.cpp:

(WebCore::HTMLElement::createContextualFragment): Call DocumentFragment::create.
(WebCore::replaceChildrenWithFragment): Call data instead of string.
(WebCore::replaceChildrenWithText): Call Text::create.
(WebCore::HTMLElement::setInnerText): Call DocumentFragment::create and
Text::create.
(WebCore::HTMLElement::setOuterText): Call Text::create.

  • html/HTMLKeygenElement.cpp:

(WebCore::HTMLKeygenElement::HTMLKeygenElement): Call Text::create.

  • html/HTMLOptionElement.cpp:

(WebCore::HTMLOptionElement::setText): Call Text::create.

  • html/HTMLParser.cpp:

(WebCore::HTMLParser::textCreateErrorCheck): Call Text::create.
(WebCore::HTMLParser::commentCreateErrorCheck): Call Comment::create.
(WebCore::HTMLParser::handleIsindex): Call Text::create.

  • html/HTMLViewSourceDocument.cpp:

(WebCore::HTMLViewSourceDocument::addText): Call Text::create.

  • html/HTMLViewSourceDocument.h:

(WebCore::HTMLViewSourceDocument::create): Call adoptRef.

  • loader/FTPDirectoryDocument.cpp:

(WebCore::FTPDirectoryTokenizer::appendEntry): Call Text::create.
(WebCore::FTPDirectoryTokenizer::createTDForFilename): Call Text::create.

  • loader/FTPDirectoryDocument.h:

(WebCore::FTPDirectoryDocument::create): Call adoptRef.

  • loader/ImageDocument.h:

(WebCore::ImageDocument::create): Ditto.

  • loader/MediaDocument.h:

(WebCore::MediaDocument::create): Ditto.

  • loader/PlaceholderDocument.h:

(WebCore::PlaceholderDocument::create): Ditto.

  • loader/PluginDocument.h:

(WebCore::PluginDocument::create): Ditto.

  • loader/TextDocument.h:

(WebCore::TextDocument::create): Ditto.

  • loader/loader.cpp:

(WebCore::Loader::Host::didFinishLoading): Use RefPtr instead of DocPtr.
(WebCore::Loader::Host::didFail): Ditto.

  • platform/TreeShared.h:

(WebCore::TreeShared::TreeShared): Added an initialRefCount argument,
defaulting to 1. Node still sometimes initializes it to 0 instead for now.

  • rendering/RenderText.cpp:

(WebCore::RenderText::originalText): Use dataImpl instead of string.

  • rendering/RenderTextFragment.cpp:

(WebCore::RenderTextFragment::originalText): Use dataImpl instead of string.
(WebCore::RenderTextFragment::previousCharacter): Ditto.

  • svg/SVGDocument.h:

(WebCore::SVGDocument::create): Use adoptRef.

  • svg/SVGElementInstance.cpp:

(WebCore::SVGElementInstance::SVGElementInstance): Updated to take a
PassRefPtr.

  • svg/SVGElementInstance.h: Made everything private.
  • wml/WMLDocument.h:

(WebCore::WMLDocument::create): Called adoptRef.

  • xml/XPathNamespace.cpp:

(WebCore::XPathNamespace::XPathNamespace): Take AtomicString arguments.

  • xml/XPathNamespace.h: Made everything private.
  • xml/XSLTProcessor.cpp:

(WebCore::createFragmentFromSource): Use DocumentFragment::create and
Text::create.

11:15 PM Changeset in webkit [47312] by jhoneycutt@apple.com
  • 8 edits
    2 adds in trunk

WebKitTools: Add a mechanism for logging MSAA focus events.

Part of <rdar://problem/6218721> No MSAA focus events fired for Webkit
nightly (20866)

​https://bugs.webkit.org/show_bug.cgi?id=20866

Reviewed by Oliver Hunt.

  • DumpRenderTree/AccessibilityController.cpp:

(logFocusEventsCallback):
Call the AccessibilityController's logFocusEvents() function.
(AccessibilityController::getJSClass):
Add a "logFocusEvents" function to the AccessibilityController's JS
class definition.

  • DumpRenderTree/AccessibilityController.h:

On Windows, include windows.h, and add a member variable to hold the
handle to the event hook for focus events. Add a declaration for a
function that enables logging of focus events.

  • DumpRenderTree/gtk/AccessibilityControllerGtk.cpp:

(AccessibilityController::logFocusEvents):
Stubbed.

  • DumpRenderTree/mac/AccessibilityControllerMac.mm:

(AccessibilityController::logFocusEvents):
Stubbed.

  • DumpRenderTree/win/AccessibilityControllerWin.cpp:

(AccessibilityController::AccessibilityController):
(AccessibilityController::~AccessibilityController):
If we hooked the focus event, unhook it.
(logFocusEventProc):
When we receive a focus event, get the accessible object for the event,
and log its name to stdout.
(AccessibilityController::logFocusEvents):
Setup the focus event hook to listen for events in the current process.

LayoutTests: Add a layout test for <rdar://problem/6218721> No MSAA focus events
fired for Webkit nightly (20866)

​https://bugs.webkit.org/show_bug.cgi?id=20866

Reviewed by Oliver Hunt.

  • platform/win/Skipped:

Remove the blanket skipping of accessibility/, and add the individual
tests.

  • platform/win/accessibility/focus-events-expected.txt: Added.
  • platform/win/accessibility/focus-events.html: Added.
11:14 PM Changeset in webkit [47311] by jhoneycutt@apple.com
  • 9 edits in trunk

WebCore:

2009-08-13 Jon Honeycutt <​jhoneycutt@apple.com>

Part of <rdar://problem/6218721> No MSAA focus events fired for Webkit
nightly (20866)

​https://bugs.webkit.org/show_bug.cgi?id=20866

Reviewed by Oliver Hunt.

  • accessibility/AXObjectCache.cpp: (WebCore::AXObjectCache::AXObjectCache): Changed to take a pointer to its owner document. This is used by AXObjectCache::handleFocusedUIElementChanged(). (WebCore::AXObjectCache::focusedUIElementForPage): Code moved from AccessibilityRenderObject::focusedUIElement(). Modified to be a static function and to take a pointer to a Page. (WebCore::AXObjectCache::platformGenerateAXID): Moved the code to generate the next AXID from getAXID() to here. Added a #if to make this non-WIN only, because Windows has its own implementation. (WebCore::AXObjectCache::getAXID): Ensure that we generate a positive AXID, ranging from 1 to LONG_MAX.
  • accessibility/AXObjectCache.h: Add a declaration for Document and Page. Removed the declaration of AccessibilityObject, because we include the header. Reordered the declaration of Node alphabetically. Moved the typedef for AXID to AccessibilityObject. Removed some trailing whitespace. Added a member variable to hold a pointer to the owner Document. (WebCore::AXObjectCache::AXObjectCache): Changed to take a pointer to its owner Document. (WebCore::AXObjectCache::focusedUIElementForPage): Added; code moved from AccessiblityRenderObject::focusedUIElement(). Returns the focused element with respect to accessibility. (WebCore::AXObjectCache::platformGenerateAXID): Declare a function to generate an AXID. (WebCore::AXObjectCache::objectFromAXID): Return the AccessibilityObject with the given AXID.
  • accessibility/AccessibilityObject.h: Moved the typedef for AXID from AXObjectCache to here. Made the m_id member use the typedef. (WebCore::AccessibilityObject::axObjectID): Changed the return type to use the typedef. (WebCore::AccessibilityObject::setAXObjectID): Changed the argument type to use the typedef.
  • accessibility/AccessibilityRenderObject.cpp: Removed some unneeded #includes. (WebCore::AccessibilityRenderObject::focusedUIElement): Moved the code to AXObjectCache::focusedUIElementForPage(), which we now call.
  • accessibility/win/AXObjectCacheWin.cpp: (WebCore::AXObjectCache::platformGenerateAXID): Ensure that we generate an AXID that is in the range 1 to LONG_MAX. (WebCore::AXObjectCache::handleFocusedUIElementChanged): If the Document has no Page, return. If the Page has not focused element (respecting accessibility), return. Assert that the accessibility of the focused element is not ignored, and that the object's AXID will be negative and fit into a LONG when negated. Broadcast a focus event for the object.
  • dom/Document.cpp: (WebCore::Document::axObjectCache): Pass this when creating the AXObjectCache. (WebCore::Document::setFocusedNode): Call AXObjectCache::handleFocusedUIElementChanged() on Windows.

WebKit/win:

2009-08-13 Jon Honeycutt <​jhoneycutt@apple.com>

Part of <rdar://problem/6218721> No MSAA focus events fired for Webkit
nightly (20866)

​https://bugs.webkit.org/show_bug.cgi?id=20866

Reviewed by Oliver Hunt.

  • AccessibleBase.cpp: (AccessibleBase::getAccessibilityObjectForChild): If the child ID is negative, negate it and treat it as an AXID.
11:08 PM Changeset in webkit [47310] by jmalonzo@webkit.org
  • 7 edits
    1 add in trunk

2009-08-14 Jiahua Huang <​jhuangjiahua@gmail.com>

Reviewed by Jan Alonzo.

[gtk] Pasteboard/GtkClipboard can't handle the "text/html" target.
​https://bugs.webkit.org/show_bug.cgi?id=27028

Remove the improper set of enum WebKitWebViewTargetInfo
by grabbing the target infos via m_helper.

  • manual-tests/gtk/copy-htmltext.html: Added.
  • platform/Pasteboard.h:
  • platform/gtk/PasteboardGtk.cpp: (WebCore::clipboard_get_contents_cb):
  • platform/gtk/PasteboardHelper.h:

Add a function in WebKit::PasteboardHelperGtk that will return the
value of WEBKIT_WEB_VIEW_TARGET_INFO_HTML,
and WebCore::clipboard_get_contents_cb will use that.

  • WebCoreSupport/PasteboardHelperGtk.cpp: (WebKit::PasteboardHelperGtk::getWebViewTargetInfoHtml):
  • WebCoreSupport/PasteboardHelperGtk.h:
10:19 PM Changeset in webkit [47309] by jmalonzo@webkit.org
  • 2 edits in trunk

2009-08-14 Jan Michael Alonzo <​jmalonzo@webkit.org>

Rubber-stamped by Gustavo Noronha.

Initialize CFLAGS and CXXFLAGS before the main body of the
WEBKIT_INIT macro so C files don't get built with "-g -O2".

  • autotools/webkit.m4:
7:53 PM Changeset in webkit [47308] by dimich@chromium.org
  • 3 edits in trunk/LayoutTests

Not reviewed, adding new test to Skipped on gtk and Qt.
DRT on these ports does not yet support layoutTestController.overridePreference.

  • platform/gtk/Skipped: skip LayoutTests/fast/harness/use-page-cache.html
  • platform/qt/Skipped: ditto.
6:36 PM Changeset in webkit [47307] by barraclough@apple.com
  • 3 edits in trunk/JavaScriptCore
  • parser/Parser.h:

(JSC::EvalExecutable::parse):
(JSC::ProgramExecutable::parse):

  • runtime/Executable.h:

Reviewed by NOBODY (build fix).

6:20 PM Changeset in webkit [47306] by dimich@chromium.org
  • 1 edit
    5 adds in trunk/LayoutTests

​https://bugs.webkit.org/show_bug.cgi?id=28324
First ever test using page cache!
The test verifies that override setting that enables page cache works correctly.
It navigates forward and then back and checks that onload is not fired again
but the timer continues firing.

Reviewed by Eric Seidel.

  • fast/harness/resources/cached-page-1.html: Added.
  • fast/harness/resources/cached-page-2.html: Added.
  • fast/harness/use-page-cache-expected.txt: Added.
  • fast/harness/use-page-cache.html: Added.
6:14 PM Changeset in webkit [47305] by Simon Fraser
  • 4 edits in trunk/WebCore

2009-08-14 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt.

Fix an assertion when bringing up the Inspector.
​https://bugs.webkit.org/show_bug.cgi?id=28318

The fix for bug 28295 assumed that image clients count the number of times
addClient/removeClient is called. That was true for CachedResource clients,
but not StyleGeneratedImage clients, which pass the call onto CSSImageGeneratorValue.

Fix by making CSSImageGeneratorValue count the number of times a client is
added/removed.

  • css/CSSCanvasValue.cpp: (WebCore::CSSCanvasValue::canvasChanged): (WebCore::CSSCanvasValue::canvasResized):
  • css/CSSImageGeneratorValue.cpp: (WebCore::CSSImageGeneratorValue::addClient): (WebCore::CSSImageGeneratorValue::removeClient): (WebCore::CSSImageGeneratorValue::getImage):
  • css/CSSImageGeneratorValue.h:
6:14 PM Changeset in webkit [47304] by barraclough@apple.com
  • 19 edits
    1 add in trunk/JavaScriptCore

Remove AST nodes from use within the Runtime (outside of parsing), stage 1
​https://bugs.webkit.org/show_bug.cgi?id=28330

Reviewed by Oliver Hunt.

Remove the EvalNode and ProgramNode from use in the runtime. They still exist
after this patch, but are hidden behind EvalExecutable and FunctionExecutable,
and are also still reachable behind CodeBlock::m_ownerNode.

The next step will be to beat back FunctionBodyNode in the same fashion.
Then remove the usage via CodeBlock, then only construct these nodes only on
demand during bytecode generation.

(JSC::GlobalCodeBlock::GlobalCodeBlock):
(JSC::GlobalCodeBlock::~GlobalCodeBlock):
(JSC::ProgramCodeBlock::ProgramCodeBlock):
(JSC::EvalCodeBlock::EvalCodeBlock):
(JSC::FunctionCodeBlock::FunctionCodeBlock):
(JSC::NativeCodeBlock::NativeCodeBlock):

  • bytecode/EvalCodeCache.h:

(JSC::EvalCodeCache::get):

  • debugger/Debugger.cpp:

(JSC::evaluateInGlobalCallFrame):

  • debugger/DebuggerCallFrame.cpp:

(JSC::DebuggerCallFrame::evaluate):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::callEval):
(JSC::Interpreter::execute):

  • interpreter/Interpreter.h:
  • parser/Nodes.cpp:

(JSC::FunctionBodyNode::createNativeThunk):
(JSC::FunctionBodyNode::generateBytecode):
(JSC::FunctionBodyNode::bytecodeForExceptionInfoReparse):

  • parser/Parser.h:

(JSC::Parser::parse):
(JSC::Parser::reparse):
(JSC::Parser::parseFunctionFromGlobalCode):
(JSC::::parse):

  • runtime/Completion.cpp:

(JSC::checkSyntax):
(JSC::evaluate):

  • runtime/Error.cpp:

(JSC::throwError):

  • runtime/Error.h:
  • runtime/Executable.h: Added.

(JSC::TemplateExecutable::TemplateExecutable):
(JSC::TemplateExecutable::markAggregate):
(JSC::TemplateExecutable::sourceURL):
(JSC::TemplateExecutable::lineNo):
(JSC::TemplateExecutable::bytecode):
(JSC::TemplateExecutable::jitCode):
(JSC::EvalExecutable::EvalExecutable):
(JSC::ProgramExecutable::ProgramExecutable):

  • runtime/FunctionConstructor.cpp:

(JSC::constructFunction):

  • runtime/FunctionConstructor.h:
  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::numericCompareFunction):

  • runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::~JSGlobalObject):
(JSC::JSGlobalObject::markChildren):

  • runtime/JSGlobalObject.h:

(JSC::JSGlobalObject::codeBlocks):

  • runtime/JSGlobalObjectFunctions.cpp:

(JSC::globalFuncEval):

4:25 PM Changeset in webkit [47303] by mrowe@apple.com
  • 2 edits in trunk/WebKit/mac

Build fix.

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::enumerate): Add the necessary .get() calls.

4:21 PM Changeset in webkit [47302] by beidson@apple.com
  • 4 edits in trunk

WebKit/mac: <rdar://problem/7091546> - Assertion failure in plugins/return-error-from-new-stream-doesnt-invoke-destroy-stream.html
on 64-bit SnowLeopard

Reviewed by Anders Carlsson.

  • Plugins/Hosted/HostedNetscapePluginStream.mm:

(WebKit::HostedNetscapePluginStream::cancelLoad): Since this might be called while the FrameLoader is switching DocumentLoaders

during the brief moment where there is no activeDocumentLoader(), accept and handle a validly null DocumentLoader.

LayoutTests: <rdar://problem/7091546> - Re-enable 3 plugins tests that now won't crash

Reviewed by Anders Carlsson.

  • platform/mac-snowleopard/Skipped:
4:18 PM Changeset in webkit [47301] by mrowe@apple.com
  • 2 edits in trunk/WebKit/mac

Fix leaks of NSNumber and NSMutableArray objects seen during layout tests.

Reviewed by Brady Eidson.

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::enumerate): Don't leak an NSMutableArray that we allocate.

4:18 PM Changeset in webkit [47300] by mrowe@apple.com
  • 2 edits in trunk/WebKit/mac

Fix leaks of HostedNetscapePluginStream and NetscapePlugInStreamLoader objects seen during layout tests.

Reviewed by Anders Carlsson.

  • Plugins/Hosted/HostedNetscapePluginStream.mm:

(WebKit::HostedNetscapePluginStream::didFail): Disconnect the stream from the plug-in instance proxy
when the load fails.

3:06 PM Changeset in webkit [47299] by bfulgham@webkit.org
  • 2 edits in trunk/WebCore

Correct libcurl crash when downloading local files.
​https://bugs.webkit.org/show_bug.cgi?id=28312

Reviewed by Kevin Ollivier.

No new tests. Test is covered by fast/encoding/char-encoding-mac.html.

  • platform/network/curl/ResourceHandleManager.cpp:

(WebCore::handleLocalReceiveResponse): New static function that

performs the local setting of valid URL and firing didReceiveResponse.
This code was moved from writeCallback.

(WebCore::writeCallback): Move local file handling to static function

(above) and use the function instead.

(WebCore::readCallback): Add check

for responseFired, and handle as a local file if it was not.

2:54 PM Changeset in webkit [47298] by beidson@apple.com
  • 2 edits in trunk/LayoutTests

Adding two more to the skipped list.
Radars 7144571 and 7144583 track exploring them.

  • platform/mac-snowleopard/Skipped:
1:54 PM Changeset in webkit [47297] by beidson@apple.com
  • 2 edits in trunk/LayoutTests

Disable media/controls-drag-timebar.html on SnowLeopard.
<rdar://problem/7144385> tracks exploring the failure.

  • platform/mac-snowleopard/Skipped:
1:52 PM Changeset in webkit [47296] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-08-14 Maxime Simon <Maxime Simon>

Reviewed by Oliver Hunt.

[Haiku] Fix RenderThemeHaiku build.
​https://bugs.webkit.org/show_bug.cgi?id=28307

  • platform/haiku/RenderThemeHaiku.cpp: Included RenderThemeHaiku.h instead of RenderTheme.h. (WebCore::RenderThemeHaiku::paintCheckbox): Removed the wrong 'virtual' before the function. (WebCore::RenderThemeHaiku::paintRadio): Removed the wrong 'virtual' before the function.
  • platform/haiku/RenderThemeHaiku.h: Corrected wrong function definitions.
1:35 PM Changeset in webkit [47295] by eric@webkit.org
  • 2 edits
    1 add in trunk/WebCore

2009-08-14 Maxime Simon <Maxime Simon>

Reviewed by Oliver Hunt.

[Haiku] Adding SharedBufferHaiku to WebCore.
​https://bugs.webkit.org/show_bug.cgi?id=28309

  • platform/haiku/SharedBufferHaiku.cpp: Added. (WebCore::SharedBuffer::createWithContentsOfFile):
  • platform/haiku/TemporaryLinkStubs.cpp: Removed unnecessary headers and functions. (WebCore::signedPublicKeyAndChallengeString): (WebCore::getSupportedKeySizes):
1:29 PM Changeset in webkit [47294] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-08-14 Eric Seidel <​eric@webkit.org>

No review. Fix 5-space indent to be 4-spaces.

  • Scripts/bugzilla-tool:
1:20 PM Changeset in webkit [47293] by Nate Chapin
  • 2 edits in trunk/WebCore

2009-08-14 Nate Chapin <Nate Chapin>

Reviewed by Dimitri Glazkov.

Ensure we're getting the right v8 context in the V8 DOMWindowEvent custom getter.

​https://bugs.webkit.org/show_bug.cgi?id=28311

This should fix Chromium failing LayoutTests/fast/events/set-event-in-another-frame.html.

  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::ACCESSOR_GETTER):
1:18 PM Changeset in webkit [47292] by oliver@apple.com
  • 10 edits in trunk

[ES5] Arguments object should inherit from Array
​https://bugs.webkit.org/show_bug.cgi?id=28298

Reviewed by Gavin Barraclough

Make the Arguments object conform to the behaviour specified in ES5.
The simple portion of this is to make Arguments use Array.prototype
as its prototype rather than Object.prototype.

The spec then requires us to set instance.constructor to the pristine
Object constructor, and instance.toString and instance.toLocaleString
to the pristine versions from Object.prototype. To do this we now
make the ObjectPrototype constructor return its toString and
toLocaleString functions (similar to the call and apply functions
from FunctionPrototype).

Oddly enough this reports itself as a slight win, but given the code
isn't hit in the tests that claim to have improved I put this down to
code motion.

12:25 PM Changeset in webkit [47291] by eric@webkit.org
  • 11 edits in trunk/WebCore

2009-08-14 Aaron Boodman <​aa@chromium.org>

Reviewed by Alexey Proskuryakov.

BUG 28134: Move the remaining parts of Access Control from XMLHttpRequest to ThreadableDocumentLoader.
​https://bugs.webkit.org/show_bug.cgi?id=28134

No new tests added since Access Control was already well tested and this is a pure refactor.

  • loader/DocumentThreadableLoader.cpp: Move a lot of the access control code from XHR in, preserving its basic strategy. Also, modify the synchronous path to not be a special case, but reuse more of the async path.

(WebCore::DocumentThreadableLoader::loadResourceSynchronously): Go through the async path and pass additional flags.
(WebCore::DocumentThreadableLoader::create): Group enum params into an options struct.
(WebCore::DocumentThreadableLoader::DocumentThreadableLoader): Ditto.
(WebCore::DocumentThreadableLoader::makeSimpleCrossOriginAccessRequest): Brought mostly from XHR.
(WebCore::DocumentThreadableLoader::makeCrossOriginAccessRequestWithPreflight): Ditto.
(WebCore::DocumentThreadableLoader::willSendRequest): Handle preflight case.
(WebCore::DocumentThreadableLoader::didReceiveResponse): Ditto.
(WebCore::DocumentThreadableLoader::didFinishLoading): Ditto.
(WebCore::DocumentThreadableLoader::getShouldUseCredentialStorage): Ditto.
(WebCore::DocumentThreadableLoader::preflightSuccess): Preflight handling.
(WebCore::DocumentThreadableLoader::preflightFailure): Ditto.
(WebCore::DocumentThreadableLoader::loadRequest): Common request function that handles async/sync.

  • loader/DocumentThreadableLoader.h: Group enum params into an options struct.
  • loader/ThreadableLoader.cpp: Ditto. (WebCore::ThreadableLoader::create): Ditto. (WebCore::ThreadableLoader::loadResourceSynchronously): Ditto.
  • loader/ThreadableLoader.h: Ditto. (WebCore::ThreadableLoaderOptions::ThreadableLoaderOptions): Ditto.
  • loader/WorkerThreadableLoader.cpp: Ditto. (WebCore::WorkerThreadableLoader::WorkerThreadableLoader):Ditto. (WebCore::WorkerThreadableLoader::loadResourceSynchronously): Ditto. (WebCore::WorkerThreadableLoader::MainThreadBridge::MainThreadBridge): Ditto. (WebCore::WorkerThreadableLoader::MainThreadBridge::mainThreadCreateLoader): Ditto.
  • loader/WorkerThreadableLoader.h: Ditto. (WebCore::WorkerThreadableLoader::create): Ditto.
  • platform/CrossThreadCopier.h: Allow ThreadableLoaderOptions to be copied across threads. (WebCore::):
  • workers/WorkerScriptLoader.cpp: More enum->struct grouping. (WebCore::WorkerScriptLoader::loadSynchronously): More enum->struct grouping. (WebCore::WorkerScriptLoader::loadAsynchronously): More enum->struct grouping.
  • xml/XMLHttpRequest.cpp: Remove all the access control code and some supporting state. (WebCore::XMLHttpRequest::XMLHttpRequest): Ditto. (WebCore::XMLHttpRequest::createRequest): Ditto. (WebCore::XMLHttpRequest::didFinishLoading): Ditto. (WebCore::XMLHttpRequest::didReceiveResponse): Ditto. (WebCore::XMLHttpRequest::didReceiveData): Ditto.
  • xml/XMLHttpRequest.h: Ditto.
11:49 AM Changeset in webkit [47290] by Darin Adler
  • 2 edits in trunk/JavaScriptGlue

Rename the confusing isObject(<class>) to inherits(<class>).
It still works on non-objects, returning false.

Patch by Darin Adler <Darin Adler> on 2009-08-14
Reviewed by Sam Weinig.

  • JSUtils.cpp:

(KJSValueToJSObject):
(KJSValueToCFTypeInternal):
Changed from isObject to inherits.

11:25 AM Changeset in webkit [47289] by eric.carlson@apple.com
  • 2 edits in trunk/LayoutTests

2009-08-14 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Adam Roben.

Remove media/video-played.html from the Windows skipped list, fixed
as part of ​https://bugs.webkit.org/show_bug.cgi?id=28275. Add
media/video-played-collapse.html for now.


​https://bugs.webkit.org/show_bug.cgi?id=28275

  • platform/win/Skipped:
11:12 AM Changeset in webkit [47288] by Darin Adler
  • 26 edits in trunk

JavaScriptCore: Rename the confusing isObject(<class>) to inherits(<class>).
It still works on non-objects, returning false.

Patch by Darin Adler <Darin Adler> on 2009-08-14
Reviewed by Sam Weinig.

  • runtime/ArrayConstructor.cpp:

(JSC::arrayConstructorIsArray): Removed unneeded isObject call
and updated remaining isObject call to new name, inherits.

  • runtime/JSCell.h: Renamed isObject(<class>) to inherits(<class>)

but more importantly, made it non-virtual (it was already inline)
so it is now as fast as JSObject::inherits was.

  • runtime/JSObject.h: Removed inherits function since the one

in the base class is fine as-is. Also made various JSCell functions
that should not be called on JSObject uncallable by making them
both private and not implemented.
(JSC::JSCell::inherits): Updated name.
(JSC::JSValue::inherits): Ditto.

  • debugger/Debugger.cpp:

(JSC::Debugger::recompileAllJSFunctions):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::unwindCallFrame):

  • runtime/ArrayPrototype.cpp:

(JSC::arrayProtoFuncToString):
(JSC::arrayProtoFuncToLocaleString):
(JSC::arrayProtoFuncConcat):

  • runtime/BooleanPrototype.cpp:

(JSC::booleanProtoFuncToString):
(JSC::booleanProtoFuncValueOf):

  • runtime/DateConstructor.cpp:

(JSC::constructDate):

  • runtime/DatePrototype.cpp:

(JSC::dateProtoFuncToString):
(JSC::dateProtoFuncToUTCString):
(JSC::dateProtoFuncToISOString):
(JSC::dateProtoFuncToDateString):
(JSC::dateProtoFuncToTimeString):
(JSC::dateProtoFuncToLocaleString):
(JSC::dateProtoFuncToLocaleDateString):
(JSC::dateProtoFuncToLocaleTimeString):
(JSC::dateProtoFuncGetTime):
(JSC::dateProtoFuncGetFullYear):
(JSC::dateProtoFuncGetUTCFullYear):
(JSC::dateProtoFuncToGMTString):
(JSC::dateProtoFuncGetMonth):
(JSC::dateProtoFuncGetUTCMonth):
(JSC::dateProtoFuncGetDate):
(JSC::dateProtoFuncGetUTCDate):
(JSC::dateProtoFuncGetDay):
(JSC::dateProtoFuncGetUTCDay):
(JSC::dateProtoFuncGetHours):
(JSC::dateProtoFuncGetUTCHours):
(JSC::dateProtoFuncGetMinutes):
(JSC::dateProtoFuncGetUTCMinutes):
(JSC::dateProtoFuncGetSeconds):
(JSC::dateProtoFuncGetUTCSeconds):
(JSC::dateProtoFuncGetMilliSeconds):
(JSC::dateProtoFuncGetUTCMilliseconds):
(JSC::dateProtoFuncGetTimezoneOffset):
(JSC::dateProtoFuncSetTime):
(JSC::setNewValueFromTimeArgs):
(JSC::setNewValueFromDateArgs):
(JSC::dateProtoFuncSetYear):
(JSC::dateProtoFuncGetYear):

  • runtime/FunctionPrototype.cpp:

(JSC::functionProtoFuncToString):

  • runtime/JSActivation.cpp:

(JSC::JSActivation::argumentsGetter):

  • runtime/JSValue.h:
  • runtime/RegExpConstructor.cpp:

(JSC::constructRegExp):

  • runtime/RegExpPrototype.cpp:

(JSC::regExpProtoFuncTest):
(JSC::regExpProtoFuncExec):
(JSC::regExpProtoFuncCompile):
(JSC::regExpProtoFuncToString):

  • runtime/ScopeChain.cpp:

(JSC::ScopeChain::localDepth):

  • runtime/StringPrototype.cpp:

(JSC::stringProtoFuncReplace):
(JSC::stringProtoFuncToString):
(JSC::stringProtoFuncMatch):
(JSC::stringProtoFuncSearch):
(JSC::stringProtoFuncSplit):
Updated to new name, inherits, from old name, isObject.

WebCore: Rename the confusing isObject(<class>) to inherits(<class>).
It still works on non-objects, returning false.

Patch by Darin Adler <Darin Adler> on 2009-08-14
Reviewed by Sam Weinig.

  • bindings/js/JSEventTarget.cpp:

(WebCore::toEventTarget):

  • bindings/js/JSGeolocationCustom.cpp:

(WebCore::createPositionCallback):
(WebCore::createPositionErrorCallback):

  • bindings/js/JSNodeFilterCustom.cpp:

(WebCore::toNodeFilter):

  • bindings/js/JSXMLHttpRequestCustom.cpp:

(WebCore::JSXMLHttpRequest::send):

  • bindings/js/JSXSLTProcessorCustom.cpp:

(WebCore::JSXSLTProcessor::importStylesheet):
(WebCore::JSXSLTProcessor::transformToFragment):
(WebCore::JSXSLTProcessor::transformToDocument):

  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/objc/objc_runtime.mm:

(JSC::Bindings::callObjCFallbackObject):

  • bridge/runtime_method.cpp:

(JSC::callRuntimeMethod):
Updated to new name, inherits, from old name, isObject.

10:58 AM Changeset in webkit [47287] by eric.carlson@apple.com
  • 4 edits
    2 moves in trunk/LayoutTests

2009-08-14 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Simon Fraser.

media/video-played.html always times out on Windows
​https://bugs.webkit.org/show_bug.cgi?id=28275

  • media/video-played-collapse.html:

Make sub-test assumptions obvious for someone reading this later.

  • media/video-played.html: Removed.
  • media/video-played-expected.txt: Removed.
  • media/video-played-ranges-1-expected.txt: Copied from LayoutTests/media/video-played-expected.txt.
  • media/video-played-ranges-1.html: Copied from LayoutTests/media/video-played.html.

Split video-played.html into video-played-ranges-1.html and video-played-ranges-2.html.

  • media/video-played-reset.html:

Make sub-test assumptions obvious for someone reading this later.

  • media/video-played.js: (logRanges):

New but not currently called. Allows all ranges to be logged for debugging.

(testRanges):

Log ranges and the amount of time each test takes when logTestTiming != 0.

(nextTest):

Record test start time when logTestTiming is true;

(startPlaying):

Run each sub-test for 100ms instead of 250ms to decrease overall test time.

(playForMillisecs.callPauseIfTimeIsReached):
(playForMillisecs):

Make error conditions trigger immediate failure instead of just logging.

8:51 AM Changeset in webkit [47286] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-08-14 Benjamin Otte <​otte@gnome.org>

Reviewed by Gustavo Noronha.

Fix webkit_web_view_set_highlight_text_matches() so that highlighting
works properly in documents with frames.

  • webkit/webkitwebview.cpp: (webkit_web_view_set_highlight_text_matches):
7:50 AM Changeset in webkit [47285] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

RVCT elftran fails to resolve QPainter::staticMetaObject coming with
QWebView::RenderHints property.

Patch by Yongjun Zhang <​yongjun.zhang@nokia.com> on 2009-08-14
Reviewed by Simon Hausmann.

This is a temporary fix and will be revereted when the right symbols
exported from Qt lib in S60.

​https://bugs.webkit.org/show_bug.cgi?id=28181

  • Api/qwebview.h:
7:44 AM Changeset in webkit [47284] by Simon Hausmann
  • 4 edits in trunk

Adding QNX as a platform. Currently only tested with Qt.

Patch by Harald Fernengel <​harald.fernengel@nokia.com> on 2009-07-31
Reviewed by Simon Hausmann.

​https://bugs.webkit.org/show_bug.cgi?id=27885

  • JavaScriptCore/runtime/Collector.cpp: Added retrieving of stack base since QNX doesn't have the pthread _nt functions
  • JavaScriptCore/wtf/Platform.h: Added WTF_PLATFORM_QNX and corresponding defines
  • WebCore/bridge/npapi.h: Build fix for missing typedefs on QNX
7:21 AM Changeset in webkit [47283] by Simon Hausmann
  • 8 edits in trunk

WebKit/qt: [Qt] Public API to configure the storage path for HTML5 localStorage
​https://bugs.webkit.org/show_bug.cgi?id=28036

Patch by Laszlo Gombos <Laszlo Gombos> on 2009-08-14
Reviewed by Simon Hausmann.

Disables LocalStorage for QtWebKit by default by setting
QWebSettings::LocalStorageEnabled to false.

Sets up a default for the LocalStorage path so that clients would only
need to enable the LocalStorageEnabled setting to turn on LocalStoragre
support.

Turn on LocalStorage support for QtLauncher and the relevant test
since LocalStorage is now disabled by default for QtWebkit.

  • Api/qwebpage.cpp:

(defaultCachePath):
(initializeApplicationCachePathIfNecessary):
(QWebPagePrivate::QWebPagePrivate):

  • Api/qwebsettings.cpp:

(QWebSettingsPrivate::apply):
(QWebSettings::QWebSettings):
(QWebSettings::setLocalStoragePath):
(QWebSettings::localStoragePath):

  • Api/qwebsettings.h:
  • QtLauncher/main.cpp:

(main):

  • tests/qwebpage/tst_qwebpage.cpp:

(tst_QWebPage::multiplePageGroupsAndLocalStorage):

WebKitTools: [Qt] Public API to configure the storage path for HTML5 localStorage
​https://bugs.webkit.org/show_bug.cgi?id=28036

Patch by Laszlo Gombos <Laszlo Gombos> on 2009-08-14
Reviewed by Simon Hausmann.

Turn on LocalStorage support for Qt DumpRenderTree since
LocalStorage is now disabled by defult for QtWebkit.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::WebPage::WebPage):

7:19 AM Changeset in webkit [47282] by yael.aharon@nokia.com
  • 7 edits in trunk

WebCore: Allow Qt API to list and to remove URL schemes that were registered as
local URL schemes.
[Qt] Allow applications to register their own local URL scheme.
​https://bugs.webkit.org/show_bug.cgi?id=28240

Patch by Yael Aharon <​yael.aharon@nokia.com> on 2009-08-14
Reviewed by Simon Hausmann.

5:54 AM Changeset in webkit [47281] by Simon Hausmann
  • 2 edits in trunk/PlanetWebKit

Add the blog of University of Szeged.
​https://bugs.webkit.org/show_bug.cgi?id=28301

Patch by Csaba Osztrogonac <​oszi@inf.u-szeged.hu> on 2009-08-14
Reviewed by Simon Hausmann.

  • config.ini:
5:05 AM Changeset in webkit [47280] by xan@webkit.org
  • 2 edits in trunk/WebKitTools

2009-08-14 Xan Lopez <​xlopez@igalia.com>

Reviewed by Jan Alonzo.

Do not unref the main webview, it's owned by its parent
container. Instead destroy the container, which should take care
of everything (not terribly important since we exit right after
that, but still).

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (main):
5:01 AM Changeset in webkit [47279] by ariya@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2009-08-14 Ariya Hidayat <​ariya.hidayat@nokia.com>

Rubber-stamped by Tor Arne Vestbø.

[Qt] Add test result after r47255.

  • platform/qt/fast/block/float/avoidance-percent-width-compat-expected.txt: Added.
  • platform/qt/fast/block/float/avoidance-percent-width-strict-expected.txt: Added.
4:49 AM Changeset in webkit [47278] by Simon Hausmann
  • 2 edits in trunk/JavaScriptCore

Currently generic ARM and ARMv7 platforms work only with JSVALUE32
​https://bugs.webkit.org/show_bug.cgi?id=28300

Patch by Gabor Loki <​loki@inf.u-szeged.hu> on 2009-08-14
Reviewed by Simon Hausmann.

  • wtf/Platform.h:
4:39 AM Changeset in webkit [47277] by pfeldman@chromium.org
  • 7 edits in trunk/WebCore

2009-08-14 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Wrap console objects late (in the frontend) so that injected
script was already in place.

​https://bugs.webkit.org/show_bug.cgi?id=28297

  • inspector/ConsoleMessage.cpp: (WebCore::ConsoleMessage::ConsoleMessage):
  • inspector/ConsoleMessage.h:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::addMessageToConsole): (WebCore::InspectorController::startGroup): (WebCore::InspectorController::setFrontendProxyObject):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::InspectorFrontend): (WebCore::InspectorFrontend::addMessageToConsole):
  • inspector/InspectorFrontend.h:
3:56 AM Changeset in webkit [47276] by vestbo@webkit.org
  • 1 edit in trunk/JavaScriptCore/runtime/ArrayConstructor.cpp

Try to fix the build when !JIT_OPTIMIZE_NATIVE_CALL

3:12 AM Changeset in webkit [47275] by ariya@webkit.org
  • 2 edits
    2 adds in trunk/LayoutTests

2009-08-14 Ariya Hidayat <​ariya.hidayat@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Update test result after r47255.
Also while I am at it, add pixel test result.

  • platform/qt/fast/block/float/035-expected.checksum: Added.
  • platform/qt/fast/block/float/035-expected.png: Added.
  • platform/qt/fast/block/float/035-expected.txt: Adjusted.
2:17 AM Changeset in webkit [47274] by Simon Hausmann
  • 2 edits in trunk/JavaScriptCore

Enable JIT on ARM for QT by default
​https://bugs.webkit.org/show_bug.cgi?id=28259

Patch by Gabor Loki <​loki@inf.u-szeged.hu> on 2009-08-14
Reviewed by Simon Hausmann.

  • wtf/Platform.h:
2:17 AM Changeset in webkit [47273] by Simon Hausmann
  • 2 edits in trunk/JavaScriptCore

Enable YARR_JIT on ARM for QT by default
​https://bugs.webkit.org/show_bug.cgi?id=28259

Patch by Gabor Loki <​loki@inf.u-szeged.hu> on 2009-08-14
Reviewed by Simon Hausmann.

  • wtf/Platform.h:
12:09 AM Changeset in webkit [47272] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Remove MarkStack::drain from the JSC exports file

MarkStack::drain is now marked inline, the including it in the exports file
produces an ld warning

12:00 AM Changeset in webkit [47271] by oliver@apple.com
  • 6 edits
    3 adds in trunk

[ES5] Implement Array.isArray
​https://bugs.webkit.org/show_bug.cgi?id=28296

Reviewed by Maciej Stachowiak

Add support for Array.isArray to the Array constructor

Note: See TracTimeline for information about the timeline view.