Load both language packs and the resource dlls in ChromeFrame and look for localized strings

chrome_frame/simple_resource_loader.h

 // 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.
 //
 // This class implements a simplified and much stripped down version of
 // Chrome's app/resource_bundle machinery. It notably avoids unwanted
 // dependencies on things like Skia.
 //
 
 #ifndef CHROME_FRAME_SIMPLE_RESOURCE_LOADER_H_
 #define CHROME_FRAME_SIMPLE_RESOURCE_LOADER_H_
 
 #include <windows.h>
 #include <string>
 #include <vector>
 
 #include "base/file_path.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/singleton.h"
+#include "ui/base/resource/data_pack.h"
 
 class SimpleResourceLoader {
  public:
 
   static SimpleResourceLoader* GetInstance();
 
   // Returns the language tag for the active language.
   static std::wstring GetLanguage();
 
   // Helper method to return the string resource identified by message_id
@@ skipping 11 matching lines @@
   // Populates |locales_path| with the path to the "Locales" directory.
   static void DetermineLocalesDirectory(FilePath* locales_path);
 
   // Returns false if |language_tag| is malformed.
   static bool IsValidLanguageTag(const std::wstring& language_tag);
 
  private:
   SimpleResourceLoader();
   ~SimpleResourceLoader();
 
-  // Finds the most-preferred resource DLL for the laguages in |language_tags|
+  // Finds the most-preferred resource dll and language pack for the laguages
+  // in |language_tags|
   // in |locales_path|.
   //
   // Returns true on success with a handle to the DLL that was loaded in
-  // |dll_handle| and its path in |file_path|.
-  static bool LoadLocaleDll(const std::vector<std::wstring>& language_tags,
-                            const FilePath& locales_path,
-                            HMODULE* dll_handle,
-                            FilePath* file_path);
+  // |dll_handle|, the data pack in data_pack and its path in |file_path|.
+  static bool LoadLocalePack(const std::vector<std::wstring>& language_tags,
+                             const FilePath& locales_path,
+                             HMODULE* dll_handle,
+                             ui::DataPack** data_pack,
+                             FilePath* file_path);

[tony, 2011/08/26 21:36:30]

Nit: Maybe renamed file_path to pak_file_path?

[ananta, 2011/08/26 21:50:17]

Replaced file_path with the language parameter as requested.

 
   // Returns the string resource identified by message_id from the currently
   // loaded locale dll.
   std::wstring GetLocalizedResource(int message_id);
 
   friend struct DefaultSingletonTraits<SimpleResourceLoader>;
 
   FRIEND_TEST_ALL_PREFIXES(SimpleResourceLoaderTest, LoadLocaleDll);
 
   std::wstring language_;
+  ui::DataPack* data_pack_;

[tony, 2011/08/26 21:36:30]

Nit: You could probably make this a scoped_ptr.  If not, you could forward
declare the type.

[ananta, 2011/08/26 21:50:17]

Removed the include of the file here and replaced it with a forward declaration.

+
   HINSTANCE locale_dll_handle_;
 };
 
 #endif  // CHROME_FRAME_SIMPLE_RESOURCE_LOADER_H_