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

Timeline



May 20, 2010:

11:17 PM Changeset in webkit [59916] by dbates@webkit.org
  • 2 edits in trunk/WebKitSite

2010-05-20 Daniel Bates <dbates@rim.com>

Rubber-stamped by Simon Fraser.

Spelling correction; change "acces" [sic] to "access".

  • quality/lifecycle.html:
11:10 PM Changeset in webkit [59915] by Philippe Normand
  • 5 edits
    2 adds in trunk/WebCore

2010-05-18 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Carlson.

[GStreamer] PlatformMedia support
https://bugs.webkit.org/show_bug.cgi?id=39300

Basic support for PlatformMedia in the gst player. The
PlatformMedia object contains a GStreamerGWorld object. That
object is empty for now. XOverlay features will be added later
on for easy review sake.

  • GNUmakefile.am:
  • platform/graphics/MediaPlayer.h: (WebCore::PlatformMedia::):
  • platform/graphics/gstreamer/GStreamerGWorld.cpp: Added. (WebCore::GStreamerGWorld::createGWorld): (WebCore::GStreamerGWorld::GStreamerGWorld): (WebCore::GStreamerGWorld::~GStreamerGWorld):
  • platform/graphics/gstreamer/GStreamerGWorld.h: Added.
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivateGStreamer::platformMedia): (WebCore::MediaPlayerPrivateGStreamer::createGSTPlayBin):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
10:58 PM Changeset in webkit [59914] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-05-20 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Add dummy SSLKeyGenerator
https://bugs.webkit.org/show_bug.cgi?id=39403

Add dummy implementation of SSLKeyGenerator.

  • platform/brew/SSLKeyGeneratorBrew.cpp: Added. (WebCore::WebCore::getSupportedKeySizes): (WebCore::WebCore::signedPublicKeyAndChallengeString):
10:23 PM Changeset in webkit [59913] by mrowe@apple.com
  • 4 edits in trunk

<rdar://problem/7848154> Remove the dependency on Foundation's private COCOA_FORMAL_PROTOCOLS_2 define.

Rubber-stamped by Dan Bernstein.

WebCore:

  • platform/mac/EmptyProtocolDefinitions.h:

WebKitTools:

  • DumpRenderTree/mac/LayoutTestControllerMac.mm:
10:15 PM Changeset in webkit [59912] by Simon Fraser
  • 3 edits in trunk/WebKit/win

2010-05-20 Simon Fraser <Simon Fraser>

Build fix, no review.

Fix the non-accelerated-compositing Windows build with some
#if USE(ACCELERATED_COMPOSITING) loving.

  • WebView.cpp:
  • WebView.h:
10:09 PM Changeset in webkit [59911] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebCore

2010-05-20 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Add Editor
https://bugs.webkit.org/show_bug.cgi?id=39404

Implement Editor::newGeneralClipboard.

  • editing/brew/EditorBrew.cpp: Added. (WebCore::Editor::newGeneralClipboard):
9:49 PM Changeset in webkit [59910] by Simon Fraser
  • 6 edits in trunk

2010-05-20 Simon Fraser <Simon Fraser>

Reviewed by Adam Roben.

Avoid flushing CA layers when a layout is pending
https://bugs.webkit.org/show_bug.cgi?id=39463
<rdar://problem/7999463>

Avoid rendering the compositing layers to the screen if there's a layout pending,
since the layer tree not in a state that should be presented to the user.

This fixes flashes in some types of content that dynamically add and remove layers.

Do this by adding a WKCACFLayerRendererClient for WKCACFLayerRenderer, which it can
call through to ask whether now is a good time to render. If not, it schedules
another render soon.

  • platform/graphics/win/WKCACFLayerRenderer.h: (WebCore::WKCACFLayerRendererClient::~WKCACFLayerRendererClient):
  • platform/graphics/win/WKCACFLayerRenderer.cpp: (WebCore::WKCACFLayerRenderer::acceleratedCompositingAvailable): (WebCore::WKCACFLayerRenderer::create): (WebCore::WKCACFLayerRenderer::WKCACFLayerRenderer): (WebCore::WKCACFLayerRenderer::render):
9:41 PM Changeset in webkit [59909] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-20 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Don't include CoreFoundation.h when building for BREW MP
https://bugs.webkit.org/show_bug.cgi?id=39405

Add !defined(BUILDING_BREWMP) guard for CoreFoundation.h.

  • WebCorePrefix.h:
9:19 PM Changeset in webkit [59908] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-05-20 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Use skia as BREW MP's graphics backend.
https://bugs.webkit.org/show_bug.cgi?id=39407

Define WTF_PLATFORM_SKIA=1 for BREW MP.

  • wtf/Platform.h:
8:56 PM Changeset in webkit [59907] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebCore

2010-05-20 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Add ScrollTheme
https://bugs.webkit.org/show_bug.cgi?id=39406

As BREW MP always uses frame flattening, there is no need to render scorllbars.
Use the default scrollbar theme with thickness 0.

  • platform/brew/ScrollbarThemeBrew.cpp: Added. (WebCore::ScrollbarTheme::nativeTheme): (WebCore::ScrollbarThemeBrew::~ScrollbarThemeBrew):
  • platform/brew/ScrollbarThemeBrew.h: Added.
8:42 PM Changeset in webkit [59906] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-05-20 Tony Gentilcore <tonyg@chromium.org>

Reviewed by Daniel Bates.

Look in /proc/registry64 for the Platform SDK on 64-bit Windows.
https://bugs.webkit.org/show_bug.cgi?id=39296

The build-webkit script failed for me on Vista 64. A web search turned
up this blog post with a patch that worked for me:
http://www.nicholaswilson.me.uk/2010/04/hacking-webkit-fail/

  • Scripts/webkitdirs.pm:
8:39 PM Changeset in webkit [59905] by ggaren@apple.com
  • 3 edits in trunk/JavaScriptCore

Fixed naming mixup: it's op_call_put_result, not ret_result.

Reviewed by Beth Dakin.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::dump):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

8:20 PM Changeset in webkit [59904] by jhoneycutt@apple.com
  • 4 edits
    1 add in trunk/WebCore

REGRESSION(r53637): DivX plug-in fails to start until window is resized
https://bugs.webkit.org/show_bug.cgi?id=39457
<rdar://problem/8006102>

Before r53637, we called Widget::move() (which caused us to call
NPP_SetWindow) before painting a plug-in. r53637 removed the call to
Widget::move(), and we end up never calling NPP_SetWindow for plug-ins
with the "DeferFirstSetWindowCall" quirk, including the DivX plug-in.

Reviewed by Simon Fraser.

  • manual-tests/divx-plugin-fails-to-draw.html: Added.
  • plugins/PluginView.cpp:

(WebCore::PluginView::PluginView):
Initialize new member to false.

  • plugins/PluginView.h:

Added new member.

  • plugins/win/PluginViewWin.cpp:

(WebCore::PluginView::paint):
Call setNPWindowRect() before painting to ensure that NPP_SetWindow gets
called.
(WebCore::PluginView::setNPWindowRect):
Set m_haveCalledSetWindow.

8:06 PM Changeset in webkit [59903] by ukai@chromium.org
  • 7 edits in trunk

2010-05-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

WebSocket handshake incompatible change in draft-hixie-thewebsocketprotocol-76
https://bugs.webkit.org/show_bug.cgi?id=35572

Fix LayoutTests for new WebSocket protocol.
It requires https://bugs.webkit.org/show_bug.cgi?id=38034 to pass websocket tests.

  • websocket/tests/handshake-error-expected.txt: Update expected data.
  • websocket/tests/handshake-error_wsh.py: Without CRLF, the wsh will sends "ThisWillCauseHandshakeErrorHTTP/1.1 101 WebSocket Protocol Handshake\r\n", which is legal for the first line of draft 76 WebSocket opening handshake. (4.1 Opening Handhshake, step 28 to 30). To fail handshake, it needs \r\n.
  • websocket/tests/long-invalid-header-expected.txt: Originally, it expects Upgrade header after status line. In draft 76, order is not important. But "pppp..\r\n" would fail because it misses ":" (end of name) before \r.

2010-05-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

WebSocket handshake incompatible change in draft-hixie-thewebsocketprotocol-76
https://bugs.webkit.org/show_bug.cgi?id=35572

WebSocket opening handshake is changed. New protocol draft could be found at http://www.whatwg.org/specs/web-socket-protocol/
It requires https://bugs.webkit.org/show_bug.cgi?id=38034 to pass websocket tests.

  • websockets/WebSocketHandshake.cpp: (WebCore::extractResponseCode): add lineLength parameter to return length of status line. (WebCore::hostName): Added. (WebCore::generateSecWebSocketKey): Added. (WebCore::generateKey3): Added. (WebCore::setChallengeNumber): Added. (WebCore::generateChallengeResponseExpected): Added. (WebCore::WebSocketHandshake::WebSocketHandshake): generate challenge response key and expected data. (WebCore::WebSocketHandshake::clientLocation): use hostName. (WebCore::WebSocketHandshake::clientHandshakeMessage): changed for draft 76 spec. (WebCore::WebSocketHandshake::clientHandshakeRequest): (WebCore::WebSocketHandshake::readServerHandshake): changed for draft 76 spec. m_mode is managed in this method. (WebCore::WebSocketHandshake::readHTTPHeaders): change error log messages. (WebCore::WebSocketHandshake::processHeaders): chagned for draft 76 spec. (WebCore::WebSocketHandshake::checkResponseHeaders): return boolean whether response header is ok or not and not change m_mode in it.
  • websockets/WebSocketHandshake.h:
8:04 PM Changeset in webkit [59902] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-05-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

ThreadedMessageQueue should use with_statement for exception safety
https://bugs.webkit.org/show_bug.cgi?id=39233

  • Scripts/webkitpy/common/thread/threadedmessagequeue.py:
7:52 PM Changeset in webkit [59901] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-05-20 Viet-Trung Luu <viettrungluu@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Don't rely on focused node to route mouse events to plugins.
https://bugs.webkit.org/show_bug.cgi?id=39375
http://crbug.com/43935

On mouse down, do a hit test and check if a plugin was hit. If so, then
save the node for future mouse events (until mouse up or capture lost).

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl): (WebKit::WebViewImpl::mouseDown): (WebKit::WebViewImpl::handleInputEvent): (WebKit::WebViewImpl::mouseCaptureLost):
  • src/WebViewImpl.h: Remove m_haveMouseCapture; add m_mouseCaptureNode.
7:44 PM Changeset in webkit [59900] by victorw@chromium.org
  • 2 edits
    4 adds in trunk/LayoutTests

2010-05-20 Victor Wang <victorw@chromium.org>

Unreviewed. Update test expectations for chromium.

  • platform/chromium-linux/fast/css/font-face-in-shadow-DOM-expected.checksum: Added.
  • platform/chromium-linux/fast/css/font-face-in-shadow-DOM-expected.png: Added.
  • platform/chromium-win/fast/css/font-face-in-shadow-DOM-expected.checksum: Added.
  • platform/chromium-win/fast/css/font-face-in-shadow-DOM-expected.png: Added.
  • platform/chromium/test_expectations.txt:
7:34 PM Changeset in webkit [59899] by eric@webkit.org
  • 4 edits in trunk/LayoutTests

2010-05-20 Zhenyao Mo <zmo@google.com>

Reviewed by Dimitri Glazkov.

fast/canvas/webgl/texture-npot.html failed on leopard bot
https://bugs.webkit.org/show_bug.cgi?id=39128

  • fast/canvas/webgl/texture-npot-expected.txt: Adding the failed test cases back.
  • fast/canvas/webgl/texture-npot.html:
  • platform/mac-leopard/Skipped: Skip texture-npot.html on leopard for now.
7:19 PM Changeset in webkit [59898] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-05-20 Dominic Mazzoni <dmazzoni@chromium.org>

Reviewed by Darin Fisher.

Add headingLevel and setFocused methods.

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

  • public/WebAccessibilityObject.h:
  • src/WebAccessibilityObject.cpp: (WebKit::WebAccessibilityObject::headingLevel): (WebKit::WebAccessibilityObject::setFocused):
7:14 PM Changeset in webkit [59897] by Adam Roben
  • 2 edits in trunk/WebCore

Stop leaking all CACFContexts

Fixes <http://webkit.org/b/39466> <rdar://problem/8007141>.

Reviewed by Simon Fraser.

  • platform/graphics/win/WKCACFContextFlusher.cpp:

(WebCore::WKCACFContextFlusher::addContext): Only retain the context
when we first add it to the m_contexts set. Otherwise we'll retain the
same context multiple times, causing it to leak.

7:02 PM Changeset in webkit [59896] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2010-05-20 Diego Gonzalez <diegohcg@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QtTestBrowser does not support websites which requires HTTP Authentication via dialogs
https://bugs.webkit.org/show_bug.cgi?id=38456

  • QtTestBrowser/webpage.cpp: (WebPage::WebPage): (WebPage::authenticationRequired):
  • QtTestBrowser/webpage.h:
7:01 PM Changeset in webkit [59895] by ggaren@apple.com
  • 6 edits in trunk/JavaScriptCore

2010-05-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Whittling away at CodeBlock use: Removed a little wonkiness in closure lookup
https://bugs.webkit.org/show_bug.cgi?id=39444


Calculate all lookup depths at compile time. I'm not sure why we couldn't
do this before, but whatever the reason, it seems to be gone now.

  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::findScopedProperty):
  • bytecompiler/BytecodeGenerator.h:
  • interpreter/Interpreter.cpp: (JSC::Interpreter::resolveSkip): (JSC::Interpreter::resolveGlobalDynamic): (JSC::Interpreter::privateExecute):
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_get_scoped_var): (JSC::JIT::emit_op_put_scoped_var): (JSC::JIT::emit_op_resolve_skip): (JSC::JIT::emit_op_resolve_global_dynamic): (JSC::JIT::emitSlow_op_resolve_global_dynamic):
  • jit/JITOpcodes32_64.cpp: (JSC::JIT::emit_op_get_scoped_var): (JSC::JIT::emit_op_put_scoped_var): (JSC::JIT::emit_op_resolve_skip):
6:45 PM Changeset in webkit [59894] by eric@webkit.org
  • 2 edits
    1 add in trunk/WebCore

2010-05-20 Young Han Lee <joybro@company100.net>

Reviewed by Darin Adler.

Create filenameFromString for all ports except GTK and EFL.
https://bugs.webkit.org/show_bug.cgi?id=39348

  • platform/FileSystem.cpp: Added. (WebCore::filenameFromString):
  • platform/FileSystem.h:
6:42 PM Changeset in webkit [59893] by mrowe@apple.com
  • 5 edits in branches/safari-533-branch

Versioning.

6:41 PM Changeset in webkit [59892] by mrowe@apple.com
  • 1 copy in tags/Safari-533.10

New tag.

6:40 PM Changeset in webkit [59891] by mthole@apple.com
  • 2 edits in trunk/WebKit/mac

When using a scale factor > 1, scrollbars sometimes appear when not necessary
https://bugs.webkit.org/show_bug.cgi?id=39458

Reviewed by Dave Hyatt.

  • WebView/WebDynamicScrollBarsView.mm:

(-[WebDynamicScrollBarsView updateScrollers]): Round up non-integral sizes from AppKit
so that they can be compared against the integral document size.

6:19 PM Changeset in webkit [59890] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-05-20 Young Han Lee <joybro@company100.net>

Reviewed by Darin Adler.

Port SharedBuffer to POSIX.
https://bugs.webkit.org/show_bug.cgi?id=39283

  • platform/posix/SharedBufferPOSIX.cpp: Added. (WebCore::SharedBuffer::createWithContentsOfFile):
6:19 PM Changeset in webkit [59889] by mrowe@apple.com
  • 13 edits
    4 adds in branches/safari-533-branch

Merge r59876.

6:06 PM Changeset in webkit [59888] by eric@webkit.org
  • 7 edits in trunk

2010-05-20 Zhenyao Mo <zmo@google.com>

Reviewed by Dimitri Glazkov.

Map RGBA4/RGB5_A1 to RGBA and RGB565 to RGB in RenderbufferStorage
https://bugs.webkit.org/show_bug.cgi?id=39077

  • fast/canvas/webgl/framebuffer-object-attachment-expected.txt: Add the test cases for color renderbuffer creation and attachment to framebuffer.
  • fast/canvas/webgl/framebuffer-object-attachment.html: Ditto.

2010-05-20 Zhenyao Mo <zmo@google.com>

Reviewed by Dimitri Glazkov.

Map RGBA4/RGB5_A1 to RGBA and RGB565 to RGB in RenderbufferStorage
https://bugs.webkit.org/show_bug.cgi?id=39077

  • platform/graphics/mac/GraphicsContext3DMac.cpp: (WebCore::GraphicsContext3D::renderbufferStorage): Mapping the enums.

2010-05-20 Zhenyao Mo <zmo@google.com>

Reviewed by Dimitri Glazkov.

Map RGBA4/RGB5_A1 to RGBA and RGB565 to RGB in RenderbufferStorage
https://bugs.webkit.org/show_bug.cgi?id=39077

  • src/WebGraphicsContext3DDefaultImpl.cpp: (WebKit::WebGraphicsContext3DDefaultImpl::renderbufferStorage): Mapping the enums.
6:05 PM Changeset in webkit [59887] by mrowe@apple.com
  • 3 edits in branches/safari-533-branch/WebKit/mac

Merge r59793.

6:05 PM Changeset in webkit [59886] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59853.

6:05 PM Changeset in webkit [59885] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59873.

6:04 PM Changeset in webkit [59884] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59871.

6:04 PM Changeset in webkit [59883] by mrowe@apple.com
  • 5 edits in branches/safari-533-branch/WebCore

Merge r59854.

6:04 PM Changeset in webkit [59882] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59864.

6:04 PM Changeset in webkit [59881] by mrowe@apple.com
  • 4 edits
    2 adds in branches/safari-533-branch

Merge r59795.

6:04 PM Changeset in webkit [59880] by mrowe@apple.com
  • 2 edits
    3 adds in branches/safari-533-branch

Merge r59859.

6:04 PM Changeset in webkit [59879] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59732.

6:04 PM Changeset in webkit [59878] by mrowe@apple.com
  • 4 edits in branches/safari-533-branch/WebCore

Merge r59731.

6:04 PM Changeset in webkit [59877] by mrowe@apple.com
  • 5 edits
    3 adds in branches/safari-533-branch

Merge r59798.

5:37 PM Changeset in webkit [59876] by mitz@apple.com
  • 13 edits
    4 adds in trunk

<rdar://problem/8007953> Textarea using custom font appears blank

Reviewed by Dave Hyatt.

WebCore:

Test: fast/css/font-face-in-shadow-DOM.html

When a remote font is loaded, CSSFontSelector forces a style recalc, which replaces all
RenderSyles that have FontFallbackLists referencing the placeholder font with fresh
RenderStyles. However, it does not descend into shadow DOM trees, so those may end up with
styles that still reference the placeholder font.

The fix is to add RenderObject::requiresForcedStyleRecalcPropagation() and have it return
true from renderers that maintain shadow DOM trees or otherwise keep their own RenderStyles.

  • dom/Element.cpp:

(WebCore::Element::recalcStyle): Check if forced style recalc needs to propagated.

  • rendering/RenderButton.h:

(WebCore::RenderButton::requiresForcedStyleRecalcPropagation):

  • rendering/RenderDataGrid.h:

(WebCore::RenderDataGrid::requiresForcedStyleRecalcPropagation):

  • rendering/RenderFileUploadControl.h:

(WebCore::RenderFileUploadControl::requiresForcedStyleRecalcPropagation):

  • rendering/RenderListItem.h:

(WebCore::RenderListItem::requiresForcedStyleRecalcPropagation):

  • rendering/RenderMedia.h:

(WebCore::RenderMedia::requiresForcedStyleRecalcPropagation):

  • rendering/RenderMenuList.h:

(WebCore::RenderMenuList::RenderMenuList::requiresForcedStyleRecalcPropagation):

  • rendering/RenderObject.h:

(WebCore::RenderObject::requiresForcedStyleRecalcPropagation):

  • rendering/RenderProgress.h:

(WebCore::RenderProgress::requiresForcedStyleRecalcPropagation):

  • rendering/RenderSlider.h:

(WebCore::RenderSlider::requiresForcedStyleRecalcPropagation):

  • rendering/RenderTextControl.h:

(WebCore::RenderTextControl::requiresForcedStyleRecalcPropagation):

LayoutTests:

  • fast/css/font-face-in-shadow-DOM.html: Added.
  • platform/mac/fast/css/font-face-in-shadow-DOM-expected.checksum: Added.
  • platform/mac/fast/css/font-face-in-shadow-DOM-expected.png: Added.
  • platform/mac/fast/css/font-face-in-shadow-DOM-expected.txt: Added.
4:58 PM Changeset in webkit [59875] by mrowe@apple.com
  • 10 edits in branches/safari-533-branch/WebCore

Merge r59776.

4:57 PM Changeset in webkit [59874] by mrowe@apple.com
  • 11 edits
    1 move
    1 delete in branches/safari-533-branch/WebCore

Merge r59677.

4:02 PM Changeset in webkit [59873] by jer.noble@apple.com
  • 2 edits in trunk/WebCore

No review; build fix only.

When WTF_USE_ACCELERATED_COMPOSITING is turned off, MediaPlayerPrivateQuickTimeVisualContext.cpp
must include CoreGraphics/CGContext.h.

  • platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
3:51 PM Changeset in webkit [59872] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Not reviewed.

Tests that use Geolocation.getCurrentPosition fail
https://bugs.webkit.org/show_bug.cgi?id=39454

  • platform/gtk/Skipped: Skip failing tests until this bug is resolved.
3:41 PM Changeset in webkit [59871] by jer.noble@apple.com
  • 2 edits in trunk/WebCore

2010-05-20 Jer Noble <jer.noble@apple.com>

Reviewed by Sam Weinig.

Video elements show no video on Windows machines that do not support accelerated compositing
https://bugs.webkit.org/show_bug.cgi?id=39446
rdar://problem/7999794

Implement the paint() method in MediaPlayerPrivateQuickTimeVisualContext. The visual context should
be set up in load() and torn down in the destructor (as opposed to setUpVideoRendering and
tearDownVideoRendering, which won't get called in a non-accelerated compositing case).


  • platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp: (WebCore::MediaPlayerPrivateQuickTimeVisualContext::~MediaPlayerPrivateQuickTimeVisualContext): (WebCore::MediaPlayerPrivateQuickTimeVisualContext::load): (WebCore::MediaPlayerPrivateQuickTimeVisualContext::paint): (WebCore::MediaPlayerPrivateQuickTimeVisualContext::visualContextTimerFired): (WebCore::MediaPlayerPrivateQuickTimeVisualContext::retrieveCurrentImage): (WebCore::MediaPlayerPrivateQuickTimeVisualContext::setUpVideoRendering): (WebCore::MediaPlayerPrivateQuickTimeVisualContext::tearDownVideoRendering):
3:32 PM Changeset in webkit [59870] by victorw@chromium.org
  • 5 edits in trunk

2010-05-20 Abhishek Arya <inferno@chromium.org>

Reviewed by Adam Barth.

Added method to WebSecurityOrigin for invoking
SecurityOrigin::canAccessPasswordManager.
https://bugs.webkit.org/show_bug.cgi?id=38916

  • public/WebSecurityOrigin.h:
  • src/WebSecurityOrigin.cpp: (WebKit::WebSecurityOrigin::canAccessPasswordManager):

2010-05-20 Abhishek Arya <inferno@chromium.org>

Reviewed by Adam Barth.

Added a method to check if the security origin can access
password manager.
https://bugs.webkit.org/show_bug.cgi?id=38916

  • page/SecurityOrigin.h: (WebCore::SecurityOrigin::canAccessPasswordManager):
3:27 PM Changeset in webkit [59869] by victorw@chromium.org
  • 2 edits
    4 adds in trunk/LayoutTests

2010-05-20 Victor Wang <victorw@chromium.org>

Unreviewed. Update test expectations for chromium.

  • platform/chromium-mac/svg/filters/feLighting-crash-expected.checksum: Added.
  • platform/chromium-mac/svg/filters/feLighting-crash-expected.png: Added.
  • platform/chromium-win/svg/filters/feLighting-crash-expected.checksum: Added.
  • platform/chromium-win/svg/filters/feLighting-crash-expected.png: Added.
  • platform/chromium/test_expectations.txt:
3:17 PM Changeset in webkit [59868] by Darin Adler
  • 12 edits in trunk

2010-05-20 Darin Adler <Darin Adler>

Reviewed by Eric Seidel.

Fix warnings seen on the buildbots today
https://bugs.webkit.org/show_bug.cgi?id=39368

  • profiler/ProfileNode.cpp: (JSC::getCount): Removed unneeded initialization of global variable. These are all initialized to zero anyway, so no need to try to initialize it and get a warning.
  • wtf/CurrentTime.cpp: (WTF::currentTime): Removed unused global variable.

2010-05-20 Darin Adler <Darin Adler>

Reviewed by Eric Seidel.

Fix warnings seen on the buildbots today
https://bugs.webkit.org/show_bug.cgi?id=39368

  • loader/icon/IconDatabase.cpp: (WebCore::IconDatabase::pruneUnretainedIcons): Use the type "long long" for things being printed with "%lli". There is no guaranteed that int64_t is the same thing as long long.
  • page/EventHandler.cpp: (WebCore::EventHandler::handleMousePressEvent): Added parentheses as suggested by warnings in newer versions of GCC.
  • platform/ContextMenu.cpp: Fix #if around the functions to match the #if around the call sites to avoid unused function warning.
  • platform/graphics/gtk/ImageGtk.cpp: (WebCore::Image::loadPlatformResource): Use NULL to end a variable argument list rather than 0. This is a case where the WebKit "use 0 instead of NULL" guideline is incorrect and won't work properly.
  • platform/win/SystemTimeWin.cpp: (WebCore::userIdleTime): Removed unnecessary initialization of LASTINPUTINFO.
  • platform/win/WebCoreInstanceHandle.h: Added property svn:eol-style. Made functions inline instead of static. It's not correct to have functions with internal linkage in a header file. Marking them inline makes more sense.
  • plugins/win/PluginMessageThrottlerWin.cpp: (WebCore::PluginMessageThrottlerWin::PluginMessageThrottlerWin): Reordered the initializers to match the order of the data members from the class definition.
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginView::paintIntoTransformedContext): Initialized all the fields of WINDOWPOS. I know this goes against the traditions of Windows programming, but it's correct and silences the warning. Also removed the unneeded separate code path for non-WinCE platforms that translates only the corner of the window rectangle; the WinCE version is correct for normal Windows as well. Not sure why this wasn't caught when the code was originally checked in.
3:05 PM Changeset in webkit [59867] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Not reviewed.

https://bugs.webkit.org/show_bug.cgi?id=39388
Re-enable geolocation tests on GTK+ now that this bug is closed.

  • platform/gtk/Skipped: Re-enable geolocation tests.
2:56 PM Changeset in webkit [59866] by yaar@chromium.org
  • 29 edits
    2 adds in trunk

2010-05-20 Justin Schuh <jschuh@chromium.org>

Reviewed by Adam Barth.

Moving frame.src checks out of the bindings
https://bugs.webkit.org/show_bug.cgi?id=37815

Moved JavaScript frame.src checks out of bindings and into
HTMLFrameElementBase. Added main thread state stack to JavaScriptCore
so ExecState is available inside core DOM. Updated affected bindings
(except for GObject, which will need to be updated to avoid origin
failures inside native code).

  • Android.jscbindings.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSCallbackData.cpp: (WebCore::JSCallbackData::invokeCallback):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::handleEvent):
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript):
  • bindings/js/JSMainThreadExecState.cpp: Added.
  • bindings/js/JSMainThreadExecState.h: Added. (WebCore::JSMainThreadExecState::currentState): (WebCore::JSMainThreadExecState::call): (WebCore::JSMainThreadExecState::evaluate): (WebCore::JSMainThreadExecState::JSMainThreadExecState): (WebCore::JSMainThreadExecState::~JSMainThreadExecState): (WebCore::JSMainThreadNullState::JSMainThreadNullState):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::executeFunctionInContext): (WebCore::ScheduledAction::execute):
  • bindings/js/ScheduledAction.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld): (WebCore::ScriptController::canAccessFromCurrentOrigin):
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptFunctionCall.cpp: (WebCore::ScriptFunctionCall::call):
  • bindings/objc/ObjCEventListener.mm:
  • bindings/objc/WebScriptObject.mm: (-[WebScriptObject callWebScriptMethod:withArguments:]): (-[WebScriptObject evaluateWebScript:]):
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/test/ObjC/DOMTestCallback.mm: (-[DOMTestCallback callbackWithClass1Param:]): (-[DOMTestCallback callbackWithClass2Param:strArg:]): (-[DOMTestCallback callbackWithNonBoolReturnType:]): (-[DOMTestCallback customCallback:class6Param:]):
  • bindings/scripts/test/ObjC/DOMTestInterface.mm:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj readOnlyIntAttr]): (-[DOMTestObj readOnlyStringAttr]): (-[DOMTestObj readOnlyTestObjAttr]): (-[DOMTestObj intAttr]): (-[DOMTestObj setIntAttr:]): (-[DOMTestObj longLongAttr]): (-[DOMTestObj setLongLongAttr:]): (-[DOMTestObj unsignedLongLongAttr]): (-[DOMTestObj setUnsignedLongLongAttr:]): (-[DOMTestObj stringAttr]): (-[DOMTestObj setStringAttr:]): (-[DOMTestObj testObjAttr]): (-[DOMTestObj setTestObjAttr:]): (-[DOMTestObj attrWithException]): (-[DOMTestObj setAttrWithException:]): (-[DOMTestObj attrWithSetterException]): (-[DOMTestObj setAttrWithSetterException:]): (-[DOMTestObj attrWithGetterException]): (-[DOMTestObj setAttrWithGetterException:]): (-[DOMTestObj customAttr]): (-[DOMTestObj setCustomAttr:]): (-[DOMTestObj scriptStringAttr]): (-[DOMTestObj voidMethod]): (-[DOMTestObj voidMethodWithArgs:strArg:objArg:]): (-[DOMTestObj intMethod]): (-[DOMTestObj intMethodWithArgs:strArg:objArg:]): (-[DOMTestObj objMethod]): (-[DOMTestObj objMethodWithArgs:strArg:objArg:]): (-[DOMTestObj methodThatRequiresAllArgs:objArg:]): (-[DOMTestObj methodThatRequiresAllArgsAndThrows:objArg:]): (-[DOMTestObj serializedValue:]): (-[DOMTestObj methodWithException]): (-[DOMTestObj customMethod]): (-[DOMTestObj customMethodWithArgs:strArg:objArg:]): (-[DOMTestObj customArgsAndException:]): (-[DOMTestObj addEventListener:listener:useCapture:]): (-[DOMTestObj removeEventListener:listener:useCapture:]): (-[DOMTestObj withDynamicFrame]): (-[DOMTestObj withDynamicFrameAndArg:]): (-[DOMTestObj withDynamicFrameAndOptionalArg:optionalArg:]): (-[DOMTestObj withDynamicFrameAndUserGesture:]): (-[DOMTestObj withDynamicFrameAndUserGestureASAD:optionalArg:]): (-[DOMTestObj withScriptStateVoid]): (-[DOMTestObj withScriptStateObj]): (-[DOMTestObj withScriptStateVoidException]): (-[DOMTestObj withScriptStateObjException]): (-[DOMTestObj methodWithOptionalArg:]): (-[DOMTestObj methodWithNonOptionalArgAndOptionalArg:opt:]): (-[DOMTestObj methodWithNonOptionalArgAndTwoOptionalArgs:opt1:opt2:]):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::canAccessFromCurrentOrigin):
  • bindings/v8/ScriptController.h:
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::isURLAllowed):

2010-05-20 Justin Schuh <jschuh@chromium.org>

Reviewed by Adam Barth.

Moving frame.src checks out of the bindings
https://bugs.webkit.org/show_bug.cgi?id=37815

  • http/tests/security/xss-DENIED-iframe-src-alias-expected.txt:
  • http/tests/security/xss-DENIED-iframe-src-alias.html:
2:55 PM Changeset in webkit [59865] by kdecker@apple.com
  • 3 edits in trunk/WebKit/mac

Reviewed by Anders Carlsson.

https://bugs.webkit.org/show_bug.cgi?id=39441
<rdar://problem/7985715> Flash context menu is both horizontally and vertically offset from point of click
<rdar://problem/7986109> Youtube video controller UI entirely missing
<rdar://problem/7986154> Flash content paints at horizontally and vertically shifted locations (within the plug-in region)

  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView updateAndSetWindow]): The base coordinates of a window and it's contentView happen to be the equal at a userSpaceScaleFactor of 1. For non-1.0 scale factors this assumption is false. Accordingly we now convert to the window contentView coordinate system when computing bounds in "window" and the visible rect.
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]): Ditto.
2:30 PM Changeset in webkit [59864] by Adam Roben
  • 2 edits in trunk/WebCore

Fix an HRGN leak in WKCACFLayerRenderer

Fixes <http://webkit.org/b/39312> <rdar://problem/7998728> REGRESSION
(r53686-r55990): Graphics corruption when watching video (affects
youtube.com)

Reviewed by Darin Adler.

  • platform/graphics/win/WKCACFLayerRenderer.cpp:

(WebCore::getDirtyRects): Use an OwnPtr to hold the HRGN we allocate.
That way we don't have to remember to destroy it (as we were
forgetting to do) when we bail out of this function early.

2:20 PM Changeset in webkit [59863] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

2010-05-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Darin Adler.

Removed some reliance on callFame[CodeBlock] by storing CodeBlock in a
local variable in the Interpreter.
https://bugs.webkit.org/show_bug.cgi?id=39447

Small speedup on SunSpider in Interpreter mode.

  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
  • jsc.cpp:
  • wtf/Platform.h:
1:49 PM Changeset in webkit [59862] by darin@chromium.org
  • 11 edits in trunk

2010-05-20 Darin Fisher <darin@chromium.org>

Reviewed by Nate Chapin.

[chromium] Provide a way to catch exceptions thrown while interacting
with a NPObject via WebBindings methods.
https://bugs.webkit.org/show_bug.cgi?id=39378

This change also includes a small bit of cleanup in V8NPObject and
friends. I moved code into the WebCore namespace where appropriate.


In _NPN_Invoke, I also moved the call to _NPN_Evaluate outside of the
context scope. I did this to avoid having nested ExceptionCatcher
objects, which is not strictly necessary, but it seemed cleaner.
_NPN_Evaluate already takes care of creating a context scope.

  • bindings/v8/NPV8Object.cpp: (WebCore::v8ObjectToNPObject): (WebCore::npCreateV8ScriptObject): (_NPN_Invoke): (_NPN_InvokeDefault): (_NPN_Evaluate): (_NPN_EvaluateHelper): (_NPN_GetProperty): (_NPN_SetProperty): (_NPN_RemoveProperty): (_NPN_HasProperty): (_NPN_HasMethod): (_NPN_SetException): (_NPN_Enumerate): (_NPN_Construct):
  • bindings/v8/NPV8Object.h:
  • bindings/v8/V8NPObject.cpp:
  • bindings/v8/V8NPObject.h:
  • bindings/v8/V8NPUtils.cpp: (WebCore::convertV8ObjectToNPVariant): (WebCore::pushExceptionHandler): (WebCore::popExceptionHandler): (WebCore::ExceptionCatcher::ExceptionCatcher): (WebCore::ExceptionCatcher::~ExceptionCatcher):
  • bindings/v8/V8NPUtils.h:
  • bindings/v8/npruntime.cpp:
1:20 PM Changeset in webkit [59861] by Martin Robinson
  • 3 edits in trunk/WebKitTools

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Eric Seidel.

editingBehavior settings needs to be set back to a reasonable default between tests
https://bugs.webkit.org/show_bug.cgi?id=39433

For now, hard code the default setting during reset, so that the serialized
version of the setting stays in sync with expectations.

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues): Reset editing behavior to the appropriate platform default.
  • DumpRenderTree/win/DumpRenderTree.cpp: (resetDefaultsToConsistentValues): Ditto.
1:12 PM Changeset in webkit [59860] by barraclough@apple.com
  • 12 edits in trunk/JavaScriptCore

Bug 39408 - Unify JIT/interpreter return behaviour.

Reviewed by Oliver Hunt.

Presently the JIT and the interpreter have differing implementations in how
they handle storing the result of a function call back to the register file.
In both cases the op_call ( / _eval / _varargs / op_construct) opcode has an
operand indicating the VM register in which the result should be stored.
The JIT plants code after the call to store the result, so the value will be
stored as soon as the callee has returned. In the interpreter the call
passes the return value register id into the callee via the callee callframe,
and the callee is responsible for writing the result back into its callers
register file after it has restored the parents callframe pointer, but before
returning.

Instead, move the task of writing the call result to the register file into a
new opcode (op_call_put_result), and after returning the callee should leave
the return value in a canonical location. In the case of the interpreter,
this canonical location is a local variable in privateExecute
(functionReturnValue), in the case of the JIT this is the normal return value
registers (regT0, or regT1:regT0 in JSVALUE32_64). op_call_put_result stores
the result from the canonical location to the registerfile.

In addition to unifying JIT & interpreter behaviour this change allows us to
remove a slot from the callframe, omit the write of the result where the
return value from the call is not used, and provides a 2% speedup on sunspider
in the interpreter.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::dump):

  • bytecode/Opcode.h:
  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitCall):
(JSC::BytecodeGenerator::emitCallVarargs):
(JSC::BytecodeGenerator::emitConstruct):

  • bytecompiler/BytecodeGenerator.h:

(JSC::BytecodeGenerator::finalDestinationOrIgnored):

  • bytecompiler/NodesCodegen.cpp:

(JSC::NewExprNode::emitBytecode):
(JSC::FunctionCallValueNode::emitBytecode):
(JSC::FunctionCallResolveNode::emitBytecode):
(JSC::FunctionCallBracketNode::emitBytecode):
(JSC::FunctionCallDotNode::emitBytecode):
(JSC::CallFunctionCallDotNode::emitBytecode):
(JSC::ApplyFunctionCallDotNode::emitBytecode):

  • interpreter/CallFrame.h:

(JSC::ExecState::init):
(JSC::ExecState::noCaller):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::dumpRegisters):
(JSC::Interpreter::throwException):
(JSC::Interpreter::privateExecute):

  • interpreter/RegisterFile.h:

(JSC::RegisterFile::):

  • jit/JIT.cpp:

(JSC::JIT::privateCompileMainPass):

  • jit/JIT.h:
  • jit/JITCall.cpp:

(JSC::JIT::compileOpCallSetupArgs):
(JSC::JIT::compileOpConstructSetupArgs):
(JSC::JIT::emit_op_call_put_result):
(JSC::JIT::compileOpCallVarargs):
(JSC::JIT::compileOpCallVarargsSlowCase):
(JSC::JIT::compileOpCall):
(JSC::JIT::compileOpCallSlowCase):
(JSC::JIT::compileOpCallVarargsSetupArgs):

1:10 PM Changeset in webkit [59859] by steveblock@google.com
  • 2 edits
    3 adds in trunk

2010-05-20 Marcus Bulach <bulach@chromium.org>

Reviewed by Steve Block.

Ensure timers are stopped on Geolocation::disconnectFrame()
https://bugs.webkit.org/show_bug.cgi?id=39388

fast/dom/Geolocation/notimer-after-unload.html, plus it should be possible to re-enable Gtk LayoutTests.

  • page/Geolocation.cpp: (WebCore::Geolocation::disconnectFrame):
12:44 PM Changeset in webkit [59858] by bfulgham@webkit.org
  • 2 edits in trunk/WebKitTools

Build fix. No review.

The WebKitAPITest targets do not use the "_debug" suffix needed
by the WinCairo port. Added Debug_Cairo target to correct this.

  • WebKitAPITest/WebKitAPITest.vcproj:
12:32 PM Changeset in webkit [59857] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Unreviewed.

editingBehavior settings needs to be set back to a reasonable default between tests
https://bugs.webkit.org/show_bug.cgi?id=39433

I forgot to revert the expected results for this test after reverting it.

  • editing/selection/extend-selection-after-double-click-expected.txt:
12:18 PM Changeset in webkit [59856] by jamesr@google.com
  • 3 edits
    2 adds in trunk

2010-05-20 James Robinson <jamesr@chromium.org>

Reviewed by Eric Seidel.

Ensures styles are up-to-date before determining selection
https://bugs.webkit.org/show_bug.cgi?id=39389

This adds a call to document::updateStyleIfNeeded() to
Frame::notifyRendererOfSelectionChange(). Without this call, if there
is a pending style update that will detach part of the DOM and the
the selection is within that part of the DOM when
notifyRendererOfSeletionChange is called, the rootEditableElement
algorithm ends up walking through detached parts of the render tree.

Test: editing/selection/focus-and-display-none.html

  • page/Frame.cpp: (WebCore::Frame::notifyRendererOfSelectionChange):

2010-05-20 James Robinson <jamesr@chromium.org>

Reviewed by Eric Seidel.

Tests putting focus into a textarea and setting display:none
https://bugs.webkit.org/show_bug.cgi?id=39389

  • editing/selection/focus-and-display-none-expected.txt: Added.
  • editing/selection/focus-and-display-none.html: Added.
12:14 PM Changeset in webkit [59855] by bfulgham@webkit.org
  • 2 edits in trunk/WebKit/win

[WinCairo] Correct scaling for print preview
https://bugs.webkit.org/show_bug.cgi?id=39329

Reviewed by Adam Roben.

Cairo does not properly deal with Windows HDCs that have been
scaled using MM_ANISOTROPIC mapping mode, and a WindowExt and
ViewportExt setting.
(see http://bugs.freedesktop.org/show_bug.cgi?id=28161)

Instead, reset the HDC's WorldTransform to be unscaled, then
scale the cairo context to the desired scaling, and perform
the drawing operation.

  • WebFrame.cpp:

(WebFrame::drawHeader): Use pre-positioned context to simplify

this method.

(WebFrame::drawFooter): Use pre-positioned context to simplify

this method.

(WebFrame::spoolPage): Revise scaling logic to turn off HDC

scaling, and scale using only Cairo. Revert scaling at end
so that user-defined GDI-based routines (e.g., header/footer)
will draw in the right position.

(WebFrame::spoolPages): Identify print preview case, and

retrieve scaling factors from preview context. Set the
Cairo context to use these factors during the spoolPage
operation.

11:51 AM Changeset in webkit [59854] by Adam Roben
  • 5 edits in trunk/WebCore

Remove code that fetched, but did nothing useful with, a CFDictionary in MediaPlayerPrivateQuickTimeVisualContext

I also removed the code in QTPixelBuffer that vended the dictionary.

Fixes <http://webkit.org/b/39435> <rdar://problem/8009278>
QTPixelBuffer passes CFDictionaries across the DLL boundary, which can
lead to crashes

Reviewed by Darin Adler.

  • platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:

(WebCore::MediaPlayerPrivateQuickTimeVisualContext::retrieveCurrentImage):
Deleted code that fetched the attachments dictionary, which was never
used other than to pass it to CFRetain/CFRelease, which could be
enough to cause a crash due to incompatible copies of
CoreFoundation.dll.

  • platform/graphics/win/QTMovieVisualContext.h: Added a now-needed

forward declaration.

  • platform/graphics/win/QTPixelBuffer.cpp:
  • platform/graphics/win/QTPixelBuffer.h:

Removed the now-unused attachments function.

11:51 AM Changeset in webkit [59853] by Adam Roben
  • 2 edits in trunk/WebCore

Fix a CFData leak in MediaPlayerPrivateQuickTimeVisualContext

Fixes <http://webkit.org/b/39432> <rdar://problem/8008992>

Reviewed by Ada Chan

  • platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:

(WebCore::QTCFDictionaryCreateWithDataCallback): Use a RetainPtr to
hold the CFData we allocate so it will get released when this function
is exited. Also pass kCFAllocatorNull as the bytes deallocator to
CFDataCreateWithBytesNoCopy so that CF doesn't try to deallocate the
bytes we pass to it.

11:36 AM Changeset in webkit [59852] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Ojan Vafai.

editingBehavior settings needs to be set back to a reasonable default between tests
https://bugs.webkit.org/show_bug.cgi?id=39433

Revert the changes from r59840 to this test until there is a method
in the WebKit APIs to consistently reset the editingBehavior setting
to a reasonable default.

11:32 AM Changeset in webkit [59851] by jianli@chromium.org
  • 2 edits in trunk/WebCore

Rename GenericTask* to CrossThreadTask* in CrossThreadTask.h.
https://bugs.webkit.org/show_bug.cgi?id=39437

Reviewed by David Levin.

  • dom/CrossThreadTask.h:

(WebCore::):
(WebCore::CrossThreadTask1::create):
(WebCore::CrossThreadTask1::CrossThreadTask1):
(WebCore::CrossThreadTask2::create):
(WebCore::CrossThreadTask2::CrossThreadTask2):
(WebCore::CrossThreadTask3::create):
(WebCore::CrossThreadTask3::CrossThreadTask3):
(WebCore::CrossThreadTask4::create):
(WebCore::CrossThreadTask4::CrossThreadTask4):
(WebCore::CrossThreadTask5::create):
(WebCore::CrossThreadTask5::CrossThreadTask5):
(WebCore::CrossThreadTask6::create):
(WebCore::CrossThreadTask6::CrossThreadTask6):
(WebCore::CrossThreadTask7::create):
(WebCore::CrossThreadTask7::CrossThreadTask7):
(WebCore::CrossThreadTask8::create):
(WebCore::CrossThreadTask8::CrossThreadTask8):
(WebCore::createCallbackTask):

11:31 AM Changeset in webkit [59850] by victorw@chromium.org
  • 2 edits in trunk/LayoutTests

2010-05-20 Victor Wang <victorw@chromium.org>

Unreviewed. Update test expectations from chromium.

  • platform/chromium/test_expectations.txt:
10:57 AM Changeset in webkit [59849] by jianli@chromium.org
  • 16 edits
    1 move in trunk

Rename GenericWorkerTask.h to CrossThreadTask.h and move it to dom directory.
https://bugs.webkit.org/show_bug.cgi?id=39135

Reviewed by David Levin.

WebCore:

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/CrossThreadTask.h: Renamed from WebCore/workers/GenericWorkerTask.h.
  • html/FileStreamProxy.cpp:
  • loader/WorkerThreadableLoader.cpp:
  • websockets/WorkerThreadableWebSocketChannel.cpp:
  • workers/DefaultSharedWorkerRepository.cpp:
  • workers/WorkerMessagingProxy.cpp:
  • workers/WorkerScriptLoader.cpp:

WebKit/chromium:

  • src/WebSharedWorkerImpl.cpp:
  • src/WebWorkerBase.cpp:
  • src/WebWorkerClientImpl.cpp:
  • src/WebWorkerImpl.cpp:
10:52 AM EFLWebKit edited by leandro@profusion.mobi
(diff)
10:44 AM Changeset in webkit [59848] by adele@apple.com
  • 2 edits in trunk/WebCore

Remove the Safari welcome page site-specific quirk, since its no longer needed.

Reviewed by Mark Rowe.

  • css/CSSStyleSheet.cpp: (WebCore::CSSStyleSheet::cssRules):
10:37 AM Changeset in webkit [59847] by steveblock@google.com
  • 34 edits
    11 adds in trunk

2010-05-20 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Provide bindings for DeviceOrientation
https://bugs.webkit.org/show_bug.cgi?id=39210

Adds ENABLE_DEVICE_ORIENTATION to XCode project file, always disabled.

  • Configurations/FeatureDefines.xcconfig:

2010-05-20 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Provide bindings for DeviceOrientation
https://bugs.webkit.org/show_bug.cgi?id=39210

Adds ENABLE_DEVICE_ORIENTATION to VisualStudio project files, always disabled.

  • win/tools/vsprops/FeatureDefines.vsprops:
  • win/tools/vsprops/FeatureDefinesCairo.vsprops:

2010-05-20 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Provide bindings for DeviceOrientation
https://bugs.webkit.org/show_bug.cgi?id=39210

Adds ENABLE_DEVICE_ORIENTATION to XCode project file, always disabled.

  • Configurations/FeatureDefines.xcconfig:

2010-05-20 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Provide bindings for DeviceOrientation
https://bugs.webkit.org/show_bug.cgi?id=39210

Tests: fast/dom/DeviceOrientation/window-property.html

fast/dom/Window/window-properties-on-device-orientation.html

Adds bindings for DeviceOrientation as defined in
http://dev.w3.org/geo/api/spec-source-orientation.html
This is guarded by ENABLE(DEVICE_ORIENTATION) which is currently disabled on
all platforms but Android.

  • Android.derived.jscbindings.mk:
  • Android.derived.v8bindings.mk:
  • Android.mk:
  • Configurations/FeatureDefines.xcconfig:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSEventCustom.cpp: (WebCore::toJS):
  • bindings/v8/custom/V8EventCustom.cpp: (WebCore::toV8):
  • dom/DeviceOrientationEvent.cpp: Added. (WebCore::DeviceOrientationEvent::DeviceOrientationEvent): (WebCore::DeviceOrientationEvent::initDeviceOrientationEvent):
  • dom/DeviceOrientationEvent.h: Added. (WebCore::DeviceOrientationEvent::create): (WebCore::DeviceOrientationEvent::alpha): (WebCore::DeviceOrientationEvent::beta): (WebCore::DeviceOrientationEvent::gamma): (WebCore::DeviceOrientationEvent::isDeviceOrientationEvent):
  • dom/DeviceOrientationEvent.idl: Added.
  • dom/Event.cpp: (WebCore::Event::isDeviceOrientationEvent):
  • dom/Event.h:
  • dom/EventNames.h:
  • page/DOMWindow.h:
  • page/DOMWindow.idl:

2010-05-20 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Provide bindings for DeviceOrientation
https://bugs.webkit.org/show_bug.cgi?id=39210

Modifies existing fast/dom/Window tests to skip window.ondeviceorientation until
it is widely supported.
Adds new fast/dom/Window/window-properties-on-device-orientation.html to test
type and presence of window.ondeviceorientation.
Adds new fast/dom/DeviceOrientation directory and initial test.
Adds all DeviceOrientation tests to all skipped lists.

  • fast/dom/DeviceOrientation: Added.
  • fast/dom/DeviceOrientation/script-tests: Added.
  • fast/dom/DeviceOrientation/script-tests/TEMPLATE.html: Added.
  • fast/dom/DeviceOrientation/script-tests/window-property.js: Added. (hasOnDeviceOrientationProperty):
  • fast/dom/DeviceOrientation/window-property-expected.txt: Added.
  • fast/dom/DeviceOrientation/window-property.html: Added.
  • fast/dom/Window/script-tests/window-property-descriptors.js:
  • fast/dom/Window/window-properties-on-device-orientation-expected.txt: Added.
  • fast/dom/Window/window-properties-on-device-orientation.html: Added.
  • fast/dom/Window/window-properties.html:
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
10:26 AM Changeset in webkit [59846] by kenneth@webkit.org
  • 2 edits in trunk/WebCore

Use 0.15 as text drag delay when Q_WS_MAC is defined, which is
consistent with the Mac OS X platform.

Rubberstamped by Antti Koivisto.

  • page/qt/EventHandlerQt.cpp:
10:24 AM Changeset in webkit [59845] by Martin Robinson
  • 2 edits in trunk/WebKit/win

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Unreviewed.

Touch WebKit.idl to ensure that interfaces rebuild.

  • Interfaces/WebKit.idl: Touched.
10:15 AM Changeset in webkit [59844] by Martin Robinson
  • 2 edits in trunk/WebKitTools

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Ojan Vafai.

editing/selection/extend-selection-after-double-click.html crashes on the Leopard Intel release bot
https://bugs.webkit.org/show_bug.cgi?id=39431

  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setEditingBehavior): Prevent a double-free by not having this variable be in the auto-release pool.
9:48 AM Changeset in webkit [59843] by Martin Robinson
  • 3 edits in trunk/WebKit/win

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Adam Roben.

Fix the Windows build and move new IDL declarations to the bottom of the file.

  • Interfaces/IWebPreferences.idl: Move new API to the bottom of the IDL file to prevent vtable mismatch.
  • WebPreferences.cpp: (WebPreferences::setEditingBehavior): Change the type of the parameter to setEditingBehavior to the proper type.
9:19 AM Changeset in webkit [59842] by Martin Robinson
  • 2 edits in trunk/WebKit/win

2010-05-20 Martin Robinson <mrobinson@igalia.com>

Unreviewed.

Fix the Windows build.

  • WebView.cpp: (WebView::notifyPreferencesChanged): Change behavior to 'editingBehavior'.
9:11 AM Changeset in webkit [59841] by tkent@chromium.org
  • 1 edit in trunk/WebKit/win/Interfaces/WebKit.idl

Unreviewed. Touch WebKit.idl as usual.

8:48 AM Changeset in webkit [59840] by Martin Robinson
  • 32 edits in trunk

2010-05-20 Martin Robinson <Martin Robinson>

Reviewed by Ojan Vafai.

Expose the editing behavior setting in DRT to test all editing code paths
https://bugs.webkit.org/show_bug.cgi?id=38603

  • editing/selection/extend-selection-after-double-click.html: Use the new DRT support for changing editing behavior.
  • editing/selection/extend-selection-after-double-click-expected.txt: Update expected results to reflect both behaviors.

2010-05-20 Martin Robinson <Martin Robinson>

Reviewed by Ojan Vafai.

Expose the editing behavior setting in DRT to test all editing code paths
https://bugs.webkit.org/show_bug.cgi?id=38603

  • public/WebSettings.h: (WebKit::WebSettings::): Added an enum for EditingBehavior types.
  • src/WebSettingsImpl.cpp: (WebKit::WebSettingsImpl::setEditingBehavior): Added.
  • src/WebSettingsImpl.h:

2010-05-20 Martin Robinson <Martin Robinson>

Reviewed by Ojan Vafai.

Expose the editing behavior setting in DRT to test all editing code paths
https://bugs.webkit.org/show_bug.cgi?id=38603

  • WebView/WebFrame.mm: (core):
  • WebView/WebFrameInternal.h: Added a conversion method from the API enum to the WebCore enum.
  • WebView/WebPreferenceKeysPrivate.h: Added a preference key for the new setting.
  • WebView/WebPreferences.mm: (+[WebPreferences initialize]): Initialize the EditingBehavior to Mac style. (-[WebPreferences editingBehavior]): Added. (-[WebPreferences setEditingBehavior:]): Added.
  • WebView/WebPreferencesPrivate.h: Added the new API points to the private API.
  • WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]): Set the editing behavior via the WebPreferences setting.

2010-05-20 Martin Robinson <Martin Robinson>

Reviewed by Ojan Vafai.

Expose the editing behavior setting in DRT to test all editing code paths
https://bugs.webkit.org/show_bug.cgi?id=38603

Expose the EditingBehavior setting in the Windows API.

  • Interfaces/IWebPreferences.idl: Add the API point for setting the editing behavior.
  • WebPreferenceKeysPrivate.h: Add a key for the editing behavior setting.
  • WebPreferences.cpp: (WebPreferences::editingBehavior): Added. (WebPreferences::setEditingBehavior): Added.
  • WebPreferences.h: Add method declarations.
  • WebView.cpp: (WebView::notifyPreferencesChanged): Update the WebCore setting based on the WebPreferences setting.

2010-05-20 Martin Robinson <Martin Robinson>

Reviewed by Ojan Vafai.

Expose the editing behavior setting in DRT to test all editing code paths
https://bugs.webkit.org/show_bug.cgi?id=38603

  • DumpRenderTree/LayoutTestController.cpp: (setEditingBehaviorCallback): Added. (LayoutTestController::staticFunctions): Expose the setEditingBehaviorCallback function.
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/chromium/LayoutTestController.cpp: Add callback method for setting editing behavior.
  • DumpRenderTree/chromium/LayoutTestController.h: Declaration for this method.
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setEditingBehavior): Implementation of editing behavior control.
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setEditingBehavior): Ditto
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setEditingBehavior): Added stub implementation of editing behavior control.
  • DumpRenderTree/qt/LayoutTestControllerQt.h: (LayoutTestController::setEditingBehavior): Add slot for controlling editor behavior.
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setEditingBehavior): Implementation of editing behavior control.
  • DumpRenderTree/wx/LayoutTestControllerWx.cpp: (LayoutTestController::setEditingBehavior): Added stub implementation of editing behavior control.
8:46 AM Changeset in webkit [59839] by tkent@chromium.org
  • 2 edits in trunk/WebKitTools

2010-05-20 Kent Tamura <tkent@chromium.org>

Reviewed by Eric Seidel.

[DRT/Chromium] Increase the time out value
https://bugs.webkit.org/show_bug.cgi?id=39203

Change the time out value of Chromium DRT to 30 seconds, which is
the same as other ports.
If a DRT process exits before new-run-webkit-tests detects time
out, new-run-webkit-tests assumes the DRT process crashed.

  • DumpRenderTree/chromium/TestShell.cpp: (TestShell::layoutTestTimeout): Change the time out value from 10 seconds to 30 seconds.
8:22 AM Changeset in webkit [59838] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-05-20 Janne Koskinen <janne.p.koskinen@digia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] WINSCW compile fix for qwebframe test
https://bugs.webkit.org/show_bug.cgi?id=38722

WINSCW cannot determine template type up the hierarchy
to common base class.

  • tests/qwebframe/tst_qwebframe.cpp:
8:13 AM Changeset in webkit [59837] by Simon Hausmann
  • 2 edits in trunk/WebCore

Fix painting when using clipToImageBuffer()

Reviewed by Simon Hausmann.

When we apply the transform of the parent painter to the painter of
the transparency layer, we adopt its coordinate system, thus offset
should not be in page coordinates, but in the coordinate system of
the parent painter.

  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::TransparencyLayer::TransparencyLayer):

8:01 AM Changeset in webkit [59836] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-05-20 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

Remove unused "old style" private APIs

These symbols were introduced after QtWebKit v4.6
so no-one should be using them

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp:
7:21 AM Changeset in webkit [59835] by agl@chromium.org
  • 3 edits
    2 adds in trunk

2010-05-20 Adam Langley <agl@chromium.org>

Reviewed by David Levin.

[chromium] Avoid crash when msttcorefonts aren't provided

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

  • platform/graphics/chromium/FontCacheLinux.cpp: (WebCore::FontCache::createFontPlatformData):
7:15 AM Changeset in webkit [59834] by zherczeg@webkit.org
  • 2 edits in trunk/LayoutTests

Unreviewed. Update Qt specific expected file.

  • platform/qt/svg/filters/feLighting-crash-expected.txt:
6:56 AM Changeset in webkit [59833] by eric@webkit.org
  • 3 edits in trunk/WebKit/qt

2010-05-20 Rajiv Ramanasankaran <rajiv.ramanasankaran@nokia.com>

Reviewed by Simon Hausmann.

[Qt] QWebPage::inputMethodQuery() returns wrong values for Qt::ImCursorPosition, Qt::ImAnchorPosition
https://bugs.webkit.org/show_bug.cgi?id=38779

The earlier implementation was written with the assumption that in this scenario the
anchor position always corresponds to the START index and that the current cursor position
always corresponds to the END index in WebKit.

Updated the implementation of QWebPage::inputMethodQuery(Qt::ImCursorPosition) and
QWebPage::inputMethodQuery(Qt::ImAnchorPosition) for the case where the Editor is not in
composition mode. In the non-composition mode, the Anchor and the Current cursor positions
correspond to the Base and Extent position offsets in WebKit.

Also added the auto-tests for the RIGHT to LEFT and LEFT to RIGHT selections.

  • Api/qwebpage.cpp: (QWebPage::inputMethodQuery): Now returning correct values for Qt::ImCursorPosition and Qt::ImAnchorPosition when the Editor is not in composition mode.
  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::inputMethods): Added auto-tests for RIGHT to LEFT and LEFT to RIGHT selections
6:53 AM Changeset in webkit [59832] by zherczeg@webkit.org
  • 3 edits
    5 adds in trunk

WebCore: Fix a crash caused by feLigthing::apply()
https://bugs.webkit.org/show_bug.cgi?id=39004

Reviewed by Dirk Schulze.

Make FELighting::apply() similar to other filters' applies
to avoid an assertion fail in putUnmultipliedImageData(),
when effectDrawingRect has negative location (x or y is less than 0).

Test: svg/filters/feLighting-crash.svg

  • svg/graphics/filters/SVGFELighting.cpp:

(WebCore::FELighting::apply):

LayoutTests: In case of lighting SVG effects, a wider output rectangle
than the input image size should not cause crash.
https://bugs.webkit.org/show_bug.cgi?id=39004

Reviewed by Dirk Schulze.

  • platform/mac/svg/filters/feLighting-crash-expected.checksum: Added.
  • platform/mac/svg/filters/feLighting-crash-expected.png: Added.
  • platform/mac/svg/filters/feLighting-crash-expected.txt: Added.
  • platform/qt/svg/filters/feLighting-crash-expected.txt: Added.
  • svg/filters/feLighting-crash.svg: Added.
6:43 AM Changeset in webkit [59831] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-05-20 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Skipping popup focus test for maemo
https://bugs.webkit.org/show_bug.cgi?id=39314

Skipping popup focus test for maemo in qwebframe auto test.

The test method tst_QWebFrame::popupFocus() was testing popup focus AND input
field focus. The input field focus has been removed from the method popupFocus()
and a new test method named inputFieldFocus() has been added. Finally the test
method popupFocus() has been skipped for maemo.

  • tests/qwebframe/tst_qwebframe.cpp:
6:02 AM Changeset in webkit [59830] by cevans@google.com
  • 2 edits in trunk/WebKitTools

Unreviewed.

Marking myself as a committer.

4:48 AM Changeset in webkit [59829] by Chris Jerdonek
  • 6 edits in trunk

2010-05-20 Chris Jerdonek <Chris Jerdonek>

Reviewed by Eric Seidel.

Modified FrameLoader::urlSelected() to accept a KURL instead of a
ResourceRequest.

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

Since ResourceRequest has non-explicit single-parameter constructors for
String and KURL, urlSelected() previously accepted any of String, KURL,
and ResourceRequest. This revision changes urlSelected() to accept only
a KURL to make the API tighter and easier to refactor.

No new functionality, so no new tests.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::urlSelected):
    • Changed the public overload to accept a KURL instead of a ResourceRequest.
  • loader/FrameLoader.h:
    • Changed the public overload to accept a KURL instead of a ResourceRequest.
  • wml/WMLAElement.cpp: (WebCore::WMLAElement::defaultEventHandler):
    • Updated the call to urlSelected().

2010-05-20 Chris Jerdonek <Chris Jerdonek>

Reviewed by Eric Seidel.

Modified FrameLoader::urlSelected() to accept a KURL instead of a
ResourceRequest.

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

Since ResourceRequest has non-explicit single-parameter constructors for
String and KURL, urlSelected() previously accepted any of String, KURL,
and ResourceRequest. This revision changes urlSelected() to accept only
a KURL to make the API tighter and easier to refactor.

  • WebCoreSupport/WebContextMenuClient.cpp: (WebContextMenuClient::searchWithGoogle):
    • Updated the call to urlSelected().
4:07 AM Changeset in webkit [59828] by morrita@google.com
  • 5 edits
    2 adds in trunk

2010-05-20 MORITA Hajime <morrita@google.com>

Reviewed by Kent Tamura.

2x execCommand ReadAV@NULL
https://bugs.webkit.org/show_bug.cgi?id=35791

This test passes if it doesn't crash.

  • editing/execCommand/35791-expected.txt: Added.
  • editing/execCommand/35791.html: Added.

2010-05-20 MORITA Hajime <morrita@google.com>

Reviewed by Kent Tamura.

2x execCommand ReadAV@NULL
https://bugs.webkit.org/show_bug.cgi?id=35791

In applyBlockStyle(), a node which is referred from
nextParagraphStart sometimes removed from the document and goes
orphan, and that causes an assertion failed. This is because
moveParagraphs() replaces the node with a new one with some new
styles applied. So we re-compute nextParagraphStart in that case.

Test: editing/execCommand/35791.html

  • dom/Position.h: (WebCore::Position::isOrphan):
  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::applyBlockStyle):
  • editing/VisiblePosition.h: (WebCore::VisiblePosition::isOrphan):
4:04 AM Changeset in webkit [59827] by Simon Hausmann
  • 2 edits in trunk/WebKitTools

[Qt] Weekly binary builds on Mac OS X don't work when launched in the Finder
https://bugs.webkit.org/show_bug.cgi?id=37273

Reviewed by Tor Arne Vestbo.

  • QtTestBrowser/QtTestBrowser.pro: Build QtLauncher as bundle in package builds
2:06 AM Changeset in webkit [59826] by steveblock@google.com
  • 3 edits in trunk/WebCore

2010-05-20 Steve Block <steveblock@google.com>

Reviewed by Eric Seidel.

PluginView contains JSC-specific code
https://bugs.webkit.org/show_bug.cgi?id=38907

Build fix only, no new tests.

  • plugins/PluginView.cpp: (WebCore::PluginView::start): (WebCore::PluginView::stop): (WebCore::PluginView::performRequest):
  • plugins/PluginView.h:
1:44 AM Changeset in webkit [59825] by Philippe Normand
  • 2 edits in trunk/WebCore

2010-05-17 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Seidel.

[GStreamer] enable gst support if USE_GSTREAMER macro is defined
https://bugs.webkit.org/show_bug.cgi?id=39205

Include gstreamer private player in the build when the
USE_GSTREAMER macro is enabled.

  • platform/graphics/MediaPlayer.cpp: (WebCore::installedMediaEngines):
1:28 AM Changeset in webkit [59824] by ukai@chromium.org
  • 2 edits in trunk/WebKitTools

Unreviewed fix for websocket test failures.

  • Scripts/new-run-webkit-websocketserver: options is named parameter for factory.get().
1:23 AM QtWebKitTriageRoster edited by Simon Hausmann
(diff)
12:45 AM Changeset in webkit [59823] by ukai@chromium.org
  • 2 edits in trunk/WebKitTools

2010-05-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Shinichiro Hamaji.

Chromium: Add --chromium option to new-run-webkit-websocketserver
https://bugs.webkit.org/show_bug.cgi?id=37664

Missed to pass options to factory.get() in r59595

  • Scripts/new-run-webkit-websocketserver: Pass options to factory.get().

May 19, 2010:

10:59 PM Changeset in webkit [59822] by eric@webkit.org
  • 9 edits in trunk

2010-05-19 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Darin Fisher.

Implementing hardware accelerated compositing of layers for Chromium.
Completely removed the existing software compositing path to replace
it with code implemented using OpenGL.
https://bugs.webkit.org/show_bug.cgi?id=38783

Tests: Covered by existing compositing tests.

  • WebCore.gyp/WebCore.gyp:
  • platform/graphics/chromium/GraphicsLayerChromium.cpp: (WebCore::GraphicsLayerChromium::~GraphicsLayerChromium): (WebCore::GraphicsLayerChromium::updateLayerDrawsContent): (WebCore::GraphicsLayerChromium::updateContentsRect):
  • platform/graphics/chromium/LayerChromium.cpp: (WebCore::LayerChromium::LayerChromium): (WebCore::LayerChromium::~LayerChromium): (WebCore::LayerChromium::updateGraphicsContext): (WebCore::LayerChromium::drawsContentUpdated): (WebCore::LayerChromium::updateContents): (WebCore::LayerChromium::setContents): (WebCore::LayerChromium::setBackingStoreSize): (WebCore::LayerChromium::setBounds): (WebCore::LayerChromium::setNeedsDisplay):
  • platform/graphics/chromium/LayerChromium.h: (WebCore::LayerChromium::contentsDirty): (WebCore::LayerChromium::drawsContent):
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::): (WebCore::checkGLError): (WebCore::loadShader): (WebCore::loadShaderProgram): (WebCore::toGLMatrix): (WebCore::orthoMatrix): (WebCore::createLayerTexture): (WebCore::LayerRendererChromium::create): (WebCore::LayerRendererChromium::LayerRendererChromium): (WebCore::LayerRendererChromium::~LayerRendererChromium): (WebCore::LayerRendererChromium::drawTexturedQuad): (WebCore::LayerRendererChromium::drawLayers): (WebCore::LayerRendererChromium::getTextureId): (WebCore::LayerRendererChromium::assignTextureForLayer): (WebCore::LayerRendererChromium::freeLayerTexture): (WebCore::LayerRendererChromium::drawDebugBorder): (WebCore::LayerRendererChromium::isLayerVisible): (WebCore::LayerRendererChromium::compositeLayersRecursive): (WebCore::LayerRendererChromium::makeContextCurrent): (WebCore::LayerRendererChromium::initGL): (WebCore::LayerRendererChromium::bindCommonAttribLocation): (WebCore::LayerRendererChromium::initializeSharedGLObjects):
  • platform/graphics/chromium/LayerRendererChromium.h: (WebCore::LayerRendererChromium::):

2010-05-19 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Darin Fisher.

Modifying WebViewImpl to support new hardware compositing for
LayerChromium layers.
https://bugs.webkit.org/show_bug.cgi?id=38783

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::paint): (WebKit::WebViewImpl::setIsAcceleratedCompositingActive): (WebKit::WebViewImpl::updateRootLayerContents): (WebKit::WebViewImpl::setRootLayerNeedsDisplay):
10:38 PM Changeset in webkit [59821] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-05-19 Eric Seidel <eric@webkit.org>

Reviewed by Adam Roben.

WinEWS should build Debug instead of Release
https://bugs.webkit.org/show_bug.cgi?id=39242

This is a workaround for
https://bugs.webkit.org/show_bug.cgi?id=39197
Adam Roben and Brian Weinstein believe this may
also make building faster since Debug builds
take less time to link.

  • Scripts/webkitpy/tool/commands/earlywarningsystem.py:
10:36 PM Changeset in webkit [59820] by barraclough@apple.com
  • 9 edits in trunk/JavaScriptCore

Errk! rename op_constructor_ret as requested in review of Bug 39399.

Reviewed by Geoff Garen.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::dump):

  • bytecode/Opcode.h:
  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitReturn):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • jit/JIT.cpp:

(JSC::JIT::privateCompileMainPass):

  • jit/JIT.h:
  • jit/JITCall.cpp:

(JSC::JIT::emit_op_ret_object_or_this):

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_ret_object_or_this):

10:35 PM Changeset in webkit [59819] by tony@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-05-19 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] disable spelling suggestions if spellcheck=false
https://bugs.webkit.org/show_bug.cgi?id=39402

This is the Chromium fix for bug 25639 that was fixed in the
other ports in r59585.

  • src/ContextMenuClientImpl.cpp: (WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
10:02 PM Changeset in webkit [59818] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-05-19 Mario Sanchez Prada <msanchez@igalia.com>

Reviewed by Darin Adler.

Misleading variable name in a11y test
https://bugs.webkit.org/show_bug.cgi?id=38465

Use a more appropriate variable name for the return
value of accessibilityController.rootElement

  • accessibility/aria-controls-with-tabs.html:
9:57 PM Changeset in webkit [59817] by barraclough@apple.com
  • 11 edits in trunk/JavaScriptCore

Bug 39399 - Move responsibility for verifying constructors return objects from the caller to the callee.

Reviewed by Geoff Garen.

This is a necessary step to move object creation from caller to callee.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::dump):

  • bytecode/Opcode.h:
  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator):
(JSC::BytecodeGenerator::emitReturn):
(JSC::BytecodeGenerator::emitConstruct):

  • bytecompiler/BytecodeGenerator.h:

(JSC::BytecodeGenerator::isConstructor):

  • bytecompiler/NodesCodegen.cpp:

(JSC::FunctionBodyNode::emitBytecode):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • jit/JIT.cpp:

(JSC::JIT::privateCompileMainPass):
(JSC::JIT::privateCompileSlowCases):

  • jit/JIT.h:
  • jit/JITCall.cpp:

(JSC::JIT::emit_op_constructor_ret):

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_constructor_ret):

9:37 PM Changeset in webkit [59816] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-19 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Timothy Hatcher.

Maintain focus on help screen while it's active.
https://bugs.webkit.org/show_bug.cgi?id=39237

9:18 PM Changeset in webkit [59815] by eric@webkit.org
  • 4 edits in trunk

2010-05-19 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Crash fix in the HistoryController.

The problem was casued by a documented feature of the QWebFrame::setHtml().
The method doesn't affect a browsing history (doesn't create a HistoryItem instance), so
m_currentItem and m_previousItem are not set.

A null check was added.

[Qt] The QWebPage crashes on history.pushState().
https://bugs.webkit.org/show_bug.cgi?id=38840

  • loader/HistoryController.cpp: (WebCore::HistoryController::pushState): (WebCore::HistoryController::replaceState):

2010-05-19 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Crash fix in the HistoryController.

The problem was casued by a documented feature of the QWebFrame::setHtml().
The method doesn't affect a browsing history (doesn't create a HistoryItem instance), so
m_currentItem and m_previousItem are not set.

A null check was added.

[Qt] The QWebPage crashes on history.pushState().
https://bugs.webkit.org/show_bug.cgi?id=38840

  • tests/qwebhistory/tst_qwebhistory.cpp: (tst_QWebHistory::popPushState_data): (tst_QWebHistory::popPushState):
8:24 PM Changeset in webkit [59814] by darin@chromium.org
  • 11 edits in trunk

2010-05-19 Darin Fisher <darin@chromium.org>

Revert r59790 due to failing Chromium npruntime tests.

  • bindings/v8/NPV8Object.cpp: (v8ObjectToNPObject): (npCreateV8ScriptObject): (_NPN_Invoke): (_NPN_InvokeDefault): (_NPN_Evaluate): (_NPN_EvaluateHelper): (_NPN_GetProperty): (_NPN_SetProperty): (_NPN_RemoveProperty): (_NPN_HasProperty): (_NPN_HasMethod): (_NPN_SetException): (_NPN_Enumerate): (_NPN_Construct):
  • bindings/v8/NPV8Object.h:
  • bindings/v8/V8NPObject.cpp: (forgetV8ObjectForNPObject):
  • bindings/v8/V8NPObject.h:
  • bindings/v8/V8NPUtils.cpp: (convertV8ObjectToNPVariant): (getStringIdentifier):
  • bindings/v8/V8NPUtils.h:
  • bindings/v8/npruntime.cpp:
8:22 PM Changeset in webkit [59813] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Fix windows build II.

Reviewed by NOBODY (build fix).

8:12 PM Changeset in webkit [59812] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Fix windows build.

Reviewed by NOBODY (build fix).

7:38 PM Changeset in webkit [59811] by barraclough@apple.com
  • 15 edits in trunk

JavaScriptCore: Bug 39393 - JSFunction need not be a subclass of InternalFunction.

Reviewed by Oliver Hunt.

re-landing r59800.

(JSC::Interpreter::retrieveCaller):
(JSC::Interpreter::findFunctionCallFrame):

  • interpreter/Interpreter.h:
  • profiler/Profiler.cpp:

(JSC::Profiler::createCallIdentifier):

  • runtime/FunctionPrototype.cpp:

(JSC::functionProtoFuncToString):

  • runtime/JSFunction.cpp:

(JSC::):
(JSC::JSFunction::JSFunction):
(JSC::JSFunction::name):
(JSC::JSFunction::displayName):
(JSC::JSFunction::calculatedDisplayName):

  • runtime/JSFunction.h:
  • runtime/JSObject.cpp:

(JSC::JSObject::putDirectFunction):
(JSC::JSObject::putDirectFunctionWithoutTransition):

  • runtime/JSObject.h:
  • runtime/Lookup.cpp:

(JSC::setUpStaticFunctionSlot):

WebCore: Rubber Stamped by Sam Weinig.

JSFunctions are no longer a subclass of InternalFunction.

  • bindings/js/ScriptCallStack.cpp:

(WebCore::ScriptCallStack::ScriptCallStack):
(WebCore::ScriptCallStack::initialize):

  • bindings/js/ScriptCallStack.h:
7:35 PM Changeset in webkit [59810] by mrowe@apple.com
  • 9 edits
    2 copies
    3 adds in branches/safari-533-branch

Merge r59719.

7:35 PM Changeset in webkit [59809] by mrowe@apple.com
  • 3 edits in branches/safari-533-branch/WebCore

Merge r59674.

7:35 PM Changeset in webkit [59808] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59684.

7:35 PM Changeset in webkit [59807] by mrowe@apple.com
  • 3 edits
    1 move in branches/safari-533-branch/WebCore

Merge r59683.

7:35 PM Changeset in webkit [59806] by mrowe@apple.com
  • 2 edits
    1 add in branches/safari-533-branch/WebCore

Merge r59682.

7:35 PM Changeset in webkit [59805] by mrowe@apple.com
  • 15 edits
    1 delete in branches/safari-533-branch/WebCore

Merge r59724.

7:35 PM Changeset in webkit [59804] by mrowe@apple.com
  • 3 edits in branches/safari-533-branch/WebCore

Merge r59716.

7:34 PM Changeset in webkit [59803] by mrowe@apple.com
  • 3 edits in branches/safari-533-branch/WebCore

Merge r59680.

7:34 PM Changeset in webkit [59802] by mrowe@apple.com
  • 6 edits in branches/safari-533-branch

Merge r59678.

6:15 PM Changeset in webkit [59801] by barraclough@apple.com
  • 11 edits in trunk/JavaScriptCore

Reverting r59800, this b0rked stuff.

Reviewed by NOBODY (build fix).

(JSC::Interpreter::retrieveCaller):
(JSC::Interpreter::findFunctionCallFrame):

  • interpreter/Interpreter.h:
  • profiler/Profiler.cpp:

(JSC::Profiler::createCallIdentifier):

  • runtime/FunctionPrototype.cpp:

(JSC::functionProtoFuncToString):

  • runtime/JSFunction.cpp:

(JSC::):
(JSC::JSFunction::JSFunction):

  • runtime/JSFunction.h:
  • runtime/JSObject.cpp:
  • runtime/JSObject.h:
  • runtime/Lookup.cpp:

(JSC::setUpStaticFunctionSlot):

6:01 PM Changeset in webkit [59800] by barraclough@apple.com
  • 11 edits in trunk/JavaScriptCore

Bug 39393 - JSFunction need not be a subclass of InternalFunction.

Reviewed by Oliver Hunt.

This may prevent us from introducing a more useful parent class to
JSFunction, e.g. a JSObject that holds an executable, which could
also reference an eval or program executable.

(JSC::Interpreter::retrieveCaller):
(JSC::Interpreter::findFunctionCallFrame):

  • interpreter/Interpreter.h:
  • profiler/Profiler.cpp:

(JSC::Profiler::createCallIdentifier):

  • runtime/FunctionPrototype.cpp:

(JSC::functionProtoFuncToString):

  • runtime/JSFunction.cpp:

(JSC::):
(JSC::JSFunction::JSFunction):
(JSC::JSFunction::name):
(JSC::JSFunction::displayName):
(JSC::JSFunction::calculatedDisplayName):

  • runtime/JSFunction.h:
  • runtime/JSObject.cpp:

(JSC::JSObject::putDirectFunction):
(JSC::JSObject::putDirectFunctionWithoutTransition):

  • runtime/JSObject.h:
  • runtime/Lookup.cpp:

(JSC::setUpStaticFunctionSlot):

5:54 PM Changeset in webkit [59799] by victorw@chromium.org
  • 4 edits
    6 adds
    1 delete in trunk/LayoutTests

2010-05-19 Victor Wang <victorw@chromium.org>

Unreviewed. Update Chromium test expectations.

  • platform/chromium-linux/fast/multicol/layers-split-across-columns-expected.checksum: Added.
  • platform/chromium-linux/fast/multicol/layers-split-across-columns-expected.png: Added.
  • platform/chromium-mac/fast/multicol/layers-in-multicol-expected.checksum:
  • platform/chromium-mac/fast/multicol/layers-in-multicol-expected.png: Removed.
  • platform/chromium-mac/fast/multicol/layers-split-across-columns-expected.checksum: Added.
  • platform/chromium-win/fast/multicol/layers-in-multicol-expected.txt:
  • platform/chromium-win/fast/multicol/layers-split-across-columns-expected.checksum: Added.
  • platform/chromium-win/fast/multicol/layers-split-across-columns-expected.png: Added.
  • platform/chromium-win/fast/multicol/layers-split-across-columns-expected.txt: Added.
  • platform/chromium-win/fast/multicol/nested-columns-expected.txt:
5:30 PM Changeset in webkit [59798] by oliver@apple.com
  • 5 edits
    3 adds in trunk

2010-05-19 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

emitJumpIfNotJSCell should special case constant immediate values
https://bugs.webkit.org/show_bug.cgi?id=39392
<rdar://problem/8001324>

Make emitJumpSlowCaseIfNotJSCell special case constant immediate
values, in addition to the immediate JSCell optimisation.

Also add assertions to make sure no one else produces code that
attempts to load constants from the register file.

  • jit/JITInlineMethods.h: (JSC::JIT::emitJumpSlowCaseIfNotJSCell):
  • jit/JSInterfaceJIT.h: (JSC::JSInterfaceJIT::emitJumpIfNotJSCell): (JSC::JSInterfaceJIT::emitLoadInt32): (JSC::JSInterfaceJIT::tagFor): (JSC::JSInterfaceJIT::payloadFor): (JSC::JSInterfaceJIT::emitLoadDouble): (JSC::JSInterfaceJIT::addressFor):
  • jit/ThunkGenerators.cpp:

2010-05-19 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

emitJumpIfNotJSCell should special case constant immediate values
https://bugs.webkit.org/show_bug.cgi?id=39392

Add tests for immediate constants being used where cells are expected.

  • fast/js/immediate-constant-instead-of-cell-expected.txt: Added.
  • fast/js/immediate-constant-instead-of-cell.html: Added.
  • fast/js/script-tests/immediate-constant-instead-of-cell.js: Added. ():
5:23 PM Changeset in webkit [59797] by jianli@chromium.org
  • 6 edits
    1 add in trunk

Refactor text encoding detection logic in FileReader.
https://bugs.webkit.org/show_bug.cgi?id=39131

Reviewed by Alexey Proskuryakov.

WebCore:

Changed FileReader::convertToText to call TextResourceDecoder::decode to
detect the encoding from BOM and decode the text. Though the File API
spec says that the supplied encoding should be used if it is valid, we
choose to ignore this requirement in order to be consistent with how
WebKit decodes the web content: always has the BOM override the provided
encoding.

  • html/FileReader.cpp:

(WebCore::FileReader::convertToText):

  • html/FileReader.h:

LayoutTests:

Add more test coverage for FileReader.

  • fast/files/file-reader-expected.txt:
  • fast/files/file-reader.html:
  • fast/files/resources/binary-file: Added.
5:19 PM Changeset in webkit [59796] by Martin Robinson
  • 2 edits in trunk/LayoutTests

Not reviewed.

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

Disabling geolocation tests until blockers are fixed.

  • platform/gtk/Skipped: Skipping geolocation tests for now.
4:59 PM Changeset in webkit [59795] by yaar@chromium.org
  • 4 edits
    2 adds in trunk

2010-05-19 Abhishek Arya <inferno@chromium.org>

Reviewed by David Hyatt.

Check that the node is a text node before doing a static cast
to a Text class pointer.
https://bugs.webkit.org/show_bug.cgi?id=38626

Test: fast/text/text-transform-nontext-node-crash.xhtml

  • rendering/RenderText.cpp: (WebCore::RenderText::originalText):
  • rendering/RenderTextFragment.cpp: (WebCore::RenderTextFragment::originalText): (WebCore::RenderTextFragment::previousCharacter):

2010-05-19 Abhishek Arya <inferno@chromium.org>

Reviewed by David Hyatt.

Tests that text transformation applied to a non-text node
does not result in crash.
https://bugs.webkit.org/show_bug.cgi?id=38626

  • fast/text/text-transform-nontext-node-crash-expected.txt: Added.
  • fast/text/text-transform-nontext-node-crash.xhtml: Added.
4:52 PM Changeset in webkit [59794] by yaar@chromium.org
  • 4 edits in trunk/WebCore

2010-05-19 Yaar Schnitman <yaar@chromium.org>

Reviewed by Eric Seidel.

Fixed warnings by JSC overloads changes. Also updated the binding reference files for JSC.
https://bugs.webkit.org/show_bug.cgi?id=39390

  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/test/JS/JSTestInterface.cpp: (WebCore::):
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::):
4:39 PM Changeset in webkit [59793] by andersca@apple.com
  • 3 edits in trunk/WebKit/mac

<rdar://problem/8004528>
REGRESSION: Coordinate system for Core Animation NPAPI plug-ins is flipped with accelerated compositing turned on

Reviewed by Kevin Decker and Simon Fraser.

When needed, create a new CALayer and set it's geometry to be flipped. Add the plug-in layer as a sublayer and then return
the newly created layer.

  • Plugins/Hosted/WebHostedNetscapePluginView.mm:

(-[WebHostedNetscapePluginView createPlugin]):

  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView createPlugin]):

4:28 PM Changeset in webkit [59792] by Martin Robinson
  • 3 edits
    1 add in trunk/LayoutTests

2010-05-19 Martin Robinson <mrobinson@igalia.com>

Not reviewed.

Updated multi-column test baselines.

  • platform/gtk/fast/multicol/layers-in-multicol-expected.txt: Updated.
  • platform/gtk/fast/multicol/layers-split-across-columns-expected.txt: Added.
  • platform/gtk/fast/multicol/nested-columns-expected.txt: Updated.
4:24 PM Changeset in webkit [59791] by ggaren@apple.com
  • 4 edits in trunk/JavaScriptCore

2010-05-19 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Slight refactoring to CodeBlock bytecode access
https://bugs.webkit.org/show_bug.cgi?id=39384

  • bytecode/CodeBlock.h: (JSC::CodeBlock::bytecodeOffset):
  • interpreter/CallFrame.h: (JSC::ExecState::returnPC):
  • interpreter/Interpreter.cpp: (JSC::Interpreter::unwindCallFrame): (JSC::Interpreter::retrieveLastCaller): Moved bytecode access into a CodeBlock helper function. Changed CallFrame to accurately represent how the returnPC is stored in JIT vs Interpreter.
3:59 PM Changeset in webkit [59790] by darin@chromium.org
  • 11 edits in trunk

2010-05-19 Darin Fisher <darin@chromium.org>

Reviewed by Nate Chapin.

[chromium] Provide a way to catch exceptions thrown while interacting
with a NPObject via WebBindings methods.
https://bugs.webkit.org/show_bug.cgi?id=39378

This change also includes a small bit of cleanup in V8NPObject and
friends. I moved code into the WebCore namespace where appropriate.


In _NPN_Invoke, I also moved the call to _NPN_Evaluate outside of the
context scope. I did this to avoid having nested ExceptionCatcher
objects, which is not strictly necessary, but it seemed cleaner.
_NPN_Evaluate already takes care of creating a context scope.

  • bindings/v8/NPV8Object.cpp: (WebCore::v8ObjectToNPObject): (WebCore::npCreateV8ScriptObject): (_NPN_Invoke): (_NPN_InvokeDefault): (_NPN_Evaluate): (_NPN_EvaluateHelper): (_NPN_GetProperty): (_NPN_SetProperty): (_NPN_RemoveProperty): (_NPN_HasProperty): (_NPN_HasMethod): (_NPN_SetException): (_NPN_Enumerate): (_NPN_Construct):
  • bindings/v8/NPV8Object.h:
  • bindings/v8/V8NPObject.cpp:
  • bindings/v8/V8NPObject.h:
  • bindings/v8/V8NPUtils.cpp: (WebCore::convertV8ObjectToNPVariant): (WebCore::pushExceptionHandler): (WebCore::popExceptionHandler): (WebCore::ExceptionCatcher::~ExceptionCatcher):
  • bindings/v8/V8NPUtils.h:
  • bindings/v8/npruntime.cpp:
3:43 PM Changeset in webkit [59789] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

2010-05-19 Dirk Pranke <dpranke@chromium.org>

Unreviewed, build fix.

  • DumpRenderTree/chromium/NotificationPresenter.cpp: (NotificationPresenter::show):
3:43 PM Changeset in webkit [59788] by yaar@chromium.org
  • 29 edits
    2 deletes in trunk

2010-05-19 Yaar Schnitman <yaar@chromium.org>

Unreviewed, rolling out r59769.
http://trac.webkit.org/changeset/59769
https://bugs.webkit.org/show_bug.cgi?id=37815

Broke GTK again

  • Android.jscbindings.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSCallbackData.cpp: (WebCore::JSCallbackData::invokeCallback):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::handleEvent):
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript):
  • bindings/js/JSMainThreadExecState.cpp: Removed.
  • bindings/js/JSMainThreadExecState.h: Removed.
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::executeFunctionInContext): (WebCore::ScheduledAction::execute):
  • bindings/js/ScheduledAction.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld):
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptFunctionCall.cpp: (WebCore::ScriptFunctionCall::call):
  • bindings/objc/ObjCEventListener.mm:
  • bindings/objc/WebScriptObject.mm: (-[WebScriptObject callWebScriptMethod:withArguments:]): (-[WebScriptObject evaluateWebScript:]):
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/test/ObjC/DOMTestCallback.mm: (-[DOMTestCallback callbackWithClass1Param:]): (-[DOMTestCallback callbackWithClass2Param:strArg:]): (-[DOMTestCallback callbackWithNonBoolReturnType:]): (-[DOMTestCallback customCallback:class6Param:]):
  • bindings/scripts/test/ObjC/DOMTestInterface.mm:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj readOnlyIntAttr]): (-[DOMTestObj readOnlyStringAttr]): (-[DOMTestObj readOnlyTestObjAttr]): (-[DOMTestObj intAttr]): (-[DOMTestObj setIntAttr:]): (-[DOMTestObj longLongAttr]): (-[DOMTestObj setLongLongAttr:]): (-[DOMTestObj unsignedLongLongAttr]): (-[DOMTestObj setUnsignedLongLongAttr:]): (-[DOMTestObj stringAttr]): (-[DOMTestObj setStringAttr:]): (-[DOMTestObj testObjAttr]): (-[DOMTestObj setTestObjAttr:]): (-[DOMTestObj attrWithException]): (-[DOMTestObj setAttrWithException:]): (-[DOMTestObj attrWithSetterException]): (-[DOMTestObj setAttrWithSetterException:]): (-[DOMTestObj attrWithGetterException]): (-[DOMTestObj setAttrWithGetterException:]): (-[DOMTestObj customAttr]): (-[DOMTestObj setCustomAttr:]): (-[DOMTestObj scriptStringAttr]): (-[DOMTestObj voidMethod]): (-[DOMTestObj voidMethodWithArgs:strArg:objArg:]): (-[DOMTestObj intMethod]): (-[DOMTestObj intMethodWithArgs:strArg:objArg:]): (-[DOMTestObj objMethod]): (-[DOMTestObj objMethodWithArgs:strArg:objArg:]): (-[DOMTestObj methodThatRequiresAllArgs:objArg:]): (-[DOMTestObj methodThatRequiresAllArgsAndThrows:objArg:]): (-[DOMTestObj serializedValue:]): (-[DOMTestObj methodWithException]): (-[DOMTestObj customMethod]): (-[DOMTestObj customMethodWithArgs:strArg:objArg:]): (-[DOMTestObj customArgsAndException:]): (-[DOMTestObj addEventListener:listener:useCapture:]): (-[DOMTestObj removeEventListener:listener:useCapture:]): (-[DOMTestObj withDynamicFrame]): (-[DOMTestObj withDynamicFrameAndArg:]): (-[DOMTestObj withDynamicFrameAndOptionalArg:optionalArg:]): (-[DOMTestObj withDynamicFrameAndUserGesture:]): (-[DOMTestObj withDynamicFrameAndUserGestureASAD:optionalArg:]): (-[DOMTestObj withScriptStateVoid]): (-[DOMTestObj withScriptStateObj]): (-[DOMTestObj withScriptStateVoidException]): (-[DOMTestObj withScriptStateObjException]): (-[DOMTestObj methodWithOptionalArg:]): (-[DOMTestObj methodWithNonOptionalArgAndOptionalArg:opt:]): (-[DOMTestObj methodWithNonOptionalArgAndTwoOptionalArgs:opt1:opt2:]):
  • bindings/v8/ScriptController.cpp:
  • bindings/v8/ScriptController.h:
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::isURLAllowed):

2010-05-19 Yaar Schnitman <yaar@chromium.org>

Unreviewed, rolling out r59769.
http://trac.webkit.org/changeset/59769
https://bugs.webkit.org/show_bug.cgi?id=37815

Broke GTK again

  • http/tests/security/xss-DENIED-iframe-src-alias-expected.txt:
  • http/tests/security/xss-DENIED-iframe-src-alias.html:
3:36 PM Changeset in webkit [59787] by dpranke@chromium.org
  • 7 edits
    4 deletes in trunk

2010-05-19 Dirk Pranke <dpranke@chromium.org>

Try again to roll out r58765

  • fast/notifications/notifications-replace-expected.txt: Removed.
  • fast/notifications/notifications-replace.html: Removed.
  • fast/notifications/notifications-rtl-expected.txt: Removed.
  • fast/notifications/notifications-rtl.html: Removed.

2010-05-19 Dirk Pranke <dpranke@chromium.org>

Try again to roll out r58675.

  • notifications/Notification.h:
  • notifications/Notification.idl:

2010-05-19 Dirk Pranke <dpranke@chromium.org>

Try again to roll out r58765.

  • public/WebNotification.h:
  • src/WebNotification.cpp:
2:53 PM Changeset in webkit [59786] by enrica@apple.com
  • 3 edits
    2 adds in trunk

WebCore: REGRESSION (r59385) crash destroying inline renderers
https://bugs.webkit.org/show_bug.cgi?id=39143
<rdar://problem/8003662>

Reviewed by Dave Hyatt.

The goal of r59385 was to make sure that the layout of block after all its children had been removed produced the identical result
as the one of a newly created empty block. In order to do so, we had to make sure that the m_inlineChildren flag was reset to true when
the block had no children (as it is upon creation).
I discovered that, by doing that for anonymous blocks it leads removeChild to conclude that the anonymous children can be removed,
without considering that the anonymous block could be part of a continuation chain. For this reason, when RenderInline::destroy()
tries to remove the continuations we are effectively deleting a renderer that had been deleted already.

Test: fast/inline-block/anonymous-block-crash.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::layoutBlock): resetting the flag m_inlineChildren only for non anonymous blocks otherwise we incurr in
a double deletion of the renderer that causes the crash.

LayoutTests: REGRESSION (r59385) crash destroying inline renderers
https://bugs.webkit.org/show_bug.cgi?id=39143
<rdar://problem/8003662>

Reviewed by Dave Hyatt.

  • fast/inline-block/anonymous-block-crash-expected.txt: Added.
  • fast/inline-block/anonymous-block-crash.html: Added.
2:30 PM Changeset in webkit [59785] by dpranke@chromium.org
  • 7 edits
    4 adds in trunk

2010-05-19 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r59782.
http://trac.webkit.org/changeset/59782
https://bugs.webkit.org/show_bug.cgi?id=39379

broke DRT build (Requested by dpranke on #webkit).

  • fast/notifications/notifications-replace-expected.txt: Added.
  • fast/notifications/notifications-replace.html: Added.
  • fast/notifications/notifications-rtl-expected.txt: Added.
  • fast/notifications/notifications-rtl.html: Added.

2010-05-19 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r59782.
http://trac.webkit.org/changeset/59782
https://bugs.webkit.org/show_bug.cgi?id=39379

broke DRT build (Requested by dpranke on #webkit).

  • notifications/Notification.h: (WebCore::Notification::dir): (WebCore::Notification::setDir): (WebCore::Notification::replaceId): (WebCore::Notification::setReplaceId):
  • notifications/Notification.idl:

2010-05-19 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r59782.
http://trac.webkit.org/changeset/59782
https://bugs.webkit.org/show_bug.cgi?id=39379

broke DRT build (Requested by dpranke on #webkit).

  • public/WebNotification.h:
  • src/WebNotification.cpp: (WebKit::WebNotification::dir): (WebKit::WebNotification::replaceId):
2:03 PM Changeset in webkit [59784] by hyatt@apple.com
  • 8 edits
    4 adds in trunk

https://bugs.webkit.org/show_bug.cgi?id=21049, opacity and position:relative fail in multi-column layouts.

Reviewed by Oliver Hunt.

Make layers split painting across columns just as normal flow content does. This is actually very
challenging, since intermediate multicol layers can exist between a child that has to be split and
its enclosing stacking context.

In order to properly paginate layers, a bit is set on all pagination roots, i.e., layers that will get
split and thus split any descendant layers that they are also responsible for painting. For a paginated
root, we walk up the layer hierarchy and collect all intermediate multicol blocks between the child
and the enclosing stacking context.

We then recur from the outside in, applying clipping and translation as we break up the layer into strips.

Composited layers remain unsplittable and have a hacked offset still in order to be placed in the right
column.

Painting and hit testing of columns has been simplified in all of the functions to use the x position of
the column rect instead of trying to increment an x offset by adding in widths and column gaps. This
makes those functions directionality-independent (and the LTR/RTL code paths are now the same).

Fix repainting of columns as well to make sure rects are split across columns only after relative position
and transforms have been applied.

Added fast/multicol/layers-split-across-columns.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::paintColumnContents):
(WebCore::RenderBlock::adjustRectForColumns):
(WebCore::RenderBlock::adjustForColumns):

  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp:

(WebCore::RenderBox::computeRectForRepaint):

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::RenderLayer):
(WebCore::RenderLayer::updateLayerPositions):
(WebCore::RenderLayer::updatePagination):
(WebCore::RenderLayer::updateLayerPosition):
(WebCore::RenderLayer::paintLayer):
(WebCore::RenderLayer::paintList):
(WebCore::RenderLayer::paintPaginatedChildLayer):
(WebCore::RenderLayer::paintChildLayerIntoColumns):
(WebCore::RenderLayer::hitTestLayer):
(WebCore::RenderLayer::hitTestList):
(WebCore::RenderLayer::hitTestPaginatedChildLayer):
(WebCore::RenderLayer::hitTestChildLayerColumns):

  • rendering/RenderLayer.h:

(WebCore::RenderLayer::):
(WebCore::RenderLayer::isPaginated):

1:54 PM Changeset in webkit [59783] by eric@webkit.org
  • 6 edits in trunk/WebKit/gtk

2010-05-19 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r59240.
http://trac.webkit.org/changeset/59240
https://bugs.webkit.org/show_bug.cgi?id=39377

"Might cause crashes in GTK+ bots" (Requested by xan_ on
#webkit).

  • WebCoreSupport/ContextMenuClientGtk.cpp: (WebKit::inputMethodsMenuItem):
  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::setInputMethodState): (WebKit::EditorClient::respondToChangedSelection): (WebKit::EditorClient::handleInputMethodKeydown): (WebKit::EditorClient::EditorClient): (WebKit::EditorClient::~EditorClient):
  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::postCommitFrameViewSetup):
  • webkit/webkitprivate.h:
  • webkit/webkitwebview.cpp: (destroy_menu_cb): (webkit_web_view_forward_context_menu_event): (webkit_web_view_key_release_event): (webkit_web_view_button_release_event): (webkit_web_view_focus_in_event): (webkit_web_view_focus_out_event): (webkit_web_view_realize): (webkit_web_view_set_scroll_adjustments): (webkit_web_view_dispose): (webkit_web_view_finalize): (webViewGetDPI): (webkit_web_view_screen_changed): (webkit_web_view_query_tooltip): (webkit_web_view_get_im_context): (webkit_web_view_update_settings): (webkit_web_view_init): (webkit_web_view_set_settings): (webkit_web_view_get_settings): (webkit_web_view_get_inspector): (webkit_web_view_set_window_features): (webkit_web_view_get_window_features): (webkit_web_view_get_back_forward_list): (webkit_web_view_zoom_in): (webkit_web_view_zoom_out): (webkit_web_view_get_encoding): (webkit_web_view_get_custom_encoding): (webkit_web_view_add_resource): (webkit_web_view_get_resource): (webkit_web_view_get_main_resource): (webkit_web_view_clear_resources): (webkit_web_view_get_subresources): (webkit_web_view_set_tooltip_text): (webkit_web_view_get_icon_uri):
1:41 PM Changeset in webkit [59782] by dpranke@chromium.org
  • 7 edits
    4 deletes in trunk

2010-05-18 Dirk Pranke <dpranke@chromium.org>

Reviewed by Dimitri Glazkov.

Attempt to revert r58765 to see if it was causing a performance
regression.

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

  • fast/notifications/notifications-replace-expected.txt: Removed.
  • fast/notifications/notifications-replace.html: Removed.
  • fast/notifications/notifications-rtl-expected.txt: Removed.
  • fast/notifications/notifications-rtl.html: Removed.

2010-05-18 Dirk Pranke <dpranke@chromium.org>

Reviewed by Dimitri Glazkov.

Attempt to revert r58765 to see if it was causing a performance
regression.

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

  • notifications/Notification.h:
  • notifications/Notification.idl:

2010-05-18 Dirk Pranke <dpranke@chromium.org>

Reviewed by Dimitri Glazkov.

Attempt to revert r58765 to see if it was causing a performance
regression.

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

  • public/WebNotification.h:
  • src/WebNotification.cpp:
1:35 PM Changeset in webkit [59781] by ggaren@apple.com
  • 4 edits in trunk/JavaScriptCore

2010-05-19 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Don't build the Interpreter into JIT builds.
https://bugs.webkit.org/show_bug.cgi?id=39373

SunSpider says no change.

  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
  • interpreter/Interpreter.h:
  • wtf/Platform.h:
12:17 PM Changeset in webkit [59780] by kevino@webkit.org
  • 3 edits in trunk/WebCore

Reviewed by Kevin Ollivier.

[wx] Ensure m_nsFont is initialized properly, and tweak fallback font handling
to handle more cases.
https://bugs.webkit.org/show_bug.cgi?id=39365

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

2010-05-19 Marcus Bulach <bulach@chromium.org>

Reviewed by Eric Seidel.

Fixes Windows Qt Release post r59772.
https://bugs.webkit.org/show_bug.cgi?id=39369

  • platform/GeolocationService.cpp:
12:02 PM Changeset in webkit [59778] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-19 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Add support for handling basic <script> tags in the HTML5 Parser
https://bugs.webkit.org/show_bug.cgi?id=39350

WebKit currently executes scripts from HTMLTokenizer in one giagantic
hack. HTML 5 requires that we execute scripts from the tree-builders/parser.

It will take me a while to re-factor enough of HTMLTokenizer to be able to
move the script execution logic without breaking things. In the interest
of allowing Adam to continue improving the HTML 5 lexer I've added this
very basic (and very incomplete) <script> support to the HTML 5 parser code
path so the he can run the HTML5 parser test suite.

<script> support is tested by most of the layout tests.

  • dom/ScriptElement.cpp: (WebCore::useHTML5Parser): (WebCore::ScriptElement::finishParsingChildren):
11:28 AM Changeset in webkit [59777] by ggaren@apple.com
  • 13 edits in trunk/JavaScriptCore

Standardized naming: "bytecode offset" wins; "bytecode index" loses.

Reviewed by Gavin Barraclough.

Also renamed getBytecodeOffset to bytecodeOffset, to match the WebKit
naming style.

  • bytecode/CodeBlock.h:

(JSC::CallReturnOffsetToBytecodeOffset::CallReturnOffsetToBytecodeOffset):
(JSC::getCallReturnOffset):
(JSC::CodeBlock::bytecodeOffset):
(JSC::CodeBlock::callReturnIndexVector):

  • interpreter/Interpreter.cpp:

(JSC::bytecodeOffsetForPC):

  • jit/JIT.cpp:

(JSC::JIT::JIT):
(JSC::JIT::privateCompileMainPass):
(JSC::JIT::privateCompileLinkPass):
(JSC::JIT::privateCompileSlowCases):
(JSC::JIT::privateCompile):

  • jit/JIT.h:

(JSC::CallRecord::CallRecord):
(JSC::JumpTable::JumpTable):
(JSC::SwitchRecord::SwitchRecord):

  • jit/JITCall.cpp:

(JSC::JIT::compileOpCallVarargsSlowCase):
(JSC::JIT::compileOpCall):

  • jit/JITInlineMethods.h:

(JSC::JIT::emitNakedCall):
(JSC::JIT::addSlowCase):
(JSC::JIT::addJump):
(JSC::JIT::emitJumpSlowToHot):
(JSC::JIT::isLabeled):
(JSC::JIT::map):
(JSC::JIT::unmap):
(JSC::JIT::isMapped):
(JSC::JIT::getMappedPayload):
(JSC::JIT::getMappedTag):
(JSC::JIT::emitGetVirtualRegister):

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_switch_imm):
(JSC::JIT::emit_op_switch_char):
(JSC::JIT::emit_op_switch_string):
(JSC::JIT::emit_op_new_error):

  • jit/JITOpcodes32_64.cpp:

(JSC::JIT::emit_op_mov):
(JSC::JIT::emit_op_get_global_var):
(JSC::JIT::emit_op_put_global_var):
(JSC::JIT::emit_op_get_scoped_var):
(JSC::JIT::emit_op_put_scoped_var):
(JSC::JIT::emit_op_to_primitive):
(JSC::JIT::emit_op_resolve_global):
(JSC::JIT::emit_op_to_jsnumber):
(JSC::JIT::emit_op_catch):
(JSC::JIT::emit_op_switch_imm):
(JSC::JIT::emit_op_switch_char):
(JSC::JIT::emit_op_switch_string):
(JSC::JIT::emit_op_new_error):
(JSC::JIT::emit_op_convert_this):

  • jit/JITPropertyAccess.cpp:

(JSC::JIT::emit_op_method_check):
(JSC::JIT::emitSlow_op_method_check):

  • jit/JITPropertyAccess32_64.cpp:

(JSC::JIT::emit_op_method_check):
(JSC::JIT::emitSlow_op_method_check):
(JSC::JIT::emit_op_get_by_val):
(JSC::JIT::emit_op_get_by_id):
(JSC::JIT::emit_op_get_by_pname):

  • jit/JITStubCall.h:

(JSC::JITStubCall::call):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

11:03 AM Changeset in webkit [59776] by andersca@apple.com
  • 10 edits in trunk/WebCore

2010-05-19 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Move member variables from NamedMappedAttrMap to its base class NamedNodeMap
https://bugs.webkit.org/show_bug.cgi?id=39367
<rdar://problem/8003304>

This lets us get rid of the vtable pointer in NamedNodeMap which saves 8 bytes, and also allows us to
fit the m_mappedAttributeCount unsigned int in the RefCounted padding (in 64-bit).


The net result is that we shrink the NamedMappedAttrMap size by 16 bytes, while keeping the size of NamedNodeMap the same.

  • dom/Element.cpp: (WebCore::Element::createAttributeMap):
  • dom/Element.h:
  • dom/NamedMappedAttrMap.cpp:
  • dom/NamedMappedAttrMap.h: (WebCore::NamedMappedAttrMap::NamedMappedAttrMap):
  • dom/NamedNodeMap.cpp: (WebCore::NamedNodeMap::clearAttributes):
  • dom/NamedNodeMap.h: (WebCore::NamedNodeMap::NamedNodeMap):
  • dom/Node.cpp: (WebCore::Node::dumpStatistics):
  • dom/StyledElement.cpp:
  • dom/StyledElement.h:
10:58 AM Changeset in webkit [59775] by weinig@apple.com
  • 2 edits in trunk/WebCore

Use ASSERT_UNUSED instead of UNUSED_PARAM.

Reviewed by Anders Carlsson.

  • dom/Attribute.cpp:

(WebCore::Attribute::unbindAttr):

10:57 AM Changeset in webkit [59774] by jianli@chromium.org
  • 3 edits in trunk/WebKit/chromium

Remove obsolete ChromiumBridge code that routes FileSystem calls to WebKitClient.
https://bugs.webkit.org/show_bug.cgi?id=39332

Reviewed by Darin Fisher.

  • public/WebKitClient.h:
  • src/ChromiumBridge.cpp:

(WebCore::ChromiumBridge::fileExists):
(WebCore::ChromiumBridge::deleteFile):
(WebCore::ChromiumBridge::deleteEmptyDirectory):
(WebCore::ChromiumBridge::getFileSize):
(WebCore::ChromiumBridge::getFileModificationTime):
(WebCore::ChromiumBridge::directoryName):
(WebCore::ChromiumBridge::pathByAppendingComponent):
(WebCore::ChromiumBridge::makeAllDirectories):
(WebCore::ChromiumBridge::getAbsolutePath):
(WebCore::ChromiumBridge::isDirectory):
(WebCore::ChromiumBridge::filePathToURL):

10:48 AM Changeset in webkit [59773] by weinig@apple.com
  • 292 edits
    1 delete in trunk/WebCore

Complete the merge of MappedAttribute and Attribute.

Rubber-stamped by Anders Carlsson.

Files elided for brevity.

10:38 AM Changeset in webkit [59772] by eric@webkit.org
  • 6 edits
    2 adds in trunk

2010-05-19 Marcus Bulach <bulach@chromium.org>

Reviewed by Steve Block.

[chromium] Adds supports for layout tests using GeolocationServiceMock.
https://bugs.webkit.org/show_bug.cgi?id=39081

Allows injection of GeolocationServiceMock factory.

Tests: existing fast/dom/Geolocation/*

  • platform/GeolocationService.cpp: (WebCore::GeolocationService::useMock): (WebCore::GeolocationService::useMockFactory):
  • platform/GeolocationService.h:
  • platform/chromium/GeolocationServiceChromium.cpp: (WebCore::GeolocationServiceChromium::GeolocationServiceChromium):

2010-05-19 Marcus Bulach <bulach@chromium.org>

Reviewed by Steve Block.

[chromium] Adds supports for layout tests using GeolocationServiceMock.
https://bugs.webkit.org/show_bug.cgi?id=39081

Allows injection of GeolocationServiceMock factory.
Tests: existing fast/dom/Geolocation/*

  • WebKit.gyp:
  • public/WebGeolocationServiceMock.h: Added.
  • src/WebGeolocationServiceMock.cpp: Added. (WebCore::GeolocationServiceChromiumMock::create): (WebCore::GeolocationServiceChromiumMock::GeolocationServiceChromiumMock): (WebCore::GeolocationServiceChromiumMock::startUpdating): (WebCore::GeolocationServiceChromiumMock::stopUpdating): (WebCore::GeolocationServiceChromiumMock::lastPosition): (WebCore::GeolocationServiceChromiumMock::lastError): (WebCore::GeolocationServiceChromiumMock::geolocationServicePositionChanged): (WebCore::GeolocationServiceChromiumMock::geolocationServiceErrorOccurred): (WebKit::WebGeolocationServiceMock::setMockGeolocationPosition): (WebKit::WebGeolocationServiceMock::setMockGeolocationError):
10:23 AM Changeset in webkit [59771] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-05-19 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

user.py throws exception when readline module is missing
https://bugs.webkit.org/show_bug.cgi?id=39239

  • Scripts/webkitpy/common/system/user.py:
    • The error handling path requires the "sys" module, so added an import sys at the top of the file.
9:38 AM Changeset in webkit [59770] by Martin Robinson
  • 1 edit
    4 adds in trunk/LayoutTests

2010-05-19 Martin Robinson <mrobinson@igalia.com>

Not reviewed.

Add baselines for new multi-column tests.

  • platform/gtk/fast/multicol/column-count-with-rules-expected.txt: Added.
  • platform/gtk/fast/multicol/layers-in-multicol-expected.txt: Added.
  • platform/gtk/fast/multicol/nested-columns-expected.txt: Added.
9:16 AM Changeset in webkit [59769] by yaar@chromium.org
  • 29 edits
    2 adds in trunk

2010-05-19 Justin Schuh <jschuh@chromium.org>

Reviewed by Adam Barth.

Moving frame.src checks out of the bindings
https://bugs.webkit.org/show_bug.cgi?id=37815

Moved JavaScript frame.src checks out of bindings and into
HTMLFrameElementBase. Added main thread state stack to JavaScriptCore
so ExecState is available inside core DOM. Updated affected bindings
(except for GObject, which will need to be updated to avoid origin
failures inside native code).

  • Android.jscbindings.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSCallbackData.cpp: (WebCore::JSCallbackData::invokeCallback):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::handleEvent):
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript):
  • bindings/js/JSMainThreadExecState.cpp: Added.
  • bindings/js/JSMainThreadExecState.h: Added. (WebCore::JSMainThreadExecState::currentState): (WebCore::JSMainThreadExecState::call): (WebCore::JSMainThreadExecState::evaluate): (WebCore::JSMainThreadExecState::JSMainThreadExecState): (WebCore::JSMainThreadExecState::~JSMainThreadExecState): (WebCore::JSMainThreadNullState::JSMainThreadNullState):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::executeFunctionInContext): (WebCore::ScheduledAction::execute):
  • bindings/js/ScheduledAction.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld): (WebCore::ScriptController::canAccessFromCurrentOrigin):
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptFunctionCall.cpp: (WebCore::ScriptFunctionCall::call):
  • bindings/objc/ObjCEventListener.mm:
  • bindings/objc/WebScriptObject.mm: (-[WebScriptObject callWebScriptMethod:withArguments:]): (-[WebScriptObject evaluateWebScript:]):
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/test/ObjC/DOMTestCallback.mm: (-[DOMTestCallback callbackWithClass1Param:]): (-[DOMTestCallback callbackWithClass2Param:strArg:]): (-[DOMTestCallback callbackWithNonBoolReturnType:]): (-[DOMTestCallback customCallback:class6Param:]):
  • bindings/scripts/test/ObjC/DOMTestInterface.mm:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj readOnlyIntAttr]): (-[DOMTestObj readOnlyStringAttr]): (-[DOMTestObj readOnlyTestObjAttr]): (-[DOMTestObj intAttr]): (-[DOMTestObj setIntAttr:]): (-[DOMTestObj longLongAttr]): (-[DOMTestObj setLongLongAttr:]): (-[DOMTestObj unsignedLongLongAttr]): (-[DOMTestObj setUnsignedLongLongAttr:]): (-[DOMTestObj stringAttr]): (-[DOMTestObj setStringAttr:]): (-[DOMTestObj testObjAttr]): (-[DOMTestObj setTestObjAttr:]): (-[DOMTestObj attrWithException]): (-[DOMTestObj setAttrWithException:]): (-[DOMTestObj attrWithSetterException]): (-[DOMTestObj setAttrWithSetterException:]): (-[DOMTestObj attrWithGetterException]): (-[DOMTestObj setAttrWithGetterException:]): (-[DOMTestObj customAttr]): (-[DOMTestObj setCustomAttr:]): (-[DOMTestObj scriptStringAttr]): (-[DOMTestObj voidMethod]): (-[DOMTestObj voidMethodWithArgs:strArg:objArg:]): (-[DOMTestObj intMethod]): (-[DOMTestObj intMethodWithArgs:strArg:objArg:]): (-[DOMTestObj objMethod]): (-[DOMTestObj objMethodWithArgs:strArg:objArg:]): (-[DOMTestObj methodThatRequiresAllArgs:objArg:]): (-[DOMTestObj methodThatRequiresAllArgsAndThrows:objArg:]): (-[DOMTestObj serializedValue:]): (-[DOMTestObj methodWithException]): (-[DOMTestObj customMethod]): (-[DOMTestObj customMethodWithArgs:strArg:objArg:]): (-[DOMTestObj customArgsAndException:]): (-[DOMTestObj addEventListener:listener:useCapture:]): (-[DOMTestObj removeEventListener:listener:useCapture:]): (-[DOMTestObj withDynamicFrame]): (-[DOMTestObj withDynamicFrameAndArg:]): (-[DOMTestObj withDynamicFrameAndOptionalArg:optionalArg:]): (-[DOMTestObj withDynamicFrameAndUserGesture:]): (-[DOMTestObj withDynamicFrameAndUserGestureASAD:optionalArg:]): (-[DOMTestObj withScriptStateVoid]): (-[DOMTestObj withScriptStateObj]): (-[DOMTestObj withScriptStateVoidException]): (-[DOMTestObj withScriptStateObjException]): (-[DOMTestObj methodWithOptionalArg:]): (-[DOMTestObj methodWithNonOptionalArgAndOptionalArg:opt:]): (-[DOMTestObj methodWithNonOptionalArgAndTwoOptionalArgs:opt1:opt2:]):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::canAccessFromCurrentOrigin):
  • bindings/v8/ScriptController.h:
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::isURLAllowed):

2010-05-19 Justin Schuh <jschuh@chromium.org>

Reviewed by Adam Barth.

Moving frame.src checks out of the bindings
https://bugs.webkit.org/show_bug.cgi?id=37815

  • http/tests/security/xss-DENIED-iframe-src-alias-expected.txt:
  • http/tests/security/xss-DENIED-iframe-src-alias.html:
9:11 AM Changeset in webkit [59768] by chang.shu@nokia.com
  • 12 edits
    3 adds in trunk

2010-05-19 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Darin Adler.

CanvasRenderingContext2D's property getters that return colors should
serialize them in accordance with the HTML5 spec (4.8.11.1.4)
Setters should ignore invalid colors.

Also, shadowColor should initially be transparent black (4.8.11.1.6)

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

Refactored CanvasStyle to store an RGBA32 instead of RGBA components and/or the color string.

Spec links:
http://www.whatwg.org/specs/web-apps/current-work/#serialization-of-a-color
http://www.whatwg.org/specs/web-apps/current-work/#dom-context-2d-shadowcolor

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::State::State): (WebCore::CanvasRenderingContext2D::shadowColor): (WebCore::CanvasRenderingContext2D::setShadowColor): (WebCore::CanvasRenderingContext2D::setShadow): (WebCore::CanvasRenderingContext2D::clearShadow): (WebCore::CanvasRenderingContext2D::applyShadow):
  • html/canvas/CanvasRenderingContext2D.h:
  • html/canvas/CanvasStyle.cpp: (WebCore::CanvasStyle::CanvasStyle): (WebCore::CanvasStyle::create): (WebCore::CanvasStyle::applyStrokeColor): (WebCore::CanvasStyle::applyFillColor):
  • html/canvas/CanvasStyle.h: (WebCore::CanvasStyle::create): (WebCore::CanvasStyle::color): (WebCore::CanvasStyle::): (WebCore::CanvasStyle::CMYKAValues::CMYKAValues):
  • platform/graphics/Color.cpp: (WebCore::Color::serialized):
  • platform/graphics/Color.h:

2010-05-19 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Darin Adler.

Added a test to verify proper color serialization in CanvasRenderingContext2D's getters.
https://bugs.webkit.org/show_bug.cgi?id=38845

Spec link:
http://www.whatwg.org/specs/web-apps/current-work/#serialization-of-a-color

  • fast/canvas/canvas-color-serialization-expected.txt: Added.
  • fast/canvas/canvas-color-serialization.html: Added.
  • fast/canvas/canvas-set-properties-with-non-invertible-ctm-expected.txt: Updated baseline.
  • fast/canvas/script-tests/canvas-color-serialization.js: Added. (trySettingStrokeStyle): (trySettingFillStyle): (trySettingShadowColor): (trySettingColor): (trySettingStrokeColorWithSetter): (trySettingFillColorWithSetter): (trySettingShadowWithSetter): (trySettingColorWithSetter): (trySettingFillColorRGBA): (trySettingStrokeColorRGBA): (trySettingShadowRGBA): (trySettingRGBA): (trySettingFillColorCMYKA): (trySettingStrokeColorCMYKA): (trySettingShadowCMYKA): (trySettingCMYKA): (trySettingFillColorWithOverrideAlpha): (trySettingStrokeColorWithOverrideAlpha): (trySettingShadowWithOverrideAlpha): (trySettingColorWithOverrideAlpha): (trySettingFillColorGrayLevel): (trySettingStrokeColorGrayLevel): (trySettingShadowGrayLevel): (trySettingGrayLevel): (trySettingFillColorGrayLevelWithAlpha): (trySettingStrokeColorGrayLevelWithAlpha): (trySettingShadowGrayLevelWithAlpha): (trySettingGrayLevelWithAlpha): (tryClearShadowAfterSettingColor): (checkDefaultValue):
  • fast/canvas/script-tests/canvas-set-properties-with-non-invertible-ctm.js:
  • fast/dom/canvasContext2d-element-attribute-js-null-expected.txt: Updated baseline.
  • fast/dom/canvasContext2d-element-attribute-js-null.html:
8:55 AM Changeset in webkit [59767] by chang.shu@nokia.com
  • 3 edits
    3 adds in trunk

2010-05-19 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Simon Hausmann.

[Qt] REGRESSION: CoolClock isn't rendered properly

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

CanvasRenderingContext2D's arc() should connect to the previous point
with a straight line (HTML5 spec 4.8.11.1.8), but if the path is empty
to begin with, we don't want a line back to (0,0)
This also fixes the rendering artifact discussed in bug 36226.

Spec link:
http://www.whatwg.org/specs/web-apps/current-work/#dom-context-2d-arc

Test: fast/canvas/canvas-arc-connecting-line.html

  • platform/graphics/qt/PathQt.cpp: (WebCore::Path::addArc):

2010-05-19 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Simon Hausmann.

Add test to verify that CanvasRenderingContext2D's arc() connects arcs
by a straight line (unless it's the first element in the path.)
https://bugs.webkit.org/show_bug.cgi?id=38526

  • fast/canvas/canvas-arc-connecting-line-expected.txt: Added.
  • fast/canvas/canvas-arc-connecting-line.html: Added.
  • fast/canvas/script-tests/canvas-arc-connecting-line.js: Added. (deg2rad):
8:20 AM Changeset in webkit [59766] by abecsi@webkit.org
  • 3 edits
    3 adds in trunk

The backtrackParenthesesOnceEnd function should store the start position
of the subpattern's non-greedy match at the place of the subpattern's
start index in the output array instead of the place of the
subpattern's end index.
Add layout test for checking of non-greedy matching of subpattern in
regular expressions.
https://bugs.webkit.org/show_bug.cgi?id=39289

Patch by Peter Varga <pvarga@inf.u-szeged.hu> on 2010-05-19
Reviewed by Darin Adler.

JavaScriptCore:

  • yarr/RegexInterpreter.cpp:

(JSC::Yarr::Interpreter::backtrackParenthesesOnceEnd):

LayoutTests:

  • fast/js/regexp-non-greedy-parentheses-expected.txt: Added.
  • fast/js/regexp-non-greedy-parentheses.html: Added.
  • fast/js/script-tests/regexp-non-greedy-parentheses.js: Added.
8:14 AM Changeset in webkit [59765] by antti.j.koivisto@nokia.com
  • 5 edits in trunk

[Qt] TiledBackingStore updates broken when not using resizesToContents mode
https://bugs.webkit.org/show_bug.cgi?id=39359

Rubber-stamped by Kenneth Rohde Christiansen.

Put the back logic that was lost in refactoring.

  • WebCoreSupport/ChromeClientQt.cpp:

(WebCore::ChromeClientQt::visibleRectForTiledBackingStore):

  • WebCoreSupport/PageClientQt.cpp:

(WebCore::PageClientQGraphicsWidget::graphicsItemVisibleRect):

  • WebCoreSupport/PageClientQt.h:

(WebCore::PageClientQWidget::viewResizesToContentsEnabled):
(WebCore::PageClientQGraphicsWidget::viewResizesToContentsEnabled):

8:13 AM Changeset in webkit [59764] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2010-05-19 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r59758.
http://trac.webkit.org/changeset/59758
https://bugs.webkit.org/show_bug.cgi?id=39361

This patch adds a test that fails on Leopard (Requested by
abarth on #webkit).

  • fast/canvas/webgl/texture-npot-expected.txt:
  • fast/canvas/webgl/texture-npot.html:
8:04 AM Changeset in webkit [59763] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

Symbian build fix.

[Qt] Updated the def file with exports needed for DumpRenderTree.

  • symbian/eabi/QtWebKitu.def:
6:25 AM Changeset in webkit [59762] by apavlov@chromium.org
  • 5 edits in trunk/WebCore

2010-05-19 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Jeremy Orlow.

Fix whitespace for the bug 38906
https://bugs.webkit.org/show_bug.cgi?id=39351

No new tests required (no code change).

  • css/CSSParser.cpp: (WebCore::CSSParser::~CSSParser): (WebCore::CSSParser::setupParser): (WebCore::CSSParser::parseSheet): (WebCore::CSSParser::parseValue): (WebCore::CSSParser::parseSelector): (WebCore::unitFromString): (WebCore::CSSParser::checkForOrphanedUnits): (WebCore::CSSParser::parseWCSSInputProperty): (WebCore::CSSParser::parseFillShorthand): (WebCore::CSSParser::parseAnimationShorthand): (WebCore::CSSParser::parseTransitionShorthand): (WebCore::CSSParser::parseShorthand): (WebCore::CSSParser::parse4Values): (WebCore::CSSParser::parseAttr): (WebCore::CSSParser::parseFillPositionXY): (WebCore::CSSParser::parseFillPosition): (WebCore::CSSParser::parseFillRepeat): (WebCore::CSSParser::parseFillProperty): (WebCore::CSSParser::parseAnimationTimingFunction): (WebCore::CSSParser::parseAnimationProperty): (WebCore::CSSParser::parseDashboardRegions): (WebCore::CSSParser::parseCounterContent): (WebCore::CSSParser::parseShape): (WebCore::CSSParser::parseFont): (WebCore::CSSParser::parseFontFamily): (WebCore::CSSParser::parseFontFaceSrc): (WebCore::parseInt): (WebCore::CSSParser::parseColor): (WebCore::CSSParser::parseShadow): (WebCore::CSSParser::parseReflect): (WebCore::BorderImageParseContext::commitNumber): (WebCore::BorderImageParseContext::commitBorderImage): (WebCore::CSSParser::parseBorderImage): (WebCore::CSSParser::parseCounter): (WebCore::parseGradientPoint): (WebCore::parseGradientColorStop): (WebCore::CSSParser::parseGradient): (WebCore::CSSParser::parseCanvas): (WebCore::CSSParser::parseTransform): (WebCore::CSSParser::parseTransformOrigin): (WebCore::CSSParser::parsePerspectiveOrigin): (WebCore::CSSParser::text): (WebCore::CSSParser::createRuleList): (WebCore::CSSParser::createVariablesRule): (WebCore::CSSParser::parseVariable): (WebCore::CSSParser::parsePropertyWithResolvedVariables): (WebCore::CSSParser::checkForVariables): (WebCore::CSSParser::createKeyframeRule):
  • css/CSSParser.h:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::setFrontend): (WebCore::InspectorController::show): (WebCore::InspectorController::unbindAllResources): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::scriptImported): (WebCore::InspectorController::getCurrentUserInitiatedProfileName): (WebCore::InspectorController::removeBreakpoint): (WebCore::drawOutlinedQuad): (WebCore::InspectorController::stopTiming):
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::didRemoveDOMNode): (WebCore::InspectorDOMAgent::getStyles): (WebCore::InspectorDOMAgent::applyStyleText):
6:24 AM Changeset in webkit [59761] by kenneth@webkit.org
  • 2 edits in trunk/WebKit/qt

When creating the UA, do not sassmue the language code is a
two-letter iso639-1 code.

Patch by Denis Dzyubenko <denis.dzyubenko@nokia.com> on 2010-05-19
Reviewed by Kenneth Rohde Christiansen.

  • Api/qwebpage.cpp:

(QWebPage::userAgentForUrl):

4:44 AM Changeset in webkit [59760] by antti.j.koivisto@nokia.com
  • 4 edits in trunk/WebKit/qt

[Qt] Move visible rect calculation from QGraphicsWebViewPrivate to PageClientQGraphicsWidget
https://bugs.webkit.org/show_bug.cgi?id=39294

Reviewed by Kenneth Rohde Christiansen.

  • Api/qgraphicswebview.cpp:
  • WebCoreSupport/PageClientQt.cpp:

(WebCore::PageClientQGraphicsWidget::graphicsItemVisibleRect):

  • WebCoreSupport/PageClientQt.h:
4:43 AM Changeset in webkit [59759] by eric@webkit.org
  • 3 edits in trunk

2010-05-19 Dan Winship <danw@gnome.org>

Reviewed by Xan Lopez.

[GTK] Install introspection files in our prefix, not
gobject-introspection's prefix

  • GNUmakefile.am:
  • configure.ac:
3:59 AM Changeset in webkit [59758] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2010-05-19 Zhenyao Mo <zmo@google.com>

Reviewed by Darin Adler.

fast/canvas/webgl/texture-npot.html failed on leopard bot
https://bugs.webkit.org/show_bug.cgi?id=39128

  • fast/canvas/webgl/texture-npot-expected.txt: Skip pixel (0, 0); print out actual color when failing.
  • fast/canvas/webgl/texture-npot.html: Ditto.
3:48 AM Changeset in webkit [59757] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-19 Steve Block <steveblock@google.com>

Reviewed by Nate Chapin.

V8 code generator does not add ENABLE guards around attribute getters and setters
https://bugs.webkit.org/show_bug.cgi?id=39222

No new tests, build fix only.

  • bindings/scripts/CodeGeneratorV8.pm:
3:17 AM Changeset in webkit [59756] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

2010-05-19 Jeremy Orlow <jorlow@chromium.org>

Unreviewed build fix for Chromium.

  • storage/IndexedDatabaseImpl.cpp:
3:04 AM Changeset in webkit [59755] by jorlow@chromium.org
  • 16 edits
    4 copies in trunk

2010-05-17 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

Hook Chromium's WebIndexedDatabaseImpl up to IndexedDatabaseImpl
https://bugs.webkit.org/show_bug.cgi?id=39216

All of these functions are supposed to be passing around the origin since
the caller of IndexedDatabaseImpl might not be able to pass in a Frame*
object. I tried to fix this in an ealier patch, but obviously messed up.

Also change passing security origin strings to passing SecurityOrigin
objects.

No behavior change because Chromium was emulating the old behavior before
even though the code wasn't properly hooked up.

  • storage/IndexedDatabase.h:
  • storage/IndexedDatabaseImpl.cpp: (WebCore::IndexedDatabaseImpl::open):
  • storage/IndexedDatabaseImpl.h:
  • storage/IndexedDatabaseRequest.cpp: (WebCore::IndexedDatabaseRequest::open):

2010-05-17 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

Hook Chromium's WebIndexedDatabaseImpl up to IndexedDatabaseImpl
https://bugs.webkit.org/show_bug.cgi?id=39216

WebImpl is for re-entering the WebKit layer from the browser layer.
Thus the previous IDBCallbacksProxy was mis-named (and should have been
WebIDBCallbacksImpl). This patch renames it and then implements the
proper proxy (which is for going from WebCore to WebKit).

Also change using security origin strings to using WebSecurityOrigin
objects.

This patch also implements the glue between WebIndexedDatabaseImpl and
IndexedDatabaseImpl.

  • WebKit.gyp:
  • src/IDBCallbacksProxy.cpp: (WebCore::IDBCallbacksProxy::create): (WebCore::IDBCallbacksProxy::IDBCallbacksProxy): (WebCore::IDBCallbacksProxy::onError): (WebCore::IDBCallbacksProxy::onSuccess):
  • src/IDBCallbacksProxy.h:
  • src/IndexedDatabaseProxy.cpp: (WebCore::IndexedDatabaseProxy::open):
  • src/IndexedDatabaseProxy.h:
  • src/WebIDBCallbacksImpl.cpp: Added. (WebCore::WebIDBCallbacksImpl::WebIDBCallbacksImpl): (WebCore::WebIDBCallbacksImpl::~WebIDBCallbacksImpl): (WebCore::WebIDBCallbacksImpl::onError): (WebCore::WebIDBCallbacksImpl::onSuccess):
  • src/WebIDBCallbacksImpl.h: Added.
  • src/WebIDBDatabaseImpl.cpp: Added. (WebKit::WebIDBDatabaseImpl::WebIDBDatabaseImpl): (WebKit::WebIDBDatabaseImpl::~WebIDBDatabaseImpl):
  • src/WebIDBDatabaseImpl.h: Added.
  • src/WebIndexedDatabaseImpl.cpp: (WebKit::WebIndexedDatabaseImpl::WebIndexedDatabaseImpl): (WebKit::WebIndexedDatabaseImpl::open):
  • src/WebIndexedDatabaseImpl.h:
2:37 AM Changeset in webkit [59754] by eric@webkit.org
  • 4 edits in trunk

2010-05-19 Ben Murdoch <benm@google.com>

Reviewed by Darin Adler.

Touch events are marked DontEnum in DOMWindow.idl
https://bugs.webkit.org/show_bug.cgi?id=39118

Update the expected results for enumerating the window object
properties now touch event listeners are no longer marked DontEnum.

  • platform/qt/fast/dom/Window/window-properties-expected.txt:

2010-05-19 Ben Murdoch <benm@google.com>

Reviewed by Darin Adler.

Touch events are marked DontEnum in DOMWindow.idl
https://bugs.webkit.org/show_bug.cgi?id=39118

Touch events are the only attribute event listeners on the
window object that are marked DontEnum. For consistency allow
them to be enumerated.

  • page/DOMWindow.idl: Remove DontEnum from touchcancel, touchend,

touchstart and touchmove attributes.

1:38 AM Changeset in webkit [59753] by Simon Hausmann
  • 2 edits in trunk/WebCore

[Qt] Fix build warning

NamedAttrMap.h has been renamed to NamedNodeMap.h

  • WebCore.pro:
1:22 AM Changeset in webkit [59752] by eric@webkit.org
  • 1 edit
    5 adds in trunk/WebCore

2010-05-19 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Add an HTML parsing benchmark
https://bugs.webkit.org/show_bug.cgi?id=39338

This benchmark parses the HTML5 specification 10 times and
reports how long it takes. It does this 21 times, discarding
the first warm-up run, and then prints the average time and
standard deviation.

Sharking the benchmark correctly shows over 60% of the time spent under HTMLTokenizer::write().
The numbers from the benchmark are rather consistent. On my
Mac Book Pro, I consistently see standard deviations < 2% of total time.

  • benchmarks/parser/html-parser.html: Added.
  • benchmarks/parser/resources/html5.html: Added.
12:33 AM Changeset in webkit [59751] by yuzo@google.com
  • 12 edits
    2 deletes in trunk

2010-05-18 Yuzo Fujishima <yuzo@google.com>

Reviewed by Shinichiro Hamaji.

Make CSS Parser properly handle only-for-pages pseudo-classes.

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

  • fast/css/misplaced-paged-media-pseudo-expected.txt: Removed.
  • fast/css/misplaced-paged-media-pseudo.html: Removed.
  • printing/page-rule-css-text-expected.txt:
  • printing/page-rule-css-text.html:
  • printing/pseudo-class-outside-page-expected.txt:
  • printing/pseudo-class-outside-page.html:

2010-05-18 Yuzo Fujishima <yuzo@google.com>

Reviewed by Shinichiro Hamaji.

Make CSS Parser properly handle only-for-pages pseudo-classes.

A new Match category, PagePseudoClass, is introduced to distinguish
only-for-pages pseudo-classes from others. A new symbol, pseudo_page,
is introduced to handle them separately.

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

  • css/CSSGrammar.y:
  • css/CSSParser.cpp: (WebCore::CSSParser::createMarginAtRule):
  • css/CSSParser.h:
  • css/CSSSelector.cpp: (WebCore::CSSSelector::extractPseudoType): (WebCore::CSSSelector::selectorText):
  • css/CSSSelector.h: (WebCore::CSSSelector::):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
12:26 AM Changeset in webkit [59750] by abarth@webkit.org
  • 5 edits in trunk/WebCore

2010-05-19 Adam Barth <abarth@webkit.org>

Reviewed by David Hyatt.

Change the API of nextToken to distinguish between emitting a token and just running out of input
https://bugs.webkit.org/show_bug.cgi?id=39349

I'm not sure there's a functional difference with this patch yet, but
as discussed with Eric, we need this to handle attributes.

  • html/HTML5Lexer.cpp: (WebCore::HTML5Lexer::reset): (WebCore::HTML5Lexer::nextToken):
  • html/HTML5Lexer.h:
  • html/HTML5Token.h: (WebCore::HTML5Token::HTML5Token): (WebCore::HTML5Token::clear): (WebCore::HTML5Token::beginStartTag): (WebCore::HTML5Token::beginEndTag): (WebCore::HTML5Token::beginCharacter):
  • html/HTML5Tokenizer.cpp: (WebCore::HTML5Tokenizer::write):
12:19 AM Changeset in webkit [59749] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-05-18 Eric Seidel <eric@webkit.org>

Reviewed by Maciej Stachowiak.

Refactor HTMLTokenizer::write to make it more readable and split out the lexer bits from flow control bits
https://bugs.webkit.org/show_bug.cgi?id=39318

No functional changes, only moving code into smaller functions.

This code is already covered by numerous layout tests.

HTML parsing benchmark shows this to be a wash.

  • dom/Tokenizer.h:
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::advance):
    • New funtion to encapulate code which will move into HTMLLexer eventually.
    • Use ALWAYS_INLINE to make sure compilers inline this large function (otherwise this patch is a regression).

(WebCore::HTMLTokenizer::willWriteHTML):

  • New function to handle calling any pre-write delegates.

(WebCore::HTMLTokenizer::didWriteHTML):

  • New function to handle calling any post-write delegates.

(WebCore::HTMLTokenizer::write):

  • Call the new functions.
  • html/HTMLTokenizer.h:
Note: See TracTimeline for information about the timeline view.