Build fix in for linux_use_heapchecker

webkit/plugins/ppapi/quota_file_io.cc

Index: webkit/plugins/ppapi/quota_file_io.cc
diff --git a/webkit/plugins/ppapi/quota_file_io.cc b/webkit/plugins/ppapi/quota_file_io.cc
index 857f8129c6495f7319d66dee87fccbb60a76a981..852d842afb2b171b1a07e934ca64a56d89e91243 100644
--- a/webkit/plugins/ppapi/quota_file_io.cc
+++ b/webkit/plugins/ppapi/quota_file_io.cc
@@ -27,6 +27,7 @@ StorageType PPFileSystemTypeToQuotaStorageType(PP_FileSystemType type) {
       return quota::kStorageTypeUnknown;
   }
   NOTREACHED();

[erikwright (departed), 2011/08/11 14:24:00]

I wonder if removing the NOTREACHED would also work. If so, I think that would
be reasonable since (1) we can apparently rely on the compiler to tell us if
there is a non-returning path and (2) having a 'default' case is as literate as
the NOTREACHED.

Otherwise, I would probably remove the default case and the NOTREACHED and have
the final return be actually used.

+  return quota::kStorageTypeUnknown;
 }
 }  // namespace