Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(165)

Issue 96158: Updates the benchmarking code to including sizes and times (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years, 10 months ago by Jasvir
Modified:
16 years, 10 months ago
Reviewers:
metaweta
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

* Added measurement of sizes of plain and cajoled code, rendered pretty or minified and encoded plain or gzipped. * Changed the measurement of performance to actual time taken rather than a V8 "score". V8 scores do not give useful information over history nor are comparable to other benchmarks. @3611

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+154 lines, -18 lines) Patch
M tests/com/google/caja/demos/benchmarks/BenchmarkRunner.java View 5 chunks +26 lines, -16 lines 0 comments Download
A tests/com/google/caja/demos/benchmarks/BenchmarkSize.java View 1 chunk +122 lines, -0 lines 0 comments Download
M tests/com/google/caja/parser/js/parsertest10.js View 1 chunk +2 lines, -1 line 0 comments Download
M tests/com/google/caja/util/CajaTestCase.java View 1 chunk +4 lines, -0 lines 0 comments Download
M tests/com/google/caja/util/RhinoTestBed.java View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 2
Jasvir
16 years, 10 months ago (2009-07-26 06:28:42 UTC) #1
metaweta
16 years, 10 months ago (2009-07-26 06:35:18 UTC) #2
On 2009/07/26 06:28:42, jasvir wrote:
> 

LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b