Timeline


and

02/26/10:

23:01 Changeset [55347] by pfeldman@chromium.org

2010-02-26 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: reload inspected page on Cmd+R / Ctrl+R / F5 key event in inspector.
Drive-by fix for couple of minor front-end problems.

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

  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::reloadPage):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::didFinishLoading): (WebCore::InspectorController::didFailLoading):
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.reloadPage):
  • inspector/front-end/InspectorFrontendHostStub.js: (.WebInspector.InspectorFrontendHostStub.prototype.copyText): (.WebInspector.InspectorFrontendHostStub.prototype.canAttachWindow):
  • inspector/front-end/TextViewer.js: (WebInspector.TextViewer.prototype._paintLine):
  • inspector/front-end/inspector.js: (WebInspector.documentKeyDown): (WebInspector.toggleAttach):

WebKit/chromium

  • src/WebDevToolsFrontendImpl.cpp: (WebKit::WebDevToolsFrontendImpl::WebDevToolsFrontendImpl): (WebKit::WebDevToolsFrontendImpl::jsCanAttachWindow):
  • src/WebDevToolsFrontendImpl.h:
  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl): (devtools.InspectorBackendImpl.prototype.toggleNodeSearch):
22:22 Changeset [55346] by eric@webkit.org

2010-02-26 James Hawkins <jhawkins@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Verify that the Nodes being enumerated are HTML elements.

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

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::forms):
22:06 Changeset [55345] by eric@webkit.org

2010-02-26 Leandro Pereira <leandro@profusion.mobi>

Reviewed by NOBODY.

Add EFL port files to platform/efl.
http://webkit.org/b/35087

  • platform/efl/ScrollbarEfl.cpp: Added.
  • platform/efl/EventLoopEfl.cpp: Added.
  • platform/efl/PlatformKeyboardEventEfl.cpp: Added.
  • platform/efl/DragImageEfl.cpp: Added.
  • platform/efl/PlatformMouseEventEfl.cpp: Added.
21:49 Changeset [55344] by eric@webkit.org

2010-02-26 Leandro Pereira <leandro@profusion.mobi>

Reviewed by NOBODY.

Add EFL port files to platform/efl.
http://webkit.org/b/35087

  • platform/efl/CursorEfl.cpp: Added.
  • platform/efl/LocalizedStringsEfl.cpp: Added.
  • platform/efl/SearchPopupMenuEfl.cpp: Added.
21:32 Changeset [55343] by eric@webkit.org

2010-02-26 Leandro Pereira <leandro@profusion.mobi>

Reviewed by NOBODY.

Add EFL port files to page/efl.
http://webkit.org/b/35087

  • WebCore/page/efl/DragControllerEfl.cpp: Added.
  • WebCore/page/efl/FrameEfl.cpp: Added.
  • WebCore/page/efl/AccessibilityObjectEfl.cpp: Added.
  • WebCore/page/efl/EventHandlerEfl.cpp: Added.
21:16 Changeset [55342] by eric@webkit.org

2010-02-26 Leandro Pereira <leandro@profusion.mobi>

Reviewed by NOBODY.

Add EFL port files to platform/efl.
http://webkit.org/b/35087

  • WebCore/platform/efl/LoggingEfl.cpp: Added.
  • WebCore/platform/efl/ScrollbarThemeEfl.cpp: Added.
  • WebCore/platform/efl/TemporaryLinkStubs.cpp: Added.
  • WebCore/platform/efl/ScrollViewEfl.cpp: Added.
  • WebCore/platform/efl/SharedBufferEfl.cpp: Added.
  • WebCore/platform/efl/DragDataEfl.cpp: Added.
20:59 Changeset [55341] by eric@webkit.org

2010-02-26 Dirk Pranke <dpranke@chromium.org>

Reviewed by Adam Barth.

Change the default port for new-run-webkit-tests when running on
a Mac from 'chromium-mac' to 'mac'. Add a '--chromium' switch to
pick up the default platform-specific version of chromium instead.

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

  • Scripts/webkitpy/layout_tests/port/factory.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
20:42 Changeset [55340] by eric@webkit.org

2010-02-26 Leandro Pereira <leandro@profusion.mobi>

Reviewed by NOBODY.

Add EFL port files to platform/efl.
http://webkit.org/b/35087

  • WebCore/platform/efl/RenderThemeEfl.cpp: Added.
19:53 Changeset [55339] by dpranke@chromium.org

010-02-26 Dirk Pranke <dpranke@chromium.org>

Unreviewed, build fix

third time's the charm getting this path right?

  • pywebsocket/mod_pywebsocket/standalone.py:
19:53 Changeset [55338] by eric@webkit.org

2010-02-26 Leandro Pereira <leandro@profusion.mobi>

Reviewed by NOBODY.

Add EFL port files to platform/efl.
http://webkit.org/b/35087

  • WebCore/platform/efl/PlatformWheelEventEfl.cpp: Added.
  • WebCore/platform/efl/FileChooserEfl.cpp: Added.
  • WebCore/platform/efl/ContextMenuEfl.cpp: Added.
  • WebCore/platform/efl/PlatformScreenEfl.cpp: Added.
  • WebCore/platform/efl/WidgetEfl.cpp: Added.
19:42 Changeset [55337] by dumi@chromium.org

Unreviewed, build fix.

  • pywebsocket/mod_pywebsocket/standalone.py:
19:27 Changeset [55336] by dumi@chromium.org

Moving the script to the right location.

Reviewed by NOBODY (OOPS!).

  • pywebsocket/mod_pywebsocket: Added.
  • pywebsocket/mod_pywebsocket/standalone.py: Copied from WebKitTools/pywebsocket/standalone.py.
  • pywebsocket/standalone.py: Removed.
19:20 Changeset [55335] by eric@webkit.org

2010-02-26 Adam Barth <abarth@webkit.org>

Reviewed by Darin Fisher.

Expose an API for ports to add schemes to the mixed content whitelist
https://bugs.webkit.org/show_bug.cgi?id=35438

Add a notion of a "secure" scheme that doesn't trigger mixed content
warnings. Let folks register new secure schemes in the same way they
can register "local" schemes.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::isMixedContent):
  • page/SecurityOrigin.cpp: (WebCore::secureSchemes): (WebCore::SecurityOrigin::registerURLSchemeAsSecure): (WebCore::SecurityOrigin::shouldTreatURLSchemeAsSecure):
  • page/SecurityOrigin.h:

2010-02-26 Adam Barth <abarth@webkit.org>

Reviewed by Darin Fisher.

Expose an API for ports to add schemes to the mixed content whitelist
https://bugs.webkit.org/show_bug.cgi?id=35438

Expose registerURLSchemeAsSecure via the WebKit API.

  • public/WebSecurityPolicy.h:
  • src/WebSecurityPolicy.cpp: (WebKit::WebSecurityPolicy::registerURLSchemeAsSecure):
18:59 Changeset [55334] by eric@webkit.org

2010-02-26 Noam Rosenthal <noam.rosenthal@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] GraphicsLayerQt: artifacts and wrong transformOrigin
This was due to wrong way of applying cache-mode and transformation
on a graphics-item with HTML. Cache-mode should be updated
when the content type updates, even if it was the same cache-mode

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

Test URL attached to the bug now works correctly.

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::GraphicsLayerQtImpl::computeTransform): (WebCore::GraphicsLayerQtImpl::flushChanges):
18:51 Changeset [55333] by dumi@chromium.org

Unreviewed, fixing the license.

  • pywebsocket/standalone.py:
18:43 Changeset [55332] by eric@webkit.org

2010-02-26 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Qt/Mac: Revert null timer removal (r51105)
https://bugs.webkit.org/show_bug.cgi?id=35396

r51105 removed the null timer event of the carbon event model. This however
breaks the flash of many sites.

Applying patch suggested by Girish Ramakrishnan in bug comment #1.

  • plugins/PluginView.h:
  • plugins/mac/PluginViewMac.cpp: (WebCore::PluginView::platformStart): (WebCore::PluginView::handleMouseEvent): (WebCore::PluginView::nullEventTimerFired): (WebCore::PluginView::mousePosForPlugin):
18:25 Changeset [55331] by eric@webkit.org

2010-02-26 Noam Rosenthal <noam.rosenthal@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Compile error with 3d-canvas
Replaced a direct gl call with a getProcAddress call
https://bugs.webkit.org/show_bug.cgi?id=35448

No new tests.

  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3D::blendColor):
18:08 Changeset [55330] by eric@webkit.org

2010-02-26 Kwang Yul Seo <skyul@company100.net>

Reviewed by Kenneth Rohde Christiansen.

[BREWMP] Port EventHandler
https://bugs.webkit.org/show_bug.cgi?id=34796

Add platform-specific code required to implement EventHandler.

  • page/brew/EventHandlerBrew.cpp: Added. (WebCore::EventHandler::tabsToAllControls): (WebCore::EventHandler::focusDocumentView): (WebCore::EventHandler::passWidgetMouseDownEventToWidget): (WebCore::EventHandler::passMouseDownEventToWidget): (WebCore::EventHandler::eventActivatedView): (WebCore::EventHandler::passSubframeEventToSubframe): (WebCore::EventHandler::passWheelEventToWidget): (WebCore::EventHandler::passMousePressEventToSubframe): (WebCore::EventHandler::passMouseMoveEventToSubframe): (WebCore::EventHandler::passMouseReleaseEventToSubframe): (WebCore::EventHandler::accessKeyModifiers):
17:51 Changeset [55329] by dumi@chromium.org

Adding pywebsocket/standalone.py back to fix the Chromium webkit
canaries. Once all layout tests pass, we can get in the patch that
moves this directorty to WebKitTools/Scripts, update all Chromium
scripts, and revert this patch.

Reviewed by Adam Barth.

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

  • pywebsocket: Added.
  • pywebsocket/standalone.py: Added.
17:51 Changeset [55328] by eric@webkit.org

2010-02-26 Leandro Pereira <leandro@profusion.mobi>

Reviewed by NOBODY.

Add EFL port files to platform/efl.
http://webkit.org/b/35087

  • WebCore/platform/efl/ClipboardEfl.cpp: Added.
  • WebCore/platform/efl/PopupMenuEfl.cpp: Added.
  • WebCore/platform/efl/SharedTimerEfl.cpp: Added.
  • WebCore/platform/efl/RenderThemeEfl.h: Added.
  • WebCore/platform/efl/Language.cpp: Added.
  • WebCore/platform/efl/CookieJarEfl.cpp: Added.
  • WebCore/platform/efl/MIMETypeRegistryEfl.cpp: Added.
17:34 Changeset [55327] by eric@webkit.org

2010-02-26 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] ignore WebKit/gtk/docs/GNUmakefile.in in .gitignore
https://bugs.webkit.org/show_bug.cgi?id=35424

  • .gitignore:
17:17 Changeset [55326] by eric@webkit.org

2010-02-26 Robert Kroeger <rjkroege@chromium.org>

Reviewed by Nate Chapin

To fire each event handler registered on an SVG node once per
event, Chromium needs an implementation of wasCreatedFromMarkup
added to V8LazyEventListener.h that matches the one in
WebKit/WebCore/bindings/js/JSLazyEventListener.h.

This patch adds such a matching implementation of wasCreatedFromMarkup
to V8LazyEventListener.h.

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

  • bindings/v8/V8LazyEventListener.h: (WebCore::V8LazyEventListener::wasCreatedFromMarkup):
17:00 Changeset [55325] by eric@webkit.org

2010-02-26 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] use geoclue_position_get_position_async to get geolocation position.
https://bugs.webkit.org/show_bug.cgi?id=35355

No new tests, behaviour depends on system.

  • platform/gtk/GeolocationServiceGtk.cpp: (WebCore::GeolocationServiceGtk::startUpdating): (WebCore::GeolocationServiceGtk::get_position):
  • platform/gtk/GeolocationServiceGtk.h:
16:44 Changeset [55324] by mthole@apple.com

https://bugs.webkit.org/show_bug.cgi?id=35449
Add ability to hide WebFrame scrollbars and to allow scrollbars to overlap with content.

Patch by Jing Jin <jjin@apple.com> on 2010-02-26
Reviewed by Timothy Hatcher.

  • WebView/WebDynamicScrollBarsView.h:
  • WebView/WebDynamicScrollBarsView.mm:

(-[WebDynamicScrollBarsView setAllowScrollersToOverlapContent:]): Allows scrollbars to
overlap with the document and re-layouts the document.
(-[WebDynamicScrollBarsView setAlwaysHideHorizontalScroller:]): Hides the horizontal scrollbar.
(-[WebDynamicScrollBarsView setAlwaysHideVerticalScroller:]): Hides the vertical scrollbar.
(-[WebDynamicScrollBarsView contentViewFrame]): Calculates the appropriate frame based
on allowScrollersToOverlapContent.
(-[WebDynamicScrollBarsView tile]): If allowScrollersToOverlapContent is YES, set the
contentView's frame so it overlaps with the scrollbar.
(-[WebDynamicScrollBarsView updateScrollers]): Take into account hideHorizontalScroller
and hideVerticalScroller.
(-[WebDynamicScrollBarsView reflectScrolledClipView:]): set drawsBackground to NO when
scrollbars are overlapping with content, so we don't get trails during scrollbar draw updates.

  • WebView/WebDynamicScrollBarsViewInternal.h:
  • WebView/WebFrame.h:
  • WebView/WebFrame.mm:

(-[WebFrame setAllowScrollersToOverlapContent:]): Hook for [WebDynamicScrollBarsView setAllowScrollersToOverlapContent:]
(-[WebFrame setAlwaysHideHorizontalScroller:]): Hook for [WebDynamicScrollBarsView setAlwaysHideHorizontalScroller:]
(-[WebFrame setAlwaysHideVerticalScroller:]): Hook for [WebDynamicScrollBarsView setAlwaysHideVerticalScroller:]

16:32 Changeset [55323] by yaar@chromium.org

2010-02-26 Yaar Schnitman <yaar@chromium.org>

Chromium Win build fix.

  • WebCore.gyp/WebCore.gyp: Added missing file to action input.
16:19 Changeset [55322] by barraclough@apple.com

Rubber Stamped by Geoff Garen.

Remove wrec. All builds should have switched to yarr by now.

(JSC::RegExp::match):

  • runtime/RegExp.h:
  • wrec: Removed.
  • wrec/CharacterClass.cpp: Removed.
  • wrec/CharacterClass.h: Removed.
  • wrec/CharacterClassConstructor.cpp: Removed.
  • wrec/CharacterClassConstructor.h: Removed.
  • wrec/Escapes.h: Removed.
  • wrec/Quantifier.h: Removed.
  • wrec/WREC.cpp: Removed.
  • wrec/WREC.h: Removed.
  • wrec/WRECFunctors.cpp: Removed.
  • wrec/WRECFunctors.h: Removed.
  • wrec/WRECGenerator.cpp: Removed.
  • wrec/WRECGenerator.h: Removed.
  • wrec/WRECParser.cpp: Removed.
  • wrec/WRECParser.h: Removed.
  • wscript:
15:47 Changeset [55321] by brettw@chromium.org

2010-02-26 Brett Wilson <brettw@chromium.org>

Fix chromium build. This test's expectation became obsolete with
recent changes to KURL which my previous change brings Chromium's
port in compliance with.

  • tests/KURLTest.cpp: (ComponentCase::TEST):
15:26 Changeset [55320] by yaar@chromium.org

2010-02-26 Yaar Schnitman <yaar@chromium.org>

Chromium build fix.

  • features.gypi: Added missing new variable enable_svg.
15:18 Changeset [55319] by brettw@chromium.org

2010-02-12 Brett Wilson <brettw@chromium.org>

Reviewed by Adam Barth.

Update the Google-URL version of KURL and the V8 bindings to the new
behavior of KURL.IsStandard.

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

This is covered by fast/dom/Window/invalid-protocol.html

  • bindings/v8/custom/V8LocationCustom.cpp: (WebCore::V8Location::protocolAccessorSetter):
  • platform/KURLGoogle.cpp: (WebCore::KURL::setProtocol): (WebCore::KURL::isHierarchical):
15:13 Changeset [55318] by yaar@chromium.org

2010-02-25 Yaar Schnitman <yaar@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Auto-generate and split DerivedSourcesAllInOne.cpp
https://bugs.webkit.org/show_bug.cgi?id=33048

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gyp/scripts/action_derivedsourcesallinone.py: Added.
  • bindings/v8/DerivedSourcesAllInOne.cpp: Removed.
14:53 Changeset [55317] by barraclough@apple.com
  • bridge/qt/qt_pixmapruntime.cpp:

(JSC::Bindings::QtPixmapInstance::invokeMethod):

Reviewed by NOBODY (Build fix following r55312).

14:39 Changeset [55316] by kov@webkit.org

WebKit/gtk

2009-12-04 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[GTK] Needs proper reporting of frame loader callbacks, in DRT
https://bugs.webkit.org/show_bug.cgi?id=32170

Add new signal to report when the document load is finished for a
frame.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidFinishDocumentLoad):
  • webkit/webkitwebview.cpp: (webkit_web_view_class_init):

WebKitTools

2010-02-26 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[GTK] Needs proper reporting of frame loader callbacks, in DRT
https://bugs.webkit.org/show_bug.cgi?id=32170

Fix reporting of unload handlers, so that it is emitted where
needed, not after it.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewLoadFinished): (webViewDocumentLoadFinished): (createWebView):

LayoutTests

2010-02-26 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[GTK] Needs proper reporting of frame loader callbacks, in DRT
https://bugs.webkit.org/show_bug.cgi?id=32170

Unskip tests that now pass.

  • LayoutTests/platform/gtk/http/tests/loading/redirect-with-no-location-crash-expected.txt: rebaselined.
  • platform/gtk/Skipped:
14:30 Changeset [55315] by ap@apple.com

Reviewed by Anders Carlsson.

https://bugs.webkit.org/show_bug.cgi?id=35450
Crash when a Java array property accessor raises an exception

Test: java/inaccessible-class.html. Adding with a FAIL result, because this doesn't raise
an exception in Firefox. I don't know what the formally correct behavior is, but it's good
to avoid crashing.

  • bridge/jni/jsc/JNIBridgeJSC.cpp: (JavaField::valueFromInstance): Check if the result is null before interpreting it as an object or array.
14:27 Changeset [55314] by krit@webkit.org

2010-02-26 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Extend AffineTransform to mapQuad
https://bugs.webkit.org/show_bug.cgi?id=35444

This makes mapQuad available for AffineTransform. So that platforms
can make use of it after the switch from TransformationMatrix to
AffineTransform in GraphicsContext.

  • platform/graphics/transforms/AffineTransform.cpp: (WebCore::AffineTransform::mapRect): mapRect already did the calculation for mapQuad but gave back the

boundingBox of the resulting FloatQuad.

(WebCore::AffineTransform::mapQuad):

  • platform/graphics/transforms/AffineTransform.h:
14:26 Changeset [55313] by beidson@apple.com

Arrow keys do not scroll source view in Resources pane or Scripts pane.
<rdar://problem/7594367> and https://bugs.webkit.org/show_bug.cgi?id=34356

Reviewed by Pavel Feldman.

Handle vertical scrolling in the Text Viewer:

  • inspector/front-end/TextViewer.js:

(WebInspector.TextViewer): Listen for the keydown event.
(WebInspector.TextViewer.prototype._handleKeyDown): If the event has no modifiers and refers

to an arrow key, scroll. The horizontal values were ripped from the default "pixels per scroll line"
value in ScrollBar.h.

14:21 Changeset [55312] by barraclough@apple.com

Bug 35401 - Fix handling of errors in handling calls over bridge,
where base object bridge-type does not match method bridge-type.

Reviewed by Alexey Proskuryakov.

The code assumes users will only attempt to invoke a Java method
on a Java base object, etc.
Add language specific subclasses of RuntimeMethod, and pass the
RuntimeMethod into invokeMethod, so we can typecheck before
casting. Throw an exception on type mismatch.

WebCore:

  • WebCore.base.exp:
  • WebCore.xcodeproj/project.pbxproj:
  • bridge/c/c_instance.cpp:

(JSC::Bindings::CRuntimeMethod::CRuntimeMethod):new class to distinguish this type of RuntimeMethod.
(JSC::Bindings::CInstance::getMethod): create an appropriate sublclass of RuntimeMethod.
(JSC::Bindings::CInstance::invokeMethod): dynamically check the type of the RuntimeMethod.

  • bridge/c/c_instance.h:
  • bridge/jni/jsc/JavaInstanceJSC.cpp:

(JavaRuntimeMethod::JavaRuntimeMethod): new class to distinguish this type of RuntimeMethod.
(JavaInstance::getMethod): create an appropriate sublclass of RuntimeMethod.
(JavaInstance::invokeMethod): dynamically check the type of the RuntimeMethod.

  • bridge/jni/jsc/JavaInstanceJSC.h:
  • bridge/jsc/BridgeJSC.h:
  • bridge/objc/objc_instance.h:
  • bridge/objc/objc_instance.mm:

(ObjcRuntimeMethod::ObjcRuntimeMethod): new class to distinguish this type of RuntimeMethod.
(ObjcInstance::getMethod): create an appropriate sublclass of RuntimeMethod.
(ObjcInstance::invokeMethod): dynamically check the type of the RuntimeMethod.
(ObjcInstance::invokeObjcMethod): new method, takes an ObjcMethod* as an argument so that we don't need to dynamically determine the type.

  • bridge/objc/objc_runtime.mm:

(JSC::Bindings::callObjCFallbackObject): use new invokeObjcMethod method.

  • bridge/runtime_method.cpp:

(JSC::callRuntimeMethod): pass RuntimeMethod as argument to invokeMethod, rather than its MethodList.

  • bridge/runtime_object.cpp:

(JSC::RuntimeObject::methodGetter): use new getMethod method.

WebKit/mac:

  • Plugins/Hosted/ProxyInstance.h:
  • Plugins/Hosted/ProxyInstance.mm:

(WebKit::PluginRuntimeMethod::PluginRuntimeMethod): new class to distinguish this type of RuntimeMethod.
(WebKit::ProxyInstance::getMethod): create an appropriate sublclass of RuntimeMethod.
(WebKit::ProxyInstance::invokeMethod): dynamically check the type of the RuntimeMethod.

LayoutTests:

  • java/java-and-plugins.html: Add tests for passing mismatched this objects to methdods.
14:17 Changeset [55311] by oliver@apple.com

2010-02-26 Oliver Hunt <oliver@apple.com>

Reviewed by Geoff Garen.

Make the lookup table generator include an explicit cast to expected
type of the function. We do this because otherwise the blind intptr_t
cast that is subsequently applied allows incorrectly typed functions
to be inserted into the table, where they will only fail at runtime.
This change makes such errors produce a compile time failure.

14:07 Changeset [55310] by eric@webkit.org

2010-02-26 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Basic tests for munder, munderover, and mover elements.

  • mathml/presentation/over.xhtml: Added.
  • mathml/presentation/under.xhtml: Added.
  • mathml/presentation/underover.xhtml: Added.
  • platform/mac/mathml/presentation/over-expected.checksum: Added.
  • platform/mac/mathml/presentation/over-expected.png: Added.
  • platform/mac/mathml/presentation/over-expected.txt: Added.
  • platform/mac/mathml/presentation/under-expected.checksum: Added.
  • platform/mac/mathml/presentation/under-expected.png: Added.
  • platform/mac/mathml/presentation/under-expected.txt: Added.
  • platform/mac/mathml/presentation/underover-expected.checksum: Added.
  • platform/mac/mathml/presentation/underover-expected.png: Added.
  • platform/mac/mathml/presentation/underover-expected.txt: Added.

2010-02-26 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Added basic support for the munder, munderover, and mover elements.

Tests: mathml/presentation/over.xhtml

mathml/presentation/under.xhtml
mathml/presentation/underover.xhtml

  • WebCore.xcodeproj/project.pbxproj:
  • mathml/MathMLInlineContainerElement.cpp: (WebCore::MathMLInlineContainerElement::createRenderer):
  • mathml/RenderMathMLUnderOver.cpp: Added. (WebCore::RenderMathMLUnderOver::RenderMathMLUnderOver): (WebCore::RenderMathMLUnderOver::addChild): (WebCore::getOffsetHeight): (WebCore::RenderMathMLUnderOver::stretchToHeight): (WebCore::RenderMathMLUnderOver::layout): (WebCore::RenderMathMLUnderOver::baselinePosition): (WebCore::RenderMathMLUnderOver::nonOperatorHeight):
  • mathml/RenderMathMLUnderOver.h: Added. (WebCore::RenderMathMLUnderOver::hasBase): (WebCore::RenderMathMLUnderOver::):
13:48 Changeset [55309] by tonikitoo@webkit.org

[Gtk] Make DRT EventSender::keyDown to consider 'hardware_keycode' field when synthesizing an event.
https://bugs.webkit.org/show_bug.cgi?id=35432

Patch by Antonio Gomes <tonikitoo@webkit.org>
Reviewed by Gustavo Noronha.

When a directional key-press event (arrow keys, End, Home, PageUp,
PageDown, etc) is synthesized by DRT's EventSender and it targets
an editor (e.g. <input type=text>, <textare>, etc), event is processed
by WebCore::EventHandler. On the other hand, if event target is not
an editor, event is bubbled up to Gtk+ for processing. In such cases,
if 'hardware_keycode' is not provided at event synthesize time
its processing fails (at some point in gtk_bindings_activate_event),
and no scroll action is performed.

Unskip fast/events/node-event-anchor-lock.html

  • DumpRenderTree/win/EventSender.cpp:

(keyDownCallback):

  • platform/gtk/Skipped:
13:43 Changeset [55308] by senorblanco@chromium.org

2010-02-24 Stephen White <senorblanco@chromium.org>

Reviewed by Darin Adler.

This CL implements a simple ad-hoc parser for CSS rgb() values.
If it fails, it returns false and the normal lex/yacc parser will
be invoked.


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

Covered by fast/canvas/canvas-bg.html, fast/canvas/canvas-bg-zoom.html,
and many more.

  • css/CSSParser.cpp: (WebCore::parseInt): (WebCore::CSSParser::parseColor):
13:40 Changeset [55307] by darin@chromium.org

2010-02-26 Darin Fisher <darin@chromium.org>

Fix mac warning (that gets treated as an error in the chromium build).

  • src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::cookiesEnabled):
13:33 Changeset [55306] by eric@webkit.org

2010-02-26 Jarkko Sakkinen <jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

https://bugs.webkit.org/show_bug.cgi?id=35419
3D canvas did not update when WTF_USE_ACCELERATED_COMPOSITING
was enabled but not from run-time settings. Added run-time
check that compositing is enabled.

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::markContextChanged):
13:24 Changeset [55305] by tonikitoo@webkit.org

2010-02-26 Antonio Gomes <tonikitoo@webkit.org>

Complementary commit of r55300. Missing "notify::" to signal name.

  • tests/testwebview.c: (test_webkit_web_view_grab_focus):
13:16 Changeset [55304] by eric@webkit.org

2010-02-26 Jarkko Sakkinen <jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

Compilation failed because functions glSampleCoverage, glBlendEquation
and glActiveTexture were not available.
https://bugs.webkit.org/show_bug.cgi?id=35423

  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3DInternal::GraphicsContext3DInternal): (WebCore::GraphicsContext3D::activeTexture): (WebCore::GraphicsContext3D::blendEquation): (WebCore::GraphicsContext3D::sampleCoverage):
12:59 Changeset [55303] by eric@webkit.org

2010-02-26 Jarkko Sakkinen <jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

https://bugs.webkit.org/show_bug.cgi?id=35380
Fixed compilation error when WTF_USE_ACCELERATED_COMPOSITING=0

  • css/MediaQueryEvaluator.cpp: (WebCore::transform_3dMediaFeatureEval):
12:42 Changeset [55302] by eric@webkit.org

2010-02-26 Nicholas Young <nicholas.young@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Incorrect painting after a size changed.
https://bugs.webkit.org/show_bug.cgi?id=35412

No new tests. Bug fix in backend.

  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: (WebCore::MediaPlayerPrivate::paint): Provide the source rectangle, rather than inferring it.
12:41 Changeset [55301] by kov@webkit.org

2010-02-26 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rebaseline of platform-specific results for test.

SVGResourceClipper needs to be moved to RenderSVGResourceClipper
https://bugs.webkit.org/show_bug.cgi?id=35421

  • platform/gtk/svg/custom/use-on-clip-path-with-transformation-expected.txt:
12:32 Changeset [55300] by tonikitoo@webkit.org

[GTK] Make webkit_web_view_grab_focus to active focus controller.
https://bugs.webkit.org/show_bug.cgi?id=35402

Reviewed by Xan Lopez.
Patch by Antonio Gomes <tonikitoo@webkit.org>

When programatically setting focus to an element in an inner document,
calling "hasFocus()" from this document returns FALSE, because
document's FocusController is not activated. It does not happen
if |document| is the main document.

Making webkit_web_view_grab_focus to actually activate the FocusController,
fixes the issue.

  • tests/testwebview.c:

(server_callback):
(test_webkit_web_view_grab_focus):

  • webkit/webkitwebview.cpp:

(webkit_web_view_grab_focus):

12:24 Changeset [55299] by eric@webkit.org

2010-02-26 Nicholas Young <nicholas.young@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Incorrect handling of MIME types in Media Player
https://bugs.webkit.org/show_bug.cgi?id=35413

No new tests. Bug fix in backend.

  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: (WebCore::MediaPlayerPrivate::supportsType): Corrected logic.
12:20 Changeset [55298] by alex@webkit.org

Unreviewed.

Reverted last patch (r55295), it causes problems with the frames.

  • WebCoreSupport/ChromeClientGtk.cpp:
12:08 Changeset [55297] by eric@webkit.org

2010-02-26 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=35418
[Gtk] Every table, including layout tables, should be exposed as a table

This patch exposes layout tables as tables. (Addressing the problematic table
hierarchy remains to be done.)

  • accessibility/gtk/AccessibilityObjectAtk.cpp: (AccessibilityObject::accessibilityPlatformIncludesObject):
  • accessibility/AccessibilityRenderObject.cpp: (AccessibilityRenderObject::determineAccessibilityRole):
11:50 Changeset [55296] by eric@webkit.org

2010-02-26 Janne Koskinen <janne.p.koskinen@digia.com>

Reviewed by Simon Hausmann.

[Qt] Symbian specific getCPUTime implemetation
https://bugs.webkit.org/show_bug.cgi?id=34742

Default implementation doesn't work on Symbian devices.
This change adds a proper implementation by
asking thread execution time from the current thread.

  • runtime/TimeoutChecker.cpp: (JSC::getCPUTime):
10:54 Changeset [55295] by alex@webkit.org

[GTK] Some region checks in scroll are not required
https://bugs.webkit.org/show_bug.cgi?id=35142

Reviewed by Xan Lopez.

Removes some of the operations checking the moved and invalidated
regions when scrolling, it is done already in
gdk_window_move_region.

  • WebCoreSupport/ChromeClientGtk.cpp:
10:51 Changeset [55294] by weinig@apple.com

Rubber-stamped by Adele Peterson.

Update Security Group Members list.

  • security/security-group-members.html:
10:47 Changeset [55293] by krit@webkit.org

2010-02-26 Dirk Schulze <krit@webkit.org>

Unreviewed LayoutTest fix.

SVGResourceClipper needs to be moved to RenderSVGResourceClipper
https://bugs.webkit.org/show_bug.cgi?id=35421

This corrects a result of a new clip test for SVG.

  • platform/mac/svg/custom/use-on-clip-path-with-transformation-expected.txt:
10:37 Changeset [55292] by dglazkov@chromium.org

2010-02-26 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by David Levin.

new-webkit-run-tests: Extraneous parenthesis in websocket_server.py
https://bugs.webkit.org/show_bug.cgi?id=35436

  • Scripts/webkitpy/layout_tests/port/websocket_server.py: Removed extra paren.
10:13 Changeset [55291] by ossy@webkit.org

Unreviewed trivial fix.

[Qt] Update platform specific expected file after r55289.

  • platform/qt/svg/css/clippath-with-shadow-expected.txt:
10:09 Changeset [55290] by abarth@webkit.org

2010-02-26 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

XSSAuditor is super super super slow
https://bugs.webkit.org/show_bug.cgi?id=35373

In this patch, we separate the decoding cache for the page's URL and
form data. Previously, we used the same cache for both, which caused
us miss the cache every time when the page had form data (because the
cache only stored one entry). When the form data is large, we were
wasting a lot of time canonicalizing.

  • page/XSSAuditor.cpp: (WebCore::XSSAuditor::findInRequest):
  • page/XSSAuditor.h:
10:02 Changeset [55289] by krit@webkit.org

2010-02-26 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGResourceClipper needs to be moved to RenderSVGResourceClipper
https://bugs.webkit.org/show_bug.cgi?id=35421

Move SVGResourceClipper to RenderSVGResourceClipper. This follows the changes
of Masker and helps to clean up the SVG code. Code snippets from SVGClipPathElement
and SVGResourceClipper got combined in RenderSVGResourceClipper. DRT results are
more readable for clipPath's now. It's possible to see the unit type of clipper, as
well as it's childs instead of just one path.

  • Android.mk:
  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • rendering/RenderSVGResource.h: (WebCore::):
  • rendering/RenderSVGResourceClipper.cpp: Added. (WebCore::RenderSVGResourceClipper::RenderSVGResourceClipper): (WebCore::RenderSVGResourceClipper::~RenderSVGResourceClipper): (WebCore::RenderSVGResourceClipper::invalidateClients): (WebCore::RenderSVGResourceClipper::invalidateClient): (WebCore::RenderSVGResourceClipper::applyResource): (WebCore::RenderSVGResourceClipper::resourceBoundingBox):
  • rendering/RenderSVGResourceClipper.h: Added. (WebCore::RenderSVGResourceClipper::renderName): (WebCore::RenderSVGResourceClipper::resourceType): (WebCore::RenderSVGResourceClipper::clipPathUnits):
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderBase::prepareToRenderSVGContent): (WebCore::SVGRenderBase::clipperBoundingBoxForRenderer): (WebCore::SVGRenderBase::deregisterFromResources):
  • rendering/SVGRenderTreeAsText.cpp: (WebCore::operator<<): (WebCore::writeStyle): (WebCore::writeSVGResource): (WebCore::writeResources):
  • svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::svgAttributeChanged): (WebCore::SVGClipPathElement::childrenChanged): (WebCore::SVGClipPathElement::createRenderer):
  • svg/SVGClipPathElement.h: (WebCore::SVGClipPathElement::isValid):
  • svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::invalidateResources):
  • svg/graphics/SVGResource.h: (WebCore::): (WebCore::SVGResource::isPaintServer): (WebCore::SVGResource::isFilter): (WebCore::SVGResource::isMarker):
  • svg/graphics/SVGResourceClipper.cpp: Removed.
  • svg/graphics/SVGResourceClipper.h: Removed.

2010-02-26 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGResourceClipper needs to be moved to RenderSVGResourceClipper
https://bugs.webkit.org/show_bug.cgi?id=35421

Update results after the move of SVGResourceClipper. Results are better
readable and more sensible now.

  • platform/mac/svg/W3C-SVG-1.1/masking-intro-01-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/masking-path-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/masking-path-02-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/masking-path-04-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/masking-path-05-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/painting-marker-02-f-expected.txt:
  • platform/mac/svg/batik/text/textEffect2-expected.txt:
  • platform/mac/svg/batik/text/textProperties-expected.txt:
  • platform/mac/svg/css/clippath-with-shadow-expected.txt:
  • platform/mac/svg/custom/clip-path-child-changes-expected.txt:
  • platform/mac/svg/custom/clip-path-display-none-child-expected.txt:
  • platform/mac/svg/custom/clip-path-href-changes-expected.txt:
  • platform/mac/svg/custom/clip-path-referencing-use-expected.txt:
  • platform/mac/svg/custom/clip-path-referencing-use2-expected.txt:
  • platform/mac/svg/custom/clip-path-units-changes-expected.txt:
  • platform/mac/svg/custom/clip-path-with-transform-expected.txt:
  • platform/mac/svg/custom/empty-clip-path-expected.txt:
  • platform/mac/svg/custom/focus-ring-expected.txt:
  • platform/mac/svg/custom/image-with-transform-clip-filter-expected.txt:
  • platform/mac/svg/custom/invalid-css-expected.txt:
  • platform/mac/svg/custom/js-late-clipPath-and-object-creation-expected.txt:
  • platform/mac/svg/custom/js-late-clipPath-creation-expected.txt:
  • platform/mac/svg/custom/recursive-clippath-expected.txt:
  • platform/mac/svg/custom/resource-invalidate-on-target-update-expected.txt:
  • platform/mac/svg/custom/text-clip-expected.txt:
  • platform/mac/svg/custom/visibility-override-clip-expected.txt:
  • platform/mac/svg/filters/filter-clip-expected.txt:
  • svg/custom/use-clipped-transform-expected.txt:
08:53 Changeset [55288] by eric@webkit.org

2010-02-26 Diego Gonzalez <diego.gonzalez@openbossa.org>

Reviewed by Simon Hausmann.

Fix documentation of QWebSettings::setUserStyleSheetUrl method

The base 64 data in the exemple shows a not valid code.

REGRESSION: QWebSettings::setUserStyleSheetUrl is not working with a data URL (Base64)
https://bugs.webkit.org/show_bug.cgi?id=34802

  • Api/qwebsettings.cpp:
08:52 Changeset [55287] by benm@google.com

The element that a touchevent is dispatched to isn't always right
https://bugs.webkit.org/show_bug.cgi?id=35431

Reviewed by Kenneth Rohde Christiansen.

WebCore:

The element that touch events are dispatched on is not always the
correct one, as the cached m_touchEventTarget member is only updated
when the first element of the PlatformTouchEvent touch list is in the
TouchPressed state.

This patch changes this behavior to dispatch the event to the target
of the touch that caused the event to be generated and eliminates the
m_touchEventTarget in favour of using the touch target hashmap. It also
simplifies the way in which the touchTargets list is generated as we
no longer have m_touchEventTarget (which was used previously to build
the list). The new behavior matches the observed behavior of the
iPhone and Android.

  • page/EventHandler.cpp:

(WebCore::assembleTargetTouches): Added. new function to build the

targetTouches list. Filters a list of touches (passed) about
another given touch.

(WebCore::EventHandler::handleTouchEvent): Remove the

m_touchEventTarget member, and simplify the generation of the
TouchLists that are bundled with the TouchEVent object. Dispatch
the event to the target of the touch that caused the event to be
fired.

  • page/EventHandler.h: Remove m_touchEventTarget.

LayoutTests:

  • fast/events/touch/basic-single-touch-events-expected.txt: Updated

the targetTouches list expections.

  • fast/events/touch/script-tests/basic-single-touch-events.js: ditto.

(verifyTouch):

08:51 Changeset [55286] by cjerdonek@webkit.org

Moved pywebsocket into the webkitpy/thirdparty directory and added
an associated README.webkit file to the pywebsocket directory.
This makes pywebsocket more consistent with the other third-party
Python code in our repository.

Reviewed by David Levin.

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

  • Scripts/run-webkit-tests:
    • Updated paths to pywebsocket.
  • Scripts/run-webkit-websocketserver:
    • Updated paths to pywebsocket.
  • Scripts/webkitpy/layout_tests/port/websocket_server.py:
    • Updated paths to pywebsocket.
  • Scripts/webkitpy/thirdparty/pywebsocket: Copied from WebKitTools/pywebsocket.
  • Scripts/webkitpy/thirdparty/pywebsocket/README.webkit: Added.
    • Added a file containing information about the contents of the pywebsocket directory. This will make it easier to understand where the third-party code came from and what local changes have been made.
  • pywebsocket: Removed.
  • pywebsocket/COPYING: Removed.
  • pywebsocket/MANIFEST.in: Removed.
  • pywebsocket/README: Removed.
  • pywebsocket/example: Removed.
  • pywebsocket/example/echo_client.py: Removed.
  • pywebsocket/example/echo_wsh.py: Removed.
  • pywebsocket/mod_pywebsocket: Removed.
  • pywebsocket/mod_pywebsocket/init.py: Removed.
  • pywebsocket/mod_pywebsocket/dispatch.py: Removed.
  • pywebsocket/mod_pywebsocket/handshake.py: Removed.
  • pywebsocket/mod_pywebsocket/headerparserhandler.py: Removed.
  • pywebsocket/mod_pywebsocket/memorizingfile.py: Removed.
  • pywebsocket/mod_pywebsocket/msgutil.py: Removed.
  • pywebsocket/mod_pywebsocket/standalone.py: Removed.
  • pywebsocket/mod_pywebsocket/util.py: Removed.
  • pywebsocket/setup.py: Removed.
  • pywebsocket/test: Removed.
  • pywebsocket/test/config.py: Removed.
  • pywebsocket/test/mock.py: Removed.
  • pywebsocket/test/run_all.py: Removed.
  • pywebsocket/test/test_dispatch.py: Removed.
  • pywebsocket/test/test_handshake.py: Removed.
  • pywebsocket/test/test_memorizingfile.py: Removed.
  • pywebsocket/test/test_mock.py: Removed.
  • pywebsocket/test/test_msgutil.py: Removed.
  • pywebsocket/test/test_util.py: Removed.
  • pywebsocket/test/testdata: Removed.
  • pywebsocket/test/testdata/handlers: Removed.
  • pywebsocket/test/testdata/handlers/blank_wsh.py: Removed.
  • pywebsocket/test/testdata/handlers/origin_check_wsh.py: Removed.
  • pywebsocket/test/testdata/handlers/sub: Removed.
  • pywebsocket/test/testdata/handlers/sub/exception_in_transfer_wsh.py: Removed.
  • pywebsocket/test/testdata/handlers/sub/no_wsh_at_the_end.py: Removed.
  • pywebsocket/test/testdata/handlers/sub/non_callable_wsh.py: Removed.
  • pywebsocket/test/testdata/handlers/sub/plain_wsh.py: Removed.
  • pywebsocket/test/testdata/handlers/sub/wrong_handshake_sig_wsh.py: Removed.
  • pywebsocket/test/testdata/handlers/sub/wrong_transfer_sig_wsh.py: Removed.
08:36 Changeset [55285] by eric@webkit.org

2010-02-26 Dirk Pranke <dpranke@chromium.org>

Reviewed by Darin Fisher.

Fix the handling of updates to #hash fragments to check for differences
post-canonicalization rather than comparing pre- to post-. Chromium
had a bug where we would set the #hash on an about:blank URL and generate
an onload event because we thought the URL (not just the fragment) was
changing.

http://bugs.webkit.org/show_bug.cgi?id=35180

  • fast/loader/about-blank-hash-change-expected.txt: Added.
  • fast/loader/about-blank-hash-change.html: Added.

2010-02-26 Dirk Pranke <dpranke@chromium.org>

Reviewed by Darin Fisher.

Fix the handling of updates to #hash fragments to check for differences
post-canonicalization rather than comparing pre- to post-. Chromium
had a bug where we would set the #hash on an about:blank URL and generate
an onload event because we thought the URL (not just the fragment) was
changing.

http://bugs.webkit.org/show_bug.cgi?id=35180

Test: fast/loader/about-blank-hash-change.html

  • bindings/v8/custom/V8LocationCustom.cpp: (WebCore::V8Location::hashAccessorSetter):
08:19 Changeset [55284] by eric@webkit.org

2010-02-26 Chris Jerdonek <cjerdonek@webkit.org>

Reviewed by Eric Seidel.

Changed the diff_parser module to log to a module-specific
logging.logger rather than the root logger.

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

  • Scripts/webkitpy/diff_parser.py:
07:48 Changeset [55283] by eric@webkit.org

2010-02-26 Zhenyao Mo <zmo@google.com>

Reviewed by David Levin.

texImage2D and texSubImage2D taking ImageData ignore flipY and premultiplyAlpha
https://bugs.webkit.org/show_bug.cgi?id=34459

  • fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data-expected.txt: Added.
  • fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data.html: Added.

2010-02-26 Zhenyao Mo <zmo@google.com>

Reviewed by David Levin.

texImage2D and texSubImage2D taking ImageData ignore flipY and premultiplyAlpha
https://bugs.webkit.org/show_bug.cgi?id=34459

Test: fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data.html

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::texImage2D): Apply flipY and premultiplyAlpha to the texture data. (WebCore::WebGLRenderingContext::texSubImage2D): Ditto.
  • platform/graphics/GraphicsContext3D.cpp: (WebCore::GraphicsContext3D::extractImageData): Extract data from ImageData, applying flipY and premultiplyAlpha.
  • platform/graphics/GraphicsContext3D.h: Add function extractImageData declaration.
07:30 Changeset [55282] by eric@webkit.org

2010-02-26 Zhenyao Mo <zmo@google.com>

Reviewed by David Levin.

Set viewport to canvas size upon context creation
https://bugs.webkit.org/show_bug.cgi?id=35057

  • fast/canvas/webgl/bug-32888.html: Remove the gl.viewport call since it's done when WebGLRenderingContext is created.
  • fast/canvas/webgl/gl-get-calls-expected.txt: The read-back viewport size should match the canvas size instead of (0, 0, 0, 0).
  • fast/canvas/webgl/gl-get-calls.html: Remove the gl.viewport call since it's done when WebGLRenderingContext is created.
  • fast/canvas/webgl/tex-image-and-sub-image-2d-with-image.html: Ditto.
  • fast/canvas/webgl/tex-sub-image-2d.html: Ditto.
  • fast/canvas/webgl/texImage2DImageDataTest.html: Ditto.
  • fast/canvas/webgl/triangle.html: Ditto.
  • fast/canvas/webgl/viewport-unchanged-upon-resize.html: Ditto.

2010-02-26 Zhenyao Mo <zmo@google.com>

Reviewed by David Levin.

Set viewport to canvas size upon context creation
https://bugs.webkit.org/show_bug.cgi?id=35057

Covered by existing tests, in particular fast/canvas/webgl/gl-get-calls.html

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::WebGLRenderingContext): Set the viewport size to match the canvas size.
06:56 Changeset [55281] by ossy@webkit.org

[Qt] fast/multicol/client-rects.html fails in QtLauncher
https://bugs.webkit.org/show_bug.cgi?id=34959

Reviewed by Kenneth Rohde Christiansen.

  • platform/qt/fast: Added.
  • platform/qt/fast/multicol: Added.
  • platform/qt/fast/multicol/client-rects-expected.txt: Platform specific expected file added.
  • platform/qt/Skipped: fast/multicol/client-rects.html unskipped.
06:49 Changeset [55280] by ossy@webkit.org

[Qt] LayoutTestController.numberOfPages() should have default parameters
https://bugs.webkit.org/show_bug.cgi?id=35428

Reviewed by Kenneth Rohde Christiansen.

WebKitTools:

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: maxViewWidth and maxViewHeight moved to

LayoutTestController to converge to platform independent implementation.
(WebCore::DumpRenderTree::DumpRenderTree):
(WebCore::DumpRenderTree::open):

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: Initialization of maxViewWidth and maxViewHeight added.
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
    • Default parameters for LayoutTestController.numberOfPages() added.
    • maxViewWidth and maxViewHeight members added.

LayoutTests:

  • platform/qt/Skipped: printing/css2.1 tests unskipped.
04:07 Changeset [55279] by philn@webkit.org

2010-02-26 Philippe Normand <pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

[GTK] plugins/setProperty.html fails on 64bit Release
https://bugs.webkit.org/show_bug.cgi?id=35425

  • platform/gtk/Skipped: Skipping another crasher related to bug 35425.
04:04 Changeset [55278] by pfeldman@chromium.org

2010-02-26 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: touch InspectorController to force front-end deployment.

  • inspector/InspectorController.cpp:
03:26 Changeset [55277] by pfeldman@chromium.org

2010-02-26 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInspector: While the current timeline view in DevTools provides a great overview of
things happening, we should make it easier to locate the cause of an event,
e.g., link to JS where relevant.
Caller info support for all kind of Timeline events and new Function Call event will be added.
JSC can be patched the same way as it will be done for V8.

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

  • bindings/js/ScriptCallStack.cpp: (WebCore::ScriptCallStack::callLocation):
  • bindings/js/ScriptCallStack.h:
  • bindings/v8/ScriptCallStack.cpp: (WebCore::ScriptCallStack::create): (WebCore::ScriptCallStack::callLocation):
  • bindings/v8/ScriptCallStack.h:
  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::callFunction):
  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::InspectorTimelineAgent): (WebCore::InspectorTimelineAgent::~InspectorTimelineAgent): (WebCore::InspectorTimelineAgent::willCallFunction): (WebCore::InspectorTimelineAgent::didCallFunction): (WebCore::InspectorTimelineAgent::pushCurrentRecord):
  • inspector/InspectorTimelineAgent.h: (WebCore::): (WebCore::InspectorTimelineAgent::instanceCount):
  • inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createGenericRecord): (WebCore::TimelineRecordFactory::createFunctionCallData):
  • inspector/TimelineRecordFactory.h:
  • inspector/front-end/Popover.js: (WebInspector.Popover.prototype.hideWhenClicked): (WebInspector.Popover.prototype._positionElement):
  • inspector/front-end/TimelineAgent.js:
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.prototype._formatRecord): (WebInspector.TimelinePanel.prototype._getRecordDetails): (WebInspector.TimelinePanel.prototype.reset): (WebInspector.TimelinePanel.prototype._closeRecordDetails): (WebInspector.TimelinePanel.prototype._onScroll): (WebInspector.TimelinePanel.prototype._refresh): (WebInspector.TimelinePanel.prototype._refreshRecords): (WebInspector.TimelineRecordListRow): (WebInspector.TimelineRecordListRow.prototype.update): (WebInspector.TimelineRecordListRow.prototype._createCell): (WebInspector.TimelineRecordListRow.prototype._createRow): (WebInspector.TimelineRecordListRow.prototype._createLinkRow): (WebInspector.TimelineRecordListRow.prototype._generateBubbleContent): (WebInspector.TimelineRecordListRow.prototype._onClick):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.js: (WebInspector.linkifyResourceAsNode):
03:04 Changeset [55276] by hamaji@chromium.org

2010-02-26 Shinichiro Hamaji <hamaji@chromium.org>

Unreviewed. Skip failing tests.

Refine LayoutTests/printing/css2.1 with numberOfPages
https://bugs.webkit.org/show_bug.cgi?id=35334

  • platform/qt/Skipped:
02:40 Changeset [55275] by hamaji@chromium.org

2010-02-26 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

Refine LayoutTests/printing/css2.1 with numberOfPages
https://bugs.webkit.org/show_bug.cgi?id=35334

Small refinements were done in paged-media-test-utils.js.

Note that WebKit doesn't pass page-break-after-003 for now.

  • printing/css2.1/README.txt:
  • printing/css2.1/page-break-after-000-expected.txt:
  • printing/css2.1/page-break-after-000.html:
  • printing/css2.1/page-break-after-001-expected.txt:
  • printing/css2.1/page-break-after-001.html:
  • printing/css2.1/page-break-after-002-expected.txt:
  • printing/css2.1/page-break-after-002.html:
  • printing/css2.1/page-break-after-003-expected.txt: Added.
  • printing/css2.1/page-break-after-003.html: Added.
  • printing/css2.1/page-break-after-004-expected.txt:
  • printing/css2.1/page-break-after-004.html:
  • printing/css2.1/page-break-before-000-expected.txt:
  • printing/css2.1/page-break-before-000.html:
  • printing/css2.1/page-break-before-001-expected.txt:
  • printing/css2.1/page-break-before-001.html:
  • printing/css2.1/page-break-before-002-expected.txt:
  • printing/css2.1/page-break-before-002.html:
  • printing/css2.1/page-break-inside-000-expected.txt:
  • printing/css2.1/page-break-inside-000.html:
  • printing/resources/paged-media-test-utils.js: (appendResult): (pageNumberForElementShouldBe): (numberOfPagesShouldBe): (runPrintingTest):
02:05 Changeset [55274] by philn@webkit.org

2010-02-26 Philippe Normand <pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

[GTK] plugins/setProperty.html fails on 64bit Release
https://bugs.webkit.org/show_bug.cgi?id=35425

  • platform/gtk/Skipped: Skipping crasher.
01:56 Changeset [55273] by eric@webkit.org

2010-02-26 Jamey Hicks <jamey.hicks@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] added QWebSettings::setInspectorUrl() and QWebSettings::inspectorUrl()

Enables the use of alternate Web Inspector frontends by changing
the location of the frontend.

This is required so that the Web Inspector may be run from an
external process or an external tool such as Eclipse or Aptana may
be used instead of the in-process Web Inspector UI.

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

  • Api/qwebsettings.cpp: (QWebSettings::QWebSettings): (QWebSettings::setInspectorUrl): (QWebSettings::inspectorUrl):
  • Api/qwebsettings.h:
  • WebCoreSupport/InspectorClientQt.cpp: (WebCore::InspectorClientQt::createPage):

2010-02-26 Jamey Hicks <jamey.hicks@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] added QWebSettings::setInspectorUrl() and QWebSettings::inspectorUrl()

Enables the use of alternate Web Inspector frontends by changing
the location of the frontend. The location is specified by providing

-inspector-url url

as an argument to QtLauncher.

This is required so that the Web Inspector may be run from an
external process or an external tool such as Eclipse or Aptana may
be used instead of the in-process Web Inspector UI.

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

  • QtLauncher/main.cpp: (LauncherWindow::init): (LauncherApplication::handleUserOptions):
01:28 BuildingQtOnSymbian edited by jocelyn.turcotte@nokia.com
(diff)
01:08 Changeset [55272] by philn@webkit.org

2010-02-26 Philippe Normand <pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

Adding two GTK layout test results for a new tests
added in r55259 and r55260.

  • platform/gtk/svg/custom/use-on-clip-path-with-transformation-expected.txt: Copied from LayoutTests/platform/mac/svg/custom/use-on-clip-path-with-transformation-expected.txt.
  • platform/gtk/svg/filters/filter-on-tspan-expected.txt: Added.
00:39 Changeset [55271] by ossy@webkit.org

Unreviewed. Roll-out r55263 because it broke fast/forms/textarea-type-spaces-pretty-diff.html.
https://bugs.webkit.org/show_bug.cgi?id=30946

WebCore:

  • editing/CompositeEditCommand.cpp:

(WebCore::isWhitespace):
(WebCore::CompositeEditCommand::rebalanceWhitespaceAt):

  • editing/InsertTextCommand.cpp:

(WebCore::InsertTextCommand::input):

  • editing/InsertTextCommand.h:
  • editing/htmlediting.cpp:
  • editing/htmlediting.h:

LayoutTests:

  • editing/execCommand/5142012-3-expected.txt:
  • editing/inserting/rebalance-whitespace-1-expected.txt: Removed.
  • editing/inserting/rebalance-whitespace-1.html: Removed.
  • editing/pasteboard/5521237-expected.txt:
  • platform/mac/editing/execCommand/5482023-expected.checksum:
  • platform/mac/editing/execCommand/5482023-expected.png: Added.
  • platform/mac/editing/execCommand/5482023-expected.txt:
  • platform/mac/editing/execCommand/remove-formatting-2-expected.checksum:
  • platform/mac/editing/execCommand/remove-formatting-2-expected.png: Added.
  • platform/mac/editing/execCommand/remove-formatting-expected.checksum:
  • platform/mac/editing/execCommand/remove-formatting-expected.png: Added.
  • platform/mac/editing/execCommand/remove-formatting-expected.txt:
  • platform/mac/editing/inserting/editable-html-element-expected.checksum:
  • platform/mac/editing/inserting/editable-html-element-expected.png: Added.
  • platform/mac/editing/inserting/editable-html-element-expected.txt:
  • platform/mac/editing/pasteboard/4989774-expected.checksum:
  • platform/mac/editing/pasteboard/4989774-expected.png: Added.
  • platform/mac/editing/pasteboard/4989774-expected.txt:
  • platform/mac/editing/selection/4983858-expected.checksum:
  • platform/mac/editing/selection/4983858-expected.png: Added.
  • platform/mac/editing/selection/4983858-expected.txt:

02/25/10:

23:40 Changeset [55270] by mrowe@apple.com

<rdar://problem/7688455> REGRESSION(r55205): Crash inside dispatchEventsOnWindowAndFocusedNode when clicking link from bookmarks view

Reviewed by Brady Eidson.

Null-check the document's page to avoid dereferencing a null page.

No regression test is added as this appears to be dependent on an interaction with Safari's bookmarks view that is
not testable from within DumpRenderTree.

  • page/FocusController.cpp:

(WebCore::dispatchEventsOnWindowAndFocusedNode):

22:49 Changeset [55269] by darin@chromium.org

2010-02-25 Darin Fisher <darin@chromium.org>

Reviewed by David Levin.

Remove deprecated cookie methods.
https://bugs.webkit.org/show_bug.cgi?id=35420

  • public/WebFrameClient.h:
  • public/WebKitClient.h:
  • src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::setCookies): (WebCore::ChromiumBridge::cookies): (WebCore::ChromiumBridge::cookieRequestHeaderFieldValue): (WebCore::ChromiumBridge::rawCookies): (WebCore::ChromiumBridge::deleteCookie): (WebCore::ChromiumBridge::cookiesEnabled):
20:56 QtWebKitMediaElementSupport edited by nicholas.young@nokia.com
Added information about QtMultimedia Integration (diff)
17:32 Changeset [55268] by dpranke@chromium.org

2010-02-25 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

Add a 'passing' port implementation to new-run-webkit-tests that
acts as a wrapper around an existing implementation but stubs out
the actual test invocations (instead, the expected results are echoed
back to the harness). This is useful for coverage and perf testing
of the harness (especially perf testing as it essentially provides
a lower bound on how fast the harness can run).

Also added a --nostart-helper flag to new-run-webkit-tests so that
you can skip starting the layout_test_helper and actually run the
harness even if you don't have a build of that port.

Also fix a bug in the 'test' port implementation to actually
create the results directory under /tmp instead of /.

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

  • Scripts/webkitpy/layout_tests/port/factory.py: Modified.
  • Scripts/webkitpy/layout_tests/port/passing.py: Added.
  • Scripts/webkitpy/layout_tests/port/test.py: Added.
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py: Modified.
17:14 Changeset [55267] by eric@webkit.org

2010-02-25 Eric Seidel <eric@webkit.org>

Fix typo in my last change. No review.

Rename run-chromium-webkit-tests to new-run-webkit-tests to stop confusion
https://bugs.webkit.org/show_bug.cgi?id=35408

  • Scripts/new-run-webkit-tests:
17:06 Changeset [55266] by oliver@apple.com

2010-02-25 Oliver Hunt <oliver@apple.com>

Reviewed by Simon Fraser.

Multiple repaints on apple.com
https://bugs.webkit.org/show_bug.cgi?id=35409

apple.com was triggering the fast scaling path for background images due to
repeated repaints as more content came in. This occured due to a two problems
in the logic to detect scaling. The first is that the main context is flipped
on mac so fails the identity or translation check. We work around this by adding
an function that allows the scaling for a flipped CTM. The other problem was that
we were looking at the destination rect size instead of the destination tile size
when deciding if the size we were drawn at would cause scaling.

  • platform/graphics/transforms/AffineTransform.h: (WebCore::AffineTransform::isIdentityOrTranslationOrFlipped):
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelScaleObserver::shouldPaintBackgroundAtLowQuality): (WebCore::RenderBoxModelObject::paintFillLayerExtended):
16:55 Changeset [55265] by dpranke@chromium.org

2010-02-25 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

Test whether or not we preserve the hash fragments for "about:blank"
URLs. (Note that currently WebKit passes this but Chromium doesn't).

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

  • fast/loader/about-blank-hash-kept-expected.txt: Added.
  • fast/loader/about-blank-hash-kept.html: Added.
16:49 Changeset [55264] by eric@webkit.org

2010-02-25 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Rename run-chromium-webkit-tests to new-run-webkit-tests to stop confusion
https://bugs.webkit.org/show_bug.cgi?id=35408

  • Scripts/new-run-webkit-tests: Added.
  • Scripts/run-chromium-webkit-tests: Removed.
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py: Renamed from WebKitTools/Scripts/webkitpy/layout_tests/run_chromium_webkit_tests.py.
16:43 Changeset [55263] by enrica@apple.com

Extra layout on keypress after a space (problem with rebalanceWhitespaceAt in InsertTextCommand).
https://bugs.webkit.org/show_bug.cgi?id=30946
<rdar://problem/7639184>

Reviewed by Adele Peterson.

WebCore:

Do text insertion and whitespace rebalancing around the insertion in one step so that there's only one layout.
This patch was originally made by Justin Garcia.

Test: editing/inserting/rebalance-whitespace-1.html

  • editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::rebalanceWhitespaceAt): No behavior changes here,
just pushed the code that determined the extent of whitespace surrounding a position
to its own helper function.

  • editing/InsertTextCommand.cpp:

(WebCore::InsertTextCommand::insertTextIntoNodeAndRebalanceWhitespace): Added. Find
whitespace surrounding the insertion position, add the text to insert, rebalance that entire
string, then insert it into the document.
(WebCore::InsertTextCommand::input):

  • editing/InsertTextCommand.h:
  • editing/htmlediting.cpp:

(WebCore::isWhitespace): Moved from CompositeEditCommand.cpp.
(WebCore::extentOfWhitespaceForRebalancingAt): Moved code from rebalanceWhitespaceAt into this helper
function. Obtains the offset of the start and end of whitespace around a particular position.

  • editing/htmlediting.h:

LayoutTests:

There are now regular spaces where nbsps were used unnecessarily before. Also,
multi-character insertions now have their whitespace rebalanced, and this is
reflected in a few tests.

  • editing/execCommand/5142012-3-expected.txt:
  • editing/inserting/rebalance-whitespace-1-expected.txt: Added.
  • editing/inserting/rebalance-whitespace-1.html: Added.
  • editing/pasteboard/5521237-expected.txt:
  • platform/mac/editing/execCommand/5482023-expected.checksum:
  • platform/mac/editing/execCommand/5482023-expected.png: Removed.
  • platform/mac/editing/execCommand/5482023-expected.txt:
  • platform/mac/editing/execCommand/remove-formatting-2-expected.checksum:
  • platform/mac/editing/execCommand/remove-formatting-2-expected.png: Removed.
  • platform/mac/editing/execCommand/remove-formatting-expected.checksum:
  • platform/mac/editing/execCommand/remove-formatting-expected.png: Removed.
  • platform/mac/editing/execCommand/remove-formatting-expected.txt:
  • platform/mac/editing/inserting/editable-html-element-expected.checksum:
  • platform/mac/editing/inserting/editable-html-element-expected.png: Removed.
  • platform/mac/editing/inserting/editable-html-element-expected.txt:
  • platform/mac/editing/pasteboard/4989774-expected.checksum:
  • platform/mac/editing/pasteboard/4989774-expected.png: Removed.
  • platform/mac/editing/pasteboard/4989774-expected.txt:
  • platform/mac/editing/selection/4983858-expected.checksum:
  • platform/mac/editing/selection/4983858-expected.png: Removed.
  • platform/mac/editing/selection/4983858-expected.txt:
16:22 Changeset [55262] by ap@apple.com

Reviewed by Anders Carlsson.

https://bugs.webkit.org/show_bug.cgi?id=35406
<rdar://problem/6945502> Make generic array methods work with JavaArray

Test: java/array-sort.html

Made RuntimeArray inherit from JSArray, keeping the promise given in ClassInfo.

  • bridge/runtime_array.cpp: (JSC::RuntimeArray::RuntimeArray): (JSC::RuntimeArray::~RuntimeArray):
  • bridge/runtime_array.h: (JSC::RuntimeArray::classInfo): (JSC::RuntimeArray::getConcreteArray):
15:21 Changeset [55261] by mitz@apple.com

Fixed a regression introduced in r44243, which made the assertion in checkListItems()
ineffective.

Reviewed by Alexey Proskuryakov.

  • dom/SelectElement.cpp:

(WebCore::SelectElementData::checkListItems): Copy the items vector before recomputing it,
then assert that the newly-computed vector is equal to the copy.

15:04 Changeset [55260] by krit@webkit.org

2010-02-25 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG's tspan is no member of SVGRenderBase, this can cause crashes on filters
https://bugs.webkit.org/show_bug.cgi?id=35354

This makes the base class RenderSVGInline of RenderSVGTSpan and RenderSVGInlineText
dependent on SVGBaseRenderer.
The SVG spec want as to use the object bounding box of the text root as the bounding box
for text-childs. So we search for the text root and use it's bounding box, stroke rect and
repaint rect for further calculations.

Test: svg/filters/filter-on-tspan.svg

  • rendering/RenderSVGInline.cpp: (WebCore::RenderSVGInline::objectBoundingBox): Find the text root and give back it's bounding box. (WebCore::RenderSVGInline::strokeBoundingBox): same for stroke rect (WebCore::RenderSVGInline::repaintRectInLocalCoordinates): same for repaint rect
  • rendering/RenderSVGInline.h: (WebCore::RenderSVGInline::toSVGRenderBase): RenderSVGInline is part of SVGRenderBase now.
  • rendering/RenderSVGTSpan.h: (WebCore::RenderSVGTSpan::renderName): Removed bounding box code. Was just a hack for filters and maskers.
  • rendering/SVGRenderSupport.cpp: (WebCore::findTextRootObject): Search for the text root.
  • rendering/SVGRenderSupport.h:
  • svg/graphics/SVGPaintServerGradient.cpp: moved findTextRootObject to SVGRenderSupport

2010-02-25 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG's tspan is no member of SVGRenderBase, this can cause crashes on filters
https://bugs.webkit.org/show_bug.cgi?id=35354

Check if filters work for tspan according to the SVG Spec. The test shouldn't
throw an assert now and a green rect should be visible.

  • platform/mac/svg/filters/filter-on-tspan-expected.checksum: Added.
  • platform/mac/svg/filters/filter-on-tspan-expected.png: Added.
  • platform/mac/svg/filters/filter-on-tspan-expected.txt: Added.
  • svg/filters/filter-on-tspan.svg: Added.
15:02 Changeset [55259] by krit@webkit.org

2010-02-25 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Use-element doesn't transform clipPath
https://bugs.webkit.org/show_bug.cgi?id=35375

If the use element is a child of clipPath, any settings on x, y or transform does not
transform the clipPath it references.
The use-element just misses this transformation in toClipPath.

Test: svg/custom/use-on-clip-path-with-transformation.svg

  • platform/graphics/cairo/PathCairo.cpp: Gtk port translated the path to the wrong direction. (WebCore::Path::translate):
  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::toClipPath):

2010-02-25 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Use-element doesn't transform clipPath
https://bugs.webkit.org/show_bug.cgi?id=35375

Check if the clipPath gets transformed on setting a transform or translation for use.

  • platform/mac/svg/custom/use-on-clip-path-with-transformation-expected.checksum: Added.
  • platform/mac/svg/custom/use-on-clip-path-with-transformation-expected.png: Added.
  • platform/mac/svg/custom/use-on-clip-path-with-transformation-expected.txt: Added.
  • svg/custom/use-on-clip-path-with-transformation.svg: Added.
14:52 Changeset [55258] by oliver@apple.com

2010-02-25 Oliver Hunt <oliver@apple.com>

Reviewed by Geoff Garen.

JSC crashes like crazy in the JSPropertyNameIterator destructor

Add back null check of m_cachedStructure. Curse last minute changes.

  • runtime/JSPropertyNameIterator.cpp: (JSC::JSPropertyNameIterator::~JSPropertyNameIterator):
14:30 Changeset [55257] by jianli@chromium.org

File.type support.
https://bugs.webkit.org/show_bug.cgi?id=35361

Reviewed by Dmitry Titov.

WebCore:

Test: LayoutTests/editing/pasteboard/file-input-files-access.html

  • html/File.cpp:

(WebCore::File::File):

  • html/File.h:

(WebCore::File::type):

  • html/File.idl:

LayoutTests:

Update the layout test to test File.type.

  • editing/pasteboard/file-input-files-access-expected.txt:
  • editing/pasteboard/script-tests/file-input-files-access.js:

(fileListShouldBe):
(runTest):

14:15 Changeset [55256] by oliver@apple.com

2010-02-25 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

Race condition in JSPropertyNameIterator and Structure destruction
https://bugs.webkit.org/show_bug.cgi?id=35398

JSPropertyNameIterator and Structure have a cyclic dependency that they
manage by clearing the appropriate reference in each other during their
destruction. However if the Structure is destroyed while the
JSPropertyNameIterator is dead but not yet finalized the Structures
WeakGCPtr will return null, and so prevent Structure from clearing
the m_cachedStructure pointer of the iterator. When the iterator is
then finalised the m_cachedStructure is invalid, and the attempt to
clear the structures back reference fails.

To fix this we simply make JSPropertyNameIterator keep the Structure
alive, using the weak pointer to break the ref cycle.

  • runtime/JSPropertyNameIterator.cpp: (JSC::JSPropertyNameIterator::~JSPropertyNameIterator): The iterator now keeps m_cachedStructure alive itself, so no longer needs to check for it being cleared
  • runtime/JSPropertyNameIterator.h: (JSC::JSPropertyNameIterator::setCachedStructure): Add an assertion to ensure correct usage (JSC::JSPropertyNameIterator::cachedStructure): Add .get()
  • runtime/Structure.cpp: (JSC::Structure::~Structure): Add an assertion that our iterator isn't already dead, and remove the now unnecessary attempt to clear the ref in the iterator
  • runtime/WeakGCPtr.h: (JSC::WeakGCPtr::hasDeadObject): An assert-only function to allow us to assert correct behaviour in the Structure destructor

2010-02-25 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

Race condition in JSPropertyNameIterator and Structure destruction
https://bugs.webkit.org/show_bug.cgi?id=35398

Add test to ensure that this race condition doesn't occur.

  • fast/js/script-tests/for-in-cached.js: (cacheClearing):
14:14 Changeset [55255] by eric@webkit.org

2010-02-25 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Javascript console should not attempt to scroll to end of console if console is not visible

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

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._scheduleScrollIntoView.scrollIntoView): (WebInspector.ConsoleView.prototype._scheduleScrollIntoView): (WebInspector.ConsoleView.prototype.addMessage):
13:56 Changeset [55254] by eric@webkit.org

2010-02-25 James Hawkins <jhawkins@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Implement WebLabelElement.

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

  • WebKit.gyp:
  • public/WebLabelElement.h: Added.
  • src/WebLabelElement.cpp: Added.
13:04 Changeset [55253] by ap@apple.com

Qt and Gtk build fixes.

  • GNUmakefile.am:
  • WebCore.pro: Added CRuntimeObject.{c,cpp}.
13:00 Changeset [55252] by ap@apple.com

Windows build fix.

  • WebCore.vcproj/WebCore.vcproj: Added CRuntimeObject.{c,cpp}.
12:56 Changeset [55251] by ap@apple.com

Tiger build fix.

  • java/Makefile: I can't get $(or ...) work on Tiger, so changed to use a single condition.
12:23 Changeset [55250] by ap@apple.com

Reviewed by Geoffrey Garen.

https://bugs.webkit.org/show_bug.cgi?id=35394
<rdar://problem/7685262> Make passing objects between Java and plug-ins work

  • WebCore.PluginHostProcess.exp: WebKit now subclasses RuntimeObject, so it needed more exports.
  • bridge/runtime_object.cpp: (JSC::Bindings::callRuntimeObject): (JSC::Bindings::callRuntimeConstructor): Assert that a runtime object is passed as function.
  • bridge/runtime_object.h: Moved RuntimeObject into Bindings namespace, matching other related classes.
  • bridge/jni/jni_jsobject.mm: (JavaJSObject::toString): Pass rootObject to convertValueToJValue(). It's not needed when constructing a string, but this function now takes it for the sake of Object. (JavaJSObject::convertValueToJObject): Check that object class is JavaRuntimeObject, not just RuntimeObject.
  • bridge/jni/jsc/JNIBridgeJSC.cpp: (JavaField::setValueToInstance): Pass rootObject to convertValueToJValue(). (JavaArray::setValueAt): Ditto.
  • bridge/jni/jsc/JNIUtilityPrivate.h: convertValueToJValue() now takes a RootObject argument, because one is needed to gcProtect an object ghtat is wrapped into JSObject.
  • bridge/jni/jsc/JNIUtilityPrivate.cpp: (JSC::Bindings::convertValueToJValue): Convert JavaScript objects to Java JSObject ones. This was already happening in other code paths, which we should change to use common code.
  • bridge/jni/jsc/JavaInstanceJSC.cpp: (JavaInstance::newRuntimeObject): Create an appropriate RuntimeObject subclass, which is JavaRuntimeObject for Java. (JavaInstance::invokeMethod): Unwrap returned JavaObjects that contain JS objects.
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/WebScriptObject.mm: (+[WebScriptObject _convertValueToObjcValue:originRootObject:rootObject:]):
  • bridge/c/CRuntimeObject.cpp: Added. (JSC::Bindings::CRuntimeObject::CRuntimeObject): (JSC::Bindings::CRuntimeObject::~CRuntimeObject): (JSC::Bindings::CRuntimeObject::getInternalCInstance):
  • bridge/c/CRuntimeObject.h: Added. (JSC::Bindings::CRuntimeObject::classInfo):
  • bridge/c/c_instance.cpp: (JSC::Bindings::CInstance::newRuntimeObject):
  • bridge/c/c_instance.h:
  • bridge/c/c_utility.cpp: (JSC::Bindings::convertValueToNPVariant):
  • bridge/jni/jsc/JavaInstanceJSC.h:
  • bridge/jni/jsc/JavaRuntimeObject.cpp: Added. (JSC::Bindings::): (JSC::Bindings::JavaRuntimeObject::JavaRuntimeObject): (JSC::Bindings::JavaRuntimeObject::~JavaRuntimeObject): (JSC::Bindings::JavaRuntimeObject::getInternalJavaInstance):
  • bridge/jni/jsc/JavaRuntimeObject.h: Added. (JSC::Bindings::JavaRuntimeObject::classInfo):
  • bridge/jsc/BridgeJSC.h:
  • bridge/objc/ObjCRuntimeObject.h: Added. (JSC::Bindings::ObjCRuntimeObject::classInfo):
  • bridge/objc/ObjCRuntimeObject.mm: Added. (JSC::Bindings::): (JSC::Bindings::ObjCRuntimeObject::ObjCRuntimeObject): (JSC::Bindings::ObjCRuntimeObject::~ObjCRuntimeObject): (JSC::Bindings::ObjCRuntimeObject::getInternalObjCInstance):
  • bridge/objc/objc_instance.h:
  • bridge/objc/objc_instance.mm: (ObjcInstance::newRuntimeObject):
  • bridge/objc/objc_runtime.mm: (JSC::Bindings::callObjCFallbackObject):
  • bridge/runtime_root.h: Added RuntimeObject subclasses for each instance type, and use them for type casting.
10:57 Changeset [55249] by kov@webkit.org

2010-02-25 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Xan Lopez.

Adopt the same string as the one that is expected by
fast/parser/fragment-parser.html for the description used by the
<isindex> element.

  • platform/gtk/LocalizedStringsGtk.cpp: (WebCore::searchableIndexIntroduction):
10:46 Changeset [55248] by pfeldman@chromium.org

2010-02-25 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: evaluate-on-hover does not work on HTML files.

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

  • inspector/front-end/SourceCSSTokenizer.re2js:
  • inspector/front-end/SourceHTMLTokenizer.js: (WebInspector.SourceHTMLTokenizer): (WebInspector.SourceHTMLTokenizer.prototype.set line): (WebInspector.SourceHTMLTokenizer.prototype._isExpectingAttribute): (WebInspector.SourceHTMLTokenizer.prototype._isExpectingAttributeValue): (WebInspector.SourceHTMLTokenizer.prototype._setExpectingAttribute): (WebInspector.SourceHTMLTokenizer.prototype._setExpectingAttributeValue): (WebInspector.SourceHTMLTokenizer.prototype._attrValueTokenType): (WebInspector.SourceHTMLTokenizer.prototype.nextToken):
  • inspector/front-end/SourceHTMLTokenizer.re2js:
  • inspector/front-end/SourceJavaScriptTokenizer.js: (WebInspector.SourceJavaScriptTokenizer):
  • inspector/front-end/SourceJavaScriptTokenizer.re2js:
  • inspector/front-end/SourceTokenizer.js: (WebInspector.SourceTokenizer.prototype.set condition): (WebInspector.SourceTokenizer.prototype.get condition): (WebInspector.SourceTokenizer.prototype.get subTokenizer): (WebInspector.SourceTokenizer.prototype.getLexCondition): (WebInspector.SourceTokenizer.prototype.setLexCondition):
  • inspector/front-end/TextEditorHighlighter.js: (WebInspector.TextEditorHighlighter): (WebInspector.TextEditorHighlighter.prototype.set mimeType): (WebInspector.TextEditorHighlighter.prototype.highlight): (WebInspector.TextEditorHighlighter.prototype._highlightInChunks): (WebInspector.TextEditorHighlighter.prototype._highlightLines): (WebInspector.TextEditorHighlighter.prototype._highlightLine):
  • inspector/front-end/TextViewer.js: (WebInspector.TextViewer.prototype._paintLine):
10:27 Changeset [55247] by jorlow@chromium.org

Make the context that was passed to the ThreadFunction accessible.
https://bugs.webkit.org/show_bug.cgi?id=35379

Patch by Jochen Eisinger <jochen@chromium.org> on 2010-02-25
Reviewed by Jeremy Orlow.

When a database is opened, right now you
don't have any context from where it is opened. The problem is that
the actual calls that open a database go through the sqlite3 vfs
layer, so there's no easy way to pass this function down to to
platform/sql/chromium/SQLFileSystemChromium*.cpp

This patch will allow you to get from anywhere within webkit a pointer
to the Thread object that actually created the thread you're currently
on (in case of the database, this can be either a thread forked of
from the main thread or from a worker thread), and query the object
for context information.

  • wtf/Threading.h:
  • wtf/ThreadingNone.cpp:

(WTF::threadContext):

  • wtf/ThreadingPthreads.cpp:

(WTF::):
(WTF::identifierByPthreadHandle):
(WTF::establishIdentifierForPthreadHandle):
(WTF::pthreadHandleForIdentifier):
(WTF::contextForIdentifier):
(WTF::createThreadInternal):
(WTF::currentThread):
(WTF::threadContext):

  • wtf/ThreadingWin.cpp:

(WTF::):
(WTF::threadMap):
(WTF::storeThreadHandleByIdentifier):
(WTF::threadHandleForIdentifier):
(WTF::contextForIdentifier):
(WTF::createThreadInternal):
(WTF::threadContext):

10:20 Changeset [55246] by jorlow@chromium.org

Reverting to re-submit with better change log.

  • wtf/Threading.h:
  • wtf/ThreadingNone.cpp:

(WTF::isMainThread):

  • wtf/ThreadingPthreads.cpp:

(WTF::identifierByPthreadHandle):
(WTF::establishIdentifierForPthreadHandle):
(WTF::pthreadHandleForIdentifier):
(WTF::createThreadInternal):
(WTF::currentThread):

  • wtf/ThreadingWin.cpp:

(WTF::threadMap):
(WTF::storeThreadHandleByIdentifier):
(WTF::threadHandleForIdentifier):
(WTF::createThreadInternal):

10:18 Changeset [55245] by kov@webkit.org

2010-02-25 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[Gtk] crashed when destroying
https://bugs.webkit.org/show_bug.cgi?id=31271

NULL-check the page before relaying the focus out event, since
this might happen when destroying the widget without destroying
its parent, and we currently crash.

  • tests/testwebview.c: (delayed_destroy): (test_webkit_web_view_destroy): (main):
  • webkit/webkitwebview.cpp: (webkit_web_view_focus_in_event):
10:16 Changeset [55244] by xan@webkit.org

2010-02-25 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Simple style cleanups.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::plugRemovedCallback): (WebCore::plugAddedCallback): (WebCore::PluginView::platformStart):
10:16 Changeset [55243] by abarth@webkit.org

2010-02-25 Adam Barth <abarth@webkit.org>

Reviewed by David Levin.

EWS leaks memory slowly
https://bugs.webkit.org/show_bug.cgi?id=35395

The EWS bots leak memory very slowly. If you run them for about a
month, each one will take up around 1 GB of virutal memory. If you run
several of them on one machine, you'll eventually exhaust all available
memory and grind the bots to a halt.

This patch introduces a --exit-after-iteration option to the queues so
that we run them for a finite amount of time. Once they exit and
restart, they'll reclaim the leaked memory. I'm not sure how many
iterations I'll end up running them for. I'll need to sort that out
operationally, but my initial guess is around 1000.

  • Scripts/webkitpy/commands/queues.py:
  • Scripts/webkitpy/commands/queues_unittest.py:
10:13 Changeset [55242] by eric@webkit.org

2010-02-25 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Darin Adler.

Optimize decoding of Latin-1 text by exploiting the fact that most of it will
be ASCII-only data.

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

  • platform/text/TextCodecLatin1.cpp: (WebCore::TextCodecLatin1::decode):
09:55 Changeset [55241] by pfeldman@chromium.org

2010-02-25 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Dimitri Glazkov.

Web Inspector: make script lines count calculation lazy.

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

  • inspector/front-end/Script.js: (WebInspector.Script): (WebInspector.Script.prototype.get linesCount):
  • inspector/front-end/ScriptsPanel.js:
  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._addBreakpoint):
09:20 Changeset [55240] by levin@chromium.org

Change hardcoded gcc paths to be Solaris friendly
https://bugs.webkit.org/show_bug.cgi?id=35213

Patch by James Choi <jchoi42@pha.jhu.edu> on 2010-02-25
Reviewed by David Levin.

  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/IDLParser.pm:
  • css/make-css-file-arrays.pl:
  • dom/make_names.pl:
09:13 Changeset [55239] by darin@chromium.org

2010-02-24 Darin Fisher <darin@chromium.org>

Reviewed by David Levin.

Add a missing WebString.h include.
https://bugs.webkit.org/show_bug.cgi?id=35360

This is required since some of the methods have implementations that
return a WebString.

  • public/WebCookieJar.h:
09:04 Changeset [55238] by xan@webkit.org

2010-02-25 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Workaround Java plugins not drawing themselves properly on first
expose by doing a late size-allocate after 'plug-added' has been
emitted on the embedding GtkSocket. It's unclear to me if this is
a bug in our side or theirs, but this should be pretty safe and
fixes the annoyance while we investigate it further.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::plugAddedCallback): (WebCore::PluginView::platformStart):
08:05 Changeset [55237] by jocelyn.turcotte@nokia.com

Add missing include guards

Reviewed by Tor Arne Vestbø.

  • loader/CrossOriginPreflightResultCache.h:
  • loader/MainResourceLoader.h:
08:04 Changeset [55236] by jocelyn.turcotte@nokia.com

[Qt] Restrict the optimization flags for Symbian to release builds.

Reviewed by Tor Arne Vestbø.

  • WebCore.pro:
07:06 Changeset [55235] by eric@webkit.org

2010-02-25 Jarkko Sakkinen <jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

Adds GraphicsContext3D QtWebKit implementation.
https://bugs.webkit.org/show_bug.cgi?id=35153

  • WebCore.pri:
  • WebCore.pro:
  • platform/graphics/GraphicsContext3D.h:
  • platform/graphics/qt/GraphicsContext3DQt.cpp: Added. (WebCore::GraphicsContext3DInternal::isValid): (WebCore::GraphicsContext3DInternal::GraphicsContext3DInternal): (WebCore::GraphicsContext3DInternal::~GraphicsContext3DInternal): (WebCore::GraphicsContext3DInternal::getProcAddress): (WebCore::GraphicsContext3D::create): (WebCore::GraphicsContext3D::GraphicsContext3D): (WebCore::GraphicsContext3D::~GraphicsContext3D): (WebCore::GraphicsContext3D::platformGraphicsContext3D): (WebCore::GraphicsContext3D::platformTexture): (WebCore::GraphicsContext3D::makeContextCurrent): (WebCore::GraphicsContext3D::beginPaint): (WebCore::GraphicsContext3D::endPaint): (WebCore::GraphicsContext3D::reshape): (WebCore::GraphicsContext3D::activeTexture): (WebCore::GraphicsContext3D::attachShader): (WebCore::GraphicsContext3D::bindAttribLocation): (WebCore::GraphicsContext3D::bindBuffer): (WebCore::GraphicsContext3D::bindFramebuffer): (WebCore::GraphicsContext3D::bindRenderbuffer): (WebCore::GraphicsContext3D::bindTexture): (WebCore::GraphicsContext3D::blendColor): (WebCore::GraphicsContext3D::blendEquation): (WebCore::GraphicsContext3D::blendEquationSeparate): (WebCore::GraphicsContext3D::blendFunc): (WebCore::GraphicsContext3D::blendFuncSeparate): (WebCore::GraphicsContext3D::bufferData): (WebCore::GraphicsContext3D::bufferSubData): (WebCore::GraphicsContext3D::checkFramebufferStatus): (WebCore::GraphicsContext3D::clearColor): (WebCore::GraphicsContext3D::clear): (WebCore::GraphicsContext3D::clearDepth): (WebCore::GraphicsContext3D::clearStencil): (WebCore::GraphicsContext3D::colorMask): (WebCore::GraphicsContext3D::compileShader): (WebCore::GraphicsContext3D::copyTexImage2D): (WebCore::GraphicsContext3D::copyTexSubImage2D): (WebCore::GraphicsContext3D::cullFace): (WebCore::GraphicsContext3D::depthFunc): (WebCore::GraphicsContext3D::depthMask): (WebCore::GraphicsContext3D::depthRange): (WebCore::GraphicsContext3D::detachShader): (WebCore::GraphicsContext3D::disable): (WebCore::GraphicsContext3D::disableVertexAttribArray): (WebCore::GraphicsContext3D::drawArrays): (WebCore::GraphicsContext3D::drawElements): (WebCore::GraphicsContext3D::enable): (WebCore::GraphicsContext3D::enableVertexAttribArray): (WebCore::GraphicsContext3D::finish): (WebCore::GraphicsContext3D::flush): (WebCore::GraphicsContext3D::framebufferRenderbuffer): (WebCore::GraphicsContext3D::framebufferTexture2D): (WebCore::GraphicsContext3D::frontFace): (WebCore::GraphicsContext3D::generateMipmap): (WebCore::GraphicsContext3D::getActiveAttrib): (WebCore::GraphicsContext3D::getActiveUniform): (WebCore::GraphicsContext3D::getAttribLocation): (WebCore::GraphicsContext3D::getContextAttributes): (WebCore::GraphicsContext3D::getError): (WebCore::GraphicsContext3D::getString): (WebCore::GraphicsContext3D::hint): (WebCore::GraphicsContext3D::isBuffer): (WebCore::GraphicsContext3D::isEnabled): (WebCore::GraphicsContext3D::isFramebuffer): (WebCore::GraphicsContext3D::isProgram): (WebCore::GraphicsContext3D::isRenderbuffer): (WebCore::GraphicsContext3D::isShader): (WebCore::GraphicsContext3D::isTexture): (WebCore::GraphicsContext3D::lineWidth): (WebCore::GraphicsContext3D::linkProgram): (WebCore::GraphicsContext3D::pixelStorei): (WebCore::GraphicsContext3D::polygonOffset): (WebCore::GraphicsContext3D::readPixels): (WebCore::GraphicsContext3D::releaseShaderCompiler): (WebCore::GraphicsContext3D::renderbufferStorage): (WebCore::GraphicsContext3D::sampleCoverage): (WebCore::GraphicsContext3D::scissor): (WebCore::GraphicsContext3D::shaderSource): (WebCore::GraphicsContext3D::stencilFunc): (WebCore::GraphicsContext3D::stencilFuncSeparate): (WebCore::GraphicsContext3D::stencilMask): (WebCore::GraphicsContext3D::stencilMaskSeparate): (WebCore::GraphicsContext3D::stencilOp): (WebCore::GraphicsContext3D::stencilOpSeparate): (WebCore::GraphicsContext3D::texParameterf): (WebCore::GraphicsContext3D::texParameteri): (WebCore::GraphicsContext3D::uniform1f): (WebCore::GraphicsContext3D::uniform1fv): (WebCore::GraphicsContext3D::uniform2f): (WebCore::GraphicsContext3D::uniform2fv): (WebCore::GraphicsContext3D::uniform3f): (WebCore::GraphicsContext3D::uniform3fv): (WebCore::GraphicsContext3D::uniform4f): (WebCore::GraphicsContext3D::uniform4fv): (WebCore::GraphicsContext3D::uniform1i): (WebCore::GraphicsContext3D::uniform1iv): (WebCore::GraphicsContext3D::uniform2i): (WebCore::GraphicsContext3D::uniform2iv): (WebCore::GraphicsContext3D::uniform3i): (WebCore::GraphicsContext3D::uniform3iv): (WebCore::GraphicsContext3D::uniform4i): (WebCore::GraphicsContext3D::uniform4iv): (WebCore::GraphicsContext3D::uniformMatrix2fv): (WebCore::GraphicsContext3D::uniformMatrix3fv): (WebCore::GraphicsContext3D::uniformMatrix4fv): (WebCore::GraphicsContext3D::useProgram): (WebCore::GraphicsContext3D::validateProgram): (WebCore::GraphicsContext3D::vertexAttrib1f): (WebCore::GraphicsContext3D::vertexAttrib1fv): (WebCore::GraphicsContext3D::vertexAttrib2f): (WebCore::GraphicsContext3D::vertexAttrib2fv): (WebCore::GraphicsContext3D::vertexAttrib3f): (WebCore::GraphicsContext3D::vertexAttrib3fv): (WebCore::GraphicsContext3D::vertexAttrib4f): (WebCore::GraphicsContext3D::vertexAttrib4fv): (WebCore::GraphicsContext3D::vertexAttribPointer): (WebCore::GraphicsContext3D::viewport): (WebCore::GraphicsContext3D::getBooleanv): (WebCore::GraphicsContext3D::getBufferParameteriv): (WebCore::GraphicsContext3D::getFloatv): (WebCore::GraphicsContext3D::getFramebufferAttachmentParameteriv): (WebCore::GraphicsContext3D::getIntegerv): (WebCore::GraphicsContext3D::getProgramiv): (WebCore::GraphicsContext3D::getProgramInfoLog): (WebCore::GraphicsContext3D::getRenderbufferParameteriv): (WebCore::GraphicsContext3D::getShaderiv): (WebCore::GraphicsContext3D::getShaderInfoLog): (WebCore::GraphicsContext3D::getShaderSource): (WebCore::GraphicsContext3D::getTexParameterfv): (WebCore::GraphicsContext3D::getTexParameteriv): (WebCore::GraphicsContext3D::getUniformfv): (WebCore::GraphicsContext3D::getUniformiv): (WebCore::GraphicsContext3D::getUniformLocation): (WebCore::GraphicsContext3D::getVertexAttribfv): (WebCore::GraphicsContext3D::getVertexAttribiv): (WebCore::GraphicsContext3D::getVertexAttribOffset): (WebCore::GraphicsContext3D::texImage2D): (WebCore::GraphicsContext3D::texSubImage2D): (WebCore::GraphicsContext3D::createBuffer): (WebCore::GraphicsContext3D::createFramebuffer): (WebCore::GraphicsContext3D::createProgram): (WebCore::GraphicsContext3D::createRenderbuffer): (WebCore::GraphicsContext3D::createShader): (WebCore::GraphicsContext3D::createTexture): (WebCore::GraphicsContext3D::deleteBuffer): (WebCore::GraphicsContext3D::deleteFramebuffer): (WebCore::GraphicsContext3D::deleteProgram): (WebCore::GraphicsContext3D::deleteRenderbuffer): (WebCore::GraphicsContext3D::deleteShader): (WebCore::GraphicsContext3D::deleteTexture): (WebCore::GraphicsContext3D::sizeInBytes): (WebCore::GraphicsContext3D::synthesizeGLError): (WebCore::GraphicsContext3D::getImageData):

2010-02-25 Jarkko Sakkinen <jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

Adds enabling and disabling of WebGL support to QWebSettings.
https://bugs.webkit.org/show_bug.cgi?id=35153

  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply): (QWebSettings::QWebSettings):
  • Api/qwebsettings.h:

2010-02-25 Jarkko Sakkinen <jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

Adds toggling of WebGL support to QtLauncher.
https://bugs.webkit.org/show_bug.cgi?id=35153

  • QtLauncher/main.cpp: (LauncherWindow::toggleWebGL): (LauncherWindow::setupUI):
06:52 BuildingQtOnSymbian edited by jocelyn.turcotte@nokia.com
(diff)
06:50 BuildingQtOnSymbian edited by jocelyn.turcotte@nokia.com
(diff)
06:49 Changeset [55234] by eric@webkit.org

2010-02-25 Jochen Eisinger <jochen@chromium.org>

Reviewed by Jeremy Orlow.

Make the context that was passed to the ThreadFunction accessible.
https://bugs.webkit.org/show_bug.cgi?id=35379

  • wtf/Threading.h:
  • wtf/ThreadingNone.cpp: (WTF::threadContext):
  • wtf/ThreadingPthreads.cpp: (WTF::): (WTF::identifierByPthreadHandle): (WTF::establishIdentifierForPthreadHandle): (WTF::pthreadHandleForIdentifier): (WTF::contextForIdentifier): (WTF::createThreadInternal): (WTF::currentThread): (WTF::threadContext):
  • wtf/ThreadingWin.cpp: (WTF::): (WTF::threadMap): (WTF::storeThreadHandleByIdentifier): (WTF::threadHandleForIdentifier): (WTF::contextForIdentifier): (WTF::createThreadInternal): (WTF::threadContext):
06:34 Changeset [55233] by pfeldman@chromium.org

2010-02-25 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Swap images for activate and deactivate breakpoints.

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

  • WebCore.gypi:
  • inspector/front-end/Images/breakpointsActivateButtonGlyph.png: Renamed from WebCore/inspector/front-end/Images/deactivateBreakpointsButtonGlyph.png.
  • inspector/front-end/Images/breakpointsDeactivateButtonGlyph.png: Renamed from WebCore/inspector/front-end/Images/deactivateBreakpointsDisabledButtonGlyph.png.
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
06:19 Changeset [55232] by kov@webkit.org

2010-02-25 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Go back on symlink resolution if we are loading the flash plugin,
and the path contains /netscape/, following what Chromium does.

Thanks to Evan Martin for the help on this!

  • plugins/gtk/PluginPackageGtk.cpp: (WebCore::PluginPackage::load):
05:45 Changeset [55231] by pfeldman@chromium.org

2010-02-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Scripts panel shows blank source when stopping on a breakpoint on refresh.

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

  • inspector/front-end/Panel.js: (WebInspector.Panel.prototype.canShowSourceLine): (WebInspector.Panel.prototype.showSourceLine):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.canShowSourceLine): (WebInspector.ResourcesPanel.prototype.showSourceLine):
  • inspector/front-end/Script.js: (WebInspector.Script):
  • inspector/front-end/ScriptView.js: (WebInspector.ScriptView.prototype.setupSourceFrameIfNeeded):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.show): (WebInspector.ScriptsPanel.prototype.get searchableViews): (WebInspector.ScriptsPanel.prototype.addScript): (WebInspector.ScriptsPanel.prototype._resourceLoadingFinished): (WebInspector.ScriptsPanel.prototype.addBreakpoint): (WebInspector.ScriptsPanel.prototype.removeBreakpoint): (WebInspector.ScriptsPanel.prototype.reset): (WebInspector.ScriptsPanel.prototype.canShowSourceLine): (WebInspector.ScriptsPanel.prototype.showSourceLine): (WebInspector.ScriptsPanel.prototype._scriptOrResourceForURLAndLine): (WebInspector.ScriptsPanel.prototype.showView): (WebInspector.ScriptsPanel.prototype._sourceFrameForScriptOrResource): (WebInspector.ScriptsPanel.prototype._showScriptOrResource): (WebInspector.ScriptsPanel.prototype._addScriptToFilesMenu.optionCompare): (WebInspector.ScriptsPanel.prototype._addScriptToFilesMenu):
  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._addBreakpoint):
  • inspector/front-end/inspector.js: (WebInspector.documentClick.followLink): (WebInspector.documentClick): (WebInspector._choosePanelToShowSourceLine): (WebInspector.canShowSourceLine): (WebInspector.showSourceLine):
05:24 Changeset [55230] by eric@webkit.org

2010-02-25 Ben Murdoch <benm@google.com>

Reviewed by Kenneth Rohde Christiansen.

The target element of a Touch should be the target where that touch originated, not where it is now.
https://bugs.webkit.org/show_bug.cgi?id=34585

  • fast/events/touch/basic-single-touch-events-expected.txt: Update expected target element.
  • fast/events/touch/script-tests/basic-single-touch-events.js: ditto.
  • fast/events/touch/script-tests/touch-target.js: Added.
  • fast/events/touch/touch-target-expected.txt: Added.
  • fast/events/touch/touch-target.html: Added.

2010-02-25 Ben Murdoch <benm@google.com>

Reviewed by Kenneth Rohde Christiansen.

The target element of a Touch should be the target where that touch originated, not where it is now.
https://bugs.webkit.org/show_bug.cgi?id=34585

Currently the target of a touch is set to the resulting node of the hit test where the touch currently
is. This does not match the behavior of iPhone or Android. This patch uses a hashmap on the EventHandler
to keep track of the target element when a touch is first started. This target is then used as the target
for subsequent touches with the same id. This matches observed behavior on iPhone and Android.

Tests:
fast/events/touch/touch-target.html: Added.
fast/events/touch/basic-single-touch-events.html: Updated.

  • page/EventHandler.cpp: (WebCore::EventHandler::handleTouchEvent): Store the originating target element of a touch in a hashmap

so that we can reuse that target for future events caused by that touch. This matches observed behavior
on iPhone and Android.

  • page/EventHandler.h: Add hashmap as a member.
  • platform/PlatformTouchPoint.h: (WebCore::PlatformTouchPoint::id): Store the touch point id as unsigned.
  • platform/qt/PlatformTouchPointQt.cpp: (WebCore::PlatformTouchPoint::PlatformTouchPoint): Cast platform touch id from signed to unsigned. Qt API

docs state that it will always be >= 0.

2010-02-25 Ben Murdoch <benm@google.com>

Reviewed by Kenneth Rohde Christiansen.

The target element of a Touch should be the target where that touch originated, not where it is now.
https://bugs.webkit.org/show_bug.cgi?id=34585

  • DumpRenderTree/qt/EventSenderQt.cpp: (EventSender::addTouchPoint): Fix a bug where touch points were not being given unique ids.
04:49 Changeset [55229] by xan@webkit.org

2010-02-25 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Unskip a bunch of (AFAICT) non-flakey HTTP tests that we currently
pass.

  • platform/gtk/Skipped:
03:48 Changeset [55228] by tonikitoo@webkit.org

Add a convenient center() method to IntRect
https://bugs.webkit.org/show_bug.cgi?id=35346

Reviewed by Simon Fraser.
Patch by Antonio Gomes <tonikitoo@webkit.org>

As it is a convenience method, and implemention is
based on FloatRect's one, patch does not provide a
layout test.

  • platform/graphics/IntRect.h:

(WebCore::IntRect::center):

03:24 Changeset [55227] by eric@webkit.org

2010-02-25 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Fake workers implementation that simulates workers using iframe and timer,
needed to support workers debugging. Also, a facility to inject scripts into
inspected page upon load.
https://bugs.webkit.org/show_bug.cgi?id=35148

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::addScriptToEvaluateOnLoad): (WebCore::InspectorBackend::removeAllScriptsToEvaluateOnLoad):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::addScriptToEvaluateOnLoad): (WebCore::InspectorController::removeAllScriptsToEvaluateOnLoad):
  • inspector/InspectorController.h:
  • inspector/front-end/InjectedFakeWorker.js: Added. (InjectedFakeWorker.Worker): (InjectedFakeWorker.FakeWorker): (InjectedFakeWorker.FakeWorker.prototype.postMessage): (InjectedFakeWorker.FakeWorker.prototype.terminate): (InjectedFakeWorker.FakeWorker.prototype._onmessageWrapper): (InjectedFakeWorker.FakeWorker.prototype._dispatchMessage): (InjectedFakeWorker.FakeWorker.prototype._handleException): (InjectedFakeWorker.FakeWorker.prototype._buildWorker): (InjectedFakeWorker.FakeWorker.prototype._setupWorkerContext.handler): (InjectedFakeWorker.FakeWorker.prototype._setupWorkerContext): (InjectedFakeWorker.FakeWorker.prototype._importScripts): (InjectedFakeWorker.FakeWorker.prototype._loadScript): (InjectedFakeWorker.URL): (InjectedFakeWorker.URL.prototype.urlRegEx.split): (InjectedFakeWorker.URL.prototype.mockLocation): (InjectedFakeWorker.URL.prototype.completeWith): (InjectedFakeWorker.URL.prototype.sameOrigin): (InjectedFakeWorker.DOMCoreException.formatError): (InjectedFakeWorker.DOMCoreException): (InjectedFakeWorker.noop):
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.setInjectedScriptSource): (.WebInspector.InspectorBackendStub.prototype.addScriptToEvaluateOnLoad): (.WebInspector.InspectorBackendStub.prototype.removeAllScriptsToEvaluateOnLoad):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
02:04 BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com
(diff)

02/24/10:

23:30 Changeset [55226] by eric@webkit.org

2010-02-24 Sam Kerner <skerner@chromium.org>

Reviewed by Darin Fisher.

Expose WebFrame::setCanHaveScrollbars(). This allows a view
which is being resized to not need scroll bars to ensure that
they are not drawn.

Existing function setAllowsScrolling() was renamed
setCanHaveScrollbars(), to be consistant with change 37159:
http://trac.webkit.org/changeset/37159

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

  • WebKit/chromium/public/WebFrame.h:
  • WebKit/chromium/src/ChromeClientImpl.cpp:
  • WebKit/chromium/src/WebFrameImpl.cpp:
  • WebKit/chromium/src/WebFrameImpl.h:
23:14 Changeset [55225] by eric@webkit.org

2010-02-24 Nicholas Young <nicholas.young@nokia.com>

Reviewed by Eric Carlson.

Add mediaPlayerOwningDocument() to MediaPlayerClient.
https://bugs.webkit.org/show_bug.cgi?id=35374

No new tests. These are interface changes only.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::mediaPlayerOwningDocument): Implementation for media element.
  • html/HTMLMediaElement.h:
  • platform/graphics/MediaPlayer.h: (WebCore::MediaPlayerClient::mediaPlayerOwningDocument): New virtual method.
22:56 Changeset [55224] by eric@webkit.org

2010-02-24 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Optimized Font::normalizeSpaces() by caching the String::characters()
instead of repeatedly calling operator[]

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

  • platform/graphics/Font.cpp: (WebCore::Font::normalizeSpaces):
22:32 Changeset [55223] by eric@webkit.org

2010-02-24 Jungshik Shin <jshin@chromium.org>

Reviewed by David Levin.

[Chromium] Traditional Chinese Chrome on Windows should use PMingLiu instead of Simsun for Han characters
https://bugs.webkit.org/show_bug.cgi?id=35319

No visible change in the layout test.

  • platform/graphics/chromium/FontUtilsChromiumWin.cpp: (WebCore::):
22:28 Changeset [55222] by mrobinson@webkit.org

2010-02-24 Krzysztof Kotlenga <pocek@users.sf.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] Creation of a WebkitWebView widget is very slow
https://bugs.webkit.org/show_bug.cgi?id=30032

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::ignoreWordInSpellDocument): Change 'langs' to 'dicts' to reflect the fact that the list just holds Enchant dictionaries now. (WebKit::EditorClient::learnWord): Ditto. (WebKit::EditorClient::checkSpellingOfString): Ditto. (WebKit::EditorClient::getGuessesForWord): Ditto.
  • webkit/webkitprivate.h: Remove the now unused SpellLanguage struct.
  • webkit/webkitwebsettings.cpp: Change function call to reflect new webkit_web_settings_get_enchant_dicts name. (get_enchant_broker): Add this method which returns the enchant broker singleton. (free_spell_checking_language): The list contents have changed, so change the way each element is freed. (webkit_web_settings_finalize): Change to reflect 'spell_checking_languages_list' to 'enchant_dicts' member name change. (webkit_web_settings_set_property): Use the broker singleton here instead of making a new one for each language. The Enchant dictionary is now the list payload. (webkit_web_settings_copy): More name-change updates. (webkit_web_settings_get_enchant_dicts): Ditto.
22:15 Changeset [55221] by eric@webkit.org

2010-02-24 Anthony Ricaud <rik@webkit.org>

Reviewed by Pavel Feldman.

Web Inspector: Reduce the timer to show the eval popover
https://bugs.webkit.org/show_bug.cgi?id=35344

  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._mouseMove):
21:40 Changeset [55220] by eric@webkit.org

2010-02-24 Chris Evans <cevans@chromium.org>

Reviewed by Darin Adler.

Add test for SVG that used to cause crash.
It has been fixed on trunk for a while, thanks to the <use> rewrite,
but we want to make sure we do not regress.

  • svg/dom/use-transform.svg: Added.
  • svg/dom/use-transform-expected.txt: Added.
21:24 Changeset [55219] by eric@webkit.org

2010-02-24 Jay Campan <jcampan@google.com>

Reviewed by David Levin.

Don't show the autofill popup when the input text is disabled or read only.

Test: manual-tests/chromium/no-autofill-on-readonly.html

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

  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::autofill):
  • manual-tests/chromium/no-autofill-on-readonly.html: Added.

2010-02-24 Jay Campan <jcampan@google.com>

Reviewed by David Levin.

Don't show the autofill popup when the input text is disabled or read only.

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

  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::autofill):
21:07 Changeset [55218] by eric@webkit.org

2010-02-24 Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QtLauncher has a border when running on QGraphicsView mode
https://bugs.webkit.org/show_bug.cgi?id=35352

Fix 2-pixels frame on each border of QtLauncher when running on QGraphicsView mode.

  • QtLauncher/webview.cpp: (WebViewGraphicsBased::WebViewGraphicsBased):
20:43 Changeset [55217] by eric@webkit.org

2010-02-24 Dominic Mazzoni <dmazzoni@chromium.org>

Reviewed by Jeremy Orlow.

Wraps includes of SVG headers so that it's possible to
compile the V8 bindings with SVG disabled, e.g. 'enable_svg=0'.
https://bugs.webkit.org/show_bug.cgi?id=35345

No new tests.

  • bindings/v8/V8DOMWrapper.cpp:
  • bindings/v8/V8Proxy.cpp:
  • bindings/v8/custom/V8CSSValueCustom.cpp:
  • bindings/v8/custom/V8DocumentCustom.cpp:
  • bindings/v8/custom/V8ElementCustom.cpp:
  • bindings/v8/custom/V8EventCustom.cpp:
  • bindings/v8/custom/V8SVGElementCustom.cpp:
20:15 Changeset [55216] by mjs@apple.com

Unreviewed build fix.

Fix gtk build.

Include JSC headers as runtime/ instead of JavaScriptCore/

  • bindings/js/JSDOMWrapper.h:
  • bindings/js/ScriptWrappable.h:
20:07 Changeset [55215] by mjs@apple.com

2010-02-24 Maciej Stachowiak <mjs@apple.com>

Reviewed by Oliver Hunt.

Cache JavaScript wrappers inline in DOM nodes
https://bugs.webkit.org/show_bug.cgi?id=35226

<rdar://problem/7664202>


7.4% speedup on Dromaeo DOM Core tests.
2.3% speedup on Hixie DOM Core tests.

This fixes the following things from the last attempt:

  • Now builds in both debug and release and passes all tests
  • Properly use a WeakGCPtr, not just a raw pointer, in ScriptWrappable
  • Properly look in the appropriate per-document or per-world map
  • Added an assert that would have caught any of the problems I found
  • Handle clearing the inline cache properly in all cases


  • bindings/js/JSDOMBinding.cpp: (WebCore::cacheDOMObjectWrapper): Adjust for name changes. (WebCore::forgetDOMNode): Clear wrapper pointer. (WebCore::cacheDOMNodeWrapper): Cache inline too if caching for normal world.
  • bindings/js/JSDOMBinding.h: (WebCore::Document::getWrapperCache): Inlined. (WebCore::domObjectWrapperMapFor): Renamed to start with lowercase. Moved to header to inline.
  • bindings/js/JSDOMWrapper.h: Added. (WebCore::DOMObject::DOMObject): Factored this out of JSDOMBinding.h to avoid include cycle. I think the class should be renamed, I picked a forward-looking header name because we already have a DOMObject.h
  • bindings/js/JSNodeCustom.cpp: (WebCore::createWrapperInline): Renamed version of original createWrapper. (WebCore::createWrapper): Call createWrapperInline. Out-of-line version. (WebCore::toJSNewlyCreated): Call createWrapperInline instead of createWrapper.
  • bindings/js/JSNodeCustom.h: Added. (WebCore::getCachedDOMNodeWrapper): Moved from JSDOMBinding.cpp and moved here, so it could inline everywhere without creating an include cycle. Consider inline cache. (WebCore::toJS): Moved to header to inline.
  • bindings/js/ScriptWrappable.h: (WebCore::ScriptWrappable::ScriptWrappable): Implement this in the obvious way for JavaScriptCore. (Using a WeakGCPtr). (WebCore::ScriptWrappable::wrapper): (WebCore::ScriptWrappable::setWrapper): (WebCore::ScriptWrappable::clearWrapper):
  • bindings/scripts/CodeGeneratorJS.pm: Include CustomHeader heaaders in the header, not just the impl file, so they can add inlining.
  • dom/Node.idl: Add CustomHeader directive.


Add new files to build.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
19:51 Changeset [55214] by eric@webkit.org

2010-02-24 Jochen Eisinger <jochen@chromium.org>

Reviewed by Jeremy Orlow.

Expose Database object of currently active task on the database thread
https://bugs.webkit.org/show_bug.cgi?id=35341

  • storage/DatabaseThread.cpp: (WebCore::DatabaseThread::DatabaseThread): (WebCore::DatabaseThread::databaseThread):
  • storage/DatabaseThread.h: (WebCore::DatabaseThread::getDatabaseOfCurrentTask):
19:34 Changeset [55213] by eric@webkit.org

2010-02-24 Nicholas Young <nicholas.young@nokia.com>

Reviewed by Eric Carlson.

supportsMuting() should be an internal interface between MediaPlayer and MediaPlayerPrivate.
https://bugs.webkit.org/show_bug.cgi?id=35327

No new tests. Refactoring Only.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::setMuted): Do not check supportsMuting() (WebCore::HTMLMediaElement::updateVolume): Set volume and muted
  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::setVolume): Check supportsMuting() (WebCore::MediaPlayer::setMuted): Check supportsMuting()
  • platform/graphics/MediaPlayer.h: Remove supportsMuting()
18:46 Changeset [55212] by mitz@apple.com

Rubber-stamped by Anders Carlsson.

Keep this test’s behavior the same after r55203.

  • fast/dom/script-tests/prototype-inheritance-2.js: Explicitly set the inner document’s body

display property to “none”. Before r55203, this happened as a side effect of parsing the
<frameset> tag.

17:09 Changeset [55211] by mrowe@apple.com

Versioning.

17:07 Changeset [55210] by dpranke@chromium.org

2010-02-23 Dirk Pranke <dpranke@chromium.org>

Reviewed by David Levin.

When the run-chromium-webkit-tests code was landed and the code was
refactored into the 'port' package, I accidentally broke using
http_server.py or websocket_server.py as command-line scripts
(the constructors needed a port object they weren't getting). This
change fixes them so that --server start|stop actually works.

As a part of this, the two files need to be able to call port.get(),
but doing that is awkward from a file inside the package, so I moved
get() into factory.py and imported that into init.py so that
http_server.py and websocket_server.py can just import factory.

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

  • Scripts/webkitpy/layout_tests/port/init.py:
  • Scripts/webkitpy/layout_tests/port/factory.py:
  • Scripts/webkitpy/layout_tests/port/http_server.py:
  • Scripts/webkitpy/layout_tests/port/websocket_server.py:
17:06 Changeset [55209] by mrowe@apple.com

New tag.

16:58 Changeset [55208] by dpranke@chromium.org

2010-02-24 Dirk Pranke <dpranke@chromium.org>

Reviewed by David Levin.

Fix the function signature for check_sys_deps on the mac port, and
fix the ordering of port_obj.check_sys_deps() and
port_obj.start_helper() (helper needs to be started before we check
the system configuration).

http://bugs.webkit.org/show_bug.cgi?id=35367

  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_chromium_webkit_tests.py:
16:24 Changeset [55207] by pkasting@chromium.org

[Chromium API] Disambiguate allowJavaScript from didNotAllowScript
https://bugs.webkit.org/show_bug.cgi?id=35205

Patch by Adam Barth <abarth@webkit.org> on 2010-02-24
Reviewed by Darin Fisher.

WebCore:

For clients that want to show a user interface element when JavaScript
was blocked on a page, we need to disambiguate between querying the
client for whether JavaScript is enabled from actually failing to
execute some script.

This patch adds a new FrameLoaderClient callback for when WebCore would
like to execute JavaScript but fails to because JavaScript is disabled.

This patch also touches every client of canExecuteScripts so they can
indicate whether we should make this callback. I was hoping there was
a better choke point, but my first two attempts were wrong in subtle
ways. pkasting points out that this will be easy to screw up in the
future, so it's better to make all the clients be explicit.

  • WebCore.PluginHostProcess.exp:
  • bindings/ScriptControllerBase.cpp:

(WebCore::ScriptController::canExecuteScripts):
(WebCore::ScriptController::executeScript):

  • bindings/js/JSEventListener.cpp:

(WebCore::JSEventListener::handleEvent):

  • bindings/js/JSLazyEventListener.cpp:

(WebCore::JSLazyEventListener::initializeJSFunction):

  • bindings/js/ScheduledAction.cpp:

(WebCore::ScheduledAction::execute):

  • bindings/js/ScriptController.cpp:

(WebCore::ScriptController::bindingRootObject):
(WebCore::ScriptController::windowScriptNPObject):
(WebCore::ScriptController::jsObjectForPluginElement):
(WebCore::ScriptController::executeScriptInWorld):

  • bindings/js/ScriptController.h:

(WebCore::):

  • bindings/js/ScriptControllerMac.mm:

(WebCore::ScriptController::windowScriptObject):

  • bindings/js/ScriptDebugServer.cpp:

(WebCore::ScriptDebugServer::setJavaScriptPaused):

  • bindings/js/ScriptEventListener.cpp:

(WebCore::createAttributeEventListener):

  • bindings/js/ScriptState.cpp:

(WebCore::scriptStateFromNode):

  • bindings/v8/ScriptController.cpp:

(WebCore::ScriptController::windowScriptNPObject):
(WebCore::ScriptController::createScriptObjectForPluginElement):

  • bindings/v8/ScriptController.h:

(WebCore::):

  • bindings/v8/ScriptEventListener.cpp:

(WebCore::createAttributeEventListener):

  • bindings/v8/V8Proxy.cpp:

(WebCore::V8Proxy::retrieve):

  • dom/ScriptElement.cpp:

(WebCore::ScriptElementData::evaluateScript):

  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::XMLTokenizer::startElementNs):

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::controls):

  • html/HTMLTokenizer.cpp:

(WebCore::HTMLTokenizer::parseTag):
(WebCore::HTMLTokenizer::processToken):

  • inspector/InspectorController.cpp:

(WebCore::canPassNodeToJavaScript):

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::isProcessingUserGesture):
(WebCore::FrameLoader::open):
(WebCore::FrameLoader::dispatchDidClearWindowObjectsInAllWorlds):
(WebCore::FrameLoader::dispatchDidClearWindowObjectInWorld):

  • loader/FrameLoaderClient.h:

(WebCore::FrameLoaderClient::didNotAllowScript):

WebKit/chromium:

Plumb didNotAllowScript through Chromium's WebKit API.

  • public/WebFrameClient.h:

(WebKit::WebFrameClient::didNotAllowScript):

  • src/DebuggerAgentImpl.cpp:

(WebKit::DebuggerAgentImpl::createUtilityContext):

  • src/FrameLoaderClientImpl.cpp:

(WebKit::FrameLoaderClientImpl::didNotAllowScript):

  • src/FrameLoaderClientImpl.h:
  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::bindToWindowObject):

WebKit/mac:

Make these two callsites explicit about not running script immediately.

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::getWindowNPObject):
(WebKit::NetscapePluginInstanceProxy::demarshalValueFromArray):

16:17 Changeset [55206] by pkasting@chromium.org

Add call to FrameLoaderClient::allowPlugins everywhere arePluginsEnabled is called
https://bugs.webkit.org/show_bug.cgi?id=34997

Patch by Adam Barth <abarth@webkit.org> on 2010-02-24
Reviewed by Darin Fisher.

If we want to let the FrameLoaderClient override arePluginsEnabled,
then we need to call out to the FrameLoaderClient every time we read
the setting.

We don't have testing infrustructure for these changes, which is lame.
I'm supposed to fix that in Bug 33991.

  • dom/DOMImplementation.cpp:

(WebCore::DOMImplementation::createDocument):

  • loader/MainResourceLoader.cpp:

(WebCore::MainResourceLoader::substituteMIMETypeFromPluginDatabase):

  • loader/PluginDocument.cpp:

(WebCore::PluginTokenizer::writeRawData):

  • page/Page.cpp:

(WebCore::Page::pluginData):

  • plugins/MimeType.cpp:

(WebCore::MimeType::enabledPlugin):

15:19 CommitQueue edited by rik@webkit.org
commiters.py is now in webkitpy (diff)
15:17 Changeset [55205] by jamesr@google.com

2010-02-24 James Robinson <jamesr@chromium.org>

Reviewed by Dmitry Titov.

Do not fire focus events while a modal dialog is up.
https://bugs.webkit.org/show_bug.cgi?id=33962

Modifies the FocusController to check the frame's page's defersLoading() flag before firing blur/focus events.
This flag is set while a modal dialog (like a window.alert or window.confirm) is up. Firing the events causes
assertion failures, since when the dialog is dismissed the PageGroupLoadDeferrer assumes that no script has run.

Manual tests only as DumpRenderTree does not support modal dialogs

  • manual-tests/modal-dialog-blur-selfclose.html: Added.
  • manual-tests/modal-dialog-blur.html: Added.
  • page/FocusController.cpp: (WebCore::dispatchEventsOnWindowAndFocusedNode):
14:58 Changeset [55204] by jamesr@google.com

2010-02-24 James Robinson <jamesr@chromium.org>

Reviewed by Eric Seidel.

Provide default username parameter to SVN.commit_with_message().

  • Scripts/webkitpy/scm.py:
14:48 Changeset [55203] by mitz@apple.com

<rdar://problem/7018611> innerHTML applies meta/link/title tags from a detached html element

Reviewed by Anders Carlsson.

WebCore:

Test: fast/parser/fragment-parser.html

Ensure that fragment parsing has no side effects on the fragment’s owner
document.

  • html/HTMLParser.cpp:

(WebCore::HTMLParser::insertNode): Don’t call
dispatchDocumentElementAvailable() for fragments.
(WebCore::HTMLParser::handleError): Don’t copy attributes to the owner
document’s <html> and <body> elements when a redundant <html> or <body>
is encountered while parsing a fragment.
(WebCore::HTMLParser::framesetCreateErrorCheck): Don’t change the owner
document’s <body> element’s style when parsing a fragment.
(WebCore::HTMLParser::createHead): Don’t attach the new <head> to the
ownder document of a fragment.

LayoutTests:

  • fast/parser/fragment-parser-expected.txt: Added.
  • fast/parser/fragment-parser.html: Added.
  • fast/parser/script-tests/fragment-parser.js: Added.
14:31 Changeset [55202] by dpranke@chromium.org

squash for checkin

14:09 Changeset [55201] by levin@chromium.org

Need to create a CanvasSurface base class for HTMLCanvasElement.
https://bugs.webkit.org/show_bug.cgi?id=35322

Reviewed by Darin Adler.

This is an initial step in making the OffscreenCanvas object.

No new functionality so no new tests.

  • GNUmakefile.am: Added CanvasSurface to the build.
  • WebCore.gypi: ditto
  • WebCore.pro: ditto
  • WebCore.vcproj/WebCore.vcproj: ditto
  • WebCore.xcodeproj/project.pbxproj: ditto
  • dom/CanvasSurface.cpp: Added.
  • dom/CanvasSurface.h: Added.
  • html/HTMLCanvasElement.h: Made HTMLCanvasElement inherit from CanvasSurface.
13:58 Changeset [55200] by ap@apple.com

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=35357
Two editing tests fail after DumpRenderTree run loop changes

AppKit decided that it wanted to group all editing commands for some reason (and thus undo
reverted them all at once).

  • DumpRenderTree/mac/DumpRenderTree.mm: (dump): Reverted the change that made DumpRenderTree use -[NSApplication run]. (runTest): Ditto. (-[DumpRenderTreeApplication isRunning]): Override isRunning with a function that always returns YES. This is another way to make the Java plug-in work.
13:52 Changeset [55199] by pkasting@chromium.org

Fix regression in calculating an animated image's start time.
https://bugs.webkit.org/show_bug.cgi?id=35115

Reviewed by Adam Barth.

  • platform/graphics/BitmapImage.cpp:

(WebCore::BitmapImage::startAnimation):

13:20 Changeset [55198] by oliver@apple.com

2010-02-24 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

[REGRESSION in r55185] EXC_BAD_ACCESS on opening inspector.
https://bugs.webkit.org/show_bug.cgi?id=35335

compileGetDirectOffset modifies the contents of the object register
when the object is not using the inline storage array. As the object
register contains our 'this' pointer we can't allow it to be clobbered.
The fix is simply to copy the register into a separate scratch register
when we're loading off an object that doesn't use inline storage.

  • jit/JITPropertyAccess.cpp: (JSC::JIT::privateCompileGetByIdSelfList):
  • jit/JITPropertyAccess32_64.cpp: (JSC::JIT::privateCompileGetByIdSelfList):

2010-02-24 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

[REGRESSION in r55185] EXC_BAD_ACCESS on opening inspector.
https://bugs.webkit.org/show_bug.cgi?id=35335

Add tests for caching getter lookup on large objects (eg. those not
using the inline storage array).

  • fast/js/pic/cached-getter-setter-expected.txt:
  • fast/js/pic/cached-getter-setter.html:
13:15 Changeset [55197] by xan@webkit.org

2010-02-24 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Unskip 3 passing cookies tests.

  • platform/gtk/Skipped:
12:51 Changeset [55196] by mitz@apple.com

<rdar://problem/7682827> Text with :first-letter lingers after being removed

Reviewed by Simon Fraser.

WebCore:

Test: fast/css/first-letter-set-text.html

  • rendering/RenderTextFragment.cpp:

(WebCore::RenderTextFragment::setTextInternal): Set this back as the
text node’s renderer, as removing the first letter has resets the node’s
renderer.

LayoutTests:

  • fast/css/first-letter-set-text-expected.checksum: Added.
  • fast/css/first-letter-set-text-expected.png: Added.
  • fast/css/first-letter-set-text-expected.txt: Added.
  • fast/css/first-letter-set-text.html: Added.
11:31 Changeset [55195] by philn@webkit.org

2010-02-24 Philippe Normand <pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

[GTK] DRT implement execCommand()
https://bugs.webkit.org/show_bug.cgi?id=35351

Skip test that fails in GTK+.

  • platform/gtk/Skipped:
10:15 Changeset [55194] by philn@webkit.org

2010-02-24 Philippe Normand <pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

Adding a GTK layout test result for a new test
added in r55159.

  • platform/gtk/fast/repaint/repaint-during-scroll-expected.txt: Added.
08:20 Changeset [55193] by ariya@webkit.org

2010-02-24 Ariya Hidayat <ariya.hidayat@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Faster cut-off for rectangle fill without shadow.
https://bugs.webkit.org/show_bug.cgi?id=35337

  • platform/graphics/qt/GraphicsContextQt.cpp:
07:39 QtWebKitTableOfFeatures46 edited by henry.haverinen@nokia.com
(diff)
07:05 Changeset [55192] by vestbo@webkit.org

[Qt] Remove QGVLauncher

Patch by Jesus Sanchez-Palencia <jesus.palencia@openbossa.org> on 2010-02-19
Reviewed by Kenneth Rohde Christiansen.

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

06:04 Changeset [55191] by xan@webkit.org

2010-02-24 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Enable SharedWorkers by default, since that's been the default for
a long time in our build-webkit configuration.

  • configure.ac:
06:01 Changeset [55190] by jocelyn.turcotte@nokia.com

[Qt] Corrects build break of QtLauncher on Windows.
The VERSION variable was not set anymore while building in trunk.

Reviewed by nobody, build fix.

  • WebCore.pro:
05:46 Changeset [55189] by xan@webkit.org

2010-02-24 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Get rid of an extra call to
setNPWindowIfNeeded. updatePluginWidget already calls this for us
if needed.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::platformStart):
05:43 Changeset [55188] by aroben@apple.com

2010-02-23 Adam Roben <aroben@apple.com>

Make commit-log-editor find just-added ChangeLog files

Fixes <http://webkit.org/b/35294> commit-log-editor doesn't find
just-added ChangeLog files

Reviewed by Dave Levin.

  • Scripts/commit-log-editor: (top level): Modified the regular expression that's used to find modified ChangeLog files to also look for just-added ChangeLog files.
05:21 Changeset [55187] by apavlov@chromium.org

2010-02-24 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Activate/Deactivate breaks does not look consistent with rest of the toolbar.

Fix style and add the new images to the related file sets.
https://bugs.webkit.org/show_bug.cgi?id=35307

  • WebCore.gypi:
  • inspector/front-end/Images/deactivateBreakpointsButtonGlyph.png:
  • inspector/front-end/Images/deactivateBreakpointsDisabledButtonGlyph.png:
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
04:44 Changeset [55186] by xan@webkit.org

2010-02-24 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Try to sanitize a bit the geometry management of plugins.

Stop doing contradictory things in updatePluginWidget (we were at
the same time pretending the geometry setting was delayed until
::paint and setting the geometry not once but twice) and just set
it any time the windowRect or clipRect for the plugin has changed.

This is closer to what the Mac port does, and fixes instances of
the plugins not being drawn until the window is resized or
scrolled. Other than that all manual and layout tests seems to
still work.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::updatePluginWidget): (WebCore::PluginView::setNPWindowIfNeeded):
04:20 BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com
(diff)
04:17 BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com
(diff)
01:02 Changeset [55185] by oliver@apple.com

2010-02-24 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

Speed up getter performance in the jit
https://bugs.webkit.org/show_bug.cgi?id=35332

Implement getter lookup caching in the interpreter.
The getter stubs are generated through basically the
same code paths as the normal get_by_id caching.
Instead of simply loading a property and returning,
we load the getter slot, and pass the getter, base value
and return address to a shared stub used for getter
dispatch.

  • jit/JIT.h: (JSC::JIT::compileGetByIdProto): (JSC::JIT::compileGetByIdSelfList): (JSC::JIT::compileGetByIdProtoList): (JSC::JIT::compileGetByIdChainList): (JSC::JIT::compileGetByIdChain):
  • jit/JITPropertyAccess.cpp: (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdSelfList): (JSC::JIT::privateCompileGetByIdProtoList): (JSC::JIT::privateCompileGetByIdChainList): (JSC::JIT::privateCompileGetByIdChain):
  • jit/JITPropertyAccess32_64.cpp: (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdSelfList): (JSC::JIT::privateCompileGetByIdProtoList): (JSC::JIT::privateCompileGetByIdChainList): (JSC::JIT::privateCompileGetByIdChain):
  • jit/JITStubs.cpp: (JSC::JITThunks::tryCacheGetByID): (JSC::DEFINE_STUB_FUNCTION):
  • jit/JITStubs.h: (JSC::):
  • runtime/GetterSetter.h:
00:51 Changeset [55184] by hamaji@chromium.org

2010-02-24 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Darin Adler.

check-webkit-style false positive for WebCore forwarding header
https://bugs.webkit.org/show_bug.cgi?id=34604

  • Scripts/webkitpy/style/checker.py:
  • Scripts/webkitpy/style/checker_unittest.py:

02/23/10:

23:40 Changeset [55183] by ossy@webkit.org

Rubber-stamped by Simon Hausmann.

[Qt] editing/execCommand/delete-image-in-anchor.html skipped
because of missing layoutTestController.execCommand()

Qt specific expected file added for editing/execCommand/insert-list-xml.xhtml
introduced in r55167, because of different font types.

  • platform/qt/Skipped: editing/execCommand/delete-image-in-anchor.html skipped.
  • platform/qt/editing/execCommand/insert-list-xml-expected.txt: Added.
23:14 Changeset [55182] by oliver@apple.com

2010-02-23 Geoff Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Simplify animation lifetime handling. Previously we manually
determined whether our base element was safe when we unregistered
our listener, now we simply ensure that the base element itself
registers and unregisters the animation listener.

  • svg/animation/SVGSMILElement.cpp: (WebCore::ConditionEventListener::create): (WebCore::ConditionEventListener::disconnectAnimation): (WebCore::ConditionEventListener::ConditionEventListener): (WebCore::ConditionEventListener::operator==): (WebCore::ConditionEventListener::handleEvent): (WebCore::SVGSMILElement::eventBaseFor): (WebCore::SVGSMILElement::connectConditions): (WebCore::SVGSMILElement::disconnectConditions):
  • svg/animation/SVGSMILElement.h:
20:27 Changeset [55181] by oliver@apple.com

2010-02-23 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

Web Inspector: Regression: r55027+: Inspector broken
https://bugs.webkit.org/show_bug.cgi?id=35253

op_get_by_id_getter_chain was not passing the correct this parameter.
The bug was caused by incorrect use of baseCell instead of baseValue,
baseValue contains the original object for the lookup (and hence the
correct this object), baseCell is clobbered as part of walking the
prototype chain.

  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):

2010-02-23 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

Web Inspector: Regression: r55027+: Inspector broken
https://bugs.webkit.org/show_bug.cgi?id=35253

Add thorough testing of getter access to hit the various
possible caching scenarios.

  • fast/js/pic/cached-getter-setter-expected.txt:
  • fast/js/pic/cached-getter-setter.html:
19:43 Google Summer of Code 2010 edited by pkasting@google.com
(diff)
18:05 Changeset [55180] by dimich@chromium.org

Chromium: Use V8::TerminateExecution to actually terminate workers.
https://bugs.webkit.org/show_bug.cgi?id=35137

Reviewed by David Levin.

Test: existing fast/workers/stress-js-execution.html which is currently failing
on Mac and Linux bots for Chromium.

  • bindings/v8/WorkerContextExecutionProxy.cpp:

(WebCore::WorkerContextExecutionProxy::evaluate):

  • bindings/v8/WorkerScriptController.cpp:

(WebCore::WorkerScriptController::forbidExecution):

17:53 Changeset [55179] by enrica@apple.com

REGRESSION: WebKit crashes when deleting images on blogger.com (34957)
<rdar://problem/7651935>
https://bugs.webkit.org/show_bug.cgi?id=34957

Reviewed by Simon Fraser.

WebCore:

Fixing a regression introduced with revision 53085. Anchor elements should not be considered
in editable content when calculating the position in the parent before the node.

Test: editing/execCommand/delete-image-in-anchor.html

  • dom/Position.cpp:

(WebCore::Position::getInlineBoxAndOffset):

LayoutTests:

  • editing/execCommand/delete-image-in-anchor-expected.txt: Added.
  • editing/execCommand/delete-image-in-anchor.html: Added.
17:46 Changeset [55178] by tony@chromium.org

2010-02-23 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=34634
Pasting a list into the middle of another list item should split the target
list item into two separate list items. This matches the behavior in other
browsers.

  • editing/pasteboard/paste-list-004-expected.txt: Added.
  • editing/pasteboard/paste-list-004.html: Added.

2010-02-23 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=34634
Pasting a list into the middle of another list item should split the target
list item into two separate list items. This matches the behavior in other
browsers.

Test: editing/pasteboard/paste-list-004.html

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::insertAsListItems):
17:22 Changeset [55177] by ap@apple.com

Reviewed by Mark Rowe.

https://bugs.webkit.org/show_bug.cgi?id=22602
Enable Java in DumpRenderTree (on Mac)

LayoutTests:

  • fast/dom/java-applet-calls-expected.txt: Removed.
  • fast/dom/java-applet-calls.html: Removed.
  • fast/dom/resources/TestApplet.class: Removed.
  • fast/dom/resources/TestApplet.java: Removed. This test was added in a failing form, even though it never worked. The new tests cover everything it was meant to cover.
  • java/SharedApplet.java: Added. This class will have methods for simple tests, but it's also possible to just add new source files.
  • java/Makefile: Added. Compile all .java files in the directory.
  • java: Added.
  • java/argument-to-object-type-expected.txt: Added.
  • java/argument-to-object-type.html: Added.
  • java/array-return-expected.txt: Added.
  • java/array-return.html: Added. Added tests for recently fixed bugs.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped: Java tests only work on Mac currently.

WebKitTools:

  • DumpRenderTree/mac/DumpRenderTree.mm: (exitApplicationRunLoop): (dump): (runTest): DumpRenderTree now runs an NSApplication, not just an event loop. This way, the Java plug-in can start without freezing.
  • Scripts/run-webkit-tests: Compile java sources in LayputTests/java.
16:56 Changeset [55176] by oliver@apple.com

2010-02-23 Oliver Hunt <oliver@apple.com>

Add expected result

  • svg/custom/animate-use-crash-expected.txt: Added.
16:49 Changeset [55175] by oliver@apple.com

2010-02-23 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

Add a testcase for an old bug to ensure it doesn't return.

  • svg/custom/animate-use-crash.xhtml: Added.
15:34 Google Summer of Code 2010 edited by jeremy@chromium.org
(diff)
15:01 Google Summer of Code 2010 created by jeremy@chromium.org
GSoC 2010 page - in case we want to participate
14:59 Changeset [55174] by eric@webkit.org

2010-02-23 James Choi <jchoi42@pha.jhu.edu>

Add Solaris definitions
https://bugs.webkit.org/show_bug.cgi?id=35214

  • WebKit/chromium/src/WebViewImpl.cpp
  • WebKit/chromium/src/WebFrameImpl.cpp
14:43 Changeset [55173] by eric@webkit.org

2010-02-23 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk]: testwebview does not work when called with absolute path
https://bugs.webkit.org/show_bug.cgi?id=34940

When testwebview is called as absolute path, chdir to executable
directory before searching resource files.

  • GNUmakefile.am:
14:39 Changeset [55172] by beidson@apple.com

Regression (r55107) - WebInspector docking is busted.
https://bugs.webkit.org/show_bug.cgi?id=35274

Reviewed by Tim Hatcher and Pavel Feldman.

WebCore:

  • inspector/InspectorController.cpp:

(WebCore::InspectorController::canAttachWindow): Use the minimum height for this calculation,

not the preferred height.

(WebCore::InspectorController::attachWindow): attachWindow() shouldn't decide the policy, as it is

often called partway through the show or dock procedure after some of the initial conditions have
changed. Let the front-end itself and the InspectorClient's make the policy decision at the start
of the show/dock operation.

  • inspector/InspectorFrontendHost.cpp:

(WebCore::InspectorFrontendHost::canAttachWindow):

  • inspector/InspectorFrontendHost.h:
  • inspector/InspectorFrontendHost.idl:
  • inspector/front-end/inspector.js:

(WebInspector.toggleAttach): Before attaching, ask the front-end-host if attaching is allowed.

WebKit/mac:

  • WebCoreSupport/WebInspectorClient.mm:

(-[WebInspectorWindowController showWindow:]): Swap the order of the "should attach?" check

to get the expected behavior.

WebKit/win:

  • WebCoreSupport/WebInspectorClient.cpp:

(WebInspectorClient::showWindowWithoutNotifications): Swap the order of the "should attach?" check

to get the expected behavior.

LayoutTests:

  • inspector/timeline-event-dispatch.html: Actually click in the right place now that the inspector isn't docked anymore.
14:17 Changeset [55171] by eric@webkit.org

2010-02-23 James Hawkins <jhawkins@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Implement WebDocument::getElementsByTagName.

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

  • public/WebDocument.h:
  • src/WebDocument.cpp: (WebKit::WebDocument::getElementsByTagName):
14:03 Changeset [55170] by jberlin@webkit.org

2010-02-23 Jessie Berlin <jberlin@webkit.org>

Reviewed by Eric Seidel and Adam Roben.

Debugging WebKit on Windows should mention WebKit.sln as an alternative to running debug-safari.
https://bugs.webkit.org/show_bug.cgi?id=35098

  • building/debug.html: Add instructions on how to build and run WebKit from within Visual Studio.
14:01 Changeset [55169] by mrowe@apple.com

<http://webkit.org/b/35287> ImageSourceCG::frameIsCompleteAtIndex returns false for all frames until image has completed loading

Reviewed by Geoff Garen.

CGImageSourceGetStatusAtIndex claims that all frames of a multi-frame image are incomplete when we've not yet received the
complete data for an image that is using an incremental data source (<rdar://problem/7679174>). We work around this by
special-casing all frames except the last in an image and treating them as complete if they are present and reported as
being incomplete. We do this on the assumption that loading new data can only modify the existing last frame or append new
frames. The last frame is only treated as being complete if the image source reports it as such. This ensures that it is
truly the last frame of the image rather than just the last that we currently have data for.

  • platform/graphics/cg/ImageSourceCG.cpp:

(WebCore::ImageSource::frameIsCompleteAtIndex):

13:31 Changeset [55168] by kov@webkit.org

2010-02-23 Mario Blättermann <mariobl@freenet.de>

Reviewed by Gustavo Noronha.

German translation update.

  • de.po:
13:31 Changeset [55167] by enrica@apple.com

WebCore: Crash in createContextualFragment when inserting a list in a non HTML document.
<rdar://problem/7666670>
https://bugs.webkit.org/show_bug.cgi?id=35305

Reviewed by Darin Adler.

createFragmentFromMarkup did not handle correctly the case where documentElement is
not an HTMLElement. The static cast to HTMLElement was causing createContextualFragment to
crash.

Test: editing/execCommand/insert-list-xml.xhtml

  • dom/Element.cpp:

(WebCore::Element::createContextualFragment): Added.

  • dom/Element.h: Added createContextualFragment virtual function.
  • editing/markup.cpp:

(WebCore::createFragmentFromMarkup): Removed static cast to HTMLElement.

  • html/HTMLElement.cpp:

(WebCore::HTMLElement::createContextualFragment): Modified to perform only checks
that are specific for an HTMLElement object. The rest of the logic has been moved to the
corresponding method in the Element class.

LayoutTests: Crash in createContextualFragment when inserting a list in a non HTML document.
<rdar://problem/7666670>
https://bugs.webkit.org/show_bug.cgi?id=35305

Reviewed by Darin Adler.

  • editing/execCommand/insert-list-xml-expected.txt: Added.
  • editing/execCommand/insert-list-xml.xhtml: Added.
13:25 Changeset [55166] by kov@webkit.org

2010-02-23 Daniel Nylander <po@danielnylander.se>

Reviewed by Gustavo Noronha.

Swedish translation update.

  • sv.po:
13:20 Changeset [55165] by kov@webkit.org

2010-02-23 Ankit Patel <ankit@redhat.com>

Reviewed by Gustavo Noronha.

Gujarati translation.

  • gu.po: Added.
13:13 Changeset [55164] by eric@webkit.org

2010-02-23 Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Need a final integration between QtLauncher and QGVLauncher
https://bugs.webkit.org/show_bug.cgi?id=35292

WebKit coding style fixes.

  • QtLauncher/main.cpp: (LauncherWindow::resetZoom):
12:39 Changeset [55163] by eric@webkit.org

2010-02-23 Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Need a final integration between QtLauncher and QGVLauncher
https://bugs.webkit.org/show_bug.cgi?id=35292

Add cloneWindow feature to QtLauncher, when running on QGraphicsView mode.

  • QtLauncher/main.cpp: (LauncherWindow::LauncherWindow): (LauncherWindow::~LauncherWindow): (LauncherWindow::init): (LauncherWindow::cloneWindow): (LauncherWindow::setupUI):
12:07 Changeset [55162] by sfalken@apple.com

2010-02-23 Steve Falkenburg <sfalken@apple.com>

Reviewed by Adam Roben.

WebCore should stop throwing away the CGImageSourceRef on Windows to improve GIF performance
https://bugs.webkit.org/show_bug.cgi?id=35309

  • platform/graphics/cg/ImageSourceCG.cpp: (WebCore::ImageSource::clear):
10:15 Changeset [55161] by dpranke@chromium.org

fix check_wdiff_install

10:07 Changeset [55160] by kov@webkit.org

2010-02-23 Gustavo Noronha Silva <gns@gnome.org>

Rubber-stamped by Dimitri Glazkov.

Chromium build fix.

09:54 Changeset [55159] by mitz@apple.com

WebKit/mac: <rdar://problem/7611158> Incomplete repaint of YouTube timeline thumb while scrolling
https://bugs.webkit.org/show_bug.cgi?id=34381

Reviewed by Simon Fraser.

Test: fast/repaint/repaint-during-scroll.html

NSClipView offsets any rects marked as needing display during scrolling
by the scroll offset. Compensate for this when -setNeedsDisplay: is called
during scrolling.

  • WebView/WebHTMLView.mm:

(-[WebHTMLView _frameOrBoundsChanged]): Set inScrollPositionChanged to YES
around to call to FrameView::scrollPositionChanged().
(-[WebHTMLView setNeedsDisplayInRect:]): When called beneath
scrollPositionChanged(), adjust the rect by the inverse of the scroll offset.

LayoutTests: Rubber-stamped by Simon Fraser.

<rdar://problem/7611158> Incomplete repaint of YouTube timeline thumb while scrolling
https://bugs.webkit.org/show_bug.cgi?id=34381

  • fast/repaint/repaint-during-scroll.html: Added.
  • platform/mac/fast/repaint/repaint-during-scroll-expected.checksum: Added.
  • platform/mac/fast/repaint/repaint-during-scroll-expected.png: Added.
  • platform/mac/fast/repaint/repaint-during-scroll-expected.txt: Added.
09:52 Changeset [55158] by kov@webkit.org

2010-02-23 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Gustavo Noronha Silva.

Changes references of GOwnPtr to reflect their new place.
http://webkit.org/b/35084

09:41 Changeset [55157] by beidson@apple.com

HTTP 307 after a 303 after a POST re-sends POST data from the original request.
<rdar://problem/7390251> and https://bugs.webkit.org/show_bug.cgi?id=31410

Reviewed by Alexey Proskuryakov.

WebCore:

Test: http/tests/loading/307-after-303-after-post.html

Remember the last HTTP method send out during a redirect chain for a resource handle:

  • platform/network/ResourceHandle.cpp:

(WebCore::ResourceHandle::lastHTTPMethod):

  • platform/network/ResourceHandle.h:
  • platform/network/ResourceHandleInternal.h:

(WebCore::ResourceHandleInternal::ResourceHandleInternal):

Refer to the last HTTP method used instead of the original method:

  • platform/network/cf/ResourceHandleCFNet.cpp:

(WebCore::willSendRequest):
(WebCore::ResourceHandle::willSendRequest):

  • platform/network/mac/ResourceHandleMac.mm:

(WebCore::ResourceHandle::willSendRequest):
(-[WebCoreResourceHandleAsDelegate connection:willSendRequest:redirectResponse:]):

LayoutTests:

  • http/tests/loading/307-after-303-after-post.html: Added.
  • http/tests/loading/resources/303-to-307-target.php: Added.
  • http/tests/loading/resources/307-post-output-target.php: Added.
  • http/tests/loading/resources/post-to-303-target.php: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
09:27 Changeset [55156] by dbates@webkit.org

2010-02-23 Crystal Zhang <haizhang@rim.com>

Unreviewed build fix.

Fix build error when enable plugin proxy: 'toRenderEmbeddedObject' identifier not found.
As RenderEmbeddedObject inherits RenderPartObject and 'toRenderEmbeddedObject' belongs
to former one.

  • html/HTMLMediaElement.cpp:
08:53 Changeset [55155] by eric@webkit.org

2010-02-23 Evan Martin <evan@chromium.org>

Reviewed by Jeremy Orlow.

[chromium] fix previous SVG-disabling patch
https://bugs.webkit.org/show_bug.cgi?id=35298

  • WebCore.gypi: rename variable
  • WebCore.gyp/WebCore.gyp: actually use the variable
08:32 Changeset [55154] by eric@webkit.org

2010-02-23 Michael Lotz <mmlr@mlotz.ch>

Reviewed by David Levin.

[Haiku] Fix conversion from BString to String
https://bugs.webkit.org/show_bug.cgi?id=35082

The previous patch to this file was broken (sorry). First of all,
the conversion from UTF8 was accidentally removed. Second, for
empty strings, we need to point the implementation at StringImpl::empty().

Covered by existing tests.

  • platform/text/haiku/StringHaiku.cpp: (WebCore::String::String):
08:00 Changeset [55153] by vestbo@webkit.org

[Qt] Prevent the media backend from handling non-audio or -video mime types

Reviewed by Eric Carlson.

  • platform/graphics/qt/MediaPlayerPrivateQt.cpp:
07:46 Changeset [55152] by evan@chromium.org

2010-02-23 Evan Martin <evan@chromium.org>

Reviewed by Jeremy Orlow.

[chromium] Allow building without SVG
https://bugs.webkit.org/show_bug.cgi?id=31522

  • WebCore.gyp/WebCore.gyp, WebCore.gypi: test enable_svg flag.
  • bindings/v8/custom/V8SVGDocumentCustom.cpp, bindings/v8/custom/V8SVGPathSegCustom.cpp: test ENABLE(SVG).
07:28 Changeset [55151] by eric@webkit.org

2010-02-23 Leandro Pereira <leandro@profusion.mobi>

Reviewed by NOBODY.

Fix false positives for 'delete *pointer' statements.
http://webkit.org/b/35235

  • WebKitTools/Scripts/webkitpy/style/processors/cpp.py:
07:21 Changeset [55150] by kov@webkit.org

2010-02-23 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Gustavo Noronha Silva.

Fixes references to GOwnPtr and GRefPtr so the GTK+ port builds
again.
http://webkit.org/b/35084

06:35 Companies and Organizations that have contributed to WebKit edited by skyul@company100.net
(diff)
06:21 Changeset [55149] by eric@webkit.org

2010-02-23 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Adding the EFL implementation of JavaScriptCore.
See https://bugs.webkit.org/show_bug.cgi?id=35084 for details.

  • GNUmakefile.am: Updated to reflect the new location of GOwnPtr and

GRefPtr.

  • wtf/efl/MainThreadEfl.cpp: Added.
  • wtf/gobject/GOwnPtr.cpp: Moved from wtf/gtk.
  • wtf/gobject/GOwnPtr.h: Moved from wtf/gtk.
  • wtf/gobject/GRefPtr.cpp: Moved from wtf/gtk.
  • wtf/gobject/GRefPtr.h: Moved from wtf/gtk.
06:17 Changeset [55148] by ariya@webkit.org

2010-02-23 Ariya Hidayat <ariya.hidayat@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Faster cut-off for rectangle drawing without shadow.
https://bugs.webkit.org/show_bug.cgi?id=35267

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::drawRect):
06:05 Changeset [55147] by eric@webkit.org

2010-02-23 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Removes redundant Settings::setGeolocationEnabled and Settings::geolocationEnabled
https://bugs.webkit.org/show_bug.cgi?id=35242

No new tests, removing dead code only.

  • page/Settings.cpp:
  • page/Settings.h:

2010-02-23 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Removes redundant Settings::setGeolocationEnabled and Settings::geolocationEnabled
https://bugs.webkit.org/show_bug.cgi?id=35242

This removes Chromium's WebSettings::setGeolocationEnabled

  • public/WebSettings.h:
  • src/WebSettingsImpl.cpp:
  • src/WebSettingsImpl.h:
05:12 Changeset [55146] by eric@webkit.org

2010-02-23 Ben Murdoch <benm@google.com>

Reviewed by Eric Seidel.

pageX/Y co-ordinates of TouchEvent should be 0,0
https://bugs.webkit.org/show_bug.cgi?id=35239

Update layout tests to reflect the WebCore changes in this bug.

  • fast/events/touch/basic-multi-touch-events-expected.txt: Updated.
  • fast/events/touch/basic-single-touch-events-expected.txt: Updated.
  • fast/events/touch/script-tests/basic-multi-touch-events.js: (verifyTouchEvent): Add checking of TouchEvent.pageX/Y
  • fast/events/touch/script-tests/basic-single-touch-events.js: (verifyTouchEvent): ditto.

2010-02-23 Ben Murdoch <benm@google.com>

Reviewed by Eric Seidel.

pageX/Y co-ordinates of TouchEvent should be 0,0
https://bugs.webkit.org/show_bug.cgi?id=35239

The co-ordinates attached to a touch event are not used for tracking touch motion, rather the co-ordinates attached to Touches within the touch event should be used. Set the co-ordinates on the event itself to 0 to avoid confusion and match observed iPhone behavior.

  • page/EventHandler.cpp: (WebCore::EventHandler::handleTouchEvent): Set the pageX/Y and screenX/Y co-ordinates attached to the touch event to 0,0 to match observed behavior on the iPhone.
  • page/EventHandler.h: Remove now unused members m_firstTouchScreenPos and m_firstTouchPagePos.
04:53 Changeset [55145] by eric@webkit.org

2010-02-23 Stephan Aßmus <superstippi@gmx.de>

Reviewed by Eric Seidel.

[Haiku] Various improvements to EventHandlerHaiku.
https://bugs.webkit.org/show_bug.cgi?id=34685

Covered by existing tests.

Fix the build by not including PlatformScrollBar.h.

  • page/haiku/EventHandlerHaiku.cpp: (WebCore::isKeyboardOptionTab):

Use the correct keycode for tab keys.

(WebCore::EventHandler::focusDocumentView):

Use proper locking. In any case, Haiku doesn't append platform
widgets to WebCore widgets. But if it did, this implementation
would be correct.

(WebCore::EventHandler::passWidgetMouseDownEventToWidget):

Implemented.

(WebCore::EventHandler::eventActivatedView):

Removed notImplemented() and added note.

(WebCore::EventHandler::passMousePressEventToSubframe):

Implemented.

(WebCore::EventHandler::passMouseMoveEventToSubframe):

Implemented.

(WebCore::EventHandler::passMouseReleaseEventToSubframe):

Implemented.

(WebCore::EventHandler::accessKeyModifiers):

Added note.

04:43 Changeset [55144] by hamaji@chromium.org

2010-02-23 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

[Gtk] Implement layoutTestController.numberOfPages
https://bugs.webkit.org/show_bug.cgi?id=35228

  • platform/gtk/Skipped:

2010-02-23 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

[Gtk] Implement layoutTestController.numberOfPages
https://bugs.webkit.org/show_bug.cgi?id=35228

  • webkit/webkitprivate.h:
  • webkit/webkitwebframe.cpp: (webkit_web_frame_number_of_pages):

2010-02-23 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

[Gtk] Implement layoutTestController.numberOfPages
https://bugs.webkit.org/show_bug.cgi?id=35228

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::numberOfPages):
04:27 Changeset [55143] by tony@chromium.org

2010-02-23 Tony Chang <tony@chromium.org>

Not reviewed.
Revert r55135 because the layout test is failing on all the bots.
https://bugs.webkit.org/show_bug.cgi?id=34634

  • editing/pasteboard/paste-list-004-expected.txt: Removed.
  • editing/pasteboard/paste-list-004.html: Removed.

2010-02-23 Tony Chang <tony@chromium.org>

Not reviewed.
Revert r55135 because the layout test is failing on all the bots.
https://bugs.webkit.org/show_bug.cgi?id=34634

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::insertAsListItems):
04:07 Changeset [55142] by steveblock@google.com

Adds cache to store Geolocation positions between browser sessions.
https://bugs.webkit.org/show_bug.cgi?id=34084

Reviewed by Ariya Hidayat.

This is required to fully implement the Geolocation maximumAge property.
See Bug 30676.

No new tests, will add tests for maximumAge once complete.

  • Android.mk: Modified. Added GeolocationPositionCache.cpp
  • GNUmakefile.am: Modified. Added GeolocationPositionCache.[cpp|h]
  • WebCore.gypi: Modified. Added GeolocationPositionCache.[cpp|h]
  • WebCore.pro: Modified. Added GeolocationPositionCache.[cpp|h]
  • WebCore.vcproj/WebCore.vcproj: Modified. Added GeolocationPositionCache.[cpp|h]
  • WebCore.xcodeproj/project.pbxproj: Modified. Added GeolocationPositionCache.[cpp|h]
  • page/GeolocationPositionCache.cpp: Added.

(WebCore::GeolocationPositionCache::GeolocationPositionCache):
(WebCore::GeolocationPositionCache::~GeolocationPositionCache):
(WebCore::GeolocationPositionCache::setCachedPosition):
(WebCore::GeolocationPositionCache::cachedPosition):
(WebCore::GeolocationPositionCache::setDatabasePath):
(WebCore::GeolocationPositionCache::readFromDB):
(WebCore::GeolocationPositionCache::writeToDB):

  • page/GeolocationPositionCache.h: Added.
04:04 Changeset [55141] by eric@webkit.org

2010-02-23 Yuta Kitamura <yutak@chromium.org>

Reviewed by Dan Bernstein.

Fix alignment of vertical-align: text-bottom inside an inline-block.

This patch is based on a fix provided by Takahito Hirano.

display: inline-block + vertical-align: text-bottom causes misalignment.
https://bugs.webkit.org/show_bug.cgi?id=30846

  • fast/inline-block/inline-block-vertical-align-2.html: Added.
  • fast/inline-block/inline-block-vertical-align-2-expected.txt: Added.

2010-02-23 Yuta Kitamura <yutak@chromium.org>

Reviewed by Dan Bernstein.

Fix alignment of vertical-align: text-bottom inside an inline-block.

This patch is based on a fix provided by Takahito Hirano.

display: inline-block + vertical-align: text-bottom causes misalignment.
https://bugs.webkit.org/show_bug.cgi?id=30846

Test: fast/inline-block/inline-block-vertical-align-2.html

  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::verticalPosition): Fixed vpos calculation. We cannot cut corners for inline blocks, since they may have their own text metrics.
03:43 Changeset [55140] by steveblock@google.com

Sets default values of V8 runtime enabler flags to match behavior with JSC
https://bugs.webkit.org/show_bug.cgi?id=35095

03:25 Changeset [55139] by eric@webkit.org

2010-02-23 José Millán Soto <jmillan@igalia.com>

Reviewed by Xan Lopez.

[Gtk] Server message not shown on http authentication
https://bugs.webkit.org/show_bug.cgi?id=34219

  • webkit/webkitsoupauthdialog.c: (show_auth_dialog): Server message is displayed, messageLabel and message variables were renamed to avoid confusion.
03:09 Changeset [55138] by eric@webkit.org

2010-02-23 Stephan Aßmus <superstippi@gmx.de>

Reviewed by Eric Seidel.

[Haiku] Fix various issues in keyboard event generation.
https://bugs.webkit.org/show_bug.cgi?id=34685

Covered by existing tests.

Fixed backspace keycode.
Fixed using the appropriate fields from the Haiku event for mapping
to the VK_* codes and added mapping a great deal more codes.
Added extraction of modifier key flags.
Completed implementation of disambiguateKeyDownEvent()
Implemented currentCapsLockState().

  • platform/haiku/PlatformKeyboardEventHaiku.cpp: (WebCore::keyIdentifierForHaikuKeyCode): (WebCore::windowsKeyCodeForKeyEvent): (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent): (WebCore::PlatformKeyboardEvent::disambiguateKeyDownEvent): (WebCore::PlatformKeyboardEvent::currentCapsLockState):
02:53 Changeset [55137] by eric@webkit.org

2010-02-23 Dmitriy Belenko <dbelenko@google.com>

Reviewed by Darin Fisher.

Chromium: Need to be able to get the bounds of selection
rectangle(s)
https://bugs.webkit.org/show_bug.cgi?id=34915

This change will enable about 30 test cases to pass in Chromium.
All of these test cases are related to selection rect boundaries.
This change will enable the test cases to retrieve the selection
rect boundary rectangle for the current selection.

  • public/WebFrame.h:
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::selectionBoundsRect):
  • src/WebFrameImpl.h:
02:35 Changeset [55136] by steveblock@google.com

Adds Client::cancelGeolocationPermissionRequestForFrame and ChromeClient::cancelGeolocationPermissionRequestForFrame
https://bugs.webkit.org/show_bug.cgi?id=34962

These methods are required so that a Geolocation object can cancel an
asynchronous permission request. This allows the chrome client to cancel
any UI it is showing for the permission request.

02:19 Changeset [55135] by eric@webkit.org

2010-02-23 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=34634
Pasting a list into the middle of another list item should split the target
list item into two separate list items. This matches the behavior in other
browsers.

  • editing/pasteboard/paste-list-004-expected.txt: Added.
  • editing/pasteboard/paste-list-004.html: Added.

2010-02-23 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=34634
Pasting a list into the middle of another list item should split the target
list item into two separate list items. This matches the behavior in other
browsers.

Test: editing/pasteboard/paste-list-004.html

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::insertAsListItems):
02:03 Changeset [55134] by eric@webkit.org

2010-02-23 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QtLauncher should not use internal JavaScriptCore and WebCore interfaces
https://bugs.webkit.org/show_bug.cgi?id=35248

  • QtLauncher/QtLauncher.pro:
  • QtLauncher/utils.h:
01:55 Changeset [55133] by jocelyn.turcotte@nokia.com

[Qt] Correct build problems while building QtWebKit inside Qt.
https://bugs.webkit.org/show_bug.cgi?id=34975

Reviewed by Laszlo Gombos.

  • WebCore.pro: Change the condition !standalone_package to !QTDIR_build
01:47 Changeset [55132] by eric@webkit.org

2010-02-23 Noam Rosenthal <noam.rosenthal@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Connect video with accelerated compositing
https://bugs.webkit.org/show_bug.cgi?id=35276

MediaControlPrivate and GraphicsLayer are patched together via
a shared PlatformLayer (QGraphicsItem). This patch makes sure that the
QGraphicsVideoItem from MediaControl becomes part of the scene
associsated with GraphicsLayer

Test: http://double.co.nz/video_test/test1.html with AC turned on

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::GraphicsLayerQtImpl::): mediaLayer member (WebCore::GraphicsLayerQtImpl::opaqueArea): video is opaque (WebCore::GraphicsLayerQtImpl::paint): don't paint video (WebCore::GraphicsLayerQtImpl::flushChanges): flush mediaLayer (WebCore::GraphicsLayerQt::setContentsToMedia): notify
  • platform/graphics/qt/GraphicsLayerQt.h: reimp setContentsToMedia
  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): m_compositing (WebCore::MediaPlayerPrivate::paint): don't paint if compositing (WebCore::MediaPlayerPrivate::acceleratedRenderingStateChanged): reimp from MediaPlayerPrivateInterface to support AC (WebCore::MediaPlayerPrivate::platformLayer): ditto
  • platform/graphics/qt/MediaPlayerPrivateQt.h: (WebCore::MediaPlayerPrivate::supportsAcceleratedRendering): ditto
01:31 Changeset [55131] by eric@webkit.org

2010-02-23 Daniel Bates <dbates@rim.com>

Reviewed by Eric Seidel.

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

Prompts a person for their SVN username if not already cached (by Subversion).

Currently, webkit-patch is unable to commit to the SVN repo. unless the
WebKit SVN username is already cached (from of a prior commit by hand)
because "svn commit" (called by webkit-patch) defaults to using the system
login name unless the username is already cached or specified on the
command line.

  • Scripts/webkitpy/scm.py: Added methods SVN.has_authorization_for_realm and modified SVN.commit_with_message to call it. Added optional username parameter to method SVN.commit_with_message.
  • Scripts/webkitpy/scm_unittest.py: Added unit test methods: SVNTest.test_commit_with_username, SVNTest.test_has_authorization_for_realm, and SVNTest.test_not_have_authorization_for_realm.
01:14 Changeset [55130] by eric@webkit.org

2010-02-23 Stephan Aßmus <superstippi@gmx.de>

Reviewed by Eric Seidel.

Fix various issues in PlatformWheelEventHaiku.
https://bugs.webkit.org/show_bug.cgi?id=34685

Covered by existing tests.

Fixed coding style violations.
Synced extracting the correct coordinates with Haiku WebKit implementation.
Added extracting modifier key flags.

  • platform/haiku/PlatformWheelEventHaiku.cpp: (WebCore::PlatformWheelEvent::PlatformWheelEvent):
01:02 Changeset [55129] by xan@webkit.org

2010-02-23 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Do not do unnecessary work during size_allocate.

FrameView::resize will already queue a layout, so there's no need
to force one ourselves. On top of that, the layout function
already adjusts the view size when needed, so there's no need to
do that manually either. No change in the layout tests or unit
tests after this.

  • webkit/webkitwebview.cpp: (webkit_web_view_size_allocate):
00:58 Changeset [55128] by eric@webkit.org

2010-02-23 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Eric Seidel.

Build fix for PluginView
https://bugs.webkit.org/show_bug.cgi?id=35230

No new tests, build fix only.

  • plugins/PluginView.cpp:
  • plugins/PluginView.h:
  • plugins/PluginViewNone.cpp:
00:42 Changeset [55127] by eric@webkit.org

2010-02-23 Diego Escalante Urrelo <descalante@igalia.com>

Reviewed by Eric Seidel.

[gtk] missing libsoup-2.4 package in gir generation
https://bugs.webkit.org/show_bug.cgi?id=35199

Include libsoup-2.4 package in gobject introspection .gir generation.

  • GNUmakefile.am:
00:26 Changeset [55126] by eric@webkit.org

2010-02-23 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Port ScriptController::createScriptInstanceForWidget
https://bugs.webkit.org/show_bug.cgi?id=34413

  • bindings/js/ScriptControllerBrew.cpp: Added. (WebCore::ScriptController::createScriptInstanceForWidget):
00:19 Changeset [55125] by philn@webkit.org

2010-02-18 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Seidel.

media/video-display-aspect-ratio.html fails
https://bugs.webkit.org/show_bug.cgi?id=34933

Original patch by Eric Carlson.

  • media/content/test-par-16-9.mp4: Change the width of the video track from 426.666 to 426.
  • platform/mac-leopard/Skipped: Unskip the test, it should pass now.
00:09 Changeset [55124] by eric@webkit.org

2010-02-23 José Millán Soto <jmillan@igalia.com>

Reviewed by Eric Seidel.

[Gtk] Webkit crashes when using orca
https://bugs.webkit.org/show_bug.cgi?id=35169

Made webkit_accessible_text_get_caret_offset check if end selection
node is null.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_text_get_caret_offset):

02/22/10:

23:53 Changeset [55123] by eric@webkit.org

2010-02-22 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Siedel.

Add more checking for missing binaries and packages to check_sys_deps()

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

  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
  • Scripts/webkitpy/layout_tests/run_chromium_webkit_tests.py:
23:37 Changeset [55122] by eric@webkit.org

2010-02-22 Stephan Aßmus <superstippi@gmx.de>

Reviewed by Eric Seidel.

[Haiku] Implemented PopupMenu support.
https://bugs.webkit.org/show_bug.cgi?id=35078

Covered by existing tests.

The implementation is currently very simple: Added a PopupMenuHaiku
class that derives from a native BPopUpMenu. It attaches a BHandler
derivative to the BApplication (WebCore main thread) which receives
the item invokation and menu hidden events and informs the PopupMenuClient
accordingly.

  • platform/PopupMenu.h:

Changed type of m_menu for Haiku.

  • platform/haiku/PopupMenuHaiku.cpp: (WebCore::PopupMenuHandler::PopupMenuHandler): (WebCore::PopupMenuHandler::MessageReceived): (WebCore::PopupMenuHaiku::PopupMenuHaiku): (WebCore::PopupMenuHaiku::~PopupMenuHaiku): (WebCore::PopupMenuHaiku::show): (WebCore::PopupMenuHaiku::hide): (WebCore::PopupMenuHaiku::Hide): (WebCore::PopupMenu::PopupMenu): (WebCore::PopupMenu::~PopupMenu):

Removed bogus code.

(WebCore::PopupMenu::show):

Implemented using new PopupMenuHaiku class.

(WebCore::PopupMenu::hide):

Implemented using new PopupMenuHaiku class.

(WebCore::PopupMenu::updateFromElement):

Implemented.

(WebCore::PopupMenu::itemWritingDirectionIsNatural):

Implemented according to Gtk port.

22:28 Changeset [55121] by hamaji@chromium.org

2010-02-22 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by David Levin.

[Gtk] check-webkit-style: GTK style should be allowed in WebKitTools/DumpRenderTree/gtk
https://bugs.webkit.org/show_bug.cgi?id=35229

  • Scripts/webkitpy/style/checker.py:
  • Scripts/webkitpy/style/checker_unittest.py:
21:03 Changeset [55120] by jchaffraix@webkit.org

Remove auto_ptr usage in JavaScriptCore.
https://bugs.webkit.org/show_bug.cgi?id=35221

Reviewed by Darin Adler.

  • parser/Nodes.h: Removed now unneeded adopt method.
  • parser/Parser.cpp: Removed <memory> include as it is not required anymore.
  • wtf/OwnPtr.h: Removed the constructor from auto_ptr.
  • wtf/VectorTraits.h: Removed a template specialization for auto_ptr.
  • wtf/unicode/Collator.h: Made userDefault return a PassOwnPtr.
  • wtf/unicode/CollatorDefault.cpp:

(WTF::Collator::userDefault): Changed the method to match the next signature.

  • wtf/unicode/icu/CollatorICU.cpp:

(WTF::Collator::userDefault): Ditto.

20:50 Changeset [55119] by jamesr@google.com

2010-02-22 James Robinson <jamesr@chromium.org>

Unreviewed. Adding myself to committers list.

  • Scripts/webkitpy/committers.py:
20:29 Changeset [55118] by simon.fraser@apple.com

2010-02-22 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler.

Remove DRT hack that turns off hardware acceleration with older QuickTimes
https://bugs.webkit.org/show_bug.cgi?id=35275

Now that WebKit does a version check to avoid a QuickTime-related
crash (r55100), DumpRenderTree does not need to.

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
17:55 Changeset [55117] by eric@webkit.org

2010-02-22 Huahui Wu <hwu@google.com>

Reviewed by Eric Seidel.

Add code that enables SquirrelFish Extreme (a.k.a JSCX, JSC JIT)
in Android. It's disabled by default, but is enabled when the
enveronment variable ENABLE_JSC_JIT is set to true.
https://bugs.webkit.org/show_bug.cgi?id=34855

  • Android.mk:

2010-02-22 Huahui Wu <hwu@google.com>

Reviewed by Eric Seidel.

Add code that enables SquirrelFish Extreme (a.k.a JSCX, JSC JIT)
in Android. It's disabled by default, but is enabled when the
enveronment variable ENABLE_JSC_JIT is set to true.
https://bugs.webkit.org/show_bug.cgi?id=34855

  • Android.mk:
  • wtf/Platform.h:
17:29 Changeset [55116] by eric@webkit.org

2010-02-22 Stephan Aßmus <superstippi@gmx.de>

Reviewed by Eric Seidel.

Fix various issues in PlatformMouseEventHaiku.
https://bugs.webkit.org/show_bug.cgi?id=34685

Covered by existing tests.

Mapping Haiku button constants (bit field) to WebCore buttons was broken.
Extracting event time was broken (supposed to be in seconds).
Wrong coordinate was being extracted, needs to be content local.
Added extracting modifier key flags.

  • platform/haiku/PlatformMouseEventHaiku.cpp: (WebCore::PlatformMouseEvent::PlatformMouseEvent):
17:12 Changeset [55115] by eric@webkit.org

2010-02-22 Stephan Aßmus <superstippi@gmx.de>

Reviewed by Eric Seidel.

[Haiku] Implement creating and filling platform gradients.
https://bugs.webkit.org/show_bug.cgi?id=34683

Covered by existing tests.

  • platform/graphics/Gradient.h:

Typedef PlatformGradient to BGradient

  • platform/graphics/haiku/GradientHaiku.cpp: (WebCore::Gradient::platformDestroy):

Delete the cached BGradient object.

(WebCore::Gradient::platformGradient):

Create a BGradient object according to the type of Gradient.
Return the cached object.

(WebCore::Gradient::fill):

use BView API to fill with the platform gradient.

17:01 Changeset [55114] by tony@chromium.org

2010-02-22 Chris Evans <cevans@chromium.org>

Reviewed by Darin Adler.

Add test for https://bugs.webkit.org/show_bug.cgi?id=32842
It has been fixed on trunk for a while, but we want to make sure we
do not regress.

  • fast/text/find-layout-crash.html: Added.
  • fast/text/find-layout-crash-expected.txt: Added.
16:55 Changeset [55113] by eric@webkit.org

2010-02-22 Stephan Aßmus <superstippi@gmx.de>

Reviewed by Eric Seidel.

Build fix for debug builds of GlyphPageTreeNode.cpp
https://bugs.webkit.org/show_bug.cgi?id=34528

Covered by existing tests.

  • platform/graphics/GlyphPageTreeNode.cpp:

include <stdio.h> since printf() is used if NDEBUG is not defined.

16:45 Changeset [55112] by sfalken@apple.com

WebKit on Windows should pick up system setting changes without requiring explicit API calls
https://bugs.webkit.org/show_bug.cgi?id=35269

Reviewed by Darin Adler.

  • WebKit.vcproj/WebKit.def: Removed WebKitSystemParameterChanged.
  • WebKit.vcproj/WebKit_debug.def: Removed WebKitSystemParameterChanged.
  • WebKitGraphics.cpp: Removed WebKitSystemParameterChanged.
  • WebKitGraphics.h: Removed WebKitSystemParameterChanged.
  • WebView.cpp:

(systemParameterChanged): Call through to wkSystemFontSmoothingChanged for font changes.
(WebView::windowReceivedMessage): Pick up WM_SETTINGCHANGE from windowReceivedMessage.

16:12 Changeset [55111] by japhet@chromium.org

2010-02-22 Nate Chapin <japhet@chromium.org>

Unreviewed, Chromium build fix.

Add special case in CodeGeneratorV8.pm for named getters for html plugin-related elements.

CodeGeneratorV8.pm was relying on HasOverridingNameGetter to hint
that a custom named getter was needed. That hint was removed in
http://trac.webkit.org/changeset/55104.

  • bindings/scripts/CodeGeneratorV8.pm:
15:28 Changeset [55110] by ossy@webkit.org

Rubber-stamped by Alexey Proskuryakov.

[Qt] fast/encoding/invalid-UTF-8-2.html fails
https://bugs.webkit.org/show_bug.cgi?id=35263

Qt Linux Release buildbot got the platform independent result for
fast/encoding/invalid-UTF-8-2.html instead of Qt specific one.

Qt specific expected file is removed to make buildbot green,
but it needs more investigation.

  • platform/qt/fast/encoding/invalid-UTF-8-2-expected.txt: Removed.
15:27 Changeset [55109] by ap@apple.com

Rubber-stamped by Geoff Garen.

Rename RuntimeObjectImp to RuntimeObject.

15:23 Changeset [55108] by pkasting@chromium.org

Avoid recursion when trying to get the size of a PNG; it's unnecessary
and in the worst case can lead to heap corruption.
https://bugs.webkit.org/show_bug.cgi?id=35167

Reviewed by Eric Seidel.

WebCore:

Test: fast/images/bad-png.html

  • platform/image-decoders/png/PNGImageDecoder.cpp:

(WebCore::PNGImageReader::decode):

LayoutTests:

  • fast/images/bad-png-expected.txt: Added.
  • fast/images/bad-png.html: Added.
  • fast/images/resources/bad-png.png: Added.
15:20 BuildingOnWindows edited by aroben@apple.com
(diff)
15:10 Changeset [55107] by beidson@apple.com

Disable WebView docking to views that are too small.
<rdar://problem/7248409> and https://bugs.webkit.org/show_bug.cgi?id=35254

Reviewed by Tim Hatcher.

WebCore:

  • WebCore.base.exp:
  • inspector/InspectorController.cpp:

(WebCore::InspectorController::canAttachWindow): Provide a utility for WebKits to make a showWindow()

decision based on if attachment would be allowed or not.

(WebCore::InspectorController::attachWindow): Don't attach if the view is too small to attach to.

  • inspector/InspectorController.h:

WebKit/mac:

  • WebCoreSupport/WebInspectorClient.mm:

(-[WebInspectorWindowController showWindow:]): No matter the preference, don't open the inspector

window attached if WebCore says it shouldn't be attached.

WebKit/win:

  • WebCoreSupport/WebInspectorClient.cpp:

(WebInspectorClient::showWindowWithoutNotifications): No matter the preference, don't open the inspector

window attached if WebCore says it shouldn't be attached.

14:32 Changeset [55106] by ap@apple.com

Build fix.

  • WebCore.base.exp: Export Instance::newRuntimeObject, it's virtual!
14:25 Changeset [55105] by ap@apple.com

Undo a small part of the previous commit.

  • bridge/runtime_method.cpp: (JSC::callRuntimeMethod): Let's keep the instance local variable as RefPtr for safety.
14:21 Changeset [55104] by ap@apple.com

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=35260
Eliminate apple_runtime_object

No change in functionality, so no tests.

  • WebCore.base.exp: Don't export Instance::newRuntimeObject, WebKit doesn't need it.
  • bindings/js/JSHTMLAppletElementCustom.cpp:
  • bindings/js/JSHTMLEmbedElementCustom.cpp:
  • bindings/js/JSHTMLObjectElementCustom.cpp:
  • html/HTMLAppletElement.idl:
  • html/HTMLEmbedElement.idl:
  • html/HTMLObjectElement.idl: These objects no longer need overriding name getters, as they no longer intercept the apple_runtime_object property.
  • bindings/js/JSPluginElementFunctions.cpp: (WebCore::pluginInstance): This is no longer static. It was used for callPlugin() in this file, and now it's also used elsewhere when calling plug-in methods. (WebCore::runtimeObjectGetter): Removed. This function was only used by the intercepting name getters.
  • bindings/js/JSPluginElementFunctions.h: Export pluginInstance().
  • bindings/objc/WebScriptObject.mm: (+[WebScriptObject _convertValueToObjcValue:originRootObject:rootObject:]):
  • bridge/runtime_method.cpp: (JSC::callRuntimeMethod): Take plug-in element's instance directly, without relying on fake attribute lookup. One change resulting from this is that RuntimeObjectImp may not be created in some cases - this code only needs an instance, but the old code had to pass the instance wrapped into RuntimeObjectImp.
14:17 Changeset [55103] by ap@apple.com

2010-02-22 Robert Hogan <robert@roberthogan.net>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=32937
LayoutTests/fast/encoding/invalid-UTF-8.html fails on Qt.

The test was verifying two things: first, that U+FFFD is rendered by the browser, and second,
that ill-formed UTF-8 sequences are turned into U+FFFD. Some aspects of the latter are not
precisely defined by the Unicode specification, so different platforms may legitimately
have different results.

Split the test into two: one to test the rendering of invalid UTF-8 sequences, and the other
to protect against regressions in each platform's decoding behaviour.

  • fast/encoding/invalid-UTF-8-2-expected.txt: Added.
  • fast/encoding/invalid-UTF-8-2.html: Added.
  • fast/encoding/invalid-UTF-8.html:
  • platform/mac/fast/encoding/invalid-UTF-8-expected.checksum:
  • platform/mac/fast/encoding/invalid-UTF-8-expected.png:
  • platform/mac/fast/encoding/invalid-UTF-8-expected.txt:
  • platform/qt/fast/encoding/invalid-UTF-8-2-expected.txt:
13:07 Changeset [55102] by cfleizach@apple.com

Bug 35186 - AX: AXFocused is not writable like it should be on nodes
https://bugs.webkit.org/show_bug.cgi?id=35186

Fixing layout test failure.

12:59 Changeset [55101] by sullivan@apple.com

https://bugs.webkit.org/show_bug.cgi?id=35256
HTMLSelectElement::setSelectedIndexByUser() can trigger unnecessary JS when there is no change to the selected index

Reviewed by Tim Hatcher.

No new tests because this code path is not used for JS-initiated changes.

  • html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::setSelectedIndexByUser):
Bail out if the to-be-selected index matches the already-selected index.

12:51 Changeset [55100] by simon.fraser@apple.com

2010-02-22 Simon Fraser <simon.fraser@apple.com>

Reviewed by John Sullivan.

<rdar://problem/7285392>
On Leopard, we have to disable hardware acceleration if we detect that the
installed Core Video framework has bug <rdar://problem/7228836>.

  • WebView/WebView.mm: (coreVideoHas7228836Fix): (-[WebView _preferencesChangedNotification:]):
12:18 Changeset [55099] by kov@webkit.org

Tagging 1.1.22.

12:05 Changeset [55098] by enrica@apple.com

Reproducible crash in WebCore::nextLinePosition on Tweeteorites.com
<rdar://problem/7615758>
https://bugs.webkit.org/show_bug.cgi?id=35060

Reviewed by Dan Bernstein.

WebCore:

Test: editing/selection/extend-byline-withfloat.html

  • editing/visible_units.cpp:

(WebCore::previousLinePosition): Skip elements with zero height.
(WebCore::nextLinePosition): Skip elements with zero height.

LayoutTests:

  • editing/selection/extend-byline-withfloat-expected.txt: Added.
  • editing/selection/extend-byline-withfloat.html: Added.
12:03 Changeset [55097] by japhet@chromium.org

2010-02-22 Nate Chapin <japhet@chromium.org>

Unreviewed, Chromium build fix.

  • bindings/scripts/CodeGeneratorV8.pm: Compile break due to bad patch merge.
11:53 Changeset [55096] by japhet@chromium.org

2010-02-22 Nate Chapin <japhet@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Streamline V8 event listener code. Merge all the variants of
V8DOMWrapper::getEventListner() into a single version and generate
addEventListener() and removeEventListener() bindings for all objects
except DOMWindow.

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

11:29 Changeset [55095] by cfleizach@apple.com

AX: AXFocused is not writable like it should be on nodes
https://bugs.webkit.org/show_bug.cgi?id=35186

Reviewed by Beth Dakin.

WebCore:

Test: platform/mac/accessibility/element-focus.html

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::canSetFocusAttribute):

LayoutTests:

  • platform/mac/accessibility/element-focus-expected.txt: Added.
  • platform/mac/accessibility/element-focus.html: Added.
11:24 Changeset [55094] by laszlo.1.gombos@nokia.com

2010-02-22 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed Symbian build fix.

[Qt] Updated the def file with new exports to enable
building DumpRenderTree.

  • symbian/eabi/QtWebKitu.def:
11:12 Changeset [55093] by barraclough@apple.com

JSStringBuilder should not CRASH if allocation fails, it should throw a JSException.

Reviewed by Oliver Hunt.

  • runtime/JSGlobalObjectFunctions.cpp:
  • runtime/JSStringBuilder.h:

(JSC::JSStringBuilder::JSStringBuilder):
(JSC::JSStringBuilder::append):
(JSC::JSStringBuilder::build):

  • runtime/StringBuilder.h:

(JSC::StringBuilder::build):

  • wtf/Vector.h:

(WTF::VectorBufferBase::tryAllocateBuffer):
(WTF::):
(WTF::VectorBuffer::tryAllocateBuffer):
(WTF::::tryExpandCapacity):
(WTF::::tryReserveCapacity):
(WTF::::tryAppend):

11:07 Changeset [55092] by xan@webkit.org

2010-02-22 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Bump library versioning for 1.1.22 release.

  • configure.ac:

WebKit/gtk:

2010-02-22 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Update for release.

  • NEWS:
11:02 Changeset [55091] by eric@webkit.org

2010-02-22 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Map FastMalloc to BREW memory allocator
https://bugs.webkit.org/show_bug.cgi?id=33570

Use MALLOC macro instead of the standard malloc function.
Although RVCT provides malloc, we can't use it in BREW
because the loader does not initialize the base address properly.

  • wtf/FastMalloc.cpp:
  • wtf/brew/SystemMallocBrew.h: Added. (mallocBrew): (callocBrew): (freeBrew): (reallocBrew):
10:48 Changeset [55090] by simon.fraser@apple.com

2010-02-22 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein, Darin Adler.

Remove addScrolledContentOffset/subtractScrolledContentOffset
https://bugs.webkit.org/show_bug.cgi?id=35225

Remove RenderLayer's addScrolledContentOffset() and subtractScrolledContentOffset()
methods, and instead use the existing scrolledContentOffset(), and use
IntSize and IntPoint instead of lots of x, y variables.

Added new IntPoint toPoint(const IntSize&) method as a convenience to convert a size to a point,
which is needed in a few places.

  • page/EventHandler.cpp: (WebCore::EventHandler::handleDrag): Use the new toPoint() convenience.
  • platform/graphics/IntPoint.h: (WebCore::toPoint): New convenience method to convert an IntSize to an IntPoint.
  • rendering/LayoutState.cpp: (WebCore::LayoutState::LayoutState):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock): (WebCore::RenderBlock::paintObject): (WebCore::RenderBlock::selectionGapRectsForRepaint): (WebCore::RenderBlock::nodeAtPoint): (WebCore::RenderBlock::offsetForContents):
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintFillLayerExtended):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::clippedOverflowRectForRepaint):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPosition):
  • rendering/RenderLayer.h: (WebCore::RenderLayer::size):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::computeRectForRepaint):
  • rendering/TextControlInnerElements.cpp: (WebCore::RenderTextControlInnerBlock::positionForPoint):
10:36 Changeset [55089] by agl@chromium.org

2009-02-22 Adam Langley <agl@google.com>

Reviewed by Darin Fisher.

fontconfig on Linux can change the render preferences on a per strike
basis (a strike a combination of face and size). Because of this, we
need to query fontconfig each time a new FontPlatformData is created
for a new size.

This patch adds support for querying this via ChromiumBridge.

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

  • platform/chromium/ChromiumBridge.h: add getRenderStyleForStrike
  • platform/graphics/chromium/FontCacheLinux.cpp: (WebCore::FontCache::createFontPlatformData):
  • platform/graphics/chromium/FontCustomPlatformData.cpp: (WebCore::FontCustomPlatformData::fontPlatformData):
  • platform/graphics/chromium/FontPlatformDataLinux.cpp: (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::operator=): (WebCore::FontPlatformData::setupPaint): (WebCore::FontPlatformData::queryStyle): add code to query fontconfig via ChromiumBridge
  • platform/graphics/chromium/FontPlatformDataLinux.h:
10:11 Changeset [55088] by kov@webkit.org

Build fix for make distcheck.

09:25 Changeset [55087] by eric@webkit.org

2010-02-22 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Fix compiler warning "suggest parentheses around"
https://bugs.webkit.org/show_bug.cgi?id=35197

No new tests as there is no new functionality.

  • wml/WMLVariables.cpp: (WebCore::substituteVariableReferences):
09:01 Changeset [55086] by kov@webkit.org

WebCore

2010-02-22 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[Soup] loses information related to message flags when converting from/to Resource{Request,Response}
https://bugs.webkit.org/show_bug.cgi?id=35093

Store message flags in WebCore objects, and set them on the
SoupMessage, when creating one from them.

No behaviour change.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::fillResponseFromMessage):
  • platform/network/soup/ResourceRequest.h: (WebCore::ResourceRequest::soupMessageFlags): (WebCore::ResourceRequest::setSoupMessageFlags):
  • platform/network/soup/ResourceRequestSoup.cpp: (WebCore::ResourceRequest::toSoupMessage): (WebCore::ResourceRequest::updateFromSoupMessage):
  • platform/network/soup/ResourceResponse.h: (WebCore::ResourceResponse::soupMessageFlags): (WebCore::ResourceResponse::setSoupMessageFlags):
  • platform/network/soup/ResourceResponseSoup.cpp: (WebCore::ResourceResponse::toSoupMessage): (WebCore::ResourceResponse::updateFromSoupMessage):

WebKit/gtk

2010-02-22 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[Soup] loses information related to message flags when converting from/to Resource{Request,Response}
https://bugs.webkit.org/show_bug.cgi?id=35093

Update the flags that are stored in the request, when the response
is received.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidReceiveResponse):
08:40 Changeset [55085] by vestbo@webkit.org

[Qt] Build the DRT in debug on Mac OS X

Reviewed by Kenneth Rohde Christiansen.

  • DumpRenderTree/qt/DumpRenderTree.pro:
08:17 Changeset [55084] by eric@webkit.org

2010-02-22 Steve Block <steveblock@google.com>

Reviewed by Nate Chapin.

Shared worker types used in globalObjectPrototypeIsDOMWindow are not properly guarded
https://bugs.webkit.org/show_bug.cgi?id=35238

No new tests, build fix only.

  • bindings/v8/V8DOMWrapper.cpp: (WebCore::globalObjectPrototypeIsDOMWindow):
07:16 Changeset [55083] by laszlo.1.gombos@nokia.com

2010-02-22 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed build fix.

[Qt] Build fix for RVCT.

Fix after r55024. The "-i" option is for perl not for the
script.

  • DerivedSources.pro:
07:12 Changeset [55082] by eric@webkit.org

2010-02-22 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add support for layout tests on Symbian
https://bugs.webkit.org/show_bug.cgi?id=31589

  • WebKit.pro:

2010-02-22 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add support for layout tests on Symbian
https://bugs.webkit.org/show_bug.cgi?id=31589

  • DumpRenderTree/qt/DumpRenderTree.pro:
  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:
  • DumpRenderTree/qt/main.cpp: (main):
06:26 Changeset [55081] by eric@webkit.org

2010-02-22 Steve Block <steveblock@google.com>

Reviewed by Simon Fraser.

Simplifies calculation of the transform in RenderLayer::paintLayer
https://bugs.webkit.org/show_bug.cgi?id=35101

No new tests, optimization only.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):
06:21 Changeset [55080] by vestbo@webkit.org

2010-02-22 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Maciej Stachowiak.

Don't reset the assignee back to default on component change

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

There's a risk of reseting a valid assignee for example when
triaging a bug. All components in WebKit have the default
assignee set to webkit-unassigned@webkit.org so this should
not cause problems for people relying on this functionality.

  • template/en/custom/bug/edit.html.tmpl:
05:42 Changeset [55079] by vestbo@webkit.org

[Qt] Switching from Phonon to QtMultimedia Backend for Qt 4.7

Patch by Nicholas Young <nicholas.young@nokia.com> on 2010-02-22
Reviewed by Tor Arne Vestbø.

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

No new tests. This patch only affects multimedia backend.

  • WebCore.pro: Build depends on Qt version.
  • css/mediaControlsQt.css: Updated media controls.
  • platform/graphics/MediaPlayer.cpp: Different backend included depending on Qt version.
  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: Added new QtMultimedia Backend.

(WebCore::MediaPlayerPrivate::create):
(WebCore::MediaPlayerPrivate::registerMediaEngine):
(WebCore::MediaPlayerPrivate::getSupportedTypes):
(WebCore::MediaPlayerPrivate::supportsType):
(WebCore::MediaPlayerPrivate::MediaPlayerPrivate):
(WebCore::MediaPlayerPrivate::~MediaPlayerPrivate):
(WebCore::MediaPlayerPrivate::hasVideo):
(WebCore::MediaPlayerPrivate::hasAudio):
(WebCore::MediaPlayerPrivate::load):
(WebCore::MediaPlayerPrivate::cancelLoad):
(WebCore::MediaPlayerPrivate::play):
(WebCore::MediaPlayerPrivate::pause):
(WebCore::MediaPlayerPrivate::paused):
(WebCore::MediaPlayerPrivate::seek):
(WebCore::MediaPlayerPrivate::seeking):
(WebCore::MediaPlayerPrivate::duration):
(WebCore::MediaPlayerPrivate::currentTime):
(WebCore::MediaPlayerPrivate::buffered):
(WebCore::MediaPlayerPrivate::maxTimeSeekable):
(WebCore::MediaPlayerPrivate::bytesLoaded):
(WebCore::MediaPlayerPrivate::totalBytes):
(WebCore::MediaPlayerPrivate::setRate):
(WebCore::MediaPlayerPrivate::setVolume):
(WebCore::MediaPlayerPrivate::supportsMuting):
(WebCore::MediaPlayerPrivate::setMuted):
(WebCore::MediaPlayerPrivate::networkState):
(WebCore::MediaPlayerPrivate::readyState):
(WebCore::MediaPlayerPrivate::setVisible):
(WebCore::MediaPlayerPrivate::mediaStatusChanged):
(WebCore::MediaPlayerPrivate::handleError):
(WebCore::MediaPlayerPrivate::stateChanged):
(WebCore::MediaPlayerPrivate::nativeSizeChanged):
(WebCore::MediaPlayerPrivate::queuedSeekTimeout):
(WebCore::MediaPlayerPrivate::seekTimeout):
(WebCore::MediaPlayerPrivate::positionChanged):
(WebCore::MediaPlayerPrivate::durationChanged):
(WebCore::MediaPlayerPrivate::volumeChanged):
(WebCore::MediaPlayerPrivate::mutedChanged):
(WebCore::MediaPlayerPrivate::updateStates):
(WebCore::MediaPlayerPrivate::setSize):
(WebCore::MediaPlayerPrivate::naturalSize):
(WebCore::MediaPlayerPrivate::paint):
(WebCore::MediaPlayerPrivate::repaint):

  • platform/graphics/qt/MediaPlayerPrivateQt.h: Added new QtMultimedia Backend.

(WebCore::MediaPlayerPrivate::isAvailable):
(WebCore::MediaPlayerPrivate::supportsFullscreen):

  • platform/qt/RenderThemeQt.cpp:

(WebCore::RenderThemeQt::mediaControlsBaselineOpacity): New method.
(WebCore::RenderThemeQt::paintMediaBackground): Background depends on baseline opacity.
(WebCore::RenderThemeQt::paintMediaMuteButton): Changed styling.
(WebCore::RenderThemeQt::paintMediaCurrentTime): Added current time display.
(WebCore::RenderThemeQt::formatMediaControlsCurrentTime): Added time formatting.
(WebCore::RenderThemeQt::formatMediaControlsRemainingTime): Added time formatting.
(WebCore::RenderThemeQt::paintMediaVolumeSliderTrack): Volume slider added.
(WebCore::RenderThemeQt::paintMediaVolumeSliderThumb): Volume slider added.
(WebCore::RenderThemeQt::paintMediaSliderTrack): Updated for QtMultimedia.
(WebCore::RenderThemeQt::paintMediaSliderThumb): Dropped an unnecessary check.
(WebCore::RenderThemeQt::adjustSliderThumbSize): Handle a missing case.

  • platform/qt/RenderThemeQt.h: Reimplemented a few more methods.
05:02 Changeset [55078] by apavlov@chromium.org

2010-02-22 Alexander Pavlov <apavlov@chromium.org>

Not reviewed: mac build fix

  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::ScriptDebugServer):
04:14 Changeset [55077] by apavlov@chromium.org

2010-02-22 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: there should be a way to "deactivate" or "skip" all breakpoints while debugging.
https://bugs.webkit.org/show_bug.cgi?id=33217

WebCore:

  • English.lproj/localizedStrings.js:
  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::ScriptDebugServer): (WebCore::ScriptDebugServer::hasBreakpoint): (WebCore::ScriptDebugServer::setBreakpointsActivated):
  • bindings/js/ScriptDebugServer.h:
  • bindings/v8/ScriptDebugServer.h: (WebCore::ScriptDebugServer::setBreakpointsActivated):
  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::activateBreakpoints): (WebCore::InspectorBackend::deactivateBreakpoints):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/front-end/Images/deactivateBreakpointsButtonGlyph.png: Added.
  • inspector/front-end/Images/deactivateBreakpointsDisabledButtonGlyph.png: Added.
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.activateBreakpoints): (.WebInspector.InspectorBackendStub.prototype.deactivateBreakpoints):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.get breakpointsActivated): (WebInspector.ScriptsPanel.prototype.addBreakpoint): (WebInspector.ScriptsPanel.prototype._stepOutClicked): (WebInspector.ScriptsPanel.prototype._toggleBreakpointsClicked):
  • inspector/front-end/inspector.css:
  • inspector/front-end/textViewer.css:

WebKit/chromium:

  • src/js/DebuggerAgent.js: (devtools.DebuggerAgent): (devtools.DebuggerAgent.prototype.setBreakpointsActivated): (devtools.DebuggerAgent.prototype.handleBreakEvent_): (devtools.DebuggerAgent.prototype.handleExceptionEvent_):
  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl.prototype.activateBreakpoints): (devtools.InspectorBackendImpl.prototype.deactivateBreakpoints):
03:18 Changeset [55076] by mrowe@apple.com

REGRESSION (r55039): Animation starts from near end when loaded over slow network
<http://webkit.org/b/35222> / <rdar://problem/7673523>

Rubber-stamped by Sam Weinig and Maciej Stachowiak.

Roll out r55039 (related to <http://webkit.org/b/35115>) as it causes animated GIFs
to skip intermediate frames when loading over a slower network.

  • platform/graphics/BitmapImage.cpp:

(WebCore::BitmapImage::startAnimation):

02:31 Changeset [55075] by mjs@apple.com

Not reviewed, build fix.

Revert the previous change.

01:28 Changeset [55074] by mjs@apple.com

2010-02-21 Maciej Stachowiak <mjs@apple.com>

Reviewed by Darin Adler.

Cache JavaScript wrappers inline in DOM nodes
https://bugs.webkit.org/show_bug.cgi?id=35226
<rdar://problem/7664202>

8.8% speedup on Dromaeo DOM Core tests.
3.3% speedup on Hixie DOM Core tests.


  • bindings/js/JSDOMBinding.cpp: (WebCore::forgetDOMNode): Clear wrapper pointer. (WebCore::cacheDOMNodeWrapper): Cache inline too if caching for normal world.
  • bindings/js/JSDOMBinding.h: (WebCore::DOMObjectWrapperMapFor):
  • bindings/js/JSDocumentCustom.cpp: (WebCore::toJS): Remove unneeded argument from getCachedDOMNodeWrapper.
  • bindings/js/JSNodeCustom.cpp: (WebCore::createWrapperInline): Renamed version of original createWrapper. (WebCore::createWrapper): Call createWrapperInline. Out-of-line version. (WebCore::toJSNewlyCreated): Call createWrapperInline instead of createWrapper.
  • bindings/js/JSNodeCustom.h: Added. (WebCore::getCachedDOMNodeWrapper): Moved here so it can be inlined. (WebCore::toJS): Moved here so it can be inlined.
  • bindings/js/ScriptWrappable.h: (WebCore::ScriptWrappable::ScriptWrappable): Implement this in the obvious way for JavaScriptCore. (WebCore::ScriptWrappable::wrapper): (WebCore::ScriptWrappable::setWrapper): (WebCore::ScriptWrappable::clearWrapper):
  • bindings/scripts/CodeGeneratorJS.pm: Include CustomHeader heaaders in the header, not just the impl file, so they can add inlining.
  • dom/Node.idl: Add CustomHeader directive.


Add new files to build.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
01:13 Changeset [55073] by pfeldman@chromium.org

2010-02-22 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: windows build fix.

00:08 Changeset [55072] by bweinstein@apple.com

Part 2 of Windows build fix.

Note: See TracTimeline for information about the timeline view.