Remove SandboxedPages and SandboxedCSP from Extension Class

chrome/browser/extensions/extension_process_manager.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 "base/bind.h"
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
@@ skipping 12 matching lines @@
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/view_type_utils.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/background_info.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_messages.h"
 #include "chrome/common/extensions/manifest_handler.h"
 #include "chrome/common/extensions/manifest_url_handler.h"
+#include "chrome/common/extensions/sandboxed_handler.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_delegate.h"
 #include "content/public/common/renderer_preferences.h"
 
@@ skipping 131 matching lines @@
   }
   event_page_suspending_time_ = base::TimeDelta::FromSeconds(5);
   unsigned suspending_time_sec = 0;
   if (base::StringToUint(CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
           switches::kEventPageSuspendingTime), &suspending_time_sec)) {
     event_page_suspending_time_ = base::TimeDelta::FromSeconds(
         suspending_time_sec);
   }
 
   (new BackgroundManifestHandler())->Register();
+  (new extensions::SandboxedHandler)->Register();

[Yoyo Zhou, 2013/03/20 22:53:53]

Does this really belong in ExtensionProcessManager (which some have argued isn't
even a good place for BackgroundManifestHandler)? It seems out of place here.
Wherever the other CSP-related handlers are registered seems more appropriate.

[Devlin, 2013/03/23 22:26:25]

Yeah, I wasn't really sure where to put this one... it's with the other CSP ones
now.

 }
 
 ExtensionProcessManager::~ExtensionProcessManager() {
   CloseBackgroundHosts();
   DCHECK(background_hosts_.empty());
 }
 
 const ExtensionProcessManager::ViewSet
 ExtensionProcessManager::GetAllViews() const {
   ViewSet result;
@@ skipping 709 matching lines @@
         if (service && service->is_ready())
           CreateBackgroundHostsForProfileStartup();
       }
       break;
     }
     default:
       ExtensionProcessManager::Observe(type, source, details);
       break;
   }
 }