Chromium Code Reviews| Index: content/browser/devtools/protocol/devtools_protocol_browsertest.cc |
| diff --git a/content/browser/devtools/protocol/devtools_protocol_browsertest.cc b/content/browser/devtools/protocol/devtools_protocol_browsertest.cc |
| index d771e0e1ec8e04c94b8970a6d580ec6fdc593205..0f44ea9a4bedf276bbe8001179e39240bf0f0379 100644 |
| --- a/content/browser/devtools/protocol/devtools_protocol_browsertest.cc |
| +++ b/content/browser/devtools/protocol/devtools_protocol_browsertest.cc |
| @@ -1330,16 +1330,18 @@ IN_PROC_BROWSER_TEST_F(DevToolsProtocolTest, MAYBE_VirtualTimeTest) { |
| params->SetString("policy", "pause"); |
| SendCommand("Emulation.setVirtualTimePolicy", std::move(params), true); |
| + // TODO(scheduler-dev): Revisit timing when we have strict ordering |
|
Sami
2017/04/05 13:16:51
Can we now also enable this for Android?
altimin
2017/04/05 13:55:18
Let's ask Alex when he's back.
Sami
2017/04/05 14:38:15
I'm pretty sure we can -- this was the bug I was f
altimin
2017/04/05 15:10:22
I'll try it in a separate CL.
Sami
2017/04/05 15:16:55
Let's do it here since this is actually the patch
altimin
2017/04/05 15:23:33
Done.
|
| + // guarantees. |
| params.reset(new base::DictionaryValue()); |
| params->SetString("expression", |
| "setTimeout(function(){console.log('before')}, 1000);" |
| - "setTimeout(function(){console.log('after')}, 1001);"); |
| + "setTimeout(function(){console.log('after')}, 1002);"); |
| SendCommand("Runtime.evaluate", std::move(params), true); |
| // Let virtual time advance for one second. |
| params.reset(new base::DictionaryValue()); |
| params->SetString("policy", "advance"); |
| - params->SetInteger("budget", 1000); |
| + params->SetInteger("budget", 1001); |
| SendCommand("Emulation.setVirtualTimePolicy", std::move(params), true); |
| WaitForNotification("Emulation.virtualTimeBudgetExpired"); |