Injected CSS localization fix (see bug no.)

chrome/browser/extensions/execute_code_in_tab_function.cc

 // Copyright (c) 2011 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/extensions/execute_code_in_tab_function.h"
 
 #include "base/callback.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_tabs_module.h"
 #include "chrome/browser/extensions/extension_tabs_module_constants.h"
 #include "chrome/browser/extensions/file_reader.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/extensions/extension_error_utils.h"
+#include "chrome/common/extensions/extension_file_util.h"
+#include "chrome/common/extensions/extension_l10n_util.h"
 #include "chrome/common/extensions/extension_messages.h"
+#include "chrome/common/extensions/extension_message_bundle.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/common/notification_service.h"
 
 namespace keys = extension_tabs_module_constants;
 
 ExecuteCodeInTabFunction::ExecuteCodeInTabFunction()
     : execute_tab_id_(-1),
       all_frames_(false) {
@@ skipping 123 matching lines @@
     return false;
   }
 
   const Extension* extension = GetExtension();
   if (!extension) {
     SendResponse(false);
     return false;
   }
 
   bool is_js_code = true;
+  // We need to do message replacement on code_string, so it must be mutable.
+  std::string code_string_to_execute = code_string;
   std::string function_name = name();
   if (function_name == TabsInsertCSSFunction::function_name()) {
     is_js_code = false;
+    extension_l10n_util::LocalizeCSSScript(extension, code_string_to_execute);

[Mihai Parparita -not on Chrome, 2011/08/11 01:58:34]

I don't think you want to do this when you're passed in a code string (callsite
at line 104), only when you loaded a file (callsite at line 131).

[Mihai Parparita -not on Chrome, 2011/08/11 01:58:34]

LocalizeCSSScript ends up doing file IO (to load the messages), and extension
functions are handled on the UI thread, so I think you'll need to execute this
on the file thread.

[adriansc, 2011/08/11 21:25:08]

I moved the localization logic to the file thread in DidLoadFile before calling
Execute.

[adriansc, 2011/08/11 21:25:08]

Done. Localization is only called in DidLoadFile now.

   } else if (function_name != TabsExecuteScriptFunction::function_name()) {
     DCHECK(false);
   }
 
   ExtensionMsg_ExecuteCode_Params params;
   params.request_id = request_id();
   params.extension_id = extension->id();
   params.is_javascript = is_js_code;
-  params.code = code_string;
+  params.code = code_string_to_execute;
   params.all_frames = all_frames_;
   params.in_main_world = false;
   contents->render_view_host()->Send(new ExtensionMsg_ExecuteCode(
       contents->render_view_host()->routing_id(), params));
 
   Observe(contents->tab_contents());
   AddRef();  // balanced in OnExecuteCodeFinished()
   return true;
 }
 
@@ skipping 24 matching lines @@
   if (!error.empty()) {
     CHECK(!success);
     error_ = error;
   }
 
   SendResponse(success);
 
   Observe(NULL);
   Release();  // balanced in Execute()
 }