Make the invisible items in the system menu of a few Indian language versions...

views/controls/menu/native_menu_win.cc

 // Copyright (c) 2009 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 "views/controls/menu/native_menu_win.h"
 
 #include "app/gfx/canvas.h"
 #include "app/gfx/font.h"
 #include "app/l10n_util.h"
 #include "app/l10n_util_win.h"
@@ skipping 284 matching lines @@
 const wchar_t* NativeMenuWin::MenuHostWindow::kMenuHostWindowKey =
     L"__MENU_HOST_WINDOW__";
 
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeMenuWin, public:
 
 NativeMenuWin::NativeMenuWin(Menu2Model* model, HWND system_menu_for)
     : model_(model),
       menu_(NULL),
-      owner_draw_(l10n_util::NeedOverrideDefaultUIFont(NULL, NULL)),
+      owner_draw_(l10n_util::NeedOverrideDefaultUIFont(NULL, NULL) &&
+                  !system_menu_for),

[Finnur, 2009/09/24 21:14:53]

I understand that you don't want the menu to be owner drawn, but when exactly do
you expect |system_menu_for| to be NULL? It is non-NULL when I test this on
Vista with lang=hi and lang=ml, so why not just change this back to:
  owner_draw_(false)
which is how it used to be? Isn't that effectively the same?

       system_menu_for_(system_menu_for),
       first_item_index_(0) {
 }
 
 NativeMenuWin::~NativeMenuWin() {
   STLDeleteContainerPointers(items_.begin(), items_.end());
   DestroyMenu(menu_);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ skipping 217 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 // MenuWrapper, public:
 
 // static
 MenuWrapper* MenuWrapper::CreateWrapper(Menu2* menu) {
   return new NativeMenuWin(menu->model(), NULL);
 }
 
 }  // namespace views