Do not use ContextMenuParams (after mutating it) to decide where to show platform menu.

Instead, keep ContextMenuParams as is and calculate the position where
    it is required (RVCMViews/RVCMMac)

Mutating ContextMenuParams has consequences, if the param is passed
   back to blink, then things break b/c it expects the unmodified params
   (specifically the coordinates). Rotating PDF is one
   example where the ContextMenuParams is passed back to blink as
   we call RenderViewContextMenu::RunPluginActionAt().
   In theory MediaPlayerActionAt() should also have the breakage.

BUG=461667
Test=Easier to test with PDF viewer,
Open a page in tab where a PDF is embedded.
e.g.
http://acroeng.adobe.com/Test_Files/browser_tests/embedded/embed2.html
Right click on PDF that are embedded in html page then in context menu
choose rotateclockwise.
This should rotate the PDF as expected.

Review URL: https://codereview.chromium.org/1115013003

Cr-Commit-Position: refs/heads/master@{#327860}

0 files changed, 0 insertions, 0 deletions