Timeline



Aug 24, 2009:

11:53 PM Changeset in webkit [47741] by abarth@webkit.org
  • 8 edits
    2 adds in trunk

2009-08-24 Hironori Bono <hbono@chromium.org>

Reviewed by Adam Barth.

Fix Bug 27827 "[Chromium] Functions Keys don't work in google spreadsheet".
<https://bugs.webkit.org/show_bug.cgi?id=27827>.

Because of the lack of mappings from GDK key-codes to WebKit key-codes,
Chromium cannot send valid key-codes to JavaScript when a user types
function keys. This change just copies the mappings from 'KeyEventGtk.cpp'.

To write layout tests for this issue, added mappings from function-key
names to platform-specific key-codes to EventSendingController objects
so that eventSender.keyDown() can send function-key events without using
platform-specific key codes. (Unfortunately, this eventSender.keyDown() change
is only for Mac. So this change adds this new test to Skipped tests for other
platforms to prevent this change from crashing the build trees.)

  • fast/events/keydown-function-keys-expected.txt: Added.
  • fast/events/keydown-function-keys.html: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:

2009-08-24 Hironori Bono <hbono@chromium.org>

Reviewed by Adam Barth.

Fix Bug 27827 "[Chromium] Functions Keys don't work in google spreadsheet".
<https://bugs.webkit.org/show_bug.cgi?id=27827>.

Because of the lack of mappings from GDK key-codes to WebKit key-codes,
Chromium cannot send valid key-codes to JavaScript when a user types
function keys. This change just copies the mappings from 'KeyEventGtk.cpp'.

To write layout tests for this issue, added mappings from function-key
names to platform-specific key-codes to EventSendingController objects
so that eventSender.keyDown() can send function-key events without using
platform-specific key codes. (Unfortunately, this eventSender.keyDown() change
is only for Mac. So this change adds this new test to Skipped tests for other
platforms to prevent this change from crashing the build trees.)

Test: fast/events/keydown-function-keys.html

  • platform/chromium/KeyCodeConversionGtk.cpp: Add mappings from GDK key-codes to WebKit key-code for function keys. (WebCore::windowsKeyCodeForKeyEvent):

2009-08-24 Hironori Bono <hbono@chromium.org>

Reviewed by Adam Barth.

Fix Bug 27827 "[Chromium] Functions Keys don't work in google spreadsheet".
<https://bugs.webkit.org/show_bug.cgi?id=27827>.

Because of the lack of mappings from GDK key-codes to WebKit key-codes,
Chromium cannot send valid key-codes to JavaScript when a user types
function keys. This change just copies the mappings from 'KeyEventGtk.cpp'.

To write layout tests for this issue, added mappings from function-key
names to platform-specific key-codes to EventSendingController objects
so that eventSender.keyDown() can send function-key events without using
platform-specific key codes. (Unfortunately, this eventSender.keyDown() change
is only for Mac. So this change adds this new test to Skipped tests for other
platforms to prevent this change from crashing the build trees.)

  • DumpRenderTree/mac/EventSendingController.mm: (-[EventSendingController keyDown:withModifiers:]):
9:23 PM Changeset in webkit [47740] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: start out with a 32-bit value to avoid a shortening warning.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-08-24

  • runtime/Collector.cpp:

(JSC::Heap::sweep):

9:09 PM Changeset in webkit [47739] by ggaren@apple.com
  • 3 edits in trunk/JavaScriptCore

Substantially reduced VM thrash in the GC heap.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-08-24
Reviewed by Oliver Hunt.

1.08x faster on v8 (1.60x faster on v8-splay).

1.40x faster on bench-alloc-nonretained.

1.90x faster on bench-alloc-retained.

SunSpider says no change.

  • runtime/Collector.cpp:

(JSC::Heap::heapAllocate): Fixed a long-standing bug: update a few local
variables unconditionally after calling collect(), since they may be used
even if we don't "goto scan". (In the bug I saw, usedBlocks got out of
sync with heap.usedBlocks).
(JSC::Heap::sweep): Keep enough free heap space to accomodate
the number of objects we'll allocate before the next GC, plus 25%, for
good measure.

  • runtime/Collector.h: Bumped the block size to 256k. This seems to give

the best cache performance, and it prevents us from initiating lots of
VM traffic to recover very small chunks of memory.

7:53 PM Changeset in webkit [47738] by barraclough@apple.com
  • 19 edits in trunk/JavaScriptCore

https://bugs.webkit.org/show_bug.cgi?id=28691
Do not retain ScopeNodes outside of parsing

Reviewed by Oliver Adler & Darin Hunt.

There is now no need for these to exist outside of parsing - their use in the runtime is replaced by Executable types.

  • bytecode/EvalCodeCache.h:

(JSC::EvalCodeCache::get):

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator):
(JSC::BytecodeGenerator::emitNewFunction):
(JSC::BytecodeGenerator::emitNewFunctionExpression):

  • bytecompiler/BytecodeGenerator.h:

(JSC::BytecodeGenerator::makeFunction):

  • debugger/Debugger.cpp:

(JSC::Debugger::recompileAllJSFunctions):
(JSC::evaluateInGlobalCallFrame):

  • debugger/DebuggerCallFrame.cpp:

(JSC::DebuggerCallFrame::evaluate):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::execute):
(JSC::Interpreter::prepareForRepeatCall):
(JSC::Interpreter::privateExecute):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • parser/Nodes.cpp:

(JSC::ScopeNodeData::ScopeNodeData):
(JSC::ProgramNode::create):
(JSC::EvalNode::create):
(JSC::FunctionBodyNode::create):

  • parser/Nodes.h:

(JSC::ScopeNode::adoptData):
(JSC::FunctionBodyNode::parameterCount):

  • parser/Parser.cpp:
  • parser/Parser.h:

(JSC::Parser::arena):
(JSC::Parser::Parser):
(JSC::Parser::parse):

  • runtime/ArrayPrototype.cpp:

(JSC::isNumericCompareFunction):
(JSC::arrayProtoFuncSort):

  • runtime/Completion.cpp:

(JSC::checkSyntax):
(JSC::evaluate):

  • runtime/Executable.cpp:

(JSC::FunctionExecutable::~FunctionExecutable):
(JSC::EvalExecutable::compile):
(JSC::ProgramExecutable::checkSyntax):
(JSC::ProgramExecutable::compile):
(JSC::FunctionExecutable::compile):
(JSC::EvalExecutable::generateJITCode):
(JSC::ProgramExecutable::generateJITCode):
(JSC::FunctionExecutable::generateJITCode):
(JSC::FunctionExecutable::reparseExceptionInfo):
(JSC::EvalExecutable::reparseExceptionInfo):
(JSC::FunctionExecutable::recompile):
(JSC::FunctionExecutable::fromGlobalCode):
(JSC::FunctionExecutable::copyParameters):
(JSC::FunctionExecutable::paramString):

  • runtime/Executable.h:

(JSC::ScriptExecutable::ScriptExecutable):
(JSC::ScriptExecutable::sourceID):
(JSC::ScriptExecutable::sourceURL):
(JSC::ScriptExecutable::lineNo):
(JSC::ScriptExecutable::lastLine):
(JSC::ScriptExecutable::usesEval):
(JSC::ScriptExecutable::usesArguments):
(JSC::ScriptExecutable::needsActivation):
(JSC::ScriptExecutable::recordParse):
(JSC::EvalExecutable::bytecode):
(JSC::EvalExecutable::jitCode):
(JSC::ProgramExecutable::bytecode):
(JSC::ProgramExecutable::reparseExceptionInfo):
(JSC::ProgramExecutable::jitCode):
(JSC::FunctionExecutable::FunctionExecutable):
(JSC::FunctionExecutable::make):
(JSC::FunctionExecutable::bytecode):
(JSC::FunctionExecutable::isGenerated):
(JSC::FunctionExecutable::name):
(JSC::FunctionExecutable::parameterCount):
(JSC::FunctionExecutable::jitCode):

  • runtime/FunctionConstructor.cpp:

(JSC::constructFunction):

  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::numericCompareFunction):

  • runtime/JSGlobalObjectFunctions.cpp:

(JSC::globalFuncEval):

5:50 PM Changeset in webkit [47737] by Antti Koivisto
  • 2 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=28698
Avoid unnecessary transformations for the window resizer rect.

Reviewed by Simon Fraser.

  • platform/Scrollbar.cpp:

(WebCore::Scrollbar::setFrameRect):

5:42 PM Changeset in webkit [47736] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Anders Carlsson.

<rdar://problem/7161900> REGRESSION: 100 MB membuster leak on windows due to 47391
(DNS prefetching support)

  • platform/network/cf/DNSCFNet.cpp: (WebCore::prefetchDNS): Use a run loop from secondary thread on Windows, because there isn't one on main one.
5:09 PM Changeset in webkit [47735] by ap@apple.com
  • 3 edits
    3 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=28374
Cross-scheme requests should not be blocked by appcache rules

Test: http/tests/appcache/different-scheme.html

  • loader/appcache/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::shouldLoadResourceFromApplicationCache): Resources with different schemes can never be loaded from appcache.
5:03 PM Changeset in webkit [47734] by ap@apple.com
  • 6 edits
    2 adds in trunk

Reviewed by Darin Adler and Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=28688
REGRESSION(r24994): Cannot create a frame with a javascript URL

Test: http/tests/security/javascriptURL/javascriptURL-in-new-iframe.html

  • bindings/js/JSAttrCustom.cpp: (WebCore::JSAttr::setValue):
  • bindings/js/JSElementCustom.cpp: (WebCore::allowSettingSrcToJavascriptURL):
  • bindings/js/JSHTMLFrameElementCustom.cpp: (WebCore::allowSettingJavascriptURL):
  • bindings/js/JSHTMLIFrameElementCustom.cpp: (WebCore::JSHTMLIFrameElement::setSrc): Omit the checks if there is no document in frame yet.
3:58 PM Changeset in webkit [47733] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-24 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Skip the correct test (media/video-currentTime-set.html, not
media/video-currentTime-set2.html).

  • platform/gtk/Skipped:
3:47 PM Changeset in webkit [47732] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-24 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Skip 3 new tests, and reenable some of the media tests, now that
the bot it fixed to handle the fact that it doesn't have a sound
card.

  • platform/gtk/Skipped:
2:56 PM Changeset in webkit [47731] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-24 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Unreviewed. Skip again tests that we enabled, since they are
failing on the bot. The tests do pass, but there is an environment
configuration problem with GStreamer on the bot (possibly related
to the fact that it has no sound card).

  • platform/gtk/Skipped:
2:37 PM Changeset in webkit [47730] by Darin Adler
  • 2 edits in trunk/JavaScriptCore
  • runtime/ObjectPrototype.cpp:

(JSC::ObjectPrototype::put): Landed revised version I had tested but forgot
to land. Leave out the branch, since we don't need one.

2:21 PM Changeset in webkit [47729] by Nate Chapin
  • 2 edits in trunk/WebCore

2009-08-24 Nate Chapin <Nate Chapin>

Reviewed by Eric Seidel.

Fix crash in fast/css/rem-dynamic-scaling.html due to a freed
RenderStyle being accessed.

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

  • dom/Element.cpp: (WebCore::Element::recalcStyle): Make currentStyle a RefPtr.
2:08 PM Changeset in webkit [47728] by adachan@apple.com
  • 3 edits in trunk/LayoutTests

Reviewed by Mark Rowe.


Skip loader/go-back-to-different-window-size.html for qt and gtk.

  • platform/gtk/Skipped:
  • platform/qt/Skipped:
2:05 PM Changeset in webkit [47727] by Darin Adler
  • 3 edits in trunk/JavaScriptCore

Array index miss case creates a string every time
https://bugs.webkit.org/show_bug.cgi?id=28664

Patch by Darin Adler <Darin Adler> on 2009-08-24
Reviewed by Geoff Garen.

SunSpider test results I saw:

0.5% faster overall
1% faster on crypto-aes
20% faster on crypto-md5
13% faster on crypto-sha1

  • runtime/ObjectPrototype.cpp:

(JSC::ObjectPrototype::ObjectPrototype): Initialize m_hasNoPropertiesWithUInt32Names
to true.
(JSC::ObjectPrototype::put): Clearly m_hasNoPropertiesWithUInt32Names if the new
property has a name that is the string form of a UInt32.
(JSC::ObjectPrototype::getOwnPropertySlot): Don't call JSObject::getOwnPropertySlot
if m_hasNoPropertiesWithUInt32Names is true, and it is highly likely to be true.

  • runtime/ObjectPrototype.h: Added declarations for the above.
1:45 PM Changeset in webkit [47726] by mitz@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/5743105> Long freezing during loading and selecting text
of a large table
https://bugs.webkit.org/show_bug.cgi?id=14918

Reviewed by Simon Fraser.

  • rendering/RenderView.cpp:

(WebCore::RenderView::setSelection): Added calls to

beginDeferredRepaints() and endDeferredRepaints() around the
invalidation of the selection, in order to better coalesce multiple
invalid rectangles.

1:26 PM Changeset in webkit [47725] by pfeldman@chromium.org
  • 5 edits in trunk/WebCore

2009-08-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Simplify findTreeElement code, get rid of potential infinite loop.

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

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.updateBreadcrumb):
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline.prototype.findTreeElement):
  • inspector/front-end/treeoutline.js: (TreeOutline.prototype.getCachedTreeElement): (TreeOutline.prototype.findTreeElement):
  • inspector/front-end/utilities.js: (isAncestorNode): ():
1:20 PM Changeset in webkit [47724] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2009-08-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Do expand local scope when hitting the break

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

  • inspector/front-end/ScopeChainSidebarPane.js: (WebInspector.ScopeChainSidebarPane.prototype.update):
1:16 PM Changeset in webkit [47723] by adachan@apple.com
  • 1 edit
    3 adds in trunk/LayoutTests

Reviewed by Brady Eidson.

Add test for https://bugs.webkit.org/show_bug.cgi?id=28659.
Cached page should be formatted according to the current window size, not the cached frame's size.

  • loader: Added.
  • loader/go-back-to-different-window-size-expected.txt: Added.
  • loader/go-back-to-different-window-size.html: Added.
12:55 PM Changeset in webkit [47722] by adachan@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Steve Falkenburg.

Make sure we use the current FrameView's frame rect when
going back to a cached page.


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

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::open):
12:10 PM Changeset in webkit [47721] by eric@webkit.org
  • 2 edits
    3 adds in trunk/LayoutTests

2009-08-24 Eric Seidel <eric@webkit.org>

No review, just updating results.

Support placeholder on textarea
https://bugs.webkit.org/show_bug.cgi?id=21248

Update expected test results form windows bot.

  • platform/win/fast/forms/basic-textareas-expected.txt:
  • platform/win/fast/forms/textarea-placeholder-pseudo-style-expected.txt: Added.
  • platform/win/fast/forms/textarea-placeholder-set-attribute-expected.txt: Added.
  • platform/win/fast/forms/textarea-placeholder-set-value-expected.txt: Added.
11:35 AM Changeset in webkit [47720] by eric@webkit.org
  • 3 edits
    3 adds in trunk

2009-08-24 Cameron McCormack <cam@mcc.id.au>

Reviewed by Darin Adler.

Modifying <text rotate=""> doesn't clear the corresponding SVGAnimatedNumberList
https://bugs.webkit.org/show_bug.cgi?id=28673

  • svg/dom/resources/text-rotate-live.js: Added. (getRotate): (getAndSetRotate):
  • svg/dom/text-rotate-live-expected.txt: Added.
  • svg/dom/text-rotate-live.html: Added.

2009-08-24 Cameron McCormack <cam@mcc.id.au>

Reviewed by Darin Adler.

Modifying <text rotate=""> doesn't clear the corresponding SVGAnimatedNumberList
https://bugs.webkit.org/show_bug.cgi?id=28673

Test: svg/dom/text-rotate-live.html

  • svg/SVGNumberList.cpp: (WebCore::SVGNumberList::parse): Clear the list before adding the parsed numbers.
11:07 AM Changeset in webkit [47719] by Simon Fraser
  • 4 edits in trunk

2009-08-24 Simon Fraser <Simon Fraser>

Reviewed by NOBODY (build fix)

Turn off ENABLE_3D_CANVAS in the xconfig files.

  • Configurations/FeatureDefines.xcconfig:
11:02 AM Changeset in webkit [47718] by kov@webkit.org
  • 1 copy in releases/WebKitGTK/webkit-1.1.13

Tagging WebKitGTK+ 1.1.13

9:40 AM Changeset in webkit [47717] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2009-08-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Only expand local scope when hitting the break

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

  • inspector/front-end/ScopeChainSidebarPane.js: (WebInspector.ScopeChainSidebarPane.prototype.update):
9:26 AM Changeset in webkit [47716] by mitz@apple.com
  • 6 edits in trunk

WebCore:
background-size fails to parse if a single length/percentage/auto is followed by a comma
https://bugs.webkit.org/show_bug.cgi?id=28674

Reviewed by Simon Fraser.

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseFillSize): Added an allowComma parameter,

which is set to false when this method consumes a comma. Moved
handling of 'contain' and 'cover' from parseFillProperty into this
method.

(WebCore::CSSParser::parseFillProperty): Set allowComma to true before

processing the next value, and pass allowComma to parseFillSize.

  • css/CSSParser.h:

LayoutTests:
background-size fails to parse if a single length/percentage/auto is followed by a comma
https://bugs.webkit.org/show_bug.cgi?id=28674

Reviewed by Simon Fraser.

  • fast/backgrounds/size/parsing-background-size-values-expected.txt:
  • fast/backgrounds/size/resources/parsing-background-size-values.js:
9:02 AM Changeset in webkit [47715] by kov@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-08-24 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Unreviewed. Fix a typo in my distcheck build fix.

  • GNUmakefile.am:
8:41 AM Changeset in webkit [47714] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-08-24 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

News about the 1.1.13 release.

  • NEWS:
8:28 AM Changeset in webkit [47713] by kov@webkit.org
  • 5 edits in trunk

2009-08-24 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Bump package and so versions for 1.1.13 release.

  • configure.ac:

WebKit/gtk

2009-08-24 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Add new symbols, and an index of new symbols for the 1.1.13
release.

  • docs/webkitgtk-docs.sgml:
  • docs/webkitgtk-sections.txt:
7:55 AM Changeset in webkit [47712] by kov@webkit.org
  • 2 edits
    3 adds in trunk/WebCore

2009-08-17 Martin Robinson <martin.james.robinson@gmail.com>

Reviewed by Gustavo Noronha.

[GTK] Support for cursors that are images
https://bugs.webkit.org/show_bug.cgi?id=28346

Added support for cursors which are images for CursorGtk. Added
a manual test for this.

  • manual-tests/gtk/cursor-image.html: Added.
  • manual-tests/gtk/resources/redcursor.cur: Added.
  • platform/gtk/CursorGtk.cpp: (WebCore::Cursor::Cursor):
7:48 AM Changeset in webkit [47711] by kenneth@webkit.org
  • 1 edit
    6 adds in trunk/LayoutTests

Unreviewed. Add missing files from former commit 47709.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-08-24

  • platform/qt/fast/backgrounds/svg-as-background-1-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-2-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-3-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-4-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-5-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-6-expected.txt: Added.
7:30 AM Changeset in webkit [47710] by kov@webkit.org
  • 6 edits in trunk

WebCore:

2009-08-24 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

[GTK] Integrate GStreamer video with the graphics backend
https://bugs.webkit.org/show_bug.cgi?id=16356

Improves GStreamer backend implementation of MediaPlayerPrivate class.

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::mediaPlayerPrivateErrorCallback): Send proper MediaPlayer error for different GStreamer errors to MediaPlayerPrivate. (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): Move static initialization boolean to a better place. (WebCore::MediaPlayerPrivate::play): (WebCore::MediaPlayerPrivate::pause): (WebCore::MediaPlayerPrivate::duration): Return if error occured, do not consider the source as stream if duration query failed. (WebCore::MediaPlayerPrivate::currentTime): Return if error occured, use an early return. (WebCore::MediaPlayerPrivate::seek): Return if error occured. (WebCore::MediaPlayerPrivate::setEndTime): Remove implementation since this function doesn't seem to be used anywhere in WebCore. (WebCore::MediaPlayerPrivate::paused): (WebCore::MediaPlayerPrivate::seeking): (WebCore::MediaPlayerPrivate::setRate): Seek to current time when rate is set. (WebCore::MediaPlayerPrivate::maxTimeBuffered): Return if error occured. (WebCore::MediaPlayerPrivate::maxTimeSeekable): Ditto. (WebCore::MediaPlayerPrivate::maxTimeLoaded): Ditto. (WebCore::MediaPlayerPrivate::bytesLoaded): (WebCore::MediaPlayerPrivate::totalBytesKnown): (WebCore::MediaPlayerPrivate::totalBytes): Ditto. (WebCore::MediaPlayerPrivate::updateStates): Return if error occured, update after seek ended with a successful change (WebCore::MediaPlayerPrivate::didEnd): Do not pause the playbin when end is reached. (WebCore::MediaPlayerPrivate::loadingFailed): Update network state with the given error. (WebCore::mimeTypeCache): Gather supported mime types from GStreamer. (WebCore::MediaPlayerPrivate::getSupportedTypes): (WebCore::MediaPlayerPrivate::supportsType): (WebCore::MediaPlayerPrivate::hasSingleSecurityOrigin): (WebCore::MediaPlayerPrivate::supportsFullscreen): First step towards fullscreen support. (WebCore::MediaPlayerPrivate::createGSTPlayBin): Use playbin2 instead of playbin, do not set playbin's audio sink.
  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.h:
  • platform/graphics/gtk/VideoSinkGStreamer.cpp: (webkit_video_sink_idle_func): Use C++ casts, decrease reference count of async queue at the end of the idle function. (webkit_video_sink_render): Increase reference count of async queue. (webkit_video_sink_stop): Remove any idle functions with the sink as data.

LayoutTests:

2009-08-24 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

[GTK] Integrate GStreamer video with the graphics backend
https://bugs.webkit.org/show_bug.cgi?id=16356

Enable media tests that now pass.

  • platform/gtk/Skipped:
7:26 AM Changeset in webkit [47709] by kenneth@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Add results with updated font metrics for passing tests added in
r28637 and remove them from Skipped.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-08-24
Reviewed by Ariya Hidayat.

  • platform/qt/Skipped:
  • platform/qt/fast/backgrounds/svg-as-background-1-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-2-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-3-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-4-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-5-expected.txt: Added.
  • platform/qt/fast/backgrounds/svg-as-background-6-expected.txt: Added.
6:38 AM Changeset in webkit [47708] by cmarrin@apple.com
  • 2 edits
    6 adds in trunk/WebCore

Remaining new files for Canvas3D
https://bugs.webkit.org/show_bug.cgi?id=28018

4:58 AM Changeset in webkit [47707] by ariya@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

2009-08-24 Andras Becsi <becsi.andras@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Add results for new tests added in r47590 and r47702.

  • platform/qt/fast/forms/textarea-placeholder-pseudo-style-expected.txt: Added.
  • platform/qt/fast/forms/textarea-placeholder-set-attribute-expected.txt: Added.
  • platform/qt/fast/forms/textarea-placeholder-set-value-expected.txt: Added.
  • platform/qt/fast/repaint/inline-block-resize-expected.txt: Added.
4:05 AM Changeset in webkit [47706] by ariya@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-24 Ariya Hidayat <ariya.hidayat@nokia.com>

Rubber-stamped by Simon Hausmann.

[Qt] Skip new test added in r47630

  • platform/qt/Skipped: Skip fast/backgrounds/size/parsing-inherit.html.
3:18 AM Changeset in webkit [47705] by ariya@webkit.org
  • 5 edits in trunk/LayoutTests

2009-08-24 Renata Hodovan <hodovan.renata@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Update the outdated metrics for doctype tests. All tests pass.

  • platform/qt/Skipped:
  • platform/qt/fast/doctypes/001-expected.txt:
  • platform/qt/fast/doctypes/002-expected.txt:
  • platform/qt/fast/doctypes/004-expected.txt:
3:09 AM Changeset in webkit [47704] by ariya@webkit.org
  • 1 edit
    19 adds in trunk/LayoutTests

2009-08-24 Andras Becsi <becsi.andras@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Add results for new tests added in r47678.

  • platform/qt/fast/block/margin-collapse/block-inside-inline: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/001-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/002-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/003-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/004-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/005-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/006-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/010-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/011-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/012-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/015-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/016-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/017-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/018-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/019-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/020-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/021-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/022-expected.txt: Added.
  • platform/qt/fast/block/margin-collapse/block-inside-inline/025-expected.txt: Added.
3:09 AM Changeset in webkit [47703] by pfeldman@chromium.org
  • 11 edits in trunk/WebCore

2009-08-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Migrate call frames interaction to the InjectedScript-based schema.

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

  • inspector/front-end/CallStackSidebarPane.js: (WebInspector.CallStackSidebarPane.prototype.update):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype.doEvalInWindow): (WebInspector.ConsoleTextMessage):
  • inspector/front-end/DOMAgent.js: (WebInspector.DOMAgent.prototype.get domWindow): (InspectorController.openInInspectedWindow): (InspectorController.getCallFrames): (InspectorController.evaluateInCallFrame):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.reset): (WebInspector.ElementsPanel.prototype.generateStylesheet):
  • inspector/front-end/InjectedScript.js: (InjectedScript.evaluate): (InjectedScript._evaluateOn): (InjectedScript.openInInspectedWindow): (InjectedScript.getCallFrames): (InjectedScript.evaluateInCallFrame): (InjectedScript._callFrameForId): (InjectedScript._objectForId): (InjectedScript.CallFrameProxy): (InjectedScript.CallFrameProxy.prototype._wrapScopeChain):
  • inspector/front-end/ObjectPropertiesSection.js: (WebInspector.ObjectPropertiesSection.prototype._update):
  • inspector/front-end/ObjectProxy.js: (WebInspector.ObjectPropertyProxy):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourceSidebarTreeElement.prototype.ondblclick):
  • inspector/front-end/ScopeChainSidebarPane.js: (WebInspector.ScopeChainSidebarPane): (WebInspector.ScopeChainSidebarPane.prototype.update): (WebInspector.ScopeVariableTreeElement.prototype.onattach): (WebInspector.ScopeVariableTreeElement.prototype.onexpand): (WebInspector.ScopeVariableTreeElement.prototype.oncollapse):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.doEvalInCallFrame): (WebInspector.ScriptsPanel.prototype.variablesInSelectedCallFrame): (WebInspector.ScriptsPanel.prototype.debuggerPaused.callback): (WebInspector.ScriptsPanel.prototype.debuggerPaused):
1:28 AM Changeset in webkit [47702] by eric@webkit.org
  • 21 edits
    9 adds in trunk

2009-08-24 Kent Tamura <tkent@chromium.org>

Reviewed by Eric Seidel.

  • fast/forms/resources/textarea-placeholder-dom-property.js: Added.
  • fast/forms/textarea-placeholder-dom-property-expected.txt: Added.
  • fast/forms/textarea-placeholder-dom-property.html: Added.
  • fast/forms/textarea-placeholder-pseudo-style.html: Added.
  • fast/forms/textarea-placeholder-set-attribute.html: Added.
  • fast/forms/textarea-placeholder-set-value.html: Added.
  • platform/mac-leopard/fast/forms/input-text-maxlength-expected.txt:
  • platform/mac/fast/forms/input-text-maxlength-expected.txt:
  • platform/mac/fast/forms/textarea-placeholder-pseudo-style-expected.txt: Added.
  • platform/mac/fast/forms/textarea-placeholder-set-attribute-expected.txt: Added.
  • platform/mac/fast/forms/textarea-placeholder-set-value-expected.txt: Added.
  • platform/qt/fast/forms/input-text-maxlength-expected.txt:
  • platform/win/fast/forms/input-text-maxlength-expected.txt:

2009-08-24 Kent Tamura <tkent@chromium.org>

Reviewed by Eric Seidel.

Add support for the placeholder attribute and DOM property of the textarea element.
https://bugs.webkit.org/show_bug.cgi?id=21248

A DOM node for an INPUT element doesn't have a flag for placeholder
visibility anymore. This patch also fixes a bug that a renderer
doesn't reflect a corresponding DOM value change.

Tests: fast/forms/textarea-placeholder-dom-property.html

fast/forms/textarea-placeholder-pseudo-style.html
fast/forms/textarea-placeholder-set-attribute.html
fast/forms/textarea-placeholder-set-value.html

  • css/html.css: Add the default style for placeholder of textarea.
  • dom/InputElement.cpp: Cleanup for m_placeholderShouldBeVisible removal (WebCore::InputElement::dispatchFocusEvent): (WebCore::InputElement::dispatchBlurEvent): (WebCore::InputElement::placeholderShouldBeVisible): (WebCore::InputElement::updatePlaceholderVisibility): (WebCore::InputElement::setValueFromRenderer): (WebCore::InputElementData::InputElementData):
  • dom/InputElement.h:
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::dispatchFocusEvent): (WebCore::HTMLInputElement::dispatchBlurEvent): (WebCore::HTMLInputElement::setValue): (WebCore::HTMLInputElement::placeholderShouldBeVisible):
  • html/HTMLInputElement.h: (WebCore::HTMLInputElement::updatePlaceholderVisibility):
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::parseMappedAttribute): (WebCore::HTMLTextAreaElement::setValue): (WebCore::HTMLTextAreaElement::placeholderShouldBeVisible): (WebCore::HTMLTextAreaElement::updatePlaceholderVisibility): (WebCore::HTMLTextAreaElement::dispatchFocusEvent): (WebCore::HTMLTextAreaElement::dispatchBlurEvent):
  • html/HTMLTextAreaElement.h:
  • html/HTMLTextAreaElement.idl:
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::RenderTextControl): (WebCore::RenderTextControl::styleDidChange): (WebCore::RenderTextControl::setInnerTextStyle): (WebCore::RenderTextControl::updatePlaceholderVisibility):
  • rendering/RenderTextControl.h:
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::RenderTextControlMultiLine): (WebCore::RenderTextControlMultiLine::nodeAtPoint): (WebCore::RenderTextControlMultiLine::updateFromElement): (WebCore::RenderTextControlMultiLine::createInnerTextStyle): (WebCore::RenderTextControlMultiLine::textBaseStyle):
  • rendering/RenderTextControlMultiLine.h:
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::RenderTextControlSingleLine): (WebCore::RenderTextControlSingleLine::textBaseStyle): (WebCore::RenderTextControlSingleLine::updateFromElement): (WebCore::RenderTextControlSingleLine::createInnerTextStyle):
  • rendering/RenderTextControlSingleLine.h:
  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::dispatchFocusEvent): (WebCore::WMLInputElement::dispatchBlurEvent): (WebCore::WMLInputElement::setValue): (WebCore::WMLInputElement::placeholderShouldBeVisible):
12:14 AM Changeset in webkit [47701] by zoltan@webkit.org
  • 1 edit in trunk/WebKitTools/Scripts/modules/committers.py

Unreviewed.

Change my mail address on the list of committers.

12:02 AM Changeset in webkit [47700] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-23 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Fix regression in editing tests by changing expected file to the
Mac one, since we now default to Mac in the EditingBehavior
setting.

  • platform/gtk/editing/execCommand/toggle-compound-styles-expected.txt:
12:01 AM Changeset in webkit [47699] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-08-23 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Fix a couple of typos.

  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init):

Aug 23, 2009:

11:00 PM Changeset in webkit [47698] by krit@webkit.org
  • 6 edits in trunk/WebCore

2009-08-23 Dirk Schulze <krit@webkit.org>

Reviewed by Mark Rowe.

Code clean-up and build fix for filters enabled builds. Moved SVGNames.h
into #if ENABLE(SVG) for Document.cpp and added SVGNames.h to the other
files. They are needed there after r47688.

  • dom/Document.cpp:
  • svg/SVGComponentTransferFunctionElement.h:
  • svg/SVGFELightElement.h:
  • svg/SVGFEMergeNodeElement.h:
  • svg/SVGFilterPrimitiveStandardAttributes.h:
8:44 PM Changeset in webkit [47697] by kov@webkit.org
  • 8 edits in trunk

WebCore:

2009-08-23 Gustavo Noronha Silva <Gustavo Noronha Silva>

Unreviewed, build fix for make distcheck on GTK+.

  • GNUmakefile.am: Added files needed for the build.
  • bindings/js/JSDOMWindowCustom.cpp: Protect SharedWorker include with #if clause.
  • bindings/js/JSEventTarget.cpp: Dito.
  • dom/Document.cpp: Dito.
  • loader/FrameLoader.cpp: Dito.

JavaScriptCore:

2009-08-23 Gustavo Noronha Silva <Gustavo Noronha Silva>

Unreviewed build fix for make distcheck.

  • GNUmakefile.am: Added files required for the build.
2:42 PM Changeset in webkit [47696] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-08-23 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Inspector: Throws an Error on "null"
https://bugs.webkit.org/show_bug.cgi?id=28665

  • inspector/front-end/InjectedScript.js: (InjectedScript.evaluate):
12:49 PM Changeset in webkit [47695] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2009-08-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Inspector: REGRESSION Formatting Function in Console is Abbreviated Too Often

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

  • inspector/front-end/InjectedScript.js: (InjectedScript.getProperties): (InjectedScript.createProxyObject):
12:24 PM Changeset in webkit [47694] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-08-23 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Update editing-behavior property here too.

  • webkit/webkitwebview.cpp: (webkit_web_view_settings_notify):
10:04 AM Changeset in webkit [47693] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-08-23 Jan Michael Alonzo <jmalonzo@webkit.org>

Rubberstamped by Gustavo Noronha.

Fix regressions introduced by http://trac.webkit.org/changeset/47690.

Update the setting whenever the property gets set.

  • webkit/webkitwebview.cpp: (webkit_web_view_settings_notify):
3:08 AM Changeset in webkit [47692] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKitTools

2009-08-23 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Xan Lopez.

[Gtk] API for disabling local file access to web URLs
https://bugs.webkit.org/show_bug.cgi?id=28663

Enable this setting for DRT.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetWebViewToConsistentStateBeforeTesting):
2:05 AM Changeset in webkit [47691] by jmalonzo@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-23 Jan Michael Alonzo <jmalonzo@webkit.org>

Not reviewed. Skip new tests from
http://trac.webkit.org/changeset/47678 and the appcache HTTP
authentication tests (https://bugs.webkit.org/show_bug.cgi?id=28668)

  • platform/gtk/Skipped:
1:45 AM Changeset in webkit [47690] by jmalonzo@webkit.org
  • 3 edits in trunk/WebKit/gtk

2009-08-23 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Xan Lopez.

[Gtk] API for disabling local file access to web URLs
https://bugs.webkit.org/show_bug.cgi?id=28663

Add a setting to enable or disable file access to web URLs. It's
disabled by default for security reasons.

  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init): (webkit_web_settings_set_property): (webkit_web_settings_get_property): (webkit_web_settings_copy):
  • webkit/webkitwebview.cpp: (webkit_web_view_update_settings):
12:18 AM Changeset in webkit [47689] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-08-22 Martin Robinson <martin.james.robinson@gmail.com>

Reviewed by Xan Lopez.

[GTK] BitmapImage::getGdkPixbuf does not handle alpha channels properly
https://bugs.webkit.org/show_bug.cgi?id=28345

When doing the conversion between cairo_surface_t* and GdkPixbuf*
account for the differences in the respective formats' in-memory
image format.

  • platform/graphics/gtk/ImageGtk.cpp: (WebCore::getCairoSurfacePixel): (WebCore::getGdkPixbufPixel): (WebCore::BitmapImage::getGdkPixbuf):

Aug 22, 2009:

11:55 PM Changeset in webkit [47688] by Darin Adler
  • 43 edits in trunk/WebCore

Make DOM classes start with a reference count of 1, like all other RefCounted

Patch by Darin Adler <Darin Adler> on 2009-08-22
Reviewed by Oliver Hunt.

Next step: Element, HTMLElement, HTMLAnchorElement, HTMLAppletElement,
HTMLAreaElement, HTMLEmbedElement, HTMLFrameElement, HTMLIFrameElement,
HTMLObjectElement, HTMLPlugInElement, HTMLPlugInImageElement.

  • DerivedSources.make: Fix error seen every time we build by escaping the $

in the build rule, since you have to use $$ for that in make.

  • WebCore.base.exp: Updated.
  • bindings/objc/DOM.mm:

Added now-needed include of SVGNames.h.

  • dom/Document.cpp: Added now-needed include of SVGNames.h.

(WebCore::Document::createElement): Use create instead of new.

  • dom/Element.cpp:

(WebCore::Element::Element): Added ConstructionType argument so the caller
can determine whether this starts with a reference count of either 0 or 1.
Later we will remove this again once they are all 1.
(WebCore::Element::create): Added.

  • dom/Element.h: Added create and made constructor protected.
  • dom/StyledElement.cpp:

(WebCore::StyledElement::StyledElement): Added ConstructionType argument.

  • dom/StyledElement.h: Made many functions protected or private, including

the constructor.

  • dom/make_names.pl: Use create to create the base elements HTMLElement,

SVGElement, and WMLElement.

  • editing/CreateLinkCommand.cpp:

(WebCore::CreateLinkCommand::doApply): Use create instead of new.

  • editing/UnlinkCommand.cpp:

(WebCore::UnlinkCommand::doApply): Ditto.

  • html/HTMLAnchorElement.cpp:

(WebCore::HTMLAnchorElement::create): Added.

  • html/HTMLAnchorElement.h: Made constructor protected and added create.

Also made m_rootEditableElementForSelectionOnMouseDown a RefPtr.

  • html/HTMLAppletElement.cpp:

(WebCore::HTMLAppletElement::create): Added.

  • html/HTMLAppletElement.h: Made constructor private and added create.
  • html/HTMLAreaElement.cpp:

(WebCore::HTMLAreaElement::create): Added.

  • html/HTMLAreaElement.h: Made constructor private and added create.
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::create): Added.

  • html/HTMLElement.h: Made constructor protected and added create.

Made other functions protected and private. Also added inline
definition of constructor.

  • html/HTMLEmbedElement.cpp:

(WebCore::HTMLEmbedElement::create): Added.

  • html/HTMLEmbedElement.h: Made constructor private and added create.

Made other functions private.

  • html/HTMLFrameElement.cpp:

(WebCore::HTMLFrameElement::create): Added.
(WebCore::HTMLFrameElement::parseMappedAttribute):

  • html/HTMLFrameElement.h: Made constructor private and added create.

Made other functions private.

  • html/HTMLFrameElementBase.h: Made functions protected and private.
  • html/HTMLFrameOwnerElement.cpp:

(WebCore::HTMLFrameOwnerElement::HTMLFrameOwnerElement): Pass
CreateElement so reference count starts at 1.

  • html/HTMLFrameOwnerElement.h: Made functions protected and private.
  • html/HTMLIFrameElement.cpp:

(WebCore::HTMLIFrameElement::create): Added.

  • html/HTMLIFrameElement.h: Made constructor private and added create.

Made other functions private.

  • html/HTMLObjectElement.cpp:

(WebCore::HTMLObjectElement::create): Added.
(WebCore::HTMLObjectElement::containsJavaApplet): Changed while loop
into a for loop and tweaked formatting.

  • html/HTMLObjectElement.h: Made constructor private and added create.

Made other functions private.

  • html/HTMLPlugInElement.h: Made constructor protected.

Made other functions protected and private.

  • html/HTMLPlugInImageElement.h: Made constructor and another function

protected.

  • html/HTMLTagNames.in: Removed createWithNew from all classes that

now have create functions. Moved conditional noscript to a separate
paragraph.

  • html/HTMLViewSourceDocument.cpp:

(WebCore::HTMLViewSourceDocument::addSpanWithClassName): Use create.
(WebCore::HTMLViewSourceDocument::addLink): Ditto.

  • page/DragController.cpp:

(WebCore::documentFragmentFromDragData): Ditto.

  • svg/SVGElement.cpp:

(WebCore::SVGElement::SVGElement): Pass CreateElementZeroRefCount
so reference count still starts at 0. Will change this class later.
(WebCore::SVGElement::create): Added.

  • svg/SVGElement.h: Added create, made constructor protected, and

made other functions protected and private. Removed unneeded
SVGNames.h include.

  • svg/SVGExternalResourcesRequired.h: Added now-needed SVGNames.h

include.

  • svg/SVGStopElement.h: Added now-needed SVGNames.h include.

Made functions private.

  • wml/WMLElement.cpp:

(WebCore::WMLElement::WMLElement): Pass CreateElementZeroRefCount
so reference count still starts at 0. Will change this class later.
(WebCore::WMLElement::create): Added.

  • wml/WMLElement.h: Added create and made constructor protected.
  • wml/WMLTagNames.in: Removed createWithNew from WMLElement.
11:36 PM Changeset in webkit [47687] by xan@webkit.org
  • 6 edits in trunk/WebKit/gtk

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

Reviewed by Jan Alonzo.

https://bugs.webkit.org/show_bug.cgi?id=28154
clicking below end of text in a textarea moves cursor to wrong position

Expose the EditingBehavior setting and set it by default to Mac,
since that what matches the GTK+ default behavior.

  • webkit/webkitprivate.cpp: (WebKit::core):
  • webkit/webkitprivate.h:
  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init): (webkit_web_settings_set_property): (webkit_web_settings_get_property): (webkit_web_settings_copy):
  • webkit/webkitwebsettings.h:
  • webkit/webkitwebview.cpp: (webkit_web_view_update_settings):
10:04 PM Changeset in webkit [47686] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

REGRESSION(r47639-r47660): Webkit crashes on launch on PowerPC
https://bugs.webkit.org/show_bug.cgi?id=28655

Reviewed by Mark Rowe.

  • runtime/JSFunction.cpp:

(JSC::JSFunction::JSFunction): Initialize properly with a VPtrHackExecutable.

  • wtf/Platform.h:
9:43 PM Changeset in webkit [47685] by abarth@webkit.org
  • 11 edits in trunk

WebKit/mac: Revert 47684. We're going to do this later once clients have had a
chance to opt into the setting they like.

Patch by Adam Barth <abarth@webkit.org> on 2009-08-22

  • Misc/WebKitVersionChecks.h:
  • WebView/WebPreferences.mm:

(+[WebPreferences initialize]):

WebKit/qt: Revert 47684. We're going to do this later once clients have had a
chance to opt into the setting they like.

Patch by Adam Barth <abarth@webkit.org> on 2009-08-22

  • Api/qwebsettings.cpp:

(QWebSettings::QWebSettings):

WebKit/win: Revert 47684. We're going to do this later once clients have had a
chance to opt into the setting they like.

Patch by Adam Barth <abarth@webkit.org> on 2009-08-22

  • WebPreferences.cpp:

(WebPreferences::initializeDefaultSettings):

WebKitTools: Revert 47684. We're going to do this later once clients have had a
chance to opt into the setting they like.

Patch by Adam Barth <abarth@webkit.org> on 2009-08-22

  • DumpRenderTree/mac/DumpRenderTree.mm:

(resetDefaultsToConsistentValues):

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::WebPage::WebPage):

  • DumpRenderTree/win/DumpRenderTree.cpp:

(resetDefaultsToConsistentValues):

9:24 PM Changeset in webkit [47684] by abarth@webkit.org
  • 11 edits in trunk

2009-08-22 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Don't let local files access web URLs
https://bugs.webkit.org/show_bug.cgi?id=28480

  • Misc/WebKitVersionChecks.m:
  • WebView/WebPreferences.mm: (+[WebPreferences initialize]):

2009-08-22 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Don't let local files access web URLs
https://bugs.webkit.org/show_bug.cgi?id=28480

  • Api/qwebsettings.cpp: (QWebSettings::QWebSettings):

2009-08-22 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Don't let local files access web URLs
https://bugs.webkit.org/show_bug.cgi?id=28480

  • WebPreferences.cpp: (WebPreferences::initializeDefaultSettings):

2009-08-22 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Don't let local files access web URLs
https://bugs.webkit.org/show_bug.cgi?id=28480

A bunch of our LayoutTests rely on our old behavior, so we explicitly
grant local files universal access during testing. Mainly, these tests
involve making XMLHttpRequests for data URLs.

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
  • DumpRenderTree/qt/DumpRenderTree.cpp: (WebCore::WebPage::WebPage):
  • DumpRenderTree/win/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
5:48 PM Changeset in webkit [47683] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

5:47 PM Changeset in webkit [47682] by mrowe@apple.com
  • 1 copy in tags/Safari-6531.16

New tag.

5:47 PM Changeset in webkit [47681] by mrowe@apple.com
  • 3 edits in branches/safari-4-branch/WebCore

Merge r47679.

5:47 PM Changeset in webkit [47680] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebCore

Merge r47663.

5:43 PM Changeset in webkit [47679] by adele@apple.com
  • 3 edits in trunk/WebCore

Fix for <rdar://problem/7161656> Crash in RenderObject::destroy when using custom scrollbars

Reviewed by Anders Carlsson.

Custom scrollbars normally get detached when a document gets detached. In this case, a crash was happening when the document had gone into the page cache
and then was getting destroyed without proper scrollbar teardown. This change makes custom scrollbars get detached when the document enters the cache.

I couldn't figure out a way to reliably cause this to happen in DRT, since the reproducible case relies on the cache being purged at a particular time.

  • history/CachedFrame.cpp: (WebCore::CachedFrame::CachedFrame): Detach custom scrollbars when the CachedFrame is created. They'll get reattached when the document comes out of the page cache.
  • page/FrameView.cpp: (WebCore::FrameView::~FrameView): Added some asserts to catch this problem earlier in debug builds.
4:26 PM Changeset in webkit [47678] by hyatt@apple.com
  • 4 edits
    74 adds in trunk

WebCore: Make sure self-collapsing blocks that clear floats are still able to collapse their bottom margins with
the bottom of their parent blocks. The old code prevented all collapsing, but all CSS2.1 states is that:
"An element that has had clearance applied to it never collapses its top margin with its parent block's bottom margin."

Reviewed by Dan Bernstein.

The implication is that the bottom margin can still collapse, and the block-inside-inline form of
fast/block/margin-collapse/025.html (added as fast/block/margin-collapse/block-inside-inline/025.html) depends
on it to get comparable results.

Added block-inside-inline tests (a whole slew of them) in fast/block/margin-collapse/block-inside-inline/

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::MarginInfo::MarginInfo):
(WebCore::RenderBlock::collapseMargins):
(WebCore::RenderBlock::clearFloatsIfNeeded):
(WebCore::RenderBlock::handleBottomOfBlock):

  • rendering/RenderBlock.h:

(WebCore::RenderBlock::MarginInfo::clearMargin):
(WebCore::RenderBlock::MarginInfo::canCollapseBottomWithChildren):

LayoutTests: Add a bunch of tests for the block-inside-inline cases of margin-collapsing. The same test
suite but converted to wrap all the blocks inside inlines to make sure they still collapse
the same way.

Reviewed by Dan Bernstein.

  • fast/block/margin-collapse/block-inside-inline: Added.
  • fast/block/margin-collapse/block-inside-inline/001.html: Added.
  • fast/block/margin-collapse/block-inside-inline/002.html: Added.
  • fast/block/margin-collapse/block-inside-inline/003.html: Added.
  • fast/block/margin-collapse/block-inside-inline/004.html: Added.
  • fast/block/margin-collapse/block-inside-inline/005.html: Added.
  • fast/block/margin-collapse/block-inside-inline/006.html: Added.
  • fast/block/margin-collapse/block-inside-inline/010.html: Added.
  • fast/block/margin-collapse/block-inside-inline/011.html: Added.
  • fast/block/margin-collapse/block-inside-inline/012.html: Added.
  • fast/block/margin-collapse/block-inside-inline/015.html: Added.
  • fast/block/margin-collapse/block-inside-inline/016.html: Added.
  • fast/block/margin-collapse/block-inside-inline/017.html: Added.
  • fast/block/margin-collapse/block-inside-inline/018.html: Added.
  • fast/block/margin-collapse/block-inside-inline/019.html: Added.
  • fast/block/margin-collapse/block-inside-inline/020.html: Added.
  • fast/block/margin-collapse/block-inside-inline/021.html: Added.
  • fast/block/margin-collapse/block-inside-inline/022.html: Added.
  • fast/block/margin-collapse/block-inside-inline/025.html: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/001-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/001-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/001-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/002-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/002-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/002-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/003-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/003-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/003-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/004-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/004-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/004-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/005-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/005-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/005-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/006-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/006-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/006-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/010-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/010-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/010-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/011-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/011-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/011-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/012-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/012-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/012-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/015-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/015-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/015-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/016-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/016-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/016-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/017-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/017-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/017-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/018-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/018-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/018-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/019-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/019-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/019-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/020-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/020-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/020-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/021-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/021-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/021-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/022-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/022-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/022-expected.txt: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/025-expected.checksum: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/025-expected.png: Added.
  • platform/mac/fast/block/margin-collapse/block-inside-inline/025-expected.txt: Added.
3:56 PM Changeset in webkit [47677] by mrowe@apple.com
  • 4 edits
    1 add in trunk/WebKitTools

Bring signed updates to the Mac nightly builds.

Rubber-stamped by Anders Carlsson.

  • WebKitLauncher/Info.plist:
  • WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj:
  • WebKitLauncher/WebKitNightlyEnablerSparkle.m:

(initializeSparkle):

  • WebKitLauncher/nightly.webkit.org.public.pem: Added.
10:54 AM Changeset in webkit [47676] by Chris Fleizach
  • 1 edit
    1 copy
    1 move in trunk/LayoutTests

Need to implement ARIA role="listitem" and role="list"
https://bugs.webkit.org/show_bug.cgi?id=28606

Fixing LayoutTest bustage.

The SnowLeopard results are the latest, so they need to move to platform/mac
The Tiger/Leopard results will default to the SnowLeopard results, so we need to move
the current test from platform/mac to platform/mac-leopard

  • platform/mac-leopard/accessibility/aria-roles-expected.txt: Copied from LayoutTests/platform/mac/accessibility/aria-roles-expected.txt.
  • platform/mac-snowleopard/accessibility/aria-roles-expected.txt: Removed.
  • platform/mac/accessibility/aria-roles-expected.txt: Replaced with LayoutTests/platform/mac-snowleopard/accessibility/aria-roles-expected.txt.
10:03 AM Changeset in webkit [47675] by Chris Fleizach
  • 5 edits
    4 adds in trunk

WebCore: Need to implement ARIA role="listitem" and role="list"
https://bugs.webkit.org/show_bug.cgi?id=28606

Reviewed by Darin Adler.

Test: platform/mac-snowleopard/accessibility/aria-list-and-listitem.html

  • accessibility/AXObjectCache.cpp:

(WebCore::AXObjectCache::getOrCreate):

  • accessibility/AccessibilityList.cpp:

(WebCore::AccessibilityList::isUnorderedList):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::RoleEntry::):

LayoutTests: Need to implement ARIA role="listitem" and role="list"
https://bugs.webkit.org/show_bug.cgi?id=28606

Reviewed by Darin Adler.

Also added a SnowLeopard specific result for aria-roles-expected.

  • platform/mac-snowleopard/accessibility: Added.
  • platform/mac-snowleopard/accessibility/aria-list-and-listitem-expected.txt: Added.
  • platform/mac-snowleopard/accessibility/aria-list-and-listitem.html: Added.
  • platform/mac-snowleopard/accessibility/aria-roles-expected.txt: Added.
9:32 AM Changeset in webkit [47674] by mitz@apple.com
  • 3 edits in trunk/WebCore

Clean up fill image geometry calculation
https://bugs.webkit.org/show_bug.cgi?id=28652

Reviewed by Darin Adler.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::calculateFillTileSize): Renamed

calculateBackgroundSize() to this. Replaced separate scaledWidth and
scaledHeight parameters with a single scaledSize parameter. Renamed
parameters and local variables.

(WebCore::RenderBoxModelObject::calculateBackgroundImageGeometry): Changed

to use CSS3 Backgrounds and Borders terminology in comments and
local variable names. Renamed, clarified and consolidated some local
variables.

  • rendering/RenderBoxModelObject.h: Made calculateFillTileSize() private.
9:16 AM Changeset in webkit [47673] by cmarrin@apple.com
  • 3 edits in trunk/WebCore

Backing out http://trac.webkit.org/changeset/47669

9:10 AM Changeset in webkit [47672] by Dimitri Glazkov
  • 7 edits in trunk/WebCore

2009-08-22 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, build fix.

Backing out http://trac.webkit.org/changeset/47671 in preparation of
rolling out http://trac.webkit.org/changeset/47669.

8:13 AM Changeset in webkit [47671] by Dimitri Glazkov
  • 7 edits in trunk/WebCore

2009-08-22 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Anders Carlsson.

Build fix, add CanvasRenderingContext to build systems.
https://bugs.webkit.org/show_bug.cgi?id=28018

  • DerivedSources.make: Added CanvasRenderinContext.
  • GNUmakefile.am: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • WebCoreSources.bkl: Ditto.
6:11 AM Changeset in webkit [47670] by cmarrin@apple.com
  • 6 edits
    18 adds
    1 delete in trunk/WebCore

New files implementing GL buffer wrappers, and support to make them platform independent
https://bugs.webkit.org/show_bug.cgi?id=28018

5:56 AM Changeset in webkit [47669] by cmarrin@apple.com
  • 2 edits in trunk/WebCore/html/canvas

stash CanvasRenderingContext2D

2:02 AM Changeset in webkit [47668] by Darin Adler
  • 2 edits in trunk/JavaScriptCore

Fix storage leak from syntax tree arena allocation patch.

Patch by Darin Adler <Darin Adler> on 2009-08-22

  • parser/Nodes.h: CommaNode needs to inherit from ParserArenaDeletable

because it has a vector.

Aug 21, 2009:

11:54 PM Changeset in webkit [47667] by Darin Adler
  • 2 edits in trunk/JavaScriptCore

Fix Qt build.

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

  • parser/Nodes.cpp:

(JSC::ScopeNodeData::ScopeNodeData): Made non-inline again.
This is used outside Nodes.cpp so can't be inline unless
it is in the header.

11:53 PM Changeset in webkit [47666] by Darin Adler
  • 3 edits in trunk/WebCore
  • WebCore.vcproj/build-generated-files.sh:
  • make-generated-sources.sh:

Removed obsolete code to set up CREATE_HASH_TABLE.

11:52 PM Changeset in webkit [47665] by Darin Adler
  • 3 edits in trunk/JavaScriptCore

Two loose ends from the last commit.

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

and create_hash_table project-internal instead of "private".

  • runtime/Executable.h: Removed accidentally-added constructor.
11:40 PM Changeset in webkit [47664] by Darin Adler
  • 19 edits in trunk/JavaScriptCore

Syntax tree nodes should use arena allocation
https://bugs.webkit.org/show_bug.cgi?id=25674

Patch by Darin Adler <Darin Adler> on 2009-08-21
Reviewed by Gavin Barraclough.

Use an actual arena now. 0.6% speedup on SunSpider.

New and improved with 100% less leaking of the universe.

Removed all exports involving the class FunctionBodyNode, which no
longer needs to be used outside JavaScriptCore.

Executable.h project-internal instead of "private".

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator): Updated since VarStack
contains const Identifier* now.

  • parser/Grammar.y: Made identifiers from the lexer be const

Identifier* and updated since VarStack contains const Identifier* now.

  • parser/Lexer.cpp:

(JSC::Lexer::setCode): Pass in ParserArena, used for identifiers.
(JSC::Lexer::makeIdentifier): Changed return type to const Identifier*
and changed to call ParserArena.
(JSC::Lexer::clear): Removed the code to manage m_identifiers and
added code to set m_arena to 0.

  • parser/Lexer.h: Updated for changes above.
  • parser/NodeConstructors.h:

(JSC::ParserArenaFreeable::operator new): Added. Calls allocateFreeable
on the arena.
(JSC::ParserArenaDeletable::operator new): Changed to call the
allocateDeletable function on the arena instead of deleteWithArena.
(JSC::PropertyNode::PropertyNode): Added new constructor that makes
numeric identifiers. Some day we might want to optimize this for
integers so it doesn't create a string for each one.
(JSC::ContinueNode::ContinueNode): Initialize m_ident to nullIdentifier
since it's now a const Identifier& so it can't be left uninitialized.
(JSC::BreakNode::BreakNode): Ditto.
(JSC::CaseClauseNode::CaseClauseNode): Updated to use SourceElements*
to keep track of the statements rather than a separate statement vector.
(JSC::BlockNode::BlockNode): Ditto.
(JSC::ForInNode::ForInNode): Initialize m_ident to nullIdentifier.

  • parser/Nodes.cpp: Moved the comment explaining emitBytecode in here.

It seemed strangely out of place in the header.
(JSC::ThrowableExpressionData::emitThrowError): Added an overload for
UString as well as Identifier.
(JSC::SourceElements::singleStatement): Added.
(JSC::SourceElements::lastStatement): Added.
(JSC::RegExpNode::emitBytecode): Changed the throwError code to use
the substitution mechanism instead of doing a string append.
(JSC::SourceElements::emitBytecode): Added. Replaces the old
statementListEmitCode function, since we now keep the SourceElements
objects around.
(JSC::BlockNode::lastStatement): Added.
(JSC::BlockNode::emitBytecode): Changed to use emitBytecode instead of
statementListEmitCode.
(JSC::CaseClauseNode::emitBytecode): Added.
(JSC::CaseBlockNode::emitBytecodeForBlock): Changed to use emitBytecode
instead of statementListEmitCode.
(JSC::ScopeNodeData::ScopeNodeData): Changed to store the
SourceElements* instead of using releaseContentsIntoVector.
(JSC::ScopeNode::emitStatementsBytecode): Added.
(JSC::ScopeNode::singleStatement): Added.
(JSC::ProgramNode::emitBytecode): Call emitStatementsBytecode instead
of statementListEmitCode.
(JSC::EvalNode::emitBytecode): Ditto.
(JSC::FunctionBodyNode::emitBytecode): Call emitStatementsBytecode
insetad of statementListEmitCode and check for the return node using
the new functions.

  • parser/Nodes.h: Changed VarStack to store const Identifier* instead

of Identifier and rely on the arena to control lifetime. Added a new
ParserArenaFreeable class. Made ParserArenaDeletable inherit from
FastAllocBase instead of having its own operator new. Base the Node
class on ParserArenaFreeable. Changed the various Node classes
to use const Identifier& instead of Identifier to avoid the need to
call their destructors and allow them to function as "freeable" in the
arena. Removed extraneous JSC_FAST_CALL on definitions of inline functions.
Changed ElementNode, PropertyNode, ArgumentsNode, ParameterNode,
CaseClauseNode, ClauseListNode, and CaseBlockNode to use ParserArenaFreeable
as a base class since they do not descend from Node. Eliminated the
StatementVector type and instead have various classes use SourceElements*
instead of StatementVector. This prevents those classes from having to
use ParserArenaDeletable to make sure the vector destructor is called.

  • parser/Parser.cpp:

(JSC::Parser::parse): Pass the arena to the lexer.

  • parser/Parser.h: Added an include of ParserArena.h, which is no longer

included by Nodes.h.
(JSC::Parser::parseFunctionFromGlobalCode): Changed to use the
singleStatement function, since there is no longer any children function.
Removed some unneeded use of RefPtr.

  • parser/ParserArena.cpp:

(JSC::ParserArena::ParserArena): Added. Initializes the new members,
m_freeableMemory, m_freeablePoolEnd, and m_identifiers.
(JSC::ParserArena::freeablePool): Added. Computes the pool pointer,
since we store only the current pointer and the end of pool pointer.
(JSC::ParserArena::deallocateObjects): Added. Contains the common
memory-deallocation logic used by both the destructor and the
reset function.
(JSC::ParserArena::~ParserArena): Changed to call deallocateObjects.
(JSC::ParserArena::reset): Ditto. Also added code to zero out the
new structures, and switched to use clear() instead of shrink(0) since
we don't really reuse arenas.
(JSC::ParserArena::makeNumericIdentifier): Added.
(JSC::ParserArena::allocateFreeablePool): Added. Used when the pool
is empty.
(JSC::ParserArena::isEmpty): Added. No longer inline, which is fine
since this is used only for assertions at the moment.
(JSC::ParserArena::derefWithArena): Make non-inline.

  • parser/ParserArena.h: Added an actual arena of "freeable" objects,

ones that don't need destructors to be called. Also added a separate
IdentifierArena object, a segmented vector of identifiers that used
to be in the Lexer.

  • runtime/Executable.h: Moved the definition of the

FunctionExecutable::make function here. It can't go in JSFunction.h
since that header has to be used outside JavaScriptCore and so can't
include this, which includes Nodes.h. The function could be moved
elswhere if we don't want to include JSFunction.h in this header, but
for now this seems to be the best place.

  • runtime/JSFunction.h: Removed the include of Executable.h and

definition of the FunctionExecutable::make function.

  • wtf/FastMalloc.cpp: Fixed an incorrect comment.
10:24 PM Changeset in webkit [47663] by adele@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/7162322> Custom style sheet ignored if UAC is enabled

Reviewed by Mark Rowe.

  • platform/win/SharedBufferWin.cpp: (WebCore::SharedBuffer::createWithContentsOfFile):

No need to open the file with write access. Only read access is needed.

10:10 PM Changeset in webkit [47662] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

<rdar://problem/7161454> Leaks seen during http/tests/xmlhttprequest/cross-origin-no-authorization.html

Also fixes leaks seen on appcache tests.

  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandle::cancel): Work around something that's likely a CFNetwork issue. Note that on Windows, there is no matching API to call, and we do not think that the leak occurs there anyway.
8:51 PM Changeset in webkit [47661] by eric@webkit.org
  • 2 edits in trunk

2009-08-22 Javier Jardón <javierjc1982@gmail.com>

[GTK] Not use deprecated symbols in WebKitWebView example code.
https://bugs.webkit.org/show_bug.cgi?id=28651

  • WebKit/gtk/webkit/webkitwebview.cpp
7:31 PM Changeset in webkit [47660] by mrowe@apple.com
  • 3 edits in trunk/JavaScriptCore

Fix the non-JIT build.

  • runtime/Executable.cpp:
  • runtime/Executable.h:
7:06 PM Changeset in webkit [47659] by dimich@chromium.org
  • 3 edits in trunk/WebCore

Remove cross-thread code from CrossOriginPreflightResultCache.
Removed mutexes, atomic initializer and string copy.
https://bugs.webkit.org/show_bug.cgi?id=28650

Reviewed by David Levin.

No new tests, the worker tests cover this.

  • loader/CrossOriginPreflightResultCache.cpp:

(WebCore::addToAccessControlAllowList):
(WebCore::CrossOriginPreflightResultCache::shared):
(WebCore::CrossOriginPreflightResultCache::appendEntry):
(WebCore::CrossOriginPreflightResultCache::canSkipPreflight):
(WebCore::CrossOriginPreflightResultCache::empty):

  • loader/CrossOriginPreflightResultCache.h:
7:02 PM Changeset in webkit [47658] by jmalonzo@webkit.org
  • 2 edits in trunk/WebCore

2009-08-21 Michelangelo De Simone <micdesim@gmail.com>

Reviewed by Jan Alonzo.

Buildfix after http://trac.webkit.org/changeset/47655.

  • html/HTMLFormControlElement.h:
6:55 PM Changeset in webkit [47657] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

<rdar://problem/7162480>
Sometimes WebKit does not layout correctly when a WebView is embedded in an HICocoaView

Reviewed by Darin Adler.

Add a null check for the current context when reflectScrolledClipView: is called from outside a
draw operation.

  • WebView/WebDynamicScrollBarsView.mm:

(-[WebDynamicScrollBarsView reflectScrolledClipView:]):

6:53 PM Changeset in webkit [47656] by Joseph Pecoraro
  • 18 edits in trunk/WebCore

2009-08-19 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Inspector: Improve Cookie DataGrid to Show Hidden Data
https://bugs.webkit.org/show_bug.cgi?id=28269

Removed Custom Bindings

  • bindings/js/JSInspectorBackendCustom.cpp:
  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8InspectorBackendCustom.cpp:
  • inspector/InspectorBackend.cpp:

Made Non-Custom Bindings in the Backend

(WebCore::InspectorBackend::getCookies):
(WebCore::InspectorBackend::deleteCookie):

  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::deleteCookie):
  • inspector/InspectorController.h:

Build the Cookie ScriptObjects, handles using document.cookie in
case the platform hasn't implemented raw cookie access.

  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getCookies): (WebCore::InspectorDOMAgent::buildObjectForCookie): (WebCore::InspectorDOMAgent::buildArrayForCookies):
  • inspector/InspectorDOMAgent.h:

Complete the Asynchronous Calls

  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::didGetCookies):
  • inspector/InspectorFrontend.h:

Asynchronous Functions to get Cookie Data

  • inspector/front-end/DOMAgent.js: (WebInspector.Cookies.getCookiesAsync): (WebInspector.Cookies.buildCookiesFromString): fallback behavior

Refactor to use the Asynchronous Functions

  • inspector/front-end/CookieItemsView.js: (WebInspector.CookieItemsView): (WebInspector.CookieItemsView.prototype.get statusBarItems): (WebInspector.CookieItemsView.prototype.update.callback): (WebInspector.CookieItemsView.prototype.update): (WebInspector.CookieItemsView.prototype.simpleDataGridForCookies): (WebInspector.CookieItemsView.prototype._deleteButtonClicked):

Cleaned/Commented Related Code

(InspectorController.searchCanceled):

  • inspector/front-end/InjectedScript.js:
  • platform/Cookie.h:
  • English.lproj/localizedStrings.js: the new strings that were supposed to have gone in last time
6:16 PM Changeset in webkit [47655] by pkasting@chromium.org
  • 11 edits
    6 adds in trunk

WebCore: https://bugs.webkit.org/show_bug.cgi?id=28145
Add support for novalidate/formnovalidate attribute, from HTML5 specs:
http://www.whatwg.org/specs/web-apps/current-work/#attr-fs-novalidate

Patch by Michelangelo De Simone <micdesim@gmail.com> on 2009-08-21
Reviewed by Adele Peterson.

Tests: fast/forms/formnovalidate-attribute.html

fast/forms/novalidate-attribute.html

  • html/HTMLAttributeNames.in: added novalidate, formnovalidate
  • html/HTMLButtonElement.idl: formnovalidate attribute exposed
  • html/HTMLFormControlElement.cpp:

(WebCore::HTMLFormControlElement::formNoValidate): attribute getter
(WebCore::HTMLFormControlElement::setFormNoValidate): attribute setter

  • html/HTMLFormControlElement.h:
  • html/HTMLFormElement.cpp:

(WebCore::HTMLFormElement::novalidate): attribute getter
(WebCore::HTMLFormElement::setNovalidate): attribute setter

  • html/HTMLFormElement.h:
  • html/HTMLFormElement.idl: novalidate attribute exposed
  • html/HTMLInputElement.idl: formNoValidate attribute exposed

LayoutTests: https://bugs.webkit.org/show_bug.cgi?id=28145
Tests for novalidate/formnovalidate attribute, from HTML5 specs:
http://www.whatwg.org/specs/web-apps/current-work/#attr-fs-novalidate

Patch by Michelangelo De Simone <micdesim@gmail.com> on 2009-08-21
Reviewed by Adele Peterson.

  • fast/dom/domListEnumeration-expected.txt:
  • fast/dom/resources/domListEnumeration.js:
  • fast/forms/formnovalidate-attribute-expected.txt: Added.
  • fast/forms/formnovalidate-attribute.html: Added.
  • fast/forms/novalidate-attribute-expected.txt: Added.
  • fast/forms/novalidate-attribute.html: Added.
  • fast/forms/resources/formnovalidate-attribute.js: Added.
  • fast/forms/resources/novalidate-attribute.js: Added.
6:13 PM Changeset in webkit [47654] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKitTools

2009-08-21 Jan Michael Alonzo <jmalonzo@webkit.org>

Rubberstamped by Simon Fraser.

Remove GNOME keyring support in build-webkit. This dependency's
already been removed in the Gtk port.

  • Scripts/build-webkit:
5:59 PM Changeset in webkit [47653] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Speculative QuickTime build fix.

Reviewed by NOBODY (build fix).

  • runtime/JSArray.cpp:
5:44 PM Changeset in webkit [47652] by mitz@apple.com
  • 1 edit in trunk/LayoutTests/fast/backgrounds/size/contain-and-cover-expected.png

Set the PNG's svn:mime-type property to image/png

5:41 PM Changeset in webkit [47651] by mitz@apple.com
  • 5 edits in trunk/LayoutTests

Rubber-stamped by Anders Carlsson.

Tweaked the test to make everything fit in a 800-by-600 view.

  • fast/backgrounds/size/contain-and-cover-expected.checksum:
  • fast/backgrounds/size/contain-and-cover-expected.png:
  • fast/backgrounds/size/contain-and-cover-expected.txt:
  • fast/backgrounds/size/contain-and-cover.html:
5:33 PM Changeset in webkit [47650] by mitz@apple.com
  • 4 edits
    5 adds in trunk

WebCore:
Rendering changes to complete
[CSS3 Backgrounds and Borders] Add support for the "contain" value for
background-size
https://bugs.webkit.org/show_bug.cgi?id=27573
and
[CSS3 Backgrounds and Borders] Add support for the "cover" value for
background-size
https://bugs.webkit.org/show_bug.cgi?id=27574

Reviewed by Beth Dakin.

Test: fast/backgrounds/size/contain-and-cover.html

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::calculateBackgroundSize):

  • rendering/RenderObject.cpp:

(WebCore::mustRepaintFillLayers):

LayoutTests:
Rendering tests for
[CSS3 Backgrounds and Borders] Add support for the "contain" value for
background-size
https://bugs.webkit.org/show_bug.cgi?id=27573
and
[CSS3 Backgrounds and Borders] Add support for the "cover" value for
background-size
https://bugs.webkit.org/show_bug.cgi?id=27574

Reviewed by Beth Dakin.

  • fast/backgrounds/size/contain-and-cover-expected.checksum: Added.
  • fast/backgrounds/size/contain-and-cover-expected.png: Added.
  • fast/backgrounds/size/contain-and-cover-expected.txt: Added.
  • fast/backgrounds/size/contain-and-cover.html: Added.
  • fast/backgrounds/size/resources/SquirrelFish.svg: Added!!!
4:55 PM Changeset in webkit [47649] by pkasting@chromium.org
  • 25 edits
    18 adds in trunk

WebCore: https://bugs.webkit.org/show_bug.cgi?id=27452
Add support for checkValidity() method and invalid event, from HTML5
specs:
http://www.whatwg.org/specs/web-apps/current-work/#dom-form-checkvalidity

Patch by Michelangelo De Simone <micdesim@gmail.com> on 2009-08-21
Reviewed by Adele Peterson.

Tests: fast/events/invalid-001.html

fast/events/invalid-002.html
fast/events/invalid-003.html
fast/events/invalid-004.html
fast/events/invalid-005.html
fast/forms/checkValidity-001.html
fast/forms/checkValidity-002.html
fast/forms/checkValidity-003.html
fast/forms/checkValidity-004.html

  • dom/Document.idl: oninvalid event handler
  • dom/Element.idl: ditto
  • dom/EventNames.h: added invalid event
  • dom/Node.cpp:

(WebCore::Node::oninvalid): ditto
(WebCore::Node::setOninvalid): ditto

  • dom/Node.h: ditto
  • html/HTMLAttributeNames.in: oninvalid attribute
  • html/HTMLButtonElement.idl: added checkValidity() method
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::parseMappedAttribute): parses oninvalid

  • html/HTMLFieldSetElement.idl: added checkValidity() method
  • html/HTMLFormControlElement.cpp:

(WebCore::HTMLFormControlElement::checkValidity): checkValidity()
implementation

  • html/HTMLFormControlElement.h:
  • html/HTMLFormElement.cpp:

(WebCore::HTMLFormElement::checkValidity): checkValidity() implementation
for form elements

  • html/HTMLFormElement.h: checkValidity() definition
  • html/HTMLFormElement.idl: added checkValidity() method
  • html/HTMLInputElement.idl: added checkValidity() method
  • html/HTMLSelectElement.idl: added checkValidity() method
  • html/HTMLTextAreaElement.idl: added checkValidity() method
  • page/DOMWindow.cpp:

(WebCore::DOMWindow::oninvalid): oninvalid event handler
(WebCore::DOMWindow::setOninvalid): ditto

  • page/DOMWindow.h: ditto
  • page/DOMWindow.idl: ditto

LayoutTests: https://bugs.webkit.org/show_bug.cgi?id=27452
Tests for checkValidity() method and invalid event, from HTML5
specs:
http://www.whatwg.org/specs/web-apps/current-work/#dom-form-checkvalidity

Patch by Michelangelo De Simone <micdesim@gmail.com> on 2009-08-21
Reviewed by Adele Peterson.

  • fast/events/invalid-001-expected.txt: Added.
  • fast/events/invalid-001.html: Added.
  • fast/events/invalid-002-expected.txt: Added.
  • fast/events/invalid-002.html: Added.
  • fast/events/invalid-003-expected.txt: Added.
  • fast/events/invalid-003.html: Added.
  • fast/events/invalid-004-expected.txt: Added.
  • fast/events/invalid-004.html: Added.
  • fast/events/invalid-005-expected.txt: Added.
  • fast/events/invalid-005.html: Added.
  • fast/forms/checkValidity-001-expected.txt: Added.
  • fast/forms/checkValidity-001.html: Added.
  • fast/forms/checkValidity-002-expected.txt: Added.
  • fast/forms/checkValidity-002.html: Added.
  • fast/forms/checkValidity-003-expected.txt: Added.
  • fast/forms/checkValidity-003.html: Added.
  • fast/forms/checkValidity-004-expected.txt: Added.
  • fast/forms/checkValidity-004.html: Added.
  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/domListEnumeration-expected.txt:
  • fast/dom/resources/domListEnumeration.js:
4:30 PM Changeset in webkit [47648] by Beth Dakin
  • 7 edits
    3 adds in trunk

WebCore: The CSS part of https://bugs.webkit.org/show_bug.cgi?id=27571 [CSS3
Backgrounds and Borders] Add support for the "round" value for
background-repeat
-and-
https://bugs.webkit.org/show_bug.cgi?id=27570 [CSS3 Backgrounds and
Borders] Add support for the "space" value for background-repeat

Reviewed by Dan Bernstein.

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseFillProperty):

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
(WebCore::CSSPrimitiveValue::operator EFillRepeat):

  • css/CSSValueKeywords.in:
  • rendering/style/FillLayer.h:
  • rendering/style/RenderStyleConstants.h:

(WebCore::):

LayoutTests: Test for the CSS part of https://bugs.webkit.org/show_bug.cgi?id=27571 [CSS3 Backgrounds and Borders] Add support for the "round" value
for background-repeat
-and-
https://bugs.webkit.org/show_bug.cgi?id=27570 [CSS3 Backgrounds and
Borders] Add support for the "space" value for background-repeat

Reviewed by Dan Bernstein.

  • fast/backgrounds/repeat/parsing-background-repeat-expected.txt: Added.
  • fast/backgrounds/repeat/parsing-background-repeat.html: Added.
  • fast/backgrounds/repeat/resources/parsing-background-repeat.js: Added.

(test):

3:57 PM Changeset in webkit [47647] by eric@webkit.org
  • 1 edit
    3 adds in trunk/LayoutTests

2009-08-21 Christian Plesner Hansen <christian.plesner.hansen@gmail.com>

Reviewed by Eric Seidel.

Regression test for bug fixed in
http://codereview.chromium.org/171039. Regexps created in one
frame were not callable outside that frame.
https://bugs.webkit.org/show_bug.cgi?id=28387

  • fast/regex/cross-frame-callable-expected.txt: Added.
  • fast/regex/cross-frame-callable.html: Added.
  • fast/regex/resources/cross-frame-callable.js: Added. (doTest):
3:46 PM Changeset in webkit [47646] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Speculative QT build fix.

Reviewed by NOBODY (build fix).

  • runtime/StringPrototype.cpp:
3:45 PM Changeset in webkit [47645] by cmarrin@apple.com
  • 1 edit
    14 adds in trunk/WebCore

New files for Canvas 3D feature
https://bugs.webkit.org/show_bug.cgi?id=28018

This is all the new files for implementing Canvas 3D. None of them are hooked up and
are not even included in the build yet. This incorporates the reviews from Simon
and Ollie for these files.

3:28 PM Changeset in webkit [47644] by rniwa@webkit.org
  • 2 edits in trunk/WebCore

No Review.

Patch by Ryosuke Niwa <rniwa@webkit.org> on 2009-08-21
Another Windows build fix.

  • editing/ApplyStyleCommand.cpp:

(WebCore::diffTextDecorations):

3:19 PM Changeset in webkit [47643] by hyatt@apple.com
  • 3 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=28625, focus rings don't respect the non-strict mode
line box shrinking quirk. Make sure outlines don't extend outside the lineTop and lineBottom
of the root line box.

Reviewed by Simon Fraser.

Covered by existing tests.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::addFocusRingRects):

  • rendering/RenderInline.cpp:

(WebCore::RenderInline::addFocusRingRects):
(WebCore::RenderInline::paintOutline):

3:08 PM Changeset in webkit [47642] by rniwa@webkit.org
  • 2 edits in trunk/WebCore

No Review.

Patch by Ryosuke Niwa <rniwa@webkit.org> on 2009-08-21
Windows build fix.

  • editing/ApplyStyleCommand.cpp:

(WebCore::setTextDecorationProperty): Made it static
(WebCore::diffTextDecorations): Made it static

2:54 PM Changeset in webkit [47641] by barraclough@apple.com
  • 15 edits in trunk/JavaScriptCore

Restructure Executable types so that host functions do not hold a FunctionExecutable.
https://bugs.webkit.org/show_bug.cgi?id=28621

Reviewed by Oliver Hunt.

All JSFunction objects have a pointer to an Executable*. This is currently always a
FunctionExecutable, however this has a couple of drawbacks. Host functions do not
store a range of information that the FunctionExecutable provides (source, name,
CodeBlock & information presently held on the FunctionBodyNode).

[ * nearly all... see below! ]

Instead, make JSFunctions hold a pointer to an ExecutableBase, move fields specific
to JS sourced executable types (source, node) into a new subclass (ScriptExecutable),
and create a new NativeExecutable type. We now provide a new method in JSFunction
to access & downcast to FunctionExecutable, but in doing so we can make an early
check (with an ASSERT) to ensure that the Executable read from a function will only
be treated as a FunctionExecutable (and thus the JS sepcific fields will only be
accessed) if the JSFunction is not a host function.

There is one JSFunction that currently does not have an Executable, which is the
object created to allow us to read out the vtable pointer. By making this change
we can also add a new Executable type fror this object (VPtrHackExecutable).
Since this means that really all JSFunctions have an Executable we no longer have
to null-check m_executable before us it - particularly in isHostFunction().

This patch removes CacheableEvalExecutable, since all subclasses of ExecutableBase
can now be ref-counted - since both JSFunction holds (and ref-counts) an ExecutableBase
that might be a FunctionExecutable or a NativeExecutable. This does now mean that all
ProgramExecutables and EvalExecutables (unnecessarily) provide an interface to be
ref-counted, however this seems less-bad than host functions unnecessarily providing
interface to access non-host specific information.

The class hierarcy has changed from this:

  • ExecutableBase
    • ProgramExecutable
    • EvalExecutable
      • CacheableEvalExecutable (also RefCounted by multiple-inheritance)
    • FunctionExecutable (also RefCounted by multiple-inheritance, 'special' FunctionExecutable also used for host functions)

To this:

  • RefCounted
    • ExecutableBase
      • NativeExecutable
      • VPtrHackExecutable
      • ScriptExecutable
        • ProgramExecutable
        • EvalExecutable
        • FunctionExecutable

This patch speeds up sunspidey by a couple of ms (presumably due to the changes to isHostFunction()).

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::CodeBlock):

  • bytecode/CodeBlock.h:

(JSC::CodeBlock::ownerExecutable):
(JSC::GlobalCodeBlock::GlobalCodeBlock):

  • bytecode/EvalCodeCache.h:

(JSC::EvalCodeCache::get):

  • debugger/Debugger.cpp:

(JSC::Debugger::recompileAllJSFunctions):

  • interpreter/CachedCall.h:

(JSC::CachedCall::CachedCall):

  • interpreter/Interpreter.cpp:

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

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • profiler/Profiler.cpp:

(JSC::createCallIdentifierFromFunctionImp):

  • runtime/Arguments.h:

(JSC::Arguments::getArgumentsData):
(JSC::Arguments::Arguments):

  • runtime/Executable.cpp:

(JSC::NativeExecutable::~NativeExecutable):
(JSC::VPtrHackExecutable::~VPtrHackExecutable):

  • runtime/Executable.h:

(JSC::ExecutableBase::ExecutableBase):
(JSC::ExecutableBase::~ExecutableBase):
(JSC::ExecutableBase::isHostFunction):
(JSC::NativeExecutable::NativeExecutable):
(JSC::VPtrHackExecutable::VPtrHackExecutable):
(JSC::ScriptExecutable::ScriptExecutable):
(JSC::ScriptExecutable::source):
(JSC::ScriptExecutable::sourceID):
(JSC::ScriptExecutable::sourceURL):
(JSC::ScriptExecutable::lineNo):
(JSC::ScriptExecutable::lastLine):
(JSC::ScriptExecutable::usesEval):
(JSC::ScriptExecutable::usesArguments):
(JSC::ScriptExecutable::needsActivation):
(JSC::EvalExecutable::EvalExecutable):
(JSC::EvalExecutable::create):
(JSC::ProgramExecutable::ProgramExecutable):
(JSC::FunctionExecutable::FunctionExecutable):

  • runtime/FunctionPrototype.cpp:

(JSC::functionProtoFuncToString):

  • runtime/JSFunction.cpp:

(JSC::JSFunction::JSFunction):
(JSC::JSFunction::~JSFunction):
(JSC::JSFunction::markChildren):
(JSC::JSFunction::getCallData):
(JSC::JSFunction::call):
(JSC::JSFunction::lengthGetter):
(JSC::JSFunction::getConstructData):
(JSC::JSFunction::construct):

  • runtime/JSFunction.h:

(JSC::JSFunction::executable):
(JSC::JSFunction::jsExecutable):
(JSC::JSFunction::isHostFunction):

2:51 PM Changeset in webkit [47640] by rniwa@webkit.org
  • 21 edits in trunk

WebCore: execCommand("Underline") uses CSS even when styleWithCSS has been turned off
https://bugs.webkit.org/show_bug.cgi?id=23892

Patch by Ryosuke Niwa <rniwa@webkit.org> on 2009-08-21
Reviewed by Eric Seidel.

This patch adds support for u and s in StyleChange and addInlineStyleIfNeeded so that
WebKit does not use CSS to decorate texts when styleWithCSS is set to false.

  • css/CSSValueList.cpp:

(WebCore::CSSValueList::copy): Added.

  • css/CSSValueList.h: Added copy.
  • editing/ApplyStyleCommand.cpp:

(WebCore::StyleChange::applyUnderline):
(WebCore::StyleChange::applyLineThrough):
(WebCore::StyleChange::StyleChange): Added a boolean trimTextDecorations argument.
(WebCore::StyleChange::init): Ditto.
(WebCore::StyleChange::extractTextStyles): Handles text decorations.
(WebCore::getPropertiesNotInComputedStyle): Handles text decorations properly.
(WebCore::ApplyStyleCommand::addInlineStyleIfNeeded): Adds u and s if needed.

LayoutTests: execCommand("Underline") uses CSS even when styleWithCSS has been turned off
https://bugs.webkit.org/show_bug.cgi?id=23892

Patch by Ryosuke Niwa <rniwa@webkit.org> on 2009-08-21
Reviewed by Eric Seidel.

This patch rebaselines tests contained spans to decorate texts even though styleWithCSS is turned off.
It replaces those spans with either u or s accordingly.

It modifies the order in which tag appear in toggle-style-2 to match the actual result
since the original ordering was rather arbitrary.

This patch breaks /editing/style/font-family-with-space.html for Qt platform.
Manual rebaseline is required once the patch is landed.

  • editing/execCommand/resources/toggle-style-2.js: Modified the order in which tag appear
  • editing/execCommand/resources/toggle-text-decorations.js: execCommand('styleWithCSS') is called after selection is set

(testSingleToggle):

  • editing/execCommand/strikethroughSelection-expected.txt: span with line-through was replaced by s
  • editing/execCommand/toggle-style-2-expected.txt: Passes all but the last two tests.
  • editing/style/remove-underline-across-paragraph-expected.txt: Underline is added by u instead of CSS.
  • editing/style/remove-underline-across-paragraph-in-bold-expected.txt: Ditto.
  • editing/style/remove-underline-after-paragraph-expected.txt: Ditto.
  • editing/style/remove-underline-after-paragraph-in-bold-expected.txt: Ditto.
  • editing/style/remove-underline-expected.txt: Ditto.
  • editing/style/remove-underline-in-bold-expected.txt: Ditto.
  • editing/style/underline-expected.txt: Ditto.
  • platform/mac/editing/execCommand/5190926-expected.txt: Ditto.
  • platform/mac/editing/pasteboard/5075944-expected.txt: Ditto.
  • platform/mac/editing/style/font-family-with-space-expected.txt: Ditto.
  • platform/qt/editing/execCommand/5190926-expected.txt: Ditto.
  • platform/qt/editing/pasteboard/5075944-expected.txt: Ditto.
1:35 PM Changeset in webkit [47639] by Joseph Pecoraro
  • 14 edits
    2 moves in trunk/WebCore

Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Inspector: Rename "Databases" Panel as "Storage"
https://bugs.webkit.org/show_bug.cgi?id=28620

Updated enum from "DatabasesPanel" to "StoragePanel" and wherever it was used.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::specialPanelForJSName): string to display StoragePanel can be either "databases" or "storage"
  • inspector/InspectorController.h: (WebCore::InspectorController::):
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::showPanel):

Updated All Direct Access from WebInspector.panels.databases to WebInspector.panels.storage,
any DatabasesPanel constructors and appropriate function names.

  • inspector/front-end/DOMStorageDataGrid.js: (WebInspector.DOMStorageDataGrid.prototype._startEditingColumnOfDataGridNode): (WebInspector.DOMStorageDataGrid.prototype._startEditing): (WebInspector.DOMStorageDataGrid.prototype._editingCommitted): (WebInspector.DOMStorageDataGrid.prototype._editingCancelled): (WebInspector.DOMStorageDataGrid.prototype.deleteSelectedRow):
  • inspector/front-end/DOMStorageItemsView.js: (WebInspector.DOMStorageItemsView.prototype.update):
  • inspector/front-end/DatabaseQueryView.js: (WebInspector.DatabaseQueryView.prototype._queryFinished):
  • inspector/front-end/DatabaseTableView.js: (WebInspector.DatabaseTableView.prototype._queryFinished):
  • inspector/front-end/inspector.js: (WebInspector._createPanels): updated string that comes from hiddenPanels to be "databases" or "storage" (WebInspector.showStoragePanel): (WebInspector.selectDatabase): (WebInspector.selectDOMStorage): (WebInspector.addDatabase): (WebInspector.addDOMStorage):

Renamed some files and Updated Accordingly.

  • inspector/front-end/StoragePanel.js: Renamed from WebCore/inspector/front-end/DatabasesPanel.js.
  • inspector/front-end/Images/storageIcon.png: Renamed from WebCore/inspector/front-end/Images/databasesIcon.png.
  • inspector/front-end/WebKit.qrc: use new file names
  • inspector/front-end/inspector.css: use new images name and class names
  • inspector/front-end/inspector.html: use new file name
  • WebCore.gypi: use new file names

Miscellaneous Updates.

  • English.lproj/localizedStrings.js: Updated Tooltip from "Databases" to "Storage"
12:57 PM Changeset in webkit [47638] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-21 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Skipping a new test, a test that requires geolocation support (we
do not enable it yet), and a test which needs font sizing to be
properly setup.

  • platform/gtk/Skipped:
12:16 PM Changeset in webkit [47637] by eric@webkit.org
  • 1 edit
    3 adds in trunk/WebCore

2009-08-21 Maxime Simon <Maxime Simon>

Reviewed by Oliver Hunt.

[Haiku] Adding three font-specific files to WebCore:
FontCacheHaiku.cpp, FontHaiku.cpp, and SimpleFontDataHaiku.cpp
https://bugs.webkit.org/show_bug.cgi?id=28131

  • platform/graphics/haiku/FontCacheHaiku.cpp: Added. (WebCore::FontCache::platformInit): (WebCore::FontCache::getFontDataForCharacters): (WebCore::FontCache::getSimilarFontPlatformData): (WebCore::FontCache::getLastResortFallbackFont): (WebCore::FontCache::createFontPlatformData): (WebCore::FontCache::getTraitsInFamily):
  • platform/graphics/haiku/FontHaiku.cpp: Added. (charUnicodeToUTF8HACK): (WebCore::Font::canReturnFallbackFontsForComplexText): (WebCore::Font::drawGlyphs): (WebCore::Font::drawComplexText): (WebCore::Font::floatWidthForComplexText): (WebCore::Font::selectionRectForComplexText): (WebCore::Font::offsetForPositionForComplexText):
  • platform/graphics/haiku/SimpleFontDataHaiku.cpp: Added. (WebCore::SimpleFontData::platformInit): (WebCore::SimpleFontData::platformCharWidthInit): (WebCore::SimpleFontData::platformDestroy): (WebCore::SimpleFontData::smallCapsFontData): (WebCore::SimpleFontData::containsCharacters): (WebCore::SimpleFontData::determinePitch): (WebCore::SimpleFontData::platformWidthForGlyph):
12:02 PM Changeset in webkit [47636] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-08-21 Philippe Beauchamp <philippe.beauchamp@gmail.com>

Reviewed by Adam Roben.

Middle click panning icon is offset by 3 pixels
https://bugs.webkit.org/show_bug.cgi?id=28611

  • platform/ScrollView.cpp: panIconSizeLength initialized to 20 instead of 16 to match icon dimension (2 pixels offset)
  • platform/win/CursorWin.cpp: (WebCore::middlePanningCursor): Hotspot initialized to (8,8) instead of (7,7) (one pixel offset)
11:54 AM Changeset in webkit [47635] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-08-21 Laszlo Gombos <Laszlo Gombos>

Reviewed by Eric Seidel.

[Qt] Remove inspector resources for Symbian
https://bugs.webkit.org/show_bug.cgi?id=28610

Based on an idea from Simon Hausmann.

  • WebCore.pro:
11:54 AM Changeset in webkit [47634] by oliver@apple.com
  • 3 edits in trunk/LayoutTests

Add test to ensure JSON parser does not accept comments.

Reviewed by Tim Hatcher.

11:20 AM Changeset in webkit [47633] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-08-21 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Eric Seidel.

Inspector: Remove Unused Variable
https://bugs.webkit.org/show_bug.cgi?id=28616

  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype._checkWarning):
11:11 AM Changeset in webkit [47632] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-08-21 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Inspector: Console Array Formatter Shows Extra Properties
https://bugs.webkit.org/show_bug.cgi?id=28615

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._formatarray): do not ignore hasOwnProperties
11:10 AM Changeset in webkit [47631] by bweinstein@apple.com
  • 2 edits in trunk/WebKitTools

2009-08-20 Brian Weinstein <bweinstein@apple.com>

Reviewed by Adam Roben.
Based on original patch by Stephanie Lewis.

Added support of the Windows malloc history format to parse-malloc history, so we can
read and parse it.

  • Scripts/parse-malloc-history:
7:43 AM Changeset in webkit [47630] by Beth Dakin
  • 13 edits
    6 adds in trunk

WebCore: The CSS part of https://bugs.webkit.org/show_bug.cgi?id=27574 [CSS3
Backgrounds and Borders] Add support for the "contain" value for
background-size
-and-
https://bugs.webkit.org/show_bug.cgi?id=27573 [CSS3 Backgrounds and
Borders] Add support for the "cover" value for background-size

Reviewed by Dan Bernstein.

Return contain or cover when appropriate, and otherwise do what we
used to do.

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

Accept contain and cover as valid values for background-size.

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseFillProperty):

Handle setting the size and the sizeLength if appropriate.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::mapFillSize):

Added new keywords contain and cover.

  • css/CSSValueKeywords.in:

Use just sizeLength instead of size to match old behavior.

  • page/animation/AnimationBase.cpp:

(WebCore::FillLayersPropertyWrapper::FillLayersPropertyWrapper):

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::calculateBackgroundSize):

  • rendering/RenderObject.cpp:

(WebCore::mustRepaintFillLayers):

The variable m_sizeType is an EBackgroundSize, and m_sizeLength is
a LengthSize (the equivalent of what m_size used to be). I got rid
of m_sizeSet and made isSizeSet() dynamic. I also defined a new
struct, FillSize that can be used to set or get m_size and
m_sizeLength both at once.

  • rendering/style/FillLayer.cpp:

(WebCore::FillLayer::FillLayer):
(WebCore::FillLayer::operator=):
(WebCore::FillLayer::operator==):
(WebCore::FillLayer::fillUnsetProperties):

  • rendering/style/FillLayer.h:

(WebCore::FillSize::FillSize):
(WebCore::FillSize::operator==):
(WebCore::FillSize::operator!=):
(WebCore::FillLayer::sizeLength):
(WebCore::FillLayer::size):
(WebCore::FillLayer::isSizeSet):
(WebCore::FillLayer::setSizeType):
(WebCore::FillLayer::setSizeLength):
(WebCore::FillLayer::setSize):
(WebCore::FillLayer::clearSize):
(WebCore::FillLayer::initialFillSizeType):
(WebCore::FillLayer::initialFillSizeLength):
(WebCore::FillLayer::initialFillSize):

This is all boiler-plate stuff to adjust to the new size() vs.
sizeType() vs. sizeLength() distinction.

  • rendering/style/RenderStyle.h:

(WebCore::InheritedFlags::backgroundSizeType):
(WebCore::InheritedFlags::backgroundSizeLength):
(WebCore::InheritedFlags::maskSize):
(WebCore::InheritedFlags::setBackgroundSize):
(WebCore::InheritedFlags::setBackgroundSizeLength):
(WebCore::InheritedFlags::setMaskSize):

Definition for EBackgroundSizeType.

  • rendering/style/RenderStyleConstants.h:

(WebCore::):

LayoutTests: Tests for the CSS part of https://bugs.webkit.org/show_bug.cgi?id=27574 [CSS3 Backgrounds and Borders] Add support for the "contain" value
for background-size
-and-
https://bugs.webkit.org/show_bug.cgi?id=27573 [CSS3 Backgrounds and
Borders] Add support for the "cover" value for background-size

Reviewed by Dan Bernstein.

  • fast/backgrounds/size/parsing-background-size-values-expected.txt: Added.
  • fast/backgrounds/size/parsing-background-size-values.html: Added.
  • fast/backgrounds/size/parsing-inherit-expected.txt: Added.
  • fast/backgrounds/size/parsing-inherit.html: Added.
  • fast/backgrounds/size/resources/parsing-background-size-values.js: Added.

(test):

  • fast/backgrounds/size/resources/parsing-inherit.js: Added.

(test):

6:32 AM Changeset in webkit [47629] by jmalonzo@webkit.org
  • 1 edit
    1 add in trunk/LayoutTests

2009-08-21 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Gustavo Noronha.

[Gtk] Check-in result for http://trac.webkit.org/changeset/47590.

  • platform/gtk/fast/repaint/inline-block-resize-expected.txt: Copied from LayoutTests/platform/mac/fast/repaint/inline-block-resize-expected.txt.
3:07 AM Changeset in webkit [47628] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-08-21 Ariya Hidayat <ariya.hidayat@nokia.com>

Not reviewed, build fix after r47580.

[Qt] Adjust the files.

  • inspector/front-end/WebKit.qrc:
12:48 AM Changeset in webkit [47627] by oliver@apple.com
  • 7 edits in trunk/JavaScriptCore

Browser hangs on opening Web Inspector.
https://bugs.webkit.org/show_bug.cgi?id=28438

Reviewed by Maciej Stachowiak.

Code generation needs to be able to walk the entire scopechain in some
cases, however the symbol table used by activations was a member of the
codeblock. Following recompilation this may no longer exist, leading
to a crash or hang on lookup.

We fix this by introducing a refcounted SymbolTable subclass, SharedSymbolTable,
for the CodeBlocks used by function code. This allows activations to
maintain ownership of a copy of the symbol table even after recompilation so
they can continue to work.

12:01 AM Changeset in webkit [47626] by Chris Fleizach
  • 18 edits
    2 adds in trunk

WebCore: Enable various "grouping" ARIA roles
https://bugs.webkit.org/show_bug.cgi?id=28486

Reviewed by Darin Adler.

Test: platform/mac/accessibility/aria-grouping-roles.html

  • accessibility/AccessibilityObject.h:
  • accessibility/AccessibilityRenderObject.cpp:
  • accessibility/mac/AccessibilityObjectWrapper.mm:
  • page/mac/WebCoreViewFactory.h:
  • platform/LocalizedStrings.h:
  • platform/mac/LocalizedStringsMac.mm:

WebKit: Enable various "grouping" ARIA roles
https://bugs.webkit.org/show_bug.cgi?id=28486

Reviewed by Darin Adler.

  • English.lproj/Localizable.strings:

WebKit/mac: Enable various "grouping" ARIA roles
https://bugs.webkit.org/show_bug.cgi?id=28486

Reviewed by Darin Adler.

  • WebCoreSupport/WebViewFactory.mm:

(-[WebViewFactory AXARIAContentGroupText:]):

WebKitTools: Enable various "grouping" ARIA roles
https://bugs.webkit.org/show_bug.cgi?id=28486

Reviewed by Darin Adler.

Expose the ability to retrieve the subrole through accessibility for DRT.

  • DumpRenderTree/AccessibilityUIElement.cpp:
  • DumpRenderTree/AccessibilityUIElement.h:
  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:
  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:
  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

LayoutTests: Enable various "grouping" ARIA roles
https://bugs.webkit.org/show_bug.cgi?id=28486

Reviewed by Darin Adler.

  • platform/mac/accessibility/aria-grouping-roles-expected.txt: Added.
  • platform/mac/accessibility/aria-grouping-roles.html: Added.
12:00 AM Changeset in webkit [47625] by xan@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-08-20 Xan Lopez <xlopez@igalia.com>

Add new file to GTK+ build.

  • GNUmakefile.am:

Aug 20, 2009:

11:11 PM Changeset in webkit [47624] by levin@chromium.org
  • 2 edits in trunk/WebCore

Unreviewed trivial comment fix.

Patch by David Levin <levin@chromium.org> on 2009-08-20
Update a bug link in a comment due to bugzilla amnesia.

  • loader/CachedResource.cpp:

(WebCore::CachedResource::setResourceToRevalidate):

10:47 PM Changeset in webkit [47623] by Joseph Pecoraro
  • 2 edits in trunk/WebKitTools

Added myself (Joseph Pecoraro) as commiter

10:41 PM Changeset in webkit [47622] by ggaren@apple.com
  • 10 edits
    1 copy in trunk/JavaScriptCore

Added a number => string cache.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-08-20
Reviewed by Maciej Stachowiak.

1.07x faster on v8 (1.7x faster on v8-splay).
1.004x faster on SunSpider.

  • runtime/JSCell.h: Moved JSValue::toString to JSString.h.
  • runtime/JSGlobalData.h: Holds the cache.
  • runtime/JSNumberCell.cpp:

(JSC::JSNumberCell::toString):
(JSC::JSNumberCell::toThisString): Removed -0 special case.
UString handles this now, since too many clients were
special-casing it.

  • runtime/JSString.h:

(JSC::JSValue::toString): Use the cache when converting
an int or double to string.

  • runtime/Operations.h:

(JSC::concatenateStrings): Call toString to take advantage
of the cache.

  • runtime/SmallStrings.h:

(JSC::NumericStrings::add):
(JSC::NumericStrings::lookup): The cache.

  • runtime/UString.cpp:

(JSC::UString::from): Added -0 special case mentioned above.
Removed appendNumeric because it's mutually exclusive with the
cache.

10:23 PM Changeset in webkit [47621] by xan@webkit.org
  • 2 edits in trunk/WebKitTools

2009-08-20 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Fix memory leaks.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::whiteListAccessFromOrigin): (LayoutTestController::pauseAnimationAtTimeOnElementWithId): (LayoutTestController::pauseTransitionAtTimeOnElementWithId):
9:21 PM Changeset in webkit [47620] by oliver@apple.com
  • 6 edits
    1 move in trunk

REGRESSION: fast/profiler/call.html is crashing occasionally
https://bugs.webkit.org/show_bug.cgi?id=28476

Reviewed by Gavin Barraclough.

Using the codeblock for information about how many parameters and
locals a function has is unsafe in certain circumstances. The
basic scenario is all function code being cleared in response to
the debugger or profiler being enabled, and then an activation is
marked before its associated function is re-executed.

To deal with this scenario we store the variable count of a function
directly in the FunctionExecutable, and then use that information.

9:10 PM Changeset in webkit [47619] by ajwong@chromium.org
  • 5 edits
    2 adds in trunk

WebCore: HTML5 media elements do not fire waiting events correctly
https://bugs.webkit.org/show_bug.cgi?id=28335

Patch by Albert J. Wong <ajwong@chromium.org> on 2009-08-20
Reviewed by David Levin.

Fire the waiting event before seeking, and fix a case where firing of
the seeked event is missed when the ready state is changed during
a seek.

Added video-waiting-seeking.html into manual tests because not
all platforms allow seeking into non-buffered ranges.

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::setReadyState): add support for
waiting event when seeking.
(WebCore::HTMLMediaElement::finishSeek): send seeked event
whenever seeking finishes.
(WebCore::HTMLMediaElement::mediaPlayerTimeChanged): dispatch to
new function.

  • html/HTMLMediaElement.h:
  • manual-tests/video-waiting-seeking.html: Added.

LayoutTests: HTML5 media elements do not fire waiting events correctly
https://bugs.webkit.org/show_bug.cgi?id=28335

Patch by Albert J. Wong <ajwong@chromium.org> on 2009-08-20
Reviewed by David Levin.

New manual test to check if waiting/seeking events are fired
when seeking into a non-buffered region.

  • http/tests/media/video-throttled-load.cgi: Added.
  • media/video-test.js:

(isInTimeRanges):

8:18 PM Changeset in webkit [47618] by mitz@apple.com
  • 4 edits in trunk/WebCore

Fix more cases of
<rdar://problem/7154521> Lots of "<Error>: doClip: empty path." spew in
the console with certain content

Reviewed by John Sullivan.

If the border box is empty, simply avoid painting instead of trying to
clip to the empty path.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintFillLayerExtended):

  • rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::paint):

  • rendering/RenderWidget.cpp:

(WebCore::RenderWidget::paint):

8:09 PM Changeset in webkit [47617] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

8:08 PM Changeset in webkit [47616] by mrowe@apple.com
  • 1 copy in tags/Safari-6531.15

New tag.

8:08 PM Changeset in webkit [47615] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebCore

Merge r47606.

7:57 PM Changeset in webkit [47614] by barraclough@apple.com
  • 6 edits in trunk/JavaScriptCore

Numbering of arguments to emitGetJITStubArg/emitPutJITStubArg incorrect
https://bugs.webkit.org/show_bug.cgi?id=28513

Reviewed by Oliver Hunt.

The argumentNumber argument to emitGetJITStubArg/emitPutJITStubArg should match
the argument number used within the stub functions in JITStubs.cpp, but it doesn't.

Firstly, all the numbers changed when we added a void* 'reserved' as the first slot
(rather than leaving argument 0 unused), and secondly in 32_64 builds the index to
peek/poke needs to be multiplies by 2 (since the argument to peek/poke is a number
of machine words, and on 32_64 build the argument slots to stub functions are two
words wide).

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

(JSC::JIT::compileOpCallSetupArgs):
(JSC::JIT::compileOpConstructSetupArgs):
(JSC::JIT::compileOpCallVarargsSetupArgs):
(JSC::JIT::compileOpCall):

  • jit/JITInlineMethods.h:

(JSC::JIT::emitPutJITStubArg):
(JSC::JIT::emitPutJITStubArgConstant):
(JSC::JIT::emitGetJITStubArg):
(JSC::JIT::emitPutJITStubArgFromVirtualRegister):

  • jit/JITOpcodes.cpp:

(JSC::JIT::privateCompileCTIMachineTrampolines):

  • jit/JITPropertyAccess.cpp:

(JSC::JIT::privateCompilePutByIdTransition):

6:49 PM Changeset in webkit [47613] by jorlow@chromium.org
  • 1 edit
    2 adds in trunk/WebCore

2009-08-20 Jeremy Orlow <jorlow@chromium.org>

Build fix. Forgot a svn add for a patch I landed. :-(

  • storage/SQLTransactionClient.cpp: Added. (WebCore::SQLTransactionClient::didCommitTransaction): (WebCore::SQLTransactionClient::didExecuteStatement): (WebCore::SQLTransactionClient::didExceedQuota):
  • storage/SQLTransactionClient.h: Added.
6:43 PM Changeset in webkit [47612] by jorlow@chromium.org
  • 12 edits in trunk/WebCore

2009-08-20 Dumitru Daniliuc <dumi@chromium.org>

Reviewed by Dimitri Glazkov.

Added a client to SQLTransaction. In addition to being a place to
get notifications about certain events in a transaction, it is
also an abstraction layer that allows us to plug in different
implementations for each port for how transactions interract with
the main DB. For example, WebCore's default implementation will
make direct calls to DatabaseTracker's methods. At the same time,
Chromium's implementation will send IPCs to the browser process
whenever a transaction needs something from the main DB.

All storage tests pass.

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

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • storage/Database.cpp: (WebCore::Database::transactionClient):
  • storage/Database.h:
  • storage/DatabaseThread.cpp: (WebCore::DatabaseThread::DatabaseThread):
  • storage/DatabaseThread.h: (WebCore::DatabaseThread::transactionClient):
  • storage/SQLTransaction.cpp: (WebCore::SQLTransaction::runCurrentStatement): (WebCore::SQLTransaction::deliverQuotaIncreaseCallback): (WebCore::SQLTransaction::postflightAndCommit):
  • storage/SQLTransactionClient.cpp: Added.
  • storage/SQLTransactionClient.h: Added.
5:27 PM Changeset in webkit [47611] by bweinstein@apple.com
  • 1 edit in trunk/WebCore/rendering/RenderLayer.cpp

Mac Build Fix.

5:07 PM Changeset in webkit [47610] by bweinstein@apple.com
  • 2 edits in trunk/WebCore

2009-08-20 Brian Weinstein <bweinstein@apple.com>

Reviewed by Steve Falkenburg.

Fix of <https://bugs.webkit.org/show_bug.cgi?id=24793>
Auto scroll speed is faster than in IE, Firefox.

Changed pan scrolling speed to match Firefox's algorithm.

  • rendering/RenderLayer.cpp: (WebCore::adjustedScrollDelta): (WebCore::RenderLayer::panScrollFromPoint):
4:49 PM Changeset in webkit [47609] by rniwa@webkit.org
  • 3 edits in trunk/LayoutTests

underline tests in /editing/style/ need not to be pixel tests but need to print markup
https://bugs.webkit.org/show_bug.cgi?id=28471

Patch by Ryosuke Niwa <rniwa@webkit.org> on 2009-08-20
Reviewed by Eric Seidel.

This patch modifies remove-underline-in-bold.html to use runDumpAsTextEditingTest.

  • editing/style/remove-underline-in-bold-expected.txt:
  • editing/style/remove-underline-in-bold.html:
4:37 PM Changeset in webkit [47608] by rniwa@webkit.org
  • 5 edits
    3 adds
    8 deletes in trunk/LayoutTests

Some tests using editing.js in editing/execCommand should use runDumpAsTextEditingTest
https://bugs.webkit.org/show_bug.cgi?id=28494

Patch by Ryosuke Niwa <rniwa@webkit.org> on 2009-08-20
Reviewed by Eric Seidel.

This patch modifies the following tests to use runDumpAsTextEditingTest

  • editing/editing.js: Fixed typo and added foreColorCommand.

(execBackColorCommand):

  • editing/execCommand/italicizeByCharacter-expected.txt: Added.
  • editing/execCommand/italicizeByCharacter.html:
  • editing/execCommand/modifyForeColorByCharacter-expected.txt: Added.
  • editing/execCommand/modifyForeColorByCharacter.html:
  • editing/execCommand/strikethroughSelection-expected.txt: Added.
  • editing/execCommand/strikethroughSelection.html:
4:30 PM Changeset in webkit [47607] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-08-20 Stephen White <senorblanco@chromium.org>

Reviewed by Eric Seidel.

Fix for assert in Chromium page cycler: drawRect() was re-using
an SkPaint, but not resetting it before calling
PlatformGraphicsSkia::setupPaintForFilling() a second time. This
CL fixes drawRect(), and re-enables the assert.
http://bugs.webkit.org/show_bug.cgi?id=28172
http://crbug.com/19797

Covered by Chromium page cycler tests.

  • platform/graphics/skia/PlatformContextSkia.cpp: (PlatformContextSkia::drawRect): (PlatformContextSkia::setupPaintCommon):
4:17 PM Changeset in webkit [47606] by andersca@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/7159016> Popup menus don't disappear when you click outside the window.

Reviewed by Dan Bernstein.

Revert r47535 which introduced this.

  • platform/win/PopupMenuWin.cpp:

(WebCore::PopupMenu::show):
(WebCore::PopupWndProc):

4:05 PM Changeset in webkit [47605] by oliver@apple.com
  • 4 edits in trunk/JavaScriptCore

EGRESSION: significant slowdown on Celtic Kane "AJAX declaration" subtest
https://bugs.webkit.org/show_bug.cgi?id=28332

Reviewed by Geoff Garen

Follow up style fixes that were missed in review.

3:58 PM Changeset in webkit [47604] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

Add new exports to windows jsc build

3:37 PM Changeset in webkit [47603] by eric@webkit.org
  • 3 edits
    2 adds in trunk/WebKitTools

2009-08-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool post-diff can post partial diffs from SVN checkouts.
https://bugs.webkit.org/show_bug.cgi?id=28445

Pass the checkout root as the cwd. Also wrote a test to ensure this.

  • Scripts/modules/scm.py:
  • Scripts/modules/scm_unittest.py:
3:37 PM Changeset in webkit [47602] by levin@chromium.org
  • 4 edits in trunk/WebCore

Crashes on sites with lots of images
https://bugs.webkit.org/show_bug.cgi?id=28473

Patch by David Levin <levin@chromium.org> on 2009-08-20
Reviewed by Alexey Proskuryakov.

The problem is that m_resourceToRevalidate::m_isBeingRevalidated is false while
CachedResource is still referring to it, so it may get deleted before
Cache::revalidationSucceeded calls CachedResource::clearResourceToRevalidate.

  • loader/Cache.cpp:

(WebCore::Cache::revalidateResource): Added assert to verify the resource
being revalidated is physically in the cache.
(WebCore::Cache::revalidationSucceeded): Changed to do one call to switch from the revalidating
resource to the revalidated resource. This allows for proper lifetime management of the
revalidated resource. Added assert to verify that the revalidatingResource is in the cache.
(WebCore::Cache::pruneDeadResources): Don't remove items from the cache that are in the
process of being validated (or else they can get added twice: Once through the normal mechanism
and then again during revalidateResource).

  • loader/CachedResource.cpp:

(WebCore::CachedResource::CachedResource):
(WebCore::CachedResource::isSafeToMakePurgeable):
Changed m_isBeingRevalidated to m_proxyResource to allow finding out
who the validating resource (or proxy) is.

(WebCore::CachedResource::~CachedResource): Removed code to handle m_resourceToRevalidate
being non-zero because this shouldn't be possible due to checks in canDelete. Added asserts
as well.
(WebCore::CachedResource::setResourceToRevalidate): Adjusted for the change to m_proxyResource
and added a comment about the assert.
(WebCore::CachedResource::clearResourceToRevalidate):
Only clean up the resourceToRevalidate if it hasn't gotten a new proxy resource.

  • loader/CachedResource.h:

(WebCore::CachedResource::setInCache): Don't clear m_proxyResource because the resource
is still being referred to by it and the code it robust to this being switched to a new proxy
resource without it being cleared first.
(WebCore::CachedResource::canDelete): Changed m_isBeingRevalidated to m_proxyResource.

3:36 PM Changeset in webkit [47601] by oliver@apple.com
  • 6 edits in trunk/JavaScriptCore

REGRESSION: significant slowdown on Celtic Kane "AJAX declaration" subtest
https://bugs.webkit.org/show_bug.cgi?id=28332

Reviewed by Gavin Barraclough.

The method check optimisation made transitions aware of the value being
assigned when a transition was assigning a function. This had the side
effect of making every assignment of a function expression result in a
new transition, and thus a new Structure. The net result of this is that
the common JS idiom of

function MyObject() {

this.myFunction = function(...){...};

}
new MyObject();

Will produce a unique structure on every iteration, meaning that all
caching is defeated and there is a significant amount of structure churn.

The fix is to return the transition to its original form where it is
keyed off a property name + attributes tuple, but have each transition
support an optional transition on a specific value.

3:36 PM Changeset in webkit [47600] by eric@webkit.org
  • 1 edit
    1 move in trunk/LayoutTests

2009-08-20 Eric Seidel <eric@webkit.org>

No review, just disabling a crashing test.

REGRESSION: fast/profiler/call.html is crashing occasionally
https://bugs.webkit.org/show_bug.cgi?id=28476

Disable this test until one of the JSC folks can find a fix.
This test will occasionally crash and cause patches to be rejected
from the commit-queue which are otherwise fine.

  • fast/profiler/call.html-disabled: Renamed from LayoutTests/fast/profiler/call.html.
3:22 PM Changeset in webkit [47599] by eric@webkit.org
  • 3 edits
    1 add in trunk/WebCore

2009-08-20 Xiaomei Ji <xji@chromium.org>

Reviewed by Eric Seidel.

Fix "Chromium RTL autocomplete popup is not layout correctly".
https://bugs.webkit.org/show_bug.cgi?id=27889

The complete fix of the issue consists 2 parts: the patch in webkit
and the patch in Chromium's own code.

This webkit patch only affects Chromium autofill. It

  1. introduces a new flag in WebCore::PopupContainerSettings to distinguish whether the width of the drop-down should be restricted or not. For autofill, the width of the drop-down is restricted to be the same as that of the input field (the new flag is set in Chromium's own code). But width is not restricted for <select> (same as before).
  2. introduce a new flag in WebCore::PopContainerSettings to indicate what heuristics to use when displaying text in drop-down menu. For autofill, use drop-down item's directionality to display drop-down items. Previously, drop-down item is displayed in the its first strong directional character's directionality. (drop-down item's directionality is set in Chromium's own code. It is set the same as the directionality of the element. For autofill, it is the same directionality as that of the input field.) For <select>, still use the text's first strong directional character's directionality to display the text.

Since the patch only affects the chromium client, not webcore part or
other clients. No automatic tests is possible.

  • manual-tests/autofill-popup-width-and-item-direction.html: Added.
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::): (WebCore::PopupListBox::paintRow): If list box width is restricted and an item is longer to fit in a list box, truncate it and draw part of the text and append ellipses. (WebCore::PopupListBox::layout): Restrict width of list box if applicable.
  • platform/chromium/PopupMenuChromium.h: (WebCore::PopupItem::PopupItem): style change. (WebCore::PopupContainerSettings::): Add 2 new flags in PopupContainerSetting to distinguish whether to restrict width of list box and in what directionality to display the text in drop-down.
3:02 PM Changeset in webkit [47598] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-08-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool post-diff can post partial diffs from SVN checkouts.
https://bugs.webkit.org/show_bug.cgi?id=28445

Pass the checkout root as the cwd. Also wrote a test to ensure this.

  • Scripts/modules/scm.py:
  • Scripts/modules/scm_unittest.py:
2:49 PM Changeset in webkit [47597] by barraclough@apple.com
  • 10 edits in trunk/JavaScriptCore

Remove FunctionCodeBlock.
https://bugs.webkit.org/show_bug.cgi?id=28502

Reviewed by Oliver Hunt.

These only exist to allow JIT code to dereference properties off the
CodeBlock for any callee, regardless of whether it is a host function.

Instead just use the FunctionExecutable. Copy the m_parameters field
from the CodeBlock into the Executable, and use this to distinguish
between host functions, functions that have been bytecompiled, and
functions that have not.

m_parameters is moved to ExecutableBase rather than FunctionExecutable
so that (as a separate change) we can move make a separate class of
executable for host code, which is not devived from FunctionExecutable
(host code does not feature any of the properties that normal executable
do and will provide, such as source, attributes, and a parsed name).

1% win on v8 tests, 0.5% on sunspider.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::derefStructures):
(JSC::CodeBlock::refStructures):
(JSC::CodeBlock::reparseForExceptionInfoIfNecessary):
(JSC::CodeBlock::handlerForBytecodeOffset):
(JSC::CodeBlock::lineNumberForBytecodeOffset):
(JSC::CodeBlock::expressionRangeForBytecodeOffset):
(JSC::CodeBlock::getByIdExceptionInfoForBytecodeOffset):
(JSC::CodeBlock::functionRegisterForBytecodeOffset):
(JSC::CodeBlock::hasGlobalResolveInstructionAtBytecodeOffset):
(JSC::CodeBlock::hasGlobalResolveInfoAtBytecodeOffset):

  • bytecode/CodeBlock.h:

(JSC::):
(JSC::CodeBlock::source):
(JSC::CodeBlock::sourceOffset):
(JSC::CodeBlock::evalCodeCache):
(JSC::CodeBlock::createRareDataIfNecessary):

remove NativeCodeBlocks and the NativeCode code type.


  • jit/JIT.cpp:

(JSC::JIT::linkCall):

Revert to previous behaviour (as currently still commented!) that Hhost functions have a null codeblock.

  • jit/JITCall.cpp:

(JSC::JIT::compileOpCallInitializeCallFrame):
(JSC::JIT::compileOpCallSetupArgs):
(JSC::JIT::compileOpCallVarargsSetupArgs):
(JSC::JIT::compileOpConstructSetupArgs):
(JSC::JIT::compileOpCallVarargs):
(JSC::JIT::compileOpCall):
(JSC::JIT::compileOpCallSlowCase):

Bring the 32_64 & non-32_64 JITs into line with each other, callee in regT0.

  • jit/JITOpcodes.cpp:

(JSC::JIT::privateCompileCTIMachineTrampolines):

Rewrite call trampolines to not use the CodeBlock.

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

Make call_JSFunction & call_arityCheck return the callee, don't expect to be passed the CodeBlock.

  • runtime/Executable.cpp:

(JSC::FunctionExecutable::generateBytecode):
(JSC::FunctionExecutable::recompile):
(JSC::FunctionExecutable::FunctionExecutable):

  • runtime/Executable.h:

(JSC::ExecutableBase::):
(JSC::ExecutableBase::ExecutableBase):
(JSC::FunctionExecutable::isHostFunction):

Add m_numParameters.

  • runtime/JSFunction.cpp:

(JSC::JSFunction::~JSFunction):

Only call generatedBytecode() on JSFunctions non-host FunctionExecutables.

2:26 PM Changeset in webkit [47596] by bweinstein@apple.com
  • 2 edits in trunk/WebCore

2009-08-20 Brian Weinstein <bweinstein@apple.com>

Reviewed by Adele Peterson.

Fix of <https://bugs.webkit.org/show_bug.cgi?id=28504>.
Pan Scrolling can scroll while showing the middlePanScroll icon.

Fixed an off by one error between RenderLayer::panScrollFromPoint and EventHandler::updatePanScrollState.
In RenderLayer::panScrollFromPoint, we were setting the icon to be an arrow if the difference between the start
of the pan scroll and the current mouseposition is > 15. However, in EventHandler::updatePanScrollState, we would
set our amount to scroll to 0 only if the difference in mouse position is < 15. I made this check a <= 15, to fix
the off by one error.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::panScrollFromPoint):
2:24 PM Changeset in webkit [47595] by hyatt@apple.com
  • 3 edits in trunk/WebCore

Pull the code in layoutBlockChildren for handling a normal flow block child into a helper method,
layoutBlockChild.

Reviewed by Dan Bernstein.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::layoutBlockChildren):
(WebCore::RenderBlock::layoutBlockChild):

  • rendering/RenderBlock.h:
2:18 PM Changeset in webkit [47594] by bweinstein@apple.com
  • 6 edits in trunk

2009-08-20 Brian Weinstein <bweinstein@apple.com>

Reviewed by Adam Roben.

Fix of <http://webkit.org/b/27850>. Panning by emulating Mouse Wheel Events breaks Google Maps.


Switched to using RenderLayer::scrollByRecursively instead of emulating a mouse wheel,
and switched to handling the hit testing in WebView from doing it as part of the mouse wheel event.
One downside of this fix is that panning will not jump out of frames anymore (same behavior as pan-scrolling), but
this is tracked in <http://webkit.org/b/28237>.

  • WebView.cpp: (WebView::gestureNotify): Added hit testing here to figure out the node the user started the gesture on. (WebView::gesture): Switched from emulating mouse wheel events for panning to calling RenderLayer::scrollByRecursively.
  • WebView.h:

2009-08-20 Brian Weinstein <bweinstein@apple.com>

Reviewed by Adam Roben.

Made subframeForHitTargetNode a public static method, so WebView can call it
during hit testing.

  • page/EventHandler.cpp: (WebCore::subframeForHitTestResult): (WebCore::EventHandler::subframeForTargetNode):
  • page/EventHandler.h:
2:03 PM Changeset in webkit [47593] by bfulgham@webkit.org
  • 2 edits in trunk/WebCore

Rubberstamped by Kevin Ollivier.

Check for null bundle before attempting to use it.

  • platform/network/curl/ResourceHandleManager.cpp:

(WebCore::certificatePath): Add null check on return of

CFBundleGetBundleWithIdentifier.

1:12 PM Changeset in webkit [47592] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-08-20 Yongjun Zhang <yongjun.zhang@nokia.com>

Reviewed by Eric Seidel.

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

Use a helper function to work around winscw compiler forward declaration bug
regarding templated classes.

Add parenthesis around (PassRefPtr::*UnspecifiedBoolType) to make winscw compiler
work with the default UnSpecifiedBoolType() operator, which removes the winscw
specific bool cast hack.

  • wtf/PassRefPtr.h: (WTF::derefIfNotNull): (WTF::PassRefPtr::~PassRefPtr):
12:56 PM Changeset in webkit [47591] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Don't leak the JSStringRef returned by AccessibilityUIElement::attributeValue.

Reviewed by Adele Peterson.

  • DumpRenderTree/AccessibilityUIElement.cpp:

(attributeValueCallback):

12:52 PM Changeset in webkit [47590] by hyatt@apple.com
  • 7 edits
    4 adds in trunk

WebCore: https://bugs.webkit.org/show_bug.cgi?id=28497, images and inline replaced elements don't propagate overflow properly on a line.

Reviewed by Dan Bernstein.

Added fast/repaint/inline-block-overflow.html and updated another test.

  • rendering/InlineFlowBox.cpp:

(WebCore::InlineFlowBox::placeBoxesHorizontally):
(WebCore::InlineFlowBox::computeVerticalOverflow):
Make sure to still propagate the border box of the replaced element as layout overflow when overflow clip is set on the
replaced element. Make sure to use the InlineBox x/y positions instead of the RenderBox x/y, since the position of the
RenderBox has not been updated to the new values yet.

  • rendering/RenderLineBoxList.cpp:

(WebCore::RenderLineBoxList::paint):
Don't use the root line box's overflow. Use the actual line box overflow values instead. In the case of an inline
with a layer, our overflow didn't propagate to the root line, so we always need to use our values.

  • rendering/RenderPartObject.cpp:

(WebCore::RenderPartObject::layout):

  • rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::layout):
Make sure to clear shadow overflow when doing layout of replaced elements, so that we don't leave a stale overflow
value around if a box-shadow goes away.

LayoutTests: https://bugs.webkit.org/show_bug.cgi?id=28497, images don't propagate overflow properly when they change
size on a line.

Reviewed by Dan Bernstein.

Add a new layout test and update an existing layout test.

  • fast/repaint/inline-block-resize.html: Added.
  • platform/mac/fast/repaint/inline-block-resize-expected.checksum: Added.
  • platform/mac/fast/repaint/inline-block-resize-expected.png: Added.
  • platform/mac/fast/repaint/inline-block-resize-expected.txt: Added.
  • platform/mac/fast/replaced/border-radius-clip-expected.txt:
12:44 PM Changeset in webkit [47589] by dimich@chromium.org
  • 2 edits in trunk/WebCore

Another attempt to fix Chromium build.

  • WebCore.gypi: now use the correct name of the image file.
12:38 PM Changeset in webkit [47588] by mitz@apple.com
  • 2 edits in trunk/WebCore

Make the Windows build even fixeder

  • platform/graphics/win/GraphicsContextCGWin.cpp:

(WebCore::GraphicsContextPlatformPrivate::flush):

12:36 PM Changeset in webkit [47587] by dimich@chromium.org
  • 2 edits in trunk/WebCore

Not reviewed, Chromium build fix.

  • WebCore.gypi: add new Webinspector image files (localStorage.png and sessionStorage.png)
12:34 PM Changeset in webkit [47586] by mitz@apple.com
  • 2 edits in trunk/WebCore

Windows build fix after the last change

  • platform/graphics/win/GraphicsContextCGWin.cpp:

(WebCore::GraphicsContext::GraphicsContext):
(WebCore::GraphicsContext::releaseWindowsContext):
(WebCore::GraphicsContext::drawWindowsBitmap):

12:24 PM Changeset in webkit [47585] by mitz@apple.com
  • 17 edits in trunk/WebCore

Replace many manually-released CFTypeRefs with RetainPtrs
https://bugs.webkit.org/show_bug.cgi?id=28498

Reviewed by Geoffrey Garen.

  • platform/graphics/cg/ColorCG.cpp:

(WebCore::createCGColor):

  • platform/graphics/cg/GradientCG.cpp:

(WebCore::Gradient::platformGradient):

  • platform/graphics/cg/GraphicsContextCG.cpp:

(WebCore::GraphicsContext::platformContext):
(WebCore::GraphicsContext::applyStrokePattern):
(WebCore::GraphicsContext::applyFillPattern):
(WebCore::GraphicsContext::setPlatformShadow):
(WebCore::GraphicsContext::setURLForRect):

  • platform/graphics/cg/GraphicsContextPlatformPrivateCG.h:

(WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate):
(WebCore::GraphicsContextPlatformPrivate::~GraphicsContextPlatformPrivate):

  • platform/graphics/cg/ImageBufferCG.cpp:

(WebCore::ImageBuffer::ImageBuffer):

  • platform/graphics/cg/ImageCG.cpp:

(WebCore::BitmapImage::checkForSolidColor):
(WebCore::Image::drawPattern):

  • platform/graphics/cg/ImageSourceCG.cpp:

(WebCore::ImageSource::setData):
(WebCore::ImageSource::isSizeAvailable):
(WebCore::ImageSource::frameSizeAtIndex):
(WebCore::ImageSource::repetitionCount):
(WebCore::ImageSource::createFrameAtIndex):
(WebCore::ImageSource::frameDurationAtIndex):

  • platform/graphics/cg/PDFDocumentImage.cpp:

(WebCore::PDFDocumentImage::dataChanged):

  • platform/graphics/cg/PathCG.cpp:

(WebCore::createScratchContext):
(WebCore::Path::contains):

  • platform/graphics/mac/FontCustomPlatformData.cpp:

(WebCore::createFontCustomPlatformData):

  • platform/graphics/mac/GraphicsContextMac.mm:

(WebCore::GraphicsContext::drawFocusRing):

  • platform/graphics/mac/ImageMac.mm:

(WebCore::BitmapImage::getTIFFRepresentation):

  • platform/mac/ClipboardMac.mm:

(WebCore::cocoaTypeFromMIMEType):
(WebCore::MIMETypeFromCocoaType):

  • platform/mac/WebCoreNSStringExtras.mm:

(stringEncodingForResource):

  • platform/network/mac/FormDataStreamMac.mm:

(WebCore::advanceCurrentStream):
(WebCore::setHTTPBody):

  • platform/text/mac/TextCodecMac.cpp:

(WebCore::TextCodecMac::encode):

11:34 AM Changeset in webkit [47584] by eric@webkit.org
  • 2 edits in trunk/WebKitSite

2009-08-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

major-objects.html should show direct Frame <-> Document pointers
https://bugs.webkit.org/show_bug.cgi?id=28479

  • coding/major-objects.html: Added pointers to diagram.
11:27 AM Changeset in webkit [47583] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-08-20 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

Output actual values of counters in showTree
https://bugs.webkit.org/show_bug.cgi?id=28481

No new tests because this patch just improves debugging outputs.

  • rendering/CounterNode.cpp: (WebCore::showTreeAndMark):
11:04 AM Changeset in webkit [47582] by mrowe@apple.com
  • 19 edits in trunk/JavaScriptCore

Roll out r47571 and related build fixes as it caused us to leak the world without warning.

10:53 AM Changeset in webkit [47581] by eric.carlson@apple.com
  • 2 edits in trunk/WebCore

2009-08-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by NOBODY (build fix).

Add file missed in last check-in.

  • html/HTMLAttributeNames.in:
10:45 AM Changeset in webkit [47580] by timothy@apple.com
  • 4 edits
    2 adds
    1 delete in trunk/WebCore

Adds new icons to the Web Inspector for Local Storage,
Session Storage and Cookies.

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

Reviewed by Eric Seidel.

  • inspector/front-end/DatabasesPanel.js:

(WebInspector.DatabasesPanel.prototype.addDOMStorage):
(WebInspector.DOMStorageSidebarTreeElement):

  • inspector/front-end/Images/cookie.png:
  • inspector/front-end/Images/domStorage.png: Removed.
  • inspector/front-end/Images/localStorage.png: Added.
  • inspector/front-end/Images/sessionStorage.png: Added.
  • inspector/front-end/inspector.css:
10:21 AM Changeset in webkit [47579] by eric.carlson@apple.com
  • 10 edits in trunk

2009-08-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Anders Carlsson.

HTML5 media elements must fire 'loadend' progress event
https://bugs.webkit.org/show_bug.cgi?id=28419

  • dom/EventNames.h:

Define loadend.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::parseMappedAttribute):

Deal with onloadend.

(WebCore::HTMLMediaElement::loadInternal):

Post 'loadend' immediately after 'abort'.

(WebCore::HTMLMediaElement::noneSupported):

Post 'loadend' immediately after 'error'.

(WebCore::HTMLMediaElement::mediaEngineError):

Ditto.

(WebCore::HTMLMediaElement::setNetworkState):

Post 'loadend' immediately after 'load'.

(WebCore::HTMLMediaElement::userCancelledLoad):

Post 'loadend' immediately after 'abort'.

2009-08-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Anders Carlsson.

HTML5 media elements must fire 'loadend' progress event
https://bugs.webkit.org/show_bug.cgi?id=28419

Add 'loadend' to existing tests.

  • media/event-attributes-expected.txt:
  • media/event-attributes.html:
  • media/media-load-event-expected.txt:
  • media/media-load-event.html:
  • media/progress-event-at-least-one-expected.txt:
  • media/progress-event-at-least-one.html:
10:18 AM Changeset in webkit [47578] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2009-08-20 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

REGESSION(r45316), Crash: WebKit crashes in Google Sites when indenting a table
https://bugs.webkit.org/show_bug.cgi?id=28474

This patch adds a test to make sure WebKit does not crash when indenting immediately right to a table.

  • editing/execCommand/indent-right-after-table-expected.txt: Added.
  • editing/execCommand/indent-right-after-table.html: Added.

2009-08-20 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

REGESSION(r45316), Crash: WebKit crashes in Google Sites when indenting a table
https://bugs.webkit.org/show_bug.cgi?id=28474

This patch fixes a crash when indenting at right after a table.
The bug was caused by Node* blockNode = enclosingBlock(endOfCurrentParagraph.deepEquivalent().node());
where node() is equal to blockNode. Because blockNode is the enclosing block node, this patch changes it to
enclosingBlock(endOfCurrentParagraph.deepEquivalent().node()->parentNode());

Test: editing/execCommand/indent-right-after-table.html

  • editing/IndentOutdentCommand.cpp: (WebCore::IndentOutdentCommand::indentRegion): Fixed so that blockNode != endOfCurrentParagraph.deepEquivalent().node()
10:11 AM Changeset in webkit [47577] by kevino@webkit.org
  • 2 edits in trunk/WebKit/wx

Reviewed by Kevin Ollivier.

Use FrameLoader::load() for loading pages from a HTML page string to address
bugs with using begin/write/end loading.

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

10:10 AM Changeset in webkit [47576] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-08-20 Andrew Scherkus <scherkus@chromium.org>

Reviewed by David Levin.

Render disabled mute button during an error or if no audio is present.

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

Covered by existing layout tests:
media/video-controls-visible-audio-only.html
media/video-no-audio.html

  • rendering/RenderThemeChromiumSkia.cpp: (WebCore::RenderThemeChromiumSkia::paintMediaMuteButton):
7:38 AM Changeset in webkit [47575] by Darin Adler
  • 4 edits in trunk/JavaScriptCore

Try to fix Windows build.

Patch by Darin Adler <Darin Adler> on 2009-08-20

Removed all exports involving the class FunctionBodyNode.

7:36 AM Changeset in webkit [47574] by ariya@webkit.org
  • 2 edits in trunk/WebKitTools

2009-08-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Unreviewed, build fix.

[Qt] The template-based qMax() compares two qreals.

  • DumpRenderTree/qt/ImageDiff.cpp: (main):
7:32 AM Changeset in webkit [47573] by Darin Adler
  • 3 edits in trunk/JavaScriptCore

Try to fix Windows build.

Patch by Darin Adler <Darin Adler> on 2009-08-20

  • parser/ParserArena.cpp:

(JSC::ParserArena::derefWithArena): Make non-inline.

  • parser/ParserArena.h: Ditto.
7:28 AM Changeset in webkit [47572] by ariya@webkit.org
  • 22 edits in trunk/LayoutTests

2009-08-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Rubber-stamped by Simon Hausmann.

[Qt] Update the (outdated) metrics for css1 padding and margin tests. All tests pass.

  • platform/qt/Skipped: Excluded passed tests.
  • platform/qt/css1/box_properties/margin-expected.txt:
  • platform/qt/css1/box_properties/margin_bottom-expected.txt:
  • platform/qt/css1/box_properties/margin_bottom_inline-expected.txt:
  • platform/qt/css1/box_properties/margin_inline-expected.txt:
  • platform/qt/css1/box_properties/margin_left-expected.txt:
  • platform/qt/css1/box_properties/margin_left_inline-expected.txt:
  • platform/qt/css1/box_properties/margin_right-expected.txt:
  • platform/qt/css1/box_properties/margin_right_inline-expected.txt:
  • platform/qt/css1/box_properties/margin_top-expected.txt:
  • platform/qt/css1/box_properties/margin_top_inline-expected.txt:
  • platform/qt/css1/box_properties/padding-expected.txt:
  • platform/qt/css1/box_properties/padding_bottom-expected.txt:
  • platform/qt/css1/box_properties/padding_bottom_inline-expected.txt:
  • platform/qt/css1/box_properties/padding_inline-expected.txt:
  • platform/qt/css1/box_properties/padding_left-expected.txt:
  • platform/qt/css1/box_properties/padding_left_inline-expected.txt:
  • platform/qt/css1/box_properties/padding_right-expected.txt:
  • platform/qt/css1/box_properties/padding_right_inline-expected.txt:
  • platform/qt/css1/box_properties/padding_top-expected.txt:
  • platform/qt/css1/box_properties/padding_top_inline-expected.txt:
7:24 AM Changeset in webkit [47571] by Darin Adler
  • 17 edits in trunk/JavaScriptCore

Syntax tree nodes should use arena allocation
https://bugs.webkit.org/show_bug.cgi?id=25674

Patch by Darin Adler <Darin Adler> on 2009-08-20
Reviewed by Gavin Barraclough.

Use an actual arena now. 0.7% speedup on SunSpider.

longer needs to be used outside JavaScriptCore.

Executable.h project-internal instead of "private".

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator): Updated since VarStack
contains const Identifier* now.

  • parser/Grammar.y: Made identifiers from the lexer be const

Identifier* and updated since VarStack contains const Identifier* now.

  • parser/Lexer.cpp:

(JSC::Lexer::setCode): Pass in ParserArena, used for identifiers.
(JSC::Lexer::makeIdentifier): Changed return type to const Identifier*
and changed to call ParserArena.
(JSC::Lexer::clear): Removed the code to manage m_identifiers and
added code to set m_arena to 0.

  • parser/Lexer.h: Updated for changes above.
  • parser/NodeConstructors.h:

(JSC::ParserArenaFreeable::operator new): Added. Calls allocateFreeable
on the arena.
(JSC::ParserArenaDeletable::operator new): Changed to call the
allocateDeletable function on the arena instead of deleteWithArena.
(JSC::PropertyNode::PropertyNode): Added new constructor that makes
numeric identifiers. Some day we might want to optimize this for
integers so it doesn't create a string for each one.
(JSC::ContinueNode::ContinueNode): Initialize m_ident to nullIdentifier
since it's now a const Identifier& so it can't be left uninitialized.
(JSC::BreakNode::BreakNode): Ditto.
(JSC::CaseClauseNode::CaseClauseNode): Updated to use SourceElements*
to keep track of the statements rather than a separate statement vector.
(JSC::BlockNode::BlockNode): Ditto.
(JSC::ForInNode::ForInNode): Initialize m_ident to nullIdentifier.

  • parser/Nodes.cpp: Moved the comment explaining emitBytecode in here.

It seemed strangely out of place in the header.
(JSC::ThrowableExpressionData::emitThrowError): Added an overload for
UString as well as Identifier.
(JSC::SourceElements::singleStatement): Added.
(JSC::SourceElements::lastStatement): Added.
(JSC::RegExpNode::emitBytecode): Changed the throwError code to use
the substitution mechanism instead of doing a string append.
(JSC::SourceElements::emitBytecode): Added. Replaces the old
statementListEmitCode function, since we now keep the SourceElements
objects around.
(JSC::BlockNode::lastStatement): Added.
(JSC::BlockNode::emitBytecode): Changed to use emitBytecode instead of
statementListEmitCode.
(JSC::CaseClauseNode::emitBytecode): Added.
(JSC::CaseBlockNode::emitBytecodeForBlock): Changed to use emitBytecode
instead of statementListEmitCode.
(JSC::ScopeNodeData::ScopeNodeData): Changed to store the
SourceElements* instead of using releaseContentsIntoVector.
(JSC::ScopeNode::emitStatementsBytecode): Added.
(JSC::ScopeNode::singleStatement): Added.
(JSC::ProgramNode::emitBytecode): Call emitStatementsBytecode instead
of statementListEmitCode.
(JSC::EvalNode::emitBytecode): Ditto.
(JSC::FunctionBodyNode::emitBytecode): Call emitStatementsBytecode
insetad of statementListEmitCode and check for the return node using
the new functions.

  • parser/Nodes.h: Changed VarStack to store const Identifier* instead

of Identifier and rely on the arena to control lifetime. Added a new
ParserArenaFreeable class. Made ParserArenaDeletable inherit from
FastAllocBase instead of having its own operator new. Base the Node
class on ParserArenaFreeable. Changed the various Node classes
to use const Identifier& instead of Identifier to avoid the need to
call their destructors and allow them to function as "freeable" in the
arena. Removed extraneous JSC_FAST_CALL on definitions of inline functions.
Changed ElementNode, PropertyNode, ArgumentsNode, ParameterNode,
CaseClauseNode, ClauseListNode, and CaseBlockNode to use ParserArenaFreeable
as a base class since they do not descend from Node. Eliminated the
StatementVector type and instead have various classes use SourceElements*
instead of StatementVector. This prevents those classes from having to
use ParserArenaDeletable to make sure the vector destructor is called.

  • parser/Parser.cpp:

(JSC::Parser::parse): Pass the arena to the lexer.

  • parser/Parser.h: Added an include of ParserArena.h, which is no longer

included by Nodes.h.
(JSC::Parser::parseFunctionFromGlobalCode): Changed to use the
singleStatement function, since there is no longer any children function.
Removed some unneeded use of RefPtr.

  • parser/ParserArena.cpp:

(JSC::ParserArena::ParserArena): Added. Initializes the new members,
m_freeableMemory, m_freeablePoolEnd, and m_identifiers.
(JSC::ParserArena::freeablePool): Added. Computes the pool pointer,
since we store only the current pointer and the end of pool pointer.
(JSC::ParserArena::deallocateObjects): Added. Contains the common
memory-deallocation logic used by both the destructor and the
reset function.
(JSC::ParserArena::~ParserArena): Changed to call deallocateObjects.
(JSC::ParserArena::reset): Ditto. Also added code to zero out the
new structures, and switched to use clear() instead of shrink(0) since
we don't really reuse arenas.
(JSC::ParserArena::makeNumericIdentifier): Added.
(JSC::ParserArena::allocateFreeablePool): Added. Used when the pool
is empty.
(JSC::ParserArena::isEmpty): Added. No longer inline, which is fine
since this is used only for assertions at the moment.

  • parser/ParserArena.h: Added an actual arena of "freeable" objects,

ones that don't need destructors to be called. Also added a separate
IdentifierArena object, a segmented vector of identifiers that used
to be in the Lexer.

  • runtime/Executable.h: Moved the definition of the

FunctionExecutable::make function here. It can't go in JSFunction.h
since that header has to be used outside JavaScriptCore and so can't
include this, which includes Nodes.h. The function could be moved
elswhere if we don't want to include JSFunction.h in this header, but
for now this seems to be the best place.

  • runtime/JSFunction.h: Removed the include of Executable.h and

definition of the FunctionExecutable::make function.

  • wtf/FastMalloc.cpp: Fixed an incorrect comment.
7:11 AM Changeset in webkit [47570] by mitz@apple.com
  • 8 edits in trunk

WebCore: <rdar://problem/7154605> Loading page with 1px wide <select> makes the
toolbar go blank
and
<rdar://problem/7154521> Lots of "<Error>: doClip: empty path." spew in
the console with certain content

Reviewed by Darin Adler.

  • rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::paintMenuListButtonGradients): Return early
if the rectangle is empty. Use RenderStyle::getBorderRadiiForRect() to
adjust the radii in case the sides of the rectangle are smaller than sum
of the radii, so that the gradient background matches the border (which
already uses getBorderRadiiForRect()). Check if bottomGradient is empty
to avoid clipping to an empty path.
(WebCore::RenderThemeMac::paintMenuListButton): Moved a
GraphicContext::save() down to avoid a save/restore imbalance in the
early return case.

  • rendering/RenderThemeSafari.cpp: Made the same changes.

(WebCore::RenderThemeSafari::paintMenuListButtonGradients):
(WebCore::RenderThemeSafari::paintMenuListButton):

LayoutTests: Updated results after fixing
<rdar://problem/7154605> Loading page with 1px wide <select> makes the
toolbar go blank
and
<rdar://problem/7154521> Lots of "<Error>: doClip: empty path." spew in
the console with certain content

Reviewed by Darin Adler.

  • platform/mac/fast/forms/001-expected.checksum:
  • platform/mac/fast/forms/001-expected.png:
  • platform/mac/fast/forms/menulist-narrow-width-expected.checksum:
  • platform/mac/fast/forms/menulist-narrow-width-expected.png:
6:40 AM Changeset in webkit [47569] by jmalonzo@webkit.org
  • 4 edits in trunk

2009-08-20 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Gustavo Noronha.

[Gtk][REGRESSION] XHR test failures after r45558
https://bugs.webkit.org/show_bug.cgi?id=27143

http/tests/xmlhttprequest/small-chunks-response-text.html

  • Enable test fixed by this patch.

http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache.html
http/tests/xmlhttprequest/cache-override.html

  • Enable tests already passing.

Updated patch originally by Gustavo Noronha.

  • platform/gtk/Skipped:

Disable sniffing if it's explicitly requested. Updated patch
originally by Gustavo Noronha.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::statusWillBeHandledBySoup): (WebCore::gotHeadersCallback): (WebCore::contentSniffedCallback): (WebCore::gotChunkCallback): (WebCore::startHttp):
6:20 AM Changeset in webkit [47568] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-08-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Swap the pixels since QImage expect ARGB while ImageData
stores them as RGBA.

This fixes the fast/canvas/canvas-copyPixels.html test.

  • platform/graphics/qt/ImageBufferQt.cpp: (WebCore::putImageData):
6:00 AM Changeset in webkit [47567] by krit@webkit.org
  • 1 edit
    3 adds in trunk/LayoutTests

2009-08-20 Dirk Schulze <krit@webkit.org>

Reviewed by Ariya Hidayat.

Check if the combination of getImageData and the afterward use of putImageData
still gives the right result.

  • fast/canvas/canvas-copyPixels-expected.txt: Added.
  • fast/canvas/canvas-copyPixels.html: Added.
  • fast/canvas/resources/canvas-copyPixels.js: Added.
4:23 AM Changeset in webkit [47566] by ariya@webkit.org
  • 3 edits in trunk/LayoutTests

2009-08-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Rubber-stamped by Simon Hausmann.

[Qt] Of course we pass ACID1 tests. Update the metrics.

  • platform/qt/Skipped:
  • platform/qt/css1/box_properties/acid_test-expected.txt:
4:19 AM Changeset in webkit [47565] by ariya@webkit.org
  • 26 edits in trunk/LayoutTests

2009-08-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Rubber-stamped by Simon Hausmann.

[Qt] Update the (outdated) metrics for css1 borders tests. All tests pass.

  • platform/qt/Skipped: Excluded passed tests.
  • platform/qt/css1/box_properties/border-expected.txt:
  • platform/qt/css1/box_properties/border_bottom-expected.txt:
  • platform/qt/css1/box_properties/border_bottom_inline-expected.txt:
  • platform/qt/css1/box_properties/border_bottom_width-expected.txt:
  • platform/qt/css1/box_properties/border_bottom_width_inline-expected.txt:
  • platform/qt/css1/box_properties/border_color-expected.txt:
  • platform/qt/css1/box_properties/border_color_inline-expected.txt:
  • platform/qt/css1/box_properties/border_inline-expected.txt:
  • platform/qt/css1/box_properties/border_left-expected.txt:
  • platform/qt/css1/box_properties/border_left_inline-expected.txt:
  • platform/qt/css1/box_properties/border_left_width-expected.txt:
  • platform/qt/css1/box_properties/border_left_width_inline-expected.txt:
  • platform/qt/css1/box_properties/border_right-expected.txt:
  • platform/qt/css1/box_properties/border_right_inline-expected.txt:
  • platform/qt/css1/box_properties/border_right_width-expected.txt:
  • platform/qt/css1/box_properties/border_right_width_inline-expected.txt:
  • platform/qt/css1/box_properties/border_style-expected.txt:
  • platform/qt/css1/box_properties/border_style_inline-expected.txt:
  • platform/qt/css1/box_properties/border_top-expected.txt:
  • platform/qt/css1/box_properties/border_top_inline-expected.txt:
  • platform/qt/css1/box_properties/border_top_width-expected.txt:
  • platform/qt/css1/box_properties/border_top_width_inline-expected.txt:
  • platform/qt/css1/box_properties/border_width-expected.txt:
  • platform/qt/css1/box_properties/border_width_inline-expected.txt:
3:55 AM Changeset in webkit [47564] by ariya@webkit.org
  • 9 edits in trunk/LayoutTests

2009-08-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Rubber-stamped by Simon Hausmann.

[Qt] Update the (outdated) metrics for css1/basic tests. All tests pass.

  • platform/qt/Skipped: Excluded passed tests.
  • platform/qt/css1/basic/class_as_selector-expected.txt:
  • platform/qt/css1/basic/comments-expected.txt:
  • platform/qt/css1/basic/containment-expected.txt:
  • platform/qt/css1/basic/contextual_selectors-expected.txt:
  • platform/qt/css1/basic/grouping-expected.txt:
  • platform/qt/css1/basic/id_as_selector-expected.txt:
  • platform/qt/css1/basic/inheritance-expected.txt:
3:40 AM Changeset in webkit [47563] by levin@chromium.org
  • 2 edits in trunk/WebKitTools

bugzilla-tool patch retrieval should handle 302 redirects.
https://bugs.webkit.org/show_bug.cgi?id=28485

Patch by David Levin <levin@chromium.org> on 2009-08-20
Reviewed by David Kilzer.

  • Scripts/modules/scm.py: Pass the --location parameter to curl

so that 302's are followed.

3:13 AM Changeset in webkit [47562] by levin@chromium.org
  • 5 edits in trunk/WebCore

[v8] Use atomic string caching for createElement
Extend atomic string caching to createElement. Fix a bug in idl
parser that caused incorrect parsing if a function argument
attribute list contained a comma.
https://bugs.webkit.org/show_bug.cgi?id=28449

Patch by Christian Plesner Hansen <christian.plesner.hansen@gmail.com> on 2009-08-20
Reviewed by David Levin.

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/IDLParser.pm:
  • bindings/v8/V8Binding.h:

(WebCore::v8ValueToAtomicWebCoreStringWithNullCheck):

  • dom/Document.idl:
3:06 AM Changeset in webkit [47561] by ariya@webkit.org
  • 8 edits in trunk/LayoutTests

2009-08-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Rubber-stamped by Simon Hausmann.

[Qt] Update the metrics after recent refactoring. All tests pass.

  • platform/qt/fast/box-shadow/basic-shadows-expected.txt:
  • platform/qt/fast/box-shadow/transform-fringing-expected.txt:
  • platform/qt/fast/dynamic/genContentDestroyChildren-expected.txt:
  • platform/qt/fast/dynamic/window-resize-scrollbars-test-expected.txt:
  • platform/qt/fast/images/image-in-map-expected.txt:
  • platform/qt/tables/mozilla/bugs/adforce_imgis_com-expected.txt:
  • platform/qt/tables/mozilla_expected_failures/bugs/bug23847-expected.txt:
2:37 AM Changeset in webkit [47560] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-08-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Not reviewed, build fix after r47515.

[Qt] There is no Phonon::MediaObject::hasAudio() function.

  • platform/graphics/qt/MediaPlayerPrivatePhonon.cpp: (WebCore::MediaPlayerPrivate::hasAudio): Assume true.
1:22 AM Changeset in webkit [47559] by levin@chromium.org
  • 2 edits in trunk/WebKitTools

One more speculative build for gtk.

Patch by Aaron Boodman <aa@chromium.org> on 2009-08-20

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:
1:12 AM Changeset in webkit [47558] by levin@chromium.org
  • 5 edits in trunk

WebKit/gtk: Hopefully the last speculative build for gtk.

Patch by Aaron Boodman <aa@chromium.org> on 2009-08-20

  • webkit/webkitprivate.cpp:

(webkit_white_list_access_from_origin):

  • webkit/webkitprivate.h:

WebKitTools: Speculative build for gtk.

Patch by Aaron Boodman <aa@chromium.org> on 2009-08-20

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::whiteListAccessFromOrigin):

12:47 AM Changeset in webkit [47557] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Ignore some leaks that are known to originate from ImageIO.

  • Scripts/run-webkit-tests:
12:29 AM Changeset in webkit [47556] by darin@chromium.org
  • 3 edits in trunk/WebCore

2009-08-20 Darin Fisher <darin@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=28483
The V8 ScriptController's cleanupScriptObjectsForPlugin should take a
Widget* instead of void* to avoid the need for manual casting to Widget*
at the callsite.

  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::cleanupScriptObjectsForPlugin):
  • bindings/v8/ScriptController.h:
12:25 AM Changeset in webkit [47555] by levin@chromium.org
  • 3 edits in trunk/WebKitTools

With David Levin.

Patch by Aaron Boodman <aa@chromium.org> on 2009-08-20
Speculative build fix for qt.

  • DumpRenderTree/qt/jsobjects.cpp:

(LayoutTestController::whiteListAccessFromOrigin):

  • DumpRenderTree/qt/jsobjects.h:
12:14 AM Changeset in webkit [47554] by levin@chromium.org
  • 2 edits in trunk/WebKit/gtk

Another speculative build for gtk.

Patch by David Levin <levin@chromium.org> on 2009-08-20

  • webkit/webkitprivate.cpp:

(webkit_white_list_access_to_origin):

12:02 AM Changeset in webkit [47553] by levin@chromium.org
  • 2 edits in trunk/WebKit/gtk

Speculative build for gtk.

Patch by David Levin <levin@chromium.org> on 2009-08-20

  • webkit/webkitprivate.cpp:

(webkit_white_list_access_to_origin):

Note: See TracTimeline for information about the timeline view.