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

Issue 621485 link

Starred by 5 users

Issue metadata

Status: Verified
Owner:
Closed: Sep 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression : Two Images are seen in print preview for Edited Image

Project Member Reported by mm00333...@techmahindra.com, Jun 20 2016

Issue description

Version: 53.0.2768.0
OS: Chrome
Platform : 8476.0.0 (Official Build) dev-channel Gnawty,Falco,Spring

What steps will reproduce the problem?
1.Sign into User -> Download image
2.Now Select Image and open it in Gallery -> Click on "Edit" ->Now increase Brightness(Dont click on "Done" Button)
3.Now Click on "Print" Option and observe print preview(Please refer screenshot)

Expected: Original Image (Only one Image)should be seen in print preview
Actual: Instead Two Images are seen in Print Preview


This is Regression Issue as it is working fine in 38.0.2125.44/6158.19.0 beta-channel daisy.
Note: Issue is seen in Latest stable M-51: 51.0.2704.79/8172.47.0 stable-channel Daisy

 
Expected_PrintPreview.png
360 KB View Download
Actual_PrintPreview.png
151 KB View Download
Cc: songsuk@chromium.org pucchakayala@chromium.org
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Gnawty using chrome version 53.0.2768.0/8476.0.0 
Components: -UI>Browser>PrintPreview Platform>Apps>FileManager>ImageLoader
Labels: -Pri-1 Pri-2
Owner: satorux@chromium.org
Status: Assigned (was: Untriaged)
Seems more likely an issue with the gallery than print preview. satorux@ can you find someone to take a look?
Project Member

Comment 3 by sheriffbot@chromium.org, Jul 4 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -M-54 M-55
Owner: fukino@chromium.org
Yes, this seems to be a Gallery issue.
I'll take this.
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 8 2016

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

commit d248394bca77540a9de057cefe28425fff954873
Author: fukino <fukino@chromium.org>
Date: Thu Sep 08 13:43:11 2016

Gallery: Leave edit mode on starting print preview.

If an image is being edited and print button is clicked, we need to leave the
editing mode to prevent unwanted images from populating in the print preview.

BUG= 621485 
TEST=manually confirmed following the repro steps.

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

[modify] https://crrev.com/d248394bca77540a9de057cefe28425fff954873/ui/file_manager/gallery/js/slide_mode.js

Project Member

Comment 6 by bugdroid1@chromium.org, Sep 8 2016

Labels: merge-merged-2854
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d248394bca77540a9de057cefe28425fff954873

commit d248394bca77540a9de057cefe28425fff954873
Author: fukino <fukino@chromium.org>
Date: Thu Sep 08 13:43:11 2016

Gallery: Leave edit mode on starting print preview.

If an image is being edited and print button is clicked, we need to leave the
editing mode to prevent unwanted images from populating in the print preview.

BUG= 621485 
TEST=manually confirmed following the repro steps.

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

[modify] https://crrev.com/d248394bca77540a9de057cefe28425fff954873/ui/file_manager/gallery/js/slide_mode.js

Labels: TE-Verified-55.0.2857.0 TE-Verified-M55
Verified the issue on Gnawty Device using version 55.0.2857.0/8793.0.0  and its working fine
Attaching screenshot for reference.
On55.0.2857.0.png
103 KB View Download

Comment 8 by fukino@chromium.org, Sep 13 2016

Status: Verified (was: Assigned)
I forgot to mark this issue as Fixed.
Thank you for the verification!

Comment 9 by fukino@chromium.org, Oct 28 2016

Cc: krishna...@chromium.org oka@chromium.org
 Issue 639252  has been merged into this issue.

Sign in to add a comment