Merge 76557 - 2011-01-24 Peter Kasting <pkasting@google.com>...

Source/WebCore/html/HTMLAnchorElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Simon Hausmann <hausmann@kde.org>
  * Copyright (C) 2003, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 523 matching lines @@
     return event->type() == eventNames().keydownEvent && event->isKeyboardEvent() && static_cast<KeyboardEvent*>(event)->keyIdentifier() == "Enter";
 }
 
 bool isMiddleMouseButtonEvent(Event* event)
 {
     return event->isMouseEvent() && static_cast<MouseEvent*>(event)->button() == MiddleButton;
 }
 
 bool isLinkClick(Event* event)
 {
-    return event->type() == eventNames().clickEvent || (event->type() == eventNames().mouseupEvent && isMiddleMouseButtonEvent(event));
+    return event->type() == eventNames().clickEvent && (!event->isMouseEvent() || static_cast<MouseEvent*>(event)->button() != RightButton);
 }
 
 void handleLinkClick(Event* event, Document* document, const String& url, const String& target, bool hideReferrer)
 {
     event->setDefaultHandled();
 
     Frame* frame = document->frame();
     if (!frame)
         return;
     // FIXME: This seems wrong.  Why are we manufactuing a user gesture?
     UserGestureIndicator indicator(DefinitelyProcessingUserGesture);
     frame->loader()->urlSelected(document->completeURL(url), target, event, false, false, hideReferrer ? NoReferrer : SendReferrer);
 }
 
 }