New issue
Advanced search Search tips

Issue 693823 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Ignition JS Engine (when turned on) fails reporting `ReferenceError: this is not defined`

Reported by kobalice...@gmail.com, Feb 18 2017

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.75 Safari/537.36

Steps to reproduce the problem:
1. npm install asmdb
   or git clone https://github.com/asmjit/asmdb.git
2. Use the attached file (chromebug.html) that includes 3 files of that package.
3. Open chromebug.html with ignition engine turned on

What is the expected behavior?
No exception should be thrown, the code is valid JS. It works in node, other browsers, and chromium with Ignition JS engine turned off.

What went wrong?
JS throws exception when it shouldn't. It seems that the first parameter in Instruction constructor (db) is incorrectly set to `this` as well, at least chrome inspector shows same value for both of them, which is incorrect.

Did this work before? N/A 

Chrome version: 55.0.2883.75  Channel: n/a
OS Version: 
Flash Version:

 
chromebug.html
583 bytes View Download

Comment 1 by woxxom@gmail.com, Feb 18 2017

Ignition is a work-in-progress so it got a lot of updates since v55 you report about. I can't reproduce the bug in the latest v58 canary.
Labels: Needs-Milestone
Components: -Blink Blink>JavaScript
Labels: TE-NeedsTriageHelp
Status: Fixed (was: Unconfirmed)
Closing according to #1. Lot's of correctness bugs fixed since 5.5.

Sign in to add a comment