 Chromium Code Reviews
 Chromium Code Reviews Issue 2697723004:
  Make regress-crbug-514081 less flaky by having max serialization size  (Closed)
    
  
    Issue 2697723004:
  Make regress-crbug-514081 less flaky by having max serialization size  (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 // Flags: --max-serializer-memory-usage=1000 | |
| 
Michael Achenbach
2017/02/15 09:35:24
Maybe we need to blacklist this flag on our intern
 
binji
2017/02/15 20:31:55
I removed the flag instead, as suggested by Yang.
 | |
| 5 if (this.Worker) { | 6 if (this.Worker) { | 
| 6 var __v_7 = new Worker('onmessage = function() {};'); | 7 var __v_7 = new Worker('onmessage = function() {};'); | 
| 7 var e; | 8 var e; | 
| 9 var ab = new ArrayBuffer(2000); | |
| 8 try { | 10 try { | 
| 9 var ab = new ArrayBuffer(2147483648); | 11 __v_7.postMessage(ab); | 
| 10 try { | 12 threw = false; | 
| 11 __v_7.postMessage(ab); | |
| 12 } catch (e) { | |
| 13 // postMessage failed, should be a DataCloneError message. | |
| 14 assertContains('cloned', e.message); | |
| 15 } | |
| 16 } catch (e) { | 13 } catch (e) { | 
| 17 // Creating the ArrayBuffer failed. | 14 // postMessage failed, should be a DataCloneError message. | 
| 18 assertInstanceof(e, RangeError); | 15 assertContains('cloned', e.message); | 
| 16 threw = true; | |
| 19 } | 17 } | 
| 18 assertTrue(threw, 'Should throw when trying to serialize large message.'); | |
| 20 } | 19 } | 
| OLD | NEW |