 Chromium Code Reviews
 Chromium Code Reviews Issue 1214053004:
  [d8] bounds-check before getting Shell::Worker internal field  (Closed) 
  Base URL: https://chromium.googlesource.com/v8/v8.git@master
    
  
    Issue 1214053004:
  [d8] bounds-check before getting Shell::Worker internal field  (Closed) 
  Base URL: https://chromium.googlesource.com/v8/v8.git@master| Index: test/mjsunit/regress/regress-4271.js | 
| diff --git a/test/mjsunit/regress/regress-4271.js b/test/mjsunit/regress/regress-4271.js | 
| new file mode 100644 | 
| index 0000000000000000000000000000000000000000..0a6005631902eb024c3cc01304c735e5de97c872 | 
| --- /dev/null | 
| +++ b/test/mjsunit/regress/regress-4271.js | 
| @@ -0,0 +1,26 @@ | 
| +// Copyright 2015 the V8 project authors. All rights reserved. | 
| +// Use of this source code is governed by a BSD-style license that can be | 
| +// found in the LICENSE file. | 
| + | 
| +if (typeof Worker === 'function') test(); | 
| 
binji
2015/07/07 21:48:20
I've just been doing 
if (this.Worker) {
  ...
}
 | 
| + | 
| +function test() { | 
| + // Throw rather than overflow internal field index | 
| + assertThrows(function() { | 
| + Worker.prototype.terminate(); | 
| + }); | 
| + | 
| + assertThrows(function() { | 
| + Worker.prototype.getMessage(); | 
| + }); | 
| + | 
| + assertThrows(function() { | 
| + Worker.prototype.postMessage({}); | 
| + }); | 
| + | 
| + // Don't throw for real worker | 
| + var worker = new Worker(''); | 
| + worker.getMessage(); | 
| + worker.postMessage({}); | 
| + worker.terminate(); | 
| +} |