Roll cacheinvalidation DEPS to r341

Roll cacheinvalidation DEPS to r341

This removes the last known dependency on guava_javalib.

Full list of changes:

r340 Removing guava dependency for the invalidation client (part of an effort to reduce method count in Android applications)
r341 Fixing package name in AndroidListenerManifest.xml and ordering factory methods by corresponding proto field in AndroidService.java

BUG=412673
TBR=zea@chromium.org

Committed: https://crrev.com/b7d5c878ba7ee66e929f299c92fd5393022b5586
Cr-Commit-Position: refs/heads/master@{#295113}

[nyquist, 2014-09-15 22:42:35]

nyquist@chromium.org changed reviewers:
+ meek@chromium.org, zea@chromium.org

[nyquist, 2014-09-15 22:42:35]

meek: PTAL
zea: PTAL OWNERS

[Colin Meek, 2014-09-16 17:54:18]

On 2014/09/15 22:42:35, nyquist wrote:
> meek: PTAL
> zea: PTAL OWNERS

LGTM

[Colin Meek, 2014-09-16 17:54:35]

https://codereview.chromium.org/573733004/diff/40001/sync/android/java/src/or...
File
sync/android/java/src/org/chromium/sync/notifier/InvalidationPreferences.java
(right):

https://codereview.chromium.org/573733004/diff/40001/sync/android/java/src/or...
sync/android/java/src/org/chromium/sync/notifier/InvalidationPreferences.java:114:
if (syncTypes == null) throw new NullPointerException("syncTypes is null.");
Do you typically use braces around 'if' block?

[nyquist, 2014-09-16 17:59:04]

moving zea@ to TBR

https://codereview.chromium.org/573733004/diff/40001/sync/android/java/src/or...
File
sync/android/java/src/org/chromium/sync/notifier/InvalidationPreferences.java
(right):

https://codereview.chromium.org/573733004/diff/40001/sync/android/java/src/or...
sync/android/java/src/org/chromium/sync/notifier/InvalidationPreferences.java:114:
if (syncTypes == null) throw new NullPointerException("syncTypes is null.");
On 2014/09/16 17:54:35, meek wrote:
> Do you typically use braces around 'if' block?

Yes, but one-liners are OK if they make sense. We use it for things like early
returns, etc. Hmm. I guess this class uses more of the brace-style though.
Then again, the other inlines of this precondition are all in-line.

Keeping as is for now.

[nyquist, 2014-09-16 17:59:08]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2014-09-16 18:00:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/573733004/40001

[commit-bot: I haz the power, 2014-09-16 18:47:57]

Committed patchset #3 (id:40001) as e02cfc0f8b4fc40420f64d5ee14b04026cdd5f31

[commit-bot: I haz the power, 2014-09-16 18:49:47]

Patchset 3 (id:??) landed as
https://crrev.com/b7d5c878ba7ee66e929f299c92fd5393022b5586
Cr-Commit-Position: refs/heads/master@{#295113}