Set ImeListView's scrollable range when updating

ash/common/system/chromeos/ime_menu/ime_menu_tray.cc

 // Copyright 2016 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 "ash/common/system/chromeos/ime_menu/ime_menu_tray.h"
 
 #include "ash/common/accessibility_delegate.h"
 #include "ash/common/ash_constants.h"
 #include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/session/session_state_delegate.h"
@@ skipping 455 matching lines @@
   if (bubble_)
     HideImeMenuBubble();
   else
     ShowImeMenuBubble();
   return true;
 }
 
 void ImeMenuTray::OnIMERefresh() {
   UpdateTrayLabel();
   if (bubble_ && ime_list_view_) {
-    SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
-    IMEInfoList list;
-    delegate->GetAvailableIMEList(&list);
-    IMEPropertyInfoList property_list;
-    delegate->GetCurrentIMEProperties(&property_list);
-    ime_list_view_->Update(list, property_list, false,
-                           ImeListView::SHOW_SINGLE_IME);
+    HideImeMenuBubble();
+    ShowImeMenuBubble();

[tdanderson, 2016/11/30 23:04:15]

I'm not sure that hiding and re-showing the bubble is really the ideal fix for
this bug; wouldn't the menu look like it's "flickering" to the user (when it
hides and re-shows quickly)?

I tried the keyboard shortcut in the IME detailed view in the system menu, and
it seems to work fine. Can you look into how the detailed view handles changes
via keyboard shortcut and do something similar for the opt-in menu?

[Azure Wei, 2016/12/01 00:33:30]

The IME detailed view and opt-in menu shares the same updating logic. And opt-in
menu used to work fine when switching with shortcuts. I suspects this could be
regression related to scroll content of TrayDetailsView. But I didn't find out
where goes wrong.

Anyway, this isn't a good solution to fix. Let me hold this CL and try other
ways.

   }
 }
 
 void ImeMenuTray::OnIMEMenuActivationChanged(bool is_activated) {
   SetVisible(is_activated);
   if (is_activated)
     UpdateTrayLabel();
   else
     HideImeMenuBubble();
 }
@@ skipping 68 matching lines @@
   WmShell::Get()->system_tray_delegate()->GetCurrentIME(&current_ime_);
 
   // Updates the tray label based on the current input method.
   if (current_ime_.third_party)
     label_->SetText(current_ime_.short_name + base::UTF8ToUTF16("*"));
   else
     label_->SetText(current_ime_.short_name);
 }
 
 }  // namespace ash