summaryrefslogtreecommitdiffstats
path: root/webkit
diff options
context:
space:
mode:
authordimich@google.com <dimich@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-07-14 06:07:28 +0000
committerdimich@google.com <dimich@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-07-14 06:07:28 +0000
commit39ab2c80b518843be90dae580e80654594c8be7a (patch)
tree95480ccefb4e23d2ffc175942efe25c2f562872b /webkit
parentde9a074b3e34ea98f2063674e4d892ebc5889dca (diff)
downloadchromium_src-39ab2c80b518843be90dae580e80654594c8be7a.zip
chromium_src-39ab2c80b518843be90dae580e80654594c8be7a.tar.gz
chromium_src-39ab2c80b518843be90dae580e80654594c8be7a.tar.bz2
Fix the ConsoleLog test and enable it. It was previously disabled because WebKit change http://trac.webkit.org/changeset/45786 had split
the 'type' and 'level' parameters of the ConsoleMessage into 2 separate parameters, which changed the way CSS styles are formatted for messages in Inspector front-end. Updated test expectations. BUG=16684 TEST=the test DevToolsSanityTest.TestConsoleLog is enabled by this CL Review URL: http://codereview.chromium.org/155482 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20609 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit')
-rw-r--r--webkit/glue/devtools/js/tests.js20
1 files changed, 10 insertions, 10 deletions
diff --git a/webkit/glue/devtools/js/tests.js b/webkit/glue/devtools/js/tests.js
index 5f73415..ade8de3 100644
--- a/webkit/glue/devtools/js/tests.js
+++ b/webkit/glue/devtools/js/tests.js
@@ -482,7 +482,7 @@ TestSuite.prototype.testConsoleLog = function() {
var index = 0;
var test = this;
- var assertNext = function(line, message, opt_level, opt_count, opt_substr) {
+ var assertNext = function(line, message, opt_class, opt_count, opt_substr) {
var elem = messages[index++].toMessageElement();
var clazz = elem.getAttribute('class');
var expectation = (opt_count || '') + 'console_test_page.html:' +
@@ -492,25 +492,25 @@ TestSuite.prototype.testConsoleLog = function() {
} else {
test.assertEquals(expectation, elem.textContent);
}
- if (opt_level) {
- test.assertContains(clazz, 'console-' + opt_level + '-level');
+ if (opt_class) {
+ test.assertContains(clazz, 'console-' + opt_class);
}
};
- assertNext('5', 'log', 'log');
- assertNext('7', 'debug', 'log');
- assertNext('9', 'info', 'log');
- assertNext('11', 'warn', 'warning');
- assertNext('13', 'error', 'error');
+ assertNext('5', 'log', 'log-level');
+ assertNext('7', 'debug', 'log-level');
+ assertNext('9', 'info', 'log-level');
+ assertNext('11', 'warn', 'warning-level');
+ assertNext('13', 'error', 'error-level');
assertNext('15', 'Message format number 1, 2 and 3.5');
assertNext('17', 'Message format for string');
assertNext('19', 'Object Object');
- assertNext('22', 'repeated', 'log', 5);
+ assertNext('22', 'repeated', 'log-level', 5);
assertNext('26', 'count: 1');
assertNext('26', 'count: 2');
assertNext('29', 'group', 'group-title');
index++;
- assertNext('33', 'timer:', 'log', '', true);
+ assertNext('33', 'timer:', 'log-level', '', true);
};