diff options
author | pavely@chromium.org <pavely@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-22 18:36:20 +0000 |
---|---|---|
committer | pavely@chromium.org <pavely@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-22 18:36:20 +0000 |
commit | df006cbc99fb695e25b0d494279a8c16e39bcca9 (patch) | |
tree | a331c547a865697f2cfa93ca957ff2b16201ff46 /sync/sync_tests.gypi | |
parent | a077ab834eba15dd0aea87a0fd9112b9b8b8a676 (diff) | |
download | chromium_src-df006cbc99fb695e25b0d494279a8c16e39bcca9.zip chromium_src-df006cbc99fb695e25b0d494279a8c16e39bcca9.tar.gz chromium_src-df006cbc99fb695e25b0d494279a8c16e39bcca9.tar.bz2 |
Client-to-server messages in GCMNetworkChannel
This change implements sending client-to-server messages in
GCMNetworkChannel. Implementation mimics what android implementation is
doing. To send message to server GCMNetworkChannel needs GCM
registration Id and access token. Registration Id is requested during
initialization and doesn’t change through process lifetime. Access token
is requested for every message. Errors are not handled,
cacheinvalidations will retry sending message if it doesn’t see
confirmation from server that the message was received. The only handled
error is HTTP_UNAUTHORIZED from tango server since access token needs to
be invalidated in this case.
Actual work of requesting registration id and access token is delegated
to GCMNetworkChannelDelegate interface. Implementation of this interface
will live in chrome/browser/invalidation directory and will use
GCMProfileService for registration and ProfileOAuth2TokenService for
token functions.
There are two things that are still not implemented:
- Register error handling: will need to implement retry.
- Building url for tango endpoint: this involves preparing and base64
encoding NetworkEndpointId protobuf from cacheinvalidations library.
Will address both in next CL.
BUG=325020
Review URL: https://codereview.chromium.org/140513002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246377 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/sync_tests.gypi')
-rw-r--r-- | sync/sync_tests.gypi | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sync/sync_tests.gypi b/sync/sync_tests.gypi index f4729c6..1adbf1c 100644 --- a/sync/sync_tests.gypi +++ b/sync/sync_tests.gypi @@ -313,6 +313,7 @@ 'suppress_wildcard': 1, 'dependencies': [ '../base/base.gyp:base', + '../google_apis/google_apis.gyp:google_apis', '../jingle/jingle.gyp:notifier_test_util', '../net/net.gyp:net_test_support', '../testing/gmock.gyp:gmock', @@ -326,6 +327,7 @@ # happens in the dependents. 'export_dependent_settings': [ '../base/base.gyp:base', + '../google_apis/google_apis.gyp:google_apis', '../jingle/jingle.gyp:notifier_test_util', '../net/net.gyp:net_test_support', '../testing/gmock.gyp:gmock', |