Bug 43832

Summary: Web Inspector: toJSON serialization for Numbers should be fixed.
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Ilya Tikhonovsky <loislo>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, bweinstein, commit-queue, joepeck, keishi, krit, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 18994    
Bug Blocks:    
Attachments:
Description Flags
[patch] initial version. none

Ilya Tikhonovsky
Reported 2010-08-11 00:10:19 PDT
Right now it is using printf("%f") and has the problem with non-english locales because they can have another decimal separators. The problem was temporary resolved for comma separator by hack. See https://bugs.webkit.org/show_bug.cgi?id=42755
Attachments
[patch] initial version. (1.50 KB, patch)
2010-10-18 07:34 PDT, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dirk Schulze
Comment 1 2010-08-11 00:23:28 PDT
This is a duplication of 18994. Let the bug open so that we won't forget to eliminate the hack, once the locale problem got fixed.
Ilya Tikhonovsky
Comment 2 2010-10-18 07:34:30 PDT
Created attachment 71033 [details] [patch] initial version.
WebKit Commit Bot
Comment 3 2010-10-18 08:24:32 PDT
Comment on attachment 71033 [details] [patch] initial version. Clearing flags on attachment: 71033 Committed r69959: <http://trac.webkit.org/changeset/69959>
WebKit Commit Bot
Comment 4 2010-10-18 08:24:39 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.