Upstream: Build net_unittests for Android.

net/base/platform_mime_util_linux.cc

Index: net/base/platform_mime_util_linux.cc
diff --git a/net/base/platform_mime_util_linux.cc b/net/base/platform_mime_util_linux.cc
index c841914dde778d9ac6850620f3e910382ac1d7f7..e0ff7c2f22962c4226f8411d110807f3d6ba654c 100644
--- a/net/base/platform_mime_util_linux.cc
+++ b/net/base/platform_mime_util_linux.cc
@@ -20,7 +20,9 @@ namespace net {
 #if defined(OS_ANDROID)
 bool PlatformMimeUtil::GetPlatformMimeTypeFromExtension(
     const FilePath::StringType& ext, std::string* result) const {
-  return android::GetMimeTypeFromExtension(ext, result);
+  // TODO(jingzhao): Replace "false" with
+  // "android::GetMimeTypeFromExtension(ext, result)" once we support JNI.

[joth, 2011/11/02 18:57:05]

is NOTIMPLEMENTED() too spammy?

[Jing Zhao, 2011/11/03 17:49:08]

Does it look better now?

+  return false;
 }
 #elif defined(USE_AURA)
 bool PlatformMimeUtil::GetPlatformMimeTypeFromExtension(
@@ -31,7 +33,7 @@ bool PlatformMimeUtil::GetPlatformMimeTypeFromExtension(
 #else
 bool PlatformMimeUtil::GetPlatformMimeTypeFromExtension(
     const FilePath::StringType& ext, std::string* result) const {
-  // TODO(thestig) This is a temporary hack until we can fix this
+  // TODO(thestig): This is a temporary hack until we can fix this

[joth, 2011/11/02 18:57:05]

side note, the : is optional in
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#TODO_Comments
(but agree it's more common to have them)

[Jing Zhao, 2011/11/03 17:49:08]

:)

   // properly in test shell / webkit.
   // We have to play dumb and not return application/x-perl here
   // to make the reload-subframe-object layout test happy.