Vertical miniature of images are covered by text
Reported by
redakcja...@gmail.com,
Sep 19 2016
|
|||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.116 Safari/537.36 Example URL: http://laracroft.pl/0816.htm#news2138 Steps to reproduce the problem: 1. Open http://laracroft.pl/0816.htm#news2138 or http://laracroft.pl/0313.htm#news1590 > Next to the news there is cover of comic book. Sometimes the image displays correctly, and sometimes the image is covered by text. I've noticed this problem yesterday. This is 14-year old website and the problem appears only on Chrome. 2. If you can see the comic cover correctly, please refresh the browser, sometimes randomly the text of the news is covering the image. 3. Refresh the browser of change the scale of the view, e.g. from 100% to 90% or 110%. What is the expected behavior? The image should be next to the text, not be covered by text. What went wrong? The problem is on every subsite with news, mainly the problem is with vertical small images. Does it occur on multiple sites: Yes Is it a problem with a plugin? No Did this work before? Yes It worked for 14 years, the website was founded in 2002. The problem is only with Chrome, I didn't noticed it before yesterday. Does this work in other browsers? Yes Chrome version: 53.0.2785.116 Channel: stable OS Version: 10.0 Flash Version: Shockwave Flash 23.0 r0
,
Sep 19 2016
I know, I'm talking about problem in Chrome, not Firefox.
,
Sep 19 2016
Please find the bisect range of the bug : You are probably looking for a change made after 400247 (known good), but no later than 400248 (first known bad). CHANGELOG URL: The script might not always return single CL as suspectas some perf builds might get missing due to failure. https://chromium.googlesource.com/chromium/src/+log/098c009df7a4ddc5c23d4d3c9dccf5eff1f24c98..5db9460f869b506fd00f88ce74d60bd23d5f761a
,
Sep 20 2016
,
Sep 24 2016
So, any solution? ;)
,
Oct 4 2016
,
Oct 12 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7937b7b2c7272149da614489b0147efed3ac9ace commit 7937b7b2c7272149da614489b0147efed3ac9ace Author: robhogan <robhogan@gmail.com> Date: Wed Oct 12 18:59:27 2016 Dirty lineboxes when adding a float to an empty inline BUG= 648342 Review-Url: https://codereview.chromium.org/2411773003 Cr-Commit-Position: refs/heads/master@{#424808} [add] https://crrev.com/7937b7b2c7272149da614489b0147efed3ac9ace/third_party/WebKit/LayoutTests/fast/block/float/float-should-dirty-line-when-adjacent-to-line-breaks-expected.html [add] https://crrev.com/7937b7b2c7272149da614489b0147efed3ac9ace/third_party/WebKit/LayoutTests/fast/block/float/float-should-dirty-line-when-adjacent-to-line-breaks.html [modify] https://crrev.com/7937b7b2c7272149da614489b0147efed3ac9ace/third_party/WebKit/Source/core/layout/line/LineBoxList.cpp
,
Oct 15 2016
,
Oct 17 2016
Fixed? I can still see this error :( http://laracroft.pl/0816.htm#news2138 the image is covered by text.
,
Oct 17 2016
The fix is in version 56.0.2889.0 or later -- which one are you using?
,
Oct 17 2016
Ok, your point. I think now it's ok.
,
Oct 19 2016
I have version 54.0.2840.59 m (64-bit) and I can't see the option to update to 56.0.2889.0. Chrome says that it has the newest verion on my PC. So how to get 56.0.2889.0 ? On 54.0.2840.59 I still can see the problem :/
,
Oct 19 2016
,
Oct 19 2016
[Automated comment] Request affecting a post-stable build (M54), manual review required.
,
Oct 19 2016
Your change meets the bar and is auto-approved for M55 (branch: 2883)
,
Oct 19 2016
M54 is already in Stable (deployed to 5% of Win users). We are taking only CRITICAL merges. Can it wait for M55?
,
Oct 20 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/08135c796776ce294caba211e7b723d0a0dab8d0 commit 08135c796776ce294caba211e7b723d0a0dab8d0 Author: robhogan <robhogan@gmail.com> Date: Thu Oct 20 00:26:13 2016 Dirty lineboxes when adding a float to an empty inline BUG= 648342 NOTRY=true NOPRESUBMIT=true TBR=eae@chromium.org Review-Url: https://codereview.chromium.org/2411773003 Cr-Commit-Position: refs/heads/master@{#424808} (cherry picked from commit 7937b7b2c7272149da614489b0147efed3ac9ace) Review-Url: https://chromiumcodereview.appspot.com/2438663002 Cr-Commit-Position: refs/branch-heads/2883@{#202} Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768} [add] https://crrev.com/08135c796776ce294caba211e7b723d0a0dab8d0/third_party/WebKit/LayoutTests/fast/block/float/float-should-dirty-line-when-adjacent-to-line-breaks-expected.html [add] https://crrev.com/08135c796776ce294caba211e7b723d0a0dab8d0/third_party/WebKit/LayoutTests/fast/block/float/float-should-dirty-line-when-adjacent-to-line-breaks.html [modify] https://crrev.com/08135c796776ce294caba211e7b723d0a0dab8d0/third_party/WebKit/Source/core/layout/line/LineBoxList.cpp
,
Oct 26 2016
Tested the issue on Windows-10, Ubuntu 14.04 and Mac OS 10.12 using chrome latest Beta M55-55.0.2883.28 by following steps mentioned in the original comment. Observed that images are displaying as expected. Hence adding TE-Verified label. Note: Able to reproduce the issue in MAC also and fixed in Beta M55-55.0.2883.28 Thank you!
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/08135c796776ce294caba211e7b723d0a0dab8d0 commit 08135c796776ce294caba211e7b723d0a0dab8d0 Author: robhogan <robhogan@gmail.com> Date: Thu Oct 20 00:26:13 2016 Dirty lineboxes when adding a float to an empty inline BUG= 648342 NOTRY=true NOPRESUBMIT=true TBR=eae@chromium.org Review-Url: https://codereview.chromium.org/2411773003 Cr-Commit-Position: refs/heads/master@{#424808} (cherry picked from commit 7937b7b2c7272149da614489b0147efed3ac9ace) Review-Url: https://chromiumcodereview.appspot.com/2438663002 Cr-Commit-Position: refs/branch-heads/2883@{#202} Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768} [add] https://crrev.com/08135c796776ce294caba211e7b723d0a0dab8d0/third_party/WebKit/LayoutTests/fast/block/float/float-should-dirty-line-when-adjacent-to-line-breaks-expected.html [add] https://crrev.com/08135c796776ce294caba211e7b723d0a0dab8d0/third_party/WebKit/LayoutTests/fast/block/float/float-should-dirty-line-when-adjacent-to-line-breaks.html [modify] https://crrev.com/08135c796776ce294caba211e7b723d0a0dab8d0/third_party/WebKit/Source/core/layout/line/LineBoxList.cpp
,
Nov 1 2016
This is not merged to M54, bugdroid updated with wrong label.
,
Nov 11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/43c0e7342917bc85ee65fe05e7b4e39e6e485362 commit 43c0e7342917bc85ee65fe05e7b4e39e6e485362 Author: robhogan <robhogan@gmail.com> Date: Fri Nov 11 08:01:08 2016 Dirty lineboxes when adding floats to a nested empty inline A follow-on from https://codereview.chromium.org/2411773003 BUG= 648342 Review-Url: https://codereview.chromium.org/2486423003 Cr-Commit-Position: refs/heads/master@{#431521} [add] https://crrev.com/43c0e7342917bc85ee65fe05e7b4e39e6e485362/third_party/WebKit/LayoutTests/fast/block/float/float-should-dirty-line-when-adjacent-to-line-breaks-2-expected.html [add] https://crrev.com/43c0e7342917bc85ee65fe05e7b4e39e6e485362/third_party/WebKit/LayoutTests/fast/block/float/float-should-dirty-line-when-adjacent-to-line-breaks-2.html [modify] https://crrev.com/43c0e7342917bc85ee65fe05e7b4e39e6e485362/third_party/WebKit/Source/core/layout/line/LineBoxList.cpp |
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by dtapu...@chromium.org
, Sep 19 2016