Closed Bug 854468 Opened 8 years ago Closed 8 years ago

Add ability to record azure page drawing

Categories

(Core :: Layout, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: bas.schouten, Assigned: bas.schouten)

References

Details

Attachments

(2 files)

It would be much easier both for automation and debugging purposes to be able to create a recording trace of a webpage from the command line. This patch adds a command line flag to all mozilla builds that, when running Moz2D content, allows us to do for example:

'firefox -recording http://www.cnn.com/ -recording-output cnn.aer'

Which will create a chromeless basic trace of the first load of cnn.com. These traces could help us in measuring performance and fixing bugs across Azure backends.

I'm a little unsure where to put this code but for now I decided to put this in layout/tools, let me know if this should be somewhere else.
Attachment #729033 - Flags: review?(roc)
Comment on attachment 729033 [details] [diff] [review]
Add moz2d recording command line flags

Review of attachment 729033 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/tools/recording/recording-cmdline.js
@@ +41,5 @@
> +        /* Ignore the platform's online/offline status while running reftests. */
> +        var ios = Components.classes["@mozilla.org/network/io-service;1"]
> +                  .getService(Components.interfaces.nsIIOService2);
> +        ios.manageOfflineStatus = false;
> +        ios.offline = false;

I don't think you want this block.

::: layout/tools/recording/recording.xul
@@ +17,5 @@
> +        onunload="OnRecordingUnload();"
> +        style="background:white; overflow:hidden; width:800px; height:600px;"
> +        >
> +    <script type="application/ecmascript" src="recording.js" />
> +    <!-- The reftest browser element is dynamically created, here -->

Dynamically creating the element seems unnecessary in this case. Just hardcode it here.
Attachment #729033 - Flags: review?(roc) → review+
https://hg.mozilla.org/mozilla-central/rev/2e93bd943482
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Attachment #730186 - Flags: review?(ted) → review+
Depends on: 894796
Depends on: 900354
You need to log in before you can comment on or make changes to this bug.