Bug 142748 - Provide a better RunLoopObserver platform abstraction
Summary: Provide a better RunLoopObserver platform abstraction
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-03-16 14:36 PDT by Matt Baker
Modified: 2015-10-22 09:46 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Baker 2015-03-16 14:36:33 PDT
Move Mac platform code out of RunLoopObserver and make the API platform agnostic. This wrapper class is used sparingly in WebKit2 and WebCore, but the Inspector relies on it to time the run loop and provide FPS timeline information. A RunLoopObserver implementation will be required for any platform where we want FPS information to the available.
Comment 1 Radar WebKit Bug Importer 2015-03-16 14:38:00 PDT
<rdar://problem/20179287>
Comment 2 BJ Burg 2015-10-22 09:46:46 PDT
This bug is blocked on non-Mac ports figuring out how to create such an abstraction for their run loops.

In bug 150392, cgarcia writes:

"The glib main loop doesn't provide any way to know its status, and there isn't any GMainLoopObserver in glib either. We could implement something like that in glib, though, but in the meantime we need a different way to implement at least the rendering frames panel. What we can do with the current glib main loop, is to use custom sources, that we can dispatch. That way we can do whatever before and after the source is actually dispatched, which is what I'm doing in this patch. We plan to change some glib sources used in WebKit with custom sources like the one I'm using in this patch, but in a more generic way. Once we have that we could implement a RunLoopObserver, but only to monitor our own custom sources."