Bug 138848 - Don't let web process go to suspended state if there is active network request.
Summary: Don't let web process go to suspended state if there is active network request.
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on: 138582
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-18 15:54 PST by Yongjun Zhang
Modified: 2014-11-19 21:49 PST (History)
5 users (show)

See Also:


Attachments
Also acquire the activity token when we have active network requests on progress. (3.76 KB, patch)
2014-11-18 16:25 PST, Yongjun Zhang
thorton: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yongjun Zhang 2014-11-18 15:54:33 PST
<rdar://problem/18846532>
Comment 1 Yongjun Zhang 2014-11-18 16:25:23 PST
Created attachment 241829 [details]
Also acquire the activity token when we have active network requests on progress.
Comment 2 Tim Horton 2014-11-18 16:35:19 PST
Comment on attachment 241829 [details]
Also acquire the activity token when we have active network requests on progress.

It seems very wrong to keep the Web process from going to sleep for very long XHR/etc., seems like this will just result in a lot of very awake processes consuming all of your power, no?
Comment 3 Yongjun Zhang 2014-11-18 16:41:23 PST
(In reply to comment #2)
> Comment on attachment 241829 [details]
> Also acquire the activity token when we have active network requests on
> progress.
> 
> It seems very wrong to keep the Web process from going to sleep for very
> long XHR/etc., seems like this will just result in a lot of very awake
> processes consuming all of your power, no?

With the changes in https://bugs.webkit.org/show_bug.cgi?id=138582, we won't track long XHR or sub-resources after the main frame has loaded, and these sub-resources won't keep the process awake.
Comment 4 Alexey Proskuryakov 2014-11-18 22:24:16 PST
It may be worth re-titling the bug for accuracy, "active network request" may not be the best description for what progress tracker does.
Comment 5 Yongjun Zhang 2014-11-19 21:49:52 PST
This patch has the risk of holding web process too long if there is a long lasting XHR that has started before main frame has loaded. I will look at a different approach.