[ash-md] Added a delay between system menu default/detailed view transitions.

base/test/test_mock_time_task_runner.h

 // Copyright 2015 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.
 
 #ifndef BASE_TEST_TEST_MOCK_TIME_TASK_RUNNER_H_
 #define BASE_TEST_TEST_MOCK_TIME_TASK_RUNNER_H_
 
 #include <stddef.h>
 
 #include <memory>
@@ skipping 147 matching lines @@
 
   // The ordinal to use for the next task. Must only be accessed while the
   // |tasks_lock_| is held.
   size_t next_task_ordinal_;
 
   Lock tasks_lock_;
 
   DISALLOW_COPY_AND_ASSIGN(TestMockTimeTaskRunner);
 };
 
+// A scoped wrapper around TestMockTimeTaskRunner that replaces message loop's

[gab, 2016/11/15 19:52:01]

s/message loop's task runner/MessageLoop::current()'s task runner (and
consequently ThreadTaskRunnerHandle)/

[bruthig, 2016/11/15 21:52:45]

Done.

+// task runner with a TestMockTimeTaskRunner and resets it back at the end of
+// the scope.
+class ScopedMockTaskRunnerWrapper {

[gab, 2016/11/15 19:52:01]

Make name ML specific, i.e. ScopedMockMessageLoopTaskRunnerWrapper

[danakj, 2016/11/15 19:58:13]

I think this class could go in its own .h and .cc files also. 1 per file is the
one true way.

[bruthig, 2016/11/15 21:52:45]

Done.

[bruthig, 2016/11/15 21:52:45]

Done.

+ public:
+  ScopedMockTaskRunnerWrapper();
+  virtual ~ScopedMockTaskRunnerWrapper();

[gab, 2016/11/15 19:52:01]

non-virtual destructor for class with no virtual methods.

[bruthig, 2016/11/15 21:52:45]

Done.

+
+  base::TestMockTimeTaskRunner* task_runner() { return task_runner_.get(); }

[gab, 2016/11/15 19:52:01]

const

[bruthig, 2016/11/15 21:52:45]

If we return a "const TestMockTimeTaskRunner*" then TrayDetailsViewTest won't be
able to invoke FastForwardBy() on it.  Or am I misunderstanding?

[gab, 2016/11/15 22:02:05]

Sorry, I meant to make the method const, i.e. 

base::TestMockTimeTaskRunner* task_runner() const { return task_runner_.get(); }
                                            ^^^^^

[bruthig, 2016/11/15 22:17:57]

Are you sure? The style guide advises not to make functions const if they are
returning a non-const pointer to a member variable.

[gab, 2016/11/15 23:16:59]

Ah interesting, had missed that part of it, please ignore me :)

+
+ private:
+  scoped_refptr<base::TestMockTimeTaskRunner> task_runner_;

[gab, 2016/11/15 19:52:01]

const (will also require to move initialization to constructor's initializer
list as suggested in .cc)

[bruthig, 2016/11/15 21:52:45]

Done, but not sure if you meant
const scoped_refptr<TestMockTimeTaskRunner> OR
scoped_refptr<const TestMockTimeTaskRunner>.

[gab, 2016/11/15 22:02:05]

const scoped_refptr<TestMockTimeTaskRunner>

(i.e. pointer is not changed outside of initializer list -- whereas
scoped_refptr<const TestMockTimeTaskRunner> makes for an unusable TaskRunner per
PostTask() not being const)

[bruthig, 2016/11/15 22:17:57]

Thx

+  scoped_refptr<base::SingleThreadTaskRunner> previous_task_runner_;

[gab, 2016/11/15 19:52:01]

nit: no base:: prefix above nor in .cc file (this is in base:: already)

[bruthig, 2016/11/15 21:52:44]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(ScopedMockTaskRunnerWrapper);
+};
+
 }  // namespace base
 
 #endif  // BASE_TEST_TEST_MOCK_TIME_TASK_RUNNER_H_