Bug 87854

Summary: Port RAMSize to WinCE
Product: WebKit Reporter: Patrick R. Gansterer <paroga>
Component: PlatformAssignee: Patrick R. Gansterer <paroga>
Status: RESOLVED FIXED    
Severity: Normal CC: ggaren, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Other   
Attachments:
Description Flags
Patch
none
Patch none

Patrick R. Gansterer
Reported 2012-05-30 07:57:43 PDT
Port RAMSiize to WinCE
Attachments
Patch (1.14 KB, patch)
2012-05-30 08:01 PDT, Patrick R. Gansterer 		no flags
DetailsFormatted DiffDiff
Patch (1.20 KB, patch)
2012-05-30 10:23 PDT, Patrick R. Gansterer 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Patrick R. Gansterer
Comment 1 2012-05-30 08:01:11 PDT
Created attachment 144818 [details] Patch
Geoffrey Garen
Comment 2 2012-05-30 10:12:52 PDT
Comment on attachment 144818 [details] Patch Please check for an error return value from GlobalMemoryStatus.
Patrick R. Gansterer
Comment 3 2012-05-30 10:14:17 PDT
Comment on attachment 144818 [details] Patch (In reply to comment #2) > (From update of attachment 144818 [details]) > Please check for an error return value from GlobalMemoryStatus. See http://msdn.microsoft.com/en-us/library/ms908470.aspx: GlobalMemoryStatus has no return value.
Geoffrey Garen
Comment 4 2012-05-30 10:19:57 PDT
> On computers with more than 4 GB of memory, the GlobalMemoryStatus function can return > incorrect information, reporting a value of –1 to indicate an overflow. This is what I meant by "error return".
Patrick R. Gansterer
Comment 5 2012-05-30 10:23:48 PDT
Created attachment 144859 [details] Patch
Patrick R. Gansterer
Comment 6 2012-06-04 09:45:21 PDT
ping?
Geoffrey Garen
Comment 7 2012-06-04 11:58:56 PDT
Comment on attachment 144859 [details] Patch r=me
WebKit Review Bot
Comment 8 2012-06-04 12:15:02 PDT
Comment on attachment 144859 [details] Patch Clearing flags on attachment: 144859 Committed r119412: <http://trac.webkit.org/changeset/119412>
WebKit Review Bot
Comment 9 2012-06-04 12:15:06 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.