Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Unified Diff: content/renderer/background_sync/background_sync_client_impl.h

Issue 1575763002: Make `content` target build without unused data members on Linux. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@use_disallow_copy_assign_linux_only
Patch Set: fix unit test Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/background_sync/background_sync_client_impl.h
diff --git a/content/renderer/background_sync/background_sync_client_impl.h b/content/renderer/background_sync/background_sync_client_impl.h
index 78d1a1a463247b9ddf193ef2817fa8628591353d..3b209b343e01f2fdfdcf5063120c87071977f7dc 100644
--- a/content/renderer/background_sync/background_sync_client_impl.h
+++ b/content/renderer/background_sync/background_sync_client_impl.h
@@ -21,15 +21,13 @@ class CONTENT_EXPORT BackgroundSyncClientImpl
: public NON_EXPORTED_BASE(BackgroundSyncServiceClient) {
public:
static void Create(
- int64_t service_worker_registration_id,
mojo::InterfaceRequest<BackgroundSyncServiceClient> request);
~BackgroundSyncClientImpl() override;
private:
using SyncCallback = mojo::Callback<void(ServiceWorkerEventStatus)>;
- BackgroundSyncClientImpl(
- int64_t service_worker_registration_id,
+ explicit BackgroundSyncClientImpl(
mojo::InterfaceRequest<BackgroundSyncServiceClient> request);
// BackgroundSyncServiceClient methods:
@@ -42,7 +40,6 @@ class CONTENT_EXPORT BackgroundSyncClientImpl
BackgroundSyncError error,
SyncRegistrationPtr registration);
- int64_t service_worker_registration_id_;
mojo::StrongBinding<BackgroundSyncServiceClient> binding_;
int64_t callback_seq_num_;

Powered by Google App Engine
This is Rietveld 408576698