 Chromium Code Reviews
 Chromium Code Reviews Issue 2643723010:
  [d8] Use ValueSerializer for postMessage (instead of ad-hoc serializer)  (Closed)
    
  
    Issue 2643723010:
  [d8] Use ValueSerializer for postMessage (instead of ad-hoc serializer)  (Closed) 
  | OLD | NEW | 
|---|---|
| 1 // Copyright 2015 the V8 project authors. All rights reserved. | 1 // Copyright 2015 the V8 project authors. All rights reserved. | 
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be | 
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. | 
| 4 | 4 | 
| 5 if (this.Worker) { | 5 if (this.Worker) { | 
| 6 var __v_7 = new Worker('onmessage = function() {};'); | 6 var __v_7 = new Worker('onmessage = function() {};'); | 
| 7 var e; | |
| 7 try { | 8 try { | 
| 8 var ab = new ArrayBuffer(2147483648); | 9 var ab = new ArrayBuffer(2147483648); | 
| 
Michael Achenbach
2017/02/07 10:17:27
Still seeing flakes of this on windows, e.g.:
http
 | |
| 9 // If creating the ArrayBuffer succeeded, then postMessage should fail. | 10 try { | 
| 10 assertThrows(function() { __v_7.postMessage(ab); }); | 11 __v_7.postMessage(ab); | 
| 12 } catch (e) { | |
| 13 // postMessage failed, should be a DataCloneError message. | |
| 14 assertContains('cloned', e.message); | |
| 15 } | |
| 11 } catch (e) { | 16 } catch (e) { | 
| 12 // Creating the ArrayBuffer failed. | 17 // Creating the ArrayBuffer failed. | 
| 13 assertInstanceof(e, RangeError); | 18 assertInstanceof(e, RangeError); | 
| 14 } | 19 } | 
| 15 } | 20 } | 
| OLD | NEW |