diff options
author | jam <jam@chromium.org> | 2016-03-21 08:51:34 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-21 15:52:47 +0000 |
commit | e1f453c8038dd60f17847dc37603c64835f9d865 (patch) | |
tree | 7f5821519246c87f5b06c0347bdb1c7bf5fdf617 /content | |
parent | 1936b3f6dcdd44c9a7db4e1686437568a7dd1aad (diff) | |
download | chromium_src-e1f453c8038dd60f17847dc37603c64835f9d865.zip chromium_src-e1f453c8038dd60f17847dc37603c64835f9d865.tar.gz chromium_src-e1f453c8038dd60f17847dc37603c64835f9d865.tar.bz2 |
Remove iOS ifdefs in src/content.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Review URL: https://codereview.chromium.org/1815563004
Cr-Commit-Position: refs/heads/master@{#382296}
Diffstat (limited to 'content')
45 files changed, 95 insertions, 219 deletions
diff --git a/content/app/content_main_runner.cc b/content/app/content_main_runner.cc index 1bf0c3e..f35f09b 100644 --- a/content/app/content_main_runner.cc +++ b/content/app/content_main_runner.cc @@ -38,7 +38,11 @@ #include "components/tracing/trace_config_file.h" #include "components/tracing/trace_to_console.h" #include "components/tracing/tracing_switches.h" +#include "content/app/mojo/mojo_init.h" #include "content/browser/browser_main.h" +#include "content/browser/gpu/gpu_process_host.h" +#include "content/browser/renderer_host/render_process_host_impl.h" +#include "content/browser/utility_process_host_impl.h" #include "content/common/set_process_title.h" #include "content/common/url_schemes.h" #include "content/gpu/in_process_gpu_thread.h" @@ -51,6 +55,10 @@ #include "content/public/common/content_switches.h" #include "content/public/common/main_function_params.h" #include "content/public/common/sandbox_init.h" +#include "content/public/gpu/content_gpu_client.h" +#include "content/public/plugin/content_plugin_client.h" +#include "content/public/renderer/content_renderer_client.h" +#include "content/public/utility/content_utility_client.h" #include "content/renderer/in_process_renderer_thread.h" #include "content/utility/in_process_utility_thread.h" #include "ipc/ipc_descriptors.h" @@ -64,17 +72,6 @@ #include "gin/v8_initializer.h" #endif -#if !defined(OS_IOS) -#include "content/app/mojo/mojo_init.h" -#include "content/browser/gpu/gpu_process_host.h" -#include "content/browser/renderer_host/render_process_host_impl.h" -#include "content/browser/utility_process_host_impl.h" -#include "content/public/gpu/content_gpu_client.h" -#include "content/public/plugin/content_plugin_client.h" -#include "content/public/renderer/content_renderer_client.h" -#include "content/public/utility/content_utility_client.h" -#endif - #if defined(OS_WIN) #include <malloc.h> #include <cstring> @@ -85,12 +82,10 @@ #include "ui/gfx/win/dpi.h" #elif defined(OS_MACOSX) #include "base/mac/scoped_nsautorelease_pool.h" -#if !defined(OS_IOS) #include "base/power_monitor/power_monitor_device_source.h" #include "content/app/mac/mac_init.h" #include "content/browser/mach_broker_mac.h" #include "content/common/sandbox_init_mac.h" -#endif // !OS_IOS #endif // OS_WIN #if defined(OS_POSIX) @@ -135,7 +130,7 @@ base::LazyInstance<ContentBrowserClient> g_empty_content_browser_client = LAZY_INSTANCE_INITIALIZER; #endif // !CHROME_MULTIPLE_DLL_CHILD -#if !defined(OS_IOS) && !defined(CHROME_MULTIPLE_DLL_BROWSER) +#if !defined(CHROME_MULTIPLE_DLL_BROWSER) base::LazyInstance<ContentGpuClient> g_empty_content_gpu_client = LAZY_INSTANCE_INITIALIZER; base::LazyInstance<ContentPluginClient> @@ -144,7 +139,7 @@ base::LazyInstance<ContentRendererClient> g_empty_content_renderer_client = LAZY_INSTANCE_INITIALIZER; base::LazyInstance<ContentUtilityClient> g_empty_content_utility_client = LAZY_INSTANCE_INITIALIZER; -#endif // !OS_IOS && !CHROME_MULTIPLE_DLL_BROWSER +#endif // !CHROME_MULTIPLE_DLL_BROWSER #if defined(V8_USE_EXTERNAL_STARTUP_DATA) && defined(OS_ANDROID) #if defined __LP64__ @@ -156,7 +151,7 @@ base::LazyInstance<ContentUtilityClient> #endif #endif -#if defined(OS_POSIX) && !defined(OS_IOS) +#if defined(OS_POSIX) // Setup signal-handling state: resanitize most signals, ignore SIGPIPE. void SetupSignalHandlers() { @@ -180,7 +175,7 @@ void SetupSignalHandlers() { CHECK_NE(SIG_ERR, signal(SIGPIPE, SIG_IGN)); } -#endif // OS_POSIX && !OS_IOS +#endif // OS_POSIX void CommonSubprocessInit(const std::string& process_type) { #if defined(OS_WIN) @@ -231,7 +226,7 @@ class ContentClientInitializer { } #endif // !CHROME_MULTIPLE_DLL_CHILD -#if !defined(OS_IOS) && !defined(CHROME_MULTIPLE_DLL_BROWSER) +#if !defined(CHROME_MULTIPLE_DLL_BROWSER) if (process_type == switches::kGpuProcess || base::CommandLine::ForCurrentProcess()->HasSwitch( switches::kSingleProcess)) { @@ -266,7 +261,7 @@ class ContentClientInitializer { if (!content_client->utility_) content_client->utility_ = &g_empty_content_utility_client.Get(); } -#endif // !OS_IOS && !CHROME_MULTIPLE_DLL_BROWSER +#endif // !CHROME_MULTIPLE_DLL_BROWSER } }; @@ -328,7 +323,6 @@ int RunZygote(const MainFunctionParams& main_function_params, } #endif // defined(OS_POSIX) && !defined(OS_MACOSX) && !defined(OS_ANDROID) -#if !defined(OS_IOS) static void RegisterMainThreadFactories() { #if !defined(CHROME_MULTIPLE_DLL_BROWSER) && !defined(CHROME_MULTIPLE_DLL_CHILD) UtilityProcessHostImpl::RegisterUtilityMainThreadFactory( @@ -414,7 +408,6 @@ int RunNamedProcessTypeMain( NOTREACHED() << "Unknown process type: " << process_type; return 1; } -#endif // !OS_IOS class ContentMainRunnerImpl : public ContentMainRunner { public: @@ -452,9 +445,7 @@ class ContentMainRunnerImpl : public ContentMainRunner { TRACE_EVENT0("startup,benchmark", "ContentMainRunnerImpl::Initialize"); #endif // OS_ANDROID -#if !defined(OS_IOS) base::GlobalDescriptors* g_fds = base::GlobalDescriptors::GetInstance(); -#endif // On Android, // - setlocale() is not supported. @@ -462,7 +453,7 @@ class ContentMainRunnerImpl : public ContentMainRunner { // stack trace when crashing. // - The ipc_fd is passed through the Java service. // Thus, these are all disabled. -#if !defined(OS_ANDROID) && !defined(OS_IOS) +#if !defined(OS_ANDROID) // Set C library locale to make sure CommandLine can parse argument values // in correct encoding. setlocale(LC_ALL, ""); @@ -472,7 +463,7 @@ class ContentMainRunnerImpl : public ContentMainRunner { kPrimaryIPCChannel + base::GlobalDescriptors::kBaseDescriptor); g_fds->Set(kMojoIPCChannel, kMojoIPCChannel + base::GlobalDescriptors::kBaseDescriptor); -#endif // !OS_ANDROID && !OS_IOS +#endif // !OS_ANDROID #if defined(OS_LINUX) || defined(OS_OPENBSD) g_fds->Set(kCrashDumpSignal, @@ -487,22 +478,18 @@ class ContentMainRunnerImpl : public ContentMainRunner { // The exit manager is in charge of calling the dtors of singleton objects. // On Android, AtExitManager is set up when library is loaded. - // On iOS, it's set up in main(), which can't call directly through to here. // A consequence of this is that you can't use the ctor/dtor-based // TRACE_EVENT methods on Linux or iOS builds till after we set this up. -#if !defined(OS_ANDROID) && !defined(OS_IOS) +#if !defined(OS_ANDROID) if (!ui_task_) { // When running browser tests, don't create a second AtExitManager as that // interfers with shutdown when objects created before ContentMain is // called are destructed when it returns. exit_manager_.reset(new base::AtExitManager); } -#endif // !OS_ANDROID && !OS_IOS +#endif // !OS_ANDROID - // Don't create this loop on iOS, since the outer loop is already handled - // and a loop that's destroyed in shutdown interleaves badly with the event - // loop pool on iOS. -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // We need this pool for all the objects created before we get to the // event loop, but we don't want to leave them hanging around until the // app quits. Each "main" needs to flush this pool right before it goes into @@ -534,9 +521,7 @@ class ContentMainRunnerImpl : public ContentMainRunner { if (delegate_ && delegate_->ShouldEnableProfilerRecording()) tracked_objects::ScopedTracker::Enable(); -#if !defined(OS_IOS) SetProcessTitleFromCommandLine(argv); -#endif #endif // !OS_ANDROID int exit_code = 0; @@ -550,10 +535,8 @@ class ContentMainRunnerImpl : public ContentMainRunner { std::string process_type = command_line.GetSwitchValueASCII(switches::kProcessType); -#if !defined(OS_IOS) // Initialize mojo here so that services can be registered. InitializeMojo(); -#endif #if defined(OS_WIN) if (command_line.HasSwitch(switches::kDeviceScaleFactor)) { @@ -619,7 +602,7 @@ class ContentMainRunnerImpl : public ContentMainRunner { TRACE_EVENT0("startup,benchmark", "ContentMainRunnerImpl::Initialize"); #endif // !OS_ANDROID -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // We need to allocate the IO Ports before the Sandbox is initialized or // the first instance of PowerMonitor is created. // It's important not to allocate the ports for processes which don't @@ -732,7 +715,7 @@ class ContentMainRunnerImpl : public ContentMainRunner { #if defined(OS_WIN) CHECK(InitializeSandbox(params.sandbox_info)); -#elif defined(OS_MACOSX) && !defined(OS_IOS) +#elif defined(OS_MACOSX) if (process_type == switches::kRendererProcess || process_type == switches::kPpapiPluginProcess || (delegate_ && delegate_->DelaySandboxInitialization(process_type))) { @@ -768,11 +751,7 @@ class ContentMainRunnerImpl : public ContentMainRunner { main_params.autorelease_pool = autorelease_pool_.get(); #endif -#if !defined(OS_IOS) return RunNamedProcessTypeMain(process_type, main_params, delegate_); -#else - return 1; -#endif } void Shutdown() override { @@ -794,7 +773,7 @@ class ContentMainRunnerImpl : public ContentMainRunner { #endif // _CRTDBG_MAP_ALLOC #endif // OS_WIN -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) autorelease_pool_.reset(NULL); #endif diff --git a/content/browser/browser_child_process_host_impl.cc b/content/browser/browser_child_process_host_impl.cc index 666fdf3..cde59de 100644 --- a/content/browser/browser_child_process_host_impl.cc +++ b/content/browser/browser_child_process_host_impl.cc @@ -134,12 +134,12 @@ BrowserChildProcessHostImpl::BrowserChildProcessHostImpl( // child process. This ensures that when a test is being run in one of the // single process modes, the global attachment broker is the privileged // attachment broker, rather than an unprivileged attachment broker. -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) IPC::AttachmentBrokerPrivileged::CreateBrokerIfNeeded( MachBroker::GetInstance()); #else IPC::AttachmentBrokerPrivileged::CreateBrokerIfNeeded(); -#endif // defined(OS_MACOSX) && !defined(OS_IOS) +#endif // defined(OS_MACOSX) #endif // USE_ATTACHMENT_BROKER child_process_host_.reset(ChildProcessHost::Create(this)); diff --git a/content/browser/browser_context.cc b/content/browser/browser_context.cc index 10db382..56382ca 100644 --- a/content/browser/browser_context.cc +++ b/content/browser/browser_context.cc @@ -14,8 +14,6 @@ #include "base/lazy_instance.h" #include "base/rand_util.h" #include "build/build_config.h" - -#if !defined(OS_IOS) #include "components/profile_service/profile_app.h" #include "content/browser/download/download_manager_impl.h" #include "content/browser/fileapi/chrome_blob_storage_context.h" @@ -35,14 +33,11 @@ #include "net/url_request/url_request_context_getter.h" #include "storage/browser/database/database_tracker.h" #include "storage/browser/fileapi/external_mount_points.h" -#endif // !OS_IOS using base::UserDataAdapter; namespace content { -// Only ~BrowserContext() is needed on iOS. -#if !defined(OS_IOS) namespace { base::LazyInstance<std::set<std::string>> g_used_user_ids = @@ -365,17 +360,13 @@ const std::string& BrowserContext::GetMojoUserIdFor( return it->second; } -#endif // !OS_IOS - BrowserContext::~BrowserContext() { CHECK(GetUserData(kMojoWasInitialized)) << "Attempting to destroy a BrowserContext that never called " << "Initialize()"; -#if !defined(OS_IOS) if (GetUserData(kDownloadManagerKeyName)) GetDownloadManager(this)->Shutdown(); -#endif } } // namespace content diff --git a/content/browser/browser_main_loop.cc b/content/browser/browser_main_loop.cc index eab70cef..ba04371 100644 --- a/content/browser/browser_main_loop.cc +++ b/content/browser/browser_main_loop.cc @@ -87,7 +87,7 @@ #include "sql/sql_memory_dump_provider.h" #include "ui/base/clipboard/clipboard.h" -#if defined(USE_AURA) || (defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(USE_AURA) || defined(OS_MACOSX) #include "content/browser/compositor/image_transport_factory.h" #endif @@ -96,10 +96,6 @@ #include "ui/aura/env.h" #endif -#if !defined(OS_IOS) -#include "content/browser/renderer_host/render_process_host_impl.h" -#endif - #if defined(OS_ANDROID) #include "base/android/jni_android.h" #include "components/tracing/graphics_memory_dump_provider_android.h" @@ -112,7 +108,7 @@ #include "ui/gl/gl_surface.h" #endif -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) #include "base/memory/memory_pressure_monitor_mac.h" #include "content/browser/bootstrap_sandbox_manager_mac.h" #include "content/browser/cocoa/system_hotkey_helper_mac.h" @@ -153,7 +149,7 @@ #include "media/capture/system_message_window_win.h" #elif defined(OS_LINUX) && defined(USE_UDEV) #include "media/capture/device_monitor_udev.h" -#elif defined(OS_MACOSX) && !defined(OS_IOS) +#elif defined(OS_MACOSX) #include "media/capture/device_monitor_mac.h" #endif @@ -341,13 +337,11 @@ NOINLINE void ResetThread_IO(scoped_ptr<BrowserProcessSubThread> thread) { thread.reset(); } -#if !defined(OS_IOS) NOINLINE void ResetThread_IndexedDb(scoped_ptr<base::Thread> thread) { volatile int inhibit_comdat = __LINE__; ALLOW_UNUSED_LOCAL(inhibit_comdat); thread.reset(); } -#endif MSVC_POP_WARNING() MSVC_ENABLE_OPTIMIZE(); @@ -392,13 +386,11 @@ class BrowserMainLoop::MemoryObserver : public base::MessageLoop::TaskObserver { void WillProcessTask(const base::PendingTask& pending_task) override {} void DidProcessTask(const base::PendingTask& pending_task) override { -#if !defined(OS_IOS) // No ProcessMetrics on IOS. scoped_ptr<base::ProcessMetrics> process_metrics( base::ProcessMetrics::CreateCurrentProcessMetrics()); size_t private_bytes; process_metrics->GetMemoryBytes(&private_bytes, NULL); LOCAL_HISTOGRAM_MEMORY_KB("Memory.BrowserUsed", private_bytes >> 10); -#endif } private: DISALLOW_COPY_AND_ASSIGN(MemoryObserver); @@ -429,9 +421,7 @@ BrowserMainLoop::BrowserMainLoop(const MainFunctionParams& parameters) BrowserMainLoop::~BrowserMainLoop() { DCHECK_EQ(this, g_current_browser_main_loop); -#if !defined(OS_IOS) ui::Clipboard::DestroyClipboardForCurrentThread(); -#endif // !defined(OS_IOS) g_current_browser_main_loop = NULL; } @@ -506,7 +496,6 @@ void BrowserMainLoop::EarlyInitialization() { crypto::EnsureNSPRInit(); #endif -#if !defined(OS_IOS) if (parsed_command_line_.HasSwitch(switches::kRendererProcessLimit)) { std::string limit_string = parsed_command_line_.GetSwitchValueASCII( switches::kRendererProcessLimit); @@ -515,7 +504,6 @@ void BrowserMainLoop::EarlyInitialization() { RenderProcessHost::SetMaxRendererProcessCount(process_limit); } } -#endif // !defined(OS_IOS) if (parts_) parts_->PostEarlyInitialization(); @@ -572,8 +560,6 @@ void BrowserMainLoop::PostMainMessageLoopStart() { TRACE_EVENT0("startup", "BrowserMainLoop::Subsystem:NetworkChangeNotifier"); network_change_notifier_.reset(net::NetworkChangeNotifier::Create()); } - -#if !defined(OS_IOS) { TRACE_EVENT0("startup", "BrowserMainLoop::Subsystem:MediaFeatures"); media::InitializeMediaLibrary(); @@ -595,7 +581,6 @@ void BrowserMainLoop::PostMainMessageLoopStart() { new base::trace_event::TraceEventSystemStatsMonitor( base::ThreadTaskRunnerHandle::Get())); } -#endif // !defined(OS_IOS) #if defined(OS_WIN) if (base::win::GetVersion() >= base::win::VERSION_WIN8) @@ -632,7 +617,6 @@ void BrowserMainLoop::PostMainMessageLoopStart() { tracing::TraceConfigFile::GetInstance()->GetTraceConfig(), TracingController::StartTracingDoneCallback()); } -#if !defined(OS_IOS) // Start tracing to a file for certain duration if needed. Only do this after // starting the main message loop to avoid calling // MessagePumpForUI::ScheduleWork() before MessagePumpForUI::Start() as it @@ -641,7 +625,6 @@ void BrowserMainLoop::PostMainMessageLoopStart() { TRACE_EVENT0("startup", "BrowserMainLoop::InitStartupTracingForDuration"); InitStartupTracingForDuration(parsed_command_line_); } -#endif // !defined(OS_IOS) #if defined(OS_ANDROID) { @@ -665,7 +648,7 @@ void BrowserMainLoop::PostMainMessageLoopStart() { } #endif -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) if (BootstrapSandboxManager::ShouldEnable()) { TRACE_EVENT0("startup", "BrowserMainLoop::Subsystem:BootstrapSandbox"); @@ -726,7 +709,7 @@ int BrowserMainLoop::PreCreateThreads() { memory_pressure_monitor_.reset(new base::chromeos::MemoryPressureMonitor( chromeos::switches::GetMemoryPressureThresholds())); } -#elif defined(OS_MACOSX) && !defined(OS_IOS) +#elif defined(OS_MACOSX) memory_pressure_monitor_.reset(new base::mac::MemoryPressureMonitor()); #elif defined(OS_WIN) memory_pressure_monitor_.reset(CreateWinMemoryPressureMonitor( @@ -744,7 +727,7 @@ int BrowserMainLoop::PreCreateThreads() { } #endif -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // The WindowResizeHelper allows the UI thread to wait on specific renderer // and GPU messages from the IO thread. Initializing it before the IO thread // starts ensures the affected IO thread messages always have somewhere to go. @@ -757,7 +740,7 @@ int BrowserMainLoop::PreCreateThreads() { // 2) Must be after parts_->PreCreateThreads to pick up chrome://flags. GpuDataManagerImpl::GetInstance()->Initialize(); -#if !defined(OS_IOS) && (!defined(GOOGLE_CHROME_BUILD) || defined(OS_ANDROID)) +#if !defined(GOOGLE_CHROME_BUILD) || defined(OS_ANDROID) // Single-process is an unsupported and not fully tested mode, so // don't enable it for official Chrome builds (except on Android). if (parsed_command_line_.HasSwitch(switches::kSingleProcess)) @@ -984,10 +967,8 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() { if (IsRunningInMojoShell()) MojoShellConnection::Destroy(); -#if !defined(OS_IOS) if (RenderProcessHost::run_renderer_in_process()) RenderProcessHostImpl::ShutDownInProcessRenderer(); -#endif if (parts_) { TRACE_EVENT0("shutdown", @@ -1001,7 +982,6 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() { system_stats_monitor_.reset(); -#if !defined(OS_IOS) // Destroying the GpuProcessHostUIShims on the UI thread posts a task to // delete related objects on the GPU thread. This must be done before // stopping the GPU thread. The GPU thread will close IPC channels to renderer @@ -1046,13 +1026,10 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() { #elif defined(OS_MACOSX) device_monitor_mac_.reset(); #endif -#endif // !defined(OS_IOS) // Shutdown Mojo shell and IPC. -#if !defined(OS_IOS) mojo_shell_context_.reset(); mojo_ipc_support_.reset(); -#endif // Must be size_t so we can subtract from it. for (size_t thread_id = BrowserThread::ID_COUNT - 1; @@ -1083,12 +1060,10 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() { } case BrowserThread::FILE: { TRACE_EVENT0("shutdown", "BrowserMainLoop::Subsystem:FileThread"); -#if !defined(OS_IOS) // Clean up state that lives on or uses the file_thread_ before // it goes away. if (resource_dispatcher_host_) resource_dispatcher_host_.get()->save_file_manager()->Shutdown(); -#endif // !defined(OS_IOS) ResetThread_FILE(std::move(file_thread_)); break; } @@ -1120,13 +1095,10 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() { break; } } - -#if !defined(OS_IOS) { TRACE_EVENT0("shutdown", "BrowserMainLoop::Subsystem:IndexedDBThread"); ResetThread_IndexedDb(std::move(indexed_db_thread_)); } -#endif // Close the blocking I/O pool after the other threads. Other threads such // as the I/O thread may need to schedule work like closing files or flushing @@ -1138,8 +1110,6 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() { TRACE_EVENT0("shutdown", "BrowserMainLoop::Subsystem:ThreadPool"); BrowserThreadImpl::ShutdownThreadPool(); } - -#if !defined(OS_IOS) // Must happen after the IO thread is shutdown since this may be accessed from // it. { @@ -1168,7 +1138,6 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() { TRACE_EVENT0("shutdown", "BrowserMainLoop::Subsystem:DeleteDataSources"); URLDataManager::DeleteDataSources(); } -#endif // !defined(OS_IOS) if (parts_) { TRACE_EVENT0("shutdown", "BrowserMainLoop::Subsystem:PostDestroyThreads"); @@ -1195,7 +1164,6 @@ void BrowserMainLoop::InitializeMainThread() { int BrowserMainLoop::BrowserThreadsStarted() { TRACE_EVENT0("startup", "BrowserMainLoop::BrowserThreadsStarted"); -#if !defined(OS_IOS) // Bring up Mojo IPC and shell as early as possible. mojo_ipc_support_.reset(new IPC::ScopedIPCSupport( BrowserThread::UnsafeGetMessageLoopForThread(BrowserThread::IO) @@ -1204,14 +1172,10 @@ int BrowserMainLoop::BrowserThreadsStarted() { #if defined(OS_MACOSX) mojo::edk::SetMachPortProvider(MachBroker::GetInstance()); #endif // defined(OS_MACOSX) -#endif // !defined(OS_IOS) -#if !defined(OS_IOS) indexed_db_thread_.reset(new base::Thread("IndexedDB")); indexed_db_thread_->Start(); -#endif -#if !defined(OS_IOS) HistogramSynchronizer::GetInstance(); #if defined(OS_ANDROID) || defined(OS_CHROMEOS) // Up the priority of the UI thread. @@ -1348,8 +1312,6 @@ int BrowserMainLoop::BrowserThreadsStarted() { SystemHotkeyHelperMac::GetInstance()->DeferredLoadSystemHotkeys(); #endif // defined(OS_MACOSX) -#endif // !defined(OS_IOS) - return result_code_; } diff --git a/content/browser/browser_main_runner.cc b/content/browser/browser_main_runner.cc index bbf47b0..ef2fcb0 100644 --- a/content/browser/browser_main_runner.cc +++ b/content/browser/browser_main_runner.cc @@ -79,10 +79,8 @@ class BrowserMainRunnerImpl : public BrowserMainRunner { SkGraphics::Init(); -#if !defined(OS_IOS) if (parameters.command_line.HasSwitch(switches::kWaitForDebugger)) base::debug::WaitForDebugger(60, true); -#endif #if defined(OS_WIN) if (base::win::GetVersion() < base::win::VERSION_VISTA) { diff --git a/content/browser/browser_process_sub_thread.cc b/content/browser/browser_process_sub_thread.cc index b1904e1..5aadda5 100644 --- a/content/browser/browser_process_sub_thread.cc +++ b/content/browser/browser_process_sub_thread.cc @@ -66,12 +66,10 @@ void BrowserProcessSubThread::IOThreadPreCleanUp() { // Destroy all URLRequests started by URLFetchers. net::URLFetcher::CancelAll(); -#if !defined(OS_IOS) // If any child processes are still running, terminate them and // and delete the BrowserChildProcessHost instances to release whatever // IO thread only resources they are referencing. BrowserChildProcessHostImpl::TerminateAll(); -#endif // !defined(OS_IOS) // Unregister GpuMemoryBuffer dump provider before IO thread is shut down. base::trace_event::MemoryDumpManager::GetInstance()->UnregisterDumpProvider( diff --git a/content/browser/frame_host/navigation_entry_impl_unittest.cc b/content/browser/frame_host/navigation_entry_impl_unittest.cc index 7319b53..a7bf9e3 100644 --- a/content/browser/frame_host/navigation_entry_impl_unittest.cc +++ b/content/browser/frame_host/navigation_entry_impl_unittest.cc @@ -24,9 +24,7 @@ class NavigationEntryTest : public testing::Test { void SetUp() override { entry1_.reset(new NavigationEntryImpl); -#if !defined(OS_IOS) instance_ = static_cast<SiteInstanceImpl*>(SiteInstance::Create(NULL)); -#endif entry2_.reset(new NavigationEntryImpl( instance_, 3, GURL("test:url"), diff --git a/content/browser/media/capture/web_contents_video_capture_device_unittest.cc b/content/browser/media/capture/web_contents_video_capture_device_unittest.cc index 0d50e4d..1fc6a8e 100644 --- a/content/browser/media/capture/web_contents_video_capture_device_unittest.cc +++ b/content/browser/media/capture/web_contents_video_capture_device_unittest.cc @@ -445,7 +445,7 @@ class StubClient : public media::VideoCaptureDevice::Client { } void* data(int plane) override { return buffer_handle_->data(plane); } ClientBuffer AsClientBuffer(int plane) override { return nullptr; } -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) base::FileDescriptor AsPlatformFile() override { return base::FileDescriptor(); } diff --git a/content/browser/renderer_host/media/video_capture_buffer_pool.cc b/content/browser/renderer_host/media/video_capture_buffer_pool.cc index a934a40c..cd3e0be 100644 --- a/content/browser/renderer_host/media/video_capture_buffer_pool.cc +++ b/content/browser/renderer_host/media/video_capture_buffer_pool.cc @@ -24,7 +24,7 @@ class SimpleBufferHandle final : public VideoCaptureBufferPool::BufferHandle { base::SharedMemoryHandle handle) : data_(data), mapped_size_(mapped_size) -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) , handle_(handle) #endif @@ -45,7 +45,7 @@ class SimpleBufferHandle final : public VideoCaptureBufferPool::BufferHandle { NOTREACHED(); return nullptr; } -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) base::FileDescriptor AsPlatformFile() override { return handle_; } @@ -54,7 +54,7 @@ class SimpleBufferHandle final : public VideoCaptureBufferPool::BufferHandle { private: void* const data_; const size_t mapped_size_; -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) const base::SharedMemoryHandle handle_; #endif }; @@ -85,7 +85,7 @@ class GpuMemoryBufferBufferHandle final DCHECK_LT(plane, static_cast<int>(gmbs_->size())); return (*gmbs_)[plane]->AsClientBuffer(); } -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) base::FileDescriptor AsPlatformFile() override { NOTREACHED(); return base::FileDescriptor(); diff --git a/content/browser/renderer_host/media/video_capture_buffer_pool.h b/content/browser/renderer_host/media/video_capture_buffer_pool.h index c69a842..e5e4e2f 100644 --- a/content/browser/renderer_host/media/video_capture_buffer_pool.h +++ b/content/browser/renderer_host/media/video_capture_buffer_pool.h @@ -58,7 +58,7 @@ class CONTENT_EXPORT VideoCaptureBufferPool virtual size_t mapped_size() const = 0; virtual void* data(int plane) = 0; virtual ClientBuffer AsClientBuffer(int plane) = 0; -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) virtual base::FileDescriptor AsPlatformFile() = 0; #endif }; diff --git a/content/browser/renderer_host/media/video_capture_device_client.cc b/content/browser/renderer_host/media/video_capture_device_client.cc index 03d64d2..d5cb149 100644 --- a/content/browser/renderer_host/media/video_capture_device_client.cc +++ b/content/browser/renderer_host/media/video_capture_device_client.cc @@ -47,7 +47,7 @@ class AutoReleaseBuffer : public media::VideoCaptureDevice::Client::Buffer { ClientBuffer AsClientBuffer(int plane) override { return buffer_handle_->AsClientBuffer(plane); } -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) base::FileDescriptor AsPlatformFile() override { return buffer_handle_->AsPlatformFile(); } diff --git a/content/browser/renderer_host/media/video_capture_gpu_jpeg_decoder.cc b/content/browser/renderer_host/media/video_capture_gpu_jpeg_decoder.cc index 8e61029..da78f9f 100644 --- a/content/browser/renderer_host/media/video_capture_gpu_jpeg_decoder.cc +++ b/content/browser/renderer_host/media/video_capture_gpu_jpeg_decoder.cc @@ -120,7 +120,7 @@ void VideoCaptureGpuJpegDecoder::DecodeCapturedData( // Mask against 30 bits, to avoid (undefined) wraparound on signed integer. next_bitstream_buffer_id_ = (next_bitstream_buffer_id_ + 1) & 0x3FFFFFFF; -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) const gfx::Size dimensions = frame_format.frame_size; base::SharedMemoryHandle out_handle = out_buffer->AsPlatformFile(); scoped_refptr<media::VideoFrame> out_frame = diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc index 4d2111f..620489e 100644 --- a/content/browser/renderer_host/render_process_host_impl.cc +++ b/content/browser/renderer_host/render_process_host_impl.cc @@ -196,7 +196,7 @@ #include "ui/gfx/win/dpi.h" #endif -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) #include "content/browser/bootstrap_sandbox_manager_mac.h" #include "content/browser/mach_broker_mac.h" #endif @@ -614,12 +614,12 @@ RenderProcessHostImpl::RenderProcessHostImpl( // render process. This ensures that when a test is being run in one of the // single process modes, the global attachment broker is the privileged // attachment broker, rather than an unprivileged attachment broker. -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) IPC::AttachmentBrokerPrivileged::CreateBrokerIfNeeded( MachBroker::GetInstance()); #else IPC::AttachmentBrokerPrivileged::CreateBrokerIfNeeded(); -#endif // defined(OS_MACOSX) && !defined(OS_IOS) +#endif // defined(OS_MACOSX) #endif // USE_ATTACHMENT_BROKER } diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc index ea33039..3aae765 100644 --- a/content/browser/renderer_host/render_widget_host_impl.cc +++ b/content/browser/renderer_host/render_widget_host_impl.cc @@ -677,7 +677,7 @@ void RenderWidgetHostImpl::DispatchColorProfile() { FROM_HERE, base::Bind(&RenderWidgetHostImpl::SendColorProfile, weak_factory_.GetWeakPtr())); -#elif !defined(OS_CHROMEOS) && !defined(OS_IOS) && !defined(OS_ANDROID) +#elif !defined(OS_CHROMEOS) && !defined(OS_ANDROID) // Only support desktop Mac and Linux at this time. SendColorProfile(); #endif diff --git a/content/browser/renderer_host/render_widget_host_unittest.cc b/content/browser/renderer_host/render_widget_host_unittest.cc index ad5ccee..c56ec9a 100644 --- a/content/browser/renderer_host/render_widget_host_unittest.cc +++ b/content/browser/renderer_host/render_widget_host_unittest.cc @@ -37,7 +37,7 @@ #include "content/browser/renderer_host/render_widget_host_view_android.h" #endif -#if defined(USE_AURA) || (defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(USE_AURA) || defined(OS_MACOSX) #include "content/browser/compositor/test/no_transport_image_transport_factory.h" #endif @@ -457,7 +457,7 @@ class RenderWidgetHostTest : public testing::Test { browser_context_.reset(new TestBrowserContext()); delegate_.reset(new MockRenderWidgetHostDelegate()); process_ = new RenderWidgetHostProcess(browser_context_.get()); -#if defined(USE_AURA) || (defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(USE_AURA) || defined(OS_MACOSX) ImageTransportFactory::InitializeForUnitTests( scoped_ptr<ImageTransportFactory>( new NoTransportImageTransportFactory)); @@ -487,7 +487,7 @@ class RenderWidgetHostTest : public testing::Test { gfx::Screen::SetScreenInstance(nullptr); screen_.reset(); #endif -#if defined(USE_AURA) || (defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(USE_AURA) || defined(OS_MACOSX) ImageTransportFactory::Terminate(); #endif diff --git a/content/browser/renderer_host/render_widget_host_view_browsertest.cc b/content/browser/renderer_host/render_widget_host_view_browsertest.cc index d7f79a0..d1f1fba 100644 --- a/content/browser/renderer_host/render_widget_host_view_browsertest.cc +++ b/content/browser/renderer_host/render_widget_host_view_browsertest.cc @@ -276,9 +276,8 @@ class FakeFrameSubscriber : public RenderWidgetHostViewFrameSubscriber { DeliverFrameCallback callback_; }; -// Disable tests for Android and IOS as these platforms have incomplete -// implementation. -#if !defined(OS_ANDROID) && !defined(OS_IOS) +// Disable tests for Android as it has an incomplete implementation. +#if !defined(OS_ANDROID) // The CopyFromBackingStore() API should work on all platforms when compositing // is enabled. @@ -1005,7 +1004,7 @@ INSTANTIATE_TEST_CASE_P( CompositingRenderWidgetHostViewBrowserTestTabCaptureHighDPI, kTestCompositingModes); -#endif // !defined(OS_ANDROID) && !defined(OS_IOS) +#endif // !defined(OS_ANDROID) } // namespace } // namespace content diff --git a/content/child/runtime_features.cc b/content/child/runtime_features.cc index 1196ffe..af698f9 100644 --- a/content/child/runtime_features.cc +++ b/content/child/runtime_features.cc @@ -48,8 +48,8 @@ static void SetRuntimeFeatureDefaultsForPlatform() { WebRuntimeFeatures::enableCompositedSelectionUpdate(true); #endif -#if !(defined OS_ANDROID || defined OS_CHROMEOS || defined OS_IOS) - // Only Android, ChromeOS, and IOS support NetInfo right now. +#if !(defined OS_ANDROID || defined OS_CHROMEOS) + // Only Android, ChromeOS support NetInfo right now. WebRuntimeFeatures::enableNetworkInformation(false); #endif } diff --git a/content/common/child_process_host_impl.cc b/content/common/child_process_host_impl.cc index d69d535..6951c6b 100644 --- a/content/common/child_process_host_impl.cc +++ b/content/common/child_process_host_impl.cc @@ -88,7 +88,7 @@ ChildProcessHostImpl::ChildProcessHostImpl(ChildProcessHostDelegate* delegate) #endif #if USE_ATTACHMENT_BROKER -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // On Mac, the privileged AttachmentBroker needs a reference to the Mach port // Provider, which is only available in the chrome/ module. The attachment // broker must already be created. @@ -97,7 +97,7 @@ ChildProcessHostImpl::ChildProcessHostImpl(ChildProcessHostDelegate* delegate) // Construct the privileged attachment broker early in the life cycle of a // child process. IPC::AttachmentBrokerPrivileged::CreateBrokerIfNeeded(); -#endif // defined(OS_MACOSX) && !defined(OS_IOS) +#endif // defined(OS_MACOSX) #endif // USE_ATTACHMENT_BROKER } diff --git a/content/common/gpu/client/gpu_jpeg_decode_accelerator_host.cc b/content/common/gpu/client/gpu_jpeg_decode_accelerator_host.cc index 05cdb8d..c9e1015 100644 --- a/content/common/gpu/client/gpu_jpeg_decode_accelerator_host.cc +++ b/content/common/gpu/client/gpu_jpeg_decode_accelerator_host.cc @@ -164,7 +164,7 @@ void GpuJpegDecodeAcceleratorHost::Decode( channel_->ShareToGpuProcess(video_frame->shared_memory_handle()); if (!base::SharedMemory::IsHandleValid(output_handle)) { DLOG(ERROR) << "Failed to duplicate handle of VideoFrame"; -#if defined(OS_POSIX) && !(defined(OS_MACOSX) && !defined(OS_IOS)) +#if defined(OS_POSIX) && !defined(OS_MACOSX) if (input_handle.auto_close) { // Defer closing task to the ScopedFD. base::ScopedFD(input_handle.fd); diff --git a/content/public/app/content_main_delegate.cc b/content/public/app/content_main_delegate.cc index 16fd804..57f04f7 100644 --- a/content/public/app/content_main_delegate.cc +++ b/content/public/app/content_main_delegate.cc @@ -6,15 +6,13 @@ #include "build/build_config.h" -#if !defined(CHROME_MULTIPLE_DLL_CHILD) -#include "content/public/browser/content_browser_client.h" -#endif - -#if !defined(OS_IOS) #include "content/public/gpu/content_gpu_client.h" #include "content/public/plugin/content_plugin_client.h" #include "content/public/renderer/content_renderer_client.h" #include "content/public/utility/content_utility_client.h" + +#if !defined(CHROME_MULTIPLE_DLL_CHILD) +#include "content/public/browser/content_browser_client.h" #endif namespace content { @@ -29,7 +27,7 @@ int ContentMainDelegate::RunProcess( return -1; } -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) bool ContentMainDelegate::ProcessRegistersWithSystemProcess( const std::string& process_type) { @@ -45,7 +43,7 @@ bool ContentMainDelegate::DelaySandboxInitialization( return false; } -#elif defined(OS_POSIX) && !defined(OS_ANDROID) && !defined(OS_IOS) +#elif defined(OS_POSIX) && !defined(OS_ANDROID) void ContentMainDelegate::ZygoteStarting( ScopedVector<ZygoteForkDelegate>* delegates) { @@ -66,7 +64,7 @@ ContentBrowserClient* ContentMainDelegate::CreateContentBrowserClient() { } ContentGpuClient* ContentMainDelegate::CreateContentGpuClient() { -#if defined(OS_IOS) || defined(CHROME_MULTIPLE_DLL_BROWSER) +#if defined(CHROME_MULTIPLE_DLL_BROWSER) return NULL; #else return new ContentGpuClient(); @@ -74,7 +72,7 @@ ContentGpuClient* ContentMainDelegate::CreateContentGpuClient() { } ContentPluginClient* ContentMainDelegate::CreateContentPluginClient() { -#if defined(OS_IOS) || defined(CHROME_MULTIPLE_DLL_BROWSER) +#if defined(CHROME_MULTIPLE_DLL_BROWSER) return NULL; #else return new ContentPluginClient(); @@ -82,7 +80,7 @@ ContentPluginClient* ContentMainDelegate::CreateContentPluginClient() { } ContentRendererClient* ContentMainDelegate::CreateContentRendererClient() { -#if defined(OS_IOS) || defined(CHROME_MULTIPLE_DLL_BROWSER) +#if defined(CHROME_MULTIPLE_DLL_BROWSER) return NULL; #else return new ContentRendererClient(); @@ -90,7 +88,7 @@ ContentRendererClient* ContentMainDelegate::CreateContentRendererClient() { } ContentUtilityClient* ContentMainDelegate::CreateContentUtilityClient() { -#if defined(OS_IOS) || defined(CHROME_MULTIPLE_DLL_BROWSER) +#if defined(CHROME_MULTIPLE_DLL_BROWSER) return NULL; #else return new ContentUtilityClient(); diff --git a/content/public/app/content_main_delegate.h b/content/public/app/content_main_delegate.h index 7fecaea..5155af88 100644 --- a/content/public/app/content_main_delegate.h +++ b/content/public/app/content_main_delegate.h @@ -50,7 +50,7 @@ class CONTENT_EXPORT ContentMainDelegate { // Called right before the process exits. virtual void ProcessExiting(const std::string& process_type) {} -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // Returns true if the process registers with the system monitor, so that we // can allocate an IO port for it before the sandbox is initialized. Embedders // are called only for process types that content doesn't know about. @@ -67,7 +67,7 @@ class CONTENT_EXPORT ContentMainDelegate { // want it at all. virtual bool DelaySandboxInitialization(const std::string& process_type); -#elif defined(OS_POSIX) && !defined(OS_ANDROID) && !defined(OS_IOS) +#elif defined(OS_POSIX) && !defined(OS_ANDROID) // Tells the embedder that the zygote process is starting, and allows it to // specify one or more zygote delegates if it wishes by storing them in // |*delegates|. diff --git a/content/public/browser/browser_child_process_host.h b/content/public/browser/browser_child_process_host.h index 1593546..07193af 100644 --- a/content/public/browser/browser_child_process_host.h +++ b/content/public/browser/browser_child_process_host.h @@ -14,7 +14,7 @@ #include "content/public/common/process_type.h" #include "ipc/ipc_sender.h" -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) #include "base/process/port_provider_mac.h" #endif @@ -85,7 +85,7 @@ class CONTENT_EXPORT BrowserChildProcessHost : public IPC::Sender { // nullptr if no service registry exists. virtual ServiceRegistry* GetServiceRegistry() = 0; -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // Returns a PortProvider used to get the task port for child processes. static base::PortProvider* GetPortProvider(); #endif diff --git a/content/public/browser/navigation_controller.h b/content/public/browser/navigation_controller.h index 76d6de2..a540ee0e 100644 --- a/content/public/browser/navigation_controller.h +++ b/content/public/browser/navigation_controller.h @@ -395,9 +395,7 @@ class NavigationController { // Random -------------------------------------------------------------------- - // Session storage depends on dom_storage that depends on blink::WebString, - // which cannot be used on iOS. -#if !defined(OS_IOS) + // Session storage depends on dom_storage that depends on blink::WebString. // Returns all the SessionStorageNamespace objects that this // NavigationController knows about, the map key is a StoragePartition id. virtual const SessionStorageNamespaceMap& @@ -406,7 +404,6 @@ class NavigationController { // TODO(ajwong): Remove this once prerendering, instant, and session restore // are migrated. virtual SessionStorageNamespace* GetDefaultSessionStorageNamespace() = 0; -#endif // Sets the max restored page ID this NavigationController has seen, if it // was restored from a previous session. diff --git a/content/public/browser/resource_context.h b/content/public/browser/resource_context.h index e96f7c2..5a9419b 100644 --- a/content/public/browser/resource_context.h +++ b/content/public/browser/resource_context.h @@ -33,9 +33,7 @@ class AppCacheService; // the UI thread. It must be destructed on the IO thread. class CONTENT_EXPORT ResourceContext : public base::SupportsUserData { public: -#if !defined(OS_IOS) ResourceContext(); -#endif ~ResourceContext() override; virtual net::HostResolver* GetHostResolver() = 0; diff --git a/content/public/common/content_client.cc b/content/public/common/content_client.cc index 34350d5..49682c6 100644 --- a/content/public/common/content_client.cc +++ b/content/public/common/content_client.cc @@ -108,7 +108,7 @@ std::string ContentClient::GetProcessTypeNameInEnglish(int type) { return std::string(); } -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) bool ContentClient::GetSandboxProfileForSandboxType( int sandbox_type, int* sandbox_profile_resource_id) const { diff --git a/content/public/common/content_client.h b/content/public/common/content_client.h index ba15b3e..3699415 100644 --- a/content/public/common/content_client.h +++ b/content/public/common/content_client.h @@ -125,7 +125,7 @@ class CONTENT_EXPORT ContentClient { // doesn't know about because they're from the embedder. virtual std::string GetProcessTypeNameInEnglish(int type); -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // Allows the embedder to define a new |sandbox_type| by mapping it to the // resource ID corresponding to the sandbox profile to use. The legal values // for |sandbox_type| are defined by the embedder and should start with diff --git a/content/public/common/content_switches.cc b/content/public/common/content_switches.cc index cc8d762..b7786ec 100644 --- a/content/public/common/content_switches.cc +++ b/content/public/common/content_switches.cc @@ -982,7 +982,7 @@ const char kDisableVaapiAcceleratedVideoEncode[] = const char kEnableSpeechDispatcher[] = "enable-speech-dispatcher"; #endif -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // Disables support for Core Animation plugins. This is triggered when // accelerated compositing is disabled. See http://crbug.com/122430. const char kDisableCoreAnimationPlugins[] = diff --git a/content/public/common/content_switches.h b/content/public/common/content_switches.h index 6e85102..8fd722e 100644 --- a/content/public/common/content_switches.h +++ b/content/public/common/content_switches.h @@ -286,7 +286,7 @@ CONTENT_EXPORT extern const char kDisableVaapiAcceleratedVideoEncode[]; CONTENT_EXPORT extern const char kEnableSpeechDispatcher[]; #endif -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) extern const char kDisableCoreAnimationPlugins[]; extern const char kDisableThreadedEventHandlingMac[]; #endif diff --git a/content/public/common/url_utils.cc b/content/public/common/url_utils.cc index 40cd67c..4133cb0 100644 --- a/content/public/common/url_utils.cc +++ b/content/public/common/url_utils.cc @@ -16,10 +16,7 @@ const char* const* GetSavableSchemes() { } bool HasWebUIScheme(const GURL& url) { - return -#if !defined(OS_IOS) - url.SchemeIs(kChromeDevToolsScheme) || -#endif + return url.SchemeIs(kChromeDevToolsScheme) || url.SchemeIs(kChromeUIScheme); } diff --git a/content/public/test/content_test_suite_base.cc b/content/public/test/content_test_suite_base.cc index 1b2914c..4c03f858 100644 --- a/content/public/test/content_test_suite_base.cc +++ b/content/public/test/content_test_suite_base.cc @@ -12,6 +12,9 @@ #include "base/threading/sequenced_worker_pool.h" #include "build/build_config.h" #include "content/browser/browser_thread_impl.h" +#include "content/browser/gpu/gpu_process_host.h" +#include "content/browser/renderer_host/render_process_host_impl.h" +#include "content/browser/utility_process_host_impl.h" #include "content/common/url_schemes.h" #include "content/gpu/in_process_gpu_thread.h" #include "content/public/common/content_client.h" @@ -20,14 +23,9 @@ #include "testing/gtest/include/gtest/gtest.h" #include "ui/base/ui_base_paths.h" -#if !defined(OS_IOS) -#include "content/browser/gpu/gpu_process_host.h" -#include "content/browser/renderer_host/render_process_host_impl.h" -#include "content/browser/utility_process_host_impl.h" #if defined(V8_USE_EXTERNAL_STARTUP_DATA) #include "gin/v8_initializer.h" #endif -#endif #if defined(OS_ANDROID) #include "base/android/jni_android.h" @@ -78,7 +76,7 @@ void ContentTestSuiteBase::Initialize() { // by tests. base::StatisticsRecorder::Initialize(); -#if !defined(OS_IOS) && defined(V8_USE_EXTERNAL_STARTUP_DATA) +#if defined(V8_USE_EXTERNAL_STARTUP_DATA) gin::V8Initializer::LoadV8Snapshot(); gin::V8Initializer::LoadV8Natives(); #endif @@ -118,13 +116,11 @@ void ContentTestSuiteBase::RegisterContentSchemes( } void ContentTestSuiteBase::RegisterInProcessThreads() { -#if !defined(OS_IOS) UtilityProcessHostImpl::RegisterUtilityMainThreadFactory( CreateInProcessUtilityThread); RenderProcessHostImpl::RegisterRendererMainThreadFactory( CreateInProcessRendererThread); GpuProcessHost::RegisterGpuMainThreadFactory(CreateInProcessGpuThread); -#endif } } // namespace content diff --git a/content/public/test/render_view_test.cc b/content/public/test/render_view_test.cc index ff945b6..572e008 100644 --- a/content/public/test/render_view_test.cc +++ b/content/public/test/render_view_test.cc @@ -367,12 +367,10 @@ void RenderViewTest::SetUp() { view_params.min_size = gfx::Size(); view_params.max_size = gfx::Size(); -#if !defined(OS_IOS) InitializeMojo(); test_io_thread_.reset(new base::TestIOThread(base::TestIOThread::kAutoStart)); ipc_support_.reset( new mojo::edk::test::ScopedIPCSupport(test_io_thread_->task_runner())); -#endif // This needs to pass the mock render thread to the view. RenderViewImpl* view = diff --git a/content/public/test/test_content_client_initializer.cc b/content/public/test/test_content_client_initializer.cc index e19f4d8..66bf033 100644 --- a/content/public/test/test_content_client_initializer.cc +++ b/content/public/test/test_content_client_initializer.cc @@ -36,11 +36,9 @@ TestContentClientInitializer::~TestContentClientInitializer() { } void TestContentClientInitializer::CreateTestRenderViewHosts() { -#if !defined(OS_IOS) rph_factory_.reset(new MockRenderProcessHostFactory()); test_render_view_host_factory_.reset( new TestRenderViewHostFactory(rph_factory_.get())); -#endif // OS_IOS } } // namespace content diff --git a/content/public/test/test_utils.cc b/content/public/test/test_utils.cc index 66a493a..2d12dd6 100644 --- a/content/public/test/test_utils.cc +++ b/content/public/test/test_utils.cc @@ -129,9 +129,7 @@ void RunThisRunLoop(base::RunLoop* run_loop) { // If we're running inside a browser test, we might need to allow the test // launcher to do extra work before/after running a nested message loop. TestLauncherDelegate* delegate = NULL; -#if !defined(OS_IOS) delegate = GetCurrentTestLauncherDelegate(); -#endif if (delegate) delegate->PreRunMessageLoop(run_loop); run_loop->Run(); diff --git a/content/public/test/unittest_test_suite.cc b/content/public/test/unittest_test_suite.cc index 10791ba..7c45d5c 100644 --- a/content/public/test/unittest_test_suite.cc +++ b/content/public/test/unittest_test_suite.cc @@ -8,6 +8,7 @@ #include "base/rand_util.h" #include "base/test/test_suite.h" #include "build/build_config.h" +#include "content/test/test_blink_web_unit_test_support.h" #include "third_party/WebKit/public/web/WebKit.h" #if defined(USE_AURA) @@ -18,10 +19,6 @@ #include <X11/Xlib.h> #endif -#if !defined(OS_IOS) -#include "content/test/test_blink_web_unit_test_support.h" -#endif - namespace content { UnitTestTestSuite::UnitTestTestSuite(base::TestSuite* test_suite) @@ -35,15 +32,11 @@ UnitTestTestSuite::UnitTestTestSuite(base::TestSuite* test_suite) aura::Env::CreateInstance(create_event_source); #endif DCHECK(test_suite); -#if !defined(OS_IOS) blink_test_support_.reset(new TestBlinkWebUnitTestSupport); -#endif } UnitTestTestSuite::~UnitTestTestSuite() { -#if !defined(OS_IOS) blink_test_support_.reset(); -#endif #if defined(USE_AURA) aura::Env::DeleteInstance(); #endif diff --git a/content/public/test/unittest_test_suite.h b/content/public/test/unittest_test_suite.h index 680ea3c..1a0f46b 100644 --- a/content/public/test/unittest_test_suite.h +++ b/content/public/test/unittest_test_suite.h @@ -32,9 +32,7 @@ class UnitTestTestSuite { private: scoped_ptr<base::TestSuite> test_suite_; -#if !defined(OS_IOS) scoped_ptr<TestBlinkWebUnitTestSupport> blink_test_support_; -#endif DISALLOW_COPY_AND_ASSIGN(UnitTestTestSuite); }; diff --git a/content/renderer/media/media_stream_audio_processor.cc b/content/renderer/media/media_stream_audio_processor.cc index 63b9b12..15fc836 100644 --- a/content/renderer/media/media_stream_audio_processor.cc +++ b/content/renderer/media/media_stream_audio_processor.cc @@ -426,7 +426,7 @@ void MediaStreamAudioProcessor::OnPlayoutData(media::AudioBus* audio_bus, int sample_rate, int audio_delay_milliseconds) { DCHECK(render_thread_checker_.CalledOnValidThread()); -#if defined(OS_ANDROID) || defined(OS_IOS) +#if defined(OS_ANDROID) DCHECK(audio_processing_->echo_control_mobile()->is_enabled()); DCHECK(!audio_processing_->echo_cancellation()->is_enabled()); #else @@ -482,17 +482,11 @@ void MediaStreamAudioProcessor::InitializeAudioProcessingModule( // disabled. audio_mirroring_ = audio_constraints.GetGoogAudioMirroring(); -#if defined(OS_IOS) - // On iOS, VPIO provides built-in AGC and AEC. - const bool echo_cancellation = false; - const bool goog_agc = false; -#else const bool echo_cancellation = audio_constraints.GetEchoCancellationProperty(); const bool goog_agc = audio_constraints.GetGoogAutoGainControl(); -#endif -#if defined(OS_IOS) || defined(OS_ANDROID) +#if defined(OS_ANDROID) const bool goog_experimental_aec = false; const bool goog_typing_detection = false; #else diff --git a/content/renderer/media/media_stream_audio_processor_options.cc b/content/renderer/media/media_stream_audio_processor_options.cc index 52c7aac..9020be0 100644 --- a/content/renderer/media/media_stream_audio_processor_options.cc +++ b/content/renderer/media/media_stream_audio_processor_options.cc @@ -55,7 +55,7 @@ struct { } const kDefaultAudioConstraints[] = { { MediaAudioConstraints::kEchoCancellation, true }, { MediaAudioConstraints::kGoogEchoCancellation, true }, -#if defined(OS_ANDROID) || defined(OS_IOS) +#if defined(OS_ANDROID) { MediaAudioConstraints::kGoogExperimentalEchoCancellation, false }, #else // Enable the extended filter mode AEC on all non-mobile platforms. @@ -148,7 +148,7 @@ void SetIfNotSet(rtc::Optional<bool>* field, bool value) { void MediaAudioConstraints::ApplyFixedAudioConstraints( cricket::AudioOptions* options) { SetIfNotSet(&options->echo_cancellation, true); -#if defined(OS_ANDROID) || defined(OS_IOS) +#if defined(OS_ANDROID) SetIfNotSet(&options->extended_filter_aec, false); #else // Enable the extended filter mode AEC on all non-mobile platforms. @@ -349,7 +349,7 @@ void EchoInformation::UpdateAecDelayStats( } void EnableEchoCancellation(AudioProcessing* audio_processing) { -#if defined(OS_ANDROID) || defined(OS_IOS) +#if defined(OS_ANDROID) // Mobile devices are using AECM. CHECK_EQ(0, audio_processing->echo_control_mobile()->set_routing_mode( webrtc::EchoControlMobile::kSpeakerphone)); @@ -408,7 +408,7 @@ void StopEchoCancellationDump(AudioProcessing* audio_processing) { } void EnableAutomaticGainControl(AudioProcessing* audio_processing) { -#if defined(OS_ANDROID) || defined(OS_IOS) +#if defined(OS_ANDROID) const webrtc::GainControl::Mode mode = webrtc::GainControl::kFixedDigital; #else const webrtc::GainControl::Mode mode = webrtc::GainControl::kAdaptiveAnalog; diff --git a/content/renderer/media/media_stream_audio_processor_unittest.cc b/content/renderer/media/media_stream_audio_processor_unittest.cc index d8ae092..5702046 100644 --- a/content/renderer/media/media_stream_audio_processor_unittest.cc +++ b/content/renderer/media/media_stream_audio_processor_unittest.cc @@ -122,7 +122,7 @@ class MediaStreamAudioProcessorTest : public ::testing::Test { // |audio_processor| does nothing when the audio processing is off in // the processor. webrtc::AudioProcessing* ap = audio_processor->audio_processing_.get(); -#if defined(OS_ANDROID) || defined(OS_IOS) +#if defined(OS_ANDROID) const bool is_aec_enabled = ap && ap->echo_control_mobile()->is_enabled(); // AEC should be turned off for mobiles. DCHECK(!ap || !ap->echo_cancellation()->is_enabled()); @@ -169,7 +169,7 @@ class MediaStreamAudioProcessorTest : public ::testing::Test { EXPECT_TRUE(audio_processing->echo_control_mobile()->routing_mode() == webrtc::EchoControlMobile::kSpeakerphone); EXPECT_FALSE(audio_processing->echo_cancellation()->is_enabled()); -#elif !defined(OS_IOS) +#else EXPECT_TRUE(audio_processing->echo_cancellation()->is_enabled()); EXPECT_TRUE(audio_processing->echo_cancellation()->suppression_level() == webrtc::EchoCancellation::kHighSuppression); @@ -183,7 +183,7 @@ class MediaStreamAudioProcessorTest : public ::testing::Test { webrtc::NoiseSuppression::kHigh); EXPECT_TRUE(audio_processing->high_pass_filter()->is_enabled()); EXPECT_TRUE(audio_processing->gain_control()->is_enabled()); -#if defined(OS_ANDROID) || defined(OS_IOS) +#if defined(OS_ANDROID) EXPECT_TRUE(audio_processing->gain_control()->mode() == webrtc::GainControl::kFixedDigital); EXPECT_FALSE(audio_processing->voice_detection()->is_enabled()); diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc index 4d2f53e..6395c05 100644 --- a/content/renderer/render_thread_impl.cc +++ b/content/renderer/render_thread_impl.cc @@ -745,7 +745,7 @@ void RenderThreadImpl::Init() { is_gpu_memory_buffer_compositor_resources_enabled_ = command_line.HasSwitch( switches::kEnableGpuMemoryBufferCompositorResources); -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) is_elastic_overscroll_enabled_ = base::mac::IsOSLionOrLater(); if (is_elastic_overscroll_enabled_) { base::ScopedCFTypeRef<CFStringRef> key( diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc index 6d3689a..e5052e8 100644 --- a/content/renderer/render_view_impl.cc +++ b/content/renderer/render_view_impl.cc @@ -1540,7 +1540,7 @@ void RenderViewImpl::ApplyWebPreferencesInternal( blink::WebView* web_view, CompositorDependencies* compositor_deps) { ApplyWebPreferences(prefs, web_view); -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) DCHECK(compositor_deps); bool is_elastic_overscroll_enabled = compositor_deps->IsElasticOverscrollEnabled(); @@ -2120,7 +2120,7 @@ void RenderViewImpl::initializeLayerTreeView() { return; bool use_threaded_event_handling = true; -#if defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_MACOSX) // Disable threaded event handling if content is not handling the elastic // overscroll effect. This includes the cases where the elastic overscroll // effect is being handled by Blink (because of command line flags) and older diff --git a/content/shell/app/shell_crash_reporter_client.cc b/content/shell/app/shell_crash_reporter_client.cc index 4e7bd0d..40eb6eb 100644 --- a/content/shell/app/shell_crash_reporter_client.cc +++ b/content/shell/app/shell_crash_reporter_client.cc @@ -35,7 +35,7 @@ void ShellCrashReporterClient::GetProductNameAndVersion( } #endif -#if defined(OS_POSIX) && !defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_POSIX) && !defined(OS_MACOSX) void ShellCrashReporterClient::GetProductNameAndVersion( const char** product_name, const char** version) { diff --git a/content/shell/app/shell_crash_reporter_client.h b/content/shell/app/shell_crash_reporter_client.h index 06e1d9b..1b2a58c 100644 --- a/content/shell/app/shell_crash_reporter_client.h +++ b/content/shell/app/shell_crash_reporter_client.h @@ -27,7 +27,7 @@ class ShellCrashReporterClient : public crash_reporter::CrashReporterClient { base::string16* channel_name) override; #endif -#if defined(OS_POSIX) && !defined(OS_MACOSX) && !defined(OS_IOS) +#if defined(OS_POSIX) && !defined(OS_MACOSX) // Returns a textual description of the product type and version to include // in the crash report. void GetProductNameAndVersion(const char** product_name, diff --git a/content/test/content_test_suite.cc b/content/test/content_test_suite.cc index 318e34e..a93c3e8 100644 --- a/content/test/content_test_suite.cc +++ b/content/test/content_test_suite.cc @@ -6,6 +6,7 @@ #include "base/base_paths.h" #include "base/base_switches.h" +#include "base/command_line.h" #include "base/logging.h" #include "base/macros.h" #include "build/build_config.h" @@ -14,7 +15,9 @@ #include "content/public/test/test_content_client_initializer.h" #include "gpu/config/gpu_info_collector.h" #include "gpu/config/gpu_util.h" +#include "media/base/media.h" #include "testing/gtest/include/gtest/gtest.h" +#include "ui/gl/test/gl_surface_test_support.h" #if defined(OS_WIN) #include "ui/gfx/win/dpi.h" @@ -22,17 +25,8 @@ #if defined(OS_MACOSX) #include "base/mac/scoped_nsautorelease_pool.h" -#if !defined(OS_IOS) #include "base/test/mock_chrome_application_mac.h" #endif -#endif - -#if !defined(OS_IOS) -#include "base/base_switches.h" -#include "base/command_line.h" -#include "media/base/media.h" -#include "ui/gl/test/gl_surface_test_support.h" -#endif #if defined(OS_ANDROID) #include "content/browser/android/in_process_surface_texture_manager.h" @@ -78,10 +72,8 @@ ContentTestSuite::~ContentTestSuite() { void ContentTestSuite::Initialize() { #if defined(OS_MACOSX) base::mac::ScopedNSAutoreleasePool autorelease_pool; -#if !defined(OS_IOS) mock_cr_app::RegisterMockCrApp(); #endif -#endif #if defined(OS_WIN) gfx::SetDefaultDeviceScaleFactor(1.0f); @@ -93,7 +85,6 @@ void ContentTestSuite::Initialize() { ContentTestSuiteBase::RegisterContentSchemes(&client); } RegisterPathProvider(); -#if !defined(OS_IOS) media::InitializeMediaLibrary(); // When running in a child process for Mac sandbox tests, the sandbox exists // to initialize GL, so don't do it here. @@ -106,7 +97,6 @@ void ContentTestSuite::Initialize() { base::CommandLine::ForCurrentProcess()); gfx::GLSurfaceTestSupport::InitializeOneOff(); } -#endif testing::TestEventListeners& listeners = testing::UnitTest::GetInstance()->listeners(); listeners.Append(new TestInitializationListener); diff --git a/content/test/run_all_unittests.cc b/content/test/run_all_unittests.cc index ac655a4..f0edbd9 100644 --- a/content/test/run_all_unittests.cc +++ b/content/test/run_all_unittests.cc @@ -25,13 +25,11 @@ int main(int argc, char** argv) { #endif content::UnitTestTestSuite test_suite( new content::ContentTestSuite(argc, argv)); -#if !defined(OS_IOS) content::InitializeMojo(); base::TestIOThread test_io_thread(base::TestIOThread::kAutoStart); scoped_ptr<mojo::edk::test::ScopedIPCSupport> ipc_support; ipc_support.reset( new mojo::edk::test::ScopedIPCSupport(test_io_thread.task_runner())); -#endif return base::LaunchUnitTests( argc, argv, base::Bind(&content::UnitTestTestSuite::Run, diff --git a/content/test/test_content_client.cc b/content/test/test_content_client.cc index edca49c..d59a911 100644 --- a/content/test/test_content_client.cc +++ b/content/test/test_content_client.cc @@ -22,7 +22,6 @@ namespace content { TestContentClient::TestContentClient() : data_pack_(ui::SCALE_FACTOR_100P) { // content_shell.pak is not built on iOS as it is not required. -#if !defined(OS_IOS) base::FilePath content_shell_pack_path; base::File pak_file; base::MemoryMappedFile::Region pak_region; @@ -47,7 +46,6 @@ TestContentClient::TestContentClient() FILE_PATH_LITERAL("content_shell.pak")); data_pack_.LoadFromPath(content_shell_pack_path); } -#endif // !defined(OS_IOS) } TestContentClient::~TestContentClient() { |