Change WebView Variations Services to use AIDL instead of Messengers |
|||||
Issue descriptionCurrently the AwVariationsSeedHandler, AwVariationsConfigurationService, and AwVariationsSeedFetchService use Messengers for IPC. It's been discussed that we use AIDL instead. Here's a brief summary of the main points. AIDL: -Overall easier to read / understand. -Easy to verify that the message received by AwVariationsConfigurationService actually originated from AwVariationsSeedFetchService. Messengers: -All messages arrive on the same handler thread, eliminating the need for locking.
,
Sep 11 2017
err, no blocking, but yes, need to just post everything to the main thread binder messages are synchronous by default, and the way binder handles synchronous call cycles is uhh... somewhat unexpected
,
Sep 11 2017
Well, we will want to return seed if it's fresh - in that case we will either have to block or use a lock to fetch the seed I think?
,
Sep 11 2017
oh yeah, probably judicious use of lock then
,
Sep 27 2017
,
May 26 2018
,
Aug 3
This bug has an owner, thus, it's been triaged. Changing status to "assigned".
,
Aug 3
This was implemented in https://crrev.com/c/954058 and https://crrev.com/c/1036852. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by gsennton@chromium.org
, Sep 4 2017