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

Side by Side Diff: remoting/host/win/worker_process_launcher_unittest.cc

Issue 2450063002: Disabling WorkerProcessLauncherTest.Crash (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "remoting/host/win/worker_process_launcher.h" 5 #include "remoting/host/win/worker_process_launcher.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <utility> 9 #include <utility>
10 10
(...skipping 463 matching lines...) Expand 10 before | Expand all | Expand 10 after
474 EXPECT_CALL(server_listener_, OnPermanentError(_)) 474 EXPECT_CALL(server_listener_, OnPermanentError(_))
475 .Times(1) 475 .Times(1)
476 .WillOnce(InvokeWithoutArgs(this, 476 .WillOnce(InvokeWithoutArgs(this,
477 &WorkerProcessLauncherTest::StopWorker)); 477 &WorkerProcessLauncherTest::StopWorker));
478 478
479 StartWorker(); 479 StartWorker();
480 base::RunLoop().Run(); 480 base::RunLoop().Run();
481 } 481 }
482 482
483 // Requests the worker to crash and expects it to honor the request. 483 // Requests the worker to crash and expects it to honor the request.
484 TEST_F(WorkerProcessLauncherTest, Crash) { 484 // Disabled: http://crbug.com/659219
485 TEST_F(WorkerProcessLauncherTest, DISABLED_Crash) {
485 EXPECT_CALL(*launcher_delegate_, LaunchProcess(_)) 486 EXPECT_CALL(*launcher_delegate_, LaunchProcess(_))
486 .Times(2) 487 .Times(2)
487 .WillRepeatedly(Invoke( 488 .WillRepeatedly(Invoke(
488 this, &WorkerProcessLauncherTest::LaunchProcessAndConnect)); 489 this, &WorkerProcessLauncherTest::LaunchProcessAndConnect));
489 490
490 EXPECT_CALL(server_listener_, OnChannelConnected(_)) 491 EXPECT_CALL(server_listener_, OnChannelConnected(_))
491 .Times(2) 492 .Times(2)
492 .WillOnce(InvokeWithoutArgs(this, 493 .WillOnce(InvokeWithoutArgs(this,
493 &WorkerProcessLauncherTest::CrashWorker)) 494 &WorkerProcessLauncherTest::CrashWorker))
494 .WillOnce(InvokeWithoutArgs(this, 495 .WillOnce(InvokeWithoutArgs(this,
(...skipping 29 matching lines...) Expand all
524 EXPECT_CALL(client_listener_, OnCrash(_, _, _)) 525 EXPECT_CALL(client_listener_, OnCrash(_, _, _))
525 .Times(1) 526 .Times(1)
526 .WillOnce(InvokeWithoutArgs( 527 .WillOnce(InvokeWithoutArgs(
527 this, &WorkerProcessLauncherTest::SendFakeMessageToLauncher)); 528 this, &WorkerProcessLauncherTest::SendFakeMessageToLauncher));
528 529
529 StartWorker(); 530 StartWorker();
530 base::RunLoop().Run(); 531 base::RunLoop().Run();
531 } 532 }
532 533
533 } // namespace remoting 534 } // namespace remoting
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698