CodeGeneratorJS.pm fails to generate visitChildren() for EventTarget interfaces with no functions. Examples of such interfaces are IDBOpenDBRequest and IDBVersionChangeRequest.
Created attachment 177288 [details] Patch
Created attachment 177300 [details] Patch
Comment on attachment 177300 [details] Patch Would be good to explain why this is needed in your ChangeLog. For example, is this only required to support a new set of classes, or are you fixing a bug in some existing classes?
(In reply to comment #3) > (From update of attachment 177300 [details]) > Would be good to explain why this is needed in your ChangeLog. For example, is this only required to support a new set of classes, or are you fixing a bug in some existing classes? This change is needed to support the classes IDBOpenDBRequest and IDBVersionChagngeRequest, which are not currently enabled for JSC. These interfaces have the EventTarget attribute but contain no members other than event listeners, and there's no reason for these interfaces to have custom implementations for visitChildren(). There seem to be no other interfaces for which this test fails, but I see no reason for the test I removed (i.e. $numFunctions > 0 || $numCachedAttributes > 0) to remain.
Created attachment 177314 [details] Patch I've updated the ChangeLog to mention the interfaces for which this change is necessary.
Comment on attachment 177314 [details] Patch r=me
Comment on attachment 177314 [details] Patch Rejecting attachment 177314 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: ripts/update-webkit line 152. Failed to run "['Tools/Scripts/update-webkit', '--chromium', '--force-update']" exit_code: 2 Updating OpenSource From http://git.chromium.org/external/Webkit 76cfc20..41a9297 HEAD -> origin/HEAD error: Ref refs/remotes/origin/master is at 41a92978811a093ec854e669c98e2c578bb1b3a1 but expected 76cfc203424f158e9f12fb85dfd4a4491fd1462e ! 76cfc20..41a9297 master -> origin/master (unable to update local ref) Died at Tools/Scripts/update-webkit line 152. Full output: http://queues.webkit.org/results/15106662
It looks like the patch is conflicting with the WebKit trunk. Please rebase your patch with the latest WebKit, fill "Geoffrey Garen" in the reviewer field in ChangeLog, and reupload it.
Created attachment 177361 [details] Patch
Comment on attachment 177361 [details] Patch Thanks!
Comment on attachment 177361 [details] Patch Rejecting attachment 177361 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: ripts/update-webkit line 152. Failed to run "['Tools/Scripts/update-webkit', '--chromium', '--force-update']" exit_code: 2 Updating OpenSource From http://git.chromium.org/external/Webkit ef0ca77..8cbb25f HEAD -> origin/HEAD error: Ref refs/remotes/origin/master is at 8cbb25f3febaaee8d69fbea1e5b98dd0173b71a9 but expected ef0ca77a554ed2a07b001ee4935266d6301b2bf8 ! ef0ca77..8cbb25f master -> origin/master (unable to update local ref) Died at Tools/Scripts/update-webkit line 152. Full output: http://queues.webkit.org/results/15098929
Comment on attachment 177361 [details] Patch Clearing flags on attachment: 177361 Committed r136482: <http://trac.webkit.org/changeset/136482>
All reviewed patches have been landed. Closing bug.
(In reply to comment #12) > (From update of attachment 177361 [details]) > Clearing flags on attachment: 177361 > > Committed r136482: <http://trac.webkit.org/changeset/136482> It broke the bindings tests. Could you fix it?
(In reply to comment #14) > It broke the bindings tests. Could you fix it? Sorry. Fixed in r136490.