Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(400)

Side by Side Diff: ui/events/platform/platform_event_source_unittest.cc

Issue 2211473003: Remove calls to deprecated MessageLoop methods on Windows and Linux. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: CR Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium 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 #include "ui/events/platform/platform_event_source.h" 5 #include "ui/events/platform/platform_event_source.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 484 matching lines...) Expand 10 before | Expand all | Expand 10 after
495 // Provides mechanism for running tests from inside an active message-loop. 495 // Provides mechanism for running tests from inside an active message-loop.
496 class PlatformEventTestWithMessageLoop : public PlatformEventTest { 496 class PlatformEventTestWithMessageLoop : public PlatformEventTest {
497 public: 497 public:
498 PlatformEventTestWithMessageLoop() {} 498 PlatformEventTestWithMessageLoop() {}
499 ~PlatformEventTestWithMessageLoop() override {} 499 ~PlatformEventTestWithMessageLoop() override {}
500 500
501 void Run() { 501 void Run() {
502 message_loop_.task_runner()->PostTask( 502 message_loop_.task_runner()->PostTask(
503 FROM_HERE, base::Bind(&PlatformEventTestWithMessageLoop::RunTest, 503 FROM_HERE, base::Bind(&PlatformEventTestWithMessageLoop::RunTest,
504 base::Unretained(this))); 504 base::Unretained(this)));
505 message_loop_.Run(); 505 base::RunLoop().Run();
506 } 506 }
507 507
508 protected: 508 protected:
509 void RunTest() { 509 void RunTest() {
510 RunTestImpl(); 510 RunTestImpl();
511 message_loop_.QuitWhenIdle(); 511 message_loop_.QuitWhenIdle();
sadrul 2016/08/08 20:42:42 Use RunLoop here too?
fdoray 2016/08/23 15:30:27 Done.
512 } 512 }
513 513
514 virtual void RunTestImpl() = 0; 514 virtual void RunTestImpl() = 0;
515 515
516 private: 516 private:
517 base::MessageLoopForUI message_loop_; 517 base::MessageLoopForUI message_loop_;
518 518
519 DISALLOW_COPY_AND_ASSIGN(PlatformEventTestWithMessageLoop); 519 DISALLOW_COPY_AND_ASSIGN(PlatformEventTestWithMessageLoop);
520 }; 520 };
521 521
(...skipping 249 matching lines...) Expand 10 before | Expand all | Expand 10 after
771 ASSERT_EQ(2u, list.size()); 771 ASSERT_EQ(2u, list.size());
772 EXPECT_EQ(15, list[0]); 772 EXPECT_EQ(15, list[0]);
773 EXPECT_EQ(10, list[1]); 773 EXPECT_EQ(10, list[1]);
774 } 774 }
775 }; 775 };
776 776
777 RUN_TEST_IN_MESSAGE_LOOP( 777 RUN_TEST_IN_MESSAGE_LOOP(
778 ConsecutiveOverriddenDispatcherInTheSameMessageLoopIteration) 778 ConsecutiveOverriddenDispatcherInTheSameMessageLoopIteration)
779 779
780 } // namespace ui 780 } // namespace ui
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698