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

Timeline



May 18, 2009:

11:06 PM Changeset in webkit [43855] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

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

Reviewed by Gavin Barraclough.


  • for polymorphic prototype lookups, increase the number of slots from 4 to 8


~4% faster on v8 raytrace benchmark

  • bytecode/Instruction.h:
10:39 PM Changeset in webkit [43854] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

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

Reviewed by Oliver Hunt.


  • tighten up the code for the load_varargs stub


~1-2% on v8-raytrace


  • jit/JITStubs.cpp: (JSC::JITStubs::cti_op_load_varargs): Hoist some loop invariants that the compiler didn't feel like hoisting for us. Remove unneeded exception check.
10:17 PM Changeset in webkit [43853] by ddkilzer@apple.com
  • 2 edits in trunk/LayoutTests

Added cookie test to Skipped list for Windows

See Bug 25861. <​https://bugs.webkit.org/show_bug.cgi?id=25861>

  • platform/win/Skipped: Added http/tests/cookies/simple-cookies-expired.html.
10:02 PM Changeset in webkit [43852] by ddkilzer@apple.com
  • 2 edits in trunk/WebCore

Tiger build fix for r43850

  • platform/mac/ClipboardMac.mm: Added typedef for NSUinteger when building on Tiger.
9:52 PM Changeset in webkit [43851] by ddkilzer@apple.com
  • 4 edits
    18 adds in trunk

Bug 24569: Add some cookies tests

<​https://bugs.webkit.org/show_bug.cgi?id=24569>

LayoutTests:

Reviewed and tweaked by David Kilzer.

This modification adds a cookie framework to check setting cookies.
It is a simple modification over the js test framework. It uses some XHRs and custom server-side script
to set / get / clear cookies.

It also adds some very basic tests to check that everything works.

  • http/tests/cookies: Added.
  • http/tests/cookies/multiple-cookies-expected.txt: Added.
  • http/tests/cookies/multiple-cookies.html: Added.
  • http/tests/cookies/resources: Added.
  • http/tests/cookies/resources/TEMPLATE.html: Copied from LayoutTests/fast/js/resources/TEMPLATE.html.
  • http/tests/cookies/resources/clearCookies.cgi: Added.
  • http/tests/cookies/resources/cookies-test-post.js: Copied from LayoutTests/fast/js/resources/js-test-post.js.
  • http/tests/cookies/resources/cookies-test-pre.js: Copied from LayoutTests/fast/js/resources/js-test-pre.js. (setCookies): Internal method to set a cookie (multiple cookies are not supported). (testCookies): Internal method to check if the cookie were set.

(clearAllCookies): Used to clear all pre-existing cookies for this domain.
(clearCookies): Used to clear the cookies set with cookiesShouldBe.
(cookiesShouldBe): Used to set a cookie and check the "Cookie" header sent. It is inspired from shouldBe from the js test framework.

  • http/tests/cookies/resources/cookies-test-style.css: Copied from LayoutTests/fast/js/resources/js-test-style.css.
  • http/tests/cookies/resources/getCookies.cgi: Added.
  • http/tests/cookies/resources/multiple-cookies.js: Added.
  • http/tests/cookies/resources/setCookies.cgi: Added.
  • http/tests/cookies/resources/simple-cookies-expired.js: Added.
  • http/tests/cookies/resources/simple-cookies-max-age.js: Added.
  • http/tests/cookies/simple-cookies-expired-expected.txt: Added.
  • http/tests/cookies/simple-cookies-expired.html: Added.
  • http/tests/cookies/simple-cookies-max-age-expected.txt: Added.
  • http/tests/cookies/simple-cookies-max-age.html: Added.
  • platform/mac-tiger/Skipped: Added http/tests/cookies/simple-cookies-max-age.html since CFNetwork on Tiger doesn't support expiring cookies with Max-Age=0.

WebKitTools:

Patch by David Kilzer.

  • Scripts/make-js-test-wrappers: Added regex to skip cookies-test-(post|pre).js files.
9:17 PM Changeset in webkit [43850] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-05-17 Eric Seidel <​eric@webkit.org>

Reviewed by Darin Adler.

ClipboardMac cleanup
​https://bugs.webkit.org/show_bug.cgi?id=25847

I'm mostly just moving code, however there are 3 changes I made while moving, detailed below.

No functional changes, so no tests.

  • platform/mac/ClipboardMac.mm: (WebCore::ClipboardMac::clearData): (WebCore::absoluteURLsFromPasteboardFilenames): Broke out logic for filenames into its own function to make the caller more readable. (WebCore::absoluteURLsFromPasteboard): Broke out logic from getData into absoluteURLsFromPasteboard. This returns an NSArray so that we can use -[NSArray componentsJoinedByString] in the caller (which is cleaner than the manual "\n" addition before). This also access to the full list of file urls for future callers. (WebCore::ClipboardMac::getData): unsigned count = (type == "URL") ? 1 : [fileList count]; is now an explicit check for "URL", before it was a check for != "text/uri-list" which was much more confusing in my opinion. text/uri-list and URL are the only two types which map to NSURLPboardType in cocoaTypeFromMIMEType(). (WebCore::ClipboardMac::types): I removed an extra if (!types) check, right before [types count]. In Obj-C messaging nil will return 0 (size of a pointer), so it's safe to message nil here and expect it to return 0.
8:40 PM Changeset in webkit [43849] by mjs@apple.com
  • 3 edits in trunk/JavaScriptCore

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

Reviewed by Geoff Garen.

  • Improve code generation for access to prototype properties


~0.4% speedup on SunSpider.


Based on a suggestion from Geoff Garen.

  • jit/JIT.h:
  • jit/JITPropertyAccess.cpp: (JSC::JIT::compileGetDirectOffset): (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdProtoList): (JSC::JIT::privateCompileGetByIdChainList): (JSC::JIT::privateCompileGetByIdChain):
8:23 PM Changeset in webkit [43848] by sfalken@apple.com
  • 4 edits in trunk/WebKit

WebKit/mac:

2009-05-18 Sam Weinig <​sam@webkit.org>

<rdar://problem/6899044> Can't see Apple ad on nytimes.com unless I spoof the user agent


Add user agent hack for pointroll.com.

Reviewed by Steve Falkenburg.

  • WebView/WebView.mm: (-[WebView WebCore::_userAgentForURL:WebCore::]):

WebKit/win:

2009-05-18 Steve Falkenburg <​sfalken@apple.com>

<rdar://problem/6899044> Can't see Apple ad on nytimes.com unless I spoof the user agent


Add user agent hack for pointroll.com.


Reviewed by Sam Weinig.

  • WebView.cpp:
5:42 PM Changeset in webkit [43847] by levin@chromium.org
  • 1 edit
    2 adds in trunk/WebCore

2009-05-18 David Levin <​levin@chromium.org>

Reviewed by Darin Fisher.

Bug 25859: Need to upstream ScriptController.* for v8.
​https://bugs.webkit.org/show_bug.cgi?id=25859

  • bindings/v8/ScriptController.cpp: Added.
  • bindings/v8/ScriptController.h: Added.
5:27 PM Changeset in webkit [43846] by kov@webkit.org
  • 5 edits in trunk

2009-05-18 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Gavin Barraclough.

Enable YARR, and disable WREC for GTK+.

  • configure.ac:
5:10 PM Changeset in webkit [43845] by staikos@webkit.org
  • 4 edits in trunk/WebCore

2009-05-18 Yichao Yin <​yichao.yin@torchmobile.com.cn>

Reviewed by George Staikos.

Change ScriptElement to support XHTMLMP in a better way

5:00 PM Changeset in webkit [43844] by kevino@webkit.org
  • 4 edits in trunk

wx build fix. Use python-config to get Python linking info on Unix OSes.

4:51 PM Changeset in webkit [43843] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wx build fix, finish up changes after the accessibility dir split.

4:48 PM Changeset in webkit [43842] by staikos@webkit.org
  • 16 edits
    2 adds in trunk/WebCore

2009-05-18 Yichao Yin <​yichao.yin@torchmobile.com.cn>

Reviewed by George Staikos (and others).

Add XHTMLMP support to Webkit
XHTMLMP is a strict subset of XHTML 1.1. It extends XHTML Basic and add enhanced
functionality. Most of the functionalities have already been implemented by Webkit.

The changes include:
1) Adding <noscript> support
2) Document conformance validation
3) User Agent conformance validation

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

Fix the build.

  • WebCore.xcodeproj/project.pbxproj:
1:40 PM Changeset in webkit [43840] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-05-18 Kenneth Rohde Christiansen <​kenneth.christiansen@openbossa.org>

Rubber-stamped by Gustavo Noronha.

Fixed build for the Qt port by adding the WebCore/accessibility
directory to the includepath.

  • WebCore.pro:
1:22 PM Changeset in webkit [43839] by barraclough@apple.com
  • 17 edits in trunk/JavaScriptCore

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

Reviewed by Xan Lopez.

[Gtk] Various autotools build refactoring and fixes
​https://bugs.webkit.org/show_bug.cgi?id=25286

Add -no-install and -no-fast-install to programs and tests that we
don't install. Also remove -O2 since this is already handled at
configure time.

  • GNUmakefile.am:
1:12 PM Changeset in webkit [43838] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-05-18 Gavin Barraclough <​barraclough@apple.com>

Reviewed by NOBODY (speculative build fix).

12:46 PM Changeset in webkit [43837] by barraclough@apple.com
  • 16 edits in trunk/JavaScriptCore

2009-05-15 Gavin Barraclough <​barraclough@apple.com>

Reviewed by Darin Adler.

Refactor JIT code-handle objects. The representation of generated code is currently
a bit of a mess. We have a class JITCode which wraps the pointer to a block of
generated code, but this object does not reference the executable pool meaning that
external events (the pool being derefed) could make the pointer become invalid.
To overcome this both the JIT and Yarr implement further (and similar) objects to
wrap the code pointer with a RefPtr to the pool. To add to the mire, as well as the
CodeBlock containing a handle onto the code the FunctionBodyNode also contains a
copy of the code pointer which is used almost (but not entirely) uniquely to access
the JIT code for a function.

Rationalization of all this:

  • Add a new type 'MacroAssembler::CodeRef' as a handle for a block of JIT generated code.
  • Change the JIT & Yarr to internally handle code using CodeRefs.
  • Move the CodeRef (formerly anow defunct JITCodeRef) from CodeBlock to its owner node.
  • Remove the (now) redundant code pointer from FunctionBodyNode.

While tidying this up I've made the PatchBuffer return code in new allocations using a CodeRef,
and have enforced an interface that the PatchBuffer will always be used, and 'finalizeCode()' or
'finalizeCodeAddendum()' will always be called exactly once on the PatchBuffer to complete code generation.

This gives us a potentially useful hook ('PatchBuffer::performFinalization()') at the end of generation,
which may have a number of uses. It may be helpful should we wish to switch our generation
model to allow RW/RX exclusive memory, and it may be useful on non-cache-coherent platforms to
give us an oportunity to cache flush as necessary.

No performance impact.

  • assembler/AbstractMacroAssembler.h: (JSC::AbstractMacroAssembler::ProcessorReturnAddress::relinkCallerToTrampoline): (JSC::AbstractMacroAssembler::CodeRef::CodeRef): (JSC::AbstractMacroAssembler::CodeRef::trampolineAt): (JSC::AbstractMacroAssembler::PatchBuffer::PatchBuffer): (JSC::AbstractMacroAssembler::PatchBuffer::~PatchBuffer): (JSC::AbstractMacroAssembler::PatchBuffer::link): (JSC::AbstractMacroAssembler::PatchBuffer::linkTailRecursive): (JSC::AbstractMacroAssembler::PatchBuffer::patch): (JSC::AbstractMacroAssembler::PatchBuffer::complete): (JSC::AbstractMacroAssembler::PatchBuffer::finalize): (JSC::AbstractMacroAssembler::PatchBuffer::entry):
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::CodeBlock): (JSC::CodeBlock::reparseForExceptionInfoIfNecessary): (JSC::CodeBlock::setJITCode):
  • bytecode/CodeBlock.h: (JSC::CodeBlock::getBytecodeIndex): (JSC::CodeBlock::executablePool):
  • interpreter/CallFrameClosure.h:
  • interpreter/Interpreter.cpp: (JSC::Interpreter::execute): (JSC::Interpreter::prepareForRepeatCall):
  • jit/JIT.cpp: (JSC::JIT::privateCompile): (JSC::JIT::privateCompileCTIMachineTrampolines): (JSC::JIT::linkCall):
  • jit/JIT.h:
  • jit/JITCode.h: (JSC::JITCode::JITCode): (JSC::JITCode::operator bool): (JSC::JITCode::addressForCall): (JSC::JITCode::offsetOf): (JSC::JITCode::execute): (JSC::JITCode::size): (JSC::JITCode::executablePool): (JSC::JITCode::HostFunction):
  • jit/JITPropertyAccess.cpp: (JSC::JIT::privateCompilePutByIdTransition): (JSC::JIT::privateCompilePatchGetArrayLength): (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdSelfList): (JSC::JIT::privateCompileGetByIdProtoList): (JSC::JIT::privateCompileGetByIdChainList): (JSC::JIT::privateCompileGetByIdChain):
  • jit/JITStubs.cpp: (JSC::JITStubs::cti_vm_dontLazyLinkCall): (JSC::JITStubs::cti_vm_lazyLinkCall):
  • parser/Nodes.cpp: (JSC::ProgramNode::generateJITCode): (JSC::EvalNode::generateJITCode): (JSC::FunctionBodyNode::FunctionBodyNode): (JSC::FunctionBodyNode::createNativeThunk): (JSC::FunctionBodyNode::generateJITCode):
  • parser/Nodes.h: (JSC::ScopeNode::generatedJITCode): (JSC::ScopeNode::getExecutablePool): (JSC::ScopeNode::setJITCode): (JSC::ProgramNode::jitCode): (JSC::EvalNode::jitCode): (JSC::FunctionBodyNode::jitCode):
  • runtime/RegExp.cpp: (JSC::RegExp::match):
  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::compile): (JSC::Yarr::jitCompileRegex): (JSC::Yarr::executeRegex):
  • yarr/RegexJIT.h: (JSC::Yarr::RegexCodeBlock::RegexCodeBlock): (JSC::Yarr::RegexCodeBlock::pcreFallback): (JSC::Yarr::RegexCodeBlock::setFallback): (JSC::Yarr::RegexCodeBlock::operator bool): (JSC::Yarr::RegexCodeBlock::set): (JSC::Yarr::RegexCodeBlock::execute):
12:11 PM Changeset in webkit [43836] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

wx build fix, initialize variable that doesn't get set on Windows.

12:10 PM Changeset in webkit [43835] by Chris Fleizach
  • 2 edits in trunk/WebCore

Bug 25776: accessibility should have its own folder in WebCore project
​https://bugs.webkit.org/show_bug.cgi?id=25776

Speculative fix for windows build failure.

11:59 AM Changeset in webkit [43834] by ddkilzer@apple.com
  • 8 edits
    2 adds in trunk

Bug 20652: WebKit doesn't display favicons with MIME type image/vnd.microsoft.icon

<​https://bugs.webkit.org/show_bug.cgi?id=20652>

Reviewed by Darin Adler.

WebCore:

Test: http/tests/misc/favicon-as-image.html

  • inspector/front-end/inspector.js: Added image/vnd.microsoft.icon to WebInspector.MIMETypes.
  • platform/MIMETypeRegistry.cpp: (WebCore::initializeSupportedImageMIMETypes): Added image/vnd.microsoft.icon to list of supported image and image resource MIME types.

LayoutTests:

Updated existing favicon-as-image.html test to display a third
image having MIME type image/vnd.microsoft.icon.

  • http/tests/misc/favicon-as-image.html: Updated.
  • http/tests/misc/resources/.htaccess: Added.
  • http/tests/misc/resources/favicon-vnd-ms.ico: Copied from LayoutTests/fast/images/resources/favicon.ico.
  • platform/mac/http/tests/misc/favicon-as-image-expected.checksum: Updated.
  • platform/mac/http/tests/misc/favicon-as-image-expected.png: Updated.
  • platform/mac/http/tests/misc/favicon-as-image-expected.txt: Updated.
11:48 AM Changeset in webkit [43833] by Chris Fleizach
  • 6 edits
    46 moves
    7 adds in trunk/WebCore

Bug 25776: accessibility should have its own folder in WebCore project
​https://bugs.webkit.org/show_bug.cgi?id=25776

10:26 AM Changeset in webkit [43832] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-05-18 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Jan Alonzo.

​https://bugs.webkit.org/show_bug.cgi?id=25854
[GTK] Should tell servers we only support identity as content encoding

Let servers know we only support identity as content
encoding. We will implement this in libsoup.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::ResourceHandle::startHttp):
8:30 AM Changeset in webkit [43831] by ap@webkit.org
  • 3 edits in trunk/WebCore

Reviewed by Darin Adler.

<rdar://problem/6881160> REGRESSION (r41425): Unable to create battle.net account due to
CAPTCHA failure.

The problem was that after loading and displaying a CAPTCHA image, it was loaded from
the server again due to preloading. So, the server expected the last loaded (invisible)
CAPTCHA text to be typed and posted back.

I don't know how to make tests for preloading.

  • loader/DocLoader.cpp: (WebCore::DocLoader::checkForPendingPreloads): There is never a reason to preload after loading for real - and if cache policy is CachePolicyReload, that actually results in another load from network layer.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::cachePolicy): It's not good for cachePolicy() to lie, but I don't know how to re-do r41425 properly. Added a FIXME.
7:13 AM Changeset in webkit [43830] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-05-18 Ariya Hidayat <​ariya.hidayat@nokia.com>

Reviewed by Simon Hausmann.

Done together with Balazs Kelemen <​kelemen.balazs@stud.u-szeged.hu>.

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

[Qt] Reuse FontPlatformData for the same FontDescription.
This effectively prevents growing heap usage for loading every web page.

  • platform/graphics/qt/FontCacheQt.cpp: (WebCore::qHash): Necessary for FontPlatformDataCache. (WebCore::FontCache::getCachedFontPlatformData): Reuse the instance if it exists, otherwise create a new one and insert it in the cache.
5:11 AM Changeset in webkit [43829] by ariya@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-05-18 Zoltan Horvath <​horvath.zoltan.6@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

Makes QtLauncher to accept multiple urls in command line and opens these in separate windows.

  • QtLauncher/main.cpp: (MainWindow::newWindow): (main):
4:33 AM Changeset in webkit [43828] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-05-18 Balazs Kelemen <​kelemen.balazs@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

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

[Qt] Fix the leak in FontFallbackList::fontDataAt() function.
When creating a new instance of SimpleFontData, put it in the font list
so that it can deleted later on.

  • platform/graphics/qt/FontFallbackListQt.cpp: (WebCore::FontFallbackList::invalidate): (WebCore::FontFallbackList::releaseFontData): (WebCore::FontFallbackList::fontDataAt):
4:30 AM Changeset in webkit [43827] by jmalonzo@webkit.org
  • 2 edits in trunk

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

Reviewed by Xan Lopez.

[Gtk] Various autotools build refactoring and fixes
​https://bugs.webkit.org/show_bug.cgi?id=25286

Add support for running unit tests. Also run the tests whenever
the 'check' target runs.

  • GNUmakefile.am:
4:30 AM Changeset in webkit [43826] by jmalonzo@webkit.org
  • 4 edits in trunk

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

Reviewed by Xan Lopez.

[Gtk] Various autotools build refactoring and fixes
​https://bugs.webkit.org/show_bug.cgi?id=25286

Style fixes

  • GNUmakefile.am:
  • WebCore/GNUmakefile.am
4:30 AM Changeset in webkit [43825] by jmalonzo@webkit.org
  • 6 edits in trunk

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

Reviewed by Xan Lopez.

[Gtk] Various autotools build refactoring and fixes
​https://bugs.webkit.org/show_bug.cgi?id=25286

Add -no-install and -no-fast-install to programs and tests that we
don't install. Also remove -O2 since this is already handled at
configure time.

  • GNUmakefile.am:
  • JavaScriptCore/GNUmakefile.am:
  • WebKitTools/GNUmakefile.am:
4:30 AM Changeset in webkit [43824] by jmalonzo@webkit.org
  • 4 edits in trunk

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

Reviewed by Xan Lopez.

[Gtk] Various autotools build refactoring and fixes
​https://bugs.webkit.org/show_bug.cgi?id=25286

Add JavaScriptCore/ to JSC include path only since it's not
required when building WebCore.

  • GNUmakefile.am:

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

Reviewed by Xan Lopez.

[Gtk] Various autotools build refactoring and fixes
​https://bugs.webkit.org/show_bug.cgi?id=25286

Refactor library cflags

  • GNUmakefile.am:
4:11 AM Changeset in webkit [43823] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2009-05-18 Julie Parent <​jparent@google.com>

Reviewed by Eric Seidel.

Skip new tests from r43822 on Gtk
​https://bugs.webkit.org/show_bug.cgi?id=25780

  • platform/gtk/Skipped:
3:55 AM Changeset in webkit [43822] by eric@webkit.org
  • 1 edit
    6 adds in trunk/LayoutTests

2009-05-18 Julie Parent <​jparent@google.com>

Reviewed by Oliver Hunt.

Add missing files from r43821
​https://bugs.webkit.org/show_bug.cgi?id=25780

  • editing/pasteboard/get-data-text-plain-drop-expected.txt: Added.
  • editing/pasteboard/get-data-text-plain-drop.html: Added.
  • editing/pasteboard/get-data-text-plain-paste-expected.txt: Added.
  • editing/pasteboard/get-data-text-plain-paste.html: Added.
  • editing/pasteboard/resources/TEMPLATE.html: Copied from LayoutTests/editing/execCommand/resources/TEMPLATE.html.
  • editing/pasteboard/resources/get-data-text-plain-paste.js: Added. (handlePaste):
3:29 AM Changeset in webkit [43821] by eric@webkit.org
  • 5 edits in trunk

2009-05-18 Julie Parent <​jparent@google.com>

Reviewed by Eric Seidel.

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

  • Scripts/make-js-test-wrappers: Added exception to avoid overwriting a custom-written test.
3:11 AM Changeset in webkit [43820] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-05-18 Julie Parent <​jparent@google.com>

Reviewed by Eric Seidel.

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

  • Scripts/make-js-test-wrappers: Added exception to avoid overwriting a custom-written test.

May 17, 2009:

10:24 PM Changeset in webkit [43819] by weinig@apple.com
  • 2 edits in branches/nitro-extreme/JavaScriptCore

2009-05-17 Sam Weinig <​sam@webkit.org>

Reviewed by Geoffrey Garen.

Remove unneeded ExecState parameter from the number JSValue constructors.

  • runtime/JSValue.h: (JSC::jsNumber): (JSC::jsNaN): (JSC::JSValue::JSValue):
6:20 PM Changeset in webkit [43818] by sfalken@apple.com
  • 3 edits in trunk/JavaScriptCore

Windows build fix

5:31 PM Changeset in webkit [43817] by mitz@apple.com
  • 90 edits
    1 move in trunk/LayoutTests

Reviewed by David Kilzer.

  • move fast/repaint/repaint.js into the resources subdirectory
  • fast/repaint/repaint.js: Removed.
  • fast/repaint/resources/repaint.js: Copied from fast/repaint/repaint.js.
  • (89 tests changed to load the script from the new location)
1:11 PM Changeset in webkit [43816] by staikos@webkit.org
  • 2 edits in trunk/WebCore

Unreviewed build fix for WML: constr->const

1:06 PM Changeset in webkit [43815] by Darin Adler
  • 2 edits
    1 add in trunk/WebCore

2009-05-17 Darin Adler <Darin Adler>

Fix done by Simon Fraser (reviewed by me).

<rdar://problem/6893775> Page with screen-only stylesheet with overflow and height
transitions has bad layout and overlapping garbled text when printing

  • manual-tests/print-with-height-transition-in-screen-stylesheet.html: Added.
  • page/animation/AnimationController.cpp: (WebCore::AnimationController::updateAnimations): Don't run transitions when printing.
12:12 PM Changeset in webkit [43814] by Darin Adler
  • 2 edits
    1 add in trunk/WebCore

2009-05-17 Darin Adler <Darin Adler>

Reviewed by Simon Fraser.

<rdar://problem/6870241> REGRESSION: Programmatically selected popup menu item not shown

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::setSelectedIndex): Added a call to updateFromElement and also a type check on the selected index to avoid a bad cast to HTMLOptionElement.
  • manual-tests/select-option-in-onload.html: Added.
11:34 AM Changeset in webkit [43813] by Simon Fraser
  • 2 edits in trunk/WebCore

2009-05-17 Simon Fraser <Simon Fraser>

Reviewed by Anders Carlsson

Rename local variables for readability.

  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintBorder):
10:50 AM Changeset in webkit [43812] by Darin Adler
  • 2 edits in trunk/LayoutTests

2009-05-17 Darin Adler <Darin Adler>

  • platform/gtk/Skipped: Disable another media test that seems to be failing at least intermittently.
10:47 AM Changeset in webkit [43811] by Darin Adler
  • 4 edits in trunk/WebCore

2009-05-17 Darin Adler <Darin Adler>

Reviewed by Mark Rowe.

Bug 25840: fix some warnings seen on the GTK bot
​https://bugs.webkit.org/show_bug.cgi?id=25840

  • platform/ScrollView.cpp: (WebCore::ScrollView::platformScrollbarModes): Initialize results here. This is really just to quiet warnings. I think this code is unreachable. Someone could put ASSERT_NOT_REACHED() in here too at some point.
  • plugins/PluginStream.cpp: (WebCore::PluginStream::startStream): Use proper printf format for int here, %d, not %lu. This could cause a real problem when compiled 64-bit, so it's good to fix.
  • rendering/RenderTextControl.cpp: (WebCore::getNextSoftBreak): Initialize breakOffset to 0 here. This is really just to quiet the warning, but still seems like a good idea.
9:50 AM Changeset in webkit [43810] by Darin Adler
  • 3 edits in trunk/WebCore

2009-05-17 Chris Fleizach <Chris Fleizach>

Reviewed by Darin Adler.

Bug 25769: Fix accessibility warnings on GTK
​https://bugs.webkit.org/show_bug.cgi?id=25769

  • page/AccessibilityTable.cpp: (WebCore::AccessibilityTable::columnCount): Get rid of excess const. (WebCore::AccessibilityTable::rowCount): Ditto.
  • page/AccessibilityTable.h: Ditto.
9:47 AM Changeset in webkit [43809] by Darin Adler
  • 3 edits
    2 adds in trunk

WebCore:

2009-05-17 Kai Brüning <​kai@granus.net>

Reviewed by Darin Adler.

bug 25822: DOM normalize does not remove empty text node between element nodes
​https://bugs.webkit.org/show_bug.cgi?id=25822

Test: fast/dom/Node/normalize.html

  • dom/Node.cpp: (WebCore::Node::normalize): Changed to remove any empty text nodes.

LayoutTests:

2009-05-17 Kai Brüning <​kai@granus.net>

Reviewed by Darin Adler.


Test case for bug 25822: DOM normalize does not remove empty text node between element nodes
​https://bugs.webkit.org/show_bug.cgi?id=25822


The test case strives to test the whole functionality of Node's normalize().

  • fast/dom/Node/normalize-expected.txt: Added.
  • fast/dom/Node/normalize.html: Added.
Note: See TracTimeline for information about the timeline view.