Timeline


and

12/07/09:

22:00 Changeset [51836] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed fix to make QtBuildBot green.
Skip failing tests caused by r51828 (Turn on (SVG) Filters for Qt.)
https://bugs.webkit.org/show_bug.cgi?id=32224
http://build.webkit.org/results/Qt%20Linux%20Release/r51828%20(4620)/results.html

  • platform/qt/Skipped:
21:58 Changeset [51835] by hamaji@chromium.org
  • 3 edits in trunk/WebKitTools

2009-12-07 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by David Levin.

[check-webkit-style] S_OK is a fine identifier
https://bugs.webkit.org/show_bug.cgi?id=32225

  • Scripts/modules/cpp_style.py:
  • Scripts/modules/cpp_style_unittest.py:
21:48 Changeset [51834] by hamaji@chromium.org
  • 5 edits in trunk/LayoutTests

2009-12-07 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Darin Adler.

editing/selection/last-empty-inline is failing in Windows
https://bugs.webkit.org/show_bug.cgi?id=31985

  • editing/selection/last-empty-inline-expected.txt:
  • editing/selection/last-empty-inline.html:
  • platform/qt/Skipped:
  • platform/win/Skipped:
21:29 Changeset [51833] by hamaji@chromium.org
  • 3 edits in trunk/WebCore

2009-12-07 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Darin Adler.

Add a function to show render tree for debugging
https://bugs.webkit.org/show_bug.cgi?id=31288

No new tests because this just adds a function for debugging.

  • rendering/RenderObject.cpp: (WebCore::RenderObject::showRenderObject): (WebCore::RenderObject::showRenderTreeAndMark): (showRenderTree):
  • rendering/RenderObject.h:
20:55 Changeset [51832] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2009-12-07 Zhe Su <suzhe@chromium.org>

Reviewed by Darin Fisher.

[Chromium] accesskey is not supported on Chromium Mac.
https://bugs.webkit.org/show_bug.cgi?id=32213

This change fixes (partially) the accesskey issue by decoupling the
code for handling accesskeys and system keys. Because on Mac,
the modifiers of accesskeys are ctrl+alt, which are not marked as
system keys.
In order to fully fix this issue, some changes in Chromium code is
also required.

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::keyEvent): (WebKit::WebViewImpl::charEvent):
20:33 Changeset [51831] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-12-07 Victor Wang <victorw@chromium.org>

Reviewed by Dimitri Glazkov.

Implement NamedPropertyEnumerator and IndexedPropertyEnumerator for
V8 NPObject. This should make the object enumerable and fix the
enumeration issue in layout test plugins/netscape-enumerate.html.

Also fix some existing style issues per webkit style guidelines.

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

Test: plugins/netscape-enumerate.html

  • bindings/v8/V8NPObject.cpp: (npObjectInvokeImpl): (npObjectInvokeDefaultHandler): (npObjectGetProperty): (npObjectPropertyEnumerator): (npObjectNamedPropertyEnumerator): (npObjectIndexedPropertyEnumerator): (createV8ObjectForNPObject):
20:19 Changeset [51830] by ukai@chromium.org
  • 5 edits
    3 deletes in trunk

2009-12-07 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

Fragments now make WebSocket URL parsing fail.
https://bugs.webkit.org/show_bug.cgi?id=32144

  • websocket/tests/script-tests/url-parsing.js:
  • websocket/tests/script-tests/url-with-fragment.js: Removed.
  • websocket/tests/url-parsing-expected.txt:
  • websocket/tests/url-with-fragment-expected.txt: Removed.
  • websocket/tests/url-with-fragment.html: Removed.

2009-12-07 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

Fragments now make WebSocket URL parsing fail.
https://bugs.webkit.org/show_bug.cgi?id=32144

  • websockets/WebSocket.cpp: (WebCore::WebSocket::connect):
20:03 Changeset [51829] by ukai@chromium.org
  • 5 edits in trunk

2009-12-07 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Darin Adler.

Fix wrong length parsing in WebSocket.
https://bugs.webkit.org/show_bug.cgi?id=32203

These two tests assumed wrong length encoding in frame: it parsed
length from bytes with 8th bit on. But spec says length is encoded
as a series of 7-bit bytes stored in octests with the 8th bit on
*but the last byte*.
These tests encodes a frame that has 129 length, so it must be
\x81\0x01 instead of \x81\0x81.

  • websocket/tests/frame-length-longer-than-buffer_wsh.py:
  • websocket/tests/frame-length-skip_wsh.py:

2009-12-07 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Darin Adler.

Fix wrong length parsing in WebSocket.
https://bugs.webkit.org/show_bug.cgi?id=32203

  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::didReceiveData):
19:44 Changeset [51828] by zimmermann@webkit.org
  • 6 edits in trunk

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Rubber-stamped by Maciej Stachowiak.

Turn on (SVG) Filters for Qt.
https://bugs.webkit.org/show_bug.cgi?id=32224

18:54 Changeset [51827] by eric@webkit.org
  • 8 edits in trunk

2009-12-07 Evan Martin <evan@chromium.org>

Reviewed by Eric Seidel.

Chromium: theme scrollbars to match GTK theme.
Add functions to RenderThemeChromiumLinux to change the scrollbar
color.

Since the change is to the Chromium WebKit API layer, testing will
be in Chromium's test shell.
http://bugs.webkit.org/show_bug.cgi?id=32048

Patch from Markus Gutschke <markus@chromium.org>.

  • platform/chromium/ScrollbarThemeChromiumLinux.cpp: (WebCore::saturateAndBrighten): (WebCore::outlineColor): (WebCore::ScrollbarThemeChromiumLinux::paintTrackPiece): (WebCore::ScrollbarThemeChromiumLinux::paintThumb):
  • rendering/RenderThemeChromiumLinux.cpp: (WebCore::RenderThemeChromiumLinux::setScrollbarColors):
  • rendering/RenderThemeChromiumLinux.h: (WebCore::RenderThemeChromiumLinux::thumbInactiveColor): (WebCore::RenderThemeChromiumLinux::thumbActiveColor): (WebCore::RenderThemeChromiumLinux::trackColor):

2009-12-07 Evan Martin <evan@chromium.org>

Reviewed by Eric Seidel.

Chromium: theme scrollbars to match GTK theme.
Add API to set the colors.

Since the change is to the Chromium WebKit API layer, testing will
be in Chromium's test shell.
http://bugs.webkit.org/show_bug.cgi?id=32048

Patch from Markus Gutschke <markus@chromium.org>.

  • public/WebView.h:
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::setScrollbarColors):
  • src/WebViewImpl.h:
18:50 Changeset [51826] by hamaji@chromium.org
  • 2 edits in trunk/BugsSite

2009-12-07 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by David Kilzer.

Bugzilla should show images in git patches
https://bugs.webkit.org/show_bug.cgi?id=31395

Attempt to go r51748 again using --directory option of git-apply.

  • PrettyPatch/PrettyPatch.rb:
18:44 Changeset [51825] by kov@webkit.org
  • 10 edits in trunk

WebCore

Reviewed by Xan Lopez.

[GTK] Doesn't respect Content-Disposition for downloads, and provides no way for apps to do that
https://bugs.webkit.org/show_bug.cgi?id=32247

Allow creating a ResourceResponse from a SoupMessage.

Covered by API test.

  • platform/network/soup/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse):
  • platform/network/soup/ResourceResponseSoup.cpp: (WebCore::ResourceResponse::updateFromSoupMessage):

WebKit/gtk

Reviewed by Xan Lopez.

[GTK] Doesn't respect Content-Disposition for downloads, and provides no way for apps to do that
https://bugs.webkit.org/show_bug.cgi?id=32247

Provide a way for applications to respect Content-Disposition, by
fetching the WebKitNetworkResponse from the frame during
mime-type-policy-decision-requested.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDecidePolicyForMIMEType): Download content when Content-Disposition is attachment.
  • tests/testmimehandling.c: (server_callback): (mime_type_policy_decision_requested_cb):
  • tests/testwebframe.c: (test_webkit_web_frame_response): (main):
  • webkit/webkitprivate.cpp: (WebKit::core):
  • webkit/webkitwebframe.cpp: (webkit_web_frame_get_network_response):
  • webkit/webkitwebframe.h:
18:40 Changeset [51824] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-07 Yuzo Fujishima <yuzo@google.com>

Reviewed by Alexey Proskuryakov.

Make run-webkit-websocketserver log to stderr rather than to a file.
https://bugs.webkit.org/show_bug.cgi?id=32234

  • Scripts/run-webkit-websocketserver:
18:29 Changeset [51823] by zimmermann@webkit.org
  • 3 edits
    3 deletes in trunk

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Reverting problematic patch, causing errors.

Revert r51789 (Avoid zero division during SVGPaintServerPattern::setup()). Crashes on all Windows slaves.
Reopen bug https://bugs.webkit.org/show_bug.cgi?id=29912.

  • svg/graphics/SVGPaintServerPattern.cpp: (WebCore::SVGPaintServerPattern::setup):
18:26 Changeset [51822] by tkent@chromium.org
  • 6 edits
    9 adds in trunk

2009-12-07 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Add tests for HTMLInputElement::stepUp() and stepDown().
https://bugs.webkit.org/show_bug.cgi?id=27451

  • fast/forms/input-step-number-expected.txt: Added.
  • fast/forms/input-step-number.html: Added.
  • fast/forms/input-step-range-expected.txt: Added.
  • fast/forms/input-step-range.html: Added.
  • fast/forms/input-step-unsupported-expected.txt: Added.
  • fast/forms/input-step-unsupported.html: Added.
  • fast/forms/script-tests/input-step-number.js: Added.
  • fast/forms/script-tests/input-step-range.js: Added.
  • fast/forms/script-tests/input-step-unsupported.js: Added.

2009-12-07 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Add support for HTMLInputElement::stepUp() and stepDown() for
type=number and type=range.
https://bugs.webkit.org/show_bug.cgi?id=27451

Our implementation of stepUp() and stepDown() rounds the resultant
value to conform to the step value.
Change the number-string conversion method for RenderSlider to be
consistent with type=number.

Tests: fast/forms/input-step-number.html

fast/forms/input-step-range.html
fast/forms/input-step-unsupported.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::rangeUnderflow): Unify the code for NUMBER and RANGE. (WebCore::HTMLInputElement::rangeOverflow): Unify the code for NUMBER and RANGE. (WebCore::HTMLInputElement::minimum): Renamed from rangeMinimum(), and support for NUMBER. (WebCore::HTMLInputElement::maximum): Renamed from rangeMaximum(), and support for NUMBER. (WebCore::HTMLInputElement::stepBase): (WebCore::HTMLInputElement::stepMismatch): Use stepBase(). (WebCore::HTMLInputElement::applyStepForNumberOrRange): (WebCore::HTMLInputElement::stepUp): (WebCore::HTMLInputElement::stepDown): (WebCore::HTMLInputElement::formStringFromDouble):
  • html/HTMLInputElement.h: (WebCore::HTMLInputElement::stepUp): (WebCore::HTMLInputElement::stepDown):
  • html/HTMLInputElement.idl: Add stepUp() and stepDown().
  • rendering/RenderSlider.cpp: (WebCore::SliderRange::SliderRange): Sync with rangeMinimum()/rangeMaximum() renaming. (WebCore::RenderSlider::updateFromElement): Use formStringFromDouble(). (WebCore::RenderSlider::setValueForPosition): Use formStringFromDouble().
18:21 Changeset [51821] by ajwong@chromium.org
  • 2 edits in trunk/WebCore

Not reviewed: Chromium build fix try 2 (dumb error).

  • bindings/v8/ScriptValue.h:

(WebCore::ScriptValue::getString):

17:55 Changeset [51820] by ajwong@chromium.org
  • 3 edits in trunk/WebCore

Not reviewed: Chromium build fix.

Fix Chromium build break caused by an added dependency from

ScriptControllerBase to JSDOMWindowShell, and an API change in
ScriptValue.

  • bindings/ScriptControllerBase.cpp:

(WebCore::ScriptController::executeIfJavaScriptURL):

  • bindings/v8/ScriptValue.h:

(WebCore::ScriptValue::getString):

17:31 Changeset [51819] by enrica@apple.com
  • 3 edits
    2 adds in trunk

WebCore: REGRESSION(4.0.4-42a12): Indent does not work for twice modified with background text in gmail.
<rdar://problem/7442387>
https://bugs.webkit.org/show_bug.cgi?id=32233

Reviewed by Darin Adler.

Fixes problem in re-creating the correct hierarchy under the new block.
Added extensive testing to cover all the cases.

Test: editing/execCommand/indent-with-style2.html

  • editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::cloneParagraphUnderNewElement):

LayoutTests: REGRESSION(4.0.4-42a12): Indent does not work for twice modified with background text in gmail.
<rdar://problem/7442387>
https://bugs.webkit.org/show_bug.cgi?id=32233

Reviewed by Darin Adler.

  • editing/execCommand/indent-with-style2-expected.txt: Added.
  • editing/execCommand/indent-with-style2.html: Added.
17:22 Changeset [51818] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2009-12-07 Finnur Thorarinsson <finnur.webkit@gmail.com>

Reviewed by Darin Fisher.

A patch for Chromium to restrict the scope of the Gmail focus fix,
where we set the Selection to 0,0 for content-editable fields and
also make sure we set the selection end-state for find to select the
text found (when not focusing a link we found).
WebKit bug: https://bugs.webkit.org/show_bug.cgi?id=32248

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::setFindEndstateFocusAndSelection):
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::setFocus):
17:15 Changeset [51817] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

Not reviewed. Add some missing files that depend on SVG Filters support, only mac supports it for now.
Still one more crash affecting the win slaves, unrelated to the filters patch though.

17:03 Changeset [51816] by aroben@apple.com
  • 2 edits in trunk/WebKitLibraries

Windows build fix for checkouts with a space in the path

  • win/tools/scripts/auto-version.sh: Quote the output file's directory

before passing it to mkdir.

16:59 Changeset [51815] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed. Skip a timeouter flakey test.

  • platform/qt/Skipped: fast/frames/sandboxed-iframe-plugins.html added.
16:51 Changeset [51814] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

Not reviewed. Skip yet another test, what a journey...

16:50 Changeset [51813] by dimich@chromium.org
  • 15 edits in trunk

Remove ENABLE_SHARED_SCRIPT flags
https://bugs.webkit.org/show_bug.cgi?id=32245
This patch was obtained by "git revert" command and then un-reverting of ChangeLog files.

Rubber-stamped by Darin Adler.

  • configure.ac:

JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig:
  • wtf/Platform.h:

WebCore:

  • Configurations/FeatureDefines.xcconfig:
  • GNUmakefile.am:
  • WebCore.pro:

WebKit/chromium:

  • features.gypi:

WebKit/mac:

  • Configurations/FeatureDefines.xcconfig:

WebKitTools:

  • Scripts/build-webkit:
16:40 Changeset [51812] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

Not reviewed. Adjust one layout test to the results of the Leopard Intel slave.

16:37 Changeset [51811] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

Not reviewed. Turn Qt slave green again, after landing filters.

16:10 Changeset [51810] by zimmermann@webkit.org
  • 2 edits in trunk/WebCore

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Force JS binding regeneration, eventually fixing the broken tests on the bots.

  • bindings/scripts/CodeGeneratorJS.pm:
16:04 Changeset [51809] by mitz@apple.com
  • 5 edits in trunk/LayoutTests

Corrected expected results

  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.checksum:
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.png:
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-expected.checksum:
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-expected.png:
16:02 Changeset [51808] by mitz@apple.com
  • 1 edit
    1 move in trunk/LayoutTests

Rubber-stamped by Simon Fraser.

Re-enabled this test now that <rdar://problem/7450481> is fixed.

  • fast/repaint/block-selection-gap-in-composited-layer.html: Copied from LayoutTests/fast/repaint/block-selection-gap-in-composited-layer.html.disabled.
  • fast/repaint/block-selection-gap-in-composited-layer.html.disabled: Removed.
16:00 Changeset [51807] by bweinstein@apple.com
  • 1 edit in trunk/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def

Add new exported symbols to finally fix the build.

15:53 Changeset [51806] by barraclough@apple.com
  • 2 edits in trunk/WebKit/win
  • WebView.cpp:

(WebView::stringByEvaluatingJavaScriptFromString):

Reviewed by NOBODY (Windows build fix part III).

15:47 Changeset [51805] by barraclough@apple.com
  • 2 edits in trunk/WebKit/win
  • WebView.cpp:

(WebView::stringByEvaluatingJavaScriptFromString):

Reviewed by NOBODY (Windows build fix part II).

15:30 Changeset [51804] by simon.fraser@apple.com
  • 11 edits in trunk

2009-12-07 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler.

<rdar://problem/7450481> One compositing test keeps DRT in "compositing mode", breaks repaint tests

The counter that WebView used to keep track of the number of enclosed WebHTMLViews using
accelerated compositing was hard to manage, and maintained incorrectly in a number of cases.
This caused one compositing test make DumpRenderTree think that all subsequent tests
were compositing too.

Replace this counter with notifications, which are only fired if a client (DRT) requests them. The
notification informs the client that a WebHTMLView entered compositing mode (or an already-
compositing WebHTML was added); it does not say when a view becomes uncomposited, or all
compositing subviews were removed, since this is tricky to get right.

Change -[WebView _isUsingAcceleratedCompositing] to manually walk the frames, and
return YES if any document view is composited.

  • WebKit.exp:
  • WebView/WebHTMLView.mm: (-[WebHTMLView close]): (-[WebHTMLView viewDidMoveToSuperview]): (-[WebHTMLView attachRootLayer:]): (-[WebHTMLView detachRootLayer]):
  • WebView/WebView.mm: (+[WebView automaticallyNotifiesObserversForKey:]): (-[WebView _postsAcceleratedCompositingNotifications]): (-[WebView _setPostsAcceleratedCompositingNotifications:]): (-[WebView _isUsingAcceleratedCompositing]):
  • WebView/WebViewData.h:
  • WebView/WebViewInternal.h:
  • WebView/WebViewPrivate.h:
15:26 Changeset [51803] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by NOBODY (Windows build fixage part I).

15:15 Changeset [51802] by zimmermann@webkit.org
  • 2 edits in trunk/WebCore

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Try to fix mac builds by touching svgtags.in - I only tried from-scratch builds.

  • svg/svgtags.in:
15:14 Changeset [51801] by barraclough@apple.com
  • 42 edits in trunk

https://bugs.webkit.org/show_bug.cgi?id=32184
Handle out-of-memory conditions with JSC Ropes with a JS exception, rather than crashing.
Switch from using fastMalloc to tryFastMalloc, pass an ExecState to record the exception on.

Reviewed by Oliver Hunt.

JavaScriptCore:

  • API/JSCallbackObjectFunctions.h:

(JSC::::toString):

  • API/JSValueRef.cpp:

(JSValueIsStrictEqual):

(JSC::BytecodeGenerator::emitEqualityOp):

  • debugger/DebuggerCallFrame.cpp:

(JSC::DebuggerCallFrame::functionName):
(JSC::DebuggerCallFrame::calculatedFunctionName):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::callEval):
(JSC::Interpreter::privateExecute):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • profiler/ProfileGenerator.cpp:

(JSC::ProfileGenerator::addParentForConsoleStart):

  • profiler/Profiler.cpp:

(JSC::Profiler::willExecute):
(JSC::Profiler::didExecute):
(JSC::Profiler::createCallIdentifier):
(JSC::createCallIdentifierFromFunctionImp):

  • profiler/Profiler.h:
  • runtime/ArrayPrototype.cpp:

(JSC::arrayProtoFuncIndexOf):
(JSC::arrayProtoFuncLastIndexOf):

  • runtime/DateConstructor.cpp:

(JSC::constructDate):

  • runtime/FunctionPrototype.cpp:

(JSC::functionProtoFuncToString):

  • runtime/InternalFunction.cpp:

(JSC::InternalFunction::name):
(JSC::InternalFunction::displayName):
(JSC::InternalFunction::calculatedDisplayName):

  • runtime/InternalFunction.h:
  • runtime/JSCell.cpp:

(JSC::JSCell::getString):

  • runtime/JSCell.h:

(JSC::JSValue::getString):

  • runtime/JSONObject.cpp:

(JSC::gap):
(JSC::Stringifier::Stringifier):
(JSC::Stringifier::appendStringifiedValue):

  • runtime/JSObject.cpp:

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

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

(JSC::JSPropertyNameIterator::get):

  • runtime/JSString.cpp:

(JSC::JSString::Rope::~Rope):
(JSC::JSString::resolveRope):
(JSC::JSString::getPrimitiveNumber):
(JSC::JSString::toNumber):
(JSC::JSString::toString):
(JSC::JSString::toThisString):
(JSC::JSString::getStringPropertyDescriptor):

  • runtime/JSString.h:

(JSC::JSString::Rope::createOrNull):
(JSC::JSString::Rope::operator new):
(JSC::JSString::value):
(JSC::JSString::tryGetValue):
(JSC::JSString::getIndex):
(JSC::JSString::getStringPropertySlot):
(JSC::JSValue::toString):

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

(JSC::NativeErrorConstructor::NativeErrorConstructor):

  • runtime/Operations.cpp:

(JSC::JSValue::strictEqualSlowCase):

  • runtime/Operations.h:

(JSC::JSValue::equalSlowCaseInline):
(JSC::JSValue::strictEqualSlowCaseInline):
(JSC::JSValue::strictEqual):
(JSC::jsLess):
(JSC::jsLessEq):
(JSC::jsAdd):
(JSC::concatenateStrings):

  • runtime/PropertyDescriptor.cpp:

(JSC::PropertyDescriptor::equalTo):

  • runtime/PropertyDescriptor.h:
  • runtime/StringPrototype.cpp:

(JSC::stringProtoFuncReplace):
(JSC::stringProtoFuncToLowerCase):
(JSC::stringProtoFuncToUpperCase):

WebCore:

  • bindings/ScriptControllerBase.cpp:

(WebCore::ScriptController::executeIfJavaScriptURL):

  • bindings/js/JSCanvasRenderingContext2DCustom.cpp:

(WebCore::toHTMLCanvasStyle):
(WebCore::JSCanvasRenderingContext2D::setFillColor):
(WebCore::JSCanvasRenderingContext2D::setStrokeColor):
(WebCore::JSCanvasRenderingContext2D::setShadow):

  • bindings/js/ScriptCallStack.cpp:

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

  • bindings/js/ScriptValue.cpp:

(WebCore::ScriptValue::getString):

  • bindings/js/ScriptValue.h:
  • bindings/js/SerializedScriptValue.cpp:

(WebCore::SerializingTreeWalker::convertIfTerminal):

  • bindings/objc/WebScriptObject.mm:

(+[WebScriptObject _convertValueToObjcValue:originRootObject:rootObject:]):

  • page/Console.cpp:

(WebCore::Console::addMessage):

WebKit/mac:

  • WebView/WebView.mm:

(aeDescFromJSValue):

15:03 Changeset [51800] by zimmermann@webkit.org
  • 249 edits
    2 adds in trunk

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Holger Hans Peter Freyther.

Turn on (SVG) Filters support, by default.
https://bugs.webkit.org/show_bug.cgi?id=32224

Only enable filters on mac for now, skipped all affected tests on the other platforms.
Will enable them one after another, to minimize breakage.

  • Configurations/FeatureDefines.xcconfig: Enable FILTERS build flag.
14:50 Changeset [51799] by eric@webkit.org
  • 5 edits in trunk

2009-12-07 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Some changes to ValidityState-typeMismatch-url.html.
https://bugs.webkit.org/show_bug.cgi?id=29913

  • Add more test cases Username, IPv6 address, mailto:, file:, tel:, fragment, and IDN.
  • Handle difference between KURL and KURLGoogle
  • fast/forms/ValidityState-typeMismatch-url-expected.txt:
  • fast/forms/script-tests/ValidityState-typeMismatch-url.js:

2009-12-07 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Fix a bug that KURL rejects a host name which has an IPv4 address
inside an IPv6 form like [::127.0.0.1].
https://bugs.webkit.org/show_bug.cgi?id=29913

  • platform/KURL.cpp: (WebCore::characterClassTable): Add IPv6Char to '.'.
14:43 Changeset [51798] by darin@chromium.org
  • 5 edits in trunk/LayoutTests

2009-12-07 Darin Fisher <darin@chromium.org>

Reviewed by Darin Adler.

http/tests/cache/subresource-failover-to-network.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=32038

Rewrite test to avoid dumpResourceLoadCallbacks, which do not
interleave predictably with the output of dumpFrameLoadCallbacks.
Other things besides resource loading can delay onload.

The newly written test uses a cookie to determine if the test has
passed.

Unskip this test for platform/win.

  • http/tests/cache/resources/subresource-failover-to-network.cgi:
  • http/tests/cache/subresource-failover-to-network-expected.txt:
  • http/tests/cache/subresource-failover-to-network.html:
  • platform/win/Skipped:
14:35 Changeset [51797] by ap@apple.com
  • 1 edit
    2 adds in trunk/LayoutTests

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=32244
Test sending and receiving small WebSocket messages of different lengths

  • websocket/tests/frame-lengths-expected.txt: Added.
  • websocket/tests/frame-lengths.html: Added.
14:27 Changeset [51796] by beidson@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/7443417> - Closing a window opened with "New windows open with: Same Page" results in crash in ::~HistoryItem

Reviewed by Sam Weinig.

The HistoryItem copy constructor has one very specific use case - to support the feature of a WebView copying another WebView's
BackForwardList. That constructor wasn't initializing its m_document pointer, leading to this crash in the destructor.

  • history/HistoryItem.cpp:

(WebCore::HistoryItem::~HistoryItem): Change the setDocument(0) to an ASSERT that the document is already 0. A HistoryItem cannot

outlive its Document and Documents will always clear the back-pointer in their ::detach() method.

(WebCore::HistoryItem::HistoryItem): Missed the m_document initializer in the copy constructor. Also remove an invalid assertion

about the original items m_cachedPage object (which is irrelevant to the copy).

13:54 Changeset [51795] by dimich@chromium.org
  • 1 edit in trunk/WebCore/ChangeLog

Not reviewed - fix typo in ChangeLog.

13:52 Changeset [51794] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-07 Yaar Schnitman <yaar@chromium.org>

Reviewed by Darin Fisher.

Chromium buildbots are not red when they fail
https://bugs.webkit.org/show_bug.cgi?id=32235

  • Scripts/build-webkit:
13:51 Changeset [51793] by dimich@chromium.org
  • 8 edits
    9 deletes in trunk/WebCore

Remove partial SharedScript implementation.
https://bugs.webkit.org/show_bug.cgi?id=32237
The patch was obtained by running "git revert" command and then un-reverting WebCore/ChangeLog.

Reviewed by Daring Adler.

  • DerivedSources.make:
  • SharedScript/SharedScriptContext.cpp: Removed.
  • SharedScript/SharedScriptContext.h: Removed.
  • SharedScript/SharedScriptContext.idl: Removed.
  • SharedScript/SharedScriptController.h: Removed.
  • SharedScript/WebKitSharedScript.cpp: Removed.
  • SharedScript/WebKitSharedScript.h: Removed.
  • SharedScript/WebKitSharedScript.idl: Removed.
  • SharedScript/WebKitSharedScriptRepository.cpp: Removed.
  • SharedScript/WebKitSharedScriptRepository.h: Removed.
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/Document.cpp:

(WebCore::Document::detach):

  • dom/EventTarget.cpp:
  • dom/EventTarget.h:
  • dom/ScriptExecutionContext.h:
13:25 Changeset [51792] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-07 Yaar Schnitman <yaar@chromium.org>

Reviewed by Darin Fisher.

Typo in chromium linux builder
https://bugs.webkit.org/show_bug.cgi?id=32238

  • Scripts/webkitdirs.pm:
13:06 Changeset [51791] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Add two tests generating new results to gtks collection of tests doing so.

  • platform/gtk/Skipped:
13:05 Changeset [51790] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Nikolas Zimmermann.

https://bugs.webkit.org/show_bug.cgi?id=32226
Random crashes in WebSocket tests on Leopard Debug builbot

  • platform/network/SocketStreamHandleBase.cpp: (WebCore::SocketStreamHandleBase::close): Make sure the object still exists when setting m_state.
12:32 Changeset [51789] by krit@webkit.org
  • 3 edits
    3 adds in trunk

2009-12-07 Shiki Okasaka <shiki@google.com>

Reviewed by Dirk Schulze.

Avoid zero division during SVGPaintServerPattern::setup()
https://bugs.webkit.org/show_bug.cgi?id=29912

Fix zero division bugs in SVGPaintServerPattern::setup() that occurred
if the tile of a pattern was bigger than the pattern and the pattern
size was < 0.5, and if the attribute overflow was set to visible.

Test: svg/custom/small-pattern.html

  • svg/graphics/SVGPaintServerPattern.cpp: (WebCore::SVGPaintServerPattern::setup):

2009-12-07 Dirk Schulze <krit@webkit.org>

Reviewed by Adam Barth.

If the tile of a pattern is bigger than the pattern and the
pattern size is < 0.5, the SVGPaintServer produced a division
by zero. This also only happens, if the attribute overflow is
set to visible.
This is a test with a pattern size of 0.1 and a tile size of 1.

  • svg/custom/script-tests/small-pattern.js: Added.
  • svg/custom/small-pattern-expected.txt: Added.
  • svg/custom/small-pattern.html: Added.
12:17 Changeset [51788] by eric@webkit.org
  • 6 edits in trunk

2009-12-07 Yaar Schnitman <yaar@chromium.org>

Reviewed by Darin Fisher.

fixes to build-webkit --chromium
https://bugs.webkit.org/show_bug.cgi?id=32179

  • DEPS:
  • WebKit.gyp:
  • gyp_webkit:

2009-12-07 Yaar Schnitman <yaar@chromium.org>

Reviewed by Darin Fisher.

fixes to build-webkit --chromium
https://bugs.webkit.org/show_bug.cgi?id=32179

  • Scripts/webkitdirs.pm:
11:46 Changeset [51787] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-07 Eric Seidel <eric@webkit.org>

No review, just adding two recently approved committers.

  • Scripts/modules/committers.py:
11:36 Changeset [51786] by ap@apple.com
  • 2 edits in trunk/WebKitTools
  • Scripts/run-webkit-websocketserver: Added property svn:executable.
11:36 Changeset [51785] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2009-12-07 Kenneth Russell <kbr@google.com>

Reviewed by Dimitri Glazkov.

[v8] WebCore::WebGLArrayBufferInternal::byteLengthAttrGetter NULL pointer
https://bugs.webkit.org/show_bug.cgi?id=31889

Fixed bug in handling of zero-argument constructor call.

Test: fast/canvas/webgl/bug-31889.html

  • fast/canvas/webgl/bug-31889-expected.txt: Added.
  • fast/canvas/webgl/bug-31889.html: Added.

2009-12-07 Kenneth Russell <kbr@google.com>

Reviewed by Dimitri Glazkov.

[v8] WebCore::WebGLArrayBufferInternal::byteLengthAttrGetter NULL pointer
https://bugs.webkit.org/show_bug.cgi?id=31889

Fixed bug in handling of zero-argument constructor call.

Test: fast/canvas/webgl/bug-31889.html

  • bindings/v8/custom/V8WebGLArrayBufferCustom.cpp: (WebCore::CALLBACK_FUNC_DECL):
11:26 Changeset [51784] by kov@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

Unreviewed. Landing files I forgot to add to my working copy before
committing r51734.

11:22 Changeset [51783] by eric@webkit.org
  • 4 edits in trunk

2009-12-07 Gyuyoung Kim <gyuyoung@gmail.com>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=32024
[GTK] WebKit does not compile without JAVASCRIPT_DEBUGGER

Fix build errors when javascript-debugger is disabled on GTK

  • GNUmakefile.am:

2009-12-07 Gyuyoung Kim <gyuyoung@gmail.com>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=32024
[GTK] WebKit does not compile without JAVASCRIPT_DEBUGGER

Fix build errors when javascript-debugger is disabled on GTK

  • webkit/webkitwebinspector.cpp: (webkit_web_inspector_set_property): (webkit_web_inspector_get_property):
11:09 Changeset [51782] by ap@apple.com
  • 2 edits in trunk/LayoutTests

Don't skip websocket/tests/url-with-credential.html, it's one of earlier tests that is
to blame.

  • platform/mac/Skipped:
11:06 Changeset [51781] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Steve Falkenburg.

Add one more fast/js test to the Skipped list - also due to:
https://bugs.webkit.org/show_bug.cgi?id=32228.

  • platform/win/Skipped:
10:51 Changeset [51780] by zimmermann@webkit.org
  • 3 edits in trunk/LayoutTests

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Cleaning up Skipped files in order to make more bots green, as I plan to land changes, that affect all bots (svg filters support).

Remove non-existant animations/3d directory in qt&win Skipped lists.
Skip one more test on windows (see bug #32228, already in radar, just guessing - not yet sure if it's a random crash issue though)
Skip 19 (!) tests on the Qt slave, failing since a longer time. Qt developers: Please investigate.

  • platform/qt/Skipped: Also remove some duplicates I spotted.
  • platform/win/Skipped:
10:34 Changeset [51779] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebKitTools

2009-12-07 Yuzo Fujishima <yuzo@google.com>

Reviewed by Alexey Proskuryakov.

Add run-webkit-websocketserver

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

  • Scripts/run-webkit-websocketserver: Added.
10:23 Changeset [51778] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Disable two flaky SnowLeopard tests, filed bug and notified author.

Tests "plugins/document-open.html" and "plugins/mouse-events.html" are flaky on SnowLeopard
https://bugs.webkit.org/show_bug.cgi?id=32229

  • platform/mac-snowleopard/Skipped:
10:23 Changeset [51777] by sfalken@apple.com
  • 4 edits in tags/Safari-532.5.1

Merge r51773.

10:22 Changeset [51776] by mitz@apple.com
  • 7 edits
    8 adds in trunk

WebCore: Fixed <rdar://problem/7437820> Weird selection artifacts

Reviewed by Darin Adler.

Tests: fast/repaint/block-selection-gap-stale-cache-2.html

fast/repaint/block-selection-gap-stale-cache.html

Instead of caching the block selection gaps’ bounds in the RenderView at setSelection()
time, cache them in each RenderLayer at paint time. This prevents the cache from getting
stale due to layout changes and overflow scrolling.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::selectionGapRectsForRepaint): Account for overflow scroll.
(WebCore::RenderBlock::paintSelection): Update the enclosing layer’s selection gaps bounds.

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::addBlockSelectionGapsBounds): Added. Updates the selection gaps
bounds to include the given rect.
(WebCore::RenderLayer::clearBlockSelectionGapsBounds): Added. Recursively clears the cached
selection gaps bounds.
(WebCore::RenderLayer::repaintBlockSelectionGaps): Added. Recursively invalidates the
selection gaps bounds.

  • rendering/RenderLayer.h:
  • rendering/RenderView.cpp:

(WebCore::RenderView::setSelection): Clear the layer-level selection gaps bounds instead
of the view-level cache.
(WebCore::RenderView::clearSelection): Changed to call repaintBlockSelectionGaps().

  • rendering/RenderView.h:

LayoutTests: Tests for <rdar://problem/7437820> Weird selection artifacts

Reviewed by Darin Adler.

  • fast/repaint/block-selection-gap-stale-cache-2.html: Added.
  • fast/repaint/block-selection-gap-stale-cache.html: Added.
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.checksum: Added.
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.png: Added.
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.txt: Added.
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-expected.checksum: Added.
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-expected.png: Added.
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-expected.txt: Added.
10:09 Changeset [51775] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed.

REGRESSION: fast/js/excessive-comma-usage.html and fast/js/math-transforms.html fail on win
https://bugs.webkit.org/show_bug.cgi?id=32228

Adding above two tests to the Skipped list on win, attempting to turn the win slave green again.

  • platform/win/Skipped:
10:06 Changeset [51774] by sfalken@apple.com
  • 1 copy in tags/Safari-532.5.1

New tag.

10:03 Changeset [51773] by sfalken@apple.com
  • 4 edits in trunk

Build fix. Be flexible about which version of ICU is used on Windows.

JavaScriptCore:

  • JavaScriptCore.vcproj/jsc/jscCommon.vsprops: Add optional xcopy commands to copy ICU 4.2.

WebKitTools:

  • DumpRenderTree/win/DumpRenderTree.vcproj: Add optional xcopy commands to copy ICU 4.2.
09:43 Changeset [51772] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Remove non-existant files in gtk/Skipped list, and exclude a printing test, without results. Should turn the gtk slave green again.

  • platform/gtk/Skipped:
09:21 Changeset [51771] by zimmermann@webkit.org
  • 2 edits in trunk/LayoutTests

2009-12-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Attempt to get all mac build slaves to green again. Disable crashing test, and notify the author of it to fix it.

New test "websocket/tests/url-with-credential.html" crashes in debug builds
https://bugs.webkit.org/show_bug.cgi?id=32226

  • platform/mac/Skipped:
09:10 Changeset [51770] by zimmermann@webkit.org
  • 1 edit
    1 add in trunk/LayoutTests

Not reviewed. Add missing layout test result.

09:07 Changeset [51769] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-12-07 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha.

Fixed whitelist comment and removed useless (tags, sdp) mimetypes
from it. Also, GStreamer doesn't handle directly SMIL.

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::mimeTypeCache):
07:38 Changeset [51768] by eric@webkit.org
  • 4 edits in trunk

2009-12-07 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25415
[GTK][ATK] Please implement support for get_text_at_offset

Eliminate the segfaults which occur when accessing the text interface now
implemented by text controls.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (getPangoLayoutForAtk):

2009-12-07 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25415
[GTK][ATK] Please implement support for get_text_at_offset

Eliminate the segfaults which occur when accessing the text interface now
implemented by text controls.

  • tests/testatk.c (test_webkit_atk_get_text_at_offset_textarea): (test_webkit_atk_get_text_at_offset_text_input): (main):
07:31 Changeset [51767] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-12-07 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Improve the autotests of QtWebkit
https://bugs.webkit.org/show_bug.cgi?id=32216

Remove the calls to qWait() of the autotest of QWebView

  • tests/qwebview/tst_qwebview.cpp: (tst_QWebView::reusePage):
07:02 Changeset [51766] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2009-12-07 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Pavel Feldman.

[Chromium] Add declarations for DevTools message dispatching on IO thread
(see http://codereview.chromium.org/460018)

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

  • public/WebDevToolsAgent.h:
  • public/WebDevToolsAgentClient.h:
06:52 Changeset [51765] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-12-07 Maciej Stachowiak <mjs@apple.com>

Reviewed by Oliver Hunt.

op_loop_if_less JIT codegen is broken for 64-bit
https://bugs.webkit.org/show_bug.cgi?id=32221

  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_loop_if_false): Fix codegen in this version - test was backwards.
06:35 Changeset [51764] by kov@webkit.org
  • 5 edits
    6 adds in trunk

Reviewed by Xan Lopez.

Build the new API test.

[GTK] REGRESSION: webkit thinks it can render PDFs
https://bugs.webkit.org/show_bug.cgi?id=32183

  • GNUmakefile.am:

WebCore

Reviewed by Xan Lopez.

Turn the MediaPlayer supported types blacklist into a whitelist

[GTK] REGRESSION: webkit thinks it can render PDFs
https://bugs.webkit.org/show_bug.cgi?id=32183

Covered by API test.

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::mimeTypeCache):

WebKit/gtk

Reviewed by Xan Lopez.

New test to make sure we do not regress this feature anymore. All
test files were created by me.

[GTK] REGRESSION: webkit thinks it can render PDFs
https://bugs.webkit.org/show_bug.cgi?id=32183

  • tests/resources/test.html: Added.
  • tests/resources/test.ogg: Added.
  • tests/resources/test.pdf: Added.
  • tests/resources/test.txt: Added.
  • tests/testmimehandling.c: Added. (server_callback): (idle_quit_loop_cb): (mime_type_policy_decision_requested_cb): (test_mime_type): (test_mime_pdf): (test_mime_html): (test_mime_text): (test_mime_ogg): (main):
06:34 Changeset [51763] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-12-07 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Improve the autotests of QtWebkit
https://bugs.webkit.org/show_bug.cgi?id=32216

Refactor tst_qwebframe to remove qWait() and use
the function waitForSignal() from util.h

  • tests/qwebframe/tst_qwebframe.cpp:
06:27 Changeset [51762] by eric@webkit.org
  • 3 edits
    3 adds in trunk

2009-12-07 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25524
[Gtk] Expose the title attribute to assistive technologies

  • platform/gtk/accessibility/title-and-alt.html: Added.
  • platform/gtk/accessibility/title-and-alt-expected.txt: Added.

2009-12-07 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25524
[Gtk] Expose the title attribute to assistive technologies

Expose 'alt' attribute from images as accessible name.
Expose the 'title' core HTML attribute as accessible description.
This is a modified version of the original fix submitted by Mario Sanchez Prada,
adjusted so that it doesn't impact other platforms.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_get_name): (webkit_accessible_get_description):
06:13 Changeset [51761] by eric@webkit.org
  • 3 edits in trunk/WebKit/qt

2009-12-07 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Improve the autotests of QtWebkit
https://bugs.webkit.org/show_bug.cgi?id=32216

Refactor the test of QGraphicsWebView:
-make waitForSignal() available to all the tests.
-remove QTest::qWait()

  • tests/qgraphicswebview/tst_qgraphicswebview.cpp: (tst_QGraphicsWebView::crashOnViewlessWebPages):
  • tests/util.h: (waitForSignal):
05:50 Changeset [51760] by oliver@apple.com
  • 5 edits in trunk

Object.create fails if properties on the descriptor are getters
https://bugs.webkit.org/show_bug.cgi?id=32219

Reviewed by Maciej Stachowiak.

Correctly initialise the PropertySlots with the descriptor object.

05:44 Changeset [51759] by girish@forwardbias.in
  • 2 edits in trunk/WebKit/qt

[Qt] Plugins: Force windowless mode when there is no native window handle

Reviewed by Simon Hausmann.

Inject wmode=opaque while instantiating the plugin for the case when the
webpage is not backed by a native window handle.

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

05:29 Changeset [51758] by vestbo@webkit.org
  • 4 edits in trunk/WebCore

[Qt] Fix support for generic font families

Reviewed by Simon Hausmann.

By not propagating the resolved font family in FontCacheQt and
only concidering the font description we ended up passing the
generic font families to Qt directly. Since most systems don't
have a mapping for these font families we ended up using the
default font in most cases.

  • platform/graphics/qt/FontCacheQt.cpp:
  • platform/graphics/qt/FontPlatformData.h:
  • platform/graphics/qt/FontPlatformDataQt.cpp:
05:06 Changeset [51757] by oliver@apple.com
  • 9 edits
    1 add in trunk

Object.getOwnPropertyDescriptor() allows cross-frame access
https://bugs.webkit.org/show_bug.cgi?id=32119

Reviewed by Maciej Stachowiak.

Make all implementations of getOwnPropertyDescriptor that have
cross domain restrictions simply fail immediately on cross domain
access, rather than trying to mimic the behaviour of normal
property access.

Test: http/tests/security/cross-frame-access-getOwnPropertyDescriptor.html

04:36 Changeset [51756] by steveblock@google.com
  • 9 edits
    3 adds in trunk/WebCore

Moves JSC-specific functions from jni_utility and moves them to new jsc/jni_utility_private files.
https://bugs.webkit.org/show_bug.cgi?id=32157

Reviewed by Adam Barth.

Build fix only, no new tests.

  • WebCore.xcodeproj/project.pbxproj: Modified. Add jni_utility_private.[cpp|h].
  • bridge/jni/jni_instance.cpp: Modified. Include jni_utility_private.h for JSC.
  • bridge/jni/jni_jsobject.mm: Modified. Include jni_utility_private.h for JSC.
  • bridge/jni/jni_objc.mm: Modified. Include jni_utility_private.h for JSC.
  • bridge/jni/jni_runtime.cpp: Modified. Include jni_utility_private.h for JSC.
  • bridge/jni/jni_utility.cpp: Modified. Removed convertValueToJValue and convertArrayInstanceToJavaArray.
  • bridge/jni/jni_utility.h: Modified. Removed convertValueToJValue and dispatchJNICall.
  • bridge/jni/jsc: Added.
  • bridge/jni/jsc/jni_utility_private.cpp: Added.

(JSC::Bindings::convertArrayInstanceToJavaArray):
(JSC::Bindings::convertValueToJValue):

  • bridge/jni/jsc/jni_utility_private.h: Added. Header for convertValueToJValue and dispatchJNICall.
  • platform/android/TemporaryLinkStubs.cpp: Modified. Include jni_utility_private.h for JSC.
04:04 Changeset [51755] by krit@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-07 Dirk Schulze <krit@webkit.org>

Not reviewed, adding myself to the reviewers list.

  • Scripts/modules/committers.py:
00:51 QtWebKitMediaElementSupport edited by vestbo@webkit.org
(diff)
00:50 QtWebKitMediaElementSupport edited by vestbo@webkit.org
(diff)
00:46 QtWebKitMediaElementSupport created by vestbo@webkit.org
00:46 QtWebKit edited by vestbo@webkit.org
(diff)

12/06/09:

23:53 Changeset [51754] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-06 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Add an API for uploading results files to StatusBot
https://bugs.webkit.org/show_bug.cgi?id=32210

Add Content-Type: plain/text which was forgotten
from the previous commit when I landed with land-patches
instead of land-diff.

  • QueueStatusServer/queue_status.py:
23:45 Changeset [51753] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-06 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Add an API for uploading results files to StatusBot
https://bugs.webkit.org/show_bug.cgi?id=32210

  • Scripts/modules/statusbot.py:
23:42 Changeset [51752] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-12-06 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Status Server needs a way to handle uploaded results
https://bugs.webkit.org/show_bug.cgi?id=32209

  • QueueStatusServer/queue_status.py: Add a ShowResults (results/*) command
  • QueueStatusServer/update_status.html: Add file upload.
23:06 Changeset [51751] by hamaji@chromium.org
  • 2 edits in trunk/BugsSite

2009-12-06 Shinichiro Hamaji <hamaji@chromium.org>

Unreviewed. Revert r51748.

Bugzilla should show images in git patches
https://bugs.webkit.org/show_bug.cgi?id=31395

  • PrettyPatch/PrettyPatch.rb:
22:42 Changeset [51750] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-06 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

[bzt] BuildAttachment should not check the builders
https://bugs.webkit.org/show_bug.cgi?id=32207

This is code that got copied here by accident when the class was created.

  • Scripts/modules/commands/download.py:
22:42 Changeset [51749] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-06 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

[bzt] Make LandingSequence.update aware of ports
https://bugs.webkit.org/show_bug.cgi?id=32208

This is required to make build work on the Chromium port because
Chromium has a custom update-webkit.

  • Scripts/modules/landingsequence.py:
22:42 Changeset [51748] by hamaji@chromium.org
  • 2 edits in trunk/BugsSite

2009-12-06 Shinichiro Hamaji <shinichiro.hamaji@gmail.com>

Reviewed by David Kilzer.

Bugzilla should show images in git patches
https://bugs.webkit.org/show_bug.cgi?id=31395

Show images in git patches using git-apply.

  • PrettyPatch/PrettyPatch.rb:
21:20 Changeset [51747] by abarth@webkit.org
  • 4 edits in trunk/WebKitTools

2009-12-06 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Add early warning system commands to bugzilla-tool.

  • Scripts/bugzilla-tool:
  • Scripts/modules/landingsequence.py:
  • Scripts/modules/webkitport.py:
21:11 Changeset [51746] by abarth@webkit.org
  • 2 edits
    1 add in trunk/WebKitTools

2009-12-06 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

[bzt] Implement Qt EarlyWarningSystem and Chromium EarlyWarningSystem
https://bugs.webkit.org/show_bug.cgi?id=32205

  • Scripts/modules/commands/early_warning_system.py: Added.
  • Scripts/modules/commands/queues.py:
19:27 Changeset [51745] by mitz@apple.com
  • 2 edits in trunk/WebKitTools

REGRESSION (r51728): update-webkit fails when the current directory is
not inside a Subversion working copy
https://bugs.webkit.org/show_bug.cgi?id=32204

Reviewed by Adele Peterson.

  • Scripts/update-webkit: Invoke isSVN() in the correct working

directory.

18:45 Changeset [51744] by eric@webkit.org
  • 5 edits in trunk/LayoutTests

2009-12-06 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Improve portability and readability of table-percent-height* tests.
https://bugs.webkit.org/show_bug.cgi?id=31858

The purpose of these tests is to confirm that an element with
height:N% inside a table cell has non-zero height. We don't need
to check concrete pixel sizes of form controls.

This updates the tests so that they check heights are non-zero and
75% heights are same as corresponding 100% heights or actually 75%.

  • fast/replaced/table-percent-height-expected.txt:
  • fast/replaced/table-percent-height-text-controls-expected.txt:
  • fast/replaced/table-percent-height-text-controls.html:
  • fast/replaced/table-percent-height.html:
18:19 Changeset [51743] by ukai@chromium.org
  • 6 edits
    8 adds in trunk/LayoutTests

2009-12-06 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

Add tests for cross-origin WebSocket usage
https://bugs.webkit.org/show_bug.cgi?id=32055

  • websocket/tests/cross-origin-expected.txt: Added.
  • websocket/tests/cross-origin.html: Added.
  • websocket/tests/fixed-origin_wsh.py: Added.
  • websocket/tests/handshake-fail-by-cross-origin-expected.txt: Added.
  • websocket/tests/handshake-fail-by-cross-origin.html: Added.
  • websocket/tests/origin-test_wsh.py: Added.
  • websocket/tests/script-tests/cross-origin.js: Added.
  • websocket/tests/script-tests/frame-length-longer-than-buffer.js:
  • websocket/tests/script-tests/frame-length-skip.js:
  • websocket/tests/script-tests/handshake-error.js:
  • websocket/tests/script-tests/handshake-fail-by-cross-origin.js: Added.
  • websocket/tests/script-tests/simple.js:
  • websocket/tests/simple-expected.txt:
17:09 Changeset [51742] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-06 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

[bzt] Rename AbstractTryQueue to AbstractReviewQueue
https://bugs.webkit.org/show_bug.cgi?id=32202

  • Scripts/modules/commands/queues.py:
09:40 Changeset [51741] by eric@webkit.org
  • 3 edits in trunk/WebKit/haiku

2009-12-06 Maxime Simon <simon.maxime@gmail.com>

Reviewed by Adam Barth.

Build fix. Add in FrameLoaderClientHaiku the declaration of three functions introduced in r51644.

  • WebCoreSupport/FrameLoaderClientHaiku.cpp: (WebCore::FrameLoaderClientHaiku::dispatchDidPushStateWithinPage): (WebCore::FrameLoaderClientHaiku::dispatchDidReplaceStateWithinPage): (WebCore::FrameLoaderClientHaiku::dispatchDidPopStateWithinPage):
  • WebCoreSupport/FrameLoaderClientHaiku.h:
03:37 Changeset [51740] by oliver@apple.com
  • 2 edits in trunk/WebCore

texImage2D pixel junk for transparency
https://bugs.webkit.org/show_bug.cgi?id=32188

Reviewed by Maciej Stachowiak

Use kCGBlendModeCopy when drawing an image to the intermediate context
used to create a GL texture.

No test as we don't currently have any mechanism to retrieve pixel data
from the webgl context.

02:30 Changeset [51739] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

Not reviewed, build fix.

Actually tested 64-bit *and* 32-bit build this time.

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_loop_if_false):

02:09 Changeset [51738] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

Not reviewed, build fix.

Really really fix 64-bit build for prior patch (actually tested this time).

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_loop_if_false):
(JSC::JIT::emitSlow_op_loop_if_false):

01:57 Changeset [51737] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

Not reviewed, build fix.

Really fix 64-bit build for prior patch.

  • jit/JITArithmetic.cpp:

(JSC::JIT::emitSlow_op_jless):

01:51 Changeset [51736] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

Not reviewed, build fix.

Fix 64-bit build for prior patch.

  • jit/JITOpcodes.cpp:

(JSC::JIT::emitSlow_op_loop_if_less):

01:42 Changeset [51735] by mjs@apple.com
  • 17 edits in trunk

2009-12-05 Maciej Stachowiak <mjs@apple.com>

Reviewed by Oliver Hunt.

conway benchmark spends half it's time in op_less (jump fusion fails)
https://bugs.webkit.org/show_bug.cgi?id=32190

<1% speedup on SunSpider and V8
2x speedup on "conway" benchmark


Two optimizations:

1) Improve codegen for logical operators &&,
and ! in a condition context


When generating code for combinations of &&,
and !, in a

condition context (i.e. in an if statement or loop condition), we
used to produce a value, and then separately jump based on its
truthiness. Now we pass the false and true targets in, and let the
logical operators generate jumps directly. This helps in four
ways:

a) Individual clauses of a short-circuit logical operator can now
jump directly to the then or else clause of an if statement (or to
the top or exit of a loop) instead of jumping to a jump.


b) It used to be that jump fusion with the condition of the first
clause of a logical operator was inhibited, because the register
was ref'd to be used later, in the actual condition jump; this no
longer happens since a jump straight to the final target is
generated directly.

c) It used to be that jump fusion with the condition of the second
clause of a logical operator was inhibited, because there was a
jump target right after the second clause and before the actual
condition jump. But now it's no longer necessary for the first
clause to jump there so jump fusion is not blocked.

d) We avoid generating excess mov statements in some cases.


As a concrete example this source:


if (!((x < q && y < q)
(t < q && z < q))) {

...

}


Used to generate this bytecode:


[ 34] less r1, r-15, r-19
[ 38] jfalse r1, 7(->45)
[ 41] less r1, r-16, r-19
[ 45] jtrue r1, 14(->59)
[ 48] less r1, r-17, r-19
[ 52] jfalse r1, 7(->59)
[ 55] less r1, r-18, r-19
[ 59] jtrue r1, 17(->76)


And now generates this bytecode (also taking advantage of the second optimization below):


[ 34] jnless r-15, r-19, 8(->42)
[ 38] jless r-16, r-19, 26(->64)
[ 42] jnless r-17, r-19, 8(->50)
[ 46] jless r-18, r-19, 18(->64)


Note the jump fusion and the fact that there's less jump
indirection - three of the four jumps go straight to the target
clause instead of indirecting through another jump.


2) Implement jless opcode to take advantage of the above, since we'll now often generate
a less followed by a jtrue where fusion is not forbidden.


  • parser/Nodes.h: (JSC::ExpressionNode::hasConditionContextCodegen): Helper function to determine whether a node supports special conditional codegen. Return false as this is the default. (JSC::ExpressionNode::emitBytecodeInConditionContext): Assert not reached - only really defined for nodes that do have conditional codegen. (JSC::UnaryOpNode::expr): Add const version. (JSC::LogicalNotNode::hasConditionContextCodegen): Returne true only if subexpression supports it. (JSC::LogicalOpNode::hasConditionContextCodegen): Return true.
  • parser/Nodes.cpp: (JSC::LogicalNotNode::emitBytecodeInConditionContext): Implemented - just swap the true and false targets for the child node. (JSC::LogicalOpNode::emitBytecodeInConditionContext): Implemented - handle jumps directly, improving codegen quality. Also handles further nested conditional codegen. (JSC::ConditionalNode::emitBytecode): Use condition context codegen when available. (JSC::IfNode::emitBytecode): ditto (JSC::IfElseNode::emitBytecode): ditto (JSC::DoWhileNode::emitBytecode): ditto (JSC::WhileNode::emitBytecode): ditto (JSC::ForNode::emitBytecode): ditto
  • bytecode/Opcode.h:
  • Added loop_if_false opcode - needed now that falsey jumps can be backwards.
  • Added jless opcode to take advantage of new fusion opportunities.
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::dump): Handle above.
  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitJumpIfTrue): Add peephole for less + jtrue ==> jless. (JSC::BytecodeGenerator::emitJumpIfFalse): Add handling of backwrds falsey jumps.
  • bytecompiler/BytecodeGenerator.h: (JSC::BytecodeGenerator::emitNodeInConditionContext): Wrapper to handle tracking of overly deep expressions etc.
  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute): Implement the two new opcodes (loop_if_false, jless).
  • jit/JIT.cpp: (JSC::JIT::privateCompileMainPass): Implement JIT support for the two new opcodes. (JSC::JIT::privateCompileSlowCases): ditto
  • jit/JIT.h:
  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_jless): (JSC::JIT::emitSlow_op_jless): ditto (JSC::JIT::emitBinaryDoubleOp): ditto
  • jit/JITOpcodes.cpp: (JSC::JIT::emitSlow_op_loop_if_less): ditto (JSC::JIT::emit_op_loop_if_false): ditto (JSC::JIT::emitSlow_op_loop_if_false): ditto
  • jit/JITStubs.cpp:
  • jit/JITStubs.h: (JSC::):

2009-12-05 Maciej Stachowiak <mjs@apple.com>

Reviewed by Oliver Hunt.

conway benchmark spends half it's time in op_less (jump fusion fails)
https://bugs.webkit.org/show_bug.cgi?id=32190

  • fast/js/codegen-loops-logical-nodes-expected.txt:
  • fast/js/script-tests/codegen-loops-logical-nodes.js: Update to test some newly sensitive cases of codegen that were not already covered.

12/05/09:

17:50 Changeset [51734] by kov@webkit.org
  • 11 edits in trunk

WebCore

2009-12-03 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha.

Advertize audio/ogg correctly and refactored mime-type cache
building.

[GTK] Recognize oga as audio/ogg
https://bugs.webkit.org/show_bug.cgi?id=31990

Test: media/media-can-play-ogg.html

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::mimeTypeCache):

LayoutTests

2009-12-05 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha.

New test for video.canPlayType and ogg containers.

[GTK] Recognize oga as audio/ogg
https://bugs.webkit.org/show_bug.cgi?id=31990

  • media/media-can-play-ogg-expected.txt: Added.
  • media/media-can-play-ogg.html: Added.
  • platform/mac-leopard/Skipped:
  • platform/mac-snowleopard/Skipped:
  • platform/mac-tiger/Skipped:
  • platform/mac/Skipped:
  • platform/qt-mac/Skipped:
  • platform/qt-win/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
16:30 Changeset [51733] by eric@webkit.org
  • 7 edits
    1 add in trunk/WebKitSite

2009-12-05 Chris Jerdonek <chris.jerdonek@gmail.com>

Reviewed by Darin Adler.

Refactored web site documentation so that advice to add
WebKitTools/Scripts to your shell path does not appear
on multiple pages.

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

  • building/build.html:
  • building/checkout.html:
  • building/run.html:
  • coding/coding-style.html:
  • coding/contributing.html:
  • coding/scripts.html: Added.
  • quality/testing.html:
15:59 Changeset [51732] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=32175
REGRESSION: websocket/tests/url-parsing.html is crashing on Leopard Bot

  • platform/network/cf/SocketStreamHandleCFNet.cpp: (WebCore::SocketStreamHandle::readStreamCallback): Call platformClose() to unschedule both streams, guaranteeing that there will be no callbacks after SocketStreamHandle is destroyed. (WebCore::SocketStreamHandle::writeStreamCallback): Edited a comment a bit.
15:44 Changeset [51731] by mitz@apple.com
  • 1 edit
    1 move in trunk/LayoutTests

Rubber-stamped by Mark Rowe.

Disabled a test because it puts the WebView into compositing mode,
which remains in effect for all subsequent repaint tests, causing them
to fail because DumpRenderTree always forces a full repaint when
generating the image of a WebView that is in compositing mode.

  • fast/repaint/block-selection-gap-in-composited-layer.html: Removed.
  • fast/repaint/block-selection-gap-in-composited-layer.html.disabled: Copied from LayoutTests/fast/repaint/block-selection-gap-in-composited-layer.html.
13:55 Changeset [51730] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

2009-12-05 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

[bzt] Implement ChromiumPort
https://bugs.webkit.org/show_bug.cgi?id=32182

  • Scripts/modules/webkitport.py:
  • Scripts/modules/webkitport_unittest.py:
13:54 Changeset [51729] by abarth@webkit.org
  • 6 edits in trunk/WebKitTools

2009-12-05 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Move update-webkit into BuildSteps
https://bugs.webkit.org/show_bug.cgi?id=32181

We need to move update-webkit out of SCM.py because SCM isn't supposed to know
that WebKit exists. The proper place for the knowledge of the existence of
update-webkit is in WebKitPort because some ports have specialized update
scripts (analogous to build-webkit).

  • Scripts/modules/buildsteps.py:
  • Scripts/modules/commands/download.py:
  • Scripts/modules/landingsequence.py:
  • Scripts/modules/scm.py:
  • Scripts/modules/webkitport.py:
13:54 Changeset [51728] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-05 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

update-webkit should call git svn rebase and resolve-ChangeLogs -f
https://bugs.webkit.org/show_bug.cgi?id=27162

Teach update-webkit about Git. I didn't add the call to
resolve-ChangeLogs -f because sometimes that script goes bananas. We
can iterate from here, however.

  • Scripts/update-webkit:
09:50 Changeset [51727] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2009-12-05 Adam Langley <agl@google.com>

Reviewed by Adam Barth.

Check that a CSS format() argument is of a valid type.

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

  • fast/css/url-format-non-string-expected.txt: Added
  • fast/css/url-format-non-string.html: Added.

2009-12-05 Adam Langley <agl@google.com>

Reviewed by Adam Barth.

Check that a CSS format() argument is of a valid type.

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

Test: fast/css/url-format-non-string.html

  • css/CSSParser.cpp: (WebCore::CSSParser::parseFontFaceSrc):
08:56 Changeset [51726] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

Reviewed by Xan Lopez.

Use a better URI for the unreachable test. localhost:80 is very
likely to be open.

  • tests/testwebdatasource.c:

(load_finished_unreachable_cb):
(test_webkit_web_data_source_unreachable_uri):

07:35 Changeset [51725] by kov@webkit.org
  • 2 edits in trunk

2009-12-05 Vincent Untz <vuntz@gnome.org>

Reviewed by Gustavo Noronha.

Fixes race for builds with introspection enabled, and parallel
make.

  • GNUmakefile.am:

12/04/09:

17:21 Changeset [51724] by eric@webkit.org
  • 3 edits
    3 adds in trunk

2009-12-04 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Darin Adler.

JavaScript delete operator should return false for string properties
https://bugs.webkit.org/show_bug.cgi?id=32012

  • runtime/StringObject.cpp: (JSC::StringObject::deleteProperty):

2009-12-04 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Darin Adler.

Add test for JavaScript string property deletion
https://bugs.webkit.org/show_bug.cgi?id=32012

  • fast/js/script-tests/string-property-deletion.js: Added.
  • fast/js/string-property-deletion-expected.txt: Added.
  • fast/js/string-property-deletion.html: Added.
16:46 Changeset [51723] by eric@webkit.org
  • 3 edits in trunk/WebKitSite

2009-12-04 Marc-Antoine Ruel <maruel@chromium.org>

Reviewed by Eric Seidel.

Add reference to the GTK and QT build wiki pages

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

  • building/tools.html:
  • css/main.css:
16:39 Changeset [51722] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-12-04 Adam Langley <agl@google.com>

Reviewed by Eric Seidel.

Chromium: make rounded borders heavier.

Skia draws its rounded corners differently from the other ports.
Whereas they have anti-aliased clipping regions, Skia only has 1-bit
regions. (Which is technically more correct, but somewhat unhelpful
for us.) Instead, with Skia we use a layer and collect all the
clipping paths in effect. When the state is popped, we paint
transparency outside the clipping paths and merge the layer down.

This appears to cause rounded borders to look a little thin, which is
addressed in this patch.

This is well covered by existing tests but will require new baselines
in the Chromium tree.

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

  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::addInnerRoundedRectClip):
16:31 Changeset [51721] by eric@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

2009-12-04 Yuzo Fujishima <yuzo@google.com>

Reviewed by Alexey Proskuryakov.

Add test for web socket send.

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

  • websocket/tests/script-tests/send.js: Added. (endTest): (ws.onopen): (ws.onmessage.ws.onmessage): (ws.onmessage): (ws.onclose): (timeOutCallback):
  • websocket/tests/send-expected.txt: Added.
  • websocket/tests/send.html: Added.
  • websocket/tests/send_wsh.py: Added.
16:24 Changeset [51720] by eric@webkit.org
  • 3 edits
    3 adds in trunk/WebKit/chromium

2009-12-04 James Hawkins <jhawkins@chromium.org>

Reviewed by Darin Fisher.

https://bugs.webkit.org/show_bug.cgi?id=32095
Implement WebRegularExpression, a WebKit API for accessing
WebCore::RegularExpression.

  • WebKit.gyp:
  • public/WebRegularExpression.h: Added.
  • public/WebTextCaseSensitivity.h: Added. (WebKit::):
  • src/AssertMatchingEnums.cpp:
  • src/WebRegularExpression.cpp: Added. (WebKit::WebRegularExpression::WebRegularExpression): (WebKit::WebRegularExpression::~WebRegularExpression): (WebKit::WebRegularExpression::match):
15:56 Changeset [51719] by pkasting@chromium.org
  • 4 edits in trunk/WebKit/chromium

[Chromium] Simplify zoom-related APIs and add a zoom level getter,
part two: Remove deprecated APIs.
https://bugs.webkit.org/show_bug.cgi?id=31893

Reviewed by Darin Fisher.

  • public/WebView.h:
  • src/WebViewImpl.cpp:
  • src/WebViewImpl.h:
15:18 Changeset [51718] by simon.fraser@apple.com
  • 3 edits
    2 adds in trunk

2009-12-04 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/7441571> Hardware-backed layers are skipped when printing

When printing, set the PaintBehaviorFlattenCompositingLayers flag so that compositing layers
paint. This also causes 3D transforms to be flattened to 2D.

Test: printing/compositing-layer-printing.html

  • page/FrameView.cpp: (WebCore::FrameView::paintContents):
15:13 Changeset [51717] by oliver@apple.com
  • 2 edits in trunk/WebCore

Fix up some issues in my SVGListTraits change from yesterday.

Reviewed by Deth Bakin

14:10 Changeset [51716] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

Skip test that is failing because of new behaviour enforced by
libsoup. Bug reported.

  • platform/gtk/Skipped:
14:00 Changeset [51715] by kenneth@webkit.org
  • 5 edits in trunk/WebKitTools

Add some missing methods for showing and hiding the
Web Inspector.

Reviewed by Adam Treat.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::WebPage::WebPage):
(WebCore::WebPage::~WebPage):
(WebCore::WebPage::webInspector):

  • DumpRenderTree/qt/DumpRenderTree.h:
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp:

(LayoutTestController::showWebInspector):
(LayoutTestController::hideWebInspector):

  • DumpRenderTree/qt/LayoutTestControllerQt.h:
13:54 Changeset [51714] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

Skip tests that require support for frame loader callbacks
reporting, and a new test.

  • platform/gtk/Skipped:
13:45 BuildBot edited by maruel@chromium.org
(diff)
13:35 Changeset [51713] by yael.aharon@nokia.com
  • 2 edits in trunk/WebKitTools

Unreviewed build fix.

[Qt] build fix after r51634 removed unused QBoxLayout include which included limits.h.

  • DumpRenderTree/qt/DumpRenderTree.cpp:
13:19 Changeset [51712] by krit@webkit.org
  • 1 edit
    1 move in trunk/LayoutTests

2009-12-04 Dirk Schulze <krit@webkit.org>

Reviewed by Gustavo Noronha Silva.

Move the render result to svg/custom as a cross platform render result.

  • platform/mac/svg/custom/radial-gradient-with-outstanding-focalPoint-expected.txt: Removed.
  • svg/custom/radial-gradient-with-outstanding-focalPoint-expected.txt: Added.
13:13 Changeset [51711] by ajwong@chromium.org
  • 2 edits in trunk/WebCore

Not reviewed: chromium build fix.

Fix Chromium build break caused by moving of isDefaultPortForProtocol() and
portAllowed() into KURL.h/KURL.cpp. Cloning code.

  • platform/KURLGoogle.cpp:

(WebCore::isDefaultPortForProtocol):
(WebCore::portAllowed):

13:03 Changeset [51710] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-12-04 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Remove suppress highlight when restoring selected node.

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

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.setDocument.selectNode): (WebInspector.ElementsPanel.prototype.setDocument.selectLastSelectedNode): (WebInspector.ElementsPanel.prototype.setDocument):
13:01 Changeset [51709] by kenneth@webkit.org
  • 6 edits in trunk

WebKitTools: Implement the setAllowUniversalAccessFromFileURLs method
for the Qt LayoutTestController.

Reviewed by Adam Treat.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::WebPage::resetSettings):

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp:

(LayoutTestController::setAllowUniversalAccessFromFileURLs):

  • DumpRenderTree/qt/LayoutTestControllerQt.h:

LayoutTests: Remove test that works after adding setAllowUniversalAccessFromFileURLs
to Qt's version of the LayoutTestController.

Reviewed by Adam Treat.

  • platform/qt/Skipped:
12:50 BuildBot edited by maruel@chromium.org
(diff)
12:24 BuildBot edited by maruel@chromium.org
(diff)
12:19 Changeset [51708] by krit@webkit.org
  • 5 edits
    4 adds in trunk

2009-12-04 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Gradient SVG animation demonstrates tearing at animation extremes
https://bugs.webkit.org/show_bug.cgi?id=11929

The focalPoint of a radial gradient is temporarly substracted by
the centralPoint, if the focalPoint is not in the radius of the
gradient. This is needed to calculate a new postion of the focalPoint
according to the specification. But the new focalPoint needs to be
moved by the centralPoint after this calculation, which is not the case
at the moment. This patch fixes this issue. It was also introduced a
deviation of maximal 0.2% for Cairo to get around the fixed point numbers
in Cairo.

W3C-SVG-1.1/pservers-grad-13-b needed an update. The missing adjustment
of the new focalPoint caused wrong results for the last three tests.

Test: svg/custom/radial-gradient-with-outstanding-focalPoint.svg

  • svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::buildGradient):

Gradient SVG animation demonstrates tearing at animation extremes
https://bugs.webkit.org/show_bug.cgi?id=11929

In SVG a focalPoint must be inside the radius of a radial gradient.
It this isn't the case, we have to move the focalPoint into the radius.
This checks the correct behavior of WebKit on false values for fx, fy.

  • platform/mac/svg/W3C-SVG-1.1/pservers-grad-13-b-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/pservers-grad-13-b-expected.png:
  • platform/mac/svg/custom/radial-gradient-with-outstanding-focalPoint-expected.checksum: Added.
  • platform/mac/svg/custom/radial-gradient-with-outstanding-focalPoint-expected.png: Added.
  • platform/mac/svg/custom/radial-gradient-with-outstanding-focalPoint-expected.txt: Added.
  • svg/custom/radial-gradient-with-outstanding-focalPoint.svg: Added.
12:16 Changeset [51707] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-12-04 Anton Muhin <antonm@chromium.org>

Reviewed by Dmitry Titov.

Do not use WebCore::String::String(const UChar*, int length) to convert
short v8 strings.

Plus added string traits.
https://bugs.webkit.org/show_bug.cgi?id=31415

  • bindings/v8/V8Binding.cpp: (WebCore::): (WebCore::v8StringToWebCoreString):
12:04 Changeset [51706] by cfleizach@apple.com
  • 12 edits
    2 adds in trunk

REGRESSION: AX: buttons now extremely repetitive
https://bugs.webkit.org/show_bug.cgi?id=32164

Reviewed by Darin Adler.

WebCore:

Test: platform/mac/accessibility/button-shouldnot-have-axvalue.html

  • accessibility/AccessibilityObject.h:

(WebCore::AccessibilityObject::isButton):

  • accessibility/mac/AccessibilityObjectWrapper.mm:

(-[AccessibilityObjectWrapper accessibilityAttributeNames]):

WebKitTools:

  • DumpRenderTree/AccessibilityUIElement.cpp:

(isAttributeSupportedCallback):
(AccessibilityUIElement::getJSClass):

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

(AccessibilityUIElement::isAttributeSupported):

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::isAttributeSupported):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::isAttributeSupported):

LayoutTests:

  • platform/mac-leopard/accessibility/table-with-aria-role-expected.txt:
  • platform/mac/accessibility/button-shouldnot-have-axvalue-expected.txt: Added.
  • platform/mac/accessibility/button-shouldnot-have-axvalue.html: Added.
  • platform/mac/accessibility/table-with-aria-role-expected.txt:
12:01 Changeset [51705] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-12-04 Zoltan Herczeg <zherczeg@inf.u-szeged.hu>

Reviewed by Darin Adler.

Check whether the right side of a string append is non-empty
https://bugs.webkit.org/show_bug.cgi?id=32151

Especially beneficial for CachedScript::script() method
which usually appends an empty string returned by flush()
to its m_script member. m_script is often hundreds of kbytes
which is duplated without reason.

  • platform/text/String.cpp: (WebCore::String::append):
11:51 BuildBot edited by maruel@chromium.org
(diff)
11:49 Changeset [51704] by enrica@apple.com
  • 3 edits
    2 adds in trunk

Mail.app crashes when indenting table pasted from Numbers.app.
<rdar://problem/7209507>
https://bugs.webkit.org/show_bug.cgi?id=32166

Reviewed by John Sullivan.

WebCore:

Test: LayoutTests/editing/execCommand/indent-partial-table.html

  • editing/IndentOutdentCommand.cpp:

(WebCore::IndentOutdentCommand::indentRegion): Added test to verify that the end of the paragraph
after the selection has not been moved.

LayoutTests:

  • editing/execCommand/indent-partial-table-expected.txt: Added.
  • editing/execCommand/indent-partial-table.html: Added.
11:16 Changeset [51703] by ap@apple.com
  • 9 edits
    3 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=32085
WebSocket should block the same ports that are blocked for resource loading

Test: websocket/tests/url-parsing.html

  • page/SecurityOrigin.cpp: (WebCore::SecurityOrigin::SecurityOrigin): (WebCore::SecurityOrigin::localURLSchemes):
  • page/SecurityOrigin.h: Move isDefaultPortForProtocol() to KURL, because that's a better place for it (SecurityOrigin is not even in WebCore/platform directory).
  • html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::host): (WebCore::HTMLAnchorElement::setHost): (WebCore::HTMLAnchorElement::setPort): Updated for the new location of isDefaultPortForProtocol().
  • platform/KURL.cpp: (WebCore::KURL::protocolIs): In an assertion, compare to "javascript" case-insensitively, since the function deosn't require lower case input. (WebCore::isDefaultPortForProtocol): Moved from SecurityOrigin. (WebCore::portAllowed): Moved from ResourceHandle.
  • platform/KURL.h:
  • platform/network/ResourceHandle.cpp: (WebCore::ResourceHandle::create): Updated for the new location of portAllowed().
  • websockets/WebSocket.cpp: (WebCore::WebSocket::connect): Per the spec, raise a SECURITY_ERR if trying to connect to a blocked port.
10:12 Changeset [51702] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-12-04 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Corrects QtLauncher style

  • QtLauncher/main.cpp: (WebPage::acceptNavigationRequest): (MainWindow::MainWindow): (MainWindow::webPage): (MainWindow::webView): (MainWindow::changeLocation): (MainWindow::loadFinished): (MainWindow::showLinkHover): (MainWindow::zoomIn): (MainWindow::zoomOut): (MainWindow::print): (MainWindow::setEditable): (MainWindow::dumpHtml): (MainWindow::selectElements): (MainWindow::newWindow): (MainWindow::setupUI): (WebPage::createWindow): (WebPage::createPlugin): (main):
10:04 Changeset [51701] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-12-04 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QtLauncher: add a menu to show or hide the web inspector.
https://bugs.webkit.org/show_bug.cgi?id=32149

  • QtLauncher/main.cpp: (WebInspector::WebInspector): (WebInspector::showEvent): (WebInspector::hideEvent): (MainWindow::MainWindow): (MainWindow::setupUI):
09:57 Changeset [51700] by eric@webkit.org
  • 6 edits in trunk/LayoutTests

2009-12-04 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] WebKit crashes when loading certain SVG images
https://bugs.webkit.org/show_bug.cgi?id=29443

Update the test results for the new FontCache implementation.

  • platform/qt/css1/box_properties/float_elements_in_series-expected.txt:
  • platform/qt/css1/font_properties/font_variant-expected.txt:
  • platform/qt/css1/pseudo/firstline-expected.txt:
  • platform/qt/css1/pseudo/multiple_pseudo_elements-expected.txt:
  • platform/qt/css2.1/t1505-c524-font-var-00-b-expected.txt:
09:50 Changeset [51699] by eric@webkit.org
  • 6 edits
    1 delete in trunk/WebCore

2009-12-04 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] WebKit crashes when loading certain SVG images
https://bugs.webkit.org/show_bug.cgi?id=29443

Remove FontFallbackListQt and rely on the common FontFallbackList
to handle the fonts. FontCache and FontPlatformData have been
updated to work with the common FontFallbackList.

In the previous implementation, FontPlatformDataCacheKey
was a clone of FontPlatformData with the hashing
capabilities added in order to use it as a key in the cache's
hashmap. FontPlatformData has been modified to handle the hashing
function directly so the data are not copied twice in memory.

FontFallbackList::fontDataAt() from FontFallbackListQt was a copy of
code from FontCache::getFontData() and FontFallbackList::fontDataAt().
The behavior is similar except currFamily->family().length() was
not tested and the fallback fonts selector were not used.

Existing tests cover the change.

  • WebCore.pro:
  • platform/graphics/qt/FontCacheQt.cpp: (WebCore::FontCache::platformInit): (WebCore::FontCache::getFontDataForCharacters): (WebCore::FontCache::getSimilarFontPlatformData): (WebCore::FontCache::getLastResortFallbackFont): (WebCore::FontCache::getTraitsInFamily): (WebCore::FontCache::createFontPlatformData):
  • platform/graphics/qt/FontCustomPlatformData.cpp: (WebCore::FontCustomPlatformData::fontPlatformData):
  • platform/graphics/qt/FontFallbackListQt.cpp: Removed. We now use the implementation from FontFallbackList.cpp
  • platform/graphics/qt/FontPlatformData.h: Add hashing capabilities to be able to use the data with the FontCache. This was previously done in FontCacheQt.cpp (WebCore::FontPlatformDataPrivate::FontPlatformDataPrivate): (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::isHashTableDeletedValue): (WebCore::FontPlatformData::font): (WebCore::FontPlatformData::size): (WebCore::FontPlatformData::family): (WebCore::FontPlatformData::bold): (WebCore::FontPlatformData::italic): (WebCore::FontPlatformData::smallCaps): (WebCore::FontPlatformData::pixelSize):
  • platform/graphics/qt/FontPlatformDataQt.cpp: (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::~FontPlatformData): (WebCore::FontPlatformData::operator=): (WebCore::FontPlatformData::operator==): (WebCore::FontPlatformData::hash):
09:43 Changeset [51698] by eric@webkit.org
  • 3 edits in trunk/WebKit/qt

2009-12-04 Kenneth Rohde Christiansen <kenneth@webkit.org>

Reviewed by Antti Koivisto.

Split out the renderPrivate in two methods, one for working on
relative coordinates (relative to the viewport) and one for
working on absolute coordinates. The latter is more effecient
for implementing tiling, as you don't need translate the coords,
and because it avoid clipping to the viewport.

No behaviour changes, so no new tests.

  • Api/qwebframe.cpp: (QWebFramePrivate::renderContentsLayerAbsoluteCoords): (QWebFramePrivate::renderRelativeCoords): (QWebFrame::render):
  • Api/qwebframe_p.h:
09:14 Changeset [51697] by treat@webkit.org
  • 2 edits in trunk/WebCore

Fix wrong assert that was only working through sheer luck.
https://bugs.webkit.org/show_bug.cgi?id=32162

Patch by Adam Treat <atreat@rim.com> on 2009-12-04
Reviewed by Dan Bernstein.

  • platform/text/BidiContext.cpp:

(WebCore::BidiContext::create):

07:36 Changeset [51696] by vestbo@webkit.org
  • 4 edits in trunk

[Qt] Allow removing 'qrc' as a local security origin scheme

Reviewed by Simon Hausmann.

WebCore:

  • page/SecurityOrigin.cpp:

WebKit/qt:

  • tests/qwebpage/tst_qwebpage.cpp:
07:36 Changeset [51695] by vestbo@webkit.org
  • 2 edits in trunk/WebKit/qt

[Qt] Clean up argument parsing in the QtLauncher

Reviewed by Simon Hausmann.

  • QtLauncher/main.cpp:
05:43 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
04:56 Changeset [51694] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-12-04 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Prevent the inspector from closing its wrapping widget.
This is not necessary anymore since we now hide the embedded
close button.
https://bugs.webkit.org/show_bug.cgi?id=32149

  • WebCoreSupport/InspectorClientQt.cpp: (WebCore::InspectorClientQt::showWindow): (WebCore::InspectorClientQt::closeWindow):
04:07 Changeset [51693] by zoltan@webkit.org
  • 2 edits in trunk/WebCore

Allow custom memory allocation control for WebCore's CollectionCache
https://bugs.webkit.org/show_bug.cgi?id=32109

Reviewed by Eric Seidel.

Inherits the following class from FastAllocBase because it is
instantiated by 'new':

struct name - instantiated at: WebCore/'location'

struct CollectionCache - dom/Document.cpp:4029

  • html/CollectionCache.h:
03:30 Changeset [51692] by eric@webkit.org
  • 4 edits
    6 adds in trunk

2009-12-04 Steve Block <steveblock@google.com>

Reviewed by Darin Adler.

Fixes a crashing bug in Geolocation when a watch is cleared from some callbacks.
https://bugs.webkit.org/show_bug.cgi?id=32111

  • fast/dom/Geolocation/resources/permission-denied-already-clear-watch.js: Added. Tests clearing the watch from the error callback after a permission denied message when permissions have already been denied.
  • fast/dom/Geolocation/permission-denied-already-clear-watch.html: Added. HTML wrapper for above test.
  • fast/dom/Geolocation/permission-denied-already-clear-watch-expected.txt: Added. Expected result for above test.
  • fast/dom/Geolocation/resources/timeout-clear-watch.js: Added. Tests clearing the watch from the error callback due to a timeout.
  • fast/dom/Geolocation/timeout-clear-watch.html: Added. HTML wrapper for above test.
  • fast/dom/Geolocation/timeout-clear-watch-expected.txt: Added. Expected result for above test.
  • platform/gtk/Skipped: Modified. Skips new tests.

2009-12-04 Steve Block <steveblock@google.com>

Reviewed by Darin Adler.

Fixes a crashing bug in Geolocation when a watch is cleared from some callbacks.
https://bugs.webkit.org/show_bug.cgi?id=32111

In Geolocation::GeoNotifier::timerFired, a JS callback may be invoked if a
fatal error was registered on this GeoNotifier or if this request has timed
out. If the request is a watch, and is cleared by a call to clearWatch in the
callback, the GeoNotifier object will be destroyed. We must therefore cache the
m_geolocation member to allow us to call Geolocation::fatalErrorOccurred or
Geolocation::requestTimedOut.

Tests: fast/dom/Geolocation/permission-denied-already-clear-watch.html

fast/dom/Geolocation/timeout-clear-watch.html

  • page/Geolocation.cpp: Modified. (WebCore::Geolocation::GeoNotifier::timerFired): Modified. Cache the m_geolocation member before invoking the JS callback.
02:37 Changeset [51691] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Re-enable audio/video tests after updated buildbot's Qt
to Phonon supported one, and put 2 bad tests into skiplist.

Reviewed by Simon Hausmann.

  • platform/qt/Skipped:
02:33 Changeset [51690] by zoltan@webkit.org
  • 3 edits in trunk/WebCore

Allow custom memory allocation control for 2 classes of the platform directory in WebCore
https://bugs.webkit.org/show_bug.cgi?id=

Reviewed by Eric Seidel.

Inherits the following class from Noncopyable because it is instantiated
by 'new' and no need to be copyable:

class/struct name - instantiated at: WebCore/'location'

class SharedTimer - (its child class) workers/WorkerRunLoop.cpp:91

Inherits the following class from FastAllocBase because it is
instantiated by 'new':

class TransformOperations - rendering/style/RenderStyle.h:1175

Noncopyable.h's include added to SharedTimer.h.

  • platform/SharedTimer.h:
  • platform/graphics/transforms/TransformOperations.h:
02:18 Changeset [51689] by xan@webkit.org
  • 4 edits in trunk

2009-12-04 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

[GTK]Enable DNS prefetching
https://bugs.webkit.org/show_bug.cgi?id=23846

Bump libsoup required version to 2.29.3 for DNS prefetching.

  • configure.ac:

WebCore:

2009-12-04 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

[GTK]Enable DNS prefetching
https://bugs.webkit.org/show_bug.cgi?id=23846

Enable DNS prefetching.

Based on a patch by José Millán.

  • platform/network/soup/DNSSoup.cpp: (WebCore::prefetchDNS):
00:56 Changeset [51688] by zoltan@webkit.org
  • 3 edits in trunk/WebCore

Roll out r51687 because it breaks the Leopard builds.

00:38 Changeset [51687] by zoltan@webkit.org
  • 3 edits in trunk/WebCore

2009-12-04 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Eric Seidel.

Allow custom memory allocation control for 2 classes of the platform directory in WebCore
https://bugs.webkit.org/show_bug.cgi?id=

Inherits the following class from Noncopyable because it is instantiated
by 'new' and no need to be copyable:

class/struct name - instantiated at: WebCore/'location'

class SharedTimer - (its child class) workers/WorkerRunLoop.cpp:91

Inherits the following class from FastAllocBase because it is
instantiated by 'new':

class TransformOperations - rendering/style/RenderStyle.h:1175

  • platform/SharedTimer.h:
  • platform/graphics/transforms/TransformOperations.h:
00:25 Changeset [51686] by eric@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

2009-12-04 Yuzo Fujishima <yuzo@google.com>

Reviewed by Darin Fisher.

Add test to see if unicode messages can be sent over web socket.

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

  • websocket/tests/script-tests/unicode.js: Added. (endTest): (ws.onopen): (ws.onmessage): (ws.onclose): (timeOutCallback):
  • websocket/tests/unicode-expected.txt: Added.
  • websocket/tests/unicode.html: Added.
  • websocket/tests/unicode_wsh.py: Added.
00:15 Changeset [51685] by zoltan@webkit.org
  • 5 edits in trunk/WebCore

2009-12-04 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Eric Seidel.

Allow custom memory allocation control for the svg directory in WebCore
https://bugs.webkit.org/show_bug.cgi?id=

Inherits the following classes from Noncopyable because these are instantiated
by 'new' and no need to be copyable:

class/struct name - instantiated at: WebCore/'location'

struct ResourceSet - svg/graphics/SVGResource.cpp:148
class SVGFontData - css/CSSFontFaceSource.cpp:156
class SVGDocumentExtensions - dom/Document.cpp:3962
class SVGViewSpec - svg/SVGSVGElement.cpp:191

  • svg/SVGDocumentExtensions.h:
  • svg/SVGFontData.h:
  • svg/SVGViewSpec.h:
  • svg/graphics/SVGResource.cpp:

12/03/09:

23:50 Changeset [51684] by cfleizach@apple.com
  • 7 edits
    4 adds in trunk

WAI-ARIA: aria-activedescendant doesn't work as intended
https://bugs.webkit.org/show_bug.cgi?id=32100

Reviewed by Beth Dakin.

WebCore:

Fixes a number of issues regarding the "tree" role and aria-activedescendant.

  1. The indexes were being reported incorrectly by treeitems.
  2. aria-activedescendant changes were not being sent to the containing item.
  3. The tree's selected rows need to consult aria-activedescendant.
  4. Since a tree changes what it returns as its children (it returns its rows) the mac-specific array indexing methods need to correctly handle the tree case.

Tests: platform/mac/accessibility/aria-tree-activedescendant.html

platform/mac/accessibility/aria-tree-index-of-items.html

  • accessibility/AXObjectCache.h:

(WebCore::AXObjectCache::):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::shouldFocusActiveDescendant):
(WebCore::AccessibilityRenderObject::activeDescendant):
(WebCore::AccessibilityRenderObject::handleActiveDescendantChanged):
(WebCore::AccessibilityRenderObject::ariaTreeSelectedRows):

  • accessibility/mac/AXObjectCacheMac.mm:

(WebCore::AXObjectCache::postPlatformNotification):

  • accessibility/mac/AccessibilityObjectWrapper.mm:

(-[AccessibilityObjectWrapper accessibilityAttributeValue:]):
(-[AccessibilityObjectWrapper accessibilityIndexOfChild:]):
(-[AccessibilityObjectWrapper accessibilityArrayAttributeValues:index:maxCount:]):

  • accessibility/win/AXObjectCacheWin.cpp:

(WebCore::AXObjectCache::postPlatformNotification):

LayoutTests:

  • platform/mac/accessibility/aria-tree-activedescendant-expected.txt: Added.
  • platform/mac/accessibility/aria-tree-activedescendant.html: Added.
  • platform/mac/accessibility/aria-tree-index-of-items-expected.txt: Added.
  • platform/mac/accessibility/aria-tree-index-of-items.html: Added.
23:48 Changeset [51683] by zoltan@webkit.org
  • 5 edits in trunk/WebCore

2009-12-03 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Eric Seidel.

Allow custom memory allocation control for the xml directory in WebCore
https://bugs.webkit.org/show_bug.cgi?id=

Inherits the following class from Noncopyable because it is instantiated
by 'new' and no need to be copyable:

class/struct name - instantiated at: WebCore/'location'

struct XMLHttpRequestStaticData - xml/XMLHttpRequest.cpp:134

Inherits the following classes from FastAllocBase because these are instantiated by 'new':

class NodeTest - xml/XPathGrammar.y:258
class NodeSet - xml/XPathValue.cpp:52
struct EvaluationContext - xml/XPathExpressionNode.cpp:40

  • xml/XMLHttpRequest.cpp:
  • xml/XPathExpressionNode.h:
  • xml/XPathNodeSet.h:
  • xml/XPathStep.h:
23:42 Changeset [51682] by hamaji@chromium.org
  • 3 edits in trunk/WebKitTools

2009-12-03 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by David Levin.

check-webkit-style should check for camelCase variable names
https://bugs.webkit.org/show_bug.cgi?id=32051

  • Scripts/modules/cpp_style.py:
  • Scripts/modules/cpp_style_unittest.py:
22:40 Changeset [51681] by atwilson@chromium.org
  • 4 edits
    1 add in trunk/WebCore

New History changes do not compile for Chromium/V8
https://bugs.webkit.org/show_bug.cgi?id=32148

Reviewed by Adam Barth.

Existing tests suffice (just trying to get code to compile).

  • WebCore.gypi:

Added V8HistoryCustom.cpp and other missing files.

  • bindings/v8/custom/V8CustomBinding.h:

Added custom handlers for History.pushState()/replaceState().

  • bindings/v8/custom/V8HistoryCustom.cpp: Added.

Added custom handlers for History.pushState()/replaceState().

  • history/BackForwardListChromium.cpp:

(WebCore::BackForwardList::pushStateItem):
Stubbed out this routine for now - will implement in the future.

22:19 Changeset [51680] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-12-03 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

OwnPtr<XBLBindingManager> Document::m_bindingManager;
https://bugs.webkit.org/show_bug.cgi?id=32147

The document actually owns the bindingManager. That's what the code
should say.

  • dom/Document.cpp: (WebCore::Document::~Document):
  • dom/Document.h: (WebCore::Document::bindingManager):
21:55 Changeset [51679] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2009-12-03 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

OwnPtr<Tokenizer> Document::m_tokenizer;
https://bugs.webkit.org/show_bug.cgi?id=32145

The document actually owns the tokenizer. That's what the code should say.

  • dom/Document.cpp: (WebCore::Document::removedLastRef): (WebCore::Document::~Document): (WebCore::Document::cancelParsing): (WebCore::Document::implicitOpen): (WebCore::Document::implicitClose):
  • dom/Document.h: (WebCore::Document::tokenizer):
21:54 Changeset [51678] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2009-12-03 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

OwnPtr<RenderArena> Document::m_renderArena;
https://bugs.webkit.org/show_bug.cgi?id=32146

The document owns the renderArena. That's what the code should say.

  • dom/Document.cpp: (WebCore::Document::Document): Also, removed a redundant initialization of the tokenizer that I missed in my previous patch. (WebCore::Document::~Document): (WebCore::Document::attach): (WebCore::Document::detach):
  • dom/Document.h: (WebCore::Document::renderArena):
21:54 Changeset [51677] by atwilson@chromium.org
  • 5 edits in trunk

Revert r51633

This reverts commit 4c9cc0b9a12d38654b9c687ffc1671c7307b397a.

21:33 QtWebKitBackportingFixes edited by laszlo.1.gombos@nokia.com
(diff)
21:28 Changeset [51676] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2009-12-03 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

The code should say that Document owns DocLoader
https://bugs.webkit.org/show_bug.cgi?id=32143

It's the truth.

  • dom/Document.cpp: (WebCore::Document::~Document):
  • dom/Document.h: (WebCore::Document::docLoader):
18:53 Changeset [51675] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Try and fix the Windows build.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def: Export a symbol that should be exported.
18:35 Changeset [51674] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

Try and fix the Mac build.

18:28 Changeset [51673] by cfleizach@apple.com
  • 3 edits in trunk/LayoutTests

Fixing layout test bustage on platforms.

AX: VO just says "term" on many web sites
https://bugs.webkit.org/show_bug.cgi?id=32139

  • platform/mac-leopard/accessibility/lists-expected.txt:
  • platform/mac/accessibility/radio-button-group-members-expected.txt:
18:17 Changeset [51672] by oliver@apple.com
  • 6 edits in trunk/JavaScriptCore

REGRESSION(4.0.3-48777): Crash in JSC::ExecState::propertyNames() (Debug-only?)
https://bugs.webkit.org/show_bug.cgi?id=32133

Reviewed by Gavin Barraclough.

Work around odd GCC-ism and correct the scopechain for use by
calls made while a cachedcall is active on the callstack.

18:15 Changeset [51671] by barraclough@apple.com
  • 9 edits in trunk/JavaScriptCore

https://bugs.webkit.org/show_bug.cgi?id=32136
Add a rope representation to JSString. Presently JSString always holds its data in UString form.
Instead, allow the result of a string concatenation to be represented in a tree form - with a
variable sized, reference-counted rope node retaining a set of UString::Reps (or other rope nopes).

Reviewed by Oliver "Brraaaaiiiinnnnnzzzzzzzz" Hunt.

Strings must still currently be resolved down to a flat UString representation before being used,
but by holding the string in a rope representation during construction we can avoid copying data
until we know the final size of the string.

~2% progression on SunSpider (~25% on date-format-xparb, ~20% on string-validate-input).

  • Update exports.
  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • Make use of new JSString::length() method to avoid prematurely resolving ropes.
  • jit/JITOpcodes.cpp:

(JSC::JIT::privateCompileCTIMachineTrampolines):

  • Switch the string length trampoline to read the length directly from JSString::m_length, rather than from the JSString's UString::Rep's 'len' property.
  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • Modify op_add such that addition of two strings, where either or both strings are already in rope representation, produces a rope as a result.
  • runtime/JSString.cpp:

(JSC::JSString::Rope::~Rope):
(JSC::copyChars):
(JSC::JSString::resolveRope):
(JSC::JSString::getPrimitiveNumber):
(JSC::JSString::toBoolean):
(JSC::JSString::toNumber):
(JSC::JSString::toString):
(JSC::JSString::toThisString):
(JSC::JSString::getStringPropertyDescriptor):

  • runtime/JSString.h:

(JSC::JSString::Rope::Fiber::Fiber):
(JSC::JSString::Rope::Fiber::destroy):
(JSC::JSString::Rope::Fiber::isRope):
(JSC::JSString::Rope::Fiber::rope):
(JSC::JSString::Rope::Fiber::string):
(JSC::JSString::Rope::create):
(JSC::JSString::Rope::initializeFiber):
(JSC::JSString::Rope::ropeLength):
(JSC::JSString::Rope::stringLength):
(JSC::JSString::Rope::fibers):
(JSC::JSString::Rope::Rope):
(JSC::JSString::Rope::operator new):
(JSC::JSString::JSString):
(JSC::JSString::value):
(JSC::JSString::length):
(JSC::JSString::isRope):
(JSC::JSString::rope):
(JSC::JSString::string):
(JSC::JSString::canGetIndex):
(JSC::jsSingleCharacterSubstring):
(JSC::JSString::getIndex):
(JSC::jsSubstring):
(JSC::JSString::getStringPropertySlot):

  • Add rope form.
  • runtime/Operations.h:

(JSC::jsAdd):
(JSC::concatenateStrings):

  • Update string concatenation, and addition of ropes, to produce ropes.
  • runtime/StringObject.cpp:

(JSC::StringObject::getOwnPropertyNames):

  • Make use of new JSString::length() method to avoid prematurely resolving ropes.
18:06 Changeset [51670] by cfleizach@apple.com
  • 13 edits
    2 adds in trunk

AX: VO just says "term" on many web sites
https://bugs.webkit.org/show_bug.cgi?id=32139

Reviewed by Eric Seidel.

WebCore:

Test: platform/mac/accessibility/definition-list-term.html

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::accessibilityDescription):

  • accessibility/mac/AccessibilityObjectWrapper.mm:

(-[AccessibilityObjectWrapper subrole]):
(-[AccessibilityObjectWrapper roleDescription]):

WebKitTools:

  • DumpRenderTree/AccessibilityUIElement.cpp:

(getRoleDescriptionCallback):
(AccessibilityUIElement::getJSClass):

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

(AccessibilityUIElement::roleDescription):

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::roleDescription):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::roleDescription):

LayoutTests:

  • platform/mac/accessibility/aria-grouping-roles-expected.txt:
  • platform/mac/accessibility/aria-grouping-roles.html:
  • platform/mac/accessibility/definition-list-term-expected.txt: Added.
  • platform/mac/accessibility/definition-list-term.html: Added.
  • platform/mac/accessibility/lists-expected.txt:
17:00 Changeset [51669] by cfleizach@apple.com
  • 16 edits
    2 adds in trunk

Implement WAI-ARIA scrollbar role and related property aria-orientation
https://bugs.webkit.org/show_bug.cgi?id=32126

Reviewed by Beth Dakin.

WebCore:

Test: accessibility/aria-scrollbar-role.html

  • accessibility/AccessibilityObject.cpp:

(WebCore::createARIARoleMap):

  • accessibility/AccessibilityObject.h:

(WebCore::AccessibilityObject::isScrollbar):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::valueForRange):
(WebCore::AccessibilityRenderObject::orientation):
(WebCore::AccessibilityRenderObject::canHaveChildren):

  • accessibility/AccessibilityRenderObject.h:
  • accessibility/mac/AccessibilityObjectWrapper.mm:

(-[AccessibilityObjectWrapper additionalAccessibilityAttributeNames]):
(-[AccessibilityObjectWrapper accessibilityAttributeValue:]):

  • html/HTMLAttributeNames.in:

WebKitTools:

  • DumpRenderTree/AccessibilityUIElement.cpp:

(getOrientationCallback):
(AccessibilityUIElement::getJSClass):

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

(AccessibilityUIElement::orientation):

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::orientation):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::orientation):

LayoutTests:

  • accessibility/aria-scrollbar-role-expected.txt: Added.
  • accessibility/aria-scrollbar-role.html: Added.
  • platform/gtk/Skipped:
  • platform/win/Skipped:
16:16 Changeset [51668] by mitz@apple.com
  • 4 edits
    4 adds in trunk

Fixed <rdar://problem/7401617> Selection painting issue in hardware-
accelerated layers
which is another part of https://bugs.webkit.org/show_bug.cgi?id=23628
Fix selection painting to do container-relative repaints

Reviewed by Darin Adler.

Test: fast/repaint/block-selection-gap-in-composited-layer.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::selectionGapRectsForRepaint): Compute and paint
gap rects in the coordinate space of the repaint container.

  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::recursiveRepaintLayerRect): Added a
FIXME.

  • rendering/RenderView.cpp:

(WebCore::RenderView::setSelection): Map block selection gap rects from
the repaint container’s coordinate space to the view’s coordinate space
before adding them to the cached selection bounds.
(WebCore::RenderView::clearSelection): Changed to use
repaintRectangleInViewAndCompositedLayers() so that the selection rect
is invalidated in composited layers as well.

16:07 Changeset [51667] by ap@apple.com
  • 4 edits in trunk/LayoutTests

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=32132
websocket/tests/simple.html fails on Windows

  • websocket/tests/script-tests/simple.js:
  • websocket/tests/simple-expected.txt: Make the test weaker - readyState could already grow by the time a posted event gets dispatched.
  • platform/win/Skipped: Un-skip the test.
15:37 Changeset [51666] by jparent@chromium.org
  • 2 edits in trunk/LayoutTests

Make shorthand-transitions.html less flaky.
The test currently allows for timing related differences by using a tolerance, but the tolerance
is inconsistent in the test, even for values read at the same time (some values are allowed a 100ms
difference, others are only allowed 50ms). This change makes all values have a 100ms tolerance.
(open question: is 100ms enough? I've looked through a bunch of logs of failures and it seems to be,
so this should definitely help reduce but this may not entirely eliminate flakiness).

Reviewed by Eric Seidel.

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

  • transitions/shorthand-transitions.html:
15:24 Changeset [51665] by eric@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2009-12-03 Csaba Osztrogonác <ossy@webkit.org>

Reviewed by Eric Seidel.

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

[Qt] Add expected files for new tests we pass.
Compared to the Mac expected png files.

  • platform/qt/fast/css/namespaces/namespaces-escapes-expected.txt: Added.
  • platform/qt/fast/css/namespaces/namespaces-invalid-at-expected.txt: Added.
14:17 Changeset [51664] by eric.carlson@apple.com
  • 2 edits in trunk/WebKitTools

2009-12-03 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben.

Minor correction to r51663.

  • DumpRenderTree/win/DumpRenderTree.cpp: (addQTDirToPATH):

GetEnvironmentVariable -> GetEnvironmentVariableW.

14:10 Changeset [51663] by eric.carlson@apple.com
  • 2 edits in trunk/WebKitTools

2009-12-03 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben.

~96 regression tests fail when using QuickTime 7.6 (they pass with QuickTime 7.3)
https://bugs.webkit.org/show_bug.cgi?id=30256

Add the QuickTime dll directory to the PATH environment variable so
inialization can succeed.

  • DumpRenderTree/win/DumpRenderTree.cpp: (addQTDirToPATH): (initialize):
14:08 Changeset [51662] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-03 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] After revision 32643, sender() is of type QWebPage instead of QWebFrame.

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::maybeDump):
14:01 Changeset [51661] by eric@webkit.org
  • 8 edits in trunk/WebKitTools

2009-12-03 Yuzo Fujishima <yuzo@google.com>

Reviewed by Eric Seidel.

Update pywebsocket to 0.4.3
This version logs friendlier and higher-level messages in WARN level, which is used for LayoutTests.
Stack trace is logged now in INFO level.
https://bugs.webkit.org/show_bug.cgi?id=32097

  • pywebsocket/mod_pywebsocket/dispatch.py:
  • pywebsocket/mod_pywebsocket/msgutil.py:
  • pywebsocket/mod_pywebsocket/standalone.py:
  • pywebsocket/mod_pywebsocket/util.py:
  • pywebsocket/setup.py:
  • pywebsocket/test/test_dispatch.py:
  • pywebsocket/test/test_util.py:
13:54 Changeset [51660] by eric@webkit.org
  • 9 edits
    1 add in trunk

2009-12-03 Jonathan Dixon <joth@chromium.org>

Reviewed by Dimitri Glazkov.

Bug 32066 - Add enable geolocation flag to WebCore::Settings for Chromium
https://bugs.webkit.org/show_bug.cgi?id=32066

Adds geolocationEnabled in Settings for runtime control of geolocaiton features. This defaults to ON
as it is intended as a development and testing aid, not a user control. To completely remove geolocation
from a given port, the compile time ENABLE_GEOLOCATION should still be used.
Adding placeholder GeolocationServiceChromium implementation, as this is required to allows
ENABLE_GEOLOCATION to be defined (in turn required for testing), even though this patch does not
make that the default just yet.

  • WebCore.gypi:
  • page/Settings.cpp: (WebCore::Settings::Settings): Add new m_geolocationEnabled flag default value (WebCore::Settings::setGeolocationEnabled): Setter for m_geolocationEnabled flag
  • page/Settings.h: (WebCore::Settings::geolocationEnabled): Add m_geolocationEnabled flag
  • platform/chromium/GeolocationServiceChromium.cpp: Added. (WebCore::GeolocationServiceChromium::GeolocationServiceChromium): Place holder GeolocationServiceChromium) (WebCore::createGeolocationService): factory function, only required when ENABLED(GEOLOCATION) is true

2009-12-03 Jonathan Dixon <joth@chromium.org>

Reviewed by Dimitri Glazkov.

Bug 32066 - Add enable geolocation flag to WebCore::Settings for Chromium
https://bugs.webkit.org/show_bug.cgi?id=32066

Add the plumbing to map WebSettings through to WebCore::Settings

  • features.gypi:
  • public/WebSettings.h: Add setGeolocationEnabled API to interface
  • src/WebSettingsImpl.cpp: (WebKit::WebSettingsImpl::setGeolocationEnabled): Calls through to WebCore::Settings::setGeolocationEnabled
  • src/WebSettingsImpl.h: Add setGeolocationEnabled interface function
13:43 Changeset [51659] by enrica@apple.com
  • 2 edits in trunk/WebCore

SpinTracer: 349 spins in Mail at WebCore::Editor::advanceToNextMisspelling(bool)
<rdar://problem/7198592>
https://bugs.webkit.org/show_bug.cgi?id=32129

Reviewed by John Sullivan.

Fixed the way the next paragraph is calculated.
I've added a repro case in Safari and attached it to the Bugzilla bug.

  • editing/Editor.cpp:

(WebCore::findFirstMisspellingOrBadGrammarInRange): Changed the way we move to the
next paragraph in the loop.

13:37 Changeset [51658] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-12-03 İsmail Dönmez <ismail@namtrac.org>

Reviewed by Eric Seidel.

Fix compilation when SVG is disabled.

  • Api/qwebframe.cpp: (qt_drt_pauseSVGAnimation):
13:27 Changeset [51657] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Alexey Proskuryakov.

Skip failing WebSocket test on Windows.

  • platform/win/Skipped:
13:25 Changeset [51656] by weinig@apple.com
  • 4 edits in trunk/WebKitTools

Fix JSClassRef leak.

Reviewed by Gavin Barraclough.

  • DumpRenderTree/AccessibilityController.cpp:

(AccessibilityController::makeWindowObject):

  • DumpRenderTree/GCController.cpp:

(GCController::makeWindowObject):

  • DumpRenderTree/LayoutTestController.cpp:

(LayoutTestController::makeWindowObject):

13:19 Changeset [51655] by beidson@apple.com
  • 3 edits in trunk/LayoutTests

Rubberstamped by Oliver Hunt.

Fix layout tests by removing the full outputting of the Date object and just outputting the milliseconds since the epoch.

  • fast/loader/stateobjects/pushstate-object-types-expected.txt:
  • fast/loader/stateobjects/pushstate-object-types.html:
13:06 Changeset [51654] by ap@apple.com
  • 1 edit
    1 copy
    2 adds in trunk/LayoutTests

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=32130
Add a test for sending non-trivial amount of data over WebSocket

  • websocket/tests/echo_wsh.py: Copied from WebKitTools/pywebsocket/example/echo_wsh.py.
  • websocket/tests/simple-stress-expected.txt: Added.
  • websocket/tests/simple-stress.html: Added.
12:59 Changeset [51653] by beidson@apple.com
  • 7 edits in trunk/LayoutTests

Rubberstamped by Sam Weinig.

Fix my absolute URL snafu by outputting only the last path component for these URLs,
which is all that is relevant.

  • fast/loader/stateobjects/document-destroyed-navigate-back-expected.txt:
  • fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll-expected.txt:
  • fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll.html:
  • fast/loader/stateobjects/document-destroyed-navigate-back.html:
  • fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange-expected.txt:
  • fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange.html:
12:38 Changeset [51652] by oliver@apple.com
  • 2 edits in trunk/WebCore

Build fix

11:51 Changeset [51651] by oliver@apple.com
  • 2 edits in trunk/WebCore

Build fix

11:39 Changeset [51650] by beidson@apple.com
  • 2 edits in trunk/WebCore

No wonder editing WebCore.exp repeatedly while I worked on this patch never seemed to stick.

Long live our auto-generated overlords that I didn't know about...

  • WebCore.base.exp:
11:34 Changeset [51649] by oliver@apple.com
  • 6 edits in trunk

REGRESSION (r51627): 3 SVG tests are failing
https://bugs.webkit.org/show_bug.cgi?id=32117

Reviewed by Dan Bernstein.

Null checking Items in the SVGList is insufficient as items may
be floats, etc so add SVGListTraits::isNull and add appropriate
specializations.

Test: svg/dom/svglist-exception-on-out-bounds-error.html

11:32 Changeset [51648] by beidson@apple.com
  • 2 edits in trunk/WebCore

No review - release build fix.

  • history/HistoryItem.cpp:

(WebCore::HistoryItem::documentDetached):

11:27 Changeset [51647] by beidson@apple.com
  • 2 edits in trunk/WebCore

No review - release build fix.

  • loader/HistoryController.cpp:

(WebCore::HistoryController::pushState):

11:25 Changeset [51646] by bweinstein@apple.com
  • 1 edit in trunk/WebKit/win/Interfaces/WebKit.idl

Touch WebKit.idl to force Interfaces rebuild and fix Windows build.

11:18 Changeset [51645] by enrica@apple.com
  • 12 edits
    2 adds in trunk

WebCore: Multiple Undos removes the last posted comment and/or the ones before it.
<rdar://problem/6557066>
https://bugs.webkit.org/show_bug.cgi?id=32079

Reviewed by Adele Peterson.

The fix consists in preventing Undos and Redos to be applied to elements that
are no longer editable. We do not attempt to remove the commands from the queue.
It is a relatively simple fix that comes with a little extra overhead for the apply
and unapply methods where we are now performing the editability check.

Test: editing/undo/undo-indent-noneditable.html

  • editing/AppendNodeCommand.cpp:

(WebCore::AppendNodeCommand::doApply):
(WebCore::AppendNodeCommand::doUnapply):

  • editing/DeleteFromTextNodeCommand.cpp:

(WebCore::DeleteFromTextNodeCommand::doApply):
(WebCore::DeleteFromTextNodeCommand::doUnapply):

  • editing/InsertIntoTextNodeCommand.cpp:

(WebCore::InsertIntoTextNodeCommand::doApply):
(WebCore::InsertIntoTextNodeCommand::doUnapply):

  • editing/InsertNodeBeforeCommand.cpp:

(WebCore::InsertNodeBeforeCommand::doApply):
(WebCore::InsertNodeBeforeCommand::doUnapply):

  • editing/JoinTextNodesCommand.cpp:

(WebCore::JoinTextNodesCommand::doApply):
(WebCore::JoinTextNodesCommand::doUnapply):

  • editing/MergeIdenticalElementsCommand.cpp:

(WebCore::MergeIdenticalElementsCommand::doApply):
(WebCore::MergeIdenticalElementsCommand::doUnapply):

  • editing/RemoveNodeCommand.cpp:

(WebCore::RemoveNodeCommand::doApply):
(WebCore::RemoveNodeCommand::doUnapply):

  • editing/SplitElementCommand.cpp:

(WebCore::SplitElementCommand::executeApply):
(WebCore::SplitElementCommand::doUnapply):

  • editing/SplitTextNodeCommand.cpp:

(WebCore::SplitTextNodeCommand::doApply):
(WebCore::SplitTextNodeCommand::doUnapply):

  • editing/WrapContentsInDummySpanCommand.cpp:

(WebCore::WrapContentsInDummySpanCommand::doUnapply):
(WebCore::WrapContentsInDummySpanCommand::doReapply):

LayoutTests: Multiple Undos removes the last posted comment and/or the ones before it.
<rdar://problem/6557066>
https://bugs.webkit.org/show_bug.cgi?id=32079

Reviewed by Adele Peterson.

Added test to verify that undo has no effect when an element is
no longer editable.

  • editing/undo/undo-indent-noneditable-expected.txt: Added.
  • editing/undo/undo-indent-noneditable.html: Added.
11:04 Changeset [51644] by beidson@apple.com
  • 67 edits
    29 adds in trunk

<rdar://problem/7214236> and http://webkit.org/b/32052 - Implement HTML5 state object history API

Reviewed by Sam Weinig.

WebCore:

Tests: fast/loader/stateobjects/document-destroyed-navigate-back.html

fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll.html
fast/loader/stateobjects/popstate-after-load-complete-addeventlistener.html
fast/loader/stateobjects/popstate-after-load-complete-body-attribute.html
fast/loader/stateobjects/popstate-after-load-complete-window-attribute.html
fast/loader/stateobjects/pushstate-object-types.html
fast/loader/stateobjects/pushstate-then-replacestate.html
fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange.html
fast/loader/stateobjects/replacestate-then-pushstate.html
http/tests/loading/state-object-security-exception.html

Derived sources and project file changes:

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am
  • WebCore.pro
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:

Add the new PopStateEvent:

  • dom/PopStateEvent.cpp: Added.

(WebCore::PopStateEvent::PopStateEvent):
(WebCore::PopStateEvent::initPopStateEvent):

  • dom/PopStateEvent.h: Added.

(WebCore::PopStateEvent::create):
(WebCore::PopStateEvent::isPopStateEvent):
(WebCore::PopStateEvent::state):

  • dom/PopStateEvent.idl: Added.
  • bindings/js/JSPopStateEventCustom.cpp: Added.

(WebCore::JSPopStateEvent::initPopStateEvent):
(WebCore::JSPopStateEvent::state):

  • bindings/js/JSEventCustom.cpp:

(WebCore::toJS):

  • dom/Event.cpp:

(WebCore::Event::isPopStateEvent):

  • dom/Event.h:
  • dom/EventNames.h:

Add the "onpopstate" attribute:

  • html/HTMLAttributeNames.in:
  • html/HTMLBodyElement.cpp:

(WebCore::HTMLBodyElement::parseMappedAttribute):

  • html/HTMLBodyElement.idl:
  • html/HTMLFrameSetElement.cpp:

(WebCore::HTMLFrameSetElement::parseMappedAttribute):

  • html/HTMLFrameSetElement.h:
  • html/HTMLFrameSetElement.idl:
  • page/DOMWindow.h:
  • page/DOMWindow.idl:

Add pushState and replaceState management to the loader and history machinery:

  • bindings/js/JSHistoryCustom.cpp:

(WebCore::JSHistory::pushState):
(WebCore::JSHistory::replaceState):

  • loader/HistoryController.cpp:

(WebCore::HistoryController::updateForSameDocumentNavigation): Augmented from "scrollToAnchor()", combining

both the same-document fragment scroll case with the new same-document state object activation case.

(WebCore::HistoryController::pushState):
(WebCore::HistoryController::replaceState):

  • loader/HistoryController.h:
  • history/BackForwardList.cpp:

(WebCore::BackForwardList::addItem): Use insertItemAfterCurrent.
(WebCore::BackForwardList::insertItemAfterCurrent): Optionally insert the item without clearing the forward

list, as pushStateItem might've selectively cleared only certain items, with the bulk of the forward list
meant to remain.

(WebCore::BackForwardList::pushStateItem): Clear the forward list *only* for the state item's document, then

insert the new item.

(WebCore::BackForwardList::removeItem):

  • history/BackForwardList.h:
  • page/History.cpp:

(WebCore::History::urlForState):
(WebCore::History::stateObjectAdded):

  • page/History.h:
  • page/History.idl:

Let HistoryItems and Documents associate with each other, as well as letting HistoryItems contain state objects:

  • history/HistoryItem.cpp:

(WebCore::HistoryItem::HistoryItem):
(WebCore::HistoryItem::~HistoryItem):
(WebCore::HistoryItem::setStateObject):
(WebCore::HistoryItem::setDocument):
(WebCore::HistoryItem::documentDetached):

  • history/HistoryItem.h:

(WebCore::HistoryItem::stateObject):
(WebCore::HistoryItem::document):

  • dom/Document.cpp:

(WebCore::Document::detach): Notify all back/forward history items owned by this Document that it

is going away.

(WebCore::Document::registerHistoryItem): Manage the list of back/forward history items this document owns.
(WebCore::Document::unregisterHistoryItem): Ditto.

  • dom/Document.h:

Add the ability for Documents, DocumentLoaders, and FrameLoaderClients to be notified when a Documents
URL changes as the result of pushState(), replaceState(), or a popstate navigation:

  • dom/Document.cpp:

(WebCore::Document::implicitClose): If there's a pending state object, dispatch the popstate event.
(WebCore::Document::updateURLForPushOrReplaceState):
(WebCore::Document::statePopped): If loading is complete, dispatch the popstate event. Otherwise, set

the pending state object.

  • loader/DocumentLoader.cpp:

(WebCore::DocumentLoader::replaceRequestURLForSameDocumentNavigation):

  • loader/DocumentLoader.h:
  • loader/FrameLoaderClient.h:
  • loader/EmptyClients.h:

(WebCore::EmptyFrameLoaderClient::dispatchDidChangeStateObjectForPageForFrame):

Change handling of "loading a HistoryItem" to distinguish between new-Document navigations and same-Document
navigations, combining the old concept of anchor scrolls with the new concept of state object navigations:

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadInSameDocument):
(WebCore::FrameLoader::continueFragmentScrollAfterNavigationPolicy):
(WebCore::FrameLoader::navigateWithinDocument):
(WebCore::FrameLoader::navigateToDifferentDocument):
(WebCore::FrameLoader::loadItem):

  • loader/FrameLoader.h:
  • page/Page.cpp:

(WebCore::Page::goToItem): Changed to allow state object activations to pass through without the load stopping.

WebKit/chromium:

  • src/FrameLoaderClientImpl.cpp:

(WebKit::FrameLoaderClientImpl::dispatchDidPushStateWithinPage):
(WebKit::FrameLoaderClientImpl::dispatchDidReplaceStateWithinPage):
(WebKit::FrameLoaderClientImpl::dispatchDidPopStateWithinPage):

  • src/FrameLoaderClientImpl.h:

WebKit/gtk:

  • WebCoreSupport/FrameLoaderClientGtk.cpp:

(WebKit::FrameLoaderClient::dispatchDidPushStateWithinPage):
(WebKit::FrameLoaderClient::dispatchDidReplaceStateWithinPage):
(WebKit::FrameLoaderClient::dispatchDidPopStateWithinPage):

  • WebCoreSupport/FrameLoaderClientGtk.h:

WebKit/mac:

  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::dispatchDidPushStateWithinPage):
(WebFrameLoaderClient::dispatchDidReplaceStateWithinPage):
(WebFrameLoaderClient::dispatchDidPopStateWithinPage):

  • WebView/WebDelegateImplementationCaching.h:
  • WebView/WebFrameLoadDelegatePrivate.h:
  • WebView/WebView.mm:

(-[WebView _cacheFrameLoadDelegateImplementations]):

WebKit/qt:

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::dispatchDidPushStateWithinPage):
(WebCore::FrameLoaderClientQt::dispatchDidReplaceStateWithinPage):
(WebCore::FrameLoaderClientQt::dispatchDidPopStateWithinPage):

  • WebCoreSupport/FrameLoaderClientQt.h:

WebKit/win:

  • Interfaces/IWebFrameLoadDelegatePrivate2.idl:
  • WebCoreSupport/WebFrameLoaderClient.cpp:

(WebFrameLoaderClient::dispatchDidPushStateWithinPage):
(WebFrameLoaderClient::dispatchDidReplaceStateWithinPage):
(WebFrameLoaderClient::dispatchDidPopStateWithinPage):

  • WebCoreSupport/WebFrameLoaderClient.h:

WebKit/wx:

  • WebKitSupport/FrameLoaderClientWx.cpp:

(WebCore::FrameLoaderClientWx::dispatchDidPushStateWithinPage):
(WebCore::FrameLoaderClientWx::dispatchDidReplaceStateWithinPage):
(WebCore::FrameLoaderClientWx::dispatchDidPopStateWithinPage):

  • WebKitSupport/FrameLoaderClientWx.h:

WebKitTools:

Keep DRT-win building...

  • DumpRenderTree/win/FrameLoadDelegate.h:

(FrameLoadDelegate::didPushStateWithinPageForFrame):
(FrameLoadDelegate::didReplaceStateWithinPageForFrame):
(FrameLoadDelegate::didPopStateWithinPageForFrame):

LayoutTests:

Update expected results of old tests:

  • fast/dom/Window/window-appendages-cleared-expected.txt:
  • fast/dom/Window/window-properties-expected.txt:
  • http/tests/security/cross-frame-access-enumeration-expected.txt:

New tests:

  • fast/loader/stateobjects: Added.
  • fast/loader/stateobjects/document-destroyed-navigate-back-expected.txt: Added.
  • fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll-expected.txt: Added.
  • fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll.html: Added.
  • fast/loader/stateobjects/document-destroyed-navigate-back.html: Added.
  • fast/loader/stateobjects/popstate-after-load-complete-addeventlistener-expected.txt: Added.
  • fast/loader/stateobjects/popstate-after-load-complete-addeventlistener.html: Added.
  • fast/loader/stateobjects/popstate-after-load-complete-body-attribute-expected.txt: Added.
  • fast/loader/stateobjects/popstate-after-load-complete-body-attribute.html: Added.
  • fast/loader/stateobjects/popstate-after-load-complete-body-inline-attribute-expected.txt: Added.
  • fast/loader/stateobjects/popstate-after-load-complete-body-inline-attribute.html: Added.
  • fast/loader/stateobjects/popstate-after-load-complete-window-attribute-expected.txt: Added.
  • fast/loader/stateobjects/popstate-after-load-complete-window-attribute.html: Added.
  • fast/loader/stateobjects/pushstate-object-types-expected.txt: Added.
  • fast/loader/stateobjects/pushstate-object-types.html: Added.
  • fast/loader/stateobjects/pushstate-then-replacestate-expected.txt: Added.
  • fast/loader/stateobjects/pushstate-then-replacestate.html: Added.
  • fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange-expected.txt: Added.
  • fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange.html: Added.
  • fast/loader/stateobjects/replacestate-then-pushstate-expected.txt: Added.
  • fast/loader/stateobjects/replacestate-then-pushstate.html: Added.
  • fast/loader/stateobjects/resources: Added.
  • fast/loader/stateobjects/resources/navigate-back.html: Added.
  • http/tests/loading/state-object-security-exception-expected.txt: Added.
  • http/tests/loading/state-object-security-exception.html: Added.
10:53 Changeset [51643] by kov@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

Reviewed by Xan Lopez.

Platform-specific result for test. GIO identifies the file as
application/javascript, as opposed to other platforms identifying
it as application/x-javascript.

  • platform/gtk/inspector/timeline-network-resource-expected.txt: Added.
10:04 Changeset [51642] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2009-12-03 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: chromium build fix.
r51621 changed JS bindings only, broke v8's.

  • bindings/v8/ScriptFunctionCall.cpp: (WebCore::ScriptFunctionCall::appendArgument):
  • bindings/v8/ScriptFunctionCall.h:
09:16 Changeset [51641] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Oliver Hunt.

Re-baseline required after r51522.

  • platform/gtk/editing/deleting/5390681-2-expected.txt:
09:09 Changeset [51640] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

Skipping new tests, and tests that require new DRT functionality.

09:06 Changeset [51639] by zoltan@webkit.org
  • 2 edits in trunk/WebKitTools

Unreviewed build fix.

Patch by Andras Becsi <abecsi@inf.u-szeged.hu> on 2009-12-03
[Qt] ARM-Linux build fix after r51634 removed unused QBoxLayout include which included limits.h on ARM.

  • DumpRenderTree/qt/DumpRenderTree.cpp:
08:57 Changeset [51638] by dglazkov@chromium.org
  • 6 edits in trunk/WebCore

2009-12-03 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Adam Barth.

[V8] Attributes and NamedNodeMaps aren't tracked correctly and may be prematurely garbage-collected.
https://bugs.webkit.org/show_bug.cgi?id=32094

Covered by existing test: LayoutTests/fast/dom/Attr/access-after-element-destruction.html

  • bindings/v8/DOMObjectsInclude.h:
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getTemplate): (WebCore::V8DOMWrapper::convertToV8Object): (WebCore::V8DOMWrapper::convertNamedNodeMapToV8Object):
  • bindings/v8/V8DOMWrapper.h:
  • bindings/v8/V8GCController.cpp: (WebCore::ObjectGrouperVisitor::visitDOMWrapper):
  • bindings/v8/custom/V8CustomBinding.h:
08:43 Changeset [51637] by pfeldman@chromium.org
  • 4 edits in trunk

2009-12-03 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Chromium: Add support for settings containing ":"

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

  • src/InspectorClientImpl.cpp: (WebKit::InspectorClientImpl::loadSettings):
06:25 Changeset [51636] by kenneth@webkit.org
  • 4 edits in trunk/WebCore

repaint events from outside the viewport aren't received
https://bugs.webkit.org/show_bug.cgi?id=32081

Reviewed by Simon Fraser.

When using a tiled backing store for painting, you need to receive
event from outside the viewport. Setting the viewport to the size
of the contents is not an option if you want to make use of WebCore's
infrastructure for drawing scrollbars etc.

A new property, paintsEntireContents, has been introduced for the
above use-case. It is settable, as tiling will be optional for Qt,
and for the not yet upstreamed EFL port, there will be two different
views, where only one of them are tiled.

No change in behavior, so no new tests added.

  • page/FrameView.cpp:

(WebCore::FrameView::repaintContentRectangle):

  • platform/ScrollView.cpp:

(WebCore::ScrollView::ScrollView):
(WebCore::ScrollView::setPaintsEntireContents):
(WebCore::ScrollView::wheelEvent):

  • platform/ScrollView.h:

(WebCore::ScrollView::paintsEntireContents):

06:24 Changeset [51635] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Skip layoutTestController.notifyDone() related tests which got revealed in r51634
until there is a fix for the issue.
https://bugs.webkit.org/show_bug.cgi?id=31626

Patch by Andras Becsi <abecsi@inf.u-szeged.hu> on 2009-12-03
Reviewed by Kenneth Rohde Christiansen.

  • platform/qt/Skipped:
06:20 Moving to Git edited by maruel@chromium.org
Add Gerrit2 link and the fact that cygwin is already a requirement on … (diff)
06:06 Changeset [51634] by ossy@webkit.org
  • 5 edits in trunk

WebKitTools: [Qt] Refactor DRT to not crash on tests which open child windows from javascript.
Prevent DRT from showing the main view if these childs get deleted.
This fixes https://bugs.webkit.org/show_bug.cgi?id=31591.

Patch by Andras Becsi <abecsi@inf.u-szeged.hu> on 2009-12-03
Reviewed by Kenneth Rohde Christiansen.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::WebPage::WebPage):
(WebCore::DumpRenderTree::DumpRenderTree):
(WebCore::DumpRenderTree::~DumpRenderTree):
(WebCore::DumpRenderTree::resetToConsistentStateBeforeTesting):
(WebCore::DumpRenderTree::open):
(WebCore::DumpRenderTree::closeRemainingWindows):
(WebCore::DumpRenderTree::createWindow):
(WebCore::DumpRenderTree::windowCount):

  • DumpRenderTree/qt/DumpRenderTree.h:

LayoutTests: [Qt] Re-enable previously skipped tests.
https://bugs.webkit.org/show_bug.cgi?id=31591

Patch by Andras Becsi <abecsi@inf.u-szeged.hu> on 2009-12-03
Reviewed by Kenneth Rohde Christiansen.

  • platform/qt/Skipped:
05:52 Changeset [51633] by jeremy@chromium.org
  • 5 edits in trunk

Reviewed by Eric Seidel.

Switch Chrome/Mac to use Core Text APIs rather than ATSUI APIs.
https://bugs.webkit.org/show_bug.cgi?id=31802

No test since this is already covered by existing pixel tests.

  • platform/graphics/SimpleFontData.h: Change #ifdef to define getNSFont() on Chrome/Mac .
  • platform/graphics/mac/ComplexTextControllerCoreText.cpp: Provide forward declarations

of Core Text functions that are public on 10.6 but SPI on 10.5.

05:15 Changeset [51632] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Put test into skiplist because of missing layoutTestController.evaluateInWebInspector().

  • platform/qt/Skipped:
  • inspector/settings-set-get.html added.
04:46 Changeset [51631] by pfeldman@chromium.org
  • 35 edits
    2 adds in trunk

2009-12-03 Pavel Feldman <pfeldman@dhcp-172-28-174-220.spb.corp.google.com>

Reviewed by Timothy Hatcher.

Web Inspector: Simplify the settings support in inspector controller.

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

Test: inspector/settings-set-get.html

  • WebCore.Inspector.exp:
  • bindings/js/JSInspectorFrontendHostCustom.cpp:
  • bindings/v8/custom/V8InspectorFrontendHostCustom.cpp:
  • inspector/InspectorClient.h:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController): (WebCore::InspectorController::setting): (WebCore::InspectorController::setSetting): (WebCore::InspectorController::setWindowVisible): (WebCore::InspectorController::attachWindow): (WebCore::InspectorController::setAttachedWindowHeight): (WebCore::InspectorController::storeLastActivePanel): (WebCore::InspectorController::scriptObjectReady): (WebCore::InspectorController::showWindow): (WebCore::InspectorController::enableResourceTracking): (WebCore::InspectorController::disableResourceTracking): (WebCore::InspectorController::ensureResourceTrackingSettingsLoaded): (WebCore::InspectorController::enableProfiler): (WebCore::InspectorController::disableProfiler): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontendHost.cpp: (WebCore::InspectorFrontendHost::setting): (WebCore::InspectorFrontendHost::setSetting):
  • inspector/InspectorFrontendHost.h:
  • inspector/InspectorFrontendHost.idl:
  • loader/EmptyClients.h: (WebCore::EmptyInspectorClient::populateSetting): (WebCore::EmptyInspectorClient::storeSetting):

WebKit/chromium

  • src/InspectorClientImpl.cpp: (WebKit::InspectorClientImpl::hiddenPanels): (WebKit::InspectorClientImpl::populateSetting): (WebKit::InspectorClientImpl::storeSetting): (WebKit::InspectorClientImpl::loadSettings): (WebKit::InspectorClientImpl::saveSettings):
  • src/InspectorClientImpl.h:

WebKit/cf

  • WebCoreSupport/WebInspectorClientCF.cpp: (WebInspectorClient::populateSetting): (WebInspectorClient::storeSetting):

WebKit/gtk

  • WebCoreSupport/InspectorClientGtk.cpp: (WebKit::InspectorClient::populateSetting): (WebKit::InspectorClient::storeSetting):
  • WebCoreSupport/InspectorClientGtk.h:

WebKit/haiku

  • WebCoreSupport/InspectorClientHaiku.cpp: (WebCore::InspectorClientHaiku::populateSetting): (WebCore::InspectorClientHaiku::storeSetting):
  • WebCoreSupport/InspectorClientHaiku.h:

WebKit/mac

  • WebCoreSupport/WebInspectorClient.h:
  • WebCoreSupport/WebInspectorClient.mm: (-[WebInspectorWindowController showWindow:]): (-[WebInspectorWindowController attach]): (-[WebInspectorWindowController detach]):

WebKit/qt

  • WebCoreSupport/InspectorClientQt.cpp: (WebCore::InspectorClientQt::populateSetting): (WebCore::InspectorClientQt::storeSetting): (WebCore::variantToSetting): (WebCore::settingToVariant):
  • WebCoreSupport/InspectorClientQt.h:

WebKit/win

  • WebCoreSupport/WebInspectorClient.cpp: (WebInspectorClient::attachWindow): (WebInspectorClient::detachWindow): (WebInspectorClient::showWindowWithoutNotifications):
  • WebCoreSupport/WebInspectorClient.h:

WebKit/wx

  • WebKitSupport/InspectorClientWx.cpp: (WebCore::InspectorClientWx::populateSetting): (WebCore::InspectorClientWx::storeSetting):
  • WebKitSupport/InspectorClientWx.h:
04:31 Changeset [51630] by benm@google.com
  • 6 edits in trunk

[Android] notifyHistoryItemChanged() should pass a pointer to the HistoryItem that changed.
https://bugs.webkit.org/show_bug.cgi?id=31915

Reviewed by Brady Eidson.

WebCore:

No change in functionality so no new tests required.

  • history/HistoryItem.cpp:

(WebCore::defaultNotifyHistoryItemChanged): Update this function to pass the HistoryItem that is being changed.
(WebCore::HistoryItem::setAlternateTitle): Update call to notifyHistoryItemChanged to include the new parameter.
(WebCore::HistoryItem::setURLString): ditto.
(WebCore::HistoryItem::setOriginalURLString): ditto.
(WebCore::HistoryItem::setReferrer): ditto.
(WebCore::HistoryItem::setTitle): ditto.
(WebCore::HistoryItem::setTarget): ditto.
(WebCore::HistoryItem::setDocumentState): On Android, add a call to notifyHistoryItemChanged. See bug for a discussion of why this is needed.
(WebCore::HistoryItem::clearDocumentState): ditto.
(WebCore::HistoryItem::setIsTargetItem): ditto.
(WebCore::HistoryItem::addChildItem): ditto.
(WebCore::HistoryItem::setFormInfoFromRequest): ditto.

  • history/HistoryItem.h: Update signature of notifyHistoryItemChanged.

WebKit/mac:

  • History/WebHistoryItem.mm:

(WKNotifyHistoryItemChanged): Update WKNotifyHistoryItemChanged() to add the new HistoryItem parameter added in the WebCore portion of this patch.

  • History/WebHistoryItemInternal.h: ditto.
04:18 Changeset [51629] by benm@google.com
  • 25 edits in trunk

[Android] The FrameLoaderClient is unaware of BackForwardList changes.
https://bugs.webkit.org/show_bug.cgi?id=31914

Reviewed by Brady Eidson.

WebCore:

This change adds three new methods on the FrameLoaderClient interface to receive notifications when the BackForwardList changes.

No new tests required. Functionality on all platforms upstream remains the same. Android is the first platform to make use of these callbacks.

  • history/BackForwardList.cpp:

(WebCore::BackForwardList::addItem): Execute the callback.
(WebCore::BackForwardList::goBack): ditto.
(WebCore::BackForwardList::goForward): ditto.
(WebCore::BackForwardList::goToItem): ditto.
(WebCore::BackForwardList::setCapacity): dito.

  • loader/EmptyClients.h:

(WebCore::EmptyFrameLoaderClient::dispatchDidAddBackForwardItem): Add an empty implementation for the callback that does nothing.
(WebCore::EmptyFrameLoaderClient::dispatchDidRemoveBackForwardItem): ditto.
(WebCore::EmptyFrameLoaderClient::dispatchDidChangeBackForwardIndex): ditto.

  • loader/FrameLoaderClient.h:

WebKit/chromium:

  • src/FrameLoaderClientImpl.cpp:

(WebKit::FrameLoaderClientImpl::dispatchDidAddBackForwardItem): Add an empty implementation. Method added to FrameLoaderClient by Android (see bug).
(WebKit::FrameLoaderClientImpl::dispatchDidRemoveBackForwardItem): ditto.
(WebKit::FrameLoaderClientImpl::dispatchDidChangeBackForwardIndex): dito.

  • src/FrameLoaderClientImpl.h:

WebKit/gtk:

  • WebCoreSupport/FrameLoaderClientGtk.cpp:

(WebKit::FrameLoaderClient::dispatchDidAddBackForwardItem): Add an empty implementation. Method added to FrameLoaderClient by Android (see bug).
(WebKit::FrameLoaderClient::dispatchDidRemoveBackForwardItem): ditto.
(WebKit::FrameLoaderClient::dispatchDidChangeBackForwardIndex): ditto.

  • WebCoreSupport/FrameLoaderClientGtk.h:

WebKit/haiku:

  • WebCoreSupport/FrameLoaderClientHaiku.cpp:

(WebCore::FrameLoaderClientHaiku::dispatchDidAddBackForwardItem): Add an empty implementation. Method added to FrameLoaderClient by Android (see bug).
(WebCore::FrameLoaderClientHaiku::dispatchDidRemoveBackForwardItem): ditto.
(WebCore::FrameLoaderClientHaiku::dispatchDidChangeBackForwardIndex): ditto.

  • WebCoreSupport/FrameLoaderClientHaiku.h:

WebKit/mac:

  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::dispatchDidAddBackForwardItem): Add an empty implementation. Method added to FrameLoaderClient by Android (see bug).
(WebFrameLoaderClient::dispatchDidRemoveBackForwardItem): ditto.
(WebFrameLoaderClient::dispatchDidChangeBackForwardIndex): ditto.

WebKit/qt:

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::dispatchDidAddBackForwardItem): Add an empty implementation. Method added to FrameLoaderClient by Android (see bug).
(WebCore::FrameLoaderClientQt::dispatchDidRemoveBackForwardItem): ditto.
(WebCore::FrameLoaderClientQt::dispatchDidChangeBackForwardIndex): ditto.

  • WebCoreSupport/FrameLoaderClientQt.h:

WebKit/win:

  • WebCoreSupport/WebFrameLoaderClient.cpp:

(WebFrameLoaderClient::dispatchDidAddBackForwardItem): Add an empty implementation. Method added to FrameLoaderClient by Android (see bug).
(WebFrameLoaderClient::dispatchDidRemoveBackForwardItem): ditto.
(WebFrameLoaderClient::dispatchDidChangeBackForwardIndex): ditto.

  • WebCoreSupport/WebFrameLoaderClient.h:

WebKit/wx:

  • WebKitSupport/FrameLoaderClientWx.cpp:

(WebCore::FrameLoaderClientWx::dispatchDidAddBackForwardItem): Add an empty implementation. Method added to FrameLoaderClient by Android (see bug).
(WebCore::FrameLoaderClientWx::dispatchDidRemoveBackForwardItem): ditto.
(WebCore::FrameLoaderClientWx::dispatchDidChangeBackForwardIndex): ditto.

  • WebKitSupport/FrameLoaderClientWx.h:
03:52 Changeset [51628] by benm@google.com
  • 2 edits
    3 adds in trunk/WebCore

[Android] Upstream WebCore/history/android: Require some platform specific state attached to HistoryItem.
https://bugs.webkit.org/show_bug.cgi?id=31913

Reviewed by Brady Eidson.

Android stores information such as the zoom scale factor and bridge back to the Java counterpart with HistoryItem.

No new tests required as this is Android specific code.

  • history/HistoryItem.h: Add Android specific member data to HistoryItem.
  • history/android: Added.
  • history/android/AndroidWebHistoryBridge.h: Added.
  • history/android/HistoryItemAndroid.cpp: Added, provides implementation for Android specific member functions in HistoryItem.

(WebCore::HistoryItem::bridge): Added.
(WebCore::HistoryItem::setBridge): Added.

01:22 Changeset [51627] by oliver@apple.com
  • 8 edits
    2 adds in trunk

NULL ptr in SVGPathSegList::getPathSegAtLength()
https://bugs.webkit.org/show_bug.cgi?id=30313

Reviewed by Maciej Stachowiak.

Add exception checks to SVGPathSegList's implementation to catch (and propagate) exceptions.
Add null checks to SVGList's content manipulation functions to prevent
null values from entering the list in the first place.

Test: svg/dom/svgpath-out-of-bounds-getPathSeg.html

Note: See TracTimeline for information about the timeline view.