Call SelectionController().sendContextMenuEvent() with up-to-date layout tree

third_party/WebKit/Source/core/input/EventHandler.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2012 Digia Plc. and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 2023 matching lines @@
 {
     FrameView* v = m_frame->view();
     if (!v)
         return WebInputEventResult::NotHandled;
 
     // Clear mouse press state to avoid initiating a drag while context menu is up.
     m_mousePressed = false;
     LayoutPoint positionInContents = v->rootFrameToContents(event.position());
     HitTestRequest request(HitTestRequest::Active);
     MouseEventWithHitTestResults mev = m_frame->document()->prepareMouseEvent(request, positionInContents, event);
+    // Since // |Document::prepareMouseEvent()| modifies layout tree for setting

[tkent, 2016/08/29 08:01:21]

Is the second '//' unnecessary?

[yosin_UTC9, 2016/08/29 08:12:10]

Done.

No, removed.

+    // hover element, we need to update layout tree for requirement of
+    // |SelectionController::sendContextMenuEvent()|.
+    m_frame->document()->updateStyleAndLayoutIgnorePendingStylesheets();
 
     selectionController().sendContextMenuEvent(mev, positionInContents);
 
     Node* targetNode = overrideTargetNode ? overrideTargetNode : mev.innerNode();
     return dispatchMouseEvent(EventTypeNames::contextmenu, targetNode, 0, event);
 }
 
 WebInputEventResult EventHandler::sendContextMenuEventForKey(Element* overrideTargetElement)
 {
     FrameView* view = m_frame->view();
@@ skipping 548 matching lines @@
 
 FrameHost* EventHandler::frameHost() const
 {
     if (!m_frame->page())
         return nullptr;
 
     return &m_frame->page()->frameHost();
 }
 
 } // namespace blink