WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
77848
Rename [HasIndexGetter], [HasNameGetter] and [HasCustomIndexSetter] IDLs
https://bugs.webkit.org/show_bug.cgi?id=77848
Summary
Rename [HasIndexGetter], [HasNameGetter] and [HasCustomIndexSetter] IDLs
Kentaro Hara
Reported
2012-02-05 21:36:03 PST
For naming consistency with [CustomGetter] and [CustomSetter], we can rename the following IDLs: [HasIndexGetter] => [IndexedGetter] (Remove "Has". This IDL is for "indexed" properties in the Web IDL:
http://dev.w3.org/2006/webapi/WebIDL/#idl-indexed-properties
) [HasCustomIndexSetter] => [CustomIndexedSetter] (Ditto.) [HasNameGetter] => [NamedGetter] (Remove "Has". This IDL is for "named" properties in the Web IDL:
http://dev.w3.org/2006/webapi/WebIDL/#idl-named-properties
)
Attachments
Patch
(49.19 KB, patch)
2012-02-05 21:57 PST
,
Kentaro Hara
no flags
Details
Formatted Diff
Diff
patch for commit
(49.00 KB, patch)
2012-02-06 03:17 PST
,
Kentaro Hara
no flags
Details
Formatted Diff
Diff
patch for commit
(48.99 KB, patch)
2012-02-06 03:18 PST
,
Kentaro Hara
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Kentaro Hara
Comment 1
2012-02-05 21:57:05 PST
Created
attachment 125569
[details]
Patch
Adam Barth
Comment 2
2012-02-05 22:12:27 PST
Comment on
attachment 125569
[details]
Patch Ok. Ideally we'd match the WebIDL syntax.
WebKit Review Bot
Comment 3
2012-02-06 01:21:29 PST
Comment on
attachment 125569
[details]
Patch Rejecting
attachment 125569
[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: 10594 --non-interactive --force --accept theirs-conflict --ignore-externals returned non-zero exit status 1 in /mnt/git/webkit-commit-queue/Source/WebKit/chromium Error: 'depot_tools/gclient sync --force --reset --delete_unversioned_trees' failed 3 tries and returned 256 at Tools/Scripts/update-webkit-chromium line 106. Re-trying 'depot_tools/gclient sync --force --reset --delete_unversioned_trees' No such file or directory at /mnt/git/webkit-commit-queue/Tools/Scripts/webkitdirs.pm line 2370. Full output:
http://queues.webkit.org/results/11434206
Kentaro Hara
Comment 4
2012-02-06 03:17:26 PST
Created
attachment 125609
[details]
patch for commit
Kentaro Hara
Comment 5
2012-02-06 03:18:59 PST
Created
attachment 125610
[details]
patch for commit
WebKit Review Bot
Comment 6
2012-02-06 04:56:26 PST
Comment on
attachment 125610
[details]
patch for commit Clearing flags on attachment: 125610 Committed
r106798
: <
http://trac.webkit.org/changeset/106798
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug