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

Timeline



Oct 22, 2009:

11:22 PM Changeset in webkit [49969] by mrowe@apple.com
  • 16 edits in trunk

Update configurations to support the mysterious future.

Rubber-stamped by Brady Eidson.

10:27 PM Changeset in webkit [49968] by barraclough@apple.com
  • 2 edits in trunk/WebKitTools

Build fix following bug #30696.

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-22
Reviewed by NOBODY (build fix).

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::evaluateScriptInIsolatedWorld):

10:16 PM Changeset in webkit [49967] by mrowe@apple.com
  • 23 edits
    2 deletes in trunk/WebKitSite

Update to WordPress 2.8.5.

  • blog:
10:06 PM Changeset in webkit [49966] by barraclough@apple.com
  • 3 edits in trunk/WebKit/qt

Build fix following bug #30696.

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-22
Reviewed by NOBODY (speculative build fix - qt is currently already broken!)

  • Api/qwebelement.cpp:

(setupScriptContext):

  • Api/qwebframe.cpp:

(QWebFrame::evaluateJavaScript):

9:57 PM Changeset in webkit [49965] by barraclough@apple.com
  • 2 edits in trunk/WebKit/mac

Build fix following bug #30696.

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-22
Reviewed by NOBODY (build fix).

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::getWindowNPObject):
(WebKit::NetscapePluginInstanceProxy::evaluate):
(WebKit::NetscapePluginInstanceProxy::invoke):
(WebKit::NetscapePluginInstanceProxy::invokeDefault):
(WebKit::NetscapePluginInstanceProxy::construct):
(WebKit::NetscapePluginInstanceProxy::getProperty):
(WebKit::NetscapePluginInstanceProxy::setProperty):
(WebKit::NetscapePluginInstanceProxy::removeProperty):
(WebKit::NetscapePluginInstanceProxy::hasProperty):
(WebKit::NetscapePluginInstanceProxy::hasMethod):
(WebKit::NetscapePluginInstanceProxy::enumerate):

9:48 PM Changeset in webkit [49964] by barraclough@apple.com
  • 3 edits in trunk/WebKit/gtk

Build fix following bug #30696.

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-22
Reviewed by NOBODY (build fix).

  • WebCoreSupport/FrameLoaderClientGtk.cpp:

(WebKit::FrameLoaderClient::windowObjectCleared):

  • webkit/webkitwebframe.cpp:

(webkit_web_frame_get_global_context):

9:33 PM Changeset in webkit [49963] by barraclough@apple.com
  • 101 edits in trunk

WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=30696
Add support for IsolatedWorlds to JSC bindings.

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-16
Reviewed by Sam Weinig & Geoff Garen.

An IsolatedWorld is basically a parallel, separate copy of the window shells and DOM wrapper objects for
a given document. To support isolation this patch:

  • Adds support to the ScriptController to track multiple window shells, one per world.
  • Adds support to Document to support multiple separate wrapper-maps, one per world.
  • Replaces the single global DOM wrapper map (on the WebCoreJSClientData) with separate maps, stored on the (new) IsolatedWorld objects.

In addition to supporting separate copies of these objects, two other features are supported:

  • It is necessary to track the current world on entry into JSC, so that within callbacks out to WebCore we can determine which world (and as such, set of DOM bindings) we should be operating on.
  • EventListeners & Callbacks are run in the world they were registered in. This requires the handler to retain a reference to the world.

No new tests. (Enabled existing isolated world layout tests.)

  • WebCore.base.exp:
  • bindings/js/JSAbstractWorkerCustom.cpp:

(WebCore::JSAbstractWorker::addEventListener):
(WebCore::JSAbstractWorker::removeEventListener):

  • bindings/js/JSCallbackData.cpp:

(WebCore::JSCallbackData::invokeCallback):

  • bindings/js/JSCallbackData.h:

(WebCore::JSCallbackData::JSCallbackData):

  • bindings/js/JSCustomXPathNSResolver.cpp:

(WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):

  • bindings/js/JSDOMApplicationCacheCustom.cpp:

(WebCore::JSDOMApplicationCache::addEventListener):
(WebCore::JSDOMApplicationCache::removeEventListener):

  • bindings/js/JSDOMBinding.cpp:

(WebCore::removeWrappers):
(WebCore::DOMObjectWrapperMap::get):
(WebCore::DOMObjectWrapperMap::set):
(WebCore::DOMObjectWrapperMap::remove):
(WebCore::DOMObjectWrapperMap::take):
(WebCore::IsolatedWorld::IsolatedWorld):
(WebCore::IsolatedWorld::~IsolatedWorld):
(WebCore::EnterIsolatedWorld::EnterIsolatedWorld):
(WebCore::EnterIsolatedWorld::~EnterIsolatedWorld):
(WebCore::JSGlobalDataWorldIterator::JSGlobalDataWorldIterator):
(WebCore::JSGlobalDataWorldIterator::operator bool):
(WebCore::JSGlobalDataWorldIterator::operator*):
(WebCore::JSGlobalDataWorldIterator::operator->):
(WebCore::JSGlobalDataWorldIterator::operator++):
(WebCore::getCurrentWorld):
(WebCore::getNormalWorld):
(WebCore::commonNormalWorld):
(WebCore::commonCurrentWorld):
(WebCore::DOMObjectHashTableMap::mapFor):
(WebCore::DOMObjectWrapperMap::mapFor):
(WebCore::forgetDOMObject):
(WebCore::getCachedDOMNodeWrapper):
(WebCore::forgetDOMNode):
(WebCore::cacheDOMNodeWrapper):
(WebCore::forgetAllDOMNodesForDocument):
(WebCore::forgetWorldOfDOMNodesForDocument):
(WebCore::isObservableThroughDOM):
(WebCore::markDOMNodesForDocument):
(WebCore::markActiveObjectsForContext):
(WebCore::takeWrappers):
(WebCore::updateDOMNodeDocument):
(WebCore::markDOMObjectWrapper):
(WebCore::allowsAccessFromFrame):
(WebCore::printErrorMessageForFrame):
(WebCore::JSC_DebuggerCallFrame_evaluateInWorld):
(WebCore::JSC_callInWorld):
(WebCore::JSC_constructInWorld):
(WebCore::JSC_evaluateInWorld):

  • bindings/js/JSDOMBinding.h:

(WebCore::IsolatedWorld::rememberDocument):
(WebCore::IsolatedWorld::forgetDocument):
(WebCore::IsolatedWorld::rememberScriptController):
(WebCore::IsolatedWorld::forgetScriptController):
(WebCore::DOMObjectHashTableMap::~DOMObjectHashTableMap):
(WebCore::DOMObjectHashTableMap::get):
(WebCore::WebCoreJSClientData::WebCoreJSClientData):
(WebCore::WebCoreJSClientData::currentWorld):
(WebCore::WebCoreJSClientData::normalWorld):
(WebCore::WebCoreJSClientData::rememberWorld):
(WebCore::WebCoreJSClientData::forgetWorld):
(WebCore::debuggerWorld):
(WebCore::pluginWorld):

  • bindings/js/JSDOMGlobalObject.cpp:

(WebCore::JSDOMGlobalObject::createJSAttributeEventListener):
(WebCore::toJSDOMGlobalObject):

  • bindings/js/JSDOMGlobalObject.h:
  • bindings/js/JSDOMWindowBase.cpp:

(WebCore::JSDOMWindowBase::printErrorMessage):
(WebCore::JSDOMWindowBase::commonJSGlobalData):
(WebCore::toJS):
(WebCore::toJSDOMWindow):

  • bindings/js/JSDOMWindowBase.h:
  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::createWindow):
(WebCore::JSDOMWindow::open):
(WebCore::JSDOMWindow::showModalDialog):
(WebCore::JSDOMWindow::setTimeout):
(WebCore::JSDOMWindow::setInterval):
(WebCore::JSDOMWindow::addEventListener):
(WebCore::JSDOMWindow::removeEventListener):

  • bindings/js/JSDOMWindowShell.cpp:

(WebCore::toJS):
(WebCore::toJSDOMWindowShell):

  • bindings/js/JSDOMWindowShell.h:
  • bindings/js/JSDesktopNotificationsCustom.cpp:

(WebCore::JSNotification::addEventListener):
(WebCore::):

  • bindings/js/JSEventListener.cpp:

(WebCore::JSEventListener::JSEventListener):
(WebCore::JSEventListener::handleEvent):
(WebCore::JSEventListener::reportError):

  • bindings/js/JSEventListener.h:

(WebCore::JSEventListener::create):

  • bindings/js/JSEventSourceCustom.cpp:

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

  • bindings/js/JSEventTarget.cpp:

(WebCore::toJS):

  • bindings/js/JSHTMLDocumentCustom.cpp:

(WebCore::JSHTMLDocument::open):

  • bindings/js/JSHTMLFrameSetElementCustom.cpp:

(WebCore::JSHTMLFrameSetElement::nameGetter):

  • bindings/js/JSInspectorBackendCustom.cpp:

(WebCore::JSInspectorBackend::databaseForId):
(WebCore::JSInspectorBackend::inspectedWindow):
(WebCore::JSInspectorBackend::nodeForId):

  • bindings/js/JSLazyEventListener.cpp:

(WebCore::JSLazyEventListener::JSLazyEventListener):
(WebCore::JSLazyEventListener::parseCode):

  • bindings/js/JSLazyEventListener.h:

(WebCore::JSLazyEventListener::create):

  • bindings/js/JSMessageChannelCustom.cpp:

(WebCore::JSMessageChannel::markChildren):

  • bindings/js/JSMessagePortCustom.cpp:

(WebCore::JSMessagePort::markChildren):
(WebCore::JSMessagePort::addEventListener):
(WebCore::JSMessagePort::removeEventListener):

  • bindings/js/JSNodeCustom.cpp:

(WebCore::JSNode::addEventListener):
(WebCore::JSNode::removeEventListener):
(WebCore::JSNode::markChildren):

  • bindings/js/JSNodeFilterCondition.cpp:

(WebCore::JSNodeFilterCondition::acceptNode):

  • bindings/js/JSQuarantinedObjectWrapper.cpp:

(WebCore::JSQuarantinedObjectWrapper::construct):
(WebCore::JSQuarantinedObjectWrapper::call):

  • bindings/js/JSSVGElementInstanceCustom.cpp:

(WebCore::JSSVGElementInstance::addEventListener):
(WebCore::JSSVGElementInstance::removeEventListener):

  • bindings/js/JSSharedWorkerCustom.cpp:

(WebCore::JSSharedWorker::markChildren):

  • bindings/js/JSWebSocketCustom.cpp:

(WebCore::JSWebSocket::addEventListener):
(WebCore::JSWebSocket::removeEventListener):

  • bindings/js/JSWorkerContextCustom.cpp:

(WebCore::JSWorkerContext::addEventListener):
(WebCore::JSWorkerContext::removeEventListener):
(WebCore::JSWorkerContext::setTimeout):
(WebCore::JSWorkerContext::setInterval):

  • bindings/js/JSXMLHttpRequestConstructor.cpp:

(WebCore::constructXMLHttpRequest):

  • bindings/js/JSXMLHttpRequestCustom.cpp:

(WebCore::JSXMLHttpRequest::markChildren):
(WebCore::JSXMLHttpRequest::addEventListener):
(WebCore::JSXMLHttpRequest::removeEventListener):

  • bindings/js/JSXMLHttpRequestUploadCustom.cpp:

(WebCore::JSXMLHttpRequestUpload::markChildren):
(WebCore::JSXMLHttpRequestUpload::addEventListener):
(WebCore::JSXMLHttpRequestUpload::removeEventListener):

  • bindings/js/ScheduledAction.cpp:

(WebCore::ScheduledAction::create):
(WebCore::ScheduledAction::ScheduledAction):
(WebCore::ScheduledAction::executeFunctionInContext):
(WebCore::ScheduledAction::execute):

  • bindings/js/ScheduledAction.h:

(WebCore::ScheduledAction::ScheduledAction):

  • bindings/js/ScriptCachedFrameData.cpp:

(WebCore::ScriptCachedFrameData::ScriptCachedFrameData):
(WebCore::ScriptCachedFrameData::restore):

  • bindings/js/ScriptController.cpp:

(WebCore::ScriptController::~ScriptController):
(WebCore::ScriptController::evaluateInWorld):
(WebCore::ScriptController::evaluate):
(WebCore::ScriptController::evaluateInIsolatedWorld):
(WebCore::ScriptController::clearWindowShell):
(WebCore::ScriptController::initScript):
(WebCore::ScriptController::processingUserGestureEvent):
(WebCore::ScriptController::attachDebugger):
(WebCore::ScriptController::updateDocument):
(WebCore::ScriptController::bindingRootObject):
(WebCore::ScriptController::createRootObject):
(WebCore::ScriptController::windowScriptNPObject):
(WebCore::ScriptController::jsObjectForPluginElement):

  • bindings/js/ScriptController.h:

(WebCore::ScriptController::windowShell):
(WebCore::ScriptController::existingWindowShell):
(WebCore::ScriptController::globalObject):
(WebCore::ScriptController::forgetWorld):

  • bindings/js/ScriptControllerMac.mm:

(WebCore::ScriptController::windowScriptObject):

  • bindings/js/ScriptEventListener.cpp:

(WebCore::createAttributeEventListener):

  • bindings/js/ScriptFunctionCall.cpp:

(WebCore::ScriptFunctionCall::call):
(WebCore::ScriptFunctionCall::construct):

  • bindings/js/ScriptObjectQuarantine.cpp:

(WebCore::getQuarantinedScriptObject):

  • bindings/js/ScriptState.cpp:

(WebCore::scriptStateFromNode):
(WebCore::scriptStateFromPage):

  • bindings/js/ScriptState.h:
  • bindings/js/WorkerScriptController.cpp:

(WebCore::WorkerScriptController::WorkerScriptController):
(WebCore::WorkerScriptController::evaluate):

  • bindings/objc/DOMInternal.mm:

(-[WebScriptObject _initializeScriptDOMNodeImp]):

  • bindings/objc/WebScriptObject.mm:

(-[WebScriptObject callWebScriptMethod:withArguments:]):
(-[WebScriptObject evaluateWebScript:]):

  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/NP_jsobject.cpp:

(_NPN_InvokeDefault):
(_NPN_Invoke):
(_NPN_Evaluate):
(_NPN_Construct):

  • bridge/jni/jni_jsobject.mm:

(JavaJSObject::call):
(JavaJSObject::eval):

  • dom/Document.cpp:

(WebCore::Document::createWrapperCache):

  • dom/Document.h:

(WebCore::Document::wrapperCacheMap):
(WebCore::Document::getWrapperCache):

  • inspector/InspectorController.cpp:

(WebCore::InspectorController::startUserInitiatedProfiling):
(WebCore::InspectorController::stopUserInitiatedProfiling):

  • inspector/JavaScriptCallFrame.cpp:

(WebCore::JavaScriptCallFrame::evaluate):

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::dispatchWindowObjectAvailable):

  • platform/network/mac/AuthenticationMac.mm:
  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::XMLHttpRequest):
(WebCore::XMLHttpRequest::dropProtection):

  • xml/XMLHttpRequest.h:

(WebCore::XMLHttpRequest::create):

WebKit/mac: ​https://bugs.webkit.org/show_bug.cgi?id=30696
Update to incorporate support for IsolatedWorlds in JSC bindings.

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-16
Reviewed by Sam Weinig & Geoff Garen.

  • WebView/WebFrame.mm:

(-[WebFrame _attachScriptDebugger]):
(-[WebFrame _stringByEvaluatingJavaScriptFromString:forceUserGesture:]):
(-[WebFrame globalContext]):

  • WebView/WebScriptDebugDelegate.mm:

(-[WebScriptCallFrame evaluateWebScript:]):

  • WebView/WebView.mm:

(-[WebView aeDescByEvaluatingJavaScriptFromString:]):

WebKit/win: ​https://bugs.webkit.org/show_bug.cgi?id=30696
Update to incorporate support for IsolatedWorlds in JSC bindings.

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-22
Reviewed by Sam Weinig & Geoff Garen.

  • Interfaces/IWebFramePrivate.idl:
  • WebFrame.cpp:

(WebFrame::globalContext):
(WebFrame::windowObjectCleared):
(WebFrame::stringByEvaluatingJavaScriptInIsolatedWorld):

  • WebFrame.h:

WebKitTools: ​https://bugs.webkit.org/show_bug.cgi?id=30696
Enable isolated-worlds tests on mac.

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-22
Reviewed by Sam Weinig & Geoff Garen.

Add private interface for DRT to invoke execution in a given world.

  • DumpRenderTree/LayoutTestController.cpp:

(evaluateScriptInIsolatedWorldCallback):
(LayoutTestController::staticFunctions):

  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/mac/LayoutTestControllerMac.mm:

(LayoutTestController::evaluateScriptInIsolatedWorld):

  • DumpRenderTree/win/LayoutTestControllerWin.cpp:

(LayoutTestController::evaluateScriptInIsolatedWorld):

LayoutTests: ​https://bugs.webkit.org/show_bug.cgi?id=30696
Enable isolated-worlds tests on mac.

  • Update the results to switch from windows to unix line-endings.
  • Update all-window-prototypes.html to allow for properties on the global object with null prototypes (just skip over them).

Patch by Gavin Barraclough <​barraclough@apple.com> on 2009-10-19
Reviewed by Sam Weinig & Geoff Garen.

  • http/tests/security/isolatedWorld/all-window-properties-expected.txt:
  • http/tests/security/isolatedWorld/all-window-prototypes-expected.txt:
  • http/tests/security/isolatedWorld/all-window-prototypes.html:
  • http/tests/security/isolatedWorld/body-properties-expected.txt:
  • http/tests/security/isolatedWorld/body-prototype-expected.txt:
  • http/tests/security/isolatedWorld/document-properties-expected.txt:
  • http/tests/security/isolatedWorld/document-prototype-expected.txt:
  • http/tests/security/isolatedWorld/global-variables-expected.txt:
  • http/tests/security/isolatedWorld/image-properties-expected.txt:
  • http/tests/security/isolatedWorld/image-prototype-expected.txt:
  • http/tests/security/isolatedWorld/location-properties-expected.txt:
  • http/tests/security/isolatedWorld/location-prototype-expected.txt:
  • http/tests/security/isolatedWorld/number-prototype-expected.txt:
  • http/tests/security/isolatedWorld/object-prototype-expected.txt:
  • http/tests/security/isolatedWorld/storage-properties-expected.txt:
  • http/tests/security/isolatedWorld/storage-prototype-expected.txt:
  • http/tests/security/isolatedWorld/string-prototype-expected.txt:
  • http/tests/security/isolatedWorld/window-properties-expected.txt:
  • platform/mac/Skipped:
9:12 PM Changeset in webkit [49962] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-22 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: EventListenersSidebarPane TODO No Longer Needed
​https://bugs.webkit.org/show_bug.cgi?id=30705

  • inspector/front-end/EventListenersSidebarPane.js:
5:22 PM Changeset in webkit [49961] by bweinstein@apple.com
  • 3 edits in trunk/LayoutTests

Updating expected results - unreviewed.

Updated expected computed style results on Windows after r49959.

  • platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
5:00 PM Changeset in webkit [49960] by ap@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Tim Hatcher.

​https://bugs.webkit.org/show_bug.cgi?id=30506
<rdar://problem/7319845> Resources that the server sent as 304 not modified are not shown
in the inspectors resource pane

I don't know how to make a test for this.

The issue here was that preloaded resources weren't added to DocLoader document resource set,
but Web Inspector asked DocLoader to fetch data from CachedResource. Even when (if) document
parser eventually requested the same resource for real and it got added to resource set, it
was too late - the Inspector wasn't updated.

  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::cachedResource): (WebCore::InspectorResource::type): (WebCore::InspectorResource::resourceData):
  • inspector/InspectorResource.h: Fix the issue by trying to fetch corresponding CachedResource harder - also look in global cache. This seems safe, and easier than updating the Inspector on transitions between resource states (revalidate vs. revalidate done and preload vs. non-preload).
4:34 PM Changeset in webkit [49959] by Beth Dakin
  • 17 edits
    36 adds in trunk

WebCore: Fix for <rdar://problem/6942706> ER: Add a CSS property that allows
shadows to work for SVG content
-and corresponding-
​https://bugs.webkit.org/show_bug.cgi?id=30479

Reviewed by Dan Bernstein.

This patch adds a new SVG-specific CSS property called -webkit-
shadow that has the same syntax as -webkit-box-shadow

Add CSSPropertyWebkitShadow to the list of SVG CSS properties, and
make valueForShadow a member function rather than a static function

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::):
(WebCore::CSSComputedStyleDeclaration::valueForShadow):
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

  • css/CSSComputedStyleDeclaration.h:

Call valueForShadow for CSSPropertyWebkitShadow

  • css/SVGCSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getSVGPropertyCSSValue):

Do standard CSS stuff for CSSPropertyWebkitShadow

  • css/SVGCSSParser.cpp:

(WebCore::CSSParser::parseSVGValue):

  • css/SVGCSSPropertyNames.in:
  • css/SVGCSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applySVGProperty):

Set the appropriate shadow on the paint context if we have one.
This seems to be all we have to do to have an SVG-shaped shadow.

  • rendering/SVGRenderSupport.cpp:

(WebCore::SVGRenderBase::prepareToRenderSVGContent):

Add shadow to SVGRenderStyle. I added a new macro that I don't like
just because that's the way things are done here right now.

  • rendering/style/SVGRenderStyle.cpp:

(WebCore::SVGRenderStyle::SVGRenderStyle):
(WebCore::SVGRenderStyle::operator==):

  • rendering/style/SVGRenderStyle.h:
  • rendering/style/SVGRenderStyleDefs.cpp:

(StyleShadowSVGData::StyleShadowSVGData):
(StyleShadowSVGData::operator==):

  • rendering/style/SVGRenderStyleDefs.h:

(WebCore::StyleShadowSVGData::create):
(WebCore::StyleShadowSVGData::copy):
(WebCore::StyleShadowSVGData::operator!=):

LayoutTests: Tests for <rdar://problem/6942706> ER: Add a CSS property that
allows shadows to work for SVG content
-and corresponding-
​https://bugs.webkit.org/show_bug.cgi?id=30479

Reviewed by Dan Bernstein.

New results that include -webkit-shadow:

  • platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:

New tests and their results.

  • platform/mac/svg/css/arrow-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/arrow-with-shadow-expected.png: Added.
  • platform/mac/svg/css/arrow-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/circle-in-mask-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/circle-in-mask-with-shadow-expected.png: Added.
  • platform/mac/svg/css/circle-in-mask-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/clippath-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/clippath-with-shadow-expected.png: Added.
  • platform/mac/svg/css/clippath-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/group-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/group-with-shadow-expected.png: Added.
  • platform/mac/svg/css/group-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/mask-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/mask-with-shadow-expected.png: Added.
  • platform/mac/svg/css/mask-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/path-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/path-with-shadow-expected.png: Added.
  • platform/mac/svg/css/path-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/stars-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/stars-with-shadow-expected.png: Added.
  • platform/mac/svg/css/stars-with-shadow-expected.txt: Added.
  • platform/mac/svg/filters/shadow-on-filter-expected.checksum: Added.
  • platform/mac/svg/filters/shadow-on-filter-expected.png: Added.
  • platform/mac/svg/filters/shadow-on-filter-expected.txt: Added.
  • platform/mac/svg/filters/shadow-on-rect-with-filter-expected.checksum: Added.
  • platform/mac/svg/filters/shadow-on-rect-with-filter-expected.png: Added.
  • platform/mac/svg/filters/shadow-on-rect-with-filter-expected.txt: Added.
  • svg/css/arrow-with-shadow.svg: Added.
  • svg/css/circle-in-mask-with-shadow.svg: Added.
  • svg/css/clippath-with-shadow.svg: Added.
  • svg/css/getComputedStyle-basic-expected.txt:
  • svg/css/group-with-shadow.svg: Added.
  • svg/css/mask-with-shadow.svg: Added.
  • svg/css/path-with-shadow.svg: Added.
  • svg/css/stars-with-shadow.html: Added.
  • svg/filters/shadow-on-filter.svg: Added.
  • svg/filters/shadow-on-rect-with-filter.svg: Added.
4:07 PM Changeset in webkit [49958] by eric@webkit.org
  • 4 edits in trunk/WebCore

2009-10-22 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=25530
[Gtk] Implement LABEL_FOR/LABELLED_BY relationship pair for labels

Implements atk_object_ref_relation_set and LABEL_FOR/LABELLED_BY.
Also causes the accessible name for labeled controls to be based on
the label as expected, rather than based on the contents.

  • accessibility/AccessibilityRenderObject.h:
  • accessibility/AccessibilityRenderObject.cpp: (correspondingLabelForControlElement):
  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (setAtkRelationSetFromCoreObject): (webkit_accessible_ref_relation_set): (webkit_accessible_class_init): (webkit_accessible_get_name):
2:32 PM Changeset in webkit [49957] by eric@webkit.org
  • 4 edits
    2 adds in trunk

2009-10-22 Shu Chang <​Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Added expected result for clicked-link-is-visited.html.
Removed test case from qt/Skipped as it works now.

  • platform/qt/Skipped:
  • platform/qt/fast/history: Added.
  • platform/qt/fast/history/clicked-link-is-visited-expected.txt: Added.

2009-10-22 Shu Chang <​Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Enable track visited links in QWebPage
​https://bugs.webkit.org/show_bug.cgi?id=30574

Test: fast/history/clicked-link-is-visited.html

  • Api/qwebpage.cpp: (QWebPagePrivate::QWebPagePrivate):
2:08 PM Changeset in webkit [49956] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-22 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: windowFocused and windowBlured Fail to Clear/Mark Inspector as "inactive"
​https://bugs.webkit.org/show_bug.cgi?id=30663

Correctly handle focusing/blurring on inner <iframe>'s such as Source Frames.

  • inspector/front-end/inspector.js: (WebInspector.windowFocused): fix for inner iframes (WebInspector.windowBlurred): fix for inner iframes (WebInspector.addMainEventListeners): change useCapture to false
1:33 PM Changeset in webkit [49955] by ggaren@apple.com
  • 4 edits in trunk/JavaScriptCore

Fixed ASSERT when opening Safari's Caches window while the Web Inspector
is open.

Patch by Geoffrey Garen <​ggaren@apple.com> on 2009-10-22
Reviewed by Alexey Proskuryakov.

  • runtime/Collector.cpp:

(JSC::typeName): Added two new types to the type name list in the Collector.
These types have been around for a while, but nobody remembered to consider them here.

  • runtime/JSCell.h:

(JSC::JSCell::isPropertyNameIterator):

  • runtime/JSPropertyNameIterator.h:

(JSC::JSPropertyNameIterator::isPropertyNameIterator): Give the Collector
a way to tell if a cell is a JSPropertyNameIterator.

1:19 PM Changeset in webkit [49954] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-22 Jaime Yap <​jaimeyap@google.com>

Reviewed by Timothy Hatcher.

Makes a page reload optional when activating resource monitoring in
InspectorController. It leaves the default behavior as performing the reload.
Reload is an optional param so callsites do not have to change.

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

No new tests added.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::enableResourceTracking):
  • inspector/InspectorController.h:
1:10 PM Changeset in webkit [49953] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-10-22 Girish Ramakrishnan <Girish Ramakrishnan>

Reviewed by Eric Seidel.

[Qt] Add Print Shortcut to QtLauncher

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

  • QtLauncher/main.cpp: (MainWindow::setupUI):
1:06 PM Changeset in webkit [49952] by sfalken@apple.com
  • 2 edits
    1 delete in trunk/JavaScriptCore

​https://bugs.webkit.org/show_bug.cgi?id=30686
Remove debug-specific def file.
Only Debug_All target uses JavaScriptCore_debug.dll naming, and since
that target is only used internally, maintaining two files just to
suppress a single link warning isn't worthwhile.

Reviewed by Jon Honeycutt.

12:56 PM Changeset in webkit [49951] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-22 Shu Chang <​Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Fix memory leak in QNetworkReplyHandler::abort().
In QNetworkReplyHandler::release(), m_reply should no longer point to its parent
after being released.
​https://bugs.webkit.org/show_bug.cgi?id=30167

No layout test available as a non-functional fix.

  • platform/network/qt/QNetworkReplyHandler.cpp: (WebCore::QNetworkReplyHandler::abort): (WebCore::QNetworkReplyHandler::release):
12:15 PM Changeset in webkit [49950] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2009-10-22 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Fix DOMAgent leak.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController): (WebCore::InspectorController::setFrontendProxyObject): (WebCore::InspectorController::close): (WebCore::InspectorController::releaseDOMAgent):
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.h: (WebCore::InspectorDOMAgent::create):
11:35 AM Changeset in webkit [49949] by Dimitri Glazkov
  • 19 edits in trunk/WebCore

2009-10-22 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Adam Barth.

[V8] Rework event listeners to not hold references to frame or V8 context.

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

Covered by existing layout tests: fast/events/attribute-listener*

  • bindings/scripts/CodeGeneratorV8.pm: Added passing ScriptExecutionContext*

to event listener handling code.

  • bindings/v8/DateExtension.cpp: (WebCore::DateExtension::setAllowSleep): Changed to use currentContext().
  • bindings/v8/ScriptEventListener.cpp: (WebCore::createAttributeEventListener): Reworked to match JSC logic. (WebCore::getEventListenerHandlerBody): Added ScriptExecutionContext* param.
  • bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::V8AbstractEventListener): Removed Frame* param

and usage.

(WebCore::V8AbstractEventListener::handleEvent): Chaged to use ScriptExecutionContext*.
(WebCore::V8AbstractEventListener::invokeEventHandler): Ditto.

  • bindings/v8/V8AbstractEventListener.h: (WebCore::V8AbstractEventListener::getListenerObject): Ditto. (WebCore::V8AbstractEventListener::prepareListenerObject): Ditto.
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::convertEventListenerToV8Object): Added ScriptExecutionContext* param. (WebCore::V8DOMWrapper::getEventListener): Ditto.
  • bindings/v8/V8DOMWrapper.h: (WebCore::V8DOMWrapper::convertEventListenerToV8Object): Ditto.
  • bindings/v8/V8EventListenerList.h: (WebCore::V8EventListenerList::findOrCreateWrapper): Removed ContextType* template param,

because it's no longer needed.

  • bindings/v8/V8IsolatedWorld.h: (WebCore::V8IsolatedWorld::sharedContext): Renamed from shared_context.
  • bindings/v8/V8LazyEventListener.cpp: (WebCore::V8LazyEventListener::V8LazyEventListener): Removed Frame* param and usage. (WebCore::V8LazyEventListener::callListenerFunction): Added ScriptExecutionContext* param. (WebCore::V8LazyEventListener::prepareListenerObject): Ditto.
  • bindings/v8/V8LazyEventListener.h: (WebCore::V8LazyEventListener::create): Reordered params to match JSC impl.
  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::V8Proxy): Adjusted formatting to match WebKit style. (WebCore::V8Proxy::evaluateInNewContext): Changed to use m_context directly. (WebCore::V8Proxy::setInjectedScriptContextDebugId): Ditto. (WebCore::V8Proxy::createWrapperFromCacheSlowCase): Ditto. (WebCore::V8Proxy::isContextInitialized): Ditto. (WebCore::V8Proxy::updateDocumentWrapperCache): Ditto. (WebCore::V8Proxy::clearDocumentWrapperCache): Ditto. (WebCore::V8Proxy::disposeContextHandles): Added explicit disposing of m_context. (WebCore::V8Proxy::clearForClose): Changed to use m_context directly. (WebCore::V8Proxy::clearForNavigation): Ditto. (WebCore::V8Proxy::setSecurityToken): Ditto. (WebCore::V8Proxy::updateDocument): Ditto. (WebCore::V8Proxy::initContextIfNeeded): Ditto. (WebCore::V8Proxy::context): Changed to use v8::Local. (WebCore::V8Proxy::mainWorldContext): Changed to use m_context directly. (WebCore::V8Proxy::setContextDebugId): Ditto. (WebCore::toV8Context):
  • bindings/v8/V8Proxy.h: Removed shared_context decl, changed to use straight

v8::Persistent for m_context.

  • bindings/v8/V8WorkerContextEventListener.cpp: (WebCore::workerProxy): Added. (WebCore::V8WorkerContextEventListener::V8WorkerContextEventListener): Removed

WorkerContextExecutionProxy* param.

(WebCore::V8WorkerContextEventListener::handleEvent): Started using ScriptExecutionContext*.
(WebCore::V8WorkerContextEventListener::reportError): Ditto.
(WebCore::V8WorkerContextEventListener::callListenerFunction): Ditto.
(WebCore::V8WorkerContextEventListener::getReceiverObject): Ditto.

  • bindings/v8/V8WorkerContextEventListener.h: (WebCore::V8WorkerContextEventListener::create): Removed

WorkerContextExecutionProxy* param.

  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::findOrCreateEventListener): Removed ContextType*

template param.

  • bindings/v8/custom/V8CustomEventListener.cpp: (WebCore::V8EventListener::V8EventListener): Removed Frame* param. (WebCore::V8EventListener::getListenerFunction): Started using ScriptExecutionContext*. (WebCore::V8EventListener::callListenerFunction): Ditto.
  • bindings/v8/custom/V8CustomEventListener.h: (WebCore::V8EventListener::create): Removed Frame* param.
11:10 AM Changeset in webkit [49948] by eric.carlson@apple.com
  • 4 edits in trunk/WebKit/mac

2009-10-22 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Dan Bernstein & Simon Fraser.

<rdar://problem/7297717>
​https://bugs.webkit.org/show_bug.cgi?id=30323
Play/pause button in full-screen video controller does not reflect current state

  • WebView/WebVideoFullscreenController.mm: (-[WebVideoFullscreenController setMediaElement:WebCore::]): Register for QTMovieRateDidChangeNotification (-[WebVideoFullscreenController rateChanged:]): Call HUD controller when rate changes.
  • WebView/WebVideoFullscreenHUDWindowController.h:
  • WebView/WebVideoFullscreenHUDWindowController.mm: (-[WebVideoFullscreenHUDWindowController scheduleTimeUpdate]): Call updateRate so play button reflects playback state. (-[WebVideoFullscreenHUDWindowController updateRate]): New, set play button state according to movie state. (-[WebVideoFullscreenHUDWindowController playingChanged:]): Only call [self playing] once.
10:49 AM Changeset in webkit [49947] by darin@chromium.org
  • 5 edits in trunk/WebCore

2009-10-14 Gaurav Shah <​gauravsh@google.com>

Reviewed by Darin Fisher.

Replaces temporary link stub for <keygen> tag handler for the Chromium
browser with a call via the Chromium Bridge.

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

  • platform/SSLKeyGenerator.h:
  • platform/chromium/ChromiumBridge.h:
  • platform/chromium/SSLKeyGeneratorChromium.cpp: (WebCore::getSupportedKeySizes): (WebCore::signedPublicKeyAndChallengeString):
  • platform/chromium/TemporaryLinkStubs.cpp: (WebCore::KURL::fileSystemPath): (WebCore::SharedBuffer::createWithContentsOfFile):
10:48 AM Changeset in webkit [49946] by atwilson@chromium.org
  • 3 edits in trunk/WebCore

Fixes Chromium Mac pasteboard handling to flow through the same code paths as other platforms.

Patch by Avi Drissman <​avi@chromium.org> on 2009-10-22
Reviewed by Eric Seidel.

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

  • platform/chromium/ChromiumBridge.h:
  • platform/chromium/PasteboardChromium.cpp:

(WebCore::Pasteboard::writeImage):

10:33 AM Changeset in webkit [49945] by Simon Fraser
  • 6 edits in trunk/WebCore

2009-10-20 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Make accelerated compositing debug indicators work in release builds
​https://bugs.webkit.org/show_bug.cgi?id=30588

Allow some debug indicators, that show which page elements go into compositing layers,
to work in release builds, when the runtime prefs are set.

  • platform/graphics/GraphicsLayer.cpp: (WebCore::GraphicsLayer::GraphicsLayer): (WebCore::GraphicsLayer::setZPosition):
  • platform/graphics/GraphicsLayer.h:
  • platform/graphics/mac/GraphicsLayerCA.h:
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::clearBorderColor): (WebCore::GraphicsLayer::showRepaintCounter): (WebCore::GraphicsLayerCA::GraphicsLayerCA): (WebCore::GraphicsLayerCA::updateMasksToBounds): (WebCore::GraphicsLayerCA::updateLayerDrawsContent): (WebCore::GraphicsLayerCA::setDebugBorder): (WebCore::GraphicsLayerCA::swapFromOrToTiledLayer): (WebCore::GraphicsLayerCA::setupContentsLayer):
  • platform/graphics/mac/WebLayer.mm:
10:30 AM Changeset in webkit [49944] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-22 Xan Lopez <​xlopez@igalia.com>

Skip failing tests that seems to be somewhat tied to the Mac
platform. See ​https://bugs.webkit.org/show_bug.cgi?id=30680

  • platform/gtk/Skipped:
9:42 AM Changeset in webkit [49943] by tonikitoo@webkit.org
  • 2 edits in trunk/WebKit/qt

Code standarlization for QGVLauncher.

Patch by Antonio Gomes <​tonikitoo@webkit.org> on 2009-10-22
RubberStamped by Tor Arne Vestbø.

1) Made member initilization lists in constructors

to be per line.

2) Made applyProxy method inline as all other methods in

WebPage class.

  • QGVLauncher/main.cpp:

(WebPage::WebPage):
(WebPage::applyProxy):
(MainView::MainView):
(MainWindow::MainWindow):
(MainWindow::init):

9:34 AM Changeset in webkit [49942] by tonikitoo@webkit.org
  • 2 edits in trunk/WebKit/qt

Add a Y-Axis rotation to QGVLauncher.

Patch by Antonio Gomes <​tonikitoo@webkit.org> on 2009-10-22
Reviewed by Tor Arne Vestbø.

It uses the QStateMachine API from Qt 4.6.

  • QGVLauncher/main.cpp:

(WebView::WebView):
(WebView::setYRotation):
(WebView::yRotation):
(MainView::flip):
(MainView::animatedYFlip):
(SharedScene::SharedScene):
(SharedScene::webView):
(MainWindow::init):
(MainWindow::animatedYFlip):
(MainWindow::buildUI):

8:53 AM Changeset in webkit [49941] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-22 Andrew Scherkus <​scherkus@chromium.org>

Reviewed by Eric Carlson.

Fix video-played.js to use an error timeout based on time rather than function call count.

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

  • media/video-played.js: (nowInSecs): Returns the current time in seconds. (playForMillisecs.callPauseIfTimeIsReached): Change from call counts to using elapsed time. (playForMillisecs): Ditto.
8:45 AM Changeset in webkit [49940] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-22 Keishi Hattori <​casey.hattori@gmail.com>

Reviewed by Pavel Feldman.

Inspector should support monitorEvents/un monitorEvents() in the command line
​https://bugs.webkit.org/show_bug.cgi?id=19879

  • inspector/front-end/EventListenersSidebarPane.js: (WebInspector.EventListenersSidebarPane.prototype.update.callback): Ignores event listeners generated by monitorEvent
  • inspector/front-end/InjectedScript.js: (InjectedScript._ensureCommandLineAPIInstalled): Added _inspectorCommandLineAPI._logEvent, _allEventTypes, _normalizeEventTypes, monitorEvent, unmonitorEvent.
8:28 AM Changeset in webkit [49939] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-21 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: windowFocused and windowBlurred Fail to Clear/Mark Inspector as "inactive"
​https://bugs.webkit.org/show_bug.cgi?id=30663

8:23 AM Changeset in webkit [49938] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-22 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Error - requestContentType [undefined] is not an object
​https://bugs.webkit.org/show_bug.cgi?id=30666

  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.prototype._refreshFormData):
4:52 AM Changeset in webkit [49937] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-22 Maxime Simon <Maxime Simon>

Reviewed by Holger Freyther.

[Haiku] Correction of the accented letter width (they were 2 characters long).
​https://bugs.webkit.org/show_bug.cgi?id=30629

  • platform/graphics/haiku/SimpleFontDataHaiku.cpp: (WebCore::SimpleFontData::platformWidthForGlyph):
3:49 AM Changeset in webkit [49936] by eric@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-10-22 Alejandro G. Castro <​alex@igalia.com>

Reviewed by Xan Lopez.

m_webView should be private attribute of the EditorClientGtk.
​https://bugs.webkit.org/show_bug.cgi?id=30664

  • WebKit/gtk/WebCoreSupport/EditorClientGtk.h:
3:39 AM Changeset in webkit [49935] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-22 Xan Lopez <​xlopez@igalia.com>

libsoup can reconstruct URIs with empty querys now, so unskip
another bunch of tests.

  • platform/gtk/Skipped:
3:24 AM Changeset in webkit [49934] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-22 Xan Lopez <​xlopez@igalia.com>

Unskip a couple of working tests.

  • platform/gtk/Skipped:
12:15 AM Changeset in webkit [49933] by jhoneycutt@apple.com
  • 6 edits in trunk

<rdar://problem/7270320> Screenshots of off-screen plug-ins are blank
<rdar://problem/7270314> After halting a transparent PluginView on
Windows, the transparency is applied twice

JavaScriptCore:

Reviewed by Dan Bernstein.

Export WTF::deleteOwnedPtr(HDC).

Ditto.

WebCore:

Replace use of Frame::nodeImage() with a function that takes a snapshot
of a PluginView.

Reviewed by Dan Bernstein.

  • plugins/PluginView.h:

Made paintWindowedPluginIntoContext() non-const, as it now calls
paintIntoTransformedContext(). Declare paintIntoTransformedContext()
and snapshot() for Windows platforms.

  • plugins/win/PluginViewWin.cpp:

(WebCore::PluginView::paintIntoTransformedContext):
Paints into the passed HDC without applying any coordinate translations.
Code moved from paintWindowedPluginIntoContext() and paint(). Removed
the memset() of windowpos in lieu of assignment.
(WebCore::PluginView::paintWindowedPluginIntoContext):
Code moved to paintIntoTransformedContext().
(WebCore::PluginView::paint):
Ditto.
(WebCore::PluginView::snapshot):
Create a context, and if the plug-in is windowless, translate it so the
plug-in will draw at the correct location. Create a bitmap, and select
it into the context. Paint the plug-in, and create a BitmapImage from
the bitmap.
(WebCore::PluginView::halt):
Use snapshot().

12:11 AM Changeset in webkit [49932] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-22 Fumitoshi Ukai <​ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

Fix calculation of length in WebSocketChannel.cpp
​https://bugs.webkit.org/show_bug.cgi?id=30656

  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::didReceiveData):

Oct 21, 2009:

11:20 PM Changeset in webkit [49931] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-10-21 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool's "patch failed to download an apply" error should give more information
​https://bugs.webkit.org/show_bug.cgi?id=30632

  • Scripts/modules/scm.py:
    • Use the common run_command method instead of custom POpen code.
    • Make run_command know how to take pipes as input.
  • Scripts/modules/scm_unittest.py:
    • Add new tests to cover change.
    • Also move test_error_handlers into new SCMClassTests so we don't run it 3 times.
9:32 PM Changeset in webkit [49930] by tkent@chromium.org
  • 2 edits in trunk/WebKitTools

committers

4:57 PM Changeset in webkit [49929] by dumi@chromium.org
  • 2 edits in trunk/WebCore

Adding the ability to enable/disable functions in V8 at runtime.

Patch by Dumitru Daniliuc <​dumi@chromium.org> on 2009-10-21
Reviewed by Dimitri Glazkov.

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

  • bindings/scripts/CodeGeneratorV8.pm:
4:47 PM Changeset in webkit [49928] by jorlow@chromium.org
  • 6 edits
    11 adds
    21 deletes in trunk/LayoutTests

Combine local and session storage tests into one and use script-tests properly
​https://bugs.webkit.org/show_bug.cgi?id=30640

Patch by Jeremy Orlow <​jorlow@chromium.org> on 2009-10-21
Reviewed by Darin Adler.

I was apparently misusing the script-tests stuff. I changed all the tests to
test both local storage and session storage so that we can use the
make-test-script-wrappers stuff as is.

  • storage/domstorage/clear-expected.txt: Added.
  • storage/domstorage/clear.html: Added.
  • storage/domstorage/complex-keys-expected.txt: Added.
  • storage/domstorage/complex-keys.html: Added.
  • storage/domstorage/complex-values-expected.txt: Added.
  • storage/domstorage/complex-values.html: Added.
  • storage/domstorage/localstorage/clear-expected.txt: Removed.
  • storage/domstorage/localstorage/clear.html: Removed.
  • storage/domstorage/localstorage/complex-keys-expected.txt: Removed.
  • storage/domstorage/localstorage/complex-keys.html: Removed.
  • storage/domstorage/localstorage/complex-values-expected.txt: Removed.
  • storage/domstorage/localstorage/complex-values.html: Removed.
  • storage/domstorage/localstorage/quota-expected.txt: Removed.
  • storage/domstorage/localstorage/quota.html: Removed.
  • storage/domstorage/localstorage/remove-item-expected.txt: Removed.
  • storage/domstorage/localstorage/remove-item.html: Removed.
  • storage/domstorage/quota-expected.txt: Added.
  • storage/domstorage/quota.html: Added.
  • storage/domstorage/remove-item-expected.txt: Added.
  • storage/domstorage/remove-item.html: Added.
  • storage/domstorage/script-tests/TEMPLATE.html: Added.
  • storage/domstorage/script-tests/clear.js:

(test):
(runTest):

  • storage/domstorage/script-tests/complex-keys.js:

(test):
(runTest):

  • storage/domstorage/script-tests/complex-values.js:

(test):
(runTest):

  • storage/domstorage/script-tests/no-quota.js: Removed.
  • storage/domstorage/script-tests/quota.js:

(testQuota):
(testNoQuota):
(runTest):

  • storage/domstorage/script-tests/remove-item.js:

(test):
(runTest):

  • storage/domstorage/sessionstorage/clear-expected.txt: Removed.
  • storage/domstorage/sessionstorage/clear.html: Removed.
  • storage/domstorage/sessionstorage/complex-keys-expected.txt: Removed.
  • storage/domstorage/sessionstorage/complex-keys.html: Removed.
  • storage/domstorage/sessionstorage/complex-values-expected.txt: Removed.
  • storage/domstorage/sessionstorage/complex-values.html: Removed.
  • storage/domstorage/sessionstorage/no-quota-expected.txt: Removed.
  • storage/domstorage/sessionstorage/no-quota.html: Removed.
  • storage/domstorage/sessionstorage/remove-item-expected.txt: Removed.
  • storage/domstorage/sessionstorage/remove-item.html: Removed.
4:39 PM Changeset in webkit [49927] by kevino@webkit.org
  • 5 edits in trunk/WebKitTools

Reviewed by Kevin Ollivier.

Update the Windows installer builder to work with Vista / Win 7 and with git.

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

4:10 PM Changeset in webkit [49926] by Darin Adler
  • 4 edits in trunk

WebCore: Swedish search (and other languages as well) is broken while fixing Japanese search
​https://bugs.webkit.org/show_bug.cgi?id=30646

Patch by Darin Adler <Darin Adler> on 2009-10-21
Rolled Japanese tailoring out that was done to fix
​https://bugs.webkit.org/show_bug.cgi?id=30437 earlier.
It was overriding the locale-specific tailoring.
We'll land a fix once we figure out how to add the
Japanese tailoring without removing anything.

  • editing/TextIterator.cpp: Rolled out r49924.

LayoutTests: Update since we rolled Japanese tailoring out that was done to fix
​https://bugs.webkit.org/show_bug.cgi?id=30437 earlier.

Patch by Darin Adler <Darin Adler> on 2009-10-21

  • fast/text/international/japanese-kana-letters-expected.txt: Update

test results to expect failure. We'll land results that expect
success once we re-fix the bug.

4:06 PM Changeset in webkit [49925] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2009-10-21 Andrew Scherkus <​scherkus@chromium.org>

Reviewed by Eric Seidel.

Update more media layout tests to use findMediaFile().

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

  • media/controls-drag-timebar.html: Switch to using findMediaFile().
  • media/video-currentTime-set.html: Remove src attribute and rely on findMediaFile() instead.
3:16 PM Changeset in webkit [49924] by bweinstein@apple.com
  • 6 edits
    3 deletes in trunk

Roll out r49922 because it broke Tiger and SL in strange ways.

2:59 PM Changeset in webkit [49923] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2009-10-21 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: chromium build fix - add missing image to gypi.

  • WebCore.gypi:
2:51 PM Changeset in webkit [49922] by bweinstein@apple.com
  • 8 edits
    3 adds in trunk

WebCore: Fixes part of <​http://webkit.org/b/30522>.
Web Inspector: DOM Exceptions throughout the Inspector should be more human readable.

Reviewed by Eric Seidel.

Expose the description attribute that is now a member of
ExceptionBase as of r49723, so when a user logs an exception, they
can see the description in the Web Inspector.

Test: fast/dom/dom-exception-description.html

  • dom/DOMCoreException.idl:
  • dom/EventException.idl:
  • dom/RangeException.idl:
  • svg/SVGException.idl:
  • xml/XMLHttpRequestException.idl:
  • xml/XPathException.idl:

LayoutTests: Fixes part of <​http://webkit.org/b/30522>.
Web Inspector: DOM Exceptions throughout the Inspector should be more human readable.

Reviewed by Eric Seidel.

Adds a test to check the name and description of some DOM Exceptions.

  • fast/dom/dom-exception-description-expected.txt: Added.
  • fast/dom/dom-exception-description.html: Added.
  • fast/dom/script-tests/dom-exception-description.js: Added.
1:47 PM Changeset in webkit [49921] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-21 Alejandro G. Castro <​alex@igalia.com>

Reviewed by Gustavo Noronha.

[GTK] Added conditional code to avoid using
gdk_window_get_root_coords if we do not have a gtk+ release newer
than 2.17.3.
​https://bugs.webkit.org/show_bug.cgi?id=30636

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
1:28 PM Writing Layout Tests for DumpRenderTree edited by jorlow@chromium.org
(diff)
1:22 PM WebKit Team edited by tkent@chromium.org
(diff)
1:02 PM Changeset in webkit [49920] by sullivan@apple.com
  • 2 edits in trunk/WebKit/mac

Rubber-stamped by Dan Bernstein.

  • WebView/WebDynamicScrollBarsView.h:

Corrected a couple of long-obsolete comments regarding Safari's use of this class.

12:53 PM Changeset in webkit [49919] by bweinstein@apple.com
  • 2 edits in trunk/WebCore

Fixes <​https://bugs.webkit.org/show_bug.cgi?id=30616>.
REGRESSION (r49036): Web Inspector: Summary graph no longer switching to size summary.

Reviewed by Timothy Hatcher.

Add a check to make sure a function we need to calculate percentages is
defined before we use it.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel.prototype._updateGraphDividersIfNeeded):

12:43 PM Changeset in webkit [49918] by eric@webkit.org
  • 4 edits in trunk/LayoutTests

2009-10-21 Alejandro G. Castro <​alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Removed 5333725.html from Skipped and modified the expected
result of 5232159-expected.txt and fake-drag-expected.txt, this
modifications come from the changes in EventSender (bug 30633 and
bug 30636).
​https://bugs.webkit.org/show_bug.cgi?id=30593

  • LayoutTests/platform/gtk/Skipped:
  • LayoutTests/platform/gtk/editing/selection/5232159-expected.txt:
  • LayoutTests/platform/gtk/editing/selection/fake-drag-expected.tx :
12:36 PM Changeset in webkit [49917] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-21 Keishi Hattori <​casey.hattori@gmail.com>

Reviewed by Timothy Hatcher.

REGRESSION: Clicking on nodes in the console should take you to the element in the DOM
​https://bugs.webkit.org/show_bug.cgi?id=27231

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._formatnode): Turns on ElementsTreeOutline.showInElementsPanelEnabled
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline): Added ElementsTreeOutline.showInElementsPanelEnabled (WebInspector.ElementsTreeElement.prototype.onmousedown): Reveals the node in the Elements panel if treeOutline.showInElementsPanelEnabled is true.
12:22 PM Changeset in webkit [49916] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=30637
Fix a compiler warning in windows.

Patch by Jeremy Orlow <​jorlow@chromium.org> on 2009-10-21
Reviewed by Eric Seidel.

  • loader/ResourceLoadNotifier.h:
12:18 PM Changeset in webkit [49915] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-10-21 Shu Chang <​Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Added dummy implementation for keepWebHistory()
​https://bugs.webkit.org/show_bug.cgi?id=30592

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::keepWebHistory):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
12:10 PM Changeset in webkit [49914] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-21 Victor Wang <​victorw@chromium.org>

Reviewed by Eric Seidel.

Fix click-focus-anchor layout test to mimic click event.

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

  • fast/events/click-focus-anchor.html:
12:03 PM Changeset in webkit [49913] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-21 Alejandro G. Castro <​alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Fixed the double click condition, it is not double click if
we move in just in one direction.
​https://bugs.webkit.org/show_bug.cgi?id=30636

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
11:55 AM Changeset in webkit [49912] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-21 Alejandro G. Castro <​alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Initialize the events completly before emitting them.
​https://bugs.webkit.org/show_bug.cgi?id=30633

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
11:32 AM Changeset in webkit [49911] by dimich@chromium.org
  • 3 edits
    2 adds in trunk

Fix the crash when a node not in a document receives dispatchEvent.
​https://bugs.webkit.org/show_bug.cgi?id=30611

Reviewed by Darin Adler.

WebCore:

Test: fast/events/dispatch-event-no-document.html

  • dom/EventTarget.cpp: Check for scriptExecutionContext() at the moment of dispatchEvent; do nothing if no context.

(WebCore::EventTarget::dispatchEvent):

LayoutTests:

  • fast/events/dispatch-event-no-document-expected.txt: Added.
  • fast/events/dispatch-event-no-document.html: Added.
11:25 AM Changeset in webkit [49910] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

wx build fix. Fix for when linking using --as-needed with gcc.

11:23 AM Changeset in webkit [49909] by agl@chromium.org
  • 2 edits in trunk/WebCore

2009-10-21 Adam Langley <​agl@google.com>

Reviewed by Eric Seidel.

Chromium Linux: disable subpixel text on layers.

​https://bugs.webkit.org/show_bug.cgi?id=30635
​http://code.google.com/p/chromium/issues/detail?id=25365

With the addition of layers for drawing rounded corners in r49641,
subpixel text on rounded rectangles broke. This is because the layer
only contains a single alpha channel and this is insufficient to
compose subpixel text correctly.

On Windows, a large body of code in TransparencyWin.cpp exists to try
to deal with this. Even then, in some cases, it downgrades to
anti-aliased text. We need a fix for the grevious effects quickly thus
this patch disables subpixel text when rendering into a layer.

This would be covered by existing tests except that subpixel text is
disabled for pixel tests on Chromium Linux.

  • platform/graphics/chromium/FontLinux.cpp: (WebCore::isCanvasMultiLayered): (WebCore::adjustTextRenderMode): (WebCore::Font::drawGlyphs): (WebCore::Font::drawComplexText):
11:13 AM Changeset in webkit [49908] by kevino@webkit.org
  • 2 edits in trunk/WebKit/wx

Reviewed by Kevin Ollivier.

Include 'WebFrame.h' declared classes in wxPython bindings.

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

11:06 AM Changeset in webkit [49907] by kevino@webkit.org
  • 5 edits in trunk

wxMac build fixes. Fix issues with linking against wrong versions of sqlite3 and curl on Mac 10.4.

9:47 AM Changeset in webkit [49906] by mitz@apple.com
  • 4 edits
    1 copy
    4 adds in trunk

WebCore: Fixed background-clip parsing regressions introduced in r46240.

Reviewed by Sam Weinig.

Test: fast/css/background-clip-text.html

  • css/CSSParser.cpp:

(WebCore::parseBackgroundClip): Added this helper method.
(WebCore::CSSParser::parseFillShorthand): Use parseBackgroundClip() to
reparse the value. The old code called parseFillProperty() again, which
didn’t work, because parseFillProperty() advances the value list.
(WebCore::CSSParser::parseFillProperty): Changed to use
parseBackgroundClip().

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::operator EFillBox): Added a case for
CSSValueWebkitText.

LayoutTests: Added a background-clip parsing test that would have caught some
regressions that were introduced in r46240.

Reviewed by Sam Weinig.

  • fast/css/background-clip-text-expected.txt: Added.
  • fast/css/background-clip-text.html: Added.
  • fast/css/script-tests: Added.
  • fast/css/script-tests/TEMPLATE.html: Copied from LayoutTests/fast/backgrounds/repeat/script-tests/TEMPLATE.html.
  • fast/css/script-tests/background-clip-text.js: Added.

(test):

6:57 AM Changeset in webkit [49905] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-21 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Timothy Hatcher.

Enable creation of custom SidebarTreeElements for different ProfileTypes

ProfileTypes can now create sidebar tree elements of custom types.
​https://bugs.webkit.org/show_bug.cgi?id=30520

  • inspector/front-end/ProfileView.js: (WebInspector.CPUProfileType.prototype.setRecordingProfile): (WebInspector.CPUProfileType.prototype.createSidebarTreeElementForProfile): (WebInspector.CPUProfileType.prototype.createView): (WebInspector.CPUProfile.prototype.get head):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfileType.prototype.buttonClicked): (WebInspector.ProfileType.prototype.viewForProfile): (WebInspector.ProfileType.prototype.createView): (WebInspector.ProfileType.prototype.createSidebarTreeElementForProfile): (WebInspector.ProfilesPanel.prototype.addProfileHeader): (WebInspector.ProfilesPanel.prototype.showProfile):
6:45 AM Changeset in webkit [49904] by Joseph Pecoraro
  • 4 edits in trunk/WebCore

2009-10-20 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: Selectors in the Styles pane should trigger a search for that selector when clicked
​https://bugs.webkit.org/show_bug.cgi?id=17126

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.addNodesToSearchResult): only change focusedDOMNode on a manual search
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertiesSection.prototype._clickSelector): trigger a search
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._startEditing): correctly start editing attribute value instead of name
6:33 AM Changeset in webkit [49903] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2009-10-20 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Fix DOM Agent leak.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController):
6:08 AM Changeset in webkit [49902] by eric@webkit.org
  • 1 edit
    3 adds in trunk/WebCore

2009-10-21 Girish Ramakrishnan <Girish Ramakrishnan>

Reviewed by Holger Freyther.

Plugins: Create manual tests for testing plugins

Create two test files:

  1. windowed.html - Test plugin in windowed mode.
  2. windowless.html - Test plugin in windowless mode.

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

  • manual-tests/plugins/windowed.html: Added.
  • manual-tests/plugins/windowless.html: Added.
2:10 AM Changeset in webkit [49901] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-21 Xan Lopez <​xlopez@igalia.com>

Skip another failing test.

  • platform/gtk/Skipped:
12:52 AM Changeset in webkit [49900] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-21 Xan Lopez <​xlopez@igalia.com>

Skip a few more libxml2 related tests.

  • platform/gtk/Skipped:
12:11 AM Changeset in webkit [49899] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2009-10-21 Satoshi Nakagawa <​psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

  • fast/text/international/japanese-kana-letters-expected.txt: Added.
  • fast/text/international/japanese-kana-letters.html: Added.

2009-10-21 Satoshi Nakagawa <​psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

Test: fast/text/international/japanese-kana-letters.html

  • editing/TextIterator.cpp: (WebCore::createCollator): (WebCore::collator): (WebCore::createSearcher):
Note: See TracTimeline for information about the timeline view.