| 5bb4d053519be10324752f323bcf73f13b9a2604 |
|
01-Apr-2014 |
Andrew Sapperstein <asapperstein@google.com> |
Allowing overriding conversation list load limit.
b/7280077
One issue we saw was that if the conversation was not
in the initial 50 conversations, we would get punted
out of the conversation list. Now we have an additional
extra in ACTION_VIEW intents that allow ignoring the
limit for the first load of the ConversationCursor.
Change-Id: I737c45e122df4dda7c63794ddc7036be61d5ebd4
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| b184bfe96fa3512af88260fce4f3cee3066fb28d |
|
26-May-2013 |
Scott Kennedy <skennedy@google.com> |
Unify LogTag
There is now only one LogTag class. The static initializer of
GmailApplication (existing) and EmailApplication (new) will now set
the log tag to "Gmail" and "Email", respectively. Up until that code
is run, it will be "UnifiedEmail".
"setprop log.tag.Gmail VERBOSE" (or .Email) will trigger all logs to
be printed as long as they go through LogUtils, regardless of what tag
is used by that individual log. This lets us still turn on logging
everywhere in one command, but also lets us use more descriptive tags
(like the class name).
And since we no longer have three com.android.mail.utils.LogTag
classes, builds will be much easier.
Also, we now use LogUtils everywhere.
Change-Id: Iec53e87cad0a46b73d2afbbd8d44aca097a5087a
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| ff8553f20964f4c31b0c503a9e1daff6ae08a9c7 |
|
06-Apr-2013 |
Scott Kennedy <skennedy@google.com> |
Cleanup
Change-Id: Iee8abf75639450d469632e43fe0e102b687558db
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 08a079c3d2857e365736432b2691187767eb116f |
|
22-Feb-2013 |
Scott Kennedy <skennedy@google.com> |
Hide conversations from list after removal via notif action
If a conversation is archived, deleted, or has a label removed via a
notification action, we should hide it from the conversation list
until either the undo notification is activated, or it times out.
Bug: 8248145
Change-Id: Ib88a14142238b5553f80d59e67be5f64ab2a28fb
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| d6020119c71ef9fe38725d0f0291cac7b9df6e79 |
|
15-Feb-2013 |
Vikram Aggarwal <viki@google.com> |
Resolve all STOPSHIP comments in UnifiedEmail
Bug 8211088 filed for the remaining case.
Bug: 8132946 Investigate the STOPSHIP comment in EmailProvider.java
wrt PII
Change-Id: I0f93e7bd68360eccbf781c144addf8a8a66ae779
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 7460a1c7b617a6a67d3f24c699d0b16a78a3c02c |
|
29-Dec-2012 |
Vikram Aggarwal <viki@google.com> |
Remove activity leak
The conversation cursor was maintaining a static reference to loaders
and the resolver. When the cursor was loaded immediately after
orientation changes, the CursorLoader framework returns the existing
cursor. This cursor contained the reference to the old activity. This
also meant that the sResolver reference was stale till the
conversation cursor was recreated.
As suggested by ath@, we remove the static ContentResolver, and rely
on a member.
Also, as suggested by ath@, we use the ApplicationContext, which
removes the inner memory leak that was being caused by the
CursorWrapper holding on to a context, which held on to the previous
application.
Just one activity on rotate (after GC).
Also contains minor code cleanup: removing unused import and cleaning
up incorrect javadoc on methods.
Bug: 7883592 mild activity leak
Change-Id: I09043c38055e5d7ed2fb345286b1f293e39bf4c3
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 9a70e919125d63971604e6dd3de8bc998e9c86da |
|
18-Aug-2012 |
Paul Westbrook <pwestbro@google.com> |
Fix resetting ConversationCursor
We need to reset the state of the Activity controller when the
ConversationCursor is reset.
It isn't necessary for ConversationCursorLoader to call the
callback's onReset, as this will happen from the loader manager
Also, the loader manager only calls onLoaderReset() if the caller has
valid data from a new instance of the loader, but onReset of the loader
is still called. We don't want to call onLoaderReset() for every call
to onReset()
Bug: 7009556
Change-Id: If943409028aa8f5e465a5dba4bac1307a4d4e0fe
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 04e0dc70c61c1d856877718922f8b1de2f25a2ff |
|
02-Aug-2012 |
Paul Westbrook <pwestbro@google.com> |
Fix some problems with conversation cursor
1) If someone called setCursor, the internal state of the
ConversationCursor would prevent new cursors from being observed
2) If onRefreshReady happend while the list is scrolling, that wasn't
handled when the list scrolling stopped
3) If both onRefreshReady and onRefreshRequired happened while
the list is scrolling, when the list stopped scrolling, only
onRefreshRequired would be called, which ends up unregistering the
observer
Change-Id: I1b07b8b41b1025b981f5fdae7d6e2e18568029de
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 741846b4b3a33011cf1913a36114b36717b7f848 |
|
02-Aug-2012 |
Marc Blank <mblank@google.com> |
Make sure reset ConversationCursorLoader gets cleaned up
Bug: 6381692
Change-Id: I4a427d0076f790091d4d166b10c322e1844f5003
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| da3b39d9f2c8d39ceb615a1d1731e06745e7c50c |
|
20-Jul-2012 |
Marc Blank <mblank@google.com> |
Fix broken logging of loader add/remove
Change-Id: Ib53a26d548ddc13b24c14ef4d501453697b1b8a5
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| bf232c3735f65b1a4746943e4a134e59e36f0bde |
|
18-Apr-2012 |
Paul Westbrook <pwestbro@google.com> |
Major refactor of ConversationCursor
* ConversationCursors are now associated with a particular loader;
there are no static methods or static state fields
* ConversationCursorLoader maintains a list of current loaders
(for tracking/debugging)
* AbstractActivityController acts as a router of conversation
changes to/from fragments (currently message view -> conversation
list)
* Comment out some ConversationCursor tests for now
Change-Id: I9b8425a19c77501796b6681142e1bc0ad20b3693
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| cf164d64bcb1da92b427bda99b97f7ec310ef704 |
|
20-Apr-2012 |
Marc Blank <mblank@google.com> |
Revert "Major refactor of ConversationCursor"
This reverts commit 5150f03723af8019169aeed8e406784da9c5f8f1.
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 5150f03723af8019169aeed8e406784da9c5f8f1 |
|
18-Apr-2012 |
Paul Westbrook <pwestbro@google.com> |
Major refactor of ConversationCursor
* ConversationCursors are now associated with a particular loader;
there are no static methods or static state fields
* ConversationCursorLoader maintains a list of current loaders
(for tracking/debugging)
* AbstractActivityController acts as a router of conversation
changes to/from fragments (currently message view -> conversation
list)
Change-Id: I0eb554615b4bf93b3547ff28e2bd5913e8703a97
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 08d50c8cfbad1aa932b1ea3134f6591a34c85633 |
|
19-Apr-2012 |
Marc Blank <mblank@google.com> |
Comment out pause/resume
Change-Id: Id96c69a9924f05e7c2129984176be293aad1aced
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 2a9dc8e41052752dbe2d254afc894b2326514f41 |
|
16-Apr-2012 |
Marc Blank <mblank@google.com> |
Don't stop loading onReset for CC
Bug: 6345560
Change-Id: I2c8c31d9da507f47919817760916bc4132ce6459
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| e1d1b07cdb0026097eb80f6c2912a16353aacec1 |
|
14-Apr-2012 |
Marc Blank <mblank@google.com> |
Lots 'o ConversationCursor goodness
* Throttling the cursor!
* Almost dead list items!
* But wait! There's more!
Change-Id: If0fe7b3d22e2dfa0e1d7bb2eaa307d20c52dc21c
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 51144944c2088299ebc2584cc09c61c23eb18659 |
|
20-Mar-2012 |
Marc Blank <mblank@google.com> |
Add "limit" to first load of ConversationCursor
* For my large convo list, time for first query is down to about
300ms from average 9000ms
* TODO: Clean all this up...
Change-Id: Iaa17f5b8b5b773cb4c8133b4f22d7096b3a3aa54
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| ff5c757ed2bcf7004a70b0675382ef894196558d |
|
16-Mar-2012 |
Paul Westbrook <pwestbro@google.com> |
Allow a Conversation to be created with just the conversation
fields
Work around for NPE when marking conversation as read before
ConvrsationCursor is created
Change-Id: I495d408146287de5d6ce7253d2e082cc4d4b4d6a
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| cfb7f33ff8ef9dc60b9143af32de64c7a3d71f36 |
|
28-Feb-2012 |
Mindy Pereira <mindyp@google.com> |
Store URI's in Uri instead of String.
Change-Id: I5492a61b80442afc7cebc5b3d560688aebae8820
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|
| 609480e14d4cb999d9b6bb1c224b3584d3f367f0 |
|
16-Feb-2012 |
Mindy Pereira <mindyp@google.com> |
Get loading folders/conversations off the ui thread.
Use loaders. Pretty.
Change-Id: I0d87ffa8e390f15718c71f05977ff6d331aeb9ca
/packages/apps/UnifiedEmail/src/com/android/mail/ui/ConversationCursorLoader.java
|