Move fileapi and dom_storage directories into the content namespace.

content/browser/fileapi/chrome_blob_storage_context.cc

 // Copyright (c) 2012 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/fileapi/chrome_blob_storage_context.h"
 
 #include "base/bind.h"
 #include "content/public/browser/browser_context.h"
 #include "webkit/blob/blob_storage_controller.h"
 
 using base::UserDataAdapter;
-using content::BrowserContext;
-using content::BrowserThread;
 using webkit_blob::BlobStorageController;
 
+namespace content {
+
 static const char* kBlobStorageContextKeyName = "content_blob_storage_context";

[tfarina, 2012/10/24 21:10:40]

nit: const char kBlobStorageContextKeyName[] = "content_blob_storage_context";

outside of the namespace, inside a unnamed namespace, although the unnamed
namespace is not strictly necessary, since const has internal linkage in c++.

[jam, 2012/10/24 21:19:48]

why? i think it's easier to keep all the code (including constants) inside the
namespace

 
 ChromeBlobStorageContext::ChromeBlobStorageContext() {}
 
 ChromeBlobStorageContext* ChromeBlobStorageContext::GetFor(
     BrowserContext* context) {
   if (!context->GetUserData(kBlobStorageContextKeyName)) {
     scoped_refptr<ChromeBlobStorageContext> blob =
         new ChromeBlobStorageContext();
     context->SetUserData(kBlobStorageContextKeyName,
                          new UserDataAdapter<ChromeBlobStorageContext>(blob));
@@ skipping 17 matching lines @@
 ChromeBlobStorageContext::~ChromeBlobStorageContext() {}
 
 void ChromeBlobStorageContext::DeleteOnCorrectThread() const {
   if (BrowserThread::IsMessageLoopValid(BrowserThread::IO) &&
       !BrowserThread::CurrentlyOn(BrowserThread::IO)) {
     BrowserThread::DeleteSoon(BrowserThread::IO, FROM_HERE, this);
     return;
   }
   delete this;
 }
+
+}  // namespace content