Add more device names to the list of known devices.

chrome/browser/extensions/api/music_manager_private/device_id_linux.cc

 // Copyright 2013 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/extensions/api/music_manager_private/device_id.h"
 
 #include <map>
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/files/file_enumerator.h"
 #include "base/files/file_path.h"
 #include "content/public/browser/browser_thread.h"
 
 namespace {
 
 const char kDiskByUuidDirectoryName[] = "/dev/disk/by-uuid";
-const char* kDeviceNames[] = { "sda1", "hda1", "dm-0" };
+const char* kDeviceNames[] = {
+  "sda1", "hda1", "dm-0", "xvda1", "sda2", "hda2", "dm-1", "xvda2",

[Marijn Kruisselbrink, 2014/01/06 19:51:30]

Would this be more flexible/future proof if instead of hard-coding the full
device name including index this only hard-coded the part without the index (so
{"sda", "hda", "dm-", "xvda"})? Then you could just loop over some short range
of indices and try them all. But maybe I misunderstand what the desired behavior
of this code is.

[rpaquay, 2014/01/06 20:47:16]

Yes, I'd think if we need another change in this list, a refactor will probably
be necessary.

+};
 
 // Map from device name to disk uuid
 typedef std::map<base::FilePath, base::FilePath> DiskEntries;
 
 void GetDiskUuid(const extensions::api::DeviceId::IdCallback& callback) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
 
   DiskEntries disk_uuids;
 
   base::FileEnumerator files(base::FilePath(kDiskByUuidDirectoryName),
@@ skipping 55 matching lines @@
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
   content::BrowserThread::PostTask(
       content::BrowserThread::FILE,
       FROM_HERE,
       base::Bind(GetDiskUuid, callback));
 }
 
 }  // namespace api
 }  // namespace extensions