Localized Chromoting Host on Mac and Linux.

remoting/host/plugin/host_plugin.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 <stdio.h>
 #include <string.h>
 
 #include <string>
 #include <vector>
 
 #include "base/at_exit.h"
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/strings/stringize_macros.h"
 #include "net/socket/ssl_server_socket.h"
 #include "remoting/base/plugin_thread_task_runner.h"
+#include "remoting/base/resources.h"
+#include "remoting/base/string_resources.h"
 #include "remoting/host/plugin/constants.h"
 #include "remoting/host/plugin/host_log_handler.h"
 #include "remoting/host/plugin/host_plugin_utils.h"
 #include "remoting/host/plugin/host_script_object.h"
 #if defined(OS_WIN)
 #include "ui/base/win/dpi.h"
 #endif
 #include "third_party/npapi/bindings/npapi.h"
 #include "third_party/npapi/bindings/npfunctions.h"
 #include "third_party/npapi/bindings/npruntime.h"
+#include "ui/base/l10n/l10n_util.h"
 
 // Symbol export is handled with a separate def file on Windows.
 #if defined (__GNUC__) && __GNUC__ >= 4
 #define EXPORT __attribute__((visibility("default")))
 #else
 #define EXPORT
 #endif
 
 #if defined(OS_WIN)
 // TODO(wez): libvpx expects these 64-bit division functions to be provided
@@ skipping 11 matching lines @@
 }
 #endif
 
 using remoting::g_npnetscape_funcs;
 using remoting::HostLogHandler;
 using remoting::HostNPScriptObject;
 using remoting::StringFromNPIdentifier;
 
 namespace {
 
+bool g_initialized = false;
+
 base::AtExitManager* g_at_exit_manager = NULL;
 
+// The plugin name and description returned by GetValue().
+std::string* g_ui_name = NULL;
+std::string* g_ui_description = NULL;
+
 // NPAPI plugin implementation for remoting host.
 // Documentation for most of the calls in this class can be found here:
 // https://developer.mozilla.org/en/Gecko_Plugin_API_Reference/Scripting_plugins
 class HostNPPlugin : public remoting::PluginThreadTaskRunner::Delegate {
  public:
   // |mode| is the display mode of plug-in. Values:
   // NP_EMBED: (1) Instance was created by an EMBED tag and shares the browser
   //               window with other content.
   // NP_FULL: (2) Instance was created by a separate file and is the primary
   //              content in the window.
@@ skipping 277 matching lines @@
   NPP instance_;
   NPObject* scriptable_object_;
 
   scoped_refptr<remoting::PluginThreadTaskRunner> plugin_task_runner_;
   scoped_refptr<remoting::AutoThreadTaskRunner> plugin_auto_task_runner_;
 
   std::map<uint32_t, DelayedTask> timers_;
   base::Lock timers_lock_;
 };
 
+void InitializePlugin() {
+  if (g_initialized)
+    return;
+
+  g_initialized = true;
+  g_at_exit_manager = new base::AtExitManager;
+
+  // Init an empty command line for common objects that use it.
+  CommandLine::Init(0, NULL);
+
+  // Do not use .pak files on Windows.
+#if !defined(OS_WIN)

[Sergey Ulanov, 2013/07/23 22:45:40]

how is g_ui_name supposed to be initialized on Windows?

[alexeypa (please no reviews), 2013/07/23 23:35:26]

g_ui_name and g_ui_description are not used on Windows and Mac. It is not
obvious from the code, so I'll update it.

+  remoting::LoadResources("");
+
+  g_ui_name = new std::string(l10n_util::GetStringUTF8(IDR_PRODUCT_NAME));
+  g_ui_description = new std::string(
+      l10n_util::GetStringUTF8(IDR_PRODUCT_DESCRIPTION));
+
+#endif  // !defined(OS_WIN)
+}
+
+void ShutdownPlugin() {
+  // Do not use .pak files on Windows.
+#if !defined(OS_WIN)
+  delete g_ui_name;
+  delete g_ui_description;
+
+  remoting::UnloadResources();
+#endif  // !defined(OS_WIN)
+
+  delete g_at_exit_manager;
+}
+
 // Utility functions to map NPAPI Entry Points to C++ Objects.
 HostNPPlugin* PluginFromInstance(NPP instance) {
   return reinterpret_cast<HostNPPlugin*>(instance->pdata);
 }
 
 NPError CreatePlugin(NPMIMEType pluginType,
                      NPP instance,
                      uint16 mode,
                      int16 argc,
                      char** argn,
@@ skipping 33 matching lines @@
     plugin->Save(save);
     delete plugin;
     instance->pdata = NULL;
     return NPERR_NO_ERROR;
   } else {
     return NPERR_INVALID_PLUGIN_ERROR;
   }
 }
 
 NPError GetValue(NPP instance, NPPVariable variable, void* value) {
+  // NP_GetValue() can be called before NP_Initialize().
+  InitializePlugin();
+
   switch(variable) {
   default:
     VLOG(2) << "GetValue - default " << variable;
     return NPERR_GENERIC_ERROR;
   case NPPVpluginNameString:
     VLOG(2) << "GetValue - name string";
-    *reinterpret_cast<const char**>(value) = HOST_PLUGIN_NAME;
+    *reinterpret_cast<const char**>(value) = g_ui_name->c_str();

[Sergey Ulanov, 2013/07/23 22:45:40]

can we call l10n_util::GetStringUTF8() here instead of storing the name in
g_ui_name?

[Sergey Ulanov, 2013/07/23 22:45:40]

Are you sure we need to localize this name and the description below? 
They are shown only in about:plugins, which is not user facing, so doesn't need
localization.
These strings do not seem to be localized for any other plugins (e.g. Flash, PDF
Viewer).

[alexeypa (please no reviews), 2013/07/23 23:35:26]

We can live without localizing them but once we got dialogs localized it is much
nicer to localize these as well. Also these strings are stored in a very
localizable way on Mac and Windows implying that they might localized as well.
They are localized by some plugins, QuickTime for instance.

[alexeypa (please no reviews), 2013/07/23 23:35:26]

No, because the plugin still owns the string and it should be kept alive after
GetValue() returns.

[Sergey Ulanov, 2013/07/25 18:40:57]

I still think that name and description shouldn't be localized. Localizing them
adds these ugly globals without giving anything in return. Yes, some plugins
have these strings localized, but I think it makes more sense to be consistent
with default plugins that ship with Chrome.

[alexeypa (please no reviews), 2013/07/25 18:48:03]

It gives something in return. It let's the plugin name and description to be
defined in one place (.grd) instead of having it defined in multiple places like
(branding files, .gyp and .cc).

[Sergey Ulanov, 2013/07/26 17:58:25]

Ok, makes sense. Maybe add translateable="false" for these strings in GRD file?

[alexeypa (please no reviews), 2013/07/26 18:55:01]

Why? They are already translated and the translations are nicely shown in the UI
(even if it just about:plugins). It is better to have a translated string if the
user prefers a translated string.

     break;
   case NPPVpluginDescriptionString:
     VLOG(2) << "GetValue - description string";
-    *reinterpret_cast<const char**>(value) = HOST_PLUGIN_DESCRIPTION;
+    *reinterpret_cast<const char**>(value) = g_ui_description->c_str();
     break;
   case NPPVpluginNeedsXEmbed:
     VLOG(2) << "GetValue - NeedsXEmbed";
     *(static_cast<NPBool*>(value)) = true;
     break;
   case NPPVpluginScriptableNPObject:
     VLOG(2) << "GetValue - scriptable object";
     HostNPPlugin* plugin = PluginFromInstance(instance);
     if (!plugin)
       return NPERR_INVALID_PLUGIN_ERROR;
@@ skipping 51 matching lines @@
 
   return NPERR_NO_ERROR;
 }
 
 EXPORT NPError API_CALL NP_Initialize(NPNetscapeFuncs* npnetscape_funcs
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
                             , NPPluginFuncs* nppfuncs
 #endif
                             ) {
   VLOG(2) << "NP_Initialize";
-  if (g_at_exit_manager)
-    return NPERR_MODULE_LOAD_FAILED_ERROR;
+  InitializePlugin();
 
   if(npnetscape_funcs == NULL)
     return NPERR_INVALID_FUNCTABLE_ERROR;
 
   if(((npnetscape_funcs->version & 0xff00) >> 8) > NP_VERSION_MAJOR)
     return NPERR_INCOMPATIBLE_VERSION_ERROR;
 
-  g_at_exit_manager = new base::AtExitManager;
   g_npnetscape_funcs = npnetscape_funcs;
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
   NP_GetEntryPoints(nppfuncs);
 #endif
-  // Init an empty command line for common objects that use it.
-  CommandLine::Init(0, NULL);
 
 #if defined(OS_WIN)
   ui::EnableHighDPISupport();
 #endif
 
   return NPERR_NO_ERROR;
 }
 
 EXPORT NPError API_CALL NP_Shutdown() {
   VLOG(2) << "NP_Shutdown";
-  delete g_at_exit_manager;
-  g_at_exit_manager = NULL;
+  ShutdownPlugin();
+
   return NPERR_NO_ERROR;
 }
 
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
 EXPORT const char* API_CALL NP_GetMIMEDescription(void) {
   VLOG(2) << "NP_GetMIMEDescription";
   return HOST_PLUGIN_MIME_TYPE "::";
 }
 
 EXPORT NPError API_CALL NP_GetValue(void* npp,
                                     NPPVariable variable,
                                     void* value) {
   return GetValue((NPP)npp, variable, value);
 }
 #endif
 
 }  // extern "C"