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

Side by Side Diff: chrome/test/ui/ppapi_uitest.cc

Issue 8789005: Simulate a user gesture in Fullscreen test so it can run on bots. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Simulate a user gesture in Fullscreen test so it can run on bots. Created 9 years 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | ppapi/tests/test_fullscreen.h » ('j') | 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "base/file_util.h" 5 #include "base/file_util.h"
6 #include "base/path_service.h" 6 #include "base/path_service.h"
7 #include "base/test/test_timeouts.h" 7 #include "base/test/test_timeouts.h"
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 #include "content/browser/plugin_service.h" 9 #include "content/browser/plugin_service.h"
10 #include "content/public/common/content_switches.h" 10 #include "content/public/common/content_switches.h"
(...skipping 493 matching lines...) Expand 10 before | Expand all | Expand 10 after
504 #define MAYBE_FlashFullscreen DISABLED_FlashFullscreen 504 #define MAYBE_FlashFullscreen DISABLED_FlashFullscreen
505 #define MAYBE_FlashFullscreen DISABLED_FlashFullscreen 505 #define MAYBE_FlashFullscreen DISABLED_FlashFullscreen
506 #endif 506 #endif
507 507
508 TEST_F(PPAPITest, MAYBE_FlashFullscreen) { 508 TEST_F(PPAPITest, MAYBE_FlashFullscreen) {
509 RunTestViaHTTP("FlashFullscreen"); 509 RunTestViaHTTP("FlashFullscreen");
510 } 510 }
511 TEST_F(OutOfProcessPPAPITest, MAYBE_FlashFullscreen) { 511 TEST_F(OutOfProcessPPAPITest, MAYBE_FlashFullscreen) {
512 RunTestViaHTTP("FlashFullscreen"); 512 RunTestViaHTTP("FlashFullscreen");
513 } 513 }
514 // New implementation only honors fullscreen requests within a context of 514
515 // a user gesture. Since we do not yet have an infrastructure for testing 515 // Fullscreen test fails on Mac.
516 // those under ppapi_tests, the tests below time out when run automtically. 516 #if defined(OS_MACOSX)
517 // To test the code, run them manually following the directions here: 517 #define MAYBE_Fullscreen FAILS_Fullscreen
518 // www.chromium.org/developers/design-documents/pepper-plugin-implementation 518 #else
519 // and click on the plugin area (gray square) to force fullscreen mode and 519 #define MAYBE_Fullscreen Fullscreen
520 // get the test unstuck. 520 #endif
521 TEST_F(PPAPITest, DISABLED_Fullscreen) { 521
522 RunTestViaHTTP("Fullscreen"); 522 // TODO(bbudge) Fix fullscreen on Mac.
523 } 523 TEST_PPAPI_IN_PROCESS_VIA_HTTP(MAYBE_Fullscreen)
524 TEST_F(OutOfProcessPPAPITest, DISABLED_Fullscreen) { 524 // TODO(bbudge) Will fail until we add an ACK message to extend user gesture.
525 RunTestViaHTTP("Fullscreen"); 525 TEST_PPAPI_OUT_OF_PROCESS_VIA_HTTP(FAILS_Fullscreen)
526 } 526 // TODO(bbudge) Enable when PPB_Testing_Dev SimulateInputEvent is proxied.
527 TEST_PPAPI_NACL_VIA_HTTP(DISABLED_Fullscreen)
527 528
528 TEST_PPAPI_IN_PROCESS(FlashClipboard) 529 TEST_PPAPI_IN_PROCESS(FlashClipboard)
529 TEST_PPAPI_OUT_OF_PROCESS(FlashClipboard) 530 TEST_PPAPI_OUT_OF_PROCESS(FlashClipboard)
530 531
531 #if defined(OS_POSIX) 532 #if defined(OS_POSIX)
532 #define MAYBE_DirectoryReader FLAKY_DirectoryReader 533 #define MAYBE_DirectoryReader FLAKY_DirectoryReader
533 #else 534 #else
534 #define MAYBE_DirectoryReader DirectoryReader 535 #define MAYBE_DirectoryReader DirectoryReader
535 #endif 536 #endif
536 537
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
600 TEST_PPAPI_OUT_OF_PROCESS(AudioConfig_InvalidConfigs) 601 TEST_PPAPI_OUT_OF_PROCESS(AudioConfig_InvalidConfigs)
601 602
602 TEST_PPAPI_IN_PROCESS(Audio_Creation) 603 TEST_PPAPI_IN_PROCESS(Audio_Creation)
603 TEST_PPAPI_IN_PROCESS(Audio_DestroyNoStop) 604 TEST_PPAPI_IN_PROCESS(Audio_DestroyNoStop)
604 TEST_PPAPI_IN_PROCESS(Audio_Failures) 605 TEST_PPAPI_IN_PROCESS(Audio_Failures)
605 TEST_PPAPI_OUT_OF_PROCESS(Audio_Creation) 606 TEST_PPAPI_OUT_OF_PROCESS(Audio_Creation)
606 TEST_PPAPI_OUT_OF_PROCESS(Audio_DestroyNoStop) 607 TEST_PPAPI_OUT_OF_PROCESS(Audio_DestroyNoStop)
607 TEST_PPAPI_OUT_OF_PROCESS(Audio_Failures) 608 TEST_PPAPI_OUT_OF_PROCESS(Audio_Failures)
608 609
609 #endif // ADDRESS_SANITIZER 610 #endif // ADDRESS_SANITIZER
OLDNEW
« no previous file with comments | « no previous file | ppapi/tests/test_fullscreen.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698