Bug 138155

Summary:

[iOS] WebProcess needs to take a background task assertion to prevent being killed for "suspended with locked system files"

Product:

WebKit

Reporter:

Daniel Bates <dbates>

Component:

WebKit2

Assignee:

Daniel Bates <dbates>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aestes, andersca, ap, barraclough, beidson, ddkilzer, sam, thorton

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

iPhone / iPad

OS:

iOS 8.1

Attachments:

Description	Flags
Patch	none

Description Daniel Bates 2014-10-28 16:02:38 PDT

Towards mitigating the web content process being killed when suspended holding lock files, we should have the web content process notify the UI process when it is holding such files so that the UI process can acquire a background task assertion to allow both to continue to run for an extended period of time in the background. In particular, we want to take a background task assertion when performing a SQLite transaction(s).

Comment 1 Daniel Bates 2014-10-28 16:03:22 PDT

<rdar://problem/17939303>

Comment 2 Daniel Bates 2014-10-28 16:05:50 PDT

Created attachment 240575 [details]
Patch

Comment 3 Daniel Bates 2014-10-28 16:23:20 PDT

I open to suggestions on this patch.

The class WebSQLiteDatabaseTracker (included in the patch) isn't specific to iOS and hence can be used by other ports. At the time of writing, the iOS port is the only port that implements a non-trivial ProcessAssertion class to acquire a background task assertion. I guarded the instantiation of this class with PLATFORM(IOS) though I placed the class files in Source/WebKit2/WebProcess/WebCoreSupport/ (should I move them to an iOS-specific directory?) because I was unclear if it was acceptable to instantiate for all ports a WebSQLiteDatabaseTracker, which would send messages to the UIProcess (*), given that iOS is the only port that makes use of such messages. I was also unclear if we should either add a new compile-time guard for this feature to avoid the instantiation on non-iOS ports, say ENABLE_SQL_DATABASE_TRACKER, or if it's acceptable to instantiate WebSQLiteDatabaseTracker for all ports and have it only enabled for iOS (say, by not calling SQLDatabaseTracker::setClient() on non-iOS ports). Let me know if there is a preferred way to expose this functionality.

(*) I didn't measure the performance implications of such messaging.

Comment 4 Daniel Bates 2014-11-06 08:40:22 PST

Comment on attachment 240575 [details]
Patch

Clearing flags on attachment: 240575

Committed r175698: <http://trac.webkit.org/changeset/175698>

Comment 5 Daniel Bates 2014-11-06 08:40:26 PST

All reviewed patches have been landed.  Closing bug.