Issue metadata
Sign in to add a comment
|
Chrome PDF Viewer: search sticks when up/down arrow is pressed in search box
Reported by
bsdman...@gmail.com,
Jun 18 2016
|
||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2767.4 Safari/537.36 Steps to reproduce the problem: 1. open any pdf in the native pdf viewer 2. press control+f in order to open search 3. type something to search for in the search box 4. after results are shown, click on the down or up arrow twice, to move to the next result What is the expected behavior? search should move to the next result in the pdf What went wrong? the search freezes and you cannot navigate to next/previous search result Did this work before? N/A Chrome version: 53.0.2767.4 Channel: dev OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version: Shockwave Flash 22.0 r0
,
Jun 18 2016
Most likely due to r398186. Probably affects all OSes.
,
Jun 20 2016
I'll look at this now. This seems worth of Pri-1 to me.
,
Jun 20 2016
,
Jun 22 2016
Issue 621872 has been merged into this issue.
,
Jun 22 2016
I have a fix for this. Just waiting on review. https://codereview.chromium.org/2082733002/
,
Jun 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e39a61d57a81807380a1d6817840d7452760e5f9 commit e39a61d57a81807380a1d6817840d7452760e5f9 Author: paulmeyer <paulmeyer@chromium.org> Date: Wed Jun 22 18:53:08 2016 Fix for PDF find-in-page bug. The problem was that the internal find identifier in PepperPluginInstanceImpl was not being updated when selectFindResult() was called, which resulted in the incorrect find request ID being reported in the find reply. BUG= 621304 Review-Url: https://codereview.chromium.org/2082733002 Cr-Commit-Position: refs/heads/master@{#401374} [modify] https://crrev.com/e39a61d57a81807380a1d6817840d7452760e5f9/content/renderer/pepper/pepper_plugin_instance_impl.cc [modify] https://crrev.com/e39a61d57a81807380a1d6817840d7452760e5f9/content/renderer/pepper/pepper_plugin_instance_impl.h [modify] https://crrev.com/e39a61d57a81807380a1d6817840d7452760e5f9/content/renderer/pepper/pepper_webplugin_impl.cc [modify] https://crrev.com/e39a61d57a81807380a1d6817840d7452760e5f9/content/renderer/pepper/pepper_webplugin_impl.h [modify] https://crrev.com/e39a61d57a81807380a1d6817840d7452760e5f9/content/renderer/render_frame_impl.cc [modify] https://crrev.com/e39a61d57a81807380a1d6817840d7452760e5f9/third_party/WebKit/public/web/WebPlugin.h
,
Jun 22 2016
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by thestig@chromium.org
, Jun 18 2016Labels: M-53
Status: Untriaged (was: Unconfirmed)