The output from runatf is as follows: Tested LayoutTests: 928 Failed LayoutTests: 0 Layout test results summary in X:/S60\LayoutTest_results.txt Performance results in X:/S60\performance_test_results.html But there is no .html file, only the .txt
confirmed performance_test_results.html not copied to toplevel
Created attachment 8564 [details] Updates WebUI to run perf test when -autotest specified
Patch causes reindeer to crash on startup. Reverting patch fixes reindeer startup crash, so r- for the moment.
Comment on attachment 8564 [details] Updates WebUI to run perf test when -autotest specified Patch seems to crash reindeer at startup. Is it doing more than it needs to?
Created attachment 8607 [details] Runs performance tests when -autotest specified
Comment on attachment 8564 [details] Updates WebUI to run perf test when -autotest specified r=me Zalan, please review.
Created attachment 8609 [details] Added Changlog to previous patch
Comment on attachment 8609 [details] Added Changlog to previous patch >if ( !err && please check for err == KErrNone (symbian coding convetion) >numTestsRun->Des().Length() use numTestsRun->Length() instead. ->Des() creates a copy of that descriptor. >PrepPerformanceResultsFileL() cleanupCount variable is not needed. >(TDesC&)KRunAllTests() change DoAutomatedTestsActionL signature from TDesC& aActionString to const TDesC& aActionString. no need for typecast.
Created attachment 8624 [details] Added Zalan's review changes to the patch This patch contains the changes recommended by Zalan's review.
Created attachment 8625 [details] Adding changelog
(In reply to comment #10) > Created an attachment (id=8625) [edit] > Adding changelog > Please add changelog (see patch for details) Also when adding a new patch, click the obsolete button on earlier patches Please request review of patches by setting review flag to ? & entering email address
Created attachment 8627 [details] Second pass at changes after review (AR2). Missed the typedef review comment. Added the typedef changes mentioned in the review.
Comment on attachment 8627 [details] Second pass at changes after review (AR2). Missed the typedef review comment. Removed the typedefs in WebUiWindowContainer.cpp. Where the call to DoAutomatedTestsActionL() is called.
Comment on attachment 8627 [details] Second pass at changes after review (AR2). Missed the typedef review comment. r=me looks great!
committed r14648 & setting to FIXED. Thanks!
Comment on attachment 8625 [details] Adding changelog Marked the change log as review+ as the bug (and hopefully) the changlog has already been commited.
Comment on attachment 8564 [details] Updates WebUI to run perf test when -autotest specified Clearing the review flag on this to remove it from the review queue, since this patch was obsoleted.
Set to CLOSED.