Timeline



Jul 3, 2006:

7:35 PM Changeset in webkit [15149] by ggaren
  • 13 edits in trunk/JavaScriptCore

Reviewed by Darin.


  • Refined value conversions in the API:
    • failed toNumber returns NaN
    • failed toObject returns NULL
    • failed toString returns empty string


  • Refined excpetion handling in the API:
    • failed value conversions do not throw exceptions
    • uncaught exceptions in JSEvaluate, JSObjectCallAsFunction, and JSObjectCallAsConstructor are returned through a JSValueRef* exception argument
    • removed JSContextHasException, because JSContextGetException does the same job


  • API/JSBase.h:
  • API/JSCharBufferRef.cpp: (JSValueCopyStringValue):
  • API/JSContextRef.cpp: (JSEvaluate):
  • API/JSContextRef.h:
  • API/JSNodeList.c: Added test code demonstrating how you would use toNumber, and why you probably don't need toUInt32, etc. (JSNodeListPrototype_item): (JSNodeList_getProperty):
  • API/JSObjectRef.cpp: (JSValueToObject): (JSObjectCallAsFunction): (JSObjectCallAsConstructor):
  • API/JSObjectRef.h:
  • API/JSValueRef.cpp: (JSValueToNumber):
  • API/JSValueRef.h:
  • API/minidom.c: (main):
  • API/testapi.c: (main): Added tests for new rules, and call to JSGCProtect to fix Intel crash
  • JavaScriptCore.exp:
6:42 PM Changeset in webkit [15148] by darin
  • 2 edits in trunk/JavaScriptCore
  • Rolled out HashMap implementation of NPRuntime, at least temporarily.

Fixes hang in the bindings section of layout tests seen on the
buildbot.

This code was using HashMap<const char*, PrivateIdentifier*>.
But that hashes based on pointer identity, not string value.
The default hash for any pointer type is to hash based on the pointer.
And WTF doesn't currently have a string hash for char*.
We'll need to fix that before re-landing this patch.

(Formatting was also incorrect -- extra spaces in parentheses.)

  • bindings/npruntime.cpp: Rolled out last change.
11:46 AM Changeset in webkit [15147] by thatcher
  • 1 edit in trunk/WebKit/Misc/WebNSDataExtras.m

Build fix.

10:36 AM Changeset in webkit [15146] by ggaren
  • 3 edits in trunk/JavaScriptCore

Reviewed, tweaked, landed by ggaren.

  • Port NPRuntime from CFDictionary to HashMap.
  • bindings/npruntime.cpp: (getStringIdentifierDictionary): (getIntIdentifierDictionary): (_NPN_GetStringIdentifier): (_NPN_GetIntIdentifier):
  • bindings/npruntime.h:
1:27 AM Changeset in webkit [15145] by thatcher
  • 3 edits in trunk/WebKitTools/Drosera/English.lproj/MainMenu.nib

Forgot the NIB file change for step over and step out.

1:20 AM Changeset in webkit [15144] by thatcher
  • 7 edits
    1 add in trunk/WebKitTools

Reviewed by Eric.

Bug 9631: [Drosera] Add "Step Over" and "Step Out"
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9631


Adds step over and step out. Along with a little code cleanup
that was minor enough to piggyback on this fix.

  • Drosera/DebuggerDocument.h:
  • Drosera/DebuggerDocument.m: (-[DebuggerDocument stepOver:]): (-[DebuggerDocument stepOut:]): (-[DebuggerDocument windowDidLoad]): (-[DebuggerDocument windowWillClose:]): (-[DebuggerDocument toolbar:itemForItemIdentifier:willBeInsertedIntoToolbar:]): (-[DebuggerDocument toolbarDefaultItemIdentifiers:]): (-[DebuggerDocument toolbarAllowedItemIdentifiers:]): (-[DebuggerDocument validateUserInterfaceItem:]):
  • Drosera/Drosera.xcodeproj/project.pbxproj:
  • Drosera/debugger.html:
  • Drosera/debugger.js:
  • Drosera/viewer.css:
1:03 AM Changeset in webkit [15143] by thatcher
  • 2 edits in trunk/WebKitTools

Reviewed by Eric.

Bug 9628: [Drosera] Split Views acting oddly
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9628


Only update the last X or Y coordinate if the new
size was not constrained. Also adds the resize cursor to
the body during the drag incase there is a constrained
over drag off of the resizer element.

  • Drosera/debugger.js:
12:42 AM Changeset in webkit [15142] by thatcher
  • 2 edits in tags/WebKit-521.14.2/WebKit

Build fix and versioning.

12:37 AM Changeset in webkit [15141] by thatcher
  • 1 copy in tags/WebKit-521.14.2

New tag.

Jul 2, 2006:

4:36 PM Changeset in webkit [15140] by ggaren
  • 5 edits in trunk/WebCore

Reviewed and landed by ggaren.

  • platform/gdk/TransferJobManager.cpp Fixed broken dowload logic
  • platform/gdk/ScrollViewGdk.cpp Add comment that scrollbarSize is a temporary variable
  • Projects/gdk/webcore-gdk.bkl Defined NDEBUG build executable is miscompiled otherwise
  • WebCoreSources.bkl added icon/SiteIcon.cpp editing/IndentOutdentCommand.cpp html/HTMLViewSourceDocument.cpp
11:17 AM Changeset in webkit [15139] by ggaren
  • 2 edits in trunk/JavaScriptCore

Reviewed by Adele.


6:05 AM Changeset in webkit [15138] by ddkilzer
  • 2 edits in trunk/WebKitTools

WebKitTools:

Reviewed by Alexy Proskuryakov. Patch by Mark Rowe.

Bug 9692: Warning about Safari extensions on every launch is obnoxious
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9692

  • WebKitLauncher/WebKitNightlyEnabler.m: (myApplicationWillTerminate): Note that we are exiting cleanly. (cleanUpAfterOurselves): Install bundle load tracking only if we failed to exit cleanly on our last invocation. This doesn't play nicely with multiple concurrent instances of WebKit.app so it can be disabled via defaults write com.apple.Safari WKNEShouldMonitorShutdowns -bool NO.
5:23 AM Changeset in webkit [15137] by ddkilzer
  • 2 edits in trunk/WebKitTools

WebKitTools:

Reviewed by Alexy Proskuryakov. Patch by Mark Rowe.

Bug 9654: Refresh Loop when accessing feed URLs
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9654

  • WebKitLauncher/Info.plist: Don't claim to handle feed:// URLs.
5:02 AM Changeset in webkit [15136] by ddkilzer
  • 3 edits in trunk/WebKitTools

WebKitTools:

Reviewed by Timothy Hatcher. Patch by Mark Rowe.

Bug 9689: Nightly builds should warn a user about potential problems when using
"Safari extensions"
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9689

  • WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj:
  • WebKitLauncher/WebKitNightlyEnabler.m: (myBundleDidLoad): Keep track of if any bundles that are loaded. (myApplicationWillFinishLaunching): Notify user if any bundles are loaded. (cleanUpAfterOurselves): Register for NSBundleDidLoadNotification and NSApplicationWillFinishLaunchingNotification notifications so that we can track bundle loads and notify the user at launch completion.
4:59 AM Changeset in webkit [15135] by ddkilzer
  • 4 edits in trunk/WebCore

WebCore:

Reviewed by Timothy Hatcher. Build fix for Linux. Patch by Shawn Stricker.

  • platform/gdk/ScrollViewGdk.cpp: (WebCore::ScrollView::setDrawable): Changed SCROLLBAR_SIZE to scrollbarSize.
  • platform/gdk/TemporaryLinkStubs.cpp: (Image::drawTiled): Added missing parameter to argument list.
  • platform/gdk/TransferJobManager.cpp: (WebCore::writeCallback): Added missing parenthesis to static_cast.
4:39 AM Changeset in webkit [15134] by ddkilzer
  • 4 edits
    4 adds in trunk

WebCore:

Reviewed by Darin. Patch by Mitz.

Replaced the requestingScript state bit added in the fix for bug 9317
with a m_requestingScript bool that is set before and restored
after calling ref() on a CachedScript.


Test: fast/tokenizer/nested-cached-scripts.html

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::HTMLTokenizer): (WebCore::HTMLTokenizer::begin): (WebCore::HTMLTokenizer::scriptHandler): (WebCore::HTMLTokenizer::notifyFinished):
  • html/HTMLTokenizer.h: (WebCore::HTMLTokenizer::State::):

LayoutTests:

Reviewed by Darin. Patch by Mitz.

  • fast/tokenizer/nested-cached-scripts-expected.txt: Added.
  • fast/tokenizer/nested-cached-scripts.html: Added.
  • fast/tokenizer/resources/nested-1.js: Added.
  • fast/tokenizer/resources/nested-2.js: Added.
Note: See TracTimeline for information about the timeline view.