Introduce WorkerStoragePartitionId and use it in SharedWorkerInstance.

content/browser/shared_worker/shared_worker_message_filter.h

 // 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.
 
 #ifndef CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_MESSAGE_FILTER_H_
 #define CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_MESSAGE_FILTER_H_
 
 #include "content/browser/worker_host/worker_storage_partition.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/browser_message_filter.h"
@@ skipping 50 matching lines @@
   void OnAllowFileSystem(int worker_route_id,
                          const GURL& url,
                          bool* result);
   void OnAllowIndexedDB(int worker_route_id,
                         const GURL& url,
                         const base::string16& name,
                         bool* result);
 
   const int render_process_id_;
   ResourceContext* const resource_context_;
-  const WorkerStoragePartition partition_;
+  const WorkerStoragePartitionId partition_id_;

[michaeln, 2014/04/24 08:04:32]

The bug has to do with the devtools holding onto references to these objects.
Why are these changes being made to the  message filter class and to the
instances held by the sharedworkerservice to no longer hold references to these
objects?

[horo, 2014/04/24 08:23:15]

Done.
Oh yes.
SharedWorkerMessageFilter should hold WorkerStoragePartition. 

   MessagePortMessageFilter* const message_port_message_filter_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(SharedWorkerMessageFilter);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_MESSAGE_FILTER_H_