aa514313a55a93f18b84e159987b39bcfdf63917 |
|
17-Feb-2017 |
Hiroaki Kuriyama <hiroaki.kuriyama@sonymobile.com> |
Merge "PrintSpooler: Remove recommendation if Play Store is not installed" am: 57b966eee1 am: 7cd04745f4 am: 2ab70d213b Change-Id: I9d4e003492c5d2b8463725e454ac985481958db7
|
36bebe842039d1ec583dfa2e000340e710b7ada4 |
|
25-Oct-2016 |
Hiroaki Kuriyama <hiroaki.kuriyama@sonymobile.com> |
PrintSpooler: Remove recommendation if Play Store is not installed Sympton: If Play Store (com.android.vending) is not installed, printer recommendation function which guides a user to Play Store is useless. Root-cause: Even if Play Store is not installed, PrintSpooler tries to start an activity for market:// with package names in Play Store. Solution: Hide printer recommendation UI and show "No print services enabled" if there is no printer service. Change-Id: I0b2eabbd57bc47c6382f53e610e33252ea359390
/frameworks/base/packages/PrintSpooler/src/com/android/printspooler/ui/AddPrinterActivity.java
|
383db5ebcc3a4a615faf249bf4f126f42e80b82e |
|
22-Jun-2016 |
Tamas Berghammer <tberghammer@google.com> |
Update package names to work with the proto3 compiler Bug: b/28974522 Change-Id: I5f3adf4946ee4ba1e09e4f40afe83c151405972a
/frameworks/base/packages/PrintSpooler/src/com/android/printspooler/ui/AddPrinterActivity.java
|
12cd55f94e3f43f4e6f3e6b3d88038045339d1d2 |
|
31-Jul-2016 |
Philip P. Moltmann <moltmann@google.com> |
Handle reset of recommended services am: fe789f3189 am: bc11f26a09 am: 36d68891a5 Change-Id: I13f55cee79c6117f95c9fe8b268b64c654af90a0
|
fe789f318976395fac69f374c7a00756a408c997 |
|
26-Jul-2016 |
Philip P. Moltmann <moltmann@google.com> |
Handle reset of recommended services Change-Id: I778f8b7a081d433ca0f21361c2ca8dcd48303935 Fixes: 30389837
/frameworks/base/packages/PrintSpooler/src/com/android/printspooler/ui/AddPrinterActivity.java
|
9599a99ee16d0873d12bf4001fbd393e741f4b0b |
|
23-Jun-2016 |
Philip P. Moltmann <moltmann@google.com> |
Merge \\\"Do not launch non exposed add printer activity\\\" into nyc-dev am: 737be111cc am: 89fe3e0169 am: de247457be Change-Id: I04ce338f871b55e6b19de016dad1e2f0a87f833c
|
61209bc9245764f86b7d9fa9f1105239b994fe0b |
|
23-Jun-2016 |
Philip P. Moltmann <moltmann@google.com> |
Do not launch non exposed add printer activity Change-Id: I0b9b11ea0ce0140c77dff2539b1a0a9dcf1558b0 Fixes: 29580579
/frameworks/base/packages/PrintSpooler/src/com/android/printspooler/ui/AddPrinterActivity.java
|
2e3012624b703a653fed44d4dc1b8904406c0d99 |
|
16-Jun-2016 |
Philip P. Moltmann <moltmann@google.com> |
Add event tracking to print UI. Bug: 29391676 Change-Id: Iec09ac7296e50329e87268bcb70f5e60d566961c
/frameworks/base/packages/PrintSpooler/src/com/android/printspooler/ui/AddPrinterActivity.java
|
9dcb86a48d73f399fb1b5c020005d76d350eeac2 |
|
14-Mar-2016 |
Philip P. Moltmann <moltmann@google.com> |
Add the print service recommendation service This service connects through the print manager to the print spooler: PrintSpooler.AddPrintersActivity <-> PrintManager <-> PrintManagerService <-> UserState <-> RemotePrintServiceRecommendationService <-> PrintRecommendationService <-> PrintRecommendationServiceImpl Hence there is a lot of mindless plumming. The actual changes are only in the AddPrintersActivity which is extended to show another list of services: The recommended services. The PrintServiceRecommendationService is based on the experimenal print service stubs provider. This provider was contributed the Android by Mopria. As this services uses Android own network discovery service most code from the experimental provider goes away. In fact the only logic left over is the selections of mdns-txt fields to look at and the printer vendor configuration. This relies on the Android MDNS to get fixed (Bug: 27696905). This also does not deal with how to update the recommendation service. Bug: 24533249 Change-Id: I6edc6e25fc08a50d478b61c71bb8ea158b08624c
/frameworks/base/packages/PrintSpooler/src/com/android/printspooler/ui/AddPrinterActivity.java
|
66c96591e2ddb464c67e60dbf4193ef4ec8a620b |
|
24-Feb-2016 |
Philip P. Moltmann <moltmann@google.com> |
Add "app printer activity" and always keep the print service state updated. Also fiddle with the UI to use more standard values. To be sure the print service state alwasy updated I changed PrintManager.getPrintServices to return a loader which just wraps a registerListener/getList/removeListener combo. I also added a new function to enabled/disable a print service to be keep all updating logic inside the PrintManagerService->UserState. Then I changed all code to use this new interface. Detailed comments: PrintServiceInfo: - I had to add the enabled state to the PrintServiceInfo as some users of PrintManager.getPrintServices want all services but then display different data depending on the enabled state. Of course I could have created two PrintManager.getPrintServices-loaders to load the two separate list of services. I think it is much easier to add this property though. It is updated every time new data is returned to the PrintManager.getPrintServices-loader. AddPrinterActivity: - This is shown as a dialog-style overlay to indicate that the user will return to the select-printers activity. It contains of three list that are updated via separate loaders. - The recommended services will be added later to keep this path set small. PrintActivity: - There are two small places where we have to update the data when we get a new list of print services. - In very, very rare conditions it can happen that the print service of the current printer gains or looses the "advancedOptions" activity - If we have no enabled print services we want to show "Add printer" instead of "All printers...". - Also the print registry is not the only loader anymore, hence we have to assign loader ids to it to not conflict with the other loaders in this activity. - Small bug in onPrintersChanged: If a printer is selected and the print service of this printer gets disabled the holder goes into "removed" state which disables the printer. When the print service is then enabled again, we forgot to re-enable the holder. PrinterRegistry: - The registry assumed that the FusedPrinterProvider was the only loader in the activity. This is not true anymore, hence it has to assign the appropriate loader ids. - The FusedPrinterProvider has an internal loader, hence we have to forward a loader Id into it. - The PrintRegistry is only called backed for a single loader, hence no need to check the loader-id. SelectPrinterActivity: - The AddPrinterDialog was removed as we now have the AddPrinterActivity. - Added a loader for the enabled services to update the empty state. - Added dedicated loader Id for the PrinterRegistry again. - If we have no enabled services, the SelectPrinterActivity chainloads the AddPrinterActivity as this is the only thing the user can do anyway. "Save a click". This should only happen when the activity is create the first time. - Moved the "add printer" from the menu item to the list of printers as suggested by UX and Zach. PrintManagerService, UserState and IPrintManagerParamtersTest: - As the only place where the print service state is updated is now the userstate, we have no more sychronization problems. Whohoo. - The users can now register for changes to the print services similar as they can register for changes to the print jobs. - UserState.getPrintServices is the only function can exposes any knowledge of the print services to the outside world. Change-Id: I9be2c7300431e06aaff9bdf7eb36120d869b56ac
/frameworks/base/packages/PrintSpooler/src/com/android/printspooler/ui/AddPrinterActivity.java
|