"ReferenceError: a is not defined | undefined | 45" in all jsreftest regress-446026-01.js runs

NEW
Unassigned

Status

()

Core
JavaScript Engine
5 years ago
4 years ago

People

(Reporter: RyanVM, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox24 affected, firefox25 affected, firefox26 affected)

Details

(Reporter)

Description

5 years ago
Due to recent TBPL log parser improvements, we are catching more errors in the logs than we used to.

https://tbpl.mozilla.org/php/getParsedLog.php?id=27115515&tree=Mozilla-Central

Ubuntu VM 12.04 mozilla-central opt test jsreftest on 2013-08-28 06:01:37 PDT for push 44e615a66f3f
slave: tst-linux32-ec2-329

06:11:26     INFO -  REFTEST TEST-START | file:///builds/slave/test/build/tests/jsreftest/tests/jsreftest.html?test=js1_5/Scope/regress-446026-01.js
06:11:26     INFO -  REFTEST TEST-LOAD | file:///builds/slave/test/build/tests/jsreftest/tests/jsreftest.html?test=js1_5/Scope/regress-446026-01.js | 1999 / 6601 (30%)
06:11:26     INFO -  BUGNUMBER: 446026
06:11:26     INFO -  STATUS: brian loves eval(s, o)
06:11:26     INFO -  JavaScript warning: file:///builds/slave/test/build/tests/jsreftest/tests/js1_5/Scope/regress-446026-01.js, line 32: Support for eval(code, scopeObject) has been removed. Use |with (scopeObject) eval(code);| instead.
06:11:26     INFO -  ReferenceError: a is not defined | undefined | 45
06:11:26     INFO -  1
06:11:26     INFO -  REFTEST TEST-PASS | file:///builds/slave/test/build/tests/jsreftest/tests/jsreftest.html?test=js1_5/Scope/regress-446026-01.js | brian loves eval(s, o)  item 1
06:11:26     INFO -  REFTEST TEST-PASS | file:///builds/slave/test/build/tests/jsreftest/tests/jsreftest.html?test=js1_5/Scope/regress-446026-01.js | brian loves eval(s, o)  item 2
06:11:26     INFO -  REFTEST INFO | Loading a blank page
06:11:26     INFO -  REFTEST TEST-END | file:///builds/slave/test/build/tests/jsreftest/tests/jsreftest.html?test=js1_5/Scope/regress-446026-01.js
Is the issue here this line in the test:

36 print(actual += " | " + foo + " | " + bar); // 21

here "actual" is an exception, so this will print a line starting with "ReferenceError: a is not defined" to the log.

Should the test just be printing something in front of that line?  Or is the log parser being over-eager here?
Flags: needinfo?(ryanvm)
(Reporter)

Comment 2

5 years ago
403 edmorley
Flags: needinfo?(ryanvm) → needinfo?(emorley)

Comment 3

5 years ago
Guess we just need bug 829684.
Depends on: 829684
Flags: needinfo?(emorley)

Updated

5 years ago
No longer blocks: 892958

Updated

5 years ago
Blocks: 910614

Updated

5 years ago
Blocks: 920191

Updated

5 years ago
No longer blocks: 910614
(Assignee)

Updated

4 years ago
Assignee: general → nobody
You need to log in before you can comment on or make changes to this bug.