Initialize AVFoundation explicitly

chrome/browser/chrome_browser_main_mac.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chrome_browser_main_mac.h"
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
@@ skipping 10 matching lines @@
 #include "chrome/browser/mac/install_from_dmg.h"
 #import "chrome/browser/mac/keystone_glue.h"
 #include "chrome/browser/mac/mac_startup_profiler.h"
 #include "chrome/browser/ui/app_list/app_list_service.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "components/crash/app/crashpad_mac.h"
 #include "components/metrics/metrics_service.h"
 #include "content/public/common/main_function_params.h"
 #include "content/public/common/result_codes.h"
+#import "media/base/mac/avfoundation_glue.h"
 #include "ui/base/l10n/l10n_util_mac.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/resource/resource_handle.h"
 
 namespace {
 
 // Writes an undocumented sentinel file that prevents Spotlight from indexing
 // below a particular path in order to reap some power savings.
 void EnsureMetadataNeverIndexFileOnFileThread(
     const base::FilePath& user_data_dir) {
@@ skipping 73 matching lines @@
 
   base::FilePath resources_pack_path;
   PathService::Get(chrome::FILE_RESOURCES_PACK, &resources_pack_path);
   ResourceBundle::GetSharedInstance().AddDataPackFromPath(
       resources_pack_path, ui::SCALE_FACTOR_NONE);
 
   // This is a no-op if the KeystoneRegistration framework is not present.
   // The framework is only distributed with branded Google Chrome builds.
   [[KeystoneGlue defaultKeystoneGlue] registerWithKeystone];
 
+  // Initialize AVFoundation if supported, for audio and video.
+  AVFoundationGlue::InitializeAVFoundation();

[jam, 2015/05/26 23:13:47]

i think we want this in src/content so that all content-based browsers get it?

[tommi (sloooow) - chröme, 2015/05/27 09:11:02]

Yes, makes sense.  Put it in BrowserMainLoop::PreCreateThreads?

+
   // Disk image installation is sort of a first-run task, so it shares the
   // no first run switches.
   //
   // This needs to be done after the resource bundle is initialized (for
   // access to localizations in the UI) and after Keystone is initialized
   // (because the installation may need to promote Keystone) but before the
   // app controller is set up (and thus before MainMenu.nib is loaded, because
   // the app controller assumes that a browser has been set up and will crash
   // upon receipt of certain notifications if no browser exists), before
   // anyone tries doing anything silly like firing off an import job, and
@@ skipping 55 matching lines @@
   // TODO(calamity): Make this gated on first_run::IsChromeFirstRun() in M45.
   content::BrowserThread::PostAfterStartupTask(
       FROM_HERE, base::ThreadTaskRunnerHandle::Get(),
       base::Bind(&EnsureMetadataNeverIndexFile, user_data_dir()));
 }
 
 void ChromeBrowserMainPartsMac::DidEndMainMessageLoop() {
   AppController* appController = [NSApp delegate];
   [appController didEndMainMessageLoop];
 }