Bug 118391

Summary: Remove support for [PassThisToCallback] extended attribute
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: BindingsAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, darin, esprehn+autocc, haraken, laszlo.gombos, rniwa, thorton, timothy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2 none

Chris Dumez
Reported 2013-07-04 05:48:50 PDT
The [PassThisToCallback] Webkit-specific IDL extended attribute is supported by the JSC bindings generator but is no longer used. We can therefore drop support for this from the generator to simplify the code.
Attachments
Patch (5.34 KB, patch)
2013-07-04 05:53 PDT, Chris Dumez
no flags
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2 (488.41 KB, application/zip)
2013-07-04 09:51 PDT, Build Bot
no flags
Chris Dumez
Comment 1 2013-07-04 05:53:34 PDT
Kentaro Hara
Comment 2 2013-07-04 05:54:55 PDT
Comment on attachment 206081 [details] Patch Looks OK.
Build Bot
Comment 3 2013-07-04 09:51:07 PDT
Comment on attachment 206081 [details] Patch Attachment 206081 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/1026325 New failing tests: http/tests/security/cross-origin-plugin-private-browsing-toggled.html
Build Bot
Comment 4 2013-07-04 09:51:09 PDT
Created attachment 206099 [details] Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-09 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.3
WebKit Commit Bot
Comment 5 2013-07-09 03:56:50 PDT
Comment on attachment 206081 [details] Patch Clearing flags on attachment: 206081 Committed r152490: <http://trac.webkit.org/changeset/152490>
WebKit Commit Bot
Comment 6 2013-07-09 03:56:53 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.