Introduce HostID and de-couple Extensions from "script injection System" [browser side]

chrome/browser/extensions/extension_user_script_loader.cc

Index: chrome/browser/extensions/extension_user_script_loader.cc
diff --git a/chrome/browser/extensions/extension_user_script_loader.cc b/chrome/browser/extensions/extension_user_script_loader.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e60f104c68b9077afc554b3ac12e44498d099d07
--- /dev/null
+++ b/chrome/browser/extensions/extension_user_script_loader.cc
@@ -0,0 +1,208 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.

[Devlin, 2015/01/14 16:45:09]

check year (everywhere)

[Xi Han, 2015/01/14 23:46:03]

Thanks. I didn't realize there are so many new files were added.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/extensions/extension_user_script_loader.h"
+
+#include <set>
+#include <string>
+
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/files/file_path.h"
+#include "base/files/file_util.h"
+#include "base/version.h"
+#include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/profiles/profile.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/notification_service.h"
+#include "content/public/browser/render_process_host.h"
+#include "extensions/browser/component_extension_resource_manager.h"
+#include "extensions/browser/content_verifier.h"
+#include "extensions/browser/extension_registry.h"
+#include "extensions/browser/extension_system.h"
+#include "extensions/browser/extensions_browser_client.h"
+#include "extensions/common/extension_messages.h"
+#include "extensions/common/file_util.h"
+#include "extensions/common/manifest_handlers/default_locale_handler.h"
+#include "extensions/common/message_bundle.h"
+#include "extensions/common/one_shot_event.h"
+#include "ui/base/resource/resource_bundle.h"
+
+using content::BrowserThread;
+
+namespace extensions {
+class ExtensionsBrowserClient;

[Devlin, 2015/01/14 16:45:09]

why do you need this?

[Xi Han, 2015/01/14 23:46:03]

Removed.

+
+namespace {
+
+// Verifies file contents as they are read.
+void VerifyContent(scoped_refptr<ContentVerifier> verifier,
+                   const ConsumerID& consumer_id,
+                   const base::FilePath& extension_root,
+                   const base::FilePath& relative_path,
+                   const std::string& content) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
+  scoped_refptr<ContentVerifyJob> job(verifier->CreateJobFor(
+      consumer_id.host_id(), extension_root, relative_path));
+  if (job.get()) {
+    job->Start();
+    job->BytesRead(content.size(), content.data());
+    job->DoneReading();
+  }
+}
+
+}  // namespace
+
+ExtensionUserScriptLoader::ExtensionUserScriptLoader(
+    Profile* profile,
+    const ConsumerID& consumer_id,
+    bool listen_for_extension_system_loaded)
+    : UserScriptLoader(profile, consumer_id),
+      extension_system_ready_(false),
+      extension_registry_observer_(this),
+      weak_factory_(this) {
+  extension_registry_observer_.Add(ExtensionRegistry::Get(profile));
+  if (listen_for_extension_system_loaded) {
+    ExtensionSystem::Get(profile)->ready().Post(
+        FROM_HERE,
+        base::Bind(&ExtensionUserScriptLoader::OnExtensionSystemReady,
+                    weak_factory_.GetWeakPtr()));

[Devlin, 2015/01/14 16:45:09]

indentation

[Xi Han, 2015/01/14 23:46:03]

Done.

+  } else

[Devlin, 2015/01/14 16:45:09]

our bracketing rules are complex.
if (one_line_if_body)
  NoBrackets()

if (multi_line_if_body) {
  UseBrackets()
  ...
}

if (multi_line_if) {
  UseBrackets()
  ...
} else if (else_with_previous_multi_line_if) {
  StillUseBrackets()
}

I believe the linter calls this rule "If an else has brackets on one side, it
must also have brackets on the other side".

[Xi Han, 2015/01/14 23:46:03]

I must misunderstood your last round of comments and changed it like this. I
also believe the else should have {} if the if has one. Thank you for the
details:)

+    extension_system_ready_ = true;
+}
+
+ExtensionUserScriptLoader::~ExtensionUserScriptLoader() {
+}
+
+void ExtensionUserScriptLoader::UpdateConsumersInfo() {
+  ExtensionRegistry* registry = ExtensionRegistry::Get(profile_);
+  for (std::set<ConsumerID>::const_iterator it = changed_consumers_.begin();

[Devlin, 2015/01/14 16:45:09]

Let's C++11 this:
for (const ConsumerID& consumer_id : changed_consumers_) {
   ...
}

[Xi Han, 2015/01/14 23:46:03]

Done.

+       it != changed_consumers_.end(); ++it) {
+    if (consumers_info_.find(*it) == consumers_info_.end()) {
+      const Extension* extension = registry->GetExtensionById(
+          it->host_id(), ExtensionRegistry::ENABLED);
+      // |changed_consumers_| may include consumers that have been removed,
+      // which leads to the above lookup failing. In this case, just continue.
+      if (!extension)
+        continue;
+      consumers_info_[*it] = ExtensionSet::ExtensionPathAndDefaultLocale(
+          extension->path(), LocaleInfo::GetDefaultLocale(extension));
+    }
+  }
+}
+
+bool ExtensionUserScriptLoader::Ready() {
+  return extension_system_ready_;
+}
+
+ContentVerifier* ExtensionUserScriptLoader::GetContentVerifier() {
+  return ExtensionSystem::Get(profile_)->content_verifier();
+}
+
+void ExtensionUserScriptLoader::SendUpdate(
+    content::RenderProcessHost* process,
+    base::SharedMemoryHandle handle_for_process,
+    const std::set<ConsumerID>& changed_consumers) {
+  // TODO(hanxi): update the IPC message to send a set of ConsumerID to render.
+  // Also, remove this function when the refactor is done on render side.
+  std::set<std::string> changed_extensions;
+  for (std::set<ConsumerID>::iterator it = changed_consumers.begin();
+       it != changed_consumers.end(); ++it) {
+    changed_extensions.insert(it->host_id());
+  }
+
+  if (base::SharedMemory::IsHandleValid(handle_for_process)) {
+    process->Send(new ExtensionMsg_UpdateUserScripts(
+        handle_for_process, consumer_id_.host_id(), changed_extensions));
+  }
+}
+
+UserScriptLoader::LoadUserScriptsFunctionCallback
+ExtensionUserScriptLoader::GetLoadUserScriptsFunction() {
+  return base::Bind(&ExtensionUserScriptLoader::LoadScriptContent);
+}
+
+// static
+bool ExtensionUserScriptLoader::LoadScriptContent(
+    const ConsumerID& consumer_id,
+    UserScript::File* script_file,
+    const SubstitutionMap* localization_messages,
+    scoped_refptr<ContentVerifier> verifier) {
+  DCHECK(script_file);
+  std::string content;
+  const base::FilePath& path = ExtensionResource::GetFilePath(
+      script_file->extension_root(), script_file->relative_path(),
+      ExtensionResource::SYMLINKS_MUST_RESOLVE_WITHIN_ROOT);
+  if (path.empty()) {
+    int resource_id = 0;
+    if (ExtensionsBrowserClient::Get()
+            ->GetComponentExtensionResourceManager()
+            ->IsComponentExtensionResource(script_file->extension_root(),
+                                           script_file->relative_path(),
+                                           &resource_id)) {
+      const ResourceBundle& rb = ResourceBundle::GetSharedInstance();
+      content = rb.GetRawDataResource(resource_id).as_string();
+    } else {
+      LOG(WARNING) << "Failed to get file path to "
+                   << script_file->relative_path().value() << " from "
+                   << script_file->extension_root().value();
+      return false;
+    }
+  } else {
+    if (!base::ReadFileToString(path, &content)) {
+      LOG(WARNING) << "Failed to load user script file: " << path.value();
+      return false;
+    }
+    if (verifier.get()) {
+      content::BrowserThread::PostTask(
+          content::BrowserThread::IO, FROM_HERE,
+          base::Bind(&VerifyContent, verifier, consumer_id,
+                     script_file->extension_root(),
+                     script_file->relative_path(), content));
+    }
+  }
+
+  // Localize the content.
+  if (localization_messages) {
+    std::string error;
+    MessageBundle::ReplaceMessagesWithExternalDictionary(*localization_messages,
+                                                         &content, &error);
+    if (!error.empty())
+      LOG(WARNING) << "Failed to replace messages in script: " << error;
+  }
+
+  // Remove BOM from the content.
+  std::string::size_type index = content.find(base::kUtf8ByteOrderMark);
+  if (index == 0) {
+    script_file->set_content(content.substr(strlen(base::kUtf8ByteOrderMark)));
+  } else
+    script_file->set_content(content);
+
+  return true;
+}
+
+void ExtensionUserScriptLoader::OnExtensionUnloaded(
+    content::BrowserContext* browser_context,
+    const Extension* extension,
+    UnloadedExtensionInfo::Reason reason) {
+  std::set<ConsumerID> ids_to_delete;
+
+  for (ConsumersInfo::iterator it = consumers_info_.begin();
+       it != consumers_info_.end(); ++it) {
+    const ConsumerID& id = it->first;
+    if (id.host_id() == extension->id())
+      ids_to_delete.insert(id);
+  }
+
+  for (std::set<ConsumerID>::iterator it = ids_to_delete.begin();
+       it != ids_to_delete.end(); ++it)
+    consumers_info_.erase(*it);
+}
+
+void ExtensionUserScriptLoader::OnExtensionSystemReady() {
+  extension_system_ready_ = true;
+  AttemptLoad();
+}
+
+}  // namespace extensions