748872 - Allow running of SPS profiling during capture

Status: RESOLVED FIXED

(Testing Graveyard :: Eideticker, defect)

Description

[William Lachance (:wlach)]

BenWa wrote a lightweight profiler called SPS, which also has a mobile variant:

https://github.com/bgirard/SPS_helper

It would be useful for eideticker to gather this information out-of-band so we could correlate it with the capture.

Comment 1

[Benoit Girard (:BenWa)]

Here's a sample of what mashing these two tools together would look like:
http://people.mozilla.org/~bgirard/cleopatra/?report=3f2f2196cc110a813b8c2dd30145133d78d57c6d

I would keep the dashboard the same with a new 'View profile' link if the profile data could be extracted. If it can't for whatever we reason eideticker would ignore profiling and keep working as it normally does.

Comment 2

[Benoit Girard (:BenWa)]

Here's a profile collected using my latest posted patch. The result is a 124KB zip per run.

William can you help me test this tomorrow with the capture. Once we get a few real runs with capture + profiling I'll do the follow up tasks:
1) Add a profile link to the dashboard
2) Add the capture video inside the profile like shown in Comment 1.

Comment 3

[Benoit Girard (:BenWa)]

Forgot the link:
http://people.mozilla.com/~bgirard/cleopatra/?report=75fa4d4047a13b2ee783faad16b9cb00ff021f47

Comment 4

[Benoit Girard (:BenWa)]

Here's a real profile pair:
http://people.mozilla.com/~bgirard/cleopatra/?zippedProfile=profiles/sps-profile-1346882045.2.zip&videoCapture=videos/video-1346882045.19.webm

Note that we're still missing a few patches to sync them up properly and I will likely add some video controls.

Comment 5

[William Lachance (:wlach)]

Let's get the video synchronization (bug 789597) done, then mark this as finished.

Comment 6

[Benoit Girard (:BenWa)]

Video synchronization is working (but will need more UI love to support zooming)!