New issue
Advanced search Search tips

Issue 647617 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Getting "Tap time not set!" RuntimeException while using Contextual Search

Project Member Reported by peconn@chromium.org, Sep 16 2016

Issue description

Playing around with the contextual search on trunk, I'm getting crashes due to this error occasionally.

java.lang.RuntimeException: Tap time not set!
   at org.chromium.chrome.browser.contextualsearch.ContextualSearchSelectionController.processSearchAction(ContextualSearchSelectionController.java:443)
   at org.chromium.chrome.browser.contextualsearch.ContextualSearchSelectionController$3.onContextReady(ContextualSearchSelectionController.java:457)
   at org.chromium.chrome.browser.contextualsearch.action.SearchAction.onContextReady(SearchAction.java:130)
   at org.chromium.chrome.browser.contextualsearch.action.SearchAction.notifyContextReady(SearchAction.java:95)
   at org.chromium.chrome.browser.contextualsearch.action.ResolvedSearchAction.onSurroundingTextReady(ResolvedSearchAction.java:50)

I'm not 100% sure on how to trigger it manually, it generally occurs if you tap enough times, I've attached videos of it.
 

Comment 1 by peconn@chromium.org, Sep 16 2016

Cc: peconn@chromium.org
Sorry - accidentally screen recorded the wrong device!
2016-09-16_11-07-02.mp4
13.2 MB View Download
2016-09-16_11-07-46.mp4
4.6 MB View Download

Comment 2 by peconn@chromium.org, Sep 16 2016

Labels: -Pri-1 Pri-2
Actually, after pulling and building trunk again, this seems to have gone away, so I think it's may just be a bug in https://codereview.chromium.org/2211353002/ .

Comment 3 by donnd@chromium.org, Sep 16 2016

Components: UI>Browser>Mobile>TouchToSearch
Labels: OS-Android
Status: Assigned (was: Untriaged)
Thanks for the repro Peter!

Sign in to add a comment