DescriptionRevert of Save pdf file instead of web page through context menu for embedded pdf file (patchset #8 id:160001 of https://codereview.chromium.org/2862583010/ )
Reason for revert:
PdfPluginContextMenuBrowserTest.EmbeddedPdfHasNoPageItems is flaky on Windows, see:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=browser_tests%20(with%20patch)&tests=PdfPluginContextMenuBrowserTest.EmbeddedPdfHasNoPageItems
Original issue's description:
> Save pdf file instead of web page through context menu for embedded pdf file
>
> When user clicks 'Save as...' in the context menu over an embedded pdf file,
> we should save the pdf file instead of the entire web page. We fix this by showing
> the correct context menu for embedded pdf versus full page pdf: only full page
> pdf's context menu shows page navigation items such as 'Reload',
> but embedded pdf's context menu doesn't show them.
>
> BUG=chromium:591071
>
> Review-Url: https://codereview.chromium.org/2862583010
> Cr-Commit-Position: refs/heads/master@{#473105}
> Committed: https://chromium.googlesource.com/chromium/src/+/fd0e534843ac5926c1b26c7e1e7a7229fcb3aeee
TBR=lazyboy@chromium.org,weili@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:591071
Review-Url: https://codereview.chromium.org/2891923005
Cr-Commit-Position: refs/heads/master@{#473171}
Committed: https://chromium.googlesource.com/chromium/src/+/ab6a896a5431f19e96cbc15ce7920730ac4ace51
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|