[Media Gallery] Added code to support mtp device media file system on Windows.

chrome/browser/system_monitor/removable_device_constants.cc

Index: chrome/browser/system_monitor/removable_device_constants.cc
diff --git a/chrome/browser/system_monitor/removable_device_constants.cc b/chrome/browser/system_monitor/removable_device_constants.cc
index df2083c95a42ed58a4a42270f5e955534b033eeb..1e268932b7bde9ce7b9a021930c4eecfeef1ed8e 100644
--- a/chrome/browser/system_monitor/removable_device_constants.cc
+++ b/chrome/browser/system_monitor/removable_device_constants.cc
@@ -15,6 +15,16 @@ const char kVendorModelVolumeStoragePrefix[] = "VendorModelVolumeStorage:";
 
 #if defined(OS_WIN)
 const char16 kWPDDevInterfaceGUID[] = L"{6ac27878-a6fa-4155-ba85-f98f491d4f33}";
+
+const char16 kbmpFormat[] = L".bmp";

[Lei Zhang, 2012/11/01 02:34:57]

kBmpFormat, ditto below.

[Peter Kasting, 2012/11/01 04:17:37]

kBMPFormat would be even better.

But must you define these in this file?  Can't we just inline these into the one
function where they're used?  It would be more readable and fewer symbols.

We should avoid extern-linkage constants unless there is no other way.

[kmadhusu, 2012/11/01 17:59:00]

It is not required to define these symbols in this file.  Removed the constants
from this file and inlined the values directly in the function.

[kmadhusu, 2012/11/01 17:59:00]

Inlined the values directly in the function.

+const char16 kexifFormat[] = L".jpg";
+const char16 kgifFormat[] = L".gif";
+const char16 kjfifFormat[] = L".jfif";
+const char16 kjpegxrFormat[] = L".jxr";
+const char16 kjpxFormat[] = L".jpx";
+const char16 kjp2Format[] = L".jp2";
+const char16 kpictFormat[] = L".pic";
+const char16 kpngFormat[] = L".png";
 #endif
 
 }  // namespace chrome