Timeline


and

03/01/10:

20:14 Changeset [55399] by jhoneycutt@apple.com
  • 4 edits in trunk/WebKit/win

Remove Windows line endings from some files.

Rubber-stamped by Alice Liu.

  • Interfaces/IWebEmbeddedView.idl:
  • WebCoreSupport/EmbeddedWidget.cpp:

(EmbeddedWidget::create):
(EmbeddedWidget::~EmbeddedWidget):
(EmbeddedWidget::createWindow):
(EmbeddedWidget::invalidateRect):
(EmbeddedWidget::setFrameRect):
(EmbeddedWidget::frameRectsChanged):
(EmbeddedWidget::setFocus):
(EmbeddedWidget::show):
(EmbeddedWidget::hide):
(EmbeddedWidget::windowClipRect):
(EmbeddedWidget::setParent):
(EmbeddedWidget::attachToWindow):
(EmbeddedWidget::detachFromWindow):
(EmbeddedWidget::didReceiveResponse):
(EmbeddedWidget::didReceiveData):
(EmbeddedWidget::didFinishLoading):
(EmbeddedWidget::didFail):

  • WebCoreSupport/EmbeddedWidget.h:

(EmbeddedWidget::EmbeddedWidget):

20:14 Changeset [55398] by jhoneycutt@apple.com
  • 2 edits in trunk/WebKit/win

Some WebKit DOMNode API is unimplemented.
https://bugs.webkit.org/show_bug.cgi?id=35554

Reviewed by Alice Liu.

  • DOMCoreClasses.cpp:

(DOMNode::nextSibling):
Create a DOMNode to wrap m_node's next sibling, and assign it to the
out param 'result'.
(DOMNode::insertBefore):
Query for the DOMNode for newChild, and return early if we fail. Query
refChild for DOMNode. Call insertBefore(), passing the newChild's
WebCore node and refChild's WebCore node (if refChild is non-null). If
we successfully insert the child, fill the result out param with
newChild, ref it, and return S_OK. Otherwise, return E_FAIL.
(DOMNode::removeChild):
Query oldChild for DOMNode. If we fail, return E_FAIL. Call
removeChild(), passing the node's WebCore node. If this fails, return
E_FAIL. Otherwise, fill the result out param with oldChild, ref it, and
return S_OK.

20:08 Changeset [55397] by oliver@apple.com
  • 7 edits in trunk

2010-03-01 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

PropertySlot::getValue(ExecState, unsigned) unnecessarily converts index to an Identifier
https://bugs.webkit.org/show_bug.cgi?id=35561

Fix this by defining a separate property getter function for index getters. This allows
us to pass an unsigned number without the conversion to an Identifier. We then update
setCustomIndex to take this new getter type.

  • runtime/PropertySlot.h: (JSC::PropertySlot::getValue): (JSC::PropertySlot::setCustom): (JSC::PropertySlot::setCustomIndex):

2010-03-01 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

PropertySlot::getValue(ExecState, unsigned) unnecessarily converts index to an Identifier
https://bugs.webkit.org/show_bug.cgi?id=35561

Update bindings generation and the few manual indexing getters we have to use
the new PropertySlot API.

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::indexGetter):
  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/runtime_array.cpp: (JSC::RuntimeArray::indexGetter):
  • bridge/runtime_array.h:
18:24 Changeset [55396] by ap@apple.com
  • 1 edit
    3 adds in trunk/LayoutTests

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=26520
Accessing a DOM node from a parent window in a child iframe moves the prototype to the child
iframe and resets it in the parent window

Adding a test for a bug that no longer occurs in ToT.

  • fast/dom/cross-frame-node-prototype-expected.txt: Added.
  • fast/dom/cross-frame-node-prototype.html: Added.
  • fast/dom/resources/cross-frame-node-prototype-iframe.html: Added.
17:58 Changeset [55395] by cfleizach@apple.com
  • 2 edits in trunk/WebKitTools

Fixing broken DRT on Leopard/Tiger. Second try.

AX: changes to WAI-ARIA grid aren't perceived correctly by VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=35514

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:
17:50 Changeset [55394] by cfleizach@apple.com
  • 2 edits in trunk/WebKitTools

Fixing broken DRT on Leopard/Tiger.

AX: changes to WAI-ARIA grid aren't perceived correctly by VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=35514

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:
17:40 Changeset [55393] by kov@webkit.org
  • 3 edits in trunk/LayoutTests

2010-03-01 Gustavo Noronha Silva <gns@gnome.org>

Rebaseline the two editing tests we stopped skipping - we failed
to take the new caret position result into account while
unskipping these.

  • platform/gtk/editing/selection/5109817-expected.txt:
  • platform/gtk/editing/selection/5354455-2-expected.txt:
17:35 Changeset [55392] by kov@webkit.org
  • 4 edits in trunk

WebKitTools

2010-03-01 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Eric Seidel.

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

Check invoke's return code before releasing the variant, since
there's a chance it won't be properly initialized, leading to
memory corruption, in some cases.

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: (testCallback): (testEnumerate): (testDocumentOpen): (testWindowOpen): (handleCallback):

LayoutTests

2010-03-01 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Eric Seidel.

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

Unskip the tests that failed as a consequence of the memory
corruption.

  • platform/gtk/Skipped:
17:29 Changeset [55391] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-01 Dirk Pranke <dpranke@chromium.org>

Reviewed by nobody. Build bustage :(

Fix stupid typo that I committed even after David Levin pointed
it out to me :(

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

  • Scripts/webkitpy/layout_tests/port/mac.py:
17:19 Changeset [55390] by cfleizach@apple.com
  • 13 edits
    2 adds in trunk

AX: changes to WAI-ARIA grid aren't perceived correctly by VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=35514

Reviewed by Darin Adler.

WebCore:

When a table's DOM is changed and an AX Table is not asked first for its children,
it would return wrong information. A table needs to make sure children are up to date in
all methods that can be called from the outside.

Test: platform/mac/accessibility/stale-table-rows.html

  • accessibility/AccessibilityARIAGrid.cpp:

(WebCore::AccessibilityARIAGrid::cellForColumnAndRow):

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::clearChildren):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::clearChildren):
(WebCore::AccessibilityRenderObject::updateChildrenIfNecessary):
(WebCore::AccessibilityRenderObject::children):

  • accessibility/AccessibilityRenderObject.h:

(WebCore::AccessibilityRenderObject::needsToUpdateChildren):
(WebCore::AccessibilityRenderObject::setNeedsToUpdateChildren):

  • accessibility/AccessibilityTable.cpp:

(WebCore::AccessibilityTable::clearChildren):
(WebCore::AccessibilityTable::columns):
(WebCore::AccessibilityTable::rows):
(WebCore::AccessibilityTable::rowHeaders):
(WebCore::AccessibilityTable::columnHeaders):
(WebCore::AccessibilityTable::cells):
(WebCore::AccessibilityTable::columnCount):
(WebCore::AccessibilityTable::rowCount):
(WebCore::AccessibilityTable::cellForColumnAndRow):

WebKitTools:

Add rowCount, columnCount for tables.

  • DumpRenderTree/AccessibilityUIElement.cpp:

(rowCountCallback):
(columnCountCallback):
(AccessibilityUIElement::getJSClass):

  • DumpRenderTree/AccessibilityUIElement.h:
  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:

(AccessibilityUIElement::rowCount):
(AccessibilityUIElement::columnCount):

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::rowCount):
(AccessibilityUIElement::columnCount):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::rowCount):
(AccessibilityUIElement::columnCount):

LayoutTests:

  • platform/mac/accessibility/stale-table-rows-expected.txt: Added.
  • platform/mac/accessibility/stale-table-rows.html: Added.
17:12 Changeset [55389] by eric@webkit.org
  • 4 edits in trunk

2010-03-01 José Millán Soto <jmillan@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Right click does not activate text entry
https://bugs.webkit.org/show_bug.cgi?id=29177

Stop skipping some tests which where failing because of right click
not being correct handled.

  • platform/gtk/Skipped:

2010-03-01 José Millán Soto <jmillan@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Right click does not activate text entry
https://bugs.webkit.org/show_bug.cgi?id=29177

Makes the frame handle the mouse click event before sending the
context menu event.

  • webkit/webkitwebview.cpp: (PopupMenuPositionFunc): Function created to make the popup menu appear in the correct position, especially when invoked from the keyboard. (webkit_web_view_forward_context_menu_event): Mouse click event is sent to frame before creating context menu, PopupMenuPositionFunc used to determine the position where the menu should appear. (webkit_web_view_popup_menu_handler): Improved focused node position detection. Event button set to right button.
17:07 Changeset [55388] by dpranke@chromium.org
  • 3 edits in trunk/WebKitTools

2010-03-01 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

Work around a bug in Python's subprocess.Popen() that keeps us from
cleaning up DumpRenderTree / test_shell properly when we finish the
tests in new-run-webkit-tests.

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

  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
16:56 Changeset [55387] by eric@webkit.org
  • 26 edits
    1 add in trunk

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Adam Barth.

Convert the zoom mode (page vs. text-only) into a proper enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • GNUmakefile.am:
  • WebCore.base.exp: Substituted symbols ZN7WebCore5Frame13setZoomFactorEfNS_8ZoomModeE and ZN7WebCore8Settings11setZoomModeENS_8ZoomModeE for ZN7WebCore5Frame13setZoomFactorEfb and ZN7WebCore8Settings16setZoomsTextOnlyEb, respectively.
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • page/Frame.cpp: (WebCore::Frame::zoomMode): (WebCore::Frame::shouldApplyTextZoom): (WebCore::Frame::shouldApplyPageZoom): (WebCore::Frame::setZoomFactor):
  • page/Frame.h:
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setZoomMode):
  • page/Settings.h: (WebCore::Settings::zoomMode):
  • page/ZoomMode.h: Added. (WebCore::):
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::setCurrentScale):

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::setZoomLevel):

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • webkit/webkitwebview.cpp: (DNDContentsRequest::webkit_web_view_apply_zoom_level):

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]): (-[WebView _setZoomMultiplier:isTextOnly:]): (-[WebView _realZoomMultiplierIsTextOnly]):

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • Api/qwebframe.cpp: (QWebFrame::setTextSizeMultiplier): (QWebFrame::setZoomFactor):
  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply):

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • WebFrame.cpp: (WebFrame::setTextSizeMultiplier):
  • WebView.cpp: (WebView::setZoomMultiplier): (WebView::zoomMultiplier): (WebView::canMakeTextLarger): (WebView::makeTextLarger): (WebView::canMakeTextSmaller): (WebView::makeTextSmaller): (WebView::notifyPreferencesChanged):

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • WebFrame.cpp: (wxWebFrame::IncreaseTextSize): (wxWebFrame::DecreaseTextSize): (wxWebFrame::ResetTextSize):
16:22 Changeset [55386] by eric@webkit.org
  • 9 edits
    1 copy
    5 adds in trunk

2010-03-01 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Added test for basic msubsup support.

  • mathml/presentation/subsup.xhtml: Added.
  • platform/mac/mathml/presentation/subsup-expected.checksum: Added.
  • platform/mac/mathml/presentation/subsup-expected.png: Added.
  • platform/mac/mathml/presentation/subsup-expected.txt: Added.

2010-03-01 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Added support for the msubsup element that also handles the msup and msub elements.

Test: mathml/presentation/subsup.xhtml

  • WebCore.xcodeproj/project.pbxproj:
  • mathml/MathMLInlineContainerElement.cpp: (WebCore::MathMLInlineContainerElement::createRenderer):
  • mathml/RenderMathMLSubSup.cpp: Added. (WebCore::RenderMathMLSubSup::RenderMathMLSubSup): (WebCore::RenderMathMLSubSup::addChild): (WebCore::RenderMathMLSubSup::stretchToHeight): (WebCore::RenderMathMLSubSup::nonOperatorHeight): (WebCore::RenderMathMLSubSup::layout): (WebCore::RenderMathMLSubSup::baselinePosition):
  • mathml/RenderMathMLSubSup.h: Added. (WebCore::RenderMathMLSubSup::hasBase): (WebCore::RenderMathMLSubSup::):
16:18 Changeset [55385] by jhoneycutt@apple.com
  • 4 edits in trunk/WebKit/win

<rdar://problem/7703368> IWebUIDelegatePrivate::embeddedViewWithArguments
is passed wrong arguments

Reviewed by Adam Roben.

  • Interfaces/IWebUIDelegatePrivate.idl:

Update copyright strings. Added a new key for the plug-in source URL.

  • Interfaces/WebKit.idl:

Update copyright strings.

  • WebCoreSupport/WebFrameLoaderClient.cpp:

(WebFrameLoaderClient::createPlugin):
Pass the URL of the plug-in as the source URL. Pass the document's
base URI for the base URL.

16:05 Changeset [55384] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-01 Arno Renevier <arno@renevier.net>

Reviewed by Xan Lopez.

webkit-build could pass unknown arguments to autogen.sh
https://bugs.webkit.org/show_bug.cgi?id=35454

  • Scripts/build-webkit:
15:49 Changeset [55383] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-01 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: use dejavu sans mono 11px on linux.

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

  • inspector/front-end/inspector.css:
15:17 Changeset [55382] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-01 Dirk Pranke <dpranke@chromium.org>

Rubber-stamped by Dimitri Glazkov.

Fix breakage from r55372.

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

  • Scripts/webkitpy/layout_tests/port/chromium.py:
14:51 Changeset [55381] by eric@webkit.org
  • 11 edits in trunk

2010-03-01 Thatcher Ulrich <tulrich@google.com>

Reviewed by Darin Fisher.

Fix chromium iframe shims. Add another test case to the
iframes-shims test. After r53637, the plugin widget doesn't get
moved every paint. This used to hide the bug that if an iframe
gets added, the plugin's cutout rectangles don't get updated until
a layout happens.
https://bugs.webkit.org/show_bug.cgi?id=35184

  • plugins/iframe-shims.html: test case that adds an iframe shim dynamically

2010-03-01 Thatcher Ulrich <tulrich@google.com>

Reviewed by Darin Fisher.

Fix chromium iframe shims. Add another test case to the
iframes-shims test. After r53637, the plugin widget doesn't get
moved every paint. This used to hide the bug that if an iframe
gets added, the plugin's cutout rectangles don't get updated until
a layout happens.
https://bugs.webkit.org/show_bug.cgi?id=35184

  • platform/Widget.h: (WebCore::Widget::widgetPositionsUpdated): new virtual method

widgetPositionsUpdated() to notify widgets when other widgets on
the page have been repositioned.

  • rendering/RenderView.cpp: (WebCore::RenderView::updateWidgetPositions): call widgetPositionsUpdated() on the widgets
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::widgetPositionsUpdated): call widgetPositionsUpdated() on the widget
  • rendering/RenderWidget.h:

2010-03-01 Thatcher Ulrich <tulrich@google.com>

Reviewed by Darin Fisher.

Fix chromium iframe shims. Add another test case to the
iframes-shims test. After r53637, the plugin widget doesn't get
moved every paint. This used to hide the bug that if an iframe
gets added, the plugin's cutout rectangles don't get updated until
a layout happens.
https://bugs.webkit.org/show_bug.cgi?id=35184

  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::widgetPositionsUpdated): do reportGeometry() to ensure that

the plugin is aware of the positions of cutouts on the page (for iframe shim behavior).

  • src/WebPluginContainerImpl.h:
14:27 Changeset [55380] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

2010-03-01 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

EWS can hang for five hours when compile output is too big
https://bugs.webkit.org/show_bug.cgi?id=35545

  • Scripts/webkitpy/commands/queues.py: Limit uploads to 1MB instead of 5MB. AppEngine seems to not like 5MB uploads. I'm not sure what the limit is. Let's try 1MB for a while and see how it goes.
  • Scripts/webkitpy/networktransaction.py: Tune the default parameters to our exponential backoff. I'm not sure why I picked five hours as the retry limit. That seems way too large.
14:14 Changeset [55379] by barraclough@apple.com
  • 5 edits in trunk

Bug 35537 - put_by_id does will incorrectly cache writes where a specific value exists,

where at the point of caching the same value is being written.

Reviewed by Oliver Hunt.

JavaScriptCore:

When performing a put_by_id that is replacing a property already present on the object,
there are three interesting cases regarding the state of the specific value:

(1) No specific value set - nothing to do, leave the structure in it's current state,

can cache.

(2) A specific value was set, the new put is not of a specified value (i.e. function),

or is of a different specific value - in these cases we need to perform a despecifying
transition to clear the specific value in the structure, but having done so this is a
normal property so as such we can again cache normally.

(3) A specific value was set, and we are overwriting with the same value - in these cases

leave the structure unchanged, but since a specific value is set we cannot cache this
put (we would need the JIT to dynamically check the value being written matched).

Unfortunately, the current behaviour does not match this. the checks for a specific value
being present & the value matching are combined in such a way that in case (2), above we
will unnecessarily prevent the transition being cached, but in case (3) we will incorrectly
fail to prevent caching.

The bug exposes itself if multiple puts of the same specific value are performed to a
property, and erroneously the put is allowed to be cached by the JIT. Method checks may be
generated caching calls of this structure. Subsequent puts performed from JIT code may
write different values without triggering a despecify transition, and as such cached method
checks will continue to pass, despite the value having changed.

  • runtime/JSObject.h:

(JSC::JSObject::putDirectInternal):

LayoutTests:

Add test case.

  • fast/js/method-check-expected.txt:
  • fast/js/script-tests/method-check.js:

(addOne):
(addOneHundred):
(totalizer.makeCall):

14:07 Changeset [55378] by eric@webkit.org
  • 2 edits in trunk/WebKit/gtk

2010-03-01 Kalle Vahlman <zuh@iki.fi>

Reviewed by Gustavo Noronha Silva.

Queue a resize when either of the content dimensions change
https://bugs.webkit.org/show_bug.cgi?id=35489

The check for size changes only queued a resize if both of the content
dimensions change, leaving the widget size out-of-sync if eg. only the
width changes.

  • WebCoreSupport/ChromeClientGtk.cpp:
13:51 Changeset [55377] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-01 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Holger Freyther.

Dist some mathml related files

  • GNUmakefile.am:
13:20 Changeset [55376] by kbr@google.com
  • 43 edits
    2 adds in trunk

2010-03-01 Kenneth Russell <kbr@google.com>

Reviewed by Oliver Hunt.

Integer overflow in WebGL arrays
https://bugs.webkit.org/show_bug.cgi?id=35241

Test: fast/canvas/webgl/webgl-array-invalid-ranges.html

  • bindings/js/JSWebGLArrayBufferConstructor.cpp: (WebCore::constructCanvasArrayBuffer):
  • bindings/js/JSWebGLArrayBufferConstructor.h: (WebCore::construct):
  • bindings/js/JSWebGLArrayHelper.h: (WebCore::setWebGLArrayFromArray):
  • bindings/js/JSWebGLByteArrayConstructor.cpp: (WebCore::constructCanvasByteArray):
  • bindings/js/JSWebGLFloatArrayConstructor.cpp: (WebCore::constructCanvasFloatArray):
  • bindings/js/JSWebGLIntArrayConstructor.cpp: (WebCore::constructCanvasIntArray):
  • bindings/js/JSWebGLShortArrayConstructor.cpp: (WebCore::constructCanvasShortArray):
  • bindings/js/JSWebGLUnsignedByteArrayConstructor.cpp: (WebCore::constructCanvasUnsignedByteArray):
  • bindings/js/JSWebGLUnsignedIntArrayConstructor.cpp: (WebCore::constructCanvasUnsignedIntArray):
  • bindings/js/JSWebGLUnsignedShortArrayConstructor.cpp: (WebCore::constructCanvasUnsignedShortArray):
  • bindings/v8/V8Binding.cpp: (WebCore::toUInt32):
  • bindings/v8/V8Binding.h: (WebCore::toUInt32):
  • bindings/v8/custom/V8WebGLArrayBufferCustom.cpp: (WebCore::V8WebGLArrayBuffer::constructorCallback):
  • bindings/v8/custom/V8WebGLArrayCustom.h: (WebCore::constructWebGLArray): (WebCore::getWebGLArrayElement): (WebCore::setWebGLArrayFromArray): (WebCore::setWebGLArray):
  • bindings/v8/custom/V8WebGLByteArrayCustom.cpp: (WebCore::V8WebGLByteArray::constructorCallback):
  • bindings/v8/custom/V8WebGLFloatArrayCustom.cpp: (WebCore::V8WebGLFloatArray::constructorCallback):
  • bindings/v8/custom/V8WebGLIntArrayCustom.cpp: (WebCore::V8WebGLIntArray::constructorCallback):
  • bindings/v8/custom/V8WebGLShortArrayCustom.cpp: (WebCore::V8WebGLShortArray::constructorCallback):
  • bindings/v8/custom/V8WebGLUnsignedByteArrayCustom.cpp: (WebCore::V8WebGLUnsignedByteArray::constructorCallback):
  • bindings/v8/custom/V8WebGLUnsignedIntArrayCustom.cpp: (WebCore::V8WebGLUnsignedIntArray::constructorCallback):
  • bindings/v8/custom/V8WebGLUnsignedShortArrayCustom.cpp: (WebCore::V8WebGLUnsignedShortArray::constructorCallback):
  • html/canvas/WebGLArray.cpp: (WebCore::WebGLArray::setImpl):
  • html/canvas/WebGLArray.h: (WebCore::WebGLArray::verifySubRange): (WebCore::WebGLArray::clampOffsetAndNumElements):
  • html/canvas/WebGLArrayBuffer.cpp: (WebCore::WebGLArrayBuffer::create): (WebCore::WebGLArrayBuffer::WebGLArrayBuffer): (WebCore::WebGLArrayBuffer::tryAllocate):
  • html/canvas/WebGLArrayBuffer.h:
  • html/canvas/WebGLByteArray.cpp: (WebCore::WebGLByteArray::create): (WebCore::WebGLByteArray::WebGLByteArray): (WebCore::WebGLByteArray::slice):
  • html/canvas/WebGLByteArray.h:
  • html/canvas/WebGLFloatArray.cpp: (WebCore::WebGLFloatArray::create): (WebCore::WebGLFloatArray::WebGLFloatArray): (WebCore::WebGLFloatArray::slice):
  • html/canvas/WebGLFloatArray.h:
  • html/canvas/WebGLIntArray.cpp: (WebCore::WebGLIntArray::create): (WebCore::WebGLIntArray::WebGLIntArray): (WebCore::WebGLIntArray::slice):
  • html/canvas/WebGLIntArray.h:
  • html/canvas/WebGLShortArray.cpp: (WebCore::WebGLShortArray::create): (WebCore::WebGLShortArray::WebGLShortArray): (WebCore::WebGLShortArray::slice):
  • html/canvas/WebGLShortArray.h:
  • html/canvas/WebGLUnsignedByteArray.cpp: (WebCore::WebGLUnsignedByteArray::create): (WebCore::WebGLUnsignedByteArray::WebGLUnsignedByteArray): (WebCore::WebGLUnsignedByteArray::slice):
  • html/canvas/WebGLUnsignedByteArray.h:
  • html/canvas/WebGLUnsignedIntArray.cpp: (WebCore::WebGLUnsignedIntArray::create): (WebCore::WebGLUnsignedIntArray::WebGLUnsignedIntArray): (WebCore::WebGLUnsignedIntArray::slice):
  • html/canvas/WebGLUnsignedIntArray.h:
  • html/canvas/WebGLUnsignedShortArray.cpp: (WebCore::WebGLUnsignedShortArray::create): (WebCore::WebGLUnsignedShortArray::WebGLUnsignedShortArray): (WebCore::WebGLUnsignedShortArray::slice):
  • html/canvas/WebGLUnsignedShortArray.h:

2010-03-01 Kenneth Russell <kbr@google.com>

Reviewed by Oliver Hunt.

Integer overflow in WebGL arrays
https://bugs.webkit.org/show_bug.cgi?id=35241

  • fast/canvas/webgl/array-unit-tests-expected.txt:
  • fast/canvas/webgl/array-unit-tests.html:
  • fast/canvas/webgl/webgl-array-invalid-ranges-expected.txt: Added.
  • fast/canvas/webgl/webgl-array-invalid-ranges.html: Added.
13:10 Changeset [55375] by beidson@apple.com
  • 14 edits
    2 adds in trunk

REGRESSION: Telling a WebView to go to its current WebHistoryItem is broken.
<rdar://problem/7699371> and https://bugs.webkit.org/show_bug.cgi?id=35532

Reviewed by Sam Weinig.

WebCore:

Test: fast/loader/api-test-go-to-current-back-forward-item.html

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadItem): If the current item is the same as the target item, don't

consider this to be a same document navigation.

  • page/Page.cpp:

(WebCore::Page::goToItem): Hard code the "going to the same item as the current item" relationship

as a precondition for stopping all loaders, as that will be a new document load.

WebKitTools:

  • DumpRenderTree/LayoutTestController.cpp:

(apiTestGoToCurrentBackForwardItemCallback):
(LayoutTestController::staticFunctions):

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

(LayoutTestController::apiTestGoToCurrentBackForwardItem):

Stubs for now:

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::apiTestGoToCurrentBackForwardItem):

  • DumpRenderTree/win/LayoutTestControllerWin.cpp:

(LayoutTestController::apiTestGoToCurrentBackForwardItem):

  • DumpRenderTree/wx/LayoutTestControllerWx.cpp:

(LayoutTestController::apiTestGoToCurrentBackForwardItem):

LayoutTests:

  • fast/loader/api-test-go-to-current-back-forward-item-expected.txt: Added.
  • fast/loader/api-test-go-to-current-back-forward-item.html: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
13:04 Changeset [55374] by robert@webkit.org
  • 2 edits in trunk/WebCore

2010-03-01 Robert Hogan <robert@webkit.org>

Reviewed by nobody, build fix.

Fix --minimal build. Add ENABLE(DATABASE) as compile time condition
for building file GeolocationPositionCache.cpp.

  • page/GeolocationPositionCache.cpp:
12:41 Changeset [55373] by dpranke@chromium.org
  • 12 edits in trunk/WebKitTools

2010-03-01 Dirk Pranke <dpranke@chromium.org>

Reviewed by David Levin.

General cleanup of error handling in new-run-webkit-tests.

Add Port.check_build() call that is separate from Port.check_sys_deps()
(and add a --nocheck-build flag to skip). This breaks a circular
dependency where you would start the layout test helper before
checking sys deps, but checking sys deps was the thing that told
you if your binaries where there.

Also, made Port.check_sys_deps(), start_helper() and stop_helper()
optional by providing default implementations in the base class
rather than requiring ports to implement the routines regardless
of whether or not they were needed.

Lastly, tweak a bunch of log messages to be cleaner, including
changing messages in thirdparty/autoinstall.py to be silent at
the default log level.

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

  • Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/passing.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/thirdparty/autoinstall.py
11:36 Changeset [55372] by dpranke@chromium.org
  • 5 edits in trunk/WebKitTools

2010-03-01 Dirk Pranke <dpranke@chromium.org>

Reviewed by David Levin.

new-chromium-webkit-tests --platform=mac-leopard diffs are backwards
https://bugs.webkit.org/show_bug.cgi?id=35265

Some parts of the code passed arguments as
"actual, expected" and some passed as "expected, actual".
As you might imagine, this lead to great confusion and wrongness.
Standardize on "expected, actual" as that's the order which is
passed to the underlying diff tool.

Based on a patch by Eric Siedel <eric@webkit.org>.

  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/test_types/image_diff.py
11:01 UsingGitWithWebKit edited by robert@webkit.org
(diff)
09:41 Changeset [55371] by jpetsovits@rim.com
  • 6 edits
    4 adds in trunk/WebCore

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Dirk Schulze.

[OpenVG] Implement support for paths
https://bugs.webkit.org/show_bug.cgi?id=34366

Adds an implementation of the Path class -
*almost* complete, but not quite because OpenVG
does not provide access to the points in a VGPath
unless one keeps track of all the points by
themselves, which we decided not to do.

Also hooked up to PainterOpenVG and GraphicsContext.

Further introduced is SharedResourceOpenVG, which is
intended as base class for paths and other OpenVG
resources (images, fonts) that WebKit creates as
long-lived objects. We are at a slight disadvantage
here as WebKit doesn't have the concept of resources
belonging to a specific (hardware graphics) context,
which is the reason why EGLDisplayOpenVG had to
provide a current display singleton; this class is
what actually requires that functionality.

Path::addArcTo() uses code by Yong Li <yoli@rim.com>.

  • platform/graphics/Path.cpp:
  • platform/graphics/Path.h:
  • platform/graphics/openvg/GraphicsContextOpenVG.cpp: (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::drawPath): (WebCore::GraphicsContext::beginPath): (WebCore::GraphicsContext::addPath):
  • platform/graphics/openvg/PainterOpenVG.cpp: (WebCore::PainterOpenVG::PainterOpenVG): (WebCore::PainterOpenVG::~PainterOpenVG): (WebCore::PainterOpenVG::transformPath): (WebCore::PainterOpenVG::beginPath): (WebCore::PainterOpenVG::addPath): (WebCore::PainterOpenVG::currentPath): (WebCore::PainterOpenVG::drawPath):
  • platform/graphics/openvg/PainterOpenVG.h:
  • platform/graphics/openvg/PathOpenVG.cpp: Added. (WebCore::PlatformPathOpenVG::PlatformPathOpenVG): (WebCore::PlatformPathOpenVG::operator=): (WebCore::PlatformPathOpenVG::~PlatformPathOpenVG): (WebCore::PlatformPathOpenVG::clear): (WebCore::PlatformPathOpenVG::createPath): (WebCore::Path::Path): (WebCore::Path::~Path): (WebCore::Path::operator=): (WebCore::Path::contains): (WebCore::Path::strokeContains): (WebCore::Path::translate): (WebCore::Path::boundingRect): (WebCore::Path::strokeBoundingRect): (WebCore::Path::moveTo): (WebCore::Path::addLineTo): (WebCore::Path::addQuadCurveTo): (WebCore::Path::addBezierCurveTo): (WebCore::Path::addArcTo): (WebCore::Path::closeSubpath): (WebCore::Path::addArc): (WebCore::Path::addRect): (WebCore::Path::addEllipse): (WebCore::Path::clear): (WebCore::Path::isEmpty): (WebCore::Path::hasCurrentPoint): (WebCore::Path::debugString): (WebCore::Path::apply): (WebCore::Path::transform): (WebCore::Path::length): (WebCore::Path::pointAtLength): (WebCore::Path::normalAngleAtLength):
  • platform/graphics/openvg/PlatformPathOpenVG.h: Added. (WebCore::PlatformPathOpenVG::vgPath):
  • platform/graphics/openvg/SharedResourceOpenVG.cpp: Added. (WebCore::SharedResourceOpenVG::makeSharedContextCurrent): (WebCore::SharedResourceOpenVG::makeCompatibleContextCurrent):
  • platform/graphics/openvg/SharedResourceOpenVG.h: Added.
08:18 Changeset [55370] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-03-01 Pavel Feldman <pfeldman@chromium.org>

Not reviewed, bring English.lproj/locallizedStrings.js back to binary (UTF) mode.

  • English.lproj/localizedStrings.js:
07:51 Changeset [55369] by jpetsovits@rim.com
  • 7 edits in trunk/WebCore

2010-03-01 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Nikolas Zimmermann.

[OpenVG] Convert from TransformationMatrix to AffineTransform.
https://bugs.webkit.org/show_bug.cgi?id=35445

Dirk Schulze previously introduced AffineTransform as
replacement for most TransformationMatrix occurrences
in GraphicsContext & Co., but did not update the OpenVG
graphics backend as there's no publicly available way
to build it. This commit now takes care of that.

  • platform/graphics/openvg/GraphicsContextOpenVG.cpp: (WebCore::GraphicsContext::getCTM): (WebCore::GraphicsContext::roundToDevicePixels): (WebCore::GraphicsContext::origin): (WebCore::GraphicsContext::concatCTM):
  • platform/graphics/openvg/PainterOpenVG.cpp: (WebCore::isNonRotatedAffineTransformation): (WebCore::PlatformPainterState::PlatformPainterState): (WebCore::PlatformPainterState::applyState): (WebCore::PlatformPainterState::applyTransformation): (WebCore::PainterOpenVG::transformation): (WebCore::PainterOpenVG::concatTransformation): (WebCore::PainterOpenVG::setTransformation): (WebCore::PainterOpenVG::scale): (WebCore::PainterOpenVG::rotate): (WebCore::PainterOpenVG::translate): (WebCore::PainterOpenVG::intersectClipRect):
  • platform/graphics/openvg/PainterOpenVG.h:
  • platform/graphics/openvg/VGUtils.cpp: (WebCore::VGMatrix::VGMatrix): (WebCore::VGMatrix::operator AffineTransform): (WebCore::VGMatrix::operator TransformationMatrix): (WebCore::AffineTransform::operator VGMatrix):
  • platform/graphics/openvg/VGUtils.h:
  • platform/graphics/transforms/AffineTransform.h:
07:34 Changeset [55368] by vestbo@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Fix segfault when drawing NPAPI plugins on Mac

Reviewed by Simon Hausmann.

Don't try to get a contexctRef for a null-pixmap. If the pixmap
size is 0,0 the private pixmap data has not yet been initialized.

07:34 Changeset [55367] by vestbo@webkit.org
  • 5 edits in trunk

Fix the Qt build on Mac OS X/Cocoa 64-bit

Add missing implementation file to resolve JSC symbols.

Use the proper event/drawing-model guards instead of global 64-bit
guard for the NPAPI plugin implementation (view and package).

06:34 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
06:06 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
06:00 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:58 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:48 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:46 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:42 QtWebKitTableOfFeatures46 edited by henry.haverinen@nokia.com
(diff)
05:41 QtWebKitTableOfFeatures20 created by henry.haverinen@nokia.com
05:41 QtWebKitFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:40 QtWebKitFeatures20 created by henry.haverinen@nokia.com
05:38 QtWebKit edited by henry.haverinen@nokia.com
(diff)
04:54 Changeset [55366] by kenneth@webkit.org
  • 1 edit in trunk/WebCore/ChangeLog

Fix ChangeLog entries not showing the reviewer due to the ChangeLogs
having been made manually without the (OOPS!!!) part.

Rubberstamped by Gustavo Noronha

02:17 Changeset [55365] by pfeldman@chromium.org
  • 2 edits in trunk/LayoutTests

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

Reviewed by Pavel Feldman.

Web Inspector: callerScriptName and callerScriptLine should be obfuscated for timeline tests.
https://bugs.webkit.org/show_bug.cgi?id=35478

  • ../../LayoutTests/inspector/timeline-test.js:
02:12 Changeset [55364] by pfeldman@chromium.org
  • 6 edits in trunk

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

Reviewed by Timothy Hatcher.

Web Inspector: disable breakpoint upon Shift-click.

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

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._format):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._mouseDown): (WebInspector.SourceFrame.prototype._mouseMove):
  • inspector/front-end/textViewer.css: Chromium: remove DevTools part that has been upstreamed.
  • src/js/DevTools.js:
02:07 Changeset [55363] by cjerdonek@webkit.org
  • 6 edits
    1 add
    2 deletes in trunk/WebKitTools

Unreviewed.

Rolling out: http://trac.webkit.org/changeset/55348

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

Rolling out since the changes to autoinstall do not work
with Python 2.4. In particular, ZipFile.extractall() was
added in Python 2.6.

  • Scripts/webkitpy/init.py:
  • Scripts/webkitpy/bugzilla.py:
  • Scripts/webkitpy/init/init.py: Removed.
  • Scripts/webkitpy/init/autoinstall.py: Removed.
  • Scripts/webkitpy/networktransaction.py:
  • Scripts/webkitpy/networktransaction_unittest.py:
  • Scripts/webkitpy/statusserver.py:
  • Scripts/webkitpy/thirdparty/autoinstall.py: Added.
01:44 Changeset [55362] by cjerdonek@webkit.org
  • 1 edit in trunk/WebKitTools/ChangeLog

Unreviewed.

Fixing up ChangeLog after: http://trac.webkit.org/changeset/55361

Restoring deleted ChangeLog entry, and added new ChangeLog entry
for r55361.

01:19 Changeset [55361] by cjerdonek@webkit.org
  • 5 edits
    3 deletes in trunk/WebKitTools

Unreviewed. Rolling out r55350: http://trac.webkit.org/changeset/55350

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

Need to roll out because this patch (pep8) depends on the newly rewritten
autoinstall.py (r55348), which is breaking for people with Python 2.4:

https://bugs.webkit.org/show_bug.cgi?id=35163#c21

That also needs to be rolled out and will be rolled out next.

02/28/10:

21:14 Changeset [55360] by cjerdonek@webkit.org
  • 3 edits in trunk/WebKitTools

Exempted WebKitTools/Scripts/webkitpy/thirdparty from all
style checks except for the whitespace/carriage_return check
and the pep8 tab and trailing white space checks.

Reviewed by Shinichiro Hamaji.

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

  • Scripts/webkitpy/style/checker.py:
    • Adjusted the _PATH_RULES_SPECIFIER configuration as necessary.
    • Added enough pep8 categories to _all_categories() for the unit tests to pass.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Updated the test_path_rules_specifier() unit test.
12:04 Changeset [55359] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=35496
<rdar://problem/7663444> Opening newsweek.com and youtube.com has become very slow in debug builds

  • css/CSSStyleSelector.cpp: (WebCore::CSSRuleSet::getIDRules): (WebCore::CSSRuleSet::getClassRules): (WebCore::CSSRuleSet::getTagRules): Removed consistency checks, as they affect performance too much in this case.
04:57 Changeset [55358] by robert@webkit.org
  • 7 edits
    2 adds in trunk

[Qt] Support private browsing mode in plugins

Add support for NPNVprivateModeBool property in plugins.

See also: https://developer.mozilla.org/En/Supporting_private_browsing_in_plugins

The NPNVprivateModeBool property is supported as scriptable property privateBrowsingEnabled
in the test WebKit plugin. The Mac platform also supports a cachedPrivateBrowsingEnabled
property implemented in the test plugin. This allows the Layout test plugins/private-browsing-mode.html
to retrieve the previous value of NPNVprivateModeBool in the test plugin. Due to the platform-specific
overhead required to support this bespoke property it is not implemented as part of this patch, instead
a new test, plugins/private-browsing-mode-2.html, is added to ensure that setting and resetting privateBrowsingEnabled works as expected.

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

02/27/10:

23:09 Changeset [55357] by joepeck@webkit.org
  • 2 edits in trunk/WebCore

2010-02-27 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Timothy Hatcher.

Web Inspector shouldn't show a white color box for "-webkit-line-break: after-white-space"
https://bugs.webkit.org/show_bug.cgi?id=33478

  • inspector/front-end/StylesSidebarPane.js:
14:45 Changeset [55356] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

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

Rubber-stamped by Gustavo Noronha.

Unskip test that we seem to pass now.

  • platform/gtk/Skipped:
14:26 Changeset [55355] by xan@webkit.org
  • 2 edits
    2 adds in trunk/LayoutTests

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

Reviewed by Gustavo Noronha.

Add platform results for plugin test and unskip it. The filename
of our test plugin is not the same than the one expected in the
default result file.

  • platform/gtk/Skipped:
  • platform/gtk/plugins/plugin-javascript-access-expected.txt: Added.
12:46 ApplicationsGtk edited by peterbloomfield@bellsouth.net
(diff)
12:16 ApplicationsGtk edited by peterbloomfield@bellsouth.net
info about the WebKit option in the Balsa email client (diff)
09:47 Changeset [55354] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Skip new test introduced in r55353 because of missing drag&drop support.

  • platform/qt/Skipped: fast/css/user-drag-none.html skipped.
04:39 Changeset [55353] by mitz@apple.com
  • 5 edits
    2 adds in trunk

<rdar://problem/7696607> Links do not respect -webkit-user-drag: none
https://bugs.webkit.org/show_bug.cgi?id=35475

Reviewed by Maciej Stachowiak.

WebCore:

Test: fast/css/user-drag-none.html

  • page/DragController.cpp:

(WebCore::DragController::mayStartDragAtEventLocation): Added a node
parameter. The image drag check is done against the node, rather than
than against the hit test result. This prevents a non-draggable image
with an auto-draggable ancestor from being dragged alone. The link drag
check now ignores links that are -webkit-user-drag: none.

  • page/DragController.h:
  • page/EventHandler.cpp:

(WebCore::EventHandler::shouldDragAutoNode): Pass the current node
to mayStartDragAtEventLocation().

LayoutTests:

  • fast/css/user-drag-none-expected.txt: Added.
  • fast/css/user-drag-none.html: Added.
02:48 Changeset [55352] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: REGRESSION: hangs when scrolling in Resource pane.

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

  • inspector/front-end/TextEditorHighlighter.js: (WebInspector.TextEditorHighlighter): (WebInspector.TextEditorHighlighter.prototype.highlight): (WebInspector.TextEditorHighlighter.prototype._highlightInChunks): (WebInspector.TextEditorHighlighter.prototype._highlightLines):
00:57 Changeset [55351] by xan@webkit.org
  • 4 edits in trunk

WebCore:

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

Reviewed by Gustavo Noronha.

Abort XEmbed plugin initialization if our parent is not anchored
in a widget hierarchy. This can happen when the browser window is
destroyed while the plugin is being loaded, and will lead to a
crash.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::platformStart):

LayoutTests:

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

Reviewed by Gustavo Noronha.

We now pass this test, unskip.

  • platform/gtk/Skipped:
00:24 Changeset [55350] by cjerdonek@webkit.org
  • 5 edits
    3 adds in trunk/WebKitTools

Added Python style checking to check-webkit-style using
the third-party pep8 module (via autoinstall).

Reviewed by David Levin.

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

  • Scripts/webkitpy/style/checker.py:
    • Added PYTHON to FileType.
    • Updated ProcessorDispatcher to return a PythonProcessor for *.py files.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Updated the ProcessorDispatcher unit tests for *.py files.
  • Scripts/webkitpy/style/processors/python.py: Added.
    • Added PythonProcessor class.
  • Scripts/webkitpy/style/processors/python_unittest.py: Added.
    • Added PythonProcessor unit tests.
  • Scripts/webkitpy/style/processors/python_unittest_input.py: Added.
    • Added a sample Python file to test the PythonProcessor.process() code path (since pep8 accepts a file path).
  • Scripts/webkitpy/style/unittests.py:
    • Updated the style unit test file to import python_unittest.py.
  • Scripts/webkitpy/style_references.py:
    • Adjusted style references to import pep8.
00:10 Changeset [55349] by eric@webkit.org
  • 4 edits in trunk/WebKit/mac

2010-02-27 Jing Jin <jjin@apple.com>

Reviewed by Timothy Hatcher.

Move implementation of Bug 35449 into WebFramePrivate.

  • WebView/WebFrame.h:
  • WebView/WebFrame.mm: (-[WebFrame setAllowScrollersToOverlapContent:]): (-[WebFrame setAlwaysHideHorizontalScroller:]): (-[WebFrame setAlwaysHideVerticalScroller:]):
  • WebView/WebFramePrivate.h:
00:04 Changeset [55348] by cjerdonek@webkit.org
  • 6 edits
    3 adds
    1 delete in trunk/WebKitTools

Rewrote autoinstall.py to support unzipping *.zip files after
download, unzipping and extracting *.tar.gz files after download,
and copying installed files to a destination directory.

Reviewed by David Levin.

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

These changes will allow us to autoinstall pep8.py from the web
and to put our third-party autoinstalled code in an explicit
directory like webkitpy/thirdparty/autoinstalled. They should
also speed up imports from autoinstalled *.zip packages slightly
since *.pyc files cannot be generated when importing from
zipped packages.

  • Scripts/webkitpy/init.py:
    • Updated the autoinstall lines to use the new autoinstall methods.
    • Added pep8.py to the list of auto-installed packages.
  • Scripts/webkitpy/bugzilla.py:
    • Updated mechanize import path.
  • Scripts/webkitpy/init/init.py: Copied from WebKitTools/QueueStatusServer/filters/init.py.
  • Scripts/webkitpy/init/autoinstall.py: Added.
    • Added AutoInstaller class.
    • Added sample testing usage to main.
  • Scripts/webkitpy/networktransaction.py:
    • Updated mechanize import path.
  • Scripts/webkitpy/networktransaction_unittest.py:
    • Updated mechanize import path.
  • Scripts/webkitpy/statusserver.py:
    • Updated mechanize import path.
  • Scripts/webkitpy/thirdparty/autoinstall.py: Removed.
    • Replaced with rewritten autoinstall in init/autoinstall.py.

02/26/10:

23:01 Changeset [55347] by pfeldman@chromium.org
  • 13 edits in trunk

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

Reviewed by Timothy Hatcher.

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

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

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

WebKit/chromium

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

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

Reviewed by Darin Fisher.

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

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

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::forms):
22:06 Changeset [55345] by eric@webkit.org
  • 1 edit
    5 adds in trunk/WebCore

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

Reviewed by NOBODY.

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

  • platform/efl/ScrollbarEfl.cpp: Added.
  • platform/efl/EventLoopEfl.cpp: Added.
  • platform/efl/PlatformKeyboardEventEfl.cpp: Added.
  • platform/efl/DragImageEfl.cpp: Added.
  • platform/efl/PlatformMouseEventEfl.cpp: Added.
21:49 Changeset [55344] by eric@webkit.org
  • 1 edit
    3 adds in trunk/WebCore

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

Reviewed by NOBODY.

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

  • platform/efl/CursorEfl.cpp: Added.
  • platform/efl/LocalizedStringsEfl.cpp: Added.
  • platform/efl/SearchPopupMenuEfl.cpp: Added.
21:32 Changeset [55343] by eric@webkit.org
  • 1 edit
    5 adds in trunk/WebCore

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

Reviewed by NOBODY.

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

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

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

Reviewed by NOBODY.

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

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

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

Reviewed by Adam Barth.

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

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

  • Scripts/webkitpy/layout_tests/port/factory.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
20:42 Changeset [55340] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

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

Reviewed by NOBODY.

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

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

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

Unreviewed, build fix

third time's the charm getting this path right?

  • pywebsocket/mod_pywebsocket/standalone.py:
19:53 Changeset [55338] by eric@webkit.org
  • 1 edit
    5 adds in trunk/WebCore

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

Reviewed by NOBODY.

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

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

Unreviewed, build fix.

  • pywebsocket/mod_pywebsocket/standalone.py:
19:27 Changeset [55336] by dumi@chromium.org
  • 1 edit
    1 move
    1 add in trunk/WebKitTools

Moving the script to the right location.

Reviewed by NOBODY (OOPS!).

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

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

Reviewed by Darin Fisher.

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

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

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

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

Reviewed by Darin Fisher.

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

Expose registerURLSchemeAsSecure via the WebKit API.

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

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

Reviewed by Kenneth Rohde Christiansen.

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

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

Test URL attached to the bug now works correctly.

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

Unreviewed, fixing the license.

  • pywebsocket/standalone.py:
18:43 Changeset [55332] by eric@webkit.org
  • 3 edits in trunk/WebCore

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

Reviewed by Kenneth Rohde Christiansen.

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

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

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

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

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

Reviewed by Kenneth Rohde Christiansen.

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

No new tests.

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

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

Reviewed by Kenneth Rohde Christiansen.

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

Add platform-specific code required to implement EventHandler.

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

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

Reviewed by Adam Barth.

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

  • pywebsocket: Added.
  • pywebsocket/standalone.py: Added.
17:51 Changeset [55328] by eric@webkit.org
  • 1 edit
    8 adds in trunk/WebCore

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

Reviewed by NOBODY.

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

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

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

Reviewed by Gustavo Noronha Silva.

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

  • .gitignore:
17:17 Changeset [55326] by eric@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Nate Chapin

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

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

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

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

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

Reviewed by Gustavo Noronha Silva.

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

No new tests, behaviour depends on system.

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

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

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

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

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

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

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

16:32 Changeset [55323] by yaar@chromium.org
  • 2 edits in trunk/WebCore

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

Chromium Win build fix.

  • WebCore.gyp/WebCore.gyp: Added missing file to action input.
16:19 Changeset [55322] by barraclough@apple.com
  • 12 edits
    1 delete in trunk/JavaScriptCore

Rubber Stamped by Geoff Garen.

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

(JSC::RegExp::match):

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

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

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

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

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

Chromium build fix.

  • features.gypi: Added missing new variable enable_svg.
15:18 Changeset [55319] by brettw@chromium.org
  • 3 edits in trunk/WebCore

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

Reviewed by Adam Barth.

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

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

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

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

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

Reviewed by Dimitri Glazkov.

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

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

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

Reviewed by NOBODY (Build fix following r55312).

14:39 Changeset [55316] by kov@webkit.org
  • 8 edits in trunk

WebKit/gtk

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

Reviewed by Xan Lopez.

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

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

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

WebKitTools

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

Reviewed by Xan Lopez.

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

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

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

LayoutTests

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

Reviewed by Xan Lopez.

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

Unskip tests that now pass.

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

Reviewed by Anders Carlsson.

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

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

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

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

Reviewed by Nikolas Zimmermann.

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

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

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

boundingBox of the resulting FloatQuad.

(WebCore::AffineTransform::mapQuad):

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

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

Reviewed by Pavel Feldman.

Handle vertical scrolling in the Text Viewer:

  • inspector/front-end/TextViewer.js:

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

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

14:21 Changeset [55312] by barraclough@apple.com
  • 24 edits
    1 add in trunk

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

Reviewed by Alexey Proskuryakov.

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

WebCore:

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

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

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

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

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

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

  • bridge/objc/objc_runtime.mm:

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

  • bridge/runtime_method.cpp:

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

  • bridge/runtime_object.cpp:

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

WebKit/mac:

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

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

LayoutTests:

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

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

Reviewed by Geoff Garen.

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

14:07 Changeset [55310] by eric@webkit.org
  • 4 edits
    1 copy
    13 adds in trunk

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

Reviewed by Kenneth Rohde Christiansen.

Basic tests for munder, munderover, and mover elements.

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

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

Reviewed by Kenneth Rohde Christiansen.

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

Tests: mathml/presentation/over.xhtml

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

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

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

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

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

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

  • DumpRenderTree/win/EventSender.cpp:

(keyDownCallback):

  • platform/gtk/Skipped:
13:43 Changeset [55308] by senorblanco@chromium.org
  • 2 edits in trunk/WebCore

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

Reviewed by Darin Adler.

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


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

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

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

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

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

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

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

Reviewed by Kenneth Rohde Christiansen.

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

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::markContextChanged):
13:24 Changeset [55305] by tonikitoo@webkit.org
  • 2 edits in trunk/WebKit/gtk

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

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

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

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

Reviewed by Kenneth Rohde Christiansen.

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

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

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

Reviewed by Kenneth Rohde Christiansen.

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

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

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

Reviewed by Tor Arne Vestbø.

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

No new tests. Bug fix in backend.

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

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

Rebaseline of platform-specific results for test.

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

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

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

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

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

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

  • tests/testwebview.c:

(server_callback):
(test_webkit_web_view_grab_focus):

  • webkit/webkitwebview.cpp:

(webkit_web_view_grab_focus):

12:24 Changeset [55299] by eric@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Tor Arne Vestbø.

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

No new tests. Bug fix in backend.

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

Unreviewed.

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

  • WebCoreSupport/ChromeClientGtk.cpp:
12:08 Changeset [55297] by eric@webkit.org
  • 3 edits in trunk/WebCore

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

Reviewed by Xan Lopez.

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

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

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

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

Reviewed by Simon Hausmann.

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

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

  • runtime/TimeoutChecker.cpp: (JSC::getCPUTime):
10:54 Changeset [55295] by alex@webkit.org
  • 2 edits in trunk/WebKit/gtk

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

Reviewed by Xan Lopez.

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

  • WebCoreSupport/ChromeClientGtk.cpp:
10:51 Changeset [55294] by weinig@apple.com
  • 2 edits in trunk/WebKitSite

Rubber-stamped by Adele Peterson.

Update Security Group Members list.

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

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

Unreviewed LayoutTest fix.

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

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

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

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

Reviewed by David Levin.

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

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

Unreviewed trivial fix.

[Qt] Update platform specific expected file after r55289.

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

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

Reviewed by Darin Adler.

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

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

  • page/XSSAuditor.cpp: (WebCore::XSSAuditor::findInRequest):
  • page/XSSAuditor.h:
10:02 Changeset [55289] by krit@webkit.org
  • 43 edits
    2 adds
    2 deletes in trunk

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

Reviewed by Nikolas Zimmermann.

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

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

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

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

Reviewed by Nikolas Zimmermann.

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

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

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

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

Reviewed by Simon Hausmann.

Fix documentation of QWebSettings::setUserStyleSheetUrl method

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

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

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

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

Reviewed by Kenneth Rohde Christiansen.

WebCore:

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

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

  • page/EventHandler.cpp:

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

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

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

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

  • page/EventHandler.h: Remove m_touchEventTarget.

LayoutTests:

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

the targetTouches list expections.

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

(verifyTouch):

08:51 Changeset [55286] by cjerdonek@webkit.org
  • 4 edits
    1 move
    1 add in trunk/WebKitTools

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

Reviewed by David Levin.

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

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

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

Reviewed by Darin Fisher.

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

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

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

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

Reviewed by Darin Fisher.

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

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

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

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

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

Reviewed by Eric Seidel.

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

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

  • Scripts/webkitpy/diff_parser.py:
07:48 Changeset [55283] by eric@webkit.org
  • 5 edits
    2 adds in trunk

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

Reviewed by David Levin.

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

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

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

Reviewed by David Levin.

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

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

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

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

Reviewed by David Levin.

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

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

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

Reviewed by David Levin.

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

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

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

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

Reviewed by Kenneth Rohde Christiansen.

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

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

Reviewed by Kenneth Rohde Christiansen.

WebKitTools:

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

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

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

LayoutTests:

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

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

Rubber-stamped by Xan Lopez.

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

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

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

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

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

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

Reviewed by Pavel Feldman.

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

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

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

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

Unreviewed. Skip failing tests.

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

  • platform/qt/Skipped:
02:40 Changeset [55275] by hamaji@chromium.org
  • 19 edits
    1 copy
    1 add in trunk/LayoutTests

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

Reviewed by Eric Seidel.

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

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

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

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

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

Rubber-stamped by Xan Lopez.

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

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

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

Reviewed by Laszlo Gombos.

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

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

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

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

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

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

Reviewed by Laszlo Gombos.

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

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

-inspector-url url

as an argument to QtLauncher.

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

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

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

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

Rubber-stamped by Xan Lopez.

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

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

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

WebCore:

  • editing/CompositeEditCommand.cpp:

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

  • editing/InsertTextCommand.cpp:

(WebCore::InsertTextCommand::input):

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

LayoutTests:

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

02/25/10:

23:40 Changeset [55270] by mrowe@apple.com
  • 2 edits in trunk/WebCore

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

Reviewed by Brady Eidson.

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

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

  • page/FocusController.cpp:

(WebCore::dispatchEventsOnWindowAndFocusedNode):

22:49 Changeset [55269] by darin@chromium.org
  • 4 edits in trunk/WebKit/chromium

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

Reviewed by David Levin.

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

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

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

Reviewed by Eric Seidel.

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

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

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

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

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

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

Fix typo in my last change. No review.

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

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

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

Reviewed by Simon Fraser.

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

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

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

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

Reviewed by Eric Seidel.

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

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

  • fast/loader/about-blank-hash-kept-expected.txt: Added.
  • fast/loader/about-blank-hash-kept.html: Added.
16:49 Changeset [55264] by eric@webkit.org
  • 1 edit
    2 moves in trunk/WebKitTools

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

Reviewed by Adam Barth.

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

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

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

Reviewed by Adele Peterson.

WebCore:

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

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

  • editing/CompositeEditCommand.cpp:

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

  • editing/InsertTextCommand.cpp:

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

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

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

  • editing/htmlediting.h:

LayoutTests:

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

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

Reviewed by Anders Carlsson.

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

Test: java/array-sort.html

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

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

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

Reviewed by Alexey Proskuryakov.

  • dom/SelectElement.cpp:

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

15:04 Changeset [55260] by krit@webkit.org
  • 8 edits
    4 adds in trunk

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

Reviewed by Nikolas Zimmermann.

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

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

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

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

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

Reviewed by Nikolas Zimmermann.

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

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

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

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

Reviewed by Nikolas Zimmermann.

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

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

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

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

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

Reviewed by Nikolas Zimmermann.

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

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

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

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

Reviewed by Geoff Garen.

JSC crashes like crazy in the JSPropertyNameIterator destructor

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

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

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

Reviewed by Dmitry Titov.

WebCore:

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

  • html/File.cpp:

(WebCore::File::File):

  • html/File.h:

(WebCore::File::type):

  • html/File.idl:

LayoutTests:

Update the layout test to test File.type.

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

(fileListShouldBe):
(runTest):

14:15 Changeset [55256] by oliver@apple.com
  • 7 edits in trunk

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

Reviewed by Maciej Stachowiak.

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

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

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

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

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

Reviewed by Maciej Stachowiak.

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

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

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

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

Reviewed by Pavel Feldman.

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

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

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

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

Reviewed by Darin Fisher.

[Chromium] Implement WebLabelElement.

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

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

Qt and Gtk build fixes.

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

Windows build fix.

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

Tiger build fix.

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

Reviewed by Geoffrey Garen.

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

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

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

Rubber-stamped by Xan Lopez.

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

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

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

Reviewed by Timothy Hatcher.

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

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

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

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

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

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

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

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

(WTF::threadContext):

  • wtf/ThreadingPthreads.cpp:

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

  • wtf/ThreadingWin.cpp:

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

10:20 Changeset [55246] by jorlow@chromium.org
  • 5 edits in trunk/JavaScriptCore

Reverting to re-submit with better change log.

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

(WTF::isMainThread):

  • wtf/ThreadingPthreads.cpp:

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

  • wtf/ThreadingWin.cpp:

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

10:18 Changeset [55245] by kov@webkit.org
  • 3 edits in trunk/WebKit/gtk

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

Reviewed by Xan Lopez.

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

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

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

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

Reviewed by Gustavo Noronha.

Simple style cleanups.

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

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

Reviewed by David Levin.

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

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

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

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

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

Reviewed by Darin Adler.

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

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

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

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

Reviewed by Dimitri Glazkov.

Web Inspector: make script lines count calculation lazy.

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

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

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

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

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

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

Reviewed by David Levin.

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

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

  • public/WebCookieJar.h:
09:04 Changeset [55238] by xan@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Gustavo Noronha.

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

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

Add missing include guards

Reviewed by Tor Arne Vestbø.

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

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

Reviewed by Tor Arne Vestbø.

  • WebCore.pro:
07:06 Changeset [55235] by eric@webkit.org
  • 9 edits
    1 add in trunk

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

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

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

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

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

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

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

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

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

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

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

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

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

Reviewed by Jeremy Orlow.

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

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

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

Reviewed by Timothy Hatcher.

Web Inspector: Swap images for activate and deactivate breakpoints.

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

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

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

Reviewed by Xan Lopez.

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

Thanks to Evan Martin for the help on this!

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

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

Reviewed by Timothy Hatcher.

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

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

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

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

Reviewed by Kenneth Rohde Christiansen.

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

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

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

Reviewed by Kenneth Rohde Christiansen.

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

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

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

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

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

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

docs state that it will always be >= 0.

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

Reviewed by Kenneth Rohde Christiansen.

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

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

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

Reviewed by Gustavo Noronha.

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

  • platform/gtk/Skipped:
03:48 Changeset [55228] by tonikitoo@webkit.org
  • 2 edits in trunk/WebCore

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

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

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

  • platform/graphics/IntRect.h:

(WebCore::IntRect::center):

03:24 Changeset [55227] by eric@webkit.org
  • 11 edits
    1 add in trunk/WebCore

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

Reviewed by Pavel Feldman.

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

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