diff options
| author | Alex Buynytskyy <alexbuy@google.com> | 2020-04-22 16:08:50 -0700 |
|---|---|---|
| committer | Alex Buynytskyy <alexbuy@google.com> | 2020-04-23 19:38:44 +0000 |
| commit | ea1390ff324a8efab225955e507911bb1c44bca2 (patch) | |
| tree | f555dcac42f4343663db8cf703f9a8f96c477c44 /services/incremental/ServiceWrappers.h | |
| parent | eab14fca5dadd2b6f57ff37cce8ce2558c8b94f6 (diff) | |
Keep only service binding in DataLoaderManager.
This simplifies:
- resource management - no extra copies of controls,
- state management - all states in one place, no more hidden (bound but
not created) state.
Bug: b/153874006
Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest
Change-Id: I3d16a099c7f42fcf14637c5a8e96bd6f99e073d1
Diffstat (limited to 'services/incremental/ServiceWrappers.h')
| -rw-r--r-- | services/incremental/ServiceWrappers.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/services/incremental/ServiceWrappers.h b/services/incremental/ServiceWrappers.h index 6b0f59e9c3e5..37928308b506 100644 --- a/services/incremental/ServiceWrappers.h +++ b/services/incremental/ServiceWrappers.h @@ -55,13 +55,12 @@ public: class DataLoaderManagerWrapper { public: virtual ~DataLoaderManagerWrapper() = default; - virtual binder::Status initializeDataLoader( + virtual binder::Status bindToDataLoader( MountId mountId, const content::pm::DataLoaderParamsParcel& params, - const content::pm::FileSystemControlParcel& control, const sp<content::pm::IDataLoaderStatusListener>& listener, bool* result) const = 0; virtual binder::Status getDataLoader(MountId mountId, sp<content::pm::IDataLoader>* result) const = 0; - virtual binder::Status destroyDataLoader(MountId mountId) const = 0; + virtual binder::Status unbindFromDataLoader(MountId mountId) const = 0; }; class IncFsWrapper { |