Tweak PathProviderPosix's FILE_EXE and FILE_MODULE handling on FreeBSD.

webkit/glue/webkitplatformsupport_impl.cc

Index: webkit/glue/webkitplatformsupport_impl.cc
diff --git a/webkit/glue/webkitplatformsupport_impl.cc b/webkit/glue/webkitplatformsupport_impl.cc
index bfa6ec04f871c0d6ad6772dece793af15261d94c..0ab77d3a87c3e29dd7cbd8ca1e290020183404d3 100644
--- a/webkit/glue/webkitplatformsupport_impl.cc
+++ b/webkit/glue/webkitplatformsupport_impl.cc
@@ -424,7 +424,7 @@ WebData WebKitPlatformSupportImpl::loadResource(const char* name) {
   // Some clients will call into this method with an empty |name| when they have
   // optional resources.  For example, the PopupMenuChromium code can have icons
   // for some Autofill items but not for others.
-  if (!strlen(name))
+  if (name == NULL || !strlen(name))

[wtc, 2011/10/25 00:07:56]

I would exclude this change.  It may be wrong to pass a null
|name| argument to this function, and it may be better to crash
in that case.

[grt (UTC plus 2), 2011/10/25 13:21:50]

Done.

[wtc, 2011/10/25 17:42:30]

Just FYI: my quick experiments showed that strlen(NULL) crashes
correctly on the three major platforms (Linux, Mac, Windows).

     return WebData();
 
   // Check the name prefix to see if it's an audio resource.