Re-enabled PPAPI crash url for non-windows platforms.

ppapi/proxy/ppb_flash_proxy.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 "ppapi/proxy/ppb_flash_proxy.h"
 
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/time.h"
 #include "ppapi/c/dev/ppb_font_dev.h"
@@ skipping 294 matching lines @@
       return PP_MakeInt32(plugin_dispatcher->preferences().number_of_cpu_cores);
   }
   return PP_MakeUndefined();
 }
 
 PP_Bool PPB_Flash_Proxy::SetCrashData(PP_Instance instance,
                                       PP_FlashCrashKey key,
                                       PP_Var value) {
   switch (key) {
     case PP_FLASHCRASHKEY_URL:
-      // TODO(raymes): This may be breaking crash reporting, see:
+#if defined(OS_WIN)
+      // TODO(raymes): This may be breaking crash reporting on Windows, see:
       // See https://code.google.com/p/chromium/issues/detail?id=135480.
       // Disable temporarily.
       return PP_FALSE;
+#endif

[yzshen1, 2012/07/11 18:22:56]

nit: it is better to use a #else to wrap code for other platforms.

Besides, please explicitly include build_config.h

 
       StringVar *url_string_var(StringVar::FromPPVar(value));
       if (!url_string_var)
         return PP_FALSE;
       std::string url_string(url_string_var->value());
       PluginGlobals::Get()->plugin_proxy_delegate()->SetActiveURL(url_string);
       return PP_TRUE;
   }
   return PP_FALSE;
 }
@@ skipping 523 matching lines @@
   // It's rarely used enough that we just request this interface when needed.
   const PPB_Flash_Print_1_0* print_interface =
       static_cast<const PPB_Flash_Print_1_0*>(
           dispatcher()->local_get_interface()(PPB_FLASH_PRINT_INTERFACE_1_0));
   if (print_interface)
     print_interface->InvokePrinting(instance);
 }
 
 }  // namespace proxy
 }  // namespace ppapi