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

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):
Note: See TracTimeline for information about the timeline view.