Revision: 3813
Author: [email protected]
Date: Mon Feb 8 00:55:01 2010
Log: Fix tests after r3811.
[email protected]
Review URL: http://codereview.chromium.org/574027
http://code.google.com/p/v8/source/detail?r=3813
Modified:
/branches/bleeding_edge/test/mjsunit/tools/logreader.js
/branches/bleeding_edge/test/mjsunit/tools/tickprocessor.js
/branches/bleeding_edge/tools/tickprocessor.js
=======================================
--- /branches/bleeding_edge/test/mjsunit/tools/logreader.js Thu Jan 21
08:42:41 2010
+++ /branches/bleeding_edge/test/mjsunit/tools/logreader.js Mon Feb 8
00:55:01 2010
@@ -80,19 +80,3 @@
assertEquals('bbbbaaaa', reader.expandBackRef_('bbbb#2:4'));
assertEquals('"#1:1"', reader.expandBackRef_('"#1:1"'));
})();
-
-
-// See http://code.google.com/p/v8/issues/detail?id=420
-(function testReadingTruncatedLog() {
- // Having an incorrect event in the middle of a log should throw an
exception.
- var reader1 = new devtools.profiler.LogReader({});
- assertThrows(function() {
- reader1.processLogChunk('alias,a,b\nxxxx\nalias,c,d\n');
- });
-
- // But having it as the last record should not.
- var reader2 = new devtools.profiler.LogReader({});
- assertDoesNotThrow(function() {
- reader2.processLogChunk('alias,a,b\nalias,c,d\nxxxx');
- });
-})();
=======================================
--- /branches/bleeding_edge/test/mjsunit/tools/tickprocessor.js Thu Jan 21
08:42:41 2010
+++ /branches/bleeding_edge/test/mjsunit/tools/tickprocessor.js Mon Feb 8
00:55:01 2010
@@ -379,9 +379,7 @@
var tp = new TickProcessor(
new CppEntriesProviderMock(), separateIc, ignoreUnknown,
stateFilter);
var pm = new PrintMonitor(testsPath + refOutput);
- tp.processLogFile(testsPath + logInput);
- // Hack file name to avoid dealing with platform specifics.
- tp.lastLogFileName_ = 'v8.log';
+ tp.processLogFileInTest(testsPath + logInput);
tp.printStatistics();
pm.finish();
};
=======================================
--- /branches/bleeding_edge/tools/tickprocessor.js Mon Feb 8 00:15:18 2010
+++ /branches/bleeding_edge/tools/tickprocessor.js Mon Feb 8 00:55:01 2010
@@ -269,6 +269,14 @@
};
+TickProcessor.prototype.processLogFileInTest = function(fileName) {
+ // Hack file name to avoid dealing with platform specifics.
+ this.lastLogFileName_ = 'v8.log';
+ var contents = readFile(fileName);
+ this.processLogChunk(contents);
+};
+
+
TickProcessor.prototype.processSharedLibrary = function(
name, startAddr, endAddr) {
var entry = this.profile_.addLibrary(name, startAddr, endAddr);
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
