bluetooth: Remove disconnect when page hidden

content/browser/bluetooth/bluetooth_dispatcher_host.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // ID Not In Map Note:
 // A service, characteristic, or descriptor ID not in the corresponding
 // BluetoothDispatcherHost map [service_to_device_, characteristic_to_service_,
 // descriptor_to_characteristic_] implies a hostile renderer because a renderer
 // obtains the corresponding ID from this class and it will be added to the map
 // at that time.
@@ skipping 683 matching lines @@
 }
 
 void BluetoothDispatcherHost::OnGATTServerDisconnect(
     int thread_id,
     int frame_routing_id,
     const std::string& device_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   RecordWebBluetoothFunctionCall(
       UMAWebBluetoothFunction::REMOTE_GATT_SERVER_DISCONNECT);
 
-  // Make sure the origin is allowed to access the device. We perform this check

[Jeffrey Yasskin, 2016/04/04 17:06:39]

Add a comment about why we're *not* checking the origin in this call.
Specifically, frames can send a disconnect request after they've started
navigating, and then the GetOrigin() call is invalid. And we have to continue
the execution so that navigating itself disconnects?

[ortuno, 2016/04/04 18:04:11]

Done.

-  // in case a hostile renderer is trying to disconnect a device that the
-  // renderer is not allowed to access.
-  if (allowed_devices_map_.GetDeviceAddress(GetOrigin(frame_routing_id),
-                                            device_id)
-          .empty()) {
-    bad_message::ReceivedBadMessage(
-        this, bad_message::BDH_DEVICE_NOT_ALLOWED_FOR_ORIGIN);
-    return;
-  }
-
   RenderFrameHostImpl* render_frame_host =
       RenderFrameHostImpl::FromID(render_process_id_, frame_routing_id);
   WebContents* web_contents =
       WebContents::FromRenderFrameHost(render_frame_host);
   if (web_contents) {
     web_contents->SetBluetoothDeviceConnected(false);
   }
 
   // The last BluetoothGattConnection for a device closes the connection when
   // it's destroyed.
@@ skipping 896 matching lines @@
 
 bool BluetoothDispatcherHost::CanFrameAccessCharacteristicInstance(
     int frame_routing_id,
     const std::string& characteristic_instance_id) {
   return QueryCacheForCharacteristic(GetOrigin(frame_routing_id),
                                      characteristic_instance_id)
              .outcome != CacheQueryOutcome::BAD_RENDERER;
 }
 
 }  // namespace content