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-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-09-03  Mark Rowe  <mrowe@apple.com>

        Reviewed by Tim Hatcher.

        <rdar://problem/5452164> Production build with in symbols directory has no debug info

        Enable debug symbol generation on all build configurations.  Production builds are stripped

        of symbols by Xcode during deployment post-processing.

        * Configurations/Base.xcconfig:

        * JavaScriptGlue.xcodeproj/project.pbxproj:

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

        Reviewed by Antti.

        <rdar://problem/5400709> Versioning in debug and release builds should include minor and tiny version before +

        * Configurations/Version.xcconfig:

        * JavaScriptGlue.xcodeproj/project.pbxproj: Add a shell script phase to make to dependency between

        Version.xcconfig and Info.plist explicit to Xcode.

2007-07-16  Sam Weinig  <sam@webkit.org>

        Reviewed by Oliver.

        Turn off -Wshorten-64-to-32 warning for 64-bit builds.

        * Configurations/Base.xcconfig:

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

        Reviewed by Oliver.

        * JavaScriptGlue.xcodeproj/project.pbxproj: Improve dependencies in Xcode project

        by marking JavaScriptGlue as a dependency of testjsglue.

2007-03-24  Mark Rowe  <mrowe@apple.com>

        Reviewed by Dave Hyatt.

        * JavaScriptGlue.xcodeproj/project.pbxproj: Use GCC 4.0 for testjsglue.

2007-03-24  Mark Rowe  <mrowe@apple.com>

        Rubber-stamped by Darin.

        * Configurations/JavaScriptGlue.xcconfig: Remove unnecessary INFOPLIST_PREPROCESS.

2007-03-19  Andrew Wellington  <proton@wiretapped.net>

        Reviewed by Maciej.

        Really set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2007-03-18  Andrew Wellington  <proton@wiretapped.net>

        Reviewed by Mark Rowe

        Set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2007-03-17  Timothy Hatcher  <timothy@apple.com>

        Reviewed by Mark Rowe.

        Made Version.xcconfig smarter when building for different configurations.

        Now uses the 522+ OpenSource version for Debug and Release, while using the    

        full 522.4 version for Production builds. The system prefix is also computed

        based on the current system, so 4522.4 on Tiger and 5522.4 on Leopard.

        * Configurations/JavaScriptGlue.xcconfig:

        * Configurations/Version.xcconfig:

2007-03-15  Timothy Hatcher  <timothy@apple.com>

        Reviewed by John.

        * Factored out most of our common build settings into .xcconfig files. Anything that was common in

          each build configuration was factored out into the shared .xcconfig file.

        * Adds a Version.xcconfig file to define the current framework version, to be used in other places.

        * Use the new $(BUNDLE_VERSION) (defined in Version.xcconfig) in the preprocessed Info.plist.

        * Use the versions defined in Version.xcconfig to set $(DYLIB_CURRENT_VERSION).

        * Configurations/Base.xcconfig: Added.

        * Configurations/DebugRelease.xcconfig: Added.

        * Configurations/JavaScriptGlue.xcconfig: Added.

        * Configurations/Version.xcconfig: Added.

        * Info.plist:

        * JavaScriptGlue.xcodeproj/project.pbxproj:

=== Safari-5522.4 ===

2007-03-12  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Oliver Hunt.

        Updated in light of fix for <rdar://problem/4681051> Installer crashes 

        in KJS::Collector::markOtherThreadConservatively(KJS::Collector::Thread*) 

        trying to install iLife 06 using Rosetta on an Intel Machine

        * JavaScriptGlue.cpp:

        (JSLockInterpreter): Ensure backwards compatibility by calling 

        registerThread() when explicitly taking the JSLock. (This doesn't happen 

        automatically anymore.) I doubt this actally matters, but in JavaScriptGlue

        territory, that kind of thinking will get you killed.

2007-03-06  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Maciej Stachowiak.

        Fixed all known crashers exposed by run-webkit-tests --threaded while using

        a PAC file (for maximum carnage). See JavaScriptCore ChangeLog for 

        more details.

        * JSBase.cpp:

        (JSBase::Release): Lock when deleting, because we may be deleting an object

        (like a JSRun) that holds thread-unsafe data.

        * JSUtils.cpp:

        (CFStringToUString): Don't lock, because our caller locks. Also, locking

        inside a function that returns thread-unsafe data by copy will only mask

        threading problems.

        * JavaScriptGlue.cpp:

        (JSRunEvaluate): Added missing JSLock.

        (JSRunCheckSyntax): Converted to JSLock.

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2007-02-22  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Darin Adler.

        Fixed <rdar://problem/4942528> Installer.app Stuck "Examining additional 

        volumes..."

        The problem was that JavaScriptGlue held a JSObject in a static pointer

        without protecting it from garbage collection. 

            sUserObjectPrototypeImp  = new UserObjectPrototypeImp();

            static ProtectedPtr<UserObjectPrototypeImp> protectPrototype; // valiant, brave, but useless

        Through an interesting comedy of errors, this object would be collected 

        and then assigned as the prototype of another object allocated at the 

        same address, thus becoming its own prototype. (See http://www.ziplo.com/grandpa.htm

        for further explanation of this phenomenon.) Hence the infinte loop in 

        property lookup.

        The solution is equally humorous: all of this code was completely useless --

        it only served to create an empty object in the prototype chain -- so

        I just removed it.

        * UserObjectImp.cpp:

        (UserObjectImp::UserObjectImp):

        * UserObjectImp.h:

2007-02-19  Timothy Hatcher  <timothy@apple.com>

        Reviewed by Geoff.

        <rdar://problem/4636301> Deprecate JavaScriptGlue

        * JavaScriptGlue.h: adds DEPRECATED_IN_MAC_OS_X_VERSION_10_5_AND_LATER to everything

        * JavaScriptGlue.xcodeproj/project.pbxproj: use -Wno-deprecated-declarations to keep the project building

        * config.h: define DEPRECATED_IN_MAC_OS_X_VERSION_10_5_AND_LATER as empty on other systems

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

        Reviewed by Anders.

        - fix <rdar://problem/5000216> JavaScriptGlue framework started turning all CFNumbers

          into signed 32bit values

        Circumstantial evidence seems to indicate that JavaScriptGlue was depending on a bug

        in CFNumber where CFNumberIsFloatType was returning true for 64-bit integer values.

        * JSUtils.cpp: (JSObjectKJSValue): Removed the call to CFNumberIsFloatType in the

        conversion of numeric values, and always use kCFNumberDoubleType, since double is

        the internal format of numbers in JavaScriptCore. There's no value to having a

        separate code path for integers. There's also no reason to check the return value

        from CFNumberGetValue -- if it returns false it still does the best it can to convert

        to a double, which is what we want.

        * UserObjectImp.cpp: (UserObjectImp::toString): Removed a similar code path that

        tried to handle integers separate from double. Instead, always use a double. For

        compatibility, use a format without a decimal point when CFNumberIsFloatType returns

        false. Also removed a bogus cast to float; the "%f" format string takes a double,

        not a float, so all the cast did was cut down on precision.

2007-01-25  Mark Rowe  <mrowe@apple.com>

        Reviewed by Maciej.

        * Info.plist: Update copyright string.

2006-12-19  Timothy Hatcher  <timothy@apple.com>

        Reviewed by Darin.

        <rdar://problem/4891774> Local WebCore/WebBrowser builds fail in 9A328 due to warning about ObjC-2.0 language features

        * JavaScriptGlue.xcodeproj/project.pbxproj:

=== Safari-521.26 ===

2006-08-15  Mark Rowe  <opendarwin.org@bdash.net.nz>

        Reviewed by Tim H.

        Build fix:  DWARF and -gfull are incompatible with symbol separation.

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2006-08-15  Mark Rowe  <opendarwin.org@bdash.net.nz>

        Reviewed by Tim H.

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

        Bug 10394: WebKit Release and Production configurations should enable dead code stripping

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2006-08-15  Mark Rowe  <opendarwin.org@bdash.net.nz>

        Reviewed by Tim H.

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

        Bug 10384: Switch to DWARF for Release configuration

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2006-08-04  Sam Weinig  <sam.weinig@gmail.com>

        Reviewed by Darin.

        - patch for http://bugs.webkit.org/show_bug.cgi?id=10192

          Make WebCore (and friends) compile with -Wshorten-64-to-32

          * Adds -Wshorten-64-to-32 flag to Xcode project.

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2006-08-03  Sam Weinig  <sam.weinig@gmail.com>

        Reviewed by Darin.

        - patch for http://bugs.webkit.org/show_bug.cgi?id=10176

          Make WebCore compile with -Wundef

          * Adds -Wundef flag to Xcode project

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2006-07-29  Sam Weinig  <sam.weinig@gmail.com>

        Reviewed by Darin.

        - patch for http://bugs.webkit.org/show_bug.cgi?id=10080

          Adopt pedantic changes from the Unity project to improve 

          cross-compiler compatibility

          Changes include:

          * Turning on gcc warning for missing newline at the end of a source file

            (GCC_WARN_ABOUT_MISSING_NEWLINE in Xcode, -Wnewline in gcc).

        * JavaScriptGlue.xcodeproj/project.pbxproj:

=== Safari-521.19 ===

2006-07-19  Geoffrey Garen  <ggaren@apple.com>

        Fixed by Darin & Maciej, reviewed by me.

        - Fixed <rdar://problem/4638934> Leopard 9A227: Installer crashes right 

        after LCA and while preparing Installation

        Compile release/production with NDEBUG set, to ensure binary compatibility

        between JavaScriptGlue and JavaScriptCore. In debug builds, JavaScriptCore 

        includes an extra debug data member in HashSet.

        * JavaScriptGlue.xcodeproj/project.pbxproj:

=== Safari-521.18 ===

2006-07-15  Maciej Stachowiak  <mjs@apple.com>

        Reviewed by Darin.

        - switch property lists to be vector+set of Identifiers instead of list of References

        * JSUtils.cpp:

        (KJSValueToCFTypeInternal): updated for JSC SPI changes

        * JSValueWrapper.cpp:

        (JSValueWrapper::JSObjectCopyPropertyNames): ditto

        * UserObjectImp.cpp:

        (UserObjectImp::getPropertyNames): ditto

        * UserObjectImp.h:

=== Safari-521.17 ===

2006-07-14  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Maciej.

        - Updated JSInterpreter to work with Interpreter ref-counting in JavaScriptCore.

        (JSInterpreter::JSInterpreter::~JSInterpreter): Now protected to catch

        manual delete.

2006-07-12  Anders Carlsson  <acarlsson@apple.com>

        Build fix

        * JSUtils.cpp:

        (KJSValueToCFTypeInternal):

        * JSValueWrapper.cpp:

        (JSValueWrapper::JSObjectCopyPropertyNames):

        * UserObjectImp.cpp:

        (UserObjectImp::getPropertyList):

=== Safari-521.15 ===

2006-07-07  Geoffrey Garen  <ggaren@apple.com>

        - build fix

        * JSUtils.cpp:

        (KJSValueToCFTypeInternal):

        * JSValueWrapper.cpp:

        (JSValueWrapper::JSObjectCopyPropertyNames):

2006-07-03  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Maciej.

        * JSRun.cpp:

        (JSRun::CheckSyntax): Updated to use new checkSyntax syntax in JSC.

        * JavaScriptGlue.xcodeproj/project.pbxproj:

=== Safari-521.14 ===

2006-06-28  Timothy Hatcher  <timothy@apple.com>

        Prefer the Stabs debugging symbols format until DWARF bugs are fixed.

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2006-06-24  David Kilzer  <ddkilzer@kilzer.net>

        Reviewed by Timothy.

        * Info.plist: Fixed copyright to include 2003-2006.

=== JavaScriptGlue-521.13 ===

2006-06-20  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Anders.

        - Required for JS API: Rename propList to getPropertyList and have it

        take its target property list by reference so that subclasses can

        add properties to the list before calling through to their superclasses.

        * JSUtils.cpp:

        (KJSValueToCFTypeInternal):

        * JSValueWrapper.cpp:

        (JSValueWrapper::JSObjectCopyPropertyNames):

        * UserObjectImp.cpp:

        (UserObjectImp::getPropertyList):

        * UserObjectImp.h:

2006-06-15  Timothy Hatcher  <timothy@apple.com>

        Reviewed by Geoff and Darin.

        Prefer the DWARF debugging symbols format for use in Xcode 2.3.

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2006-06-12  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by TimO, Maciej.

        - JSGlue part of merging InterpreterImp into Interpreter.

        Since there's now a kjs forwarding header, I moved all forwarding

        headers into a "ForwardingHeaders" directory, like in WebCore.

        * ForwardingHeaders: Added.

        * ForwardingHeaders/kjs: Added.

        * ForwardingHeaders/kjs/ExecState.h: Added.

        * ForwardingHeaders/wtf: Added.

        * JavaScriptGlue.xcodeproj/project.pbxproj:

        * config.h:

        * wtf: Removed.

        * wtf/AlwaysInline.h: Removed.

        * wtf/Assertions.h: Removed.

        * wtf/FastMalloc.h: Removed.

        * wtf/Forward.h: Removed.

        * wtf/HashCountedSet.h: Removed.

        * wtf/HashSet.h: Removed.

        * wtf/Noncopyable.h: Removed.

        * wtf/OwnArrayPtr.h: Removed.

        * wtf/OwnPtr.h: Removed.

        * wtf/PassRefPtr.h: Removed.

        * wtf/Platform.h: Removed.

        * wtf/RefPtr.h: Removed.

        * wtf/Vector.h: Removed.

=== JavaScriptGlue-521.10 ===

2006-05-09  Maciej Stachowiak  <mjs@apple.com>

        Rubber stamped by Anders.

        - renamed kxmlcore to wtf

        kxmlcore --> wtf

        KXMLCore --> WTF

        KXC --> WTF

        * config.h:

        * kxmlcore: Removed.

        * kxmlcore/AlwaysInline.h: Removed.

        * kxmlcore/Assertions.h: Removed.

        * kxmlcore/FastMalloc.h: Removed.

        * kxmlcore/Forward.h: Removed.

        * kxmlcore/HashCountedSet.h: Removed.

        * kxmlcore/HashSet.h: Removed.

        * kxmlcore/Noncopyable.h: Removed.

        * kxmlcore/OwnArrayPtr.h: Removed.

        * kxmlcore/OwnPtr.h: Removed.

        * kxmlcore/PassRefPtr.h: Removed.

        * kxmlcore/Platform.h: Removed.

        * kxmlcore/RefPtr.h: Removed.

        * kxmlcore/Vector.h: Removed.

        * wtf: Added.

2006-04-11  Darin Adler  <darin@apple.com>

        - try to fix Windows build

        * kxmlcore/HashForward.h: Removed.

2006-04-10  Darin Adler  <darin@apple.com>

        Rubber-stamped by John Sullivan.

        - added forwarding headers for the new Forward.h and HashForward.h files

        * kxmlcore/Forward.h: Added.

        * kxmlcore/HashForward.h: Added.

2006-04-04  Timothy Hatcher  <timothy@apple.com>

        Reviewed by Darin.

        The Debug and Release frameworks are now built with install paths relative to the build products directory.

        This removes the need for other projects to build with -framework WebCore and -framework JavaScriptCore. 

        * JavaScriptGlue.xcodeproj/project.pbxproj:

2006-04-02  Eric Seidel  <eseidel@apple.com>

        Reviewed by andersca.

        Add missing forwarding header.

        * kxmlcore/Vector.h: Added.

2006-03-31  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Darin.

        - Fixed <rdar://problem/4430836> JavaScript patch crashing Quartz

        Composer

        JSValueWrappers used to hold on to the ExecState that pertained at

        the time of their creation. Since ExecStates are transient, that

        design was totally bogus, and it would crash once the ExecState had

        been deallocated.

        Unfortunately, there's no clean solution to this problem, since

        the JSGlue API has no notion of state. The solution here is to use

        a shared, global ExecState for the purpose of JSGlue calls. Given

        the pre-existing limitations in the JSGlue API, this design

        shouldn't actually introduce any new limitations (see comments in

        JSValueWrapper.cpp).

        I tested with Quartz Composer and PAC files, neither of which are

        layout-testable.

        * JSUtils.cpp:

        (KJSValueToJSObject):

        * JSValueWrapper.cpp:

        (JSValueWrapper::JSValueWrapper):

        (JSValueWrapper::GetValue):

        (getProcessGlobalExecState):

        (JSValueWrapper::JSObjectCopyPropertyNames):

        (JSValueWrapper::JSObjectCopyProperty):

        (JSValueWrapper::JSObjectSetProperty):

        (JSValueWrapper::JSObjectCallFunction):

        (JSValueWrapper::JSObjectCopyCFValue):

        * JSValueWrapper.h:

        Also added a test harness file. It was helpful while I was testing,