Disambiguate OnMouseCaptureLost from OnMouseReleased, etc.

views/controls/link.cc

-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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/link.h"
 
 #if defined(OS_LINUX)
 #include <gdk/gdk.h>
 #endif
 
 #include "base/logging.h"
@@ skipping 78 matching lines @@
 }
 
 void Link::SetController(LinkController* controller) {
   controller_ = controller;
 }
 
 const LinkController* Link::GetController() {
   return controller_;
 }
 
-bool Link::OnMousePressed(const MouseEvent& e) {
-  if (!enabled_ || (!e.IsLeftMouseButton() && !e.IsMiddleMouseButton()))
+bool Link::OnMousePressed(const MouseEvent& event) {
+  if (!enabled_ || (!event.IsLeftMouseButton() && !event.IsMiddleMouseButton()))
     return false;
   SetHighlighted(true);
   return true;
 }
 
-bool Link::OnMouseDragged(const MouseEvent& e) {
+bool Link::OnMouseDragged(const MouseEvent& event) {
   SetHighlighted(enabled_ &&
-                 (e.IsLeftMouseButton() || e.IsMiddleMouseButton()) &&
-                 HitTest(e.location()));
+                 (event.IsLeftMouseButton() || event.IsMiddleMouseButton()) &&
+                 HitTest(event.location()));
   return true;
 }
 
-void Link::OnMouseReleased(const MouseEvent& e, bool canceled) {
+void Link::OnMouseReleased(const MouseEvent& event) {
   // Change the highlight first just in case this instance is deleted
   // while calling the controller
   SetHighlighted(false);
-  if (enabled_ && !canceled &&
-      (e.IsLeftMouseButton() || e.IsMiddleMouseButton()) &&
-      HitTest(e.location())) {
+  if (enabled_ && (event.IsLeftMouseButton() || event.IsMiddleMouseButton()) &&
+      HitTest(event.location())) {
     // Focus the link on click.
     RequestFocus();
 
     if (controller_)
-      controller_->LinkActivated(this, e.flags());
+      controller_->LinkActivated(this, event.flags());
   }
 }
 
-bool Link::OnKeyPressed(const KeyEvent& e) {
-  bool activate = ((e.key_code() == ui::VKEY_SPACE) ||
-                   (e.key_code() == ui::VKEY_RETURN));
+void Link::OnMouseCaptureLost() {
+  SetHighlighted(false);
+}
+
+bool Link::OnKeyPressed(const KeyEvent& event) {
+  bool activate = ((event.key_code() == ui::VKEY_SPACE) ||
+                   (event.key_code() == ui::VKEY_RETURN));
   if (!activate)
     return false;
 
   SetHighlighted(false);
 
   // Focus the link on key pressed.
   RequestFocus();
 
   if (controller_)
-    controller_->LinkActivated(this, e.flags());
+    controller_->LinkActivated(this, event.flags());
 
   return true;
 }
 
-bool Link::SkipDefaultKeyEventProcessing(const KeyEvent& e) {
+bool Link::SkipDefaultKeyEventProcessing(const KeyEvent& event) {
   // Make sure we don't process space or enter as accelerators.
-  return (e.key_code() == ui::VKEY_SPACE) ||
-      (e.key_code() == ui::VKEY_RETURN);
+  return (event.key_code() == ui::VKEY_SPACE) ||
+      (event.key_code() == ui::VKEY_RETURN);
 }
 
 void Link::GetAccessibleState(ui::AccessibleViewState* state) {
   Label::GetAccessibleState(state);
   state->role = ui::AccessibilityTypes::ROLE_LINK;
 }
 
 void Link::SetFont(const gfx::Font& font) {
   Label::SetFont(font);
   ValidateStyle();
@@ skipping 62 matching lines @@
   } else {
     if (font().GetStyle() & gfx::Font::UNDERLINED) {
       Label::SetFont(
           font().DeriveFont(0, font().GetStyle() & ~gfx::Font::UNDERLINED));
     }
     Label::SetColor(disabled_color_);
   }
 }
 
 }  // namespace views