Slight code change to make some global variables const.

chrome/views/window.cc

 // Copyright (c) 2006-2008 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/views/window.h"
 
 #include <shellapi.h>
 
 #include "base/win_util.h"
 #include "chrome/app/chrome_dll_resource.h"
@@ skipping 215 matching lines @@
   // Update the native frame's text. We do this regardless of whether or not
   // the native frame is being used, since this also updates the taskbar, etc.
   std::wstring window_title = window_delegate_->GetWindowTitle();
   std::wstring localized_text;
   if (l10n_util::AdjustStringForLocaleDirection(window_title, &localized_text))
     window_title.assign(localized_text);
   SetWindowText(GetHWND(), window_title.c_str());
 }
 
 void Window::UpdateWindowIcon() {
-  // If the non-client view is rendering its own icon, we need to tell it to repaint.
+  // If the non-client view is rendering its own icon, we need to tell it to
+  // repaint.
   non_client_view_->SchedulePaint();
 
   // Update the native frame's icon. We do this regardless of whether or not
   // the native frame is being used, since this also updates the taskbar, etc.
   SkBitmap icon = window_delegate_->GetWindowIcon();
   if (!icon.isNull()) {
     HICON windows_icon = IconUtil::CreateHICONFromSkBitmap(icon);
     // We need to make sure to destroy the previous icon, otherwise we'll leak
     // these GDI objects until we crash!
     HICON old_icon = reinterpret_cast<HICON>(
@@ skipping 443 matching lines @@
 }
 
 void Window::OnNCLButtonDown(UINT ht_component, const CPoint& point) {
   // When we're using a native frame, window controls work without us
   // interfering.
   if (!non_client_view_->UseNativeFrame()) {
     switch (ht_component) {
       case HTCLOSE:
       case HTMINBUTTON:
       case HTMAXBUTTON: {
-        // When the mouse is pressed down in these specific non-client areas, we
-        // need to tell the RootView to send the mouse pressed event (which sets
-        // capture, allowing subsequent WM_LBUTTONUP (note, _not_ WM_NCLBUTTONUP)
-        // to fire so that the appropriate WM_SYSCOMMAND can be sent by the
-        // applicable button's ButtonListener. We _have_ to do this this way
-        // rather than letting Windows just send the syscommand itself (as would
-        // happen if we never did this dance) because for some insane reason
-        // DefWindowProc for WM_NCLBUTTONDOWN also renders the pressed window
-        // control button appearance, in the Windows classic style, over our
-        // view! Ick! By handling this message we prevent Windows from doing this
-        // undesirable thing, but that means we need to roll the sys-command
-        // handling ourselves.
+        // When the mouse is pressed down in these specific non-client areas,
+        // we need to tell the RootView to send the mouse pressed event (which
+        // sets capture, allowing subsequent WM_LBUTTONUP (note, _not_
+        // WM_NCLBUTTONUP) to fire so that the appropriate WM_SYSCOMMAND can be
+        // sent by the applicable button's ButtonListener. We _have_ to do this
+        // way rather than letting Windows just send the syscommand itself (as
+        // would happen if we never did this dance) because for some insane
+        // reason DefWindowProc for WM_NCLBUTTONDOWN also renders the pressed
+        // window control button appearance, in the Windows classic style, over
+        // our view! Ick! By handling this message we prevent Windows from
+        // doing this undesirable thing, but that means we need to roll the
+        // sys-command handling ourselves.
         ProcessNCMousePress(point, MK_LBUTTON);
         return;
       }
     }
   }
 
   // TODO(beng): figure out why we need to run the system menu manually
   //             ourselves. This is wrong and causes many subtle bugs.
   //             From my initial research, it looks like DefWindowProc tries
   //             to run it but fails before sending the initial WM_MENUSELECT
@@ skipping 429 matching lines @@
     resize_cursors_[RC_VERTICAL] = LoadCursor(NULL, IDC_SIZENS);
     resize_cursors_[RC_HORIZONTAL] = LoadCursor(NULL, IDC_SIZEWE);
     resize_cursors_[RC_NESW] = LoadCursor(NULL, IDC_SIZENESW);
     resize_cursors_[RC_NWSE] = LoadCursor(NULL, IDC_SIZENWSE);
     initialized = true;
   }
 }
 
 }  // namespace views