New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 784931 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Chrome on android automatically does a search query whenever a word is highlighted.

Project Member Reported by thanhdle@chromium.org, Nov 14 2017

Issue description

Chrome on android automatically does a search query whenever a word is highlighted, which makes the logs flooded with unecessary search query.

Highlight should disabled on the page to prevent is issue, and have some mechanics to copy url information to the clipboard with some gesture.
 
Description: Show this description
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 16 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/be625f5ecdee0700c4ffac2ceab8b5aa15d753ae

commit be625f5ecdee0700c4ffac2ceab8b5aa15d753ae
Author: Thanh Le <thanhdle@chromium.org>
Date: Thu Nov 16 01:12:30 2017

Disable highlighting on chrome://interventions-internals.

Chrome on android automatically does a search query whenever a word is
highlighted, which makes the logs flooded with unecessary search query.

Highlight is disabled on the page to prevent is issue.

Add copy button to url cells in previews log table.

Screenshot:
https://drive.google.com/a/chromium.org/file/d/1MMIBzN5e7pHXUgrCpeTJNw8DCeZYHHe0
https://drive.google.com/a/chromium.org/file/d/1fxGrhosLFEBVK4UVjWSFd-u33ZIHpQIF

Bug:  784931 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I43335c250077929f678f558e5d2b9b7dcb7ee6cb
Reviewed-on: https://chromium-review.googlesource.com/769860
Commit-Queue: Thanh Le <thanhdle@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Ryan Sturm (OOO until 11/27) <ryansturm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516934}
[modify] https://crrev.com/be625f5ecdee0700c4ffac2ceab8b5aa15d753ae/chrome/browser/resources/interventions_internals/index.css
[modify] https://crrev.com/be625f5ecdee0700c4ffac2ceab8b5aa15d753ae/chrome/browser/resources/interventions_internals/index.js
[modify] https://crrev.com/be625f5ecdee0700c4ffac2ceab8b5aa15d753ae/chrome/test/data/webui/interventions_internals_browsertest.js

Status: Fixed (was: Started)

Comment 4 by efoo@chromium.org, Dec 5 2017

Components: Blink>Previews

Comment 5 by efoo@chromium.org, Dec 5 2017

Components: -UI>Browser>Previews

Sign in to add a comment