gdata: Export PlatformFileInfo/Proto conversion functions

gdata: Export PlatformFileInfo/Proto conversion functions

These functions will be used in an upcoming patch, when
we extract file info from GDataFileProto/GDataDirectoryProto
(i.e. in some places, we need PlatFormFileInfo).

BUG=chromium-os:30251
TEST=GDataFileSystemTest.CachedFeedLoading

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=134679

[achuithb, 2012-04-30 23:51:33]

lgtm with nits.

http://codereview.chromium.org/10270027/diff/1/chrome/browser/chromeos/gdata/...
File chrome/browser/chromeos/gdata/gdata_files.h (right):

http://codereview.chromium.org/10270027/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_files.h:36: class PlatformFileInfoProto;
I'd make this the first declaration in this proto group since that's how it
appears in the proto file.

http://codereview.chromium.org/10270027/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_files.h:140: // Convert the proto
representation to the platform file.
Can we keep all the proto methods grouped together? So maybe move this up after
the declaration of FromProtoString? 

Let's also keep the style consistent (one comment for both functions, and no
newline between the declarations).

[satorux1, 2012-05-01 00:13:46]

http://codereview.chromium.org/10270027/diff/1/chrome/browser/chromeos/gdata/...
File chrome/browser/chromeos/gdata/gdata_files.h (right):

http://codereview.chromium.org/10270027/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_files.h:140: // Convert the proto
representation to the platform file.
On 2012/04/30 23:51:33, achuith.bhandarkar wrote:
> Can we keep all the proto methods grouped together? So maybe move this up
after
> the declaration of FromProtoString? 
> 
> Let's also keep the style consistent (one comment for both functions, and no
> newline between the declarations).

Done.