Move responsibility for l10n parameters to the caller.

remoting/host/plugin/host_script_object.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 "remoting/host/plugin/host_script_object.h"
 
 #include "base/bind.h"
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/message_loop.h"
@@ skipping 1033 matching lines @@
 #else
       /*i18n-content*/"DISCONNECT_BUTTON_PLUS_SHORTCUT_LINUX",
 #endif
       &ui_strings.disconnect_button_text_plus_shortcut);
   LocalizeString(localize_func, /*i18n-content*/"CONTINUE_PROMPT",
                  &ui_strings.continue_prompt);
   LocalizeString(localize_func, /*i18n-content*/"CONTINUE_BUTTON",
                  &ui_strings.continue_button_text);
   LocalizeString(localize_func, /*i18n-content*/"STOP_SHARING_BUTTON",
                  &ui_strings.stop_sharing_button_text);
-  LocalizeString(localize_func, /*i18n-content*/"MESSAGE_SHARED",
-                 &ui_strings.disconnect_message);
+  const char* substitutions[] = { "$1" };
+  LocalizeStringWithSubstitutions(localize_func,
+                                  /*i18n-content*/"MESSAGE_SHARED",
+                                  substitutions, arraysize(substitutions),
+                                  &ui_strings.disconnect_message);
 
   base::AutoLock auto_lock(ui_strings_lock_);
   ui_strings_ = ui_strings;
 }
 
 bool HostNPScriptObject::LocalizeString(NPObject* localize_func,
                                         const char* tag, string16* result) {
-  NPVariant args[2];
+  return LocalizeStringWithSubstitutions(localize_func, tag, NULL, 0, result);
+}
+
+bool HostNPScriptObject::LocalizeStringWithSubstitutions(
+    NPObject* localize_func, const char* tag, const char** substitutions,
+    int number_of_substitutions, string16* result) {
+  scoped_array<NPVariant> args(new NPVariant[number_of_substitutions + 1]);
   STRINGZ_TO_NPVARIANT(tag, args[0]);
+  for (int i = 0; i < number_of_substitutions; ++i) {
+    STRINGZ_TO_NPVARIANT(substitutions[i], args[i + 1]);
+  }
   NPVariant np_result;
   bool is_good = g_npnetscape_funcs->invokeDefault(
-      plugin_, localize_func, &args[0], 1, &np_result);
+      plugin_, localize_func, args.get(), number_of_substitutions + 1,

[Sergey Ulanov, 2012/06/28 01:11:24]

Actually chrome.i18n.getMessage() takes only two parameters - the second
parameter is either one string or an array of strings, so this works correctly
only for the case when there is only one substitution.

[Jamie, 2012/06/28 23:02:41]

So it does--good catch! Passing arrays over NPAPI is quite painful, so I've
opted to support only one substitution for now. A relatively simple alternative
would be to pass the strings as individual parameters and turn them into an
array in Javascript, but I don't think that's worth doing right now.

+      &np_result);
   if (!is_good) {
     LOG(ERROR) << "Localization failed for " << tag;
     return false;
   }
   std::string translation = StringFromNPVariant(np_result);
   g_npnetscape_funcs->releasevariantvalue(&np_result);
   if (translation.empty()) {
     LOG(ERROR) << "Missing translation for " << tag;
     return false;
   }
@@ skipping 148 matching lines @@
   return is_good;
 }
 
 void HostNPScriptObject::SetException(const std::string& exception_string) {
   DCHECK(plugin_message_loop_proxy_->BelongsToCurrentThread());
   g_npnetscape_funcs->setexception(parent_, exception_string.c_str());
   LOG(INFO) << exception_string;
 }
 
 }  // namespace remoting