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

Side by Side Diff: chrome/browser/apps/guest_view/web_view_browsertest.cc

Issue 1066563006: GuestView: Move GuestViewManager extension dependencies to ExtensionsGuestViewManagerDelegate (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@extensions_guest_view_message_filter
Patch Set: Fixed tests I hope Created 5 years, 8 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/path_service.h" 5 #include "base/path_service.h"
6 #include "base/process/process.h" 6 #include "base/process/process.h"
7 #include "base/strings/stringprintf.h" 7 #include "base/strings/stringprintf.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "chrome/app/chrome_command_ids.h" 9 #include "chrome/app/chrome_command_ids.h"
10 #include "chrome/browser/apps/app_browsertest_util.h" 10 #include "chrome/browser/apps/app_browsertest_util.h"
(...skipping 18 matching lines...) Expand all
29 #include "content/public/browser/web_contents_delegate.h" 29 #include "content/public/browser/web_contents_delegate.h"
30 #include "content/public/common/content_switches.h" 30 #include "content/public/common/content_switches.h"
31 #include "content/public/test/browser_test_utils.h" 31 #include "content/public/test/browser_test_utils.h"
32 #include "content/public/test/fake_speech_recognition_manager.h" 32 #include "content/public/test/fake_speech_recognition_manager.h"
33 #include "content/public/test/test_renderer_host.h" 33 #include "content/public/test/test_renderer_host.h"
34 #include "extensions/browser/api/declarative/rules_registry.h" 34 #include "extensions/browser/api/declarative/rules_registry.h"
35 #include "extensions/browser/api/declarative/rules_registry_service.h" 35 #include "extensions/browser/api/declarative/rules_registry_service.h"
36 #include "extensions/browser/api/declarative/test_rules_registry.h" 36 #include "extensions/browser/api/declarative/test_rules_registry.h"
37 #include "extensions/browser/api/declarative_webrequest/webrequest_constants.h" 37 #include "extensions/browser/api/declarative_webrequest/webrequest_constants.h"
38 #include "extensions/browser/app_window/native_app_window.h" 38 #include "extensions/browser/app_window/native_app_window.h"
39 #include "extensions/browser/guest_view/extensions_guest_view_manager_delegate.h "
39 #include "extensions/browser/guest_view/guest_view_manager.h" 40 #include "extensions/browser/guest_view/guest_view_manager.h"
40 #include "extensions/browser/guest_view/guest_view_manager_factory.h" 41 #include "extensions/browser/guest_view/guest_view_manager_factory.h"
41 #include "extensions/browser/guest_view/test_guest_view_manager.h" 42 #include "extensions/browser/guest_view/test_guest_view_manager.h"
42 #include "extensions/browser/guest_view/web_view/web_view_guest.h" 43 #include "extensions/browser/guest_view/web_view/web_view_guest.h"
43 #include "extensions/common/extension.h" 44 #include "extensions/common/extension.h"
44 #include "extensions/common/extensions_client.h" 45 #include "extensions/common/extensions_client.h"
45 #include "extensions/test/extension_test_message_listener.h" 46 #include "extensions/test/extension_test_message_listener.h"
46 #include "media/base/media_switches.h" 47 #include "media/base/media_switches.h"
47 #include "net/test/embedded_test_server/embedded_test_server.h" 48 #include "net/test/embedded_test_server/embedded_test_server.h"
48 #include "net/test/embedded_test_server/http_request.h" 49 #include "net/test/embedded_test_server/http_request.h"
(...skipping 11 matching lines...) Expand all
60 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h" 61 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
61 #include "chrome/browser/chromeos/accessibility/speech_monitor.h" 62 #include "chrome/browser/chromeos/accessibility/speech_monitor.h"
62 #endif 63 #endif
63 64
64 // For fine-grained suppression on flaky tests. 65 // For fine-grained suppression on flaky tests.
65 #if defined(OS_WIN) 66 #if defined(OS_WIN)
66 #include "base/win/windows_version.h" 67 #include "base/win/windows_version.h"
67 #endif 68 #endif
68 69
69 using extensions::ContextMenuMatcher; 70 using extensions::ContextMenuMatcher;
71 using extensions::ExtensionsGuestViewManagerDelegate;
72 using extensions::GuestViewManager;
73 using extensions::TestGuestViewManager;
70 using extensions::MenuItem; 74 using extensions::MenuItem;
71 using prerender::PrerenderLinkManager; 75 using prerender::PrerenderLinkManager;
72 using prerender::PrerenderLinkManagerFactory; 76 using prerender::PrerenderLinkManagerFactory;
73 using task_manager::browsertest_util::MatchAboutBlankTab; 77 using task_manager::browsertest_util::MatchAboutBlankTab;
74 using task_manager::browsertest_util::MatchAnyApp; 78 using task_manager::browsertest_util::MatchAnyApp;
75 using task_manager::browsertest_util::MatchAnyBackground; 79 using task_manager::browsertest_util::MatchAnyBackground;
76 using task_manager::browsertest_util::MatchAnyTab; 80 using task_manager::browsertest_util::MatchAnyTab;
77 using task_manager::browsertest_util::MatchAnyWebView; 81 using task_manager::browsertest_util::MatchAnyWebView;
78 using task_manager::browsertest_util::MatchApp; 82 using task_manager::browsertest_util::MatchApp;
79 using task_manager::browsertest_util::MatchBackground; 83 using task_manager::browsertest_util::MatchBackground;
(...skipping 729 matching lines...) Expand 10 before | Expand all | Expand 10 after
809 return guest_web_contents_; 813 return guest_web_contents_;
810 } 814 }
811 815
812 content::WebContents* GetEmbedderWebContents() { 816 content::WebContents* GetEmbedderWebContents() {
813 if (!embedder_web_contents_) { 817 if (!embedder_web_contents_) {
814 embedder_web_contents_ = GetFirstAppWindowWebContents(); 818 embedder_web_contents_ = GetFirstAppWindowWebContents();
815 } 819 }
816 return embedder_web_contents_; 820 return embedder_web_contents_;
817 } 821 }
818 822
819 extensions::TestGuestViewManager* GetGuestViewManager() { 823 TestGuestViewManager* GetGuestViewManager() {
820 return static_cast<extensions::TestGuestViewManager*>( 824 TestGuestViewManager* manager = static_cast<TestGuestViewManager*>(
821 extensions::TestGuestViewManager::FromBrowserContext( 825 TestGuestViewManager::FromBrowserContext(browser()->profile()));
822 browser()->profile())); 826 // TestGuestViewManager::WaitForSingleGuestCreated may and will get called
827 // before a guest is created.
828 if (!manager) {
829 manager = static_cast<TestGuestViewManager*>(
830 GuestViewManager::CreateWithDelegate(
831 browser()->profile(),
832 scoped_ptr<guestview::GuestViewManagerDelegate>(
833 new ExtensionsGuestViewManagerDelegate(
834 browser()->profile()))));
835 }
836 return manager;
823 } 837 }
824 838
825 WebViewTest() : guest_web_contents_(NULL), 839 WebViewTest() : guest_web_contents_(NULL),
826 embedder_web_contents_(NULL) { 840 embedder_web_contents_(NULL) {
827 extensions::GuestViewManager::set_factory_for_testing(&factory_); 841 extensions::GuestViewManager::set_factory_for_testing(&factory_);
828 } 842 }
829 843
830 private: 844 private:
831 bool UsesFakeSpeech() { 845 bool UsesFakeSpeech() {
832 const testing::TestInfo* const test_info = 846 const testing::TestInfo* const test_info =
(...skipping 1962 matching lines...) Expand 10 before | Expand all | Expand 10 after
2795 IN_PROC_BROWSER_TEST_F(WebViewTest, AllowTransparencyAndAllowScalingPropagate) { 2809 IN_PROC_BROWSER_TEST_F(WebViewTest, AllowTransparencyAndAllowScalingPropagate) {
2796 LoadAppWithGuest("web_view/simple"); 2810 LoadAppWithGuest("web_view/simple");
2797 2811
2798 ASSERT_TRUE(!!GetGuestWebContents()); 2812 ASSERT_TRUE(!!GetGuestWebContents());
2799 extensions::WebViewGuest* guest = 2813 extensions::WebViewGuest* guest =
2800 extensions::WebViewGuest::FromWebContents(GetGuestWebContents()); 2814 extensions::WebViewGuest::FromWebContents(GetGuestWebContents());
2801 ASSERT_TRUE(guest->allow_transparency()); 2815 ASSERT_TRUE(guest->allow_transparency());
2802 ASSERT_TRUE(guest->allow_scaling()); 2816 ASSERT_TRUE(guest->allow_scaling());
2803 } 2817 }
2804 2818
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698