Use Windows Display Language

ci/licenses_golden/licenses_flutter

@@ -3230,8 +3230,6 @@ ORIGIN: ../../../flutter/shell/platform/windows/windows_lifecycle_manager.cc + .
 ORIGIN: ../../../flutter/shell/platform/windows/windows_lifecycle_manager.h + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/shell/platform/windows/windows_proc_table.cc + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/shell/platform/windows/windows_proc_table.h + ../../../flutter/LICENSE
-ORIGIN: ../../../flutter/shell/platform/windows/windows_registry.cc + ../../../flutter/LICENSE
-ORIGIN: ../../../flutter/shell/platform/windows/windows_registry.h + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/shell/platform/windows/windowsx_shim.h + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/shell/profiling/sampling_profiler.cc + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/shell/profiling/sampling_profiler.h + ../../../flutter/LICENSE
@@ -5928,8 +5926,6 @@ FILE: ../../../flutter/shell/platform/windows/windows_lifecycle_manager.cc
 FILE: ../../../flutter/shell/platform/windows/windows_lifecycle_manager.h
 FILE: ../../../flutter/shell/platform/windows/windows_proc_table.cc
 FILE: ../../../flutter/shell/platform/windows/windows_proc_table.h
-FILE: ../../../flutter/shell/platform/windows/windows_registry.cc
-FILE: ../../../flutter/shell/platform/windows/windows_registry.h
 FILE: ../../../flutter/shell/platform/windows/windowsx_shim.h
 FILE: ../../../flutter/shell/profiling/sampling_profiler.cc
 FILE: ../../../flutter/shell/profiling/sampling_profiler.h

shell/platform/windows/BUILD.gn

@@ -107,8 +107,6 @@ source_set("flutter_windows_source") {
     "windows_lifecycle_manager.h",
     "windows_proc_table.cc",
     "windows_proc_table.h",
-    "windows_registry.cc",
-    "windows_registry.h",
     "windowsx_shim.h",
   ]
 

shell/platform/windows/flutter_windows_engine.cc

@@ -156,12 +156,9 @@ FlutterLocale CovertToFlutterLocale(const LanguageInfo& info) {
 
 }  // namespace
 
-FlutterWindowsEngine::FlutterWindowsEngine(
-    const FlutterProjectBundle& project,
-    std::unique_ptr<WindowsRegistry> registry)
+FlutterWindowsEngine::FlutterWindowsEngine(const FlutterProjectBundle& project)
     : project_(std::make_unique<FlutterProjectBundle>(project)),
       aot_data_(nullptr, nullptr),
-      windows_registry_(std::move(registry)),
       lifecycle_manager_(std::make_unique<WindowsLifecycleManager>(this)) {
   embedder_api_.struct_size = sizeof(FlutterEngineProcTable);
   FlutterEngineGetProcAddresses(&embedder_api_);
@@ -571,8 +568,7 @@ void FlutterWindowsEngine::SetLifecycleState(flutter::AppLifecycleState state) {
 }
 
 void FlutterWindowsEngine::SendSystemLocales() {
-  std::vector<LanguageInfo> languages =
-      GetPreferredLanguageInfo(*windows_registry_);
+  std::vector<LanguageInfo> languages = GetPreferredLanguageInfo();
   std::vector<FlutterLocale> flutter_locales;
   flutter_locales.reserve(languages.size());
   for (const auto& info : languages) {

shell/platform/windows/flutter_windows_engine.h

@@ -37,7 +37,6 @@
 #include "flutter/shell/platform/windows/window_proc_delegate_manager.h"
 #include "flutter/shell/platform/windows/window_state.h"
 #include "flutter/shell/platform/windows/windows_lifecycle_manager.h"
-#include "flutter/shell/platform/windows/windows_registry.h"
 #include "third_party/rapidjson/include/rapidjson/document.h"
 
 namespace flutter {
@@ -77,13 +76,8 @@ static void WindowsPlatformThreadPrioritySetter(
 // run in headless mode.
 class FlutterWindowsEngine {
  public:
-  // Creates a new Flutter engine with an injectible windows registry.
-  FlutterWindowsEngine(const FlutterProjectBundle& project,
-                       std::unique_ptr<WindowsRegistry> windows_registry);
-
   // Creates a new Flutter engine object configured to run |project|.
-  explicit FlutterWindowsEngine(const FlutterProjectBundle& project)
-      : FlutterWindowsEngine(project, std::make_unique<WindowsRegistry>()) {}
+  explicit FlutterWindowsEngine(const FlutterProjectBundle& project);
 
   virtual ~FlutterWindowsEngine();
 
@@ -405,9 +399,6 @@ class FlutterWindowsEngine {
   // The on frame drawn callback.
   fml::closure next_frame_callback_;
 
-  // Wrapper providing Windows registry access.
-  std::unique_ptr<WindowsRegistry> windows_registry_;
-
   // Handler for top level window messages.
   std::unique_ptr<WindowsLifecycleManager> lifecycle_manager_;
 

shell/platform/windows/system_utils.cc

@@ -12,9 +12,8 @@
 
 namespace flutter {
 
-std::vector<LanguageInfo> GetPreferredLanguageInfo(
-    const WindowsRegistry& registry) {
-  std::vector<std::wstring> languages = GetPreferredLanguages(registry);
+std::vector<LanguageInfo> GetPreferredLanguageInfo() {
+  std::vector<std::wstring> languages = GetPreferredLanguages();
   std::vector<LanguageInfo> language_info;
   language_info.reserve(languages.size());
 
@@ -24,18 +23,6 @@ std::vector<LanguageInfo> GetPreferredLanguageInfo(
   return language_info;
 }
 
-std::wstring GetPreferredLanguagesFromRegistry(const WindowsRegistry& registry,
-                                               ULONG buffer_size) {
-  std::wstring buffer(buffer_size, '\0');
-  if (registry.GetRegistryValue(HKEY_CURRENT_USER, kGetPreferredLanguageRegKey,
-                                kGetPreferredLanguageRegValue,
-                                RRF_RT_REG_MULTI_SZ, NULL, buffer.data(),
-                                &buffer_size) != ERROR_SUCCESS) {
-    return std::wstring();
-  }
-  return buffer;
-}
-
 std::wstring GetPreferredLanguagesFromMUI() {
   ULONG buffer_size;
   ULONG count = 0;
@@ -51,36 +38,11 @@ std::wstring GetPreferredLanguagesFromMUI() {
   return buffer;
 }
 
-std::vector<std::wstring> GetPreferredLanguages(
-    const WindowsRegistry& registry) {
+std::vector<std::wstring> GetPreferredLanguages() {
   std::vector<std::wstring> languages;
-  BOOL languages_from_registry = TRUE;
-  ULONG buffer_size = 0;
-  ULONG count = 0;
-  DWORD flags = MUI_LANGUAGE_NAME | MUI_UI_FALLBACK;
-
-  // Determine where languages are defined and get buffer length
-  if (registry.GetRegistryValue(HKEY_CURRENT_USER, kGetPreferredLanguageRegKey,
-                                kGetPreferredLanguageRegValue,
-                                RRF_RT_REG_MULTI_SZ, NULL, NULL,
-                                &buffer_size) != ERROR_SUCCESS) {
-    languages_from_registry = FALSE;
-  }
-
-  // Multi-string must be at least 3-long if non-empty,
-  // as a multi-string is terminated with 2 nulls.
-  //
-  // See:
-  // https://learn.microsoft.com/windows/win32/sysinfo/registry-value-types
-  if (languages_from_registry && buffer_size < 3) {
-    languages_from_registry = FALSE;
-  }
 
   // Initialize the buffer
-  std::wstring buffer =
-      languages_from_registry
-          ? GetPreferredLanguagesFromRegistry(registry, buffer_size)
-          : GetPreferredLanguagesFromMUI();
+  std::wstring buffer = GetPreferredLanguagesFromMUI();
 
   // Extract the individual languages from the buffer.
   size_t start = 0;

shell/platform/windows/system_utils.h

@@ -10,8 +10,6 @@
 #include <string>
 #include <vector>
 
-#include "flutter/shell/platform/windows/windows_registry.h"
-
 namespace flutter {
 
 // Registry key for user-preferred languages.
@@ -28,21 +26,15 @@ struct LanguageInfo {
 
 // Returns the list of user-preferred languages, in preference order,
 // parsed into LanguageInfo structures.
-std::vector<LanguageInfo> GetPreferredLanguageInfo(
-    const WindowsRegistry& registry);
-
-// Retrieve the preferred languages from the registry.
-std::wstring GetPreferredLanguagesFromRegistry(const WindowsRegistry& registry,
-                                               ULONG buffer_size);
+std::vector<LanguageInfo> GetPreferredLanguageInfo();
 
 // Retrieve the preferred languages from the MUI API.
 std::wstring GetPreferredLanguagesFromMUI();
 
 // Returns the list of user-preferred languages, in preference order.
 // The language names are as described at:
 // https://docs.microsoft.com/en-us/windows/win32/intl/language-names
-std::vector<std::wstring> GetPreferredLanguages(
-    const WindowsRegistry& registry);
+std::vector<std::wstring> GetPreferredLanguages();
 
 // Parses a Windows language name into its components.
 LanguageInfo ParseLanguageName(std::wstring language_name);

shell/platform/windows/system_utils_unittests.cc

@@ -12,66 +12,35 @@
 namespace flutter {
 namespace testing {
 
-class MockWindowsRegistry : public WindowsRegistry {
- public:
-  MockWindowsRegistry() = default;
-  virtual ~MockWindowsRegistry() = default;
-
-  virtual LSTATUS GetRegistryValue(HKEY hkey,
-                                   LPCWSTR key,
-                                   LPCWSTR value,
-                                   DWORD flags,
-                                   LPDWORD type,
-                                   PVOID data,
-                                   LPDWORD data_size) const {
-    using namespace std::string_literals;
-    static const std::wstring locales =
-        L"en-US\0zh-Hans-CN\0ja\0zh-Hant-TW\0he\0\0"s;
-    static DWORD locales_len = locales.size() * sizeof(wchar_t);
-    if (data != nullptr) {
-      if (*data_size < locales_len) {
-        return ERROR_MORE_DATA;
-      }
-      std::memcpy(data, locales.data(), locales_len);
-      *data_size = locales_len;
-    } else if (data_size != NULL) {
-      *data_size = locales_len;
-    }
-    return ERROR_SUCCESS;
-  }
-
- private:
-  FML_DISALLOW_COPY_AND_ASSIGN(MockWindowsRegistry);
-};
-
 TEST(SystemUtils, GetPreferredLanguageInfo) {
-  WindowsRegistry registry;
-  std::vector<LanguageInfo> languages = GetPreferredLanguageInfo(registry);
+  std::vector<LanguageInfo> languages = GetPreferredLanguageInfo();
   // There should be at least one language.
   ASSERT_GE(languages.size(), 1);
   // The info should have a valid languge.
   EXPECT_GE(languages[0].language.size(), 2);
 }
 
 TEST(SystemUtils, GetPreferredLanguages) {
-  WindowsRegistry registry;
-  std::vector<std::wstring> languages = GetPreferredLanguages(registry);
+  std::vector<std::wstring> languages = GetPreferredLanguages();
   // There should be at least one language.
   ASSERT_GE(languages.size(), 1);
   // The language should be non-empty.
   EXPECT_FALSE(languages[0].empty());
   // There should not be a trailing null from the parsing step.
   EXPECT_EQ(languages[0].size(), wcslen(languages[0].c_str()));
 
-  // Test mock results
-  MockWindowsRegistry mock_registry;
-  languages = GetPreferredLanguages(mock_registry);
-  ASSERT_EQ(languages.size(), 5);
-  ASSERT_EQ(languages[0], std::wstring(L"en-US"));
-  ASSERT_EQ(languages[1], std::wstring(L"zh-Hans-CN"));
-  ASSERT_EQ(languages[2], std::wstring(L"ja"));
-  ASSERT_EQ(languages[3], std::wstring(L"zh-Hant-TW"));
-  ASSERT_EQ(languages[4], std::wstring(L"he"));
+  // Get actual system language.
+  std::wstring wdl_msz = GetPreferredLanguagesFromMUI();
+  auto wdl_itr = wdl_msz.begin();
+  for (auto languages_itr = languages.begin(); languages_itr != languages.end();
+       languages_itr++) {
+    for (auto lang_itr = languages_itr->begin();
+         lang_itr != languages_itr->end(); lang_itr++) {
+      EXPECT_EQ(*lang_itr, *wdl_itr++);
+    }
+    EXPECT_EQ(*wdl_itr++, 0);
+  }
+  EXPECT_EQ(*wdl_itr, 0);
 }
 
 TEST(SystemUtils, ParseLanguageNameGeneric) {