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

Timeline



Aug 6, 2009:

11:53 PM WebKit Team edited by jmalonzo@webkit.org
add janm to the team! (diff)
11:53 PM Changeset in webkit [46888] by abarth@webkit.org
  • 5 edits
    4 adds in trunk

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

Reviewed by Adam Barth.

box-shadow's spread is ignored with <table>
​https://bugs.webkit.org/show_bug.cgi?id=28017

Use RenderStyle::getBoxShadowExtent just like RenderBlock.

  • fast/box-shadow/box-shadow-overflow-scroll-expected.txt: Added.
  • fast/box-shadow/box-shadow-overflow-scroll.html: Added.
  • fast/box-shadow/resources/box-shadow-overflow-scroll.js: Added.

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

Reviewed by Adam Barth.

box-shadow's spread is ignored with <table>
​https://bugs.webkit.org/show_bug.cgi?id=28017

Use RenderStyle::getBoxShadowExtent just like RenderBlock.

Test: fast/box-shadow/box-shadow-overflow-scroll.html

  • rendering/RenderTable.cpp: (WebCore::RenderTable::layout):

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

Reviewed by Adam Barth.

box-shadow's spread is ignored with <table>
​https://bugs.webkit.org/show_bug.cgi?id=28017

Use RenderStyle::getBoxShadowExtent just like RenderBlock.

  • Scripts/make-js-test-wrappers: Added regexp to skip box-shadow-overflow-scroll.js
11:44 PM HackingGtk edited by jmalonzo@webkit.org
Add l10n, docs and tests folder; Removed unused backends and … (diff)
11:14 PM Changeset in webkit [46887] by jmalonzo@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

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

Reviewed by Xan Lopez.

[Gtk] Copy expected (txt only) result from Mac for
fast/images/icon-decoding.html from r46871 since we seem to pass this
test as well.

  • platform/gtk/fast/images/icon-decoding-expected.txt: Copied from LayoutTests/platform/mac/fast/images/icon-decoding-expected.txt.
10:51 PM Changeset in webkit [46886] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2009-08-06 Joseph Pecoraro <​joepeck02@gmail.com>

Reviewed by Timothy Hatcher.

Inspector: NodeLists Don't Display Well in the Console
​https://bugs.webkit.org/show_bug.cgi?id=28061

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._format): Specific check if the object is a NodeList, display as an "array"
10:39 PM Changeset in webkit [46885] by abarth@webkit.org
  • 4 edits in trunk/WebCore

2009-08-06 Yusuke Sato <​yusukes@chromium.org>

Reviewed by David Levin.

Chromium Linux: add support for @font-face
​https://bugs.webkit.org/show_bug.cgi?id=28015

  • loader/CachedFont.cpp: (WebCore::CachedFont::~CachedFont): (WebCore::CachedFont::ensureCustomFontData): (WebCore::CachedFont::platformDataFromCustomData): (WebCore::CachedFont::allClientsRemoved):

Modified #ifdefs so that chromium linux can load remote fonts.

  • platform/graphics/chromium/FontCustomPlatformData.cpp: (WebCore::FontCustomPlatformData::~FontCustomPlatformData): (WebCore::FontCustomPlatformData::fontPlatformData): (WebCore::createFontCustomPlatformData):

Create SkTypeface for a web font by calling SkTypeface::CreateFromStream function.

(WebCore::RemoteFontStream::RemoteFontStream):
(WebCore::RemoteFontStream::~RemoteFontStream):
(WebCore::RemoteFontStream::rewind):
(WebCore::RemoteFontStream::read):

New class that implements SkStream interface and wraps wtf's SharedBuffer.

  • platform/graphics/chromium/FontCustomPlatformData.h: (WebCore::FontCustomPlatformData::FontCustomPlatformData):

Added member variables for Linux.

10:09 PM Changeset in webkit [46884] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2009-08-06 Yong Li <​yong.li@torchmobile.com>

Reviewed by Eric Seidel.

Fix JPEGImageDecoder.cpp compile error on WinCE
​https://bugs.webkit.org/show_bug.cgi?id=28051

  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp:
9:45 PM Changeset in webkit [46883] by kevino@webkit.org
  • 1 edit
    1 add in trunk/WebKit/wx

Reviewed by Eric Seidel.

Adding support for building wx Python bindings using the waf build system.

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

9:24 PM Changeset in webkit [46882] by kevino@webkit.org
  • 1 edit
    1 add in trunk/WebCore

Reviewed by Eric Seidel.

Adding WebCore support for the waf build system for wx.

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

9:11 PM Changeset in webkit [46881] by abarth@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-08-06 Norbert Leser <​norbert.leser@nokia.com>

Reviewed by Darin Adler.

Updated patch for bug #27059:
Symbian platform always uses little endian encoding,
regardless of compiler.
We need to make sure that we correctly detect EABI architecture
for armv5 targets on Symbian,
where EABI is set but not ARM_EABI

  • wtf/Platform.h:
8:30 PM Changeset in webkit [46880] by abarth@webkit.org
  • 7 edits in trunk

JavaScriptCore:

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

Unreviewed revert.

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

Revert 46877 because it broke GTK.

  • wtf/Noncopyable.h:

WebCore:

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

Unreviewed revert.

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

Revert 46877 because it broke GTK.

  • platform/graphics/wince/GraphicsContextWince.cpp:

WebKit/win:

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

Unreviewed revert.

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

Revert 46877 because it broke GTK.

  • COMPropertyBag.h:
8:05 PM Changeset in webkit [46879] by barraclough@apple.com
  • 14 edits in trunk/JavaScriptCore

2009-08-06 Gavin Barraclough <​barraclough@apple.com>

Reviewed by Oliver Hunt.

Make get_by_id/put_by_id/method_check/call defer optimization using a data flag rather than a code modification.
( ​https://bugs.webkit.org/show_bug.cgi?id=27635 )

This improves performance of ENABLE(ASSEMBLER_WX_EXCLUSIVE) builds by 2-2.5%, reducing the overhead to about 2.5%.
(No performance impact with ASSEMBLER_WX_EXCLUSIVE disabled).

  • bytecode/CodeBlock.cpp: (JSC::printStructureStubInfo):
    • Make StructureStubInfo store the type as an integer, rather than an OpcodeID.
  • bytecode/CodeBlock.h: (JSC::): (JSC::CallLinkInfo::seenOnce): (JSC::CallLinkInfo::setSeen): (JSC::MethodCallLinkInfo::seenOnce): (JSC::MethodCallLinkInfo::setSeen):
    • Change a pointer in CallLinkInfo/MethodCallLinkInfo to use a PtrAndFlags, use a flag to track when an op has been executed once.
  • bytecode/StructureStubInfo.cpp: (JSC::StructureStubInfo::deref):
    • Make StructureStubInfo store the type as an integer, rather than an OpcodeID.
  • bytecode/StructureStubInfo.h: (JSC::StructureStubInfo::StructureStubInfo): (JSC::StructureStubInfo::initGetByIdSelf): (JSC::StructureStubInfo::initGetByIdProto): (JSC::StructureStubInfo::initGetByIdChain): (JSC::StructureStubInfo::initGetByIdSelfList): (JSC::StructureStubInfo::initGetByIdProtoList): (JSC::StructureStubInfo::initPutByIdTransition): (JSC::StructureStubInfo::initPutByIdReplace): (JSC::StructureStubInfo::seenOnce): (JSC::StructureStubInfo::setSeen):
    • Make StructureStubInfo store the type as an integer, rather than an OpcodeID, add a flag to track when an op has been executed once.
  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitGetById): (JSC::BytecodeGenerator::emitPutById):
    • Make StructureStubInfo store the type as an integer, rather than an OpcodeID.
  • jit/JIT.cpp: (JSC::JIT::privateCompileCTIMachineTrampolines): (JSC::JIT::unlinkCall):
    • Remove the "don't lazy link" stage of calls.
  • jit/JIT.h: (JSC::JIT::compileCTIMachineTrampolines):
    • Remove the "don't lazy link" stage of calls.
  • jit/JITCall.cpp: (JSC::JIT::compileOpCallSlowCase):
    • Remove the "don't lazy link" stage of calls.
  • jit/JITStubs.cpp: (JSC::JITThunks::JITThunks): (JSC::JITThunks::tryCachePutByID): (JSC::JITThunks::tryCacheGetByID): (JSC::JITStubs::DEFINE_STUB_FUNCTION): (JSC::JITStubs::getPolymorphicAccessStructureListSlot):
    • Remove the "don't lazy link" stage of calls, and the "_second" stage of get_by_id/put_by_id/method_check.
  • jit/JITStubs.h: (JSC::JITThunks::ctiStringLengthTrampoline): (JSC::JITStubs::):
    • Remove the "don't lazy link" stage of calls, and the "_second" stage of get_by_id/put_by_id/method_check.
  • wtf/PtrAndFlags.h: (WTF::PtrAndFlags::PtrAndFlags): (WTF::PtrAndFlags::operator!): (WTF::PtrAndFlags::operator->):
    • Add ! and -> operators, add constuctor with pointer argument.
6:56 PM Changeset in webkit [46878] by Chris Fleizach
  • 7 edits in trunk/WebCore

2009-08-06 Chris Fleizach <Chris Fleizach>

Reviewed by Eric Seidel.

Bug 27956 - AX: roleValue should be cached for performance
​https://bugs.webkit.org/show_bug.cgi?id=27956

Caches the role value of an accessibility object instead of calculating it everytime.
The changes gained a 5% speedup using VoiceOver to navigate a webpage.

  • accessibility/AXObjectCache.cpp: (WebCore::AXObjectCache::handleAriaRoleChanged):

Use generic role setter instead of specific ARIA role setter.

  • accessibility/AccessibilityObject.cpp: (WebCore::AccessibilityObject::AccessibilityObject):
  • accessibility/AccessibilityObject.h: (WebCore::AccessibilityObject::headingLevel):

Make headingLevel into instance method so that it can *safely* be called in the constructor.

(WebCore::AccessibilityObject::setRoleValue):
(WebCore::AccessibilityObject::roleValue):

  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::AccessibilityRenderObject): (WebCore::AccessibilityRenderObject::headingLevel): (WebCore::AccessibilityRenderObject::intValue): (WebCore::AccessibilityRenderObject::updateAccessibilityRole): (WebCore::AccessibilityRenderObject::determineAccessibilityRole):
  • accessibility/AccessibilityRenderObject.h:
  • accessibility/mac/AccessibilityObjectWrapper.mm: (AXAttributeStringSetHeadingLevel):
6:09 PM Changeset in webkit [46877] by abarth@webkit.org
  • 6 edits in trunk

2009-08-06 Zoltan Horvath <​hzoltan@inf.u-szeged.hu>

Reviewed by Adam Barth.

Allow custom memory allocation control for Noncopyable class
​https://bugs.webkit.org/show_bug.cgi?id=27879

Several classes which inherited from Noncopyable are instantiated by
operator new, so Noncopyable class has been inherited from FastAllocBase.

  • wtf/Noncopyable.h:

2009-08-06 Zoltan Horvath <​hzoltan@inf.u-szeged.hu>

Reviewed by Adam Barth.

Change Noncopyable inheriting to public
​http://bugs.webkit.org/show_bug.cgi?id=27879

Since Noncopyable is inherited from FastAllocBase, Noncopyable's
inheriting has been changed to public.

  • platform/graphics/wince/GraphicsContextWince.cpp:

2009-08-06 Zoltan Horvath <​hzoltan@inf.u-szeged.hu>

Reviewed by Adam Barth.

Change Noncopyable inheriting to public
​http://bugs.webkit.org/show_bug.cgi?id=27879

Since Noncopyable is inherited from FastAllocBase,
Noncopyable's inheriting has been changed to public.

  • COMPropertyBag.h:
5:59 PM Changeset in webkit [46876] by mrowe@apple.com
  • 8 edits in trunk

Add explicit dependencies for our build verification scripts to ensure that they always run after linking has completed.

Rubber-stamped by Sam Weinig.

5:59 PM Changeset in webkit [46875] by Simon Fraser
  • 2 edits in trunk/LayoutTests

2009-08-06 Simon Fraser <Simon Fraser>

Make the test a little more sloppy to try to fix sporadic failures on Windows.

  • animations/play-state.html:
5:32 PM Changeset in webkit [46874] by Simon Fraser
  • 17 edits
    2 copies
    3 adds in trunk

2009-08-05 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Implement the matchMedium method on the Media interface described in the CSSOM View Module.
​https://bugs.webkit.org/show_bug.cgi?id=22495

Add a new Media interface, obtainable via the AbstractView, that can
evaluate queries via its matchesMedium() method.

Test: fast/media/matchmedium-query-api.html

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl: Add Media.* files to the build.
  • css/Media.cpp: Added. (WebCore::Media::Media): (WebCore::Media::type): (WebCore::Media::matchMedium):
  • css/Media.h: Added. (WebCore::Media::create): (WebCore::Media::document):
  • css/Media.idl: Added. Media::matchMedium() uses a MediaQueryEvaluator() to test the query.
  • page/AbstractView.idl:
  • page/DOMWindow.cpp: (WebCore::DOMWindow::media):
  • page/DOMWindow.h:
  • page/DOMWindow.idl: Add a readonly attribute for Media.
5:28 PM Changeset in webkit [46873] by Darin Adler
  • 3 edits in trunk/LayoutTests

2009-08-06 Darin Adler <Darin Adler>

Reviewed by Mark Rowe.

  • fast/dom/gc-9-expected.txt: Generated new results.
  • fast/dom/gc-9.html: Turned off tests that are unpredictable based on what happens during garbage collection. These do represent bugs, so we want to turn them back on again once the bugs are fixed.
5:27 PM Changeset in webkit [46872] by pkasting@chromium.org
  • 2 edits in trunk/WebKitLibraries

2009-08-06 Peter Kasting <​pkasting@google.com>

Reviewed by Adam Barth.

​https://bugs.webkit.org/show_bug.cgi?id=27323
Strip line endings at all points auto-version.sh reads data, not just
the one I happened to run into.

  • win/tools/scripts/auto-version.sh:
5:25 PM Changeset in webkit [46871] by pkasting@chromium.org
  • 1 edit
    8 adds in trunk/LayoutTests

2009-08-06 Peter Kasting <​pkasting@google.com>

Reviewed by Eric Seidel.

​https://bugs.webkit.org/show_bug.cgi?id=28025
Add a test for .ico decoding.

  • fast/images/icon-decoding.html: Added.
  • fast/images/resources/1bit.ico: Added.
  • fast/images/resources/2entries.ico: Added.
  • fast/images/resources/32bit.ico: Added.
  • fast/images/resources/8bit.ico: Added.
  • platform/mac/fast/images/icon-decoding-expected.checksum: Added.
  • platform/mac/fast/images/icon-decoding-expected.png: Added.
  • platform/mac/fast/images/icon-decoding-expected.txt: Added.
4:46 PM Changeset in webkit [46870] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-08-06 Anders Carlsson <​andersca@apple.com>

Reviewed by Darin Adler and Dan Bernstein.

  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::hasHTMLView): Always return true when we're in viewless mode.
4:45 PM Changeset in webkit [46869] by abarth@webkit.org
  • 12 edits
    8 adds in trunk

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

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=27455
Test suite for form controls' custom validation message.

  • fast/dom/domListEnumeration-expected.txt: Corrected
  • fast/dom/resources/domListEnumeration.js: Corrected
  • fast/forms/ValidityState-customError-001-expected.txt: Added.
  • fast/forms/ValidityState-customError-001.html: Added.
  • fast/forms/ValidityState-customError-002-expected.txt: Added.
  • fast/forms/ValidityState-customError-002.html: Added.
  • fast/forms/ValidityState-customError-003-expected.txt: Added.
  • fast/forms/ValidityState-customError-003.html: Added.
  • fast/forms/ValidityState-customError-004-expected.txt: Added.
  • fast/forms/ValidityState-customError-004.html: Added.

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

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=27455
Support for a custom validation message in ValidityState and related
form controls.
​http://www.whatwg.org/specs/web-apps/current-work/multipage/forms.html#dom-cva-setcustomvalidity

Tests: fast/forms/ValidityState-customError-001.html

fast/forms/ValidityState-customError-002.html
fast/forms/ValidityState-customError-003.html
fast/forms/ValidityState-customError-004.html

  • html/HTMLButtonElement.idl: setCustomValidity DOM method
  • html/HTMLFieldSetElement.idl: ditto
  • html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElement::setCustomValidity): ditto
  • html/HTMLFormControlElement.h: (WebCore::HTMLFormControlElement::valueMissing): moved down
  • html/HTMLInputElement.idl: ditto
  • html/HTMLSelectElement.idl: ditto
  • html/HTMLTextAreaElement.idl:
  • html/ValidityState.h: (WebCore::ValidityState::setCustomErrorMessage): method to set custom validation message (WebCore::ValidityState::customError): validation flag
4:00 PM Changeset in webkit [46868] by mrowe@apple.com
  • 6 edits in trunk

Bring a little order to our otherwise out of control lives.

3:56 PM Changeset in webkit [46867] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Stop copying .idl files in to the framework resources directory.

  • WebCore.xcodeproj/project.pbxproj:
3:51 PM Changeset in webkit [46866] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-08-06 Anders Carlsson <​andersca@apple.com>

Remove WebGraphicsExtras.h include.


  • Plugins/WebNetscapePluginView.mm:
3:51 PM Changeset in webkit [46865] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix the build some more.

Add JSWebSocket.cpp and JSWebSocket.h to the Xcode project, and move the WebSocket-related
files and groups in to the appropriate locations.

  • WebCore.xcodeproj/project.pbxproj:
3:38 PM Changeset in webkit [46864] by mrowe@apple.com
  • 2 edits in trunk/WebKit

Build fix.

Don't attempt to build a file that Anders just deleted.

  • WebKit.xcodeproj/project.pbxproj:
3:30 PM Changeset in webkit [46863] by jorlow@chromium.org
  • 4 edits
    7 adds in trunk/WebCore

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

Reviewed by Alexey Proskuryakov.

Add WebSocket.idl
​https://bugs.webkit.org/show_bug.cgi?id=27209

Add WebSocket.idl in WebCore/websockets/.
Add build systems only for GNUmakefile.am and WebCore.xcodeproj now.
Other build systems will be updated once the code is functional.

  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSWebSocketConstructor.cpp: Added.
  • bindings/js/JSWebSocketConstructor.h: Added.
  • bindings/js/JSWebSocketCustom.cpp: Added.
  • websockets/WebSocket.cpp: Added.
  • websockets/WebSocket.h: Added.
  • websockets/WebSocket.idl: Added.
3:08 PM Changeset in webkit [46862] by andersca@apple.com
  • 1 edit
    2 deletes in trunk/WebKit/mac

2009-08-06 Anders Carlsson <​andersca@apple.com>

Reviewed by Sam Weinig.

Remove WebGraphicsExtras, it is no longer used.

  • Misc/WebGraphicsExtras.c: Removed.
  • Misc/WebGraphicsExtras.h: Removed.
2:57 PM Changeset in webkit [46861] by Simon Fraser
  • 2 edits in trunk/LayoutTests

2009-08-06 Simon Fraser <Simon Fraser>

Fix a test in http/tests/media that was missed in the earlier 'media' rename.

  • http/tests/media/video-play-stall.html:
2:41 PM Changeset in webkit [46860] by rniwa@webkit.org
  • 2 edits
    1 add
    4 deletes in trunk/LayoutTests

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

Reviewed by Eric Seidel.

LayoutTests/editing/execCommand/boldSelection.html must be a dumpAsText test
​https://bugs.webkit.org/show_bug.cgi?id=28055

This patch converts boldSelection.html to a dumpAsText and adds description on what it tests.
Namely, it tests bolding the first 21 letters of a sentence.
Since bolding must be done by adding b tag instead of a style span, we print innerHTML of the result.

  • editing/execCommand/boldSelection-expected.txt: Added.
  • editing/execCommand/boldSelection.html:
  • platform/mac/editing/execCommand/boldSelection-expected.checksum: Removed.
  • platform/mac/editing/execCommand/boldSelection-expected.png: Removed.
  • platform/mac/editing/execCommand/boldSelection-expected.txt: Removed.
  • platform/qt/editing/execCommand/boldSelection-expected.txt: Removed.
2:38 PM BuildingQtOnWindows edited by robert@roberthogan.net
(diff)
2:29 PM Changeset in webkit [46859] by jianli@chromium.org
  • 10 edits in trunk/WebCore

2009-08-06 Jian Li <​jianli@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Style cleaning for WorkerContextExecutionProxy.
​https://bugs.webkit.org/show_bug.cgi?id=27997

Cleanup WorkerContextExecutioonProxy related files to follow
WebKit coding styles after all V8 binding codes are upstreamed.
1) Lower-case the first letter of remaining functins in class

WorkerContextExecutionProxy. Update the V8 code generator
and all other references accordingly.

2) Integrate getConstructor from WorkerContextExecutionProxy

to V8DOMWrapper.

3) Other misc changes, like removing trailing whitespaces and

sorting the include files.

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/ScheduledAction.cpp: (WebCore::ScheduledAction::execute):
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getConstructorForContext): (WebCore::V8DOMWrapper::convertToV8Object):
  • bindings/v8/V8DOMWrapper.h:
  • bindings/v8/V8WorkerContextEventListener.cpp: (WebCore::V8WorkerContextEventListener::~V8WorkerContextEventListener): (WebCore::V8WorkerContextEventListener::handleEvent): (WebCore::V8WorkerContextEventListener::reportError): (WebCore::V8WorkerContextEventListener::getReceiverObject):
  • bindings/v8/V8WorkerContextObjectEventListener.cpp: (WebCore::weakObjectEventListenerCallback):
  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::handleConsoleMessage): (WebCore::WorkerContextExecutionProxy::dispose): (WebCore::WorkerContextExecutionProxy::initContextIfNeeded): (WebCore::WorkerContextExecutionProxy::convertToV8Object): (WebCore::WorkerContextExecutionProxy::convertEventToV8Object): (WebCore::WorkerContextExecutionProxy::convertEventTargetToV8Object): (WebCore::WorkerContextExecutionProxy::convertWorkerContextToV8Object): (WebCore::WorkerContextExecutionProxy::toV8): (WebCore::WorkerContextExecutionProxy::forgetV8EventObject): (WebCore::WorkerContextExecutionProxy::removeEventListener):
  • bindings/v8/WorkerContextExecutionProxy.h: (WebCore::WorkerContextExecutionProxy::context): (WebCore::WorkerContextExecutionProxy::convertToV8Object):
  • bindings/v8/custom/V8WorkerContextCustom.cpp: (WebCore::ACCESSOR_GETTER): (WebCore::CALLBACK_FUNC_DECL):
2:21 PM BuildingQtOnWindows edited by robert@roberthogan.net
(diff)
2:15 PM Changeset in webkit [46858] by Simon Fraser
  • 1 edit
    2 moves in trunk/LayoutTests

2009-08-06 Simon Fraser <Simon Fraser>

Rubber-stamped by Dan Bernstein.

Move a video test from fast/media to media/, since fast/media is all about
CSS media queries.

  • media/video-controls-with-mutation-event-handler-expected.txt: Renamed from LayoutTests/fast/media/video-controls-with-mutation-event-handler-expected.txt.
  • media/video-controls-with-mutation-event-handler.html: Renamed from LayoutTests/fast/media/video-controls-with-mutation-event-handler.html.
2:09 PM Changeset in webkit [46857] by Simon Fraser
  • 29 edits in trunk/LayoutTests

2009-08-06 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

​https://bugs.webkit.org/show_bug.cgi?id=28056
Change media tests to not use a variable in global scope with the name
'media', to avoid conflict with window.media.

  • media/audio-constructor-autobuffer-expected.txt:
  • media/audio-constructor-autobuffer.html:
  • media/audio-constructor-src.html:
  • media/audio-constructor.html:
  • media/audio-data-url-expected.txt:
  • media/audio-data-url.html:
  • media/audio-mpeg-supported-expected.txt:
  • media/audio-mpeg-supported.html:
  • media/media-load-event-expected.txt:
  • media/media-load-event.html:
  • media/video-can-play-type.html:
  • media/video-loop-expected.txt:
  • media/video-loop.html:
  • media/video-seek-past-end-paused-expected.txt:
  • media/video-seek-past-end-paused.html:
  • media/video-seek-past-end-playing-expected.txt:
  • media/video-seek-past-end-playing.html:
  • media/video-source-media.html:
  • media/video-source-type-params.html:
  • media/video-source-type.html:
  • media/video-source.html:
  • media/video-src-invalid-remove.html:
  • media/video-src-plus-source-expected.txt:
  • media/video-src-plus-source.html:
  • media/video-src-source.html:
  • media/video-src.html:
  • media/video-test.js: (findMediaElement): (waitForEvent): (waitForEventAndTest): (relativeURL):
  • media/video-timeupdate-during-playback.html:
1:50 PM Changeset in webkit [46856] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

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

No review, only changing make-js-test-wrappers.

Fix make-js-test-wrappers to ignore a few more js tests
with custom templates.

  • Scripts/make-js-test-wrappers:
1:20 PM Changeset in webkit [46855] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

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

No review, only changing make-js-test-wrappers.

Fix make-js-test-wrappers to ignore a few more js tests
with custom templates.

  • Scripts/make-js-test-wrappers:
12:17 PM Changeset in webkit [46854] by abarth@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-08-06 Zoltan Horvath <​hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

Allow custom memory allocation control for JavaScriptCore's PolymorphicAccessStructureList struct
​https://bugs.webkit.org/show_bug.cgi?id=27877

Inherits PolymorphicAccessStructureList struct from FastAllocBase because it has been instantiated by
'new' in JavaScriptCore/jit/JITStubs.cpp:1229.

  • bytecode/Instruction.h:
11:49 AM Changeset in webkit [46853] by bfulgham@webkit.org
  • 2 edits in trunk/PlanetWebKit

2009-08-06 Brent Fulgham <​bfulgham@webkit.org>

Rubber-stamped by Adam Barth.

Update my blog's address.

  • config.ini: Update Blog address.
11:25 AM Changeset in webkit [46852] by atwilson@chromium.org
  • 9 edits
    5 copies
    9 adds in trunk/LayoutTests

2009-08-06 Drew Wilson <​atwilson@google.com>

Reviewed by David Levin.

Refactored worker tests to share code with shared worker versions. Shared worker tests are currently disabled by default.

Worker layout tests should be refactored to enable testing SharedWorkers also.
​https://bugs.webkit.org/show_bug.cgi?id=27963

  • fast/workers/resources/shared-worker-common.js: (onconnect): (gc): (handleMessage):
  • fast/workers/resources/shared-worker-create-common.js: Added. (createWorker.worker.port.onmessage): (createWorker.worker.postMessage): (createWorker):
  • fast/workers/resources/worker-context-gc.js: Added. (log): (worker.onmessage):
  • fast/workers/resources/worker-create-common.js: Added. (createWorker):
  • fast/workers/resources/worker-location.js: Added. (log): (gc): (worker.onmessage):
  • fast/workers/resources/worker-navigator.js: Added. (log): (worker.onmessage):
  • fast/workers/resources/worker-replace-global-constructor.js:
  • fast/workers/resources/worker-replace-self.js: Added. (log): (worker.onmessage):
  • fast/workers/shared-worker-context-gc-expected.txt: Added.
  • fast/workers/shared-worker-context-gc.html-disabled: Added.
  • fast/workers/shared-worker-location-expected.txt: Added.
  • fast/workers/shared-worker-location.html-disabled: Added.
  • fast/workers/shared-worker-navigator-expected.txt: Added.
  • fast/workers/shared-worker-navigator.html-disabled: Added.
  • fast/workers/shared-worker-replace-global-constructor.html-disabled:
  • fast/workers/shared-worker-replace-self-expected.txt: Added.
  • fast/workers/shared-worker-replace-self.html-disabled: Added.
  • fast/workers/worker-context-gc.html:
  • fast/workers/worker-location.html:
  • fast/workers/worker-navigator.html:
  • fast/workers/worker-replace-global-constructor.html:
  • fast/workers/worker-replace-self.html:
11:02 AM Changeset in webkit [46851] by Darin Adler
  • 2 edits in trunk/WebCore

2009-08-06 Darin Adler <Darin Adler>

Fix Mac Leopard debug build.

  • platform/text/StringImpl.cpp: (WebCore::equalIgnoringCase): Removed meaningless assertion; an unsigned is always >= 0 and checking causes a warning in the newer versions of gcc.
9:43 AM Changeset in webkit [46850] by abarth@webkit.org
  • 10 edits
    4 deletes in trunk

WebCore:

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

Unreview rollout.

Revert 46840 because it broke the reliability tests.

  • bindings/v8/ScheduledAction.cpp: (WebCore::ScheduledAction::ScheduledAction): (WebCore::ScheduledAction::execute):
  • bindings/v8/ScheduledAction.h: (WebCore::ScheduledAction::ScheduledAction):
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::V8Custom::WindowSetTimeoutImpl):
  • bindings/v8/custom/V8WorkerContextCustom.cpp: (WebCore::SetTimeoutOrInterval):

LayoutTests:

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

Unreviewed rollout.

Revert 46840 because it broke the reliability tests.

  • http/tests/security/isolatedWorld/window-setTimeout-function-expected.txt: Removed.
  • http/tests/security/isolatedWorld/window-setTimeout-function.html: Removed.
  • http/tests/security/isolatedWorld/window-setTimeout-string-expected.txt: Removed.
  • http/tests/security/isolatedWorld/window-setTimeout-string.html: Removed.
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
9:43 AM Changeset in webkit [46849] by abarth@webkit.org
  • 3 edits in trunk/WebCore

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

Unreviewed rollout.

Revert 46840 in preparation to revert 46838, which broke the reliability tests.

  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::updateDocumentWrapperCache): (WebCore::V8Proxy::initContextIfNeeded): (WebCore::V8Proxy::mainWorldContext):
  • bindings/v8/V8Proxy.h: (WebCore::V8Proxy::context):
9:37 AM Changeset in webkit [46848] by pfeldman@chromium.org
  • 5 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

WebInspector: move style-related utilities into InjectedScript.

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

  • inspector/front-end/DOMAgent.js: (WebInspector.CSSStyleDeclaration): (WebInspector.CSSStyleDeclaration.prototype.isPropertyImplicit): (WebInspector.CSSStyleDeclaration.prototype.styleTextWithShorthands): (WebInspector.CSSStyleDeclaration.prototype.getLonghandProperties): (WebInspector.CSSStyleDeclaration.prototype.getShorthandPriority):
  • inspector/front-end/InjectedScript.js: (InjectedScript.applyStyleText): (InjectedScript.toggleStyleEnabled): (InjectedScript._serializeStyle): (InjectedScript._getUniqueStyleProperties): (InjectedScript._getLonghandProperties): (InjectedScript._getShorthandPriority):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype._update): (WebInspector.StylePropertiesSection.prototype.isPropertyOverloaded): (WebInspector.StylePropertiesSection.prototype.onpopulate): (WebInspector.StylePropertyTreeElement.prototype.get priority): (WebInspector.StylePropertyTreeElement.prototype.get value): (WebInspector.StylePropertyTreeElement.prototype):
  • inspector/front-end/utilities.js:
8:58 AM Changeset in webkit [46847] by Simon Hausmann
  • 4 edits in trunk/WebCore

2009-08-06 Simon Hausmann <​simon.hausmann@nokia.com>

Unreviewed build fix, add #ifdef guards around
inclusions of shared worker specific files.

  • bindings/js/JSEventTarget.cpp:
  • bindings/js/JSWorkerContextBase.cpp:
  • bindings/js/WorkerScriptController.cpp:
8:57 AM Changeset in webkit [46846] by Simon Hausmann
  • 4 edits in trunk/WebCore

2009-08-06 Laszlo Gombos <Laszlo Gombos>

Reviewed by Darin Adler.

cssgrammar.cpp fails to compile with RVCT compiler
​https://bugs.webkit.org/show_bug.cgi?id=27952

  • css/CSSGrammar.y: Do not convert to String to get rid of the stray memory allocation
  • platform/text/StringImpl.cpp: (WebCore::equalIgnoringCase):
  • platform/text/StringImpl.h: (WebCore::equalIgnoringCase): Add charactersEqualIgnoringCase function that works with a character pointer and length
8:30 AM Changeset in webkit [46845] by atwilson@chromium.org
  • 30 edits
    9 copies
    13 adds in trunk

WebCore:

2009-08-01 Drew Wilson <​atwilson@google.com>

Reviewed by NOBODY (OOPS!).

Created first working implementation of SharedWorkers (execution only, no sharing).
​https://bugs.webkit.org/show_bug.cgi?id=27927

Added initial implementations of SharedWorkerThread and SharedWorkerContext.
No v8 bindings yet.

  • DerivedSources.cpp: Added shared worker files.
  • DerivedSources.make: Added shared worker files.
  • GNUmakefile.am: Added shared worker files.
  • WebCore.gypi: Added shared worker files.
  • WebCore.pro: Added shared worker files.
  • WebCore.vcproj/WebCore.vcproj: Added shared worker files.
  • WebCore.xcodeproj/project.pbxproj: Added shared worker files.
  • bindings/js/JSEventTarget.cpp: (WebCore::toJS): Added code to convert from EventTarget to correct JS class. (WebCore::toEventTarget): Added code to cast from JS object to appropriate EventTarget impl class.
  • bindings/js/JSSharedWorkerConstructor.cpp: (WebCore::JSSharedWorkerConstructor::JSSharedWorkerConstructor): (WebCore::constructSharedWorker): Tweaked the constructor code to pass in the lexical global object like normal workers.
  • bindings/js/JSSharedWorkerContextCustom.cpp: Added. (WebCore::JSSharedWorkerContext::mark): Custom marking of the onconnect handler.
  • bindings/js/JSWorkerContextBase.cpp: (WebCore::toJSSharedWorkerContext): (WebCore::toJSWorkerContext): Added proper conversion to the correct WorkerContext derived class.
  • bindings/js/JSWorkerContextBase.h:
  • bindings/js/WorkerScriptController.cpp: (WebCore::WorkerScriptController::initScript): Updated WorkerScriptController to create the correct type of binding object (JSSharedWorkerContext vs JSDedicatedWorkerContext)
  • bindings/scripts/CodeGeneratorV8.pm: Added support for SharedWorkerContext.
  • bindings/v8/DOMObjectsInclude.h: Added shared worker bindings files.
  • dom/EventNames.h: Added connect event.
  • dom/EventTarget.cpp: (WebCore::EventTarget::toSharedWorkerContext): Added API for casting to new derived class (SharedWorkerContext).
  • dom/EventTarget.h:
  • workers/AbstractWorker.cpp: (WebCore::AbstractWorker::resolveURL): Moved code that resolves/validates URLs to base class so it can be shared between dedicated/shared workers.
  • workers/AbstractWorker.h:
  • workers/DedicatedWorkerContext.cpp: (WebCore::DedicatedWorkerContext::logException): Refactored exception handling code - moved onerror handling to base class.
  • workers/DedicatedWorkerContext.h: (WebCore::DedicatedWorkerContext::isDedicatedWorkerContext):
  • workers/SharedWorker.cpp: (WebCore::SharedWorker::SharedWorker): Changed constructor to fire up worker thread.
  • workers/SharedWorker.h:
  • workers/SharedWorkerContext.cpp: Added. (WebCore::SharedWorkerContext::SharedWorkerContext): (WebCore::SharedWorkerContext::~SharedWorkerContext): (WebCore::SharedWorkerContext::logException): (WebCore::SharedWorkerContext::addMessage): Placeholder methods until we add support for sending exceptions/messages to console. (WebCore::SharedWorkerContext::dispatchConnect): (WebCore::SharedWorkerContext::thread):
  • workers/SharedWorkerContext.h: Added. (WebCore::SharedWorkerContext::create): (WebCore::SharedWorkerContext::isSharedWorkerContext): (WebCore::SharedWorkerContext::toSharedWorkerContext): (WebCore::SharedWorkerContext::setOnconnect): (WebCore::SharedWorkerContext::onconnect): (WebCore::SharedWorkerContext::name):
  • workers/SharedWorkerContext.idl: Added.
  • workers/SharedWorkerRepository.h: Added. (WebCore::SharedWorkerRepository::~SharedWorkerRepository):
  • workers/DefaultSharedWorkerRepository.cpp: Added. Implementation of core singleton class that will be used to implement sharing. (WebCore::SharedWorkerProxy::setThread): Proxy object used to handle resource loading for a given shared worker. (WebCore::SharedWorkerProxy::postTaskToLoader): (WebCore::SharedWorkerProxy::postTaskForModeToWorkerContext): Placeholder routines until we implement loading. (WebCore::SharedWorkerConnectTask::create): Creates a task to fire off a connect event on the worker thread. (WebCore::SharedWorkerConnectTask::SharedWorkerConnectTask): (WebCore::SharedWorkerConnectTask::performTask): (WebCore::SharedWorkerLoader::SharedWorkerLoader): Added helper object to load the initial script for the worker. (WebCore::SharedWorkerLoader::load): (WebCore::SharedWorkerLoader::notifyFinished): (WebCore::SharedWorkerRepository::instance): (WebCore::DefaultSharedWorkerRepository::instance): (WebCore::DefaultSharedWorkerRepository::workerScriptLoaded): Fires off the worker thread once the script is loaded. (WebCore::SharedWorkerRepository::connect): (WebCore::DefaultSharedWorkerRepository::DefaultSharedWorkerRepository): (WebCore::DefaultSharedWorkerRepository::~DefaultSharedWorkerRepository):
  • workers/DefaultSharedWorkerRepository.h: Added.
  • workers/SharedWorkerThread.cpp: Added. (WebCore::SharedWorkerThread::create): (WebCore::SharedWorkerThread::SharedWorkerThread): (WebCore::SharedWorkerThread::~SharedWorkerThread): (WebCore::SharedWorkerThread::createWorkerContext):
  • workers/SharedWorkerThread.h: Added.
  • workers/Worker.cpp: (WebCore::Worker::Worker):
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::reportException):
  • workers/WorkerContext.h: (WebCore::WorkerContext::isSharedWorkerContext): (WebCore::WorkerContext::isDedicatedWorkerContext): Added APIs to determine the type of a given context.

LayoutTests:

2009-08-05 Drew Wilson <​atwilson@google.com>

Reviewed by NOBODY (OOPS!).

SharedWorkers should instantiate a thread and a SharedWorkerContext
​https://bugs.webkit.org/show_bug.cgi?id=27927

Added disabled tests for SharedWorker minimal functionality.

Began refactoring of dedicated worker tests to allow sharing test cases between shared and dedicated worker (will continue in another patch).

  • fast/workers/resources/shared-worker-common.js: Added.
  • fast/workers/resources/shared-worker-script-error.js: Added.
  • fast/workers/resources/worker-replace-global-constructor.js: Moved common code into shared .js file.
  • fast/workers/shared-worker-constructor.html-disabled:
  • fast/workers/shared-worker-gc-expected.txt: Added.
  • fast/workers/shared-worker-gc.html-disabled: Added.
  • fast/workers/shared-worker-load-error-expected.txt: Added.
  • fast/workers/shared-worker-load-error.html-disabled: Added.
  • fast/workers/shared-worker-replace-global-constructor-expected.txt: Added.
  • fast/workers/shared-worker-replace-global-constructor.html-disabled: Added.
  • fast/workers/shared-worker-script-error-expected.txt: Added.
  • fast/workers/shared-worker-script-error.html-disabled: Added.
  • fast/workers/shared-worker-simple-expected.txt: Added.
  • fast/workers/shared-worker-simple.html-disabled: Added.
  • fast/workers/worker-replace-global-constructor.html:
8:12 AM Changeset in webkit [46844] by cmarrin@apple.com
  • 6 edits in trunk

Added ENABLE_3D_CANVAS flag to build, default to off

7:59 AM BuildingGtk edited by oszi@inf.u-szeged.hu
(diff)
7:26 AM Changeset in webkit [46843] by Simon Hausmann
  • 2 edits in trunk/WebCore

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

Reviewed by Simon Hausmann.

[Qt] windowsKeyCodeForKeyEvent fix

Fix windowsKeyCodeForKeyEvent to determine wheter the event comes from the keypad.

  • platform/qt/PlatformKeyboardEventQt.cpp: (WebCore::windowsKeyCodeForKeyEvent): (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
7:12 AM Changeset in webkit [46842] by staikos@webkit.org
  • 1 edit
    1 delete in trunk/WebCore

2009-08-06 George Staikos <​george.staikos@torchmobile.com>

Rubber-stamped by Ariya Hidayat.

Remove the dead wince/ directory that should never have gone in.

  • svg/graphics/wince: Removed.
  • svg/graphics/wince/SVGResourceFilterWince.cpp: Removed.
2:52 AM Changeset in webkit [46841] by pfeldman@chromium.org
  • 6 edits
    1 add in trunk/WebCore

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

Reviewed by Timothy Hatcher.

WebInspector: Extract style editing into a separate file that is
going to be loaded in page context.

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

  • inspector/front-end/InjectedScript.js: Added. (InjectedScript.applyStyleText): (InjectedScript.setStyleText): (InjectedScript.toggleStyleEnabled): (InjectedScript.setStyleRule):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertiesSection.prototype.editingSelectorCommitted.callback): (WebInspector.StylePropertiesSection.prototype.editingSelectorCommitted): (WebInspector.StylePropertyTreeElement.prototype.): (WebInspector.StylePropertyTreeElement.prototype):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
1:27 AM Changeset in webkit [46840] by abarth@webkit.org
  • 3 edits in trunk/WebCore

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

Reviewed by David Levin.

[V8] Remove bug-bait V8Proxy::context()
​https://bugs.webkit.org/show_bug.cgi?id=27826

  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::updateDocumentWrapperCache): (WebCore::V8Proxy::initContextIfNeeded): (WebCore::V8Proxy::mainWorldContext):
  • bindings/v8/V8Proxy.h:
1:17 AM Changeset in webkit [46839] by abarth@webkit.org
  • 2 edits in trunk/WebCore

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

Unreviewed attempt to fix the Chromium Linux build. This function is
not used.

  • bindings/v8/OwnHandle.h:
12:42 AM Changeset in webkit [46838] by abarth@webkit.org
  • 10 edits
    4 adds in trunk

WebCore:

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

Reviewed by Eric Seidel.

[V8] Teach ScheduledAction::execute about isolated worlds
​https://bugs.webkit.org/show_bug.cgi?id=27703

We now save a weak handle to the original context. We use that handle
to call the timeout in the right context / world.

Tests: http/tests/security/isolatedWorld/window-setTimeout-function.html

http/tests/security/isolatedWorld/window-setTimeout-string.html

  • bindings/v8/ScheduledAction.cpp: (WebCore::ScheduledAction::ScheduledAction): (WebCore::ScheduledAction::execute):
  • bindings/v8/ScheduledAction.h: (WebCore::ScheduledAction::ScheduledAction):
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::V8Custom::WindowSetTimeoutImpl):
  • bindings/v8/custom/V8WorkerContextCustom.cpp: (WebCore::SetTimeoutOrInterval):

LayoutTests:

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

Reviewed by Eric Seidel.

[V8] Teach ScheduledAction::execute about isolated worlds
​https://bugs.webkit.org/show_bug.cgi?id=27703

Add test coverage for the interaction between setTimeout and isolated
worlds.

  • http/tests/security/isolatedWorld/window-setTimeout-function-expected.txt: Added.
  • http/tests/security/isolatedWorld/window-setTimeout-function.html: Added.
  • http/tests/security/isolatedWorld/window-setTimeout-string-expected.txt: Added.
  • http/tests/security/isolatedWorld/window-setTimeout-string.html: Added.

Aug 5, 2009:

11:57 PM Changeset in webkit [46837] by abarth@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-08-05 Zoltan Horvath <​hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

Allow custom memory allocation control for JavaScriptCore's ScopeNodeData struct
​https://bugs.webkit.org/show_bug.cgi?id=27875

Inherits ScopeNodeData struct from FastAllocBase because it has been instantiated by
'new' in JavaScriptCore/parser/Nodes.cpp:1848.

  • parser/Nodes.h:
11:53 PM Changeset in webkit [46836] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

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

Skip new test using eventSender, since the GTK+ port does not have
it yet.

  • platform/gtk/Skipped:
11:10 PM Changeset in webkit [46835] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

11:10 PM Changeset in webkit [46834] by mrowe@apple.com
  • 1 copy in tags/Safari-6531.10

New tag.

11:08 PM Changeset in webkit [46833] by mrowe@apple.com
  • 3 edits in branches/safari-4-branch/WebKit/win

Merge r46638.

10:33 PM Changeset in webkit [46832] by barraclough@apple.com
  • 5 edits in trunk/JavaScriptCore

2009-08-05 Zoltan Herczeg <​zherczeg@inf.u-szeged.hu>

Reviewed by Gavin Barraclough.

Add floating point support for generic ARM port.
​https://bugs.webkit.org/show_bug.cgi?id=24986

  • assembler/ARMAssembler.cpp: (JSC::ARMAssembler::doubleTransfer):
  • assembler/ARMAssembler.h: (JSC::ARM::): (JSC::ARMAssembler::): (JSC::ARMAssembler::faddd_r): (JSC::ARMAssembler::fsubd_r): (JSC::ARMAssembler::fmuld_r): (JSC::ARMAssembler::fcmpd_r): (JSC::ARMAssembler::fdtr_u): (JSC::ARMAssembler::fdtr_d): (JSC::ARMAssembler::fmsr_r): (JSC::ARMAssembler::fsitod_r): (JSC::ARMAssembler::fmstat):
  • assembler/MacroAssemblerARM.h: (JSC::MacroAssemblerARM::): (JSC::MacroAssemblerARM::supportsFloatingPoint): (JSC::MacroAssemblerARM::loadDouble): (JSC::MacroAssemblerARM::storeDouble): (JSC::MacroAssemblerARM::addDouble): (JSC::MacroAssemblerARM::subDouble): (JSC::MacroAssemblerARM::mulDouble): (JSC::MacroAssemblerARM::convertInt32ToDouble): (JSC::MacroAssemblerARM::branchDouble):
  • jit/JIT.h:
10:22 PM Changeset in webkit [46831] by barraclough@apple.com
  • 11 edits in trunk/JavaScriptCore

2009-08-05 Zoltan Herczeg <​zherczeg@inf.u-szeged.hu>

Reviewed by Gavin Barraclough.

Add JIT support for generic ARM port without optimizations.
​https://bugs.webkit.org/show_bug.cgi?id=24986

All JIT optimizations are disabled.

Signed off by Zoltan Herczeg <​zherczeg@inf.u-szeged.hu>
Signed off by Gabor Loki <​loki@inf.u-szeged.hu>

  • assembler/ARMAssembler.cpp: (JSC::ARMAssembler::baseIndexTransfer32):
  • assembler/AbstractMacroAssembler.h: (JSC::AbstractMacroAssembler::Imm32::Imm32):
  • assembler/MacroAssemblerARM.h: (JSC::MacroAssemblerARM::store32): (JSC::MacroAssemblerARM::move): (JSC::MacroAssemblerARM::branch32): (JSC::MacroAssemblerARM::add32): (JSC::MacroAssemblerARM::sub32): (JSC::MacroAssemblerARM::load32):
  • bytecode/CodeBlock.h: (JSC::CodeBlock::getBytecodeIndex):
  • jit/JIT.h:
  • jit/JITInlineMethods.h: (JSC::JIT::restoreArgumentReference):
  • jit/JITOpcodes.cpp:
  • jit/JITStubs.cpp:
  • jit/JITStubs.h: (JSC::JITStackFrame::returnAddressSlot):
  • wtf/Platform.h:
10:19 PM Changeset in webkit [46830] by abarth@webkit.org
  • 4 edits in trunk/LayoutTests

2009-08-05 Drew Wilson <​atwilson@google.com>

Reviewed by David Levin.

Need to test throwing exceptions from Workers after calling close().
​https://bugs.webkit.org/show_bug.cgi?id=27924

  • fast/workers/resources/worker-close.js:
  • fast/workers/worker-close-expected.txt:
  • fast/workers/worker-close.html:
8:50 PM Changeset in webkit [46829] by Dimitri Glazkov
  • 5 edits in trunk/WebCore

2009-08-05 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, build fix.

[V8] Revert ​http://trac.webkit.org/changeset/46821, which was
landed to supplement ​http://trac.webkit.org/changeset/46816, but
now that the latter is reverted, the former needs the same.

  • bindings/v8/DOMObjectsInclude.h:
  • bindings/v8/DerivedSourcesAllInOne.cpp:
  • bindings/v8/V8Index.cpp:
  • bindings/v8/V8Index.h:
8:27 PM Changeset in webkit [46828] by abarth@webkit.org
  • 4 edits
    1 add in trunk/WebCore

2009-08-05 Joseph Pecoraro <​joepeck02@gmail.com>

Reviewed by Timothy Hatcher.

Color is only reported in rgb() format; should toggle between rgb() and hex
​https://bugs.webkit.org/show_bug.cgi?id=13516

New Class to Parse and Change Between Color Representations:

Representations of Colors in Raw Form. (May be Lazy Loaded):

(WebInspector.Color.prototype.get shorthex):
(WebInspector.Color.prototype.get hex):
(WebInspector.Color.prototype.set hex):
(WebInspector.Color.prototype.get rgb):
(WebInspector.Color.prototype.set rgb):
(WebInspector.Color.prototype.get hsl):
(WebInspector.Color.prototype.set hsl):
(WebInspector.Color.prototype.get nickname):
(WebInspector.Color.prototype.set nickname):
(WebInspector.Color.prototype.get rgba):
(WebInspector.Color.prototype.set rgba):
(WebInspector.Color.prototype.get hsla):
(WebInspector.Color.prototype.set hsla):

Helpers and Standard String Representations:

(WebInspector.Color.prototype.hasShortHex):
(WebInspector.Color.prototype.toRgb):
(WebInspector.Color.prototype.toHsl):
(WebInspector.Color.prototype.toShortHex):
(WebInspector.Color.prototype.toHex):
(WebInspector.Color.prototype.toRgba):
(WebInspector.Color.prototype.toHsla):
(WebInspector.Color.prototype.toNickname):

Conversion Functions to Determine Other Representations:
My Sources for all algorithms and sample data:
Wikipedia: ​http://en.wikipedia.org/wiki/HSV_color_space
CSS Specification: ​http://www.w3.org/TR/css3-color/#hsla-color

(WebInspector.Color.prototype.rgbToHex):
(WebInspector.Color.prototype.hexToRgb):
(WebInspector.Color.prototype.rgbToHsl):
(WebInspector.Color.prototype.hslToRgb.hueToRgb):
(WebInspector.Color.prototype.hslToRgb):
(WebInspector.Color.prototype.rgbaToHsla):
(WebInspector.Color.prototype.hslaToRgba):

Called from the constructor, attempts to parse and will throw an error
if it cannot parse. If simple then the "hex" value is guarenteed.
If advanced then both the "rgba" and "hsla" are guarenteed. If it is
a nickname in any way (e.g. "transparent", "black") then this is
guarenteed to be set as well.

(WebInspector.Color.prototype.parse):

Clicking on the swatch will rotate through Color Representations.

Simple: rgb -> hsl -> nickname? -> shorthex? -> hex -> (loop around)
Advanced: rgba -> hsla -> nickname? -> (loop around)

  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype.updateTitle.value.): (WebInspector.StylePropertyTreeElement.prototype.updateTitle):

Load Color.js:

  • inspector/front-end/inspector.html:
8:05 PM Changeset in webkit [46827] by abarth@webkit.org
  • 4 edits in trunk/WebCore

2009-08-05 John Abd-El-Malek <​jam@chromium.org>

Reviewed by Dimitri Glazkov.

Support setting event listeners for message ports in the worker process.

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

  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::retrieve): (WebCore::WorkerContextExecutionProxy::EventTargetToV8Object):
  • bindings/v8/WorkerContextExecutionProxy.h:
  • bindings/v8/custom/V8MessagePortCustom.cpp: (WebCore::getEventListener): (WebCore::ACCESSOR_SETTER): (WebCore::CALLBACK_FUNC_DECL):
7:39 PM Changeset in webkit [46826] by abarth@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Dimitri Glazkov.

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

Minor fix for media code on chromium/skia.

  • rendering/RenderThemeChromiumSkia.cpp: (WebCore::RenderThemeChromiumSkia::paintMediaSliderTrack):
6:37 PM Changeset in webkit [46825] by Darin Adler
  • 3 edits
    2 adds in trunk

WebCore:

2009-08-05 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

Client side image maps did not handle zoom correctly.
Client side image maps did not handle border and padding correctly.

Test: fast/images/image-map-zoom.html

  • rendering/RenderImage.cpp: (WebCore::RenderImage::nodeAtPoint): Compute the values to pass in to the map element using the content box and the effective zoom.

LayoutTests:

2009-08-05 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

Client side image maps did not handle zoom correctly.
Client side image maps did not handle border and padding correctly.

  • fast/images/image-map-zoom-expected.txt: Added.
  • fast/images/image-map-zoom.html: Added.
6:30 PM Changeset in webkit [46824] by beidson@apple.com
  • 5 edits in trunk

Add more missing ResourceLoadDelegate methods for WinDRT, and remove another test from the skipped list.

5:52 PM WebKit Team edited by atwilson@chromium.org
(diff)
5:12 PM Changeset in webkit [46823] by mrowe@apple.com
  • 14 edits
    5 deletes in trunk

Roll out r46816 as it broke many tests.

5:11 PM Changeset in webkit [46822] by ap@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Oliver Hunt.

​https://bugs.webkit.org/show_bug.cgi?id=28027
<rdar://7064428> Files with accents in names are submitted incorrectly due to use of
precomposed Unicode

  • platform/network/mac/FormDataStreamMac.mm: (WebCore::setHTTPBody): Use getFileSize() from FileSystem.h instead of custom code for getting file size. This adds path normalization necessary with some file systems on Mac. This also loses a S_IFMT check, which didn't seem important anyway.
4:47 PM Changeset in webkit [46821] by Dimitri Glazkov
  • 5 edits in trunk/WebCore

2009-08-05 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, build fix.

[V8] Include Media into V8 bindings.

  • bindings/v8/DOMObjectsInclude.h: Added Media include.
  • bindings/v8/DerivedSourcesAllInOne.cpp: Added V8Media include.
  • bindings/v8/V8Index.cpp: Added decl for V8Media.
  • bindings/v8/V8Index.h: Added V8Media include.
4:41 PM Changeset in webkit [46820] by beidson@apple.com
  • 5 edits
    1 delete in trunk

WebKitTools:
2009-08-05 Brady Eidson <​beidson@apple.com>

Reviewed by Darin Adler.

Win DRT's resource load delegate is missing didReceiveResponse
​https://bugs.webkit.org/show_bug.cgi?id=28033

  • DumpRenderTree/win/ResourceLoadDelegate.cpp: (ResourceLoadDelegate::didReceiveResponse):
  • DumpRenderTree/win/ResourceLoadDelegate.h:

LayoutTests:
2009-08-05 Brady Eidson <​beidson@apple.com>

Reviewed by Darin Adler.

Win DRT's resource load delegate is missing didReceiveResponse
​https://bugs.webkit.org/show_bug.cgi?id=28033

  • platform/win/Skipped: Remove a test that now passes


Remove platform specific results for the XFrameOptions suite which now matches cross platform results:

  • platform/win/http/tests/security: Removed.
  • platform/win/http/tests/security/XFrameOptions: Removed.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt: Removed.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt: Removed.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-allow-expected.txt: Removed.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt: Removed.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-allow-expected.txt: Removed.
4:18 PM Changeset in webkit [46819] by jhoneycutt@apple.com
  • 4 edits in trunk/WebCore

2009-08-04 Jon Honeycutt <​jhoneycutt@apple.com>

Fix Windows layout test crashes.

PluginView::removeFromUnstartedListIfNecessary() would try to remove
any unstarted PluginView from the Page's set of unstarted plug-ins,
regardless of why the PluginView wasn't started. If we tried to start a
plug-in, but it failed to start, we would hit an assertion in
Page::removeUnstartedPlugin() that asserts that we would only modify the
unstarted plug-ins set if we were not currently starting plug-ins.

Fix this by having PluginView track whether it's waiting to be started,
and to check this before trying to remove itself from the Page's
unstarted plug-ins set.

Reviewed by Darin Adler.

  • page/Page.cpp: (WebCore::Page::removeUnstartedPlugin): Assert that the passed PluginView is in the unstarted set.
  • plugins/PluginView.cpp: (WebCore::PluginView::start): Set that we are not waiting to be started. (WebCore::PluginView::startOrAddtoUnstartedList): Set that we are waiting to be started. (WebCore::PluginView::removeFromUnstartedListIfNecessary): Check not whether we are started, but whether we are waiting to be started. (WebCore::PluginView::PluginView):
  • plugins/PluginView.h:
3:48 PM Changeset in webkit [46818] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

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

Fix DOM Storage memory leak
​https://bugs.webkit.org/show_bug.cgi?id=28029

Forgot to use adoptRef in one place. This caused the = operator to
increment the ref count when it shouldn't have, so StorageAreaImpl's were
never freed.

  • storage/StorageNamespaceImpl.cpp: (WebCore::StorageNamespaceImpl::storageArea):
2:51 PM Changeset in webkit [46817] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-08-05 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, build fix.

[Chromium] Let CodeGeneratorV8.pm know that Media is ref-counted.

  • bindings/scripts/CodeGeneratorV8.pm: Added Media check to IsRefPtrType.
2:26 PM Changeset in webkit [46816] by Simon Fraser
  • 14 edits
    2 copies
    3 adds in trunk

2009-08-05 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Implement the matchMedium method on the Media interface described in the CSSOM View Module.
​https://bugs.webkit.org/show_bug.cgi?id=22495

Add a new Media interface, obtainable via the AbstractView, that can
evaluate queries via its matchesMedium() method.

Test: fast/media/matchmedium-query-api.html

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl: Add Media.* files to the build.
  • css/Media.cpp: Added. (WebCore::Media::Media): (WebCore::Media::type): (WebCore::Media::matchMedium):
  • css/Media.h: Added. (WebCore::Media::create): (WebCore::Media::document):
  • css/Media.idl: Added. Media::matchMedium() uses a MediaQueryEvaluator() to test the query.
  • page/AbstractView.idl:
  • page/DOMWindow.cpp: (WebCore::DOMWindow::media):
  • page/DOMWindow.h:
  • page/DOMWindow.idl: Add a readonly attribute for Media.
2:25 PM Changeset in webkit [46815] by Darin Adler
  • 96 edits in trunk/WebCore

2009-08-05 Darin Adler <Darin Adler>

Reviewed by David Levin.

Use checked casts for render tree
​https://bugs.webkit.org/show_bug.cgi?id=23522

Last patch. Makes all the casts of RenderObject use checked
casts instead of using static_cast directly.

  • accessibility/AccessibilityListBox.cpp: (WebCore::AccessibilityListBox::visibleChildren): (WebCore::AccessibilityListBox::doAccessibilityHitTest):
  • accessibility/AccessibilityListBoxOption.cpp: (WebCore::AccessibilityListBoxOption::elementRect):
  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::textUnderElement): (WebCore::AccessibilityRenderObject::stringValue): (WebCore::AccessibilityRenderObject::titleUIElement): (WebCore::AccessibilityRenderObject::accessibilityIsIgnored):
  • accessibility/AccessibilitySlider.cpp: (WebCore::AccessibilitySliderThumb::elementRect):
  • dom/InputElement.cpp: (WebCore::InputElement::updatePlaceholderVisibility):
  • dom/SelectElement.cpp: (WebCore::SelectElement::scrollToSelection): (WebCore::SelectElement::recalcStyle): (WebCore::SelectElement::setRecalcListItems): (WebCore::SelectElement::menuListDefaultEventHandler): (WebCore::SelectElement::listBoxDefaultEventHandler):
  • editing/BreakBlockquoteCommand.cpp: (WebCore::BreakBlockquoteCommand::doApply):
  • html/HTMLAppletElement.cpp: (WebCore::HTMLAppletElement::renderWidgetForJSBindings):
  • html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::reset):
  • html/HTMLEmbedElement.cpp: (WebCore::HTMLEmbedElement::updateWidget):
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::attach):
  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::defaultEventHandler):
  • html/HTMLIFrameElement.cpp: (WebCore::HTMLIFrameElement::attach):
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::defaultEventHandler): (WebCore::HTMLInputElement::addSearchResult): (WebCore::HTMLInputElement::onSearch):
  • html/HTMLLIElement.cpp: (WebCore::HTMLLIElement::parseMappedAttribute): (WebCore::HTMLLIElement::attach):
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::setReadyState): (WebCore::HTMLMediaElement::mediaPlayerDurationChanged): (WebCore::HTMLMediaElement::mediaPlayerSizeChanged): (WebCore::HTMLMediaElement::mediaPlayerRenderingCanBeAccelerated): (WebCore::HTMLMediaElement::mediaPlayerGraphicsLayer): (WebCore::HTMLMediaElement::defaultEventHandler): (WebCore::HTMLMediaElement::finishParsingChildren):
  • html/HTMLOListElement.cpp: (WebCore::HTMLOListElement::parseMappedAttribute):
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::updateWidget):
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::defaultEventHandler):
  • page/DragController.cpp: (WebCore::DragController::concludeEditDrag):
  • page/EventHandler.cpp: (WebCore::EventHandler::selectCursor): (WebCore::EventHandler::capsLockStateMayHaveChanged):
  • page/Frame.cpp: (WebCore::Frame::ownerRenderer):
  • page/FrameView.cpp: (WebCore::FrameView::detachCustomScrollbars):
  • page/wince/FrameWince.cpp: (WebCore::computePageRectsForFrame):
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::placeBoxesHorizontally):
  • rendering/MediaControlElements.cpp: (WebCore::MediaControlTimelineElement::defaultEventHandler):
  • rendering/RenderBlockLineLayout.cpp: (WebCore::RenderBlock::findNextLineBreak):
  • rendering/RenderCounter.cpp: (WebCore::planCounter):
  • rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::computeEdgeInfo): (WebCore::RenderFrameSet::setIsResizing):
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::contentsBox):
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::updateBacking): (WebCore::RenderLayerCompositor::requiresCompositingForVideo):
  • rendering/RenderListItem.cpp: (WebCore::previousListItem): (WebCore::RenderListItem::explicitValueChanged):
  • rendering/RenderObject.cpp: (WebCore::updateListMarkerNumbers):
  • rendering/RenderObjectChildList.cpp: (WebCore::updateListMarkerNumbers): (WebCore::invalidateCountersInContainer):
  • rendering/RenderScrollbarTheme.cpp: (WebCore::RenderScrollbarTheme::minimumThumbLength): (WebCore::RenderScrollbarTheme::backButtonRect): (WebCore::RenderScrollbarTheme::forwardButtonRect): (WebCore::RenderScrollbarTheme::trackRect): (WebCore::RenderScrollbarTheme::constrainTrackRectToTrackPieces): (WebCore::RenderScrollbarTheme::paintScrollbarBackground): (WebCore::RenderScrollbarTheme::paintTrackBackground): (WebCore::RenderScrollbarTheme::paintTrackPiece): (WebCore::RenderScrollbarTheme::paintButton): (WebCore::RenderScrollbarTheme::paintThumb):
  • rendering/RenderSlider.cpp: (WebCore::SliderThumbElement::defaultEventHandler):
  • rendering/RenderThemeChromiumMac.mm: (WebCore::RenderThemeChromiumMac::paintSliderThumb): (WebCore::RenderThemeChromiumMac::paintMediaSliderTrack):
  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::determineSliderThumbState):
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::paintSliderThumb):
  • rendering/RenderThemeSafari.cpp: (WebCore::RenderThemeSafari::paintSliderThumb):
  • rendering/RenderThemeWin.cpp: (WebCore::RenderThemeWin::determineSliderThumbState):
  • rendering/SVGCharacterLayoutInfo.cpp: (WebCore::SVGCharacterLayoutInfo::addLayoutInformation):
  • rendering/SVGRootInlineBox.cpp: (WebCore::findSVGRootObject):
  • rendering/TextControlInnerElements.cpp: (WebCore::RenderTextControlInnerBlock::nodeAtPoint): (WebCore::SearchFieldResultsButtonElement::defaultEventHandler):
  • svg/SVGImageElement.cpp: (WebCore::SVGImageElement::attach):
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::canvasResource):
  • svg/SVGTextContentElement.cpp: (WebCore::rootInlineBoxForTextContentElement):
  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::defaultEventHandler): Use checked casts.
  • rendering/RenderApplet.h:
  • rendering/RenderCounter.h:
  • rendering/RenderFieldset.h:
  • rendering/RenderFileUploadControl.h:
  • rendering/RenderFrame.h:
  • rendering/RenderFrameSet.h:
  • rendering/RenderHTMLCanvas.h:
  • rendering/RenderListBox.h:
  • rendering/RenderListItem.h:
  • rendering/RenderListMarker.h:
  • rendering/RenderMedia.h:
  • rendering/RenderMenuList.h:
  • rendering/RenderPart.h:
  • rendering/RenderPartObject.h:
  • rendering/RenderPath.h:
  • rendering/RenderSVGContainer.h:
  • rendering/RenderSVGRoot.h:
  • rendering/RenderSVGTextPath.h:
  • rendering/RenderSVGViewportContainer.h:
  • rendering/RenderScrollbar.h:
  • rendering/RenderSlider.h:
  • rendering/RenderTextControlMultiLine.h:
  • rendering/RenderTextControlSingleLine.h:
  • rendering/RenderVideo.h: Added a checked cast. Also made most members protected or private.
  • rendering/RenderBlock.h:
  • rendering/RenderInline.h:
  • rendering/RenderReplaced.h:
  • rendering/RenderSVGBlock.h:
  • rendering/RenderSVGImage.h:
  • rendering/RenderSVGInlineText.h:
  • rendering/RenderSVGText.h:
  • rendering/RenderTextControl.h: Made most members protected or private.
  • rendering/RenderInline.cpp:
  • rendering/RenderSVGContainer.cpp:
  • rendering/RenderSVGHiddenContainer.cpp:
  • rendering/RenderSVGImage.cpp:
  • rendering/RenderSVGRoot.cpp: Removed unneeded destructor.
  • rendering/RenderSVGHiddenContainer.h: Removed unneeded virtual function overrides that did the same as the default and base class.
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::paint): Updated to use m_viewport directly instead of using a function call. (WebCore::RenderSVGViewportContainer::applyViewportClip): Ditto. (WebCore::RenderSVGViewportContainer::viewportTransform): Ditto. (WebCore::RenderSVGViewportContainer::localToParentTransform): Ditto. (WebCore::RenderSVGViewportContainer::pointIsInsideViewportClip): Ditto.
  • rendering/RenderScrollbar.cpp: (WebCore::pseudoForScrollbarPart): Changed switch statement to use cases for all values instead of using a default case.
  • rendering/RenderTreeAsText.cpp: (WebCore::operator<<): Removed dependency on casting to types that don't have any public members we need to get at. Instead, use function names to identify the type rather than doing so much overloading. Changed to use checked casts. (WebCore::write): Ditto.
  • rendering/SVGRenderSupport.cpp: (WebCore::renderSubtreeToImage): Changed to use a checked cast and also added a comment about the fact that the code does not do sufficient type checking before doing the cast.
  • rendering/SVGRenderTreeAsText.cpp: (WebCore::writeRenderSVGTextBox): Renamed this from operator<<. (WebCore::writeSVGInlineTextBoxes): Renamed this from writeSVGInlineText. (WebCore::writeSVGText): Renamed this from write. (WebCore::writeSVGInlineText): Ditto. (WebCore::writeSVGImage): Ditto.
  • rendering/SVGRenderTreeAsText.h: Updated for name changes. Removed unneeded default arguments.
2:14 PM Changeset in webkit [46814] by pkasting@chromium.org
  • 3 edits in trunk/WebCore

2009-08-05 Peter Kasting <​pkasting@google.com>

Reviewed by Sam Weinig.

​https://bugs.webkit.org/show_bug.cgi?id=27851
Fix a pair of compiler warnings on the GTK Linux buildbot.

  • platform/image-decoders/bmp/BMPImageDecoder.cpp: (WebCore::BMPImageDecoder::processFileHeader): Don't use multi-character constants, they're not portable.
  • platform/image-decoders/ico/ICOImageDecoder.cpp: (WebCore::ICOImageDecoder::setSize): Simpler code that also doesn't compare signed with unsigned.
2:00 PM Changeset in webkit [46813] by beidson@apple.com
  • 1 edit
    4 copies in trunk/LayoutTests

2009-08-05 Brady Eidson <​beidson@apple.com>

Rubberstamped by Mark Rowe.

As a followup to ​http://trac.webkit.org/changeset/46811, Windows-specific results *did* have some differences, so restoring those results.

  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt: Copied from platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt: Copied from platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-allow-expected.txt: Copied from platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-allow-expected.txt.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt: Copied from platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt.
1:46 PM Changeset in webkit [46812] by beidson@apple.com
  • 1 edit
    6 deletes in trunk/LayoutTests

2009-08-05 Brady Eidson <​beidson@apple.com>

Rubberstamped by Mark Rowe.

Actually, for all 6 of these tests Tiger now matches the cross-platform results, so removing all the Tiger results.

  • platform/mac-tiger/http/tests/loading/redirect-methods-expected.txt: Removed.
  • platform/mac-tiger/http/tests/misc/favicon-loads-with-images-disabled-expected.txt: Removed.
  • platform/mac-tiger/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt: Removed.
  • platform/mac-tiger/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt: Removed.
  • platform/mac-tiger/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-allow-expected.txt: Removed.
  • platform/mac-tiger/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt: Removed.
1:31 PM Changeset in webkit [46811] by beidson@apple.com
  • 1 edit
    4 deletes in trunk/LayoutTests

2009-08-05 Brady Eidson <​beidson@apple.com>

Rubberstamped by Mark Rowe.

As a followup to ​http://trac.webkit.org/changeset/46809, Windows-specific results now match the cross-platform results,
so I'm removing them.

  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt: Removed.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt: Removed.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-allow-expected.txt: Removed.
  • platform/win/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt: Removed.
1:26 PM Changeset in webkit [46810] by beidson@apple.com
  • 3 edits in trunk/LayoutTests

2009-08-05 Brady Eidson <​beidson@apple.com>

Rubberstamped by Mark Rowe.

As a followup to ​http://trac.webkit.org/changeset/46809, update Tiger platform specific results for these tests.

  • platform/mac-tiger/http/tests/loading/redirect-methods-expected.txt:
  • platform/mac-tiger/http/tests/misc/favicon-loads-with-images-disabled-expected.txt:
1:16 PM Changeset in webkit [46809] by darin@chromium.org
  • 12 edits in trunk

2009-08-05 Darin Fisher <​darin@chromium.org>

Reviewed by Darin Adler.

Add layoutTestController.dumpWillCacheResponse
​https://bugs.webkit.org/show_bug.cgi?id=28010

willCacheResponse is only interesting for embedders of mainline WebKit on Mac.
Splitting off a new dumpWillCacheResponse allows a number of existings tests
to run across platforms.

The test that was specifically verifying willCacheResponse now calls
dumpWillCacheResponse.

  • DumpRenderTree/LayoutTestController.cpp: Add dumpWillCacheResponse (LayoutTestController::LayoutTestController): (dumpWillCacheResponseCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h: (LayoutTestController::dumpWillCacheResponse): (LayoutTestController::setDumpWillCacheResponse):
  • DumpRenderTree/mac/ResourceLoadDelegate.mm: Inspect dumpWillCacheResponse instead of dumpResourceLoadCallbacks. (-[ResourceLoadDelegate webView:resource:willCacheResponse:fromDataSource:]):
12:56 PM Changeset in webkit [46808] by mrowe@apple.com
  • 4 edits in trunk/LayoutTests

Fix two tests to not load a resource that takes five seconds to load.

Reviewed by Brady Eidson.

We achieve this by making the delay in slow-resource.pl configurable in a query parameter.
This drops the run time of each of the tests from over five seconds to less than a third
of a second.

  • http/tests/misc/DOMContentLoaded-event.html:
  • http/tests/navigation/multiple-back-forward-entries.html:
  • http/tests/navigation/resources/slow-resource.pl:
12:36 PM Changeset in webkit [46807] by pkasting@chromium.org
  • 7 edits
    1 delete in trunk/WebCore

2009-08-05 Peter Kasting <​pkasting@google.com>

Reviewed by Eric Seidel.

​https://bugs.webkit.org/show_bug.cgi?id=26460
Return multiple icon entries from the ICO decoder as separate frames,
sorted by decreasing quality (much like the CG ICO decoder does).

As a result of this change, we can eliminate the Skia-specific setData()
hack that the Chromium port used to select the desired icon size -- now
callers can just enumerate the frames and ask for the data from the one
they like.


Under the hood, the ICO decoder now keeps vectors for a number of things
(including directory entries and image decoders) where it used to have
single members. However, callers (that I have seen) will only request
one frame from the icon, so practically there aren't going to be lots of
instantiated image decoders.

  • platform/graphics/ImageSource.h: Move |m_decoder| back to private now that Skia no longer needs to access it.
  • platform/graphics/cairo/ImageSourceCairo.cpp: (WebCore::createDecoder): Remove size argument from ICO decoder instantiation.
  • platform/graphics/skia/ImageSourceSkia.cpp: (WebCore::createDecoder): Remove size argument from ICO decoder instantiation. (WebCore::ImageSource::setData): Remove function to ask for a particular icon size.
  • platform/graphics/skia/ImageSourceSkia.h: Removed.
  • platform/graphics/wx/ImageSourceWx.cpp: (WebCore::createDecoder): Remove size argument from ICO decoder instantiation.
  • platform/image-decoders/ico/ICOImageDecoder.cpp: (WebCore::ICOImageDecoder::ICOImageDecoder): (WebCore::ICOImageDecoder::~ICOImageDecoder): Delete all instantiated per-frame decoders. (WebCore::ICOImageDecoder::setData): Send data to all instantiated per-frame decoders. (WebCore::ICOImageDecoder::isSizeAvailable): Use size from icon directory instead of PNG decoder (if applicable) so we can report it without decoding the PNG frames. (WebCore::ICOImageDecoder::size): Report frame-specific size if BMP decoder is calling. Otherwise, use size from icon directory instead of PNG decoder (if applicable). (WebCore::ICOImageDecoder::frameSizeAtIndex): Implement. (WebCore::ICOImageDecoder::setSize): Sanity check value if BMP decoder is calling. (WebCore::ICOImageDecoder::frameCount): Implement. (WebCore::ICOImageDecoder::frameBufferAtIndex): Sanity check size for PNG frames. (WebCore::ICOImageDecoder::compareEntries): Add utility function for sorting entries. (WebCore::ICOImageDecoder::setDataForPNGDecoderAtIndex): Factor out utility function for passing correct data blob to a PNG decoder. (WebCore::ICOImageDecoder::decodeWithCheckForDataEnded): Split decode() into two pieces. (WebCore::ICOImageDecoder::decodeDirectory): The first part of the old decode(). (WebCore::ICOImageDecoder::decodeAtIndex): The second part of the old decode(), split off so we avoid decoding an entry until it's requested. (WebCore::ICOImageDecoder::processDirectory): Add resizing of internal data structures based on directory contents. (WebCore::ICOImageDecoder::processDirectoryEntries): Sort entries by quality. (WebCore::ICOImageDecoder::imageTypeAtIndex): Return type to caller instead of setting a member.
  • platform/image-decoders/ico/ICOImageDecoder.h:
11:38 AM Changeset in webkit [46806] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2009-08-05 Szabo Carol <​carol.szabo@nokia.com>

Reviewed by Darin Adler.

Test for ​https://bugs.webkit.org/show_bug.cgi?id=27942
Form Reset does not work per w3c standard when single selection
SELECT element has more than one option marked as selected.

  • fast/forms/select-reset-multiple-selections-4-single-selection-expected.txt: Added.
  • fast/forms/select-reset-multiple-selections-4-single-selection.html: Added.

2009-08-05 Szabo Carol <​carol.szabo@nokia.com>

Reviewed by Darin Adler.

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

Test: fast/forms/select-reset-multiple-selections-4-single-selection.html

  • dom/SelectElement.cpp: (WebCore::SelectElement::reset):

Added HTML 4.01 compliant handling of broken webpages that
specify more than one selected option for single selection SELECT
element, in a manner that emulates Firefox and is similar to IE 7

11:22 AM Changeset in webkit [46805] by jorlow@chromium.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Adam Barth.

Add my name to committers.py
​https://bugs.webkit.org/show_bug.cgi?id=28013

Add my name to committers.py.

  • Scripts/modules/committers.py:
8:17 AM Changeset in webkit [46804] by Simon Hausmann
  • 2 edits in trunk/WebCore

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

Reviewed by Simon Hausmann.

WebCore.pro bugfix

Fix the gperf lines to properly include the string.h header at compilation with gperf version 3.0.3 and newer.

  • WebCore.pro:
8:14 AM Changeset in webkit [46803] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2009-08-05 Csaba Osztrogonac <​oszi@inf.u-szeged.hu>

Reviewed by Simon Hausmann.

[Qt] Fix build error caused by ​http://trac.webkit.org/changeset/46763

  • Api/qwebpluginfactory.h: Export macro added.
7:56 AM Changeset in webkit [46802] by kenneth@webkit.org
  • 3 edits in trunk/WebKitTools

2009-08-05 Kenneth Rohde Christiansen <​kenneth@webkit.org>

Reviewed by Simon Hausmann.

A minor refactoring of the Qt DRT to be a bit more similar to
the other DRTs, as well as more understandable.

Splitting up resetJSObjects into a LayoutTestController->reset()
plus a new closeRemainingWindows() method.

Added a resetToConsistentStateBeforeTesting() method to keep
the code in one place, making it easier to verify that we are
doing things properly.

  • DumpRenderTree/qt/DumpRenderTree.cpp: (WebCore::DumpRenderTree::resetToConsistentStateBeforeTesting): (WebCore::DumpRenderTree::open): (WebCore::DumpRenderTree::closeRemainingWindows):
  • DumpRenderTree/qt/DumpRenderTree.h:
7:51 AM Changeset in webkit [46801] by kenneth@webkit.org
  • 5 edits in trunk/WebKitTools

2009-08-05 Kenneth Rohde Christiansen <​kenneth@webkit.org>

Reviewed by Simon Hausmann.

Make the Qt DRT WorkQueue work similarily to the cross platform
one. This means that all items return true if they started
loading.

This change paves the way for unforking the WorkQueue, if we [Qt]
would like that.

  • DumpRenderTree/qt/WorkQueue.cpp: (WorkQueue::processWork):
  • DumpRenderTree/qt/WorkQueue.h:
  • DumpRenderTree/qt/WorkQueueItem.h:
  • DumpRenderTree/qt/jsobjects.cpp: (LoadItem::invoke): (ReloadItem::invoke): (ScriptItem::invoke): (BackForwardItem::invoke): (LayoutTestController::processWork): (LayoutTestController::maybeDump):
7:48 AM Changeset in webkit [46800] by kenneth@webkit.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Simon Hausmann.

If load of a test fails, don't dump as it will be dumped
in the preceding test, resulting in a invalid incorrect layout.

  • DumpRenderTree/qt/jsobjects.cpp: (LayoutTestController::maybeDump):
7:46 AM Changeset in webkit [46799] by kenneth@webkit.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Simon Hausmann.

Make the Qt DumpRenderTree more similar to the mac one.

Changes include:
1) Reset zoom factor before each test
2) Only dump the backforward list when we got other dump result
3) When we dump the render tree, and got zero result print out:

[mainFrame renderTreeAsExternalRepresentation]
and not:
[frame renderTreeAsExternalRepresentation]

  • DumpRenderTree/qt/DumpRenderTree.cpp: (WebCore::DumpRenderTree::open): (WebCore::methodNameStringForFailedTest): (WebCore::DumpRenderTree::dump):
7:39 AM Changeset in webkit [46798] by Chris Fleizach
  • 4 edits in trunk

WebCore:

2009-08-05 Chris Fleizach <Chris Fleizach>

Fix Tiger build breakage.

  • accessibility/mac/AccessibilityObjectWrapper.mm:

WebKitTools:

2009-08-05 chris fleizach <Chris Fleizach>

Fix Tiger build breakage.

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:
5:40 AM EFLWebKit edited by tonikitoo@gmail.com
(diff)
4:40 AM Changeset in webkit [46797] by jmalonzo@webkit.org
  • 2 edits in trunk

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

Reviewed by Xan Lopez.

[Gtk] macros in webkit.m4 should be defined once only
​https://bugs.webkit.org/show_bug.cgi?id=27929

The macros in webkit.m4 are not meant to be called multiple times
so we'll make them AC_DEFUN_ONCE. Also make AC_HEADER_STDC and AC_PROG_CXX
AC_REQUIRE to remove the automake warnings when doing autogen.sh.

Lastly, make sure CXXFLAGS and CFLAGS are defined before
AC_PROG_CXX gets expanded so it doesn't add "-g -O2" on Release builds.

  • autotools/webkit.m4:
1:13 AM Changeset in webkit [46796] by xan@webkit.org
  • 2 edits in trunk

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

Revert r46714, it was making us build debug images by default.

  • autotools/webkit.m4:
Note: See TracTimeline for information about the timeline view.