Refactor: Shelf Widget

chrome/browser/ui/ash/launcher/shell_window_launcher_item_controller.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 "chrome/browser/ui/ash/launcher/shell_window_launcher_item_controller.h"
 
 #include "ash/launcher/launcher_util.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_app_menu_item.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_app_menu_item_v2app.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
@@ skipping 61 matching lines @@
     ShellWindow* shell_window = *iter;
     shell_windows_.erase(iter);
     shell_windows_.push_front(shell_window);
   }
 }
 
 string16 ShellWindowLauncherItemController::GetTitle() {
   return GetAppTitle();
 }
 
-bool ShellWindowLauncherItemController::HasWindow(
-    aura::Window* window) const {
+bool ShellWindowLauncherItemController::HasWindow(aura::Window* window) const {

[Mr4D (OOO till 08-26), 2013/03/02 02:02:12]

Not that I don't think that this is a good change - but the CL is humangous as
it is. Maybe you leave it the way it is?

[Harry McCleave, 2013/03/04 02:47:41]

Done.

   ShellWindowList::const_iterator iter =
       std::find_if(shell_windows_.begin(), shell_windows_.end(),
                    ShellWindowHasWindow(window));
   return iter != shell_windows_.end();
 }
 
 bool ShellWindowLauncherItemController::IsOpen() const {
   return !shell_windows_.empty();
 }
 
@@ skipping 95 matching lines @@
 
 void ShellWindowLauncherItemController::RestoreOrShow(
     ShellWindow* shell_window) {
   if (shell_window->GetBaseWindow()->IsMinimized())
     shell_window->GetBaseWindow()->Restore();
   else
     shell_window->GetBaseWindow()->Show();
   // Always activate windows when shown from the launcher.
   shell_window->GetBaseWindow()->Activate();
 }