CompositorReport as used by DefaultDisplayBufferCompositor can't measure render time

[Daniel van Vugt]

CompositorReport as used by DefaultDisplayBufferCompositor can't measure render time.

Presently we have:

    // [A]
    report->began_frame(this);
    ...
    if (display_buffer.post_renderables_if_optimizable(renderable_list)) // [B] before page flip
    {
        renderer->suspend();
        report->finished_frame(true, this);
    }
    else
    {
        display_buffer.make_current();

        renderer->set_rotation(display_buffer.orientation());

        renderer->begin(); // TODO deprecatable now?
        renderer->render(renderable_list);
        display_buffer.post_update(); // swap buffers, [B], then page flip
        renderer->end();

        report->finished_frame(false, this);
    }
    // [C]

Compositor report correctly reports "frame time" as [C]-[A]. However a more useful measure is render time which is [B]-[A]. But we can't do that yet as we'd have to insert a reporting point in the middle of the post_ functions (or split them) so as to not include page flipping time in "render time", while still including the final eglSwapBuffers (if any).

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:mir at revision 2183, scheduled for release in mir, milestone 0.10.0

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:mir at revision None, scheduled for release in mir, milestone 0.10.0

[Daniel van Vugt (vanvugt)]

mir (0.10.0+15.04.20150107.2-0ubuntu1) vivid; urgency=medium