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

Issue 679307 link

Starred by 2 users
Status: Fixed
Owner:
fmalita@chromium.org
Closed: Jan 2017
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
reed@google.com
vmp...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Selected text appears weird after dragging it

Reported by adha...@etouch.net, Jan 9 2017 
Chrome Version : 57.0.2976.0 (Official Build) 062d7b445d6d0ebf509fd923ef027a0a3c6818b1-refs/heads/master@{#442165} 32/64 bit
OS : Mac(10.12.1, 10.11.6, 10.12), Windows(7,8,10), Linux(14.04 LTS)

What steps will reproduce the problem?
(1)Launch chrome and open NTP.
(2)Type any keyword like "balloon" or "car".
(3)Drag the text and observe.

Actual:Selected text appears weird after dragging it.

Expected:Selected text should appear properly after dragging it.

This is a Regression issue broken in M-57,will soon update other info.

Good build:57.0.2974.0
Bad build:57.0.2975.0

Kindly find below the attached screen shot for reference.
Actual_Expected_text.png
40.3 KB View Download

Comment 1 by msrchandra@chromium.org, Jan 9 2017

Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: fmalita@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 57.0.2974.0 (Revision: 441962).
Bad build: 57.0.2975.0 (Revision: 442120).

You are probably looking for a change made after 442038 (known good), but no later than 442039 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/4aa74f3897170953b1df843a46eef77f8a6a8055..bef18d10d18bd46e0c8dce3e8ece1176cdfec9fa

@fmalita -- Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.

Comment 2 by tkent@chromium.org, Jan 9 2017

Components: -Blink>Forms>Text Blink>DataTransfer Blink>Paint

Comment 3 by fmalita@chromium.org, Jan 10 2017

Cc: reed@google.com
Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 11 2017 

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

commit ae554451211cf78a57a6257abf4943f679fa5298
Author: fmalita <fmalita@chromium.org>
Date: Wed Jan 11 04:56:53 2017

Disable LCD text in drag images

Since we don't know know whether the drag image is going to have an
opaque backdrop, disable LCD text conservatively.

This was previously handled by SkPictureImageGenerator, internally, but
broke in http://crrev.com/2619573002.

BUG= 679307 
R=reed@google.com

Review-Url: https://codereview.chromium.org/2625803002
Cr-Commit-Position: refs/heads/master@{#442804}

[modify] https://crrev.com/ae554451211cf78a57a6257abf4943f679fa5298/third_party/WebKit/Source/core/frame/LocalFrame.cpp

Comment 5 by fmalita@chromium.org, Jan 11 2017

Status: Fixed (was: Started)

Comment 6 by rbasuvula@chromium.org, Jan 12 2017

Cc: rbasuvula@chromium.org
Labels: TE-Verified-M57 TE-Verified-57.0.2979.0
Tested the issue on Windows-10, Ubuntu 14.04 & Mac 10.12.2 using chrome latest M57-57.0.2979.0 by following steps mentioned in the original comment. Observed that Selected text displaying as expected. Hence adding TE-Verified label.
Please find the screen shot for reference.

Thank you!
679307.png
132 KB View Download
►

Sign in to add a comment