Bug 34259 - [BREWMP] Port userIdleTime
Summary: [BREWMP] Port userIdleTime
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Other
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 35061 33564
  Show dependency treegraph
 
Reported: 2010-01-28 06:15 PST by Kwang Yul Seo
Modified: 2010-02-17 17:19 PST (History)
2 users (show)

See Also:


Attachments
Port userIdleTime (2.53 KB, patch)
2010-01-28 06:17 PST, Kwang Yul Seo
eric: review-
Details | Formatted Diff | Diff
Port userIdleTime (2.47 KB, patch)
2010-02-04 23:48 PST, Kwang Yul Seo
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kwang Yul Seo 2010-01-28 06:15:18 PST
Return an arbitrarily high userIdleTime so that releasing pages from the page cache isn't postponed.
Comment 1 Kwang Yul Seo 2010-01-28 06:17:22 PST
Created attachment 47613 [details]
Port userIdleTime
Comment 2 Eric Seidel (no email) 2010-02-01 15:46:49 PST
Comment on attachment 47613 [details]
Port userIdleTime

This seems like a bad way to do this.  Isn't there another way besides this hack?
Comment 3 Eric Seidel (no email) 2010-02-01 15:47:21 PST
Also, the year is now 2010, and I don't see how Apple's copyright would be needed on this file, since it looks nothing like any Apple version of this file.
Comment 4 Kwang Yul Seo 2010-02-04 23:47:07 PST
(In reply to comment #2)
> (From update of attachment 47613 [details])
> This seems like a bad way to do this.  Isn't there another way besides this
> hack?

Because BREWMP is a mobile platform, I don't want to postpone releasing pages. I think returning FLT_MAX is not so bad. Besides many ports do the same thing.
Comment 5 Kwang Yul Seo 2010-02-04 23:48:00 PST
Created attachment 48205 [details]
Port userIdleTime

Remove Apple's copyright and modified the year to 2010.
Comment 6 Eric Seidel (no email) 2010-02-17 15:35:40 PST
Comment on attachment 48205 [details]
Port userIdleTime

Bleh.  It sucks that so many ports use this hack.
Comment 7 WebKit Commit Bot 2010-02-17 17:19:16 PST
Comment on attachment 48205 [details]
Port userIdleTime

Clearing flags on attachment: 48205

Committed r54915: <http://trac.webkit.org/changeset/54915>
Comment 8 WebKit Commit Bot 2010-02-17 17:19:21 PST
All reviewed patches have been landed.  Closing bug.