New issue
Advanced search Search tips

Issue 732550 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Cleanup pageCountEstimationDone in PrintingControllerImpl.java

Project Member Reported by ctzsm@chromium.org, Jun 12 2017

Issue description

A followup of https://chromium-review.googlesource.com/521205, since we removed page count setting in onLayout, method pageCountEstimationDone doesn't make sense now. However, pageCountEstimationDone is still needed to set necessary info.

We need to clean the method as long as other small improvements mentioned in the CL discussion.
 

Comment 1 by ctzsm@chromium.org, Jun 12 2017

Summary: Cleanup pageCountEstimationDone in PrintingControllerImpl.java (was: Cleanup for android printing)

Comment 2 by ctzsm@chromium.org, Jun 16 2017

Status: Fixed (was: Assigned)
Landed https://chromium-review.googlesource.com/c/538042/, mark as fixed.

Sign in to add a comment