Declarative content scripts: Renderer-side: per-extension shared memory regions (lazily loaded)

extensions/renderer/user_script_set.cc

Index: extensions/renderer/user_script_set.cc
diff --git a/extensions/renderer/user_script_set.cc b/extensions/renderer/user_script_set.cc
index 5ab873955f026e98159d5567a4c111821d130c98..1ba38cb5e22b83c0c17139f6e7ab8fc151de7240 100644
--- a/extensions/renderer/user_script_set.cc
+++ b/extensions/renderer/user_script_set.cc
@@ -79,22 +79,12 @@ void UserScriptSet::GetInjections(
   }
 }
 
-bool UserScriptSet::OnControlMessageReceived(
-    const IPC::Message& message) {
-  bool handled = true;
-  IPC_BEGIN_MESSAGE_MAP(UserScriptSet, message)
-    IPC_MESSAGE_HANDLER(ExtensionMsg_UpdateUserScripts, OnUpdateUserScripts)
-    IPC_MESSAGE_UNHANDLED(handled = false)
-  IPC_END_MESSAGE_MAP()
-  return handled;
-}
-
-void UserScriptSet::OnUpdateUserScripts(
+bool UserScriptSet::OnUpdateUserScripts(
     base::SharedMemoryHandle shared_memory,
     const std::set<std::string>& changed_extensions) {
   if (!base::SharedMemory::IsHandleValid(shared_memory)) {
     NOTREACHED() << "Bad scripts handle";
-    return;
+    return false;
   }
 
   for (std::set<std::string>::const_iterator iter = changed_extensions.begin();
@@ -102,15 +92,18 @@ void UserScriptSet::OnUpdateUserScripts(
        ++iter) {
     if (!Extension::IdIsValid(*iter)) {
       NOTREACHED() << "Invalid extension id: " << *iter;
-      return;
+      return false;
     }
   }
 
-  if (UpdateScripts(shared_memory)) {
-    FOR_EACH_OBSERVER(Observer,
-                      observers_,
-                      OnUserScriptsUpdated(changed_extensions, scripts_.get()));
+  if (!UpdateScripts(shared_memory)) {
+    return false;
   }
+
+  FOR_EACH_OBSERVER(Observer,
+                    observers_,
+                    OnUserScriptsUpdated(changed_extensions, scripts_.get()));
+  return true;
 }
 
 bool UserScriptSet::UpdateScripts(