GDataDB support with leveldb.

chrome/browser/chromeos/gdata/gdata_db_factory.h

Index: chrome/browser/chromeos/gdata/gdata_db_factory.h
===================================================================
--- chrome/browser/chromeos/gdata/gdata_db_factory.h	(revision 0)
+++ chrome/browser/chromeos/gdata/gdata_db_factory.h	(revision 0)
@@ -0,0 +1,25 @@
+// 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 CHROME_BROWSER_CHROMEOS_GDATA_GDATA_DB_FACTORY_H_
+#define CHROME_BROWSER_CHROMEOS_GDATA_GDATA_DB_FACTORY_H_
+#pragma once
+
+#include "base/memory/scoped_ptr.h"
+
+class FilePath;
+
+namespace gdata {
+
+class GDataDB;
+
+// Factory class to create a GDataDB instance at |db_path|.
+class GDataDBFactory {

[satorux1, 2012/04/24 18:26:44]

If all functions are static, we don't need a class. I'd create db_factory
namespace here instead.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Nonmem...

Rather than creating classes only to group static member functions which do not
share static data, use namespaces instead.

[achuithb, 2012/04/24 19:43:58]

Done.

+ public:
+  static scoped_ptr<GDataDB> CreateGDataDB(const FilePath& db_path);
+};

[satorux1, 2012/04/24 18:26:44]

if we are to use the class, we should have private: and
DISALLOW_COPY_AND_ASSIGN. namespace is more concise. :)

[achuithb, 2012/04/24 19:43:58]

Done.

+
+}  // namespace gdata
+
+#endif  // CHROME_BROWSER_CHROMEOS_GDATA_GDATA_DB_FACTORY_H_
Property changes on: chrome/browser/chromeos/gdata/gdata_db_factory.h
___________________________________________________________________
Added: svn:eol-style
   + LF