Use std::default_delete as the default deleter for scoped_ptr.

content/browser/service_worker/service_worker_process_manager.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.
 
 #include "content/browser/service_worker/service_worker_process_manager.h"
 
 #include "content/browser/renderer_host/render_process_host_impl.h"
 #include "content/browser/service_worker/service_worker_context_wrapper.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/site_instance.h"
@@ skipping 249 matching lines @@
   std::sort(counted.begin(), counted.end(), SecondGreater());
 
   std::vector<int> result(counted.size());
   for (size_t i = 0; i < counted.size(); ++i)
     result[i] = counted[i].first;
   return result;
 }
 
 }  // namespace content
 
-namespace base {
+namespace std {
 // Destroying ServiceWorkerProcessManagers only on the UI thread allows the
 // member WeakPtr to safely guard the object's lifetime when used on that
 // thread.
-void DefaultDeleter<content::ServiceWorkerProcessManager>::operator()(
+void default_delete<content::ServiceWorkerProcessManager>::operator()(

[dcheng, 2015/11/16 18:22:59]

This is permitted by the C++ standard but probably looks a bit wonky. From
http://en.cppreference.com/w/cpp/language/extending_std:

"It is allowed to add template specializations for any standard library template
to the namespace std only if the declaration depends on a user-defined type and
the specialization satisfies all requirements for the original template, except
where such specializations are prohibited."

[danakj, 2015/11/16 22:19:03]

Oh.. thanks for the linky!

Why is this written as a function overload, but the previous one is written as a
template specialization?

[dcheng, 2015/11/17 17:08:42]

This is the definition of operator() in the previous template specialization.

[danakj, 2015/11/17 19:09:27]

Er, oh yes. Magnets..

     content::ServiceWorkerProcessManager* ptr) const {
   content::BrowserThread::DeleteSoon(
       content::BrowserThread::UI, FROM_HERE, ptr);
 }
-}  // namespace base
+}  // namespace std