Closed
Bug 854468
Opened 11 years ago
Closed 11 years ago
Add ability to record azure page drawing
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
mozilla22
People
(Reporter: bas.schouten, Assigned: bas.schouten)
References
Details
Attachments
(2 files)
9.16 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
1.68 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
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+
Assignee | ||
Comment 2•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/2e93bd943482
Comment 3•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2e93bd943482
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #730186 -
Flags: review?(ted)
Updated•11 years ago
|
Attachment #730186 -
Flags: review?(ted) → review+
Comment 5•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4573d094fa71
You need to log in
before you can comment on or make changes to this bug.
Description
•