Issue metadata
Sign in to add a comment
|
Regression:Page turns blank on rotating pdf multiple times.
Reported by
shruti.j...@etouch.net,
May 30 2018
|
||||||||||||||||||||||
Issue descriptionChrome version: 68.0.3440.7 (Official Build) Revision 9bbd2d0687bd83181f22ac13565501690ee5b4c7-refs/branch-heads/3440@{#16}(64-bit) OS:Mac OS X(10.12.6,10.13.1,10.13.5) Steps to reproduce: 1. Launch chrome, crash the browser using 'chrome://inducebrowsercrashforrealz' 2. After relaunch click on restore button and navigate to google.com and search for 'test pdf'. 3. Open the first link shown in the result. 4. Click on rotate icon multiple times shown in toolbar and observe. Actual Result: Page turns blank on rotating pdf multiple times. Expected Result: Pdf should appear properly even after multiple rotation. This is regression issue broken in ‘M-68’ and providing per-revision bisect info below: Good Build:68.0.3439.0(Revision:561389) Bad Build:68.0.3440.0(Revision:561733) You are probably looking for a change made after 561614 (known good), but no later than 561615 (first known bad). CHANGE-LOG URL: The script might not always return single CL as suspect as some perf builds might get missing due to failure. https://chromium.googlesource.com/chromium/src/+log/e257d3b0b2ce380369dc8c0038fa2304cf251cca..4255c18dd8d9b3411e6cdbaa92782c4cf2a97581 Suspect:https://chromium.googlesource.com/chromium/src/+/4255c18dd8d9b3411e6cdbaa92782c4cf2a97581 @Fady Samuel: Could you please help to reassign if your change is not the cause for this change. Note: 1.Issue is only reproducible on MAC(10.12.6,10.13.1,10.13.5) OS. 2.Issue is also reproducible on latest canary M69# 69.0.3444.0. Kindly review the attached screen-cast . Thank You!
,
May 30 2018
marking as RBS, please change if required.
,
Jun 11 2018
I cannot repro this anymore... it somehow got fixed although I think it might still be in M68. I'll investigate further later today.
,
Jun 21 2018
Friendly ping to get an update on this issue as it is marked as RBS. Thanks..!
,
Jul 3
Bulk update: M68 stable cut is scheduled for July 19th. This issue is marked as RB-Stable, so please take a look at it before. Thanks!
,
Jul 16
any updates on this fsamuel@
,
Jul 16
Is this still occurring in M68?
,
Jul 16
Ahh I lost track of this bug...trying it out now on M68.
,
Jul 17
Looks like this is the fix: https://chromium-review.googlesource.com/c/chromium/src/+/1109448 This landed in M69 but was not merged in M68. Shall I merge back?
,
Jul 17
+ccameron
,
Jul 19
How safe is this fix overall?
,
Jul 19
I would say quite safe.
,
Jul 23
Gentle ping to get an update on this issue as it is marked as RBS for M68 & M68 stable release is coming this week. Thanks..!
,
Jul 25
,
Jul 27
Let's punt this to M69.
,
Aug 1
Gentle ping to get an update on this issue as it is marked as RBS. Thanks..!
,
Aug 7
Just to update: Still seeing the same issue on latest stable-68.0.3440.84 as per C#0. fsamuel@, Please take a look and update as it is marked as RBS. Thanks..!
,
Aug 7
M69 Stable promotion is coming VERY soon. Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge into the release branch ASAP. Thank you.
,
Aug 8
Per comment #9, this change is already landed in M69. Pls confirm and mark bug as fixed if nothing else is pending.
,
Aug 8
This is already in M69 so there's nothing to do here.
,
Aug 8
[Auto-generated comment by a script] We noticed that this issue is targeted for M-69; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-69 label, otherwise remove Merge-TBD label. Thanks.
,
Aug 9
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by fsam...@chromium.org
, May 30 2018