summaryrefslogtreecommitdiffstats
path: root/packages/PrintSpooler
Commit message (Collapse)AuthorAgeFilesLines
* Crash apps that print malformed or password protected PDFs.Svet Ganov2014-11-016-35/+113
| | | | | | | | | If apps are writing malformed content (typically not a PDF file) or if the PDF content they provide to the print system is password protected, are now crashed as both of these are app bugs. bug:17636435 Change-Id: Ifce6a3199e587448dd38f6a84290a965c24b698b
* Merge "Save to a PDF file should look like print preview." into lmp-mr1-devSvetoslav2014-10-313-15/+134
|\
| * Save to a PDF file should look like print preview.Svetoslav2014-10-313-15/+134
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When rendering a PDF file for print preview we take into account the selected print options such as paper size, orientation, etc without modifying the document. To print we send the doc in its original form and the print options so the print service can apply the necessary transforms in addition to the optional custom options it supports. When saving to PDF we have to actually change the document as we act as a print service. bug:13545980 Change-Id: Icdcecf962bec6ff742cc6015df5af9d9086ce760
* | Print preview disappearing if printer is changed.Svetoslav2014-10-291-1/+2
| | | | | | | | | | | | This is a regression caused by my previous patch. Change-Id: I80a0b4d880fd4e48e6a44fac303a4c8204e8ec0d
* | Merge "Printing from two apps at the same time not working." into lmp-mr1-devSvet Ganov2014-10-251-0/+2
|\ \
| * | Printing from two apps at the same time not working.Svet Ganov2014-10-241-0/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When printing from two apps at the same time the second print UI is getting stuck. There were a couple of issues here: AdapterView was not notifying for item selection if the data changes after scheduling a dalayed selection notification and the notification execution. The code assumed that a layout pass will occur and posponed the notification after the layout pass but it is not guaranteed that such a layout pass will occur. Now we delay only if a layout pass is being scheduled. Also when binding to the PDF rendering service the print spooler was using the same intent and as a result two print activites were getting the same renderer instance while they should get separate ones. Now we use different data in the intent to ensure we get separate renderer instances. Change-Id: I6aa7c7b041957804b4273549dd837a6d70064efc
* | Merge "Crash in print spooler if printing app killed from recents." into ↵Svet Ganov2014-10-246-79/+51
|\ \ | | | | | | | | | lmp-mr1-dev
| * | Crash in print spooler if printing app killed from recents.Svet Ganov2014-10-246-79/+51
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | If the printing app with the print UI on top is killed from recents we get a crash because: 1) the remote print document was not transitioned to a failed state if the printing app dies (this is an unrecoverable failure); 2) the print preview controller was destroyed asyncronosly during which it also asynchronously disconnects from the rendering service which however happens after the system has already cleaned up all connections of the print UI activity as it is being destoryed. bug:18109386 Change-Id: If6200b14a8aa90622228bbb659e9c4962226f561
* | Sometimes historical printers not properly ordered.Svet Ganov2014-10-231-13/+7
|/ | | | | | | | | | We order printers based on past usage. In some cases the ordering does not work. The reason for that was an incorrect assumption that the entries in an ArrayMap are ordered in the order they are added. bug:18109283 Change-Id: Ie367e4ca5e6bd79f335a060074c9211054e3a931
* Merge "Set copies to one when saving to PDF." into lmp-mr1-devSvet Ganov2014-10-231-1/+5
|\
| * Set copies to one when saving to PDF.Svet Ganov2014-10-231-1/+5
| | | | | | | | | | | | | | | | | | One can save only one copy to PDF. Showing more that one copy when saving to PDF enven in a disabled input box is confusing to the user. We now set the copies to one when the PDF printer is selected. bug:17631303 Change-Id: Ia04d144efc899e7f3836c19349d9c87f3ae8f0f0
* | am 60cdb909: am 0a1ffdb3: Merge "Handle if the printing app is killed while ↵Svetoslav2014-10-141-1/+3
|\ \ | |/ |/| | | | | | | | | saving to PDF." into lmp-dev * commit '60cdb909304f0d4b93a1a17228bdcc888f16cf41': Handle if the printing app is killed while saving to PDF.
| * am 0a1ffdb3: Merge "Handle if the printing app is killed while saving to ↵Svetoslav2014-10-141-1/+3
| |\ | | | | | | | | | | | | | | | | | | PDF." into lmp-dev * commit '0a1ffdb3f89705eeeb89349fcd31c82b2d15b0a5': Handle if the printing app is killed while saving to PDF.
| | * Handle if the printing app is killed while saving to PDF.Svetoslav2014-10-141-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When saving to PDF we bring up the file picker UI which covers the print UI. In this case the printing app may get killed as it is not in the forground (the system is bound to the print spooler so it cannot get killed). If that happens we are ending up with an empty file. However, if the file chooser UI is up we already have everything from the app we need and it dying does not matter. This change takes care if ignoring printing app death while saving to PDF. bug:17922948 Change-Id: I10d808e3c3e93e850dbc2a948f2482381a887928
| | * Import translations. DO NOT MERGEGeoff Mendal2014-10-062-4/+4
| | | | | | | | | | | | | | | Change-Id: I2100b649d1f2866cdc5ffd13f264afba347b6685 Auto-generated-cl: translation import
| * | Import translations. DO NOT MERGEGeoff Mendal2014-10-061-1/+1
| | | | | | | | | | | | | | | Change-Id: I391b269b0d535fa7d8a4082cb0271b0aeb16a497 Auto-generated-cl: translation import
* | | Custom print settings cannot change resolution.Svet Ganov2014-10-071-31/+51
| | | | | | | | | | | | | | | | | | bug:17677133 Change-Id: Ib8a24101f12d431fe221c1e91711d93a6a517273
* | | Import translations. DO NOT MERGEGeoff Mendal2014-10-061-1/+1
|/ / | | | | | | | | Change-Id: Ieb7aeaea1204f4ba7daf3abd7a30f647564807dd Auto-generated-cl: translation import
* | am 922fcc2f: Merge "Crash in print spooler if save to PDF selected early." ↵Svetoslav2014-09-291-1/+5
|\ \ | |/ | | | | | | | | | | into lmp-dev * commit '922fcc2f9e2370b34672af0e91e5754c6f8d6da7': Crash in print spooler if save to PDF selected early.
| * Merge "Crash in print spooler if save to PDF selected early." into lmp-devSvetoslav2014-09-281-1/+5
| |\
| | * Crash in print spooler if save to PDF selected early.Svetoslav2014-09-281-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The first issue is that the save to pdf button is enabled before the first layout completed and at this point we do not know anything about the document. The second is that if a layout is in progress and we select save to pdf we end up starting the documents UI twice and finishing the second instance carshes the spooler. bug:17676878 Change-Id: Id0b1f06e4d377347286c2dde09669d431441234b
* | | am b179bed3: Merge "Fix carsh in print spooler if custom print setings ↵Svetoslav2014-09-291-1/+26
|\ \ \ | |/ / | | | | | | | | | | | | | | | return bad pages." into lmp-dev * commit 'b179bed35302704bce809029ab424f919de351c3': Fix carsh in print spooler if custom print setings return bad pages.
| * | Fix carsh in print spooler if custom print setings return bad pages.Svetoslav2014-09-281-1/+26
| |/ | | | | | | | | | | | | | | | | | | The custom settings activity of a print serivce can change the selected pages and the spooler was not checking whether the returned pages were valid leading to a crash. bug:17678553 Change-Id: Id8ea3d482d2909b97a06752d61b416dfe5948483
* | am 299a89b5: Merge "Print spooler should not crash if fed non-PDF content." ↵Svetoslav2014-09-242-5/+5
|\ \ | |/ | | | | | | | | | | into lmp-dev * commit '299a89b50f1268f3c4cd0e58727f7d7805b573b2': Print spooler should not crash if fed non-PDF content.
| * Print spooler should not crash if fed non-PDF content.Svetoslav2014-09-242-5/+5
| | | | | | | | | | | | | | | | | | | | | | It is possible that a buggy app breaks the contract and provides content to be printed in format other than PDF. This was leading to a crash in the print spooler. This change fixes the crash and shows a user friendly error message. bug:17642690 Change-Id: I5a4acb06080a152562655da6851467b3e71d8658
* | am 15c86a84: Merge "Missing pages in print preview when scorolling." into ↵Svetoslav2014-09-241-4/+0
|\ \ | |/ | | | | | | | | | | lmp-dev * commit '15c86a84a5b595b43eecec611de22e6bd0c0eea4': Missing pages in print preview when scorolling.
| * Merge "Missing pages in print preview when scorolling." into lmp-devSvetoslav2014-09-241-4/+0
| |\
| | * Missing pages in print preview when scorolling.Svetoslav2014-09-231-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is possible that two providers for the same page are assigned to two preview views despite temporariliy while rebinding. We were however releasing the provider from the view which is being bound if not for the same page which as a result was cancelling rendering of this page which is provided to another view via another provider. Hence, having an empty page. Removed the unnecessary code to release a provider when binding the view - release is happening only if the view holder is recycled. bug:17515670 Change-Id: I197438c16cfdb363f521cd1320f154da54eea5f5
* | | am 857f8013: Merge "Fix ANR in print spooler." into lmp-devSvetoslav2014-09-241-0/+1
|\ \ \ | |/ / | | | | | | | | | * commit '857f80134d6c23894f5589457883c57137b2ed27': Fix ANR in print spooler.
| * | Merge "Fix ANR in print spooler." into lmp-devSvetoslav2014-09-231-0/+1
| |\ \
| | * | Fix ANR in print spooler.Svetoslav2014-09-231-0/+1
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Saving to PDF a subset of the pages requires trimming the undesired pages and when this is done writing the ready doc to a URI. To write the file to URI we have to obtain it but it is aquired by the shredding task and never released. Now we are releasing the document immediately after trimming it. bug:17631301 Change-Id: I4db7966c65c75f0f14c3cb52fd83b4d8fd5a4687
* | | am ea5d75d8: Merge "Print spooler crash on back before first layout is ↵Svetoslav2014-09-242-5/+1
|\ \ \ | |/ / | | | | | | | | | | | | | | | complete." into lmp-dev * commit 'ea5d75d807b379592f3bfd6ccd4cb43d02f72f49': Print spooler crash on back before first layout is complete.
| * | Print spooler crash on back before first layout is complete.Svetoslav2014-09-232-5/+1
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the user presses back before the remote print document update has completed we get a crash. The reason is that after the update completion we were trying to finish the activity if we are in a cancelled state but this is not needed as we aready handled the back key so we will finish. Handling finish twice was creating the problem as classes with lifecycles are not designed to be used after being finished. In particular, we were calling doFinish() twice. bug:17630561 Change-Id: If418f237a2def7c8e4a072ac8826283f4dd7fc85
* | am e9456781: Merge "Loosen the constraint for releasing a shared file." into ↵Svetoslav2014-09-221-1/+1
|\ \ | |/ | | | | | | | | | | lmp-dev * commit 'e9456781c0b4d6ccc70a5378ecd802fd28554796': Loosen the constraint for releasing a shared file.
| * Loosen the constraint for releasing a shared file.Svetoslav2014-09-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the spooler we have the renderer reading a file to visualize content and the app writing a file to produce the content. Since we have to swap the file under the renderer we have a mutex file provider that both parties can request, use when released, and release when required. This enables us to request the file which closes the renderer and when the renderer is closed ask the app to write some more pages, then open the renderer, and so on. The mutex file provider was throwing of a thread that does not own the file thries to relase it which is not needed, this should be just a nop. bug:17607134 Change-Id: Id6a2ce92d70077f57978b95315648faf02c13c68
| * Import translations. DO NOT MERGEBaligh Uddin2014-09-202-3/+3
| | | | | | | | | | Change-Id: Ib08e8991a092d8404352d9709d837be2eb392b55 Auto-generated-cl: translation import
* | Import translations. DO NOT MERGEBaligh Uddin2014-09-202-3/+3
| | | | | | | | | | Change-Id: I87cc837c719968aa778af92a4f119a266d2c2c42 Auto-generated-cl: translation import
* | am a2201272: Merge "Fix a print spooler crash when printing." into lmp-devSvetoslav2014-09-194-18/+33
|\ \ | |/ | | | | | | * commit 'a2201272b5b27f3ac9cf90d6332e81fb474c1dbb': Fix a print spooler crash when printing.
| * Fix a print spooler crash when printing.Svetoslav2014-09-194-18/+33
| | | | | | | | | | | | | | | | | | | | | | | | The spooler communicates with he remote PDF renderer asynchronously. When print is confirmed we close the renderer, destroy it, and unbind from its service. If we unbind from the service after the print activiy is finished we get a crash. The bug was that we did not wait until we disconnect from the remote renderer before finishing the print activity. bug:17583115 Change-Id: I55b0135f9c5658b3a4fda2901b8b3bdef044e211
* | am d972e2df: Merge "Print UI stuck when backing out of documents UI." into ↵Svetoslav2014-09-181-2/+2
|\ \ | |/ | | | | | | | | | | lmp-dev * commit 'd972e2dff33679a3c7fddd058396fd6437266709': Print UI stuck when backing out of documents UI.
| * Merge "Print UI stuck when backing out of documents UI." into lmp-devSvetoslav2014-09-181-2/+2
| |\
| | * Print UI stuck when backing out of documents UI.Svetoslav2014-09-181-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When user chooses to save to PDF we open the document chooser UI. Now if the user now backs to the print UI without choosing a file, the print UI has all options disabled and the back button does not work - the user is stuck. This is because we were not transitioning the print UI in the correct state. bug:17557454 Change-Id: I385129e106ed41564b89137ca54d409127ba8ff0
| * | Merge "Import translations. DO NOT MERGE" into lmp-devBaligh Uddin2014-09-187-36/+22
| |\ \ | | |/ | |/|
| | * Import translations. DO NOT MERGEBaligh Uddin2014-09-167-36/+22
| | | | | | | | | | | | | | | Change-Id: Ie9fb53c8c5a93a7059ede5961f2967c1892e0faf Auto-generated-cl: translation import
* | | Merge "Import translations. DO NOT MERGE" into lmp-sprout-devBaligh Uddin2014-09-1855-2/+442
|\ \ \
| * | | Import translations. DO NOT MERGEBaligh Uddin2014-09-1655-2/+442
| | | | | | | | | | | | | | | | | | | | Change-Id: Ib05ba70bddef4f0c406498a0005da785fcde2138 Auto-generated-cl: translation import
* | | | am 0bb6e05d: Page content sometimes missing in preview when scrolling.Svetoslav2014-09-172-31/+5
|\ \ \ \ | | |/ / | |/| | | | | | | | | | * commit '0bb6e05dcd5e809ca25ac79fc4d81c4c5957a8e2': Page content sometimes missing in preview when scrolling.
| * | | Page content sometimes missing in preview when scrolling.Svetoslav2014-09-172-31/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed the restriction for the page content providers to be singletons as the RecyclerView may hold on views with providers and ask us to bind other views for the same page. Note that the views that RecyclerView is holding will be soon rebound or recycled just they are not at the time we bind other views for the same position. We were guarding against this but caused some pages not appearing sometimes on scrolling. Also now keeping the media size and magins of recycled page view (will be updated if needed on rebinding) or avoid changes of the background to null as setting it to not null later causes an undesired layout pass. bug:17537922 Change-Id: If029f6c413e15d4f5c1ba1ac5e4d52ad2cad5904
* | | | am 84ad5f75: Merge "Page content sometimes disappears when scrolling a long ↵Svetoslav2014-09-175-12/+10
|\ \ \ \ | |/ / / | | | | | | | | | | | | | | | | | | | | doc in print preview." into lmp-dev * commit '84ad5f75e1c7d4f4d5113b413e814bde3aaa1747': Page content sometimes disappears when scrolling a long doc in print preview.
| * | | Page content sometimes disappears when scrolling a long doc in print preview.Svetoslav2014-09-175-12/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The operations of the remote renderer have to be performed in order, open, render some pages, close. One of the tasks was executed on the wrong executor resuling in a race and an occassional bad state. Also fixed a NPE if the preview list is flinged and then the user presses back. bug:17537922 Change-Id: I5048078ba2b875a2a8335f3a4324afaa34d014a2