diff options
Diffstat (limited to 'chromium/content/browser/permissions/permission_service_context.cc')
-rw-r--r-- | chromium/content/browser/permissions/permission_service_context.cc | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/chromium/content/browser/permissions/permission_service_context.cc b/chromium/content/browser/permissions/permission_service_context.cc index c3ab81294ed..cb409a27cd0 100644 --- a/chromium/content/browser/permissions/permission_service_context.cc +++ b/chromium/content/browser/permissions/permission_service_context.cc @@ -32,7 +32,7 @@ class PermissionServiceContext::PermissionSubscription { PermissionSubscription& operator=(const PermissionSubscription&) = delete; ~PermissionSubscription() { - DCHECK_NE(id_, 0); + DCHECK(id_); BrowserContext* browser_context = context_->GetBrowserContext(); if (browser_context) { PermissionControllerImpl::FromBrowserContext(browser_context) @@ -41,7 +41,7 @@ class PermissionServiceContext::PermissionSubscription { } void OnConnectionError() { - DCHECK_NE(id_, 0); + DCHECK(id_); context_->ObserverHadConnectionError(id_); } @@ -49,12 +49,12 @@ class PermissionServiceContext::PermissionSubscription { observer_->OnPermissionStatusChange(status); } - void set_id(int id) { id_ = id; } + void set_id(PermissionController::SubscriptionId id) { id_ = id; } private: PermissionServiceContext* const context_; mojo::Remote<blink::mojom::PermissionObserver> observer_; - int id_ = 0; + PermissionController::SubscriptionId id_; }; PermissionServiceContext::PermissionServiceContext( @@ -108,7 +108,7 @@ void PermissionServiceContext::CreateSubscription( } GURL requesting_origin(origin.Serialize()); - int subscription_id = + auto subscription_id = PermissionControllerImpl::FromBrowserContext(browser_context) ->SubscribePermissionStatusChange( permission_type, render_frame_host_, requesting_origin, @@ -119,7 +119,8 @@ void PermissionServiceContext::CreateSubscription( subscriptions_[subscription_id] = std::move(subscription); } -void PermissionServiceContext::ObserverHadConnectionError(int subscription_id) { +void PermissionServiceContext::ObserverHadConnectionError( + PermissionController::SubscriptionId subscription_id) { size_t erased = subscriptions_.erase(subscription_id); DCHECK_EQ(1u, erased); } |