Load pdf library on all platforms.

chrome/app/chrome_main_delegate.cc

 // 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/app/chrome_main_delegate.h"
 
 #include "base/command_line.h"
 #include "base/environment.h"
 #include "base/files/file_path.h"
 #include "base/i18n/rtl.h"
@@ skipping 678 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);
 #endif
     CHECK(!loaded_locale.empty()) << "Locale could not be found for " <<
         locale;
 
 #if !defined(CHROME_MULTIPLE_DLL_BROWSER)
-    if (process_type == switches::kUtilityProcess)
+    if (process_type == switches::kUtilityProcess ||
+        process_type == switches::kZygoteProcess) {
       chrome::ChromeContentUtilityClient::PreSandboxStartup();

[jln (very slow on Chromium), 2013/11/14 23:14:18]

Do you mind calling this from content/zygote/zygote_main_linux.cc
PreSandboxInit() /  
 PreloadPepperPlugins() instead ?

This is confusing and should be refactored, but the vast majority of pre-sandbox
library load is done there.

[Vitaly Buka (NO REVIEWS), 2013/11/14 23:53:18]

PreSandboxInit() /  
 PreloadPepperPlugins()
are parts of content, I can't call chrome/code from there without re-factoring.

On 2013/11/14 23:14:18, jln wrote:

[jln (very slow on Chromium), 2013/11/15 00:11:22]

Ahh yeah, of course, silly me.

I hope that the Zygote trying to reload the same thing will not cause much
computation to be duplicated. It's fairly unlikely that it would if the library
doesn't get loaded twice.

Is there a clean way to exclude the PDF library from PreloadPepperPlugins() with
a comment pointing here?

+    }
 #endif
   }
 
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
   // Zygote needs to call InitCrashReporter() in RunZygote().
   if (process_type != switches::kZygoteProcess) {
 #if defined(OS_ANDROID)
     if (process_type.empty())
       breakpad::InitCrashReporter();
     else
@@ skipping 140 matching lines @@
 }
 
 content::ContentUtilityClient*
 ChromeMainDelegate::CreateContentUtilityClient() {
 #if defined(CHROME_MULTIPLE_DLL_BROWSER)
   return NULL;
 #else
   return &g_chrome_content_utility_client.Get();
 #endif
 }