1626978 - Find in view source shows incorrect second match

Status: RESOLVED DUPLICATE of bug 1517551

(Core :: Layout: Text and Fonts, defect)

Description

[Mike Kaply [:mkaply]]

If you view the source of this article:

https://www.yahoo.com/huffpost/nazi-zoombombing-jewish-yeshiva-university-213626401.html

And search on "this article origin"

You get two hits. The first one is correct, but the second one matches incorrectly in the document. "this article origin" is in that line, but it's earlier.

I'm guessing the \u values are throwing off the computations?

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The priority flag is not set for this bug.
:Honza, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Attachment: image.png

Thanks for the report Mike!

Yes, I am seeing two matches, but I don's understand what's wrong.
See my attached screenshot, the string this article origin seems to be highlighted just fine.
What am I missing?

Honza

Comment 3

[Mike Kaply [:mkaply]]

Attachment: What I see for the second hit

This is what I see for the second hit

I did a view-source: on the URL bar.

This is on Windows, Firefox 75.0.

Note the unicode values - \u003E - these seem to be throwing things off for me.

Comment 4

[Mike Kaply [:mkaply]]

Just tried nightly and I get the same result.

Comment 5

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Attachment: image.png

I am able to reproduce the problem in Firefox 74/75/76, but not in Nightly.

The selection offset varies in different Firefox versions for me
(perhaps the offset is wrongly calculates due to the unicode chars?)

Honza

Comment 6

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Attachment: image.png

Screenshot for 74

Comment 7

[Alice0775 White]

I can reproduce the issue on Nightly77.0a1 Windows10 if "Wrap Long Lines" is disabled and certain zoom level.

And another scenario, select some text by mouse in the line. and ctrl+C.

• Highlight Background color is shifted a few pixels from the font boundary.
• Selected font color would not change to Highlight Foreground.
• Text other than the selected text is copied to the clipboard.

Comment 8

[Alice0775 White]

Attachment: aaaaa.html

STR

1. Open attached
2. Zoom in or out (e.g. 80%)
3. Find "firefox"
   OR
   Select text around the end of the line.

Comment 9

[Alice0775 White]

Regression window:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=485a03afaa233e311bfb20e15bac9f50158acd00&tochange=d734e6acf7778df7c933d33540203f08b44ff977

Suspect: Bug 1408612

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

The product::component has been changed since the backlog priority was decided, so we're resetting it.
For more information, please visit auto_nag documentation.

Comment 11

[Jonathan Kew [:jfkthame]]

I believe this is another example of bug 1517551.