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

Issue 709089 link

Starred by 2 users

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Pasting into Find in Page textbox strips out extra spaces

Reported by lx45...@gmail.com, Apr 6 2017

Issue description

Chrome Version       : 58.0.3029.54
OS Version: 6.3
URLs (if applicable) :
Other browsers tested:
  Add OK or FAIL after other browsers where you have tested this issue:
     Safari 5: Not tested. (update your template guys)
   Firefox 45: OK
        IE 11: FAIL

What steps will reproduce the problem?
1. Copy the text: test  words
2. Open the find dialog
3. Paste

What is the expected result?
The copied text will be found in the page.

What happens instead of that?
Multiple consecutive spaces are stripped on pasting, and Chrome searches for the wrong phrase, either not finding the intended text or finding the wrong thing.

Please provide any additional information below. Attach a screenshot if
possible.

UserAgentString: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.54 Safari/537.36


 
Labels: Needs-Triage-M58 Needs-Bisect
Cc: krajshree@chromium.org
Labels: Needs-Feedback
Unable to reproduce the issue on Win-7 and Win-10 using chrome reported version #58.0.3029.54 and latest canary #59.0.3064.0.

Attached a screen cast for reference.

Following are the steps followed to reproduce the issue.
------------
1. Opened wikipedia.com in chrome.
2. Copied a text and pasted in the find in page.
3. Observed that copied text was found in the page as expected.

lx45803@ - Could you please verify the screen cast and please let us know if anything missed from our side. Also if possible please provide a sample URL and screen cast of the issue to understand it better.

Thanks...!!
709089.mp4
4.0 MB View Download

Comment 3 by lx45...@gmail.com, Apr 7 2017

@krajshree - The issue only happens when pasting text with multiple consecutive spaces. Well formed text like that on Wikipedia won't demonstrate the issue.

I've attached a screencast showing the issue with the reproduction steps I gave.
2017-04-07_10-27-45.mp4
352 KB View Download
Project Member

Comment 4 by sheriffbot@chromium.org, Apr 7 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "krajshree@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Components: UI>Browser>FindInPage
Labels: -Type-Bug -Pri-3 -Needs-Bisect -Needs-Triage-M58 hasbisect-per-revision M-59 OS-Linux Pri-1 Type-Bug-Regression
Owner: ananta@chromium.org
Status: Assigned (was: Unconfirmed)
lx45803@ - Thanks for clarification...!!

Able to reproduce the issue on Windows 10 and Ubuntu 14.04 using chrome reported version #58.0.3029.54 and latest canary #59.0.3066.0.
Issue is not reproducible on OS-Mac.

Bisect Information:
=====================
Good build: 49.0.2579.0  Revision(362585)
Bad Build : 49.0.2580.0  Revision(362859)

Change Log URL: 
https://chromium.googlesource.com/chromium/src/+log/3b5cac24943ce9e682c806f844a6aec90481774c..ae9bb92fa0ca5ea9991c82a5f5910491d0f9aaf1

From the above change log suspecting below change
Review url: https://codereview.chromium.org/1492633004

ananta@ - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Thanks...!!

Comment 6 by lx45...@gmail.com, Apr 10 2017

Skimming the commit, it definitely looks like that change is the culprit, assuming CollapseWhitespace does what it says on the tin. Is there a better fix for the font issue in the original bug? Although I've never needed it personally, I can imagine cases where you might want to search for whitespace patterns or the like. 
Friendly Ping!
ananta@ could you please look into this issue.

Thanks.

Comment 8 by lx45...@gmail.com, Dec 9 2017

krajshree@, could you reassign this? ananta@ hasn't visited in nearly three months, and this issue is still present in Canary 65.0.3289.2.

Comment 9 Deleted

This issue is still present in the 67 branch.
It prevents from searching code fragments on source pages.
Say, you cannot find this string:
"  -v, --version              print Node.js version\n"
here:
https://github.com/nodejs/node/blob/master/src/node.cc
until you manually add all the extra spaces in the string pasted in the search bar.
Owner: ----
Status: Untriaged (was: Assigned)
Unassigning all bugs for ananta@chromium.org
Labels: Hotlist-DesktopUIValid Hotlist-DesktopUIChecked
** UI Mass triage**

Able to reproduce this bug using Chrome canary 72.0.3616.0. Tagging as a valid bug for expert review.

Sign in to add a comment