[scheduler] Add TaskQueue::Observer

content/browser/devtools/protocol/devtools_protocol_browsertest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 1312 matching lines @@
 #define MAYBE_VirtualTimeTest VirtualTimeTest
 #endif
 IN_PROC_BROWSER_TEST_F(DevToolsProtocolTest, MAYBE_VirtualTimeTest) {
   NavigateToURLBlockUntilNavigationsComplete(shell(), GURL("about:blank"), 1);
   Attach();
 
   std::unique_ptr<base::DictionaryValue> params(new base::DictionaryValue());
   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 fixing with my EnqueueOrder
change. Could you turn this on here please?

[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 that fixes the test. The
trybots will tell us if I'm wrong -- at that point we can split.

[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");
 
   params.reset(new base::DictionaryValue());
   params->SetString("expression", "console.log('done')");
   SendCommand("Runtime.evaluate", std::move(params), true);
 
   // The second timer shold not fire.
   EXPECT_THAT(console_messages_, ElementsAre("before", "done"));
@@ skipping 411 matching lines @@
       EXPECT_EQ("polyglottal", value);
       found++;
     } else {
       FAIL();
     }
   }
   EXPECT_EQ(2u, found);
 }
 
 }  // namespace content