Add a null check for PermissionBubbleManager::FromWebcontents.

There is a small chance that the permission bubble manager for a given
web contents has been deleted before an asychronous permission request
has been handled.

BUG=371085

Review URL: https://codereview.chromium.org/282693002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269987 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/chrome_quota_permission_context.cc

@@ -283,11 +283,13 @@ void ChromeQuotaPermissionContext::RequestQuotaPermission(
   if (PermissionBubbleManager::Enabled()) {
     PermissionBubbleManager* bubble_manager =
         PermissionBubbleManager::FromWebContents(web_contents);
-    bubble_manager->AddRequest(new QuotaPermissionRequest(this,
-            params.origin_url, params.requested_size, params.user_gesture,
-            Profile::FromBrowserContext(web_contents->GetBrowserContext())->
-                GetPrefs()->GetString(prefs::kAcceptLanguages),
-            callback));
+    if (bubble_manager) {
+      bubble_manager->AddRequest(new QuotaPermissionRequest(this,
+              params.origin_url, params.requested_size, params.user_gesture,
+              Profile::FromBrowserContext(web_contents->GetBrowserContext())->
+                  GetPrefs()->GetString(prefs::kAcceptLanguages),
+              callback));
+    }
     return;
   }
 

chrome/browser/geolocation/chrome_geolocation_permission_context.cc

@@ -242,9 +242,11 @@ void ChromeGeolocationPermissionContext::DecidePermission(
       if (PermissionBubbleManager::Enabled()) {
         PermissionBubbleManager* mgr =
             PermissionBubbleManager::FromWebContents(web_contents);
-        mgr->AddRequest(new GeolocationPermissionRequest(
-                this, id, requesting_frame, user_gesture, callback,
-                profile_->GetPrefs()->GetString(prefs::kAcceptLanguages)));
+        if (mgr) {
+          mgr->AddRequest(new GeolocationPermissionRequest(
+                  this, id, requesting_frame, user_gesture, callback,
+                  profile_->GetPrefs()->GetString(prefs::kAcceptLanguages)));
+        }
       } else {
         // setting == ask. Prompt the user.
         QueueController()->CreateInfoBarRequest(

chrome/browser/media/chrome_midi_permission_context.cc

@@ -199,10 +199,12 @@ void ChromeMidiPermissionContext::DecidePermission(
       if (PermissionBubbleManager::Enabled()) {
         PermissionBubbleManager* bubble_manager =
             PermissionBubbleManager::FromWebContents(web_contents);
-        bubble_manager->AddRequest(new MidiPermissionRequest(
-            this, id, requesting_frame, user_gesture,
-            profile_->GetPrefs()->GetString(prefs::kAcceptLanguages),
-            callback));
+        if (bubble_manager) {
+          bubble_manager->AddRequest(new MidiPermissionRequest(
+              this, id, requesting_frame, user_gesture,
+              profile_->GetPrefs()->GetString(prefs::kAcceptLanguages),
+              callback));
+        }
         return;
       }
 

chrome/browser/media/media_capture_devices_dispatcher.cc

@@ -75,7 +75,7 @@ const content::MediaStreamDevice* FindDeviceWithId(
     }
   }
   return NULL;
-};
+}
 
 // This is a short-term solution to grant camera and/or microphone access to
 // extensions:
@@ -631,8 +631,10 @@ void MediaCaptureDevicesDispatcher::ProcessQueuedAccessRequest(
                        base::Unretained(this), web_contents)));
     if (controller->DismissInfoBarAndTakeActionOnSettings())
       return;
-    PermissionBubbleManager::FromWebContents(web_contents)->
-        AddRequest(controller.release());
+    PermissionBubbleManager* bubble_manager =
+        PermissionBubbleManager::FromWebContents(web_contents);
+    if (bubble_manager)
+      bubble_manager->AddRequest(controller.release());
     return;
   }
 

chrome/browser/notifications/desktop_notification_service.cc

@@ -514,12 +514,14 @@ void DesktopNotificationService::RequestPermission(
     if (PermissionBubbleManager::Enabled()) {
       PermissionBubbleManager* bubble_manager =
           PermissionBubbleManager::FromWebContents(web_contents);
-      bubble_manager->AddRequest(new NotificationPermissionRequest(
-          this,
-          origin,
-          DisplayNameForOriginInProcessId(
-              origin, render_frame_host->GetProcess()->GetID()),
-          callback));
+      if (bubble_manager) {
+        bubble_manager->AddRequest(new NotificationPermissionRequest(
+            this,
+            origin,
+            DisplayNameForOriginInProcessId(
+                origin, render_frame_host->GetProcess()->GetID()),
+            callback));
+      }
       return;
     }