=== Start merge of feature-branch 2007-10-12 ===

2007-10-11  Andrew Wellington  <proton@wiretapped.net>

        Reviewed by Eric Seidel.

        Fix for http://bugs.webkit.org/show_bug.cgi?id=15076

        "deg2rad has multiple definitions"

        Define deg2rad, rad2deg, deg2grad, grad2deg, rad2grad, grad2rad

        These are used through WebKit.

        Change based on original patch by Rob Buis.

        * wtf/MathExtras.h:

        (deg2rad):

        (rad2deg):

        (deg2grad):

        (grad2deg):

        (rad2grad):

        (grad2rad):

2007-10-10  Maciej Stachowiak  <mjs@apple.com>

        Reviewed by Eric.

        - fix assertion failures on quit.

        * kjs/array_object.cpp:

        (ArrayProtoFunc::callAsFunction): Dynamically alocate function-scope static

        UStrings to avoid the static destructor getting called later.

        * kjs/lookup.h: Dynamically alocate function-scope static

        Identifiers to avoid the static destructor getting called later.

2007-10-07  Ed Schouten  <ed@fxq.nl>

        Reviewed and landed by Alexey Proskuryakov.

        Add PLATFORM(FREEBSD), so we can fix the build on FreeBSD-like

        systems by including <pthread_np.h>. Also fix some (disabled)

        regcomp()/regexec() code; it seems some variable names have

        changed.

        * kjs/config.h:

        * kjs/regexp.cpp:

        (KJS::RegExp::RegExp):

        * wtf/Platform.h:

2007-10-02  Alexey Proskuryakov  <ap@webkit.org>

        Reviewed by Darin.

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

        RegExp fails to match non-ASCII characters against [\S\s]

        Test: fast/js/regexp-negative-special-characters.html

        * pcre/pcre_compile.c:

        (compile_branch): Adjust opcode and bitmap as necessary to include (or exclude)

        character codes >255. Fix suggested by Philip Hazel.

        * pcre/pcre_exec.c:

        (match): Merged fix for PCRE bug 580 (\S\S vs. \S{2}).

        * tests/mozilla/expected.html: One test was fixed.

        * pcre/MERGING: Added information about this fix.

2007-10-02  Maciej Stachowiak  <mjs@apple.com>

        Reviewed by Oliver.

        - skip extra hash lookup and avoid converting char* to UString for 19% speedup on CK JS array test 

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

        * kjs/array_object.cpp:

        (ArrayProtoFunc::callAsFunction): Implement the two mentioned optimizations.

2007-10-02  Maciej Stachowiak  <mjs@apple.com>

        Reviewed by Mark.

        - Efficiently handle regexp property identifiers for 19% speedup on Celtic Kane regexp test

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

        * kjs/CommonIdentifiers.h:

        * kjs/regexp_object.cpp:

        (RegExpProtoFunc::callAsFunction):

        (RegExpObjectImp::arrayOfMatches):

        (RegExpObjectImp::construct):

2007-10-02  Maciej Stachowiak  <mjs@apple.com>

        Reviewed by Mark.

        - Cache global prorotypes more efficiently for 10% speedup on CK AJAX benchmark

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

        * kjs/lookup.h:

2007-10-01  Oliver Hunt  <oliver@apple.com>

        Reviewed by Mark.

        Enable Experimental SVG features by default when building from Xcode

        * Configurations/JavaScriptCore.xcconfig:

2007-09-29  Rob Buis  <buis@kde.org>

        Reviewed by Adam.

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

        Misparsing date in javascript leads to year value of -1

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

        Some date values not handled consistently with IE/Firefox

        Allow an optional comma between month and year, and year and time.

        * kjs/date_object.cpp:

        (KJS::parseDate):

2007-07-11  Nikolas Zimmermann  <zimmermann@kde.org>

        Reviewed by Mark.

        Forwardport the hash table fix from CodeGeneratorJS.pm to create_hash_table.

        Reran run-jsc-tests, couldn't find any regressions. Suggested by Darin.

        * kjs/create_hash_table:

2007-06-25  Antti Koivisto  <antti@apple.com>

        Reviewed by Maciej.

        Use intHash to hash floats and doubles too.

        * ChangeLog:

        * wtf/HashFunctions.h:

        (WTF::FloatHash::hash):

        (WTF::FloatHash::equal):

        (WTF::):

        * wtf/HashTraits.h:

        (WTF::FloatHashTraits::emptyValue):

        (WTF::FloatHashTraits::deletedValue):

        (WTF::):

=== End merge of feature-branch 2007-10-12 ===

2007-10-11  Mark Rowe  <mrowe@apple.com>

        Reviewed by Tim Hatcher.

        Fix for <rdar://problem/5488678>.  Disable debugging symbols in production builds for 10.4

        PowerPC to prevent a huge STABS section from being generated.

        * Configurations/Base.xcconfig:

2007-10-08  George Staikos  <staikos@kde.org>

        Reviewed by Adam Roben.

        Fix Qt build on Win32.

        * kjs/testkjs.cpp:

        (main):

2007-10-10  Simon Hausmann  <hausmann@kde.org>

        Reviewed by Lars.

        Fix compilation using gcc 4.3. Header files have been reorganized and as a result some extra

        includes are needed for INT_MAX, std::auto_ptr and the like.

        * kjs/collector.cpp:

        * kjs/collector.h:

        * kjs/lexer.cpp:

        * kjs/scope_chain.cpp:

        * kjs/ustring.cpp:

        * wtf/Vector.h:

2007-10-09  Lars Knoll  <lars@trolltech.com>

        Reviewed by Simon.

        fix the invokation of slots with return types. Add a JSLock around the conversion from QVariant to JSValue.

        * bindings/qt/qt_instance.cpp:

        (KJS::Bindings::QtInstance::invokeMethod):

        * bindings/qt/qt_runtime.cpp:

        (KJS::Bindings::convertValueToQVariant):

        (KJS::Bindings::convertQVariantToValue):

2007-10-05  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Sam Weinig.

        Added JSObject::removeDirect, to support the fix for 

        <rdar://problem/5522487> REGRESSION: With JavaScript disabled, any 

        page load causes a crash in PropertyMap::put

        * kjs/object.cpp:

        (KJS::JSObject::removeDirect):

        * kjs/object.h:

2007-10-04  Mark Rowe  <mrowe@apple.com>

        Reviewed by Oliver.

        Switch to default level of debugging symbols to resolve <rdar://problem/5488678>.

        The "full" level appears to offer no observable benefits even though the documentation

        suggests it be used for dead code stripping.  This should also decrease link times.

        * Configurations/Base.xcconfig:

2007-10-03  Lars Knoll  <lars@trolltech.com>

        Reviewed by Rob.

        Fix a stupid bug in Unicode::toUpper/toLower.

        Fixes all three test failures in the JavaScriptCore test

        suite.

        * wtf/unicode/qt4/UnicodeQt4.h:

        (WTF::Unicode::toLower):

        (WTF::Unicode::toUpper):

2007-10-02  Darin Adler  <darin@apple.com>

        Reviewed by Adam.

        - add support for GDI objects to OwnPtr; I plan to use this

          to fix some GDI handle leaks

        * kjs/grammar.y: Change parser to avoid macros that conflict

        with macros defined in Windows system headers: THIS, DELETE,

        VOID, IN, and CONST. This is needed because OwnPtr.h will now

        include <windows.h>.

        * kjs/keywords.table: Ditto.

        * wtf/OwnPtr.h: For PLATFORM(WIN), add support so that OwnPtr can be

        a GDI handle, and it will call DeleteObject. Also change to use the

        RemovePointer technique used by RetainPtr, so you can say OwnPtr<HBITMAP>

        rather than having to pass in the type pointed to by HBITMAP.

        * wtf/OwnPtrWin.cpp: Added.

        (WebCore::deleteOwnedPtr): Put this in a separate file so that we

        don't have to include <windows.h> in OwnPtr.h.

        * JavaScriptCore.vcproj/WTF/WTF.vcproj: Added OwnPtrWin.cpp.

2007-09-29  Holger Hans Peter Freyther  <zecke@selfish.org>

        Reviewed by Mark.

        -Fix http://bugs.webkit.org/show_bug.cgi?id=13226.

        Remove Bakefiles from svn.

        * JavaScriptCoreSources.bkl: Removed.

        * jscore.bkl: Removed.

2007-09-27  Kevin Decker  <kdecker@apple.com>

        Rubber stamped by John Sullivan.

        <rdar://problem/5493093>

        * JavaScriptCore.order: Added.

        * JavaScriptCore.xcodeproj/project.pbxproj: We're changing from using an order file built by

         another team to using one we actually check into our project repository. Linker settings for

         Symbol Ordering Flags have been updated accordingly.

2007-09-26  Adam Roben  <aroben@apple.com>

        Make testkjs delay-load WebKit.dll so WebKitInitializer can work its magic

        Rubberstamped by Anders.

        * JavaScriptCore.vcproj/testkjs/testkjs.vcproj:

2007-09-25  Adam Roben  <aroben@apple.com>

        Make testkjs delay-load its dependencies

        This lets WebKitInitializer re-route the dependencies to be loaded out

        of the Safari installation directory.

        Rubberstamped by Sam.

        * JavaScriptCore.vcproj/testkjs/testkjs.vcproj:

2007-09-25  David Kilzer  <ddkilzer@webkit.org>

        Reviewed by Adam.

        - Fix http://bugs.webkit.org/show_bug.cgi?id=14885

          LGPL'ed files contain incorrect FSF address

        * COPYING.LIB:

        * bindings/testbindings.cpp:

        * kjs/AllInOneFile.cpp:

        * kjs/DateMath.cpp:

        * kjs/PropertyNameArray.cpp:

        * kjs/PropertyNameArray.h:

        * kjs/config.h:

2007-09-25  Sam Weinig  <sam@webkit.org>

        Fix location for build products for Debug_Internal.

        Reviewed by Adam Roben.

        * JavaScriptCore.vcproj/testkjs/testkjs.vcproj:

2007-09-25  Adam Roben  <aroben@apple.com>

        Make testkjs use WebKitInitializer

        Reviewed by Sam.

        * JavaScriptCore.vcproj/JavaScriptCore.sln: Add WebKitInitializer and

        make testkjs depend on it.

        * JavaScriptCore.vcproj/testkjs/testkjs.vcproj: Link against

        WebKitInitializer.lib.

        * kjs/testkjs.cpp:

        (main): Call initializeWebKit.

2007-09-24  Kevin McCullough  <kmccullough@apple.com>

        Reviewed by Sam.

        - Continued to update project files to not use Edit and Continue for Debug Information since it doesn't work and breaks some functionality.

        * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:

        * JavaScriptCore.vcproj/WTF/WTF.vcproj:

2007-09-21  Kevin McCullough  <kmccullough@apple.com>

        Reviewed by Sam.

        - Updated project files to not use Edit and Continue for Debug Information since it doesn't work and breaks some functionality.

        * JavaScriptCore.vcproj/dftables/dftables.vcproj:

        * JavaScriptCore.vcproj/testkjs/testkjs.vcproj:

2007-09-20  Holger Hans Peter Freyther  <zecke@selfish.org>

        Rubber stamped by Adam.

        Renamed files from *Gdk to *Gtk (see #14732) using the

        work of Juan A. Suarez Romero as a base.

        GDK -> GTK 

        * JavaScriptCore.pri:

        * kjs/testkjs.pro:

        * pcre/dftables.pro:

        * wtf/Platform.h: PLATFORM(GDK) to PLATFORM(GTK)

2007-09-21  Mark Rowe  <mrowe@apple.com>

        Reviewed by Antti Koivisto.

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

        <rdar://problem/5496942> REGRESSION: Reproducible crash in Safari when evaluating script in Drosera console (15250)

        * kjs/function.cpp:

        (KJS::GlobalFuncImp::callAsFunction): Null-check thisObj before passing it to interpreterForGlobalObject.

2007-09-19  Holger Hans Peter Freyther  <zecke@selfish.org>

        Rubber stamped by Adam.

        Make the guard/#if use the same name (ENABLE_FTPDIR) as the #define. This follows

        the ENABLE_ICONDATABASE example from a couple of lines above.

        * wtf/Platform.h:

2007-09-19  Mark Rowe  <mrowe@apple.com>

        Reviewed by Maciej.

        <rdar://problem/5487107> NULL dereference crash in FastMallocZone::enumerate when running leaks against Safari

        Storing remote pointers to their local equivalents in mapped memory was leading to the local pointer being

        interpreted as a remote pointer.  This caused a crash when using the result of mapping this invalid remote pointer.

        The fix is to follow the pattern used elsewhere in FastMallocZone by always doing the mapping after reading and

        never storing the mapped pointer.

        * wtf/FastMalloc.cpp:

        (WTF::FastMallocZone::enumerate):

2007-09-15  Darin Adler  <darin@apple.com>

        - fix Mac build

        * JavaScriptCore.exp: Export WTFLogVerbose.

2007-09-14  Kevin McCullough  <kmccullough@apple.com>

        Reviewed by Sam.

        - Copy JSRetainPtr to include folder.

        * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:

2007-09-13  Geoffrey Garen  <ggaren@apple.com>

        Try to fix GDK build.

        * wtf/MathExtras.h:

        (wtf_random_init):

2007-09-12  Geoff Garen  <ggaren@apple.com>

        Reviewed by Sam Weinig.

        Fixed <rdar://problem/5429064> 141885 Safari JavaScript: Math.random() slightly less randomly distributed than on Safari / Mac

        Math.random was skewed slightly upward because it assumed that RAND_MAX was outside the range of

        values that rand() might return. This problem was particularly pronounced on Windows because

        the range of values returned by rand() on Windows is 2^16 smaller than the range of values

        return by rand() on Mac.

        Fixed by accounting for RAND_MAX return values. Also, switched Windows over to rand_s, which has

        a range that's equal to rand()'s range on Mac.

        * kjs/config.h:

        * kjs/math_object.cpp:

        (MathFuncImp::callAsFunction): Use the new new thing.

        * wtf/MathExtras.h: Platform abstraction for random numbers, to cover over differences on Windows.

        (wtf_random_init):

        (wtf_random):

2007-09-13  Antti Koivisto  <antti@apple.com>

        Reviewed by Maciej.

        Small addition to previous path to cover

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

        window.eval runs in the global scope of the calling window

        Switch variable scope as well.

        * kjs/function.cpp:

        (KJS::GlobalFuncImp::callAsFunction):

2007-09-12  Antti Koivisto  <antti@apple.com>

        Reviewed by Geoff, Maciej.

        Fix <rdar://problem/5445058>

        REGRESSION: Unable to upload picture to eBay auction due to domain security check

        eBay uses window.eval() between windows. In Firefox window.eval() switches execution

        and security context to the target window, something WebKit did not do. With WebKit

        security tightening in r24781, this broke picture uploads.

        Fix by making WebKit switch context in window.eval().

        * kjs/Context.cpp:

        (KJS::Context::Context):

        (KJS::Context::~Context):

        * kjs/context.h:

        Save and restore interpreter context independently from calling context.

        * kjs/function.cpp:

        (KJS::GlobalFuncImp::callAsFunction):

        If eval is called for global object different than current one, switch execution context

        to that object and push it to scope.

2007-09-12  Sam Weinig  <sam@webkit.org>

        Reviewed by Geoffrey Garen.

        <rdar://problem/5478717> JSStringCreateWithCFString leaks when passed a zero length CFStringRef

        * API/JSStringRefCF.cpp:

        (JSStringCreateWithCFString): Special case the zero length string and remove the

        UTF16 optimized path since it will always leak due to the fact that we won't be 

        able to free the backing store that the CFStringRef provides.

2007-09-10  Timothy Hatcher  <timothy@apple.com>

        Reviewed by Darin Adler.

        <rdar://problem/5456224> CrashTracer: [USER] 2 crashes in Toast Titanium at com.apple.CoreServices.CarbonCore: CSMemDisposePtr + 37

        Removed the implementation of these malloc zone functions. We do not have the ability to 

        check if a pointer is valid or not, so we can't correctly implement them. The system free 

        does not fail if you pass in a bad pointer.

        * wtf/FastMalloc.cpp:

        (WTF::FastMallocZone::size):

        (WTF::FastMallocZone::zoneMalloc):

        (WTF::FastMallocZone::zoneCalloc):

        (WTF::FastMallocZone::zoneFree):

        (WTF::FastMallocZone::zoneRealloc):

2007-09-07  Darin Adler  <darin@apple.com>

        Reviewed by Steve Falkenburg.

        - fix crash seen on Windows release builds

        * wtf/FastMalloc.cpp: Change pthread_getspecific optimization to be done only

        on the DARWIN platform. Also correct a couple reinterpret_cast that should be

        static_cast instead.

2007-09-06  Kevin McCullough  <kmccullough@apple.com>

        Reviewed by Maciej.

        - Moved JSRetainPtr to the API.

        * API/JSRetainPtr.h: Copied from kjs/JSRetainPtr.h.

        (JSRetain):

        (JSRelease):

        (JSRetainPtr::JSRetainPtr):

        (JSRetainPtr::~JSRetainPtr):

        (JSRetainPtr::get):

        (JSRetainPtr::releaseRef):

        (JSRetainPtr::operator->):

        (JSRetainPtr::operator!):

        (JSRetainPtr::operator UnspecifiedBoolType):

        (::operator):

        (::adopt):