Bug 16681 - Apollo Differences against JavaScriptCore
Summary: Apollo Differences against JavaScriptCore
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-12-30 16:35 PST by Brent Fulgham
Modified: 2008-06-11 16:18 PDT (History)
6 users (show)

See Also:

Attachments
Diff against Apollo 1.0 JavaScriptCore implementation (48.56 KB, patch)
2007-12-30 16:35 PST, Brent Fulgham 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2007-12-30 16:35:01 PST 
The attached patch contains differences between Safari 55233.3 and Apollo/AIR 1.0.  Some of these differences are not relevant (various Context object manipulation).  Some of the comments they added may be helpful from a documentation standpoint.
Comment 1 Brent Fulgham 2007-12-30 16:35:41 PST 
Created attachment 18199 [details]
Diff against Apollo 1.0 JavaScriptCore implementation
Comment 2 David Kilzer (:ddkilzer) 2007-12-30 18:11:10 PST 
Adam:  The Formatted Diff link for Attachment #18199 [details] returns a blank web page:

http://bugs.webkit.org/attachment.cgi?id=18199&action=prettypatch
Comment 3 Maciej Stachowiak 2007-12-31 17:59:03 PST 
It's hard to understand the purpose of many of these changes, and there don't appear to be any test cases.
Comment 4 Cameron Zwarich (cpst) 2008-06-10 23:15:27 PDT 
Is this still relevant after SquirrelFish?
Comment 5 Brent Fulgham 2008-06-11 16:18:41 PDT 
(In reply to comment #4)
> Is this still relevant after SquirrelFish?

Good point.  I'll close this out as WebKit has drifted far from the Apollo/AIR base with respect to JSC.

The AIR team will have to figure out how best to integrate their customized routines into JavaScriptCore under the new architecture, so this patch is probably no longer useful.