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

chrome/browser/system_monitor/removable_device_constants.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 "chrome/browser/system_monitor/removable_device_constants.h"
 
 namespace chrome {
 
 const char kFSUniqueIdPrefix[] = "UUID:";
 const char kVendorModelSerialPrefix[] = "VendorModelSerial:";
 
 #if defined(OS_LINUX)
 const char kVendorModelVolumeStoragePrefix[] = "VendorModelVolumeStorage:";
 #endif
 
 #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