summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Attachments in Conversation List: Backend changes so that the appropriateMark Wei2013-07-0828-415/+1802
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | attachments are passed to the UI. Implement PhotoManager so we can start showing images. Upon sync, we extract two image attachments from the first unread message, and add them to the conversations table. UI changes as well. Load BEST if that is available, else load SIMPLE. No vertical dividers in attachment previews. Attachment previews are above labels. Round badge appears for unread conversations when there are more images than shown. Only appears after the previews have loaded. Clicking on the attachment previews launches photoviewer. Attempt to load bigger images than necessary to prevent reloading during mark read/unread and rotate. Show animated loading icon while previews are loading. Previews load from top to bottom, from left to right. Big performance improvements: No longer decoding bitmap on UI thread. This kills most of the jank. Only load when we are not scrolling. This kills a lot of jank. Remaining jank will be delt with later: DividedImageCanvas.setDimensions() creates a huge empty bitmap on the UI thread, which causes a GC. ath will fix. Multiple images in one line now load whatever's ready first. Added tracing. Bitmap cache given more memory. Following ath's advice, decode Bitmap from InputStream rather than byte[]. Don't compress into BitmapHolder. Only use the bigger Bitmap cache. Bug: 7481049 Change-Id: Icf823f56abc9345e6d0b800a1e6d45444d8fa028
* Merge "Don't pass custom Parcelables to AlarmManager" into jb-ub-mail-ur10Scott Kennedy2013-07-092-16/+50
|\
| * Don't pass custom Parcelables to AlarmManagerScott Kennedy2013-07-082-16/+50
| | | | | | | | | | | | | | | | | | | | | | | | AlarmManager unparcels everything in the Bundle, which causes a ClassNotFoundException. So we'll take some code from Clock, and shove a NotificationAction into a byte array, and parcel that. Other processes can unparcel this without causing any problems. Bug: 9051575 Change-Id: Ie8fbf1f8b4e8a1523f02c2d84fe50a780a4d90c7
* | Conversation View UI Refresh Message Headers.Andrew Sapperstein2013-07-0829-140/+169
| | | | | | | | | | | | | | Still need to do the new snap header spec and the icons are a bit too small, but it's a good start. Change-Id: I0e2e622a1100c96dab022fd3836bc6666d6330cd
* | Merge "Import translations. DO NOT MERGE" into jb-ub-mail-ur10Baligh Uddin2013-07-0849-53/+146
|\ \
| * | Import translations. DO NOT MERGEBaligh Uddin2013-07-0849-53/+146
| |/ | | | | | | | | Change-Id: Ia3511c86b221e88892f4840ab443a4612e5a436d Auto-generated-cl: translation import
* | smoother progress barAndy Huang2013-07-0829-52/+159
| | | | | | | | | | | | | | | | | | | | Remove hard-coded assets for the swipe-refresh progress bar and use a new procedurally drawn progress bar that runs at 60fps and reduces memory footprint and GC significantly. Some colors and sizes are now configurable now, to boot. Bug: 9737423 Change-Id: I46d34d4c6c49b9c57fb80ff4cb03bdc99cfe13f4
* | Disable drawer for child foldersVikram Aggarwal2013-07-081-19/+42
|/ | | | | | Bug: 9696028 Disable drawer for child folders Change-Id: Iba2353e71c9b4789611737173db874fc37981b05
* Merge "Parent folder as a URI" into jb-ub-mail-ur10Vikram Aggarwal2013-07-036-36/+43
|\
| * Parent folder as a URIVikram Aggarwal2013-07-036-36/+43
| | | | | | | | | | | | | | | | | | | | 1. Change folder to a URI 2. Fix references. Bug: 9694857 Populate folder.parent correctly in the provider Bug: 9694899 Clean up the hierarchy folder code Change-Id: I4eff33d0259777e447cfe5769813db990f142077
* | am 4ba11c20: (-s ours) Import translations. DO NOT MERGEBaligh Uddin2013-07-030-0/+0
|\ \ | | | | | | | | | | | | * commit '4ba11c2040cb1fbc868c1a6ff96e49ba1e0d0e1b': Import translations. DO NOT MERGE
| * | Import translations. DO NOT MERGEBaligh Uddin2013-07-032-3/+3
| | | | | | | | | | | | | | | Change-Id: I8d3f248f7e1ec32ef4e61cd286824a93493d9776 Auto-generated-cl: translation import
* | | Merge "Import translations. DO NOT MERGE" into jb-ub-mail-ur10Baligh Uddin2013-07-033-4/+4
|\ \ \ | |_|/ |/| |
| * | Import translations. DO NOT MERGEBaligh Uddin2013-07-033-4/+4
| | | | | | | | | | | | | | | Change-Id: I8d9b5ed6813c038530f88f727f200619ad9b9ba2 Auto-generated-cl: translation import
* | | Pluralize action bar unread message for i18n, bug 9571455Tony Mantler2013-07-032-8/+8
| | | | | | | | | | | | Change-Id: I8ac53f9b9ac8363c149c63f9e5d650db210fda87
* | | Fix highlight in Email with nested foldersVikram Aggarwal2013-07-031-5/+7
|/ / | | | | | | | | | | | | | | | | Calculate the offset correctly without calling getPosition() to ensure the correct conversation is highlighted. Bug: 9680612 Nested folders highlight incorrect conversation Change-Id: I4aea4bbe88c726d4644f202ae3bf57f979aebc74
* | Revert "Highlight the correct conversation in the presence of nested folders"Mark Wei2013-07-031-7/+6
| | | | | | | | | | | | This reverts commit 11324ad037cb9a2735c23295972934b106da2980. Change-Id: If05a7f706ebad2fbd8c9b5702f7f066bb1504581
* | Highlight the correct conversation in the presence of nested foldersVikram Aggarwal2013-07-021-6/+7
| | | | | | | | | | | | | | | | | | Nested folders go right on top, and fleeting views specify position. This change corrects the cursor position -> list position calculation and simplifies it. Bug: 9680612 Nested folders highlight incorrect conversation Change-Id: I119af8efac16375445d5ba90cd62cedbfbe845c9
* | Merge "Import translations. DO NOT MERGE" into jb-ub-mail-ur10Baligh Uddin2013-07-0249-146/+48
|\ \
| * | Import translations. DO NOT MERGEBaligh Uddin2013-07-0149-146/+48
| | | | | | | | | | | | | | | Change-Id: Ie7a882d3b4000b55cb2cc34e8f86ae4ecc5ce525 Auto-generated-cl: translation import
* | | Hash account when building base uri in conv view.Andrew Sapperstein2013-07-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes b/9170480. Previously we just used account name to build the base uri for viewing the conversation in web view. That breaks when the account name has an apostrophe. So instead just hash the name and use that instead. Change-Id: I84f6dd4f67371795bc46c6f97573230e3981a4cc
* | | More bidi notification fixes.Andrew Sapperstein2013-07-012-4/+8
| | | | | | | | | | | | Change-Id: Id0934adf6a41025ad54131de874fc344e5689086
* | | Merge "Return when reset." into jb-ub-mail-ur10Andrew Sapperstein2013-07-011-0/+1
|\ \ \ | |/ / |/| |
| * | Return when reset.Andrew Sapperstein2013-06-281-0/+1
| | | | | | | | | | | | Change-Id: Ia356ad3f0bfdb929ea872d6749ed1aa64e9292d9
* | | Merge "list caching tweaks" into jb-ub-mail-ur10Andy Huang2013-06-281-6/+26
|\ \ \
| * | | list caching tweaksAndy Huang2013-06-271-6/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stop background caching earlier - it's pointless to continue caching when the dataset changes and a new cursor is inbound. avoids creating unnecessary garbage. * add explicit GCs to sidestep common cases that usually result in GC_FOR_ALLOC: after a query and after background caching Bug: 9032162 Change-Id: Ic2e7819e6887b965515a63db3cac7e4dfdf70969
* | | | Change how we get a LetterTileProvider for notificationsScott Kennedy2013-06-282-5/+3
| |/ / |/| | | | | | | | | | | | | | Because conflicts. Change-Id: I82966a309f8b3a92ebd1e35f56af6ddaa966f3f3
* | | Prevent NPE.Andrew Sapperstein2013-06-281-7/+22
|/ / | | | | | | | | | | | | | | Fixes b/9265059 by adding some null checks. Also added some wtf logs so we can figure out what is null. Change-Id: I2c8c660effa63abdddaa3fa075952ed46681192b
* | Merge "Hide Dial&SMS actions if no apps can handle them." into jb-ub-mail-ur10Andrew Sapperstein2013-06-281-8/+32
|\ \
| * | Hide Dial&SMS actions if no apps can handle them.Andrew Sapperstein2013-06-271-8/+32
| | | | | | | | | | | | | | | | | | Fixes b/9057637. Change-Id: If00a322225676c9075cfbd6170ab04c2b43f7e77
* | | Use letter tiles in notificationsScott Kennedy2013-06-272-29/+63
| | | | | | | | | | | | | | | | | | | | | | | | For notifications where we would display a contact photo if it existed, we will now show a letter tile if we can make one. Bug: 9597536 Change-Id: Idb060d2c055074ff930154c12ddfff7d960dbf75
* | | Some cleanupScott Kennedy2013-06-272-10/+6
|/ / | | | | | | Change-Id: Ic81e40d2b85031cb9f5f17d34bd960d5984fe21e
* | Render bidi text in inbox notifications correctly.Andrew Sapperstein2013-06-271-2/+8
| | | | | | | | | | | | | | | | Mixed language text where the directions of the languages differed could display errors because they were not unicode wrapped. Now they are. Fixes b/8841323. Change-Id: Ic55efeeacde2a7e4a9b78f1399324806e4310190
* | Merge "Import translations. DO NOT MERGE" into jb-ub-mail-ur10Baligh Uddin2013-06-2749-2/+100
|\ \
| * | Import translations. DO NOT MERGEBaligh Uddin2013-06-2649-2/+100
| | | | | | | | | | | | | | | Change-Id: I4a7c2987570e7f8f5a54f3f1a2299c32193d08fd Auto-generated-cl: translation import
* | | Merge "yielding cache construction to reduce scroll+refresh jank" into ↵Andy Huang2013-06-273-45/+210
|\ \ \ | | | | | | | | | | | | jb-ub-mail-ur10
| * | | yielding cache construction to reduce scroll+refresh jankAndy Huang2013-06-263-45/+210
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Background caching of conversation objects triggers GC_FOR_ALLOC, which blocks the main thread and causes jank. We do want to cache to improve scroll framerate, but we also want to avoid causing GCs while drawing is in progress. Add cancel-handling to the cache's async task, and connect it to a new DrawIdler class that monitors a view hierarchy for draws. Together, they cause any drawing to pause background caching until 100ms of drawing silence. If you scroll before the cache is filled, scroll framerate will not benefit from the cache, so it will be somewhat lower. Previously, caching would continue during the scroll, so depending on where you are in the list, it may have been faster, except that it was punctuated by annoying GC pauses. The framerate now will be generally more even now, although it may be lower if you scroll before the cache is allowed to be filled. The DrawIdler is 1:1 owned by the Activity/Controller, but a single Cursor may be shared across Activity instances (e.g. rotate). So connect each DrawIdler to the shared cursor in onLoadFinished. This works because the rotated Activity's onLoadFinished is called *after* the first Activity's onDestroy. Bug: 9032162 Change-Id: I217dac012f2d74f47a158aee3bd2de78d9b14171
* | | | Merge "Adding UNREAD folder type." into jb-ub-mail-ur10Andrew Sapperstein2013-06-271-0/+2
|\ \ \ \ | |/ / / |/| | |
| * | | Adding UNREAD folder type.Andrew Sapperstein2013-06-261-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Needed for support in Email. Change-Id: I9e26add44754b65919e01352303fe4607d026a21
* | | | Remove an unused stringScott Kennedy2013-06-261-4/+0
|/ / / | | | | | | | | | Change-Id: Ieab0b206b44ec6e41fd5da1bba961930693bcd41
* | | Merge "Clean up a bunch of warnings" into jb-ub-mail-ur10Scott Kennedy2013-06-2642-148/+75
|\ \ \ | |/ / |/| |
| * | Clean up a bunch of warningsScott Kennedy2013-06-2542-148/+75
| | | | | | | | | | | | | | | Bug: 9565838 Change-Id: Ie3a380581e81c63aaa0764875c55bfdccd5f2b13
* | | Merge "Final eml viewer cleanup." into jb-ub-mail-ur10Andrew Sapperstein2013-06-2510-42/+238
|\ \ \
| * | | Final eml viewer cleanup.Andrew Sapperstein2013-06-2510-42/+238
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes b/9560839. Now we show the title of the eml file and we also show an overflow button with settings/send feedback/help. Change-Id: I5eab245d9fd47e37b83eadcb419255939117ab39
* | | | am 440e405b: (-s ours) Import translations. DO NOT MERGEBaligh Uddin2013-06-250-0/+0
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | * commit '440e405b64bd871c08764cdc5bccc1cc2abd03d8': Import translations. DO NOT MERGE
| * | | Import translations. DO NOT MERGEBaligh Uddin2013-06-241-1/+1
| | | | | | | | | | | | | | | | | | | | Change-Id: I4961892c6ffebdfcd06af6e206d290f3ac2e431c Auto-generated-cl: translation import
* | | | Import translations. DO NOT MERGEBaligh Uddin2013-06-2431-293/+148
| |_|/ |/| | | | | | | | | | | Change-Id: Ic5124001b822cf78b20da084c07a4a1dfaea91a6 Auto-generated-cl: translation import
* | | Remove an unnecessary lineScott Kennedy2013-06-241-3/+0
| |/ |/| | | | | | | | | | | Since the notification content title is the same as the big content title, there's no point in setting it separately. Change-Id: Ia2f7e24a2f333248ea46059dd396b298504aba50
* | Allow installing APKs from eml viewer.Andrew Sapperstein2013-06-241-0/+6
| | | | | | | | | | | | | | Fixes b/9528672. Need to change the content uri to a direct file uri so install will work. Change-Id: Iac9e17e45ecf5c9551e271dbde6cc3f18bf4f8ee
* | Attachments in eml files.Andrew Sapperstein2013-06-2413-17/+701
| | | | | | | | | | | | | | | | | | | | It's kinda slow for large eml files (because we have to parse the entire file) but it works. Hooray. Fixes b/6393073. Change-Id: I2d7a87a484cd282a000a0905fe069f3ab45e2061