Reduce use of DatabaseIdentifier in Indexed DB entry points.

content/public/browser/indexed_db_context.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_INDEXED_DB_CONTEXT_H_
 #define CONTENT_PUBLIC_BROWSER_INDEXED_DB_CONTEXT_H_
 
 #include <stdint.h>
 
 #include <string>
@@ skipping 24 matching lines @@
   // Deletes all indexed db files for the given origin.
   virtual void DeleteForOrigin(const GURL& origin_url) = 0;
 
   // Copies the indexed db files from this context to another. The
   // indexed db directory in the destination context needs to be empty.
   virtual void CopyOriginData(const GURL& origin_url,
                               IndexedDBContext* dest_context) = 0;
 
   // Get the file name of the local storage file for the given origin.
   virtual base::FilePath GetFilePathForTesting(
-      const std::string& origin_id) const = 0;
+      const GURL& origin_url) const = 0;

[jochen (gone - plz use gerrit), 2016/03/10 15:46:51]

should this be Origin?

[jsbell, 2016/03/10 17:27:55]

In general, yes we should switch all of these uses from GURL to Origin.

All the storage backends use GURL internally for handling origins, so I'm first
minimizing use of DatabaseIdentifier for entry points, then will tackle GURL to
Origin later - tracked as https://crbug.com/591482

+
   // Set the task runner for tests if browser main loop is not initialized.
   virtual void SetTaskRunnerForTesting(
       base::SequencedTaskRunner* task_runner) = 0;
 
  protected:
   friend class base::RefCountedThreadSafe<IndexedDBContext>;
   virtual ~IndexedDBContext() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_INDEXED_DB_CONTEXT_H_