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

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

Index: third_party/WebKit/Source/core/input/EventHandler.cpp
diff --git a/third_party/WebKit/Source/core/input/EventHandler.cpp b/third_party/WebKit/Source/core/input/EventHandler.cpp
index 74df242db22b8562c12d392a7a3c1960aec199d0..dcaa114eabe652b1331ba9bcb0825e4110f84ebd 100644
--- a/third_party/WebKit/Source/core/input/EventHandler.cpp
+++ b/third_party/WebKit/Source/core/input/EventHandler.cpp
@@ -2041,6 +2041,10 @@ WebInputEventResult EventHandler::sendContextMenuEvent(const PlatformMouseEvent&
     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);