Bug 115024

Summary: Remove fast/js/i18n-bindings-locale.html
Product: WebKit Reporter: Seokju Kwon <seokju>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ggaren, gyuyoung.kim, kadam, oliver, rakuco, zan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Seokju Kwon
Reported 2013-04-23 02:09:11 PDT
Remove V8-i18n-extension test (r78095) from all ports.
Attachments
Patch (14.18 KB, patch)
2013-04-23 02:11 PDT, Seokju Kwon 		no flags
DetailsFormatted DiffDiff
Patch (14.19 KB, patch)
2013-04-23 02:56 PDT, Seokju Kwon 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Seokju Kwon
Comment 1 2013-04-23 02:11:42 PDT
Created attachment 199204 [details] Patch
Zan Dobersek
Comment 2 2013-04-23 02:53:21 PDT
Comment on attachment 199204 [details] Patch This seems V8-specific and should be safe to remove. Also I can't find any support for the v8Locale or the Locale object in the current V8 code. Despite that I don't feel all that comfortable cq+ing the patch. I'd much more like the patch to be properly reviewed by the JSC people.
Seokju Kwon
Comment 3 2013-04-23 02:56:23 PDT
Created attachment 199209 [details] Patch
Geoffrey Garen
Comment 4 2013-04-23 11:41:02 PDT
Comment on attachment 199209 [details] Patch r=me
WebKit Commit Bot
Comment 5 2013-04-23 12:08:33 PDT
Comment on attachment 199209 [details] Patch Clearing flags on attachment: 199209 Committed r148977: <http://trac.webkit.org/changeset/148977>
WebKit Commit Bot
Comment 6 2013-04-23 12:08:36 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.