[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Gnash-commit] [SCM] Gnash branch, master, updated. 56046006b64572d2d738

From: Sandro Santilli
Subject: [Gnash-commit] [SCM] Gnash branch, master, updated. 56046006b64572d2d738f988e3f77e8cc722a0f4
Date: Sat, 06 Nov 2010 15:24:03 +0000

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Gnash".

The branch, master has been updated
       via  56046006b64572d2d738f988e3f77e8cc722a0f4 (commit)
      from  13fb267d7a2718bf2da14cc4400045a64d82c484 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------

commit 56046006b64572d2d738f988e3f77e8cc722a0f4
Author: Sandro Santilli <address@hidden>
Date:   Sat Nov 6 16:09:06 2010 +0100

    Log a debug line when browser doesn't support reading cookies

diff --git a/plugin/npapi/plugin.cpp b/plugin/npapi/plugin.cpp
index 4e28313..a4f3b5c 100644
--- a/plugin/npapi/plugin.cpp
+++ b/plugin/npapi/plugin.cpp
@@ -99,6 +99,13 @@
 #define PATH_MAX 1024
+// Macro to prevent repeated logging calls for the same
+// event
+#define LOG_ONCE(x) { \
+    static bool warned = false; \
+    if (!warned) { warned = true; x; } \
 // For scriptable plugin support
 #include "pluginScriptObject.h"
@@ -966,7 +973,10 @@ nsPluginInstance::setupCookies(const std::string& pageurl)
     // like IceWeasel on Debian lenny, which pre dates the cookie support
     // in NPAPI, you have to block all Cookie for sites like YouTube to
     // allow Gnash to work.
-    if (!NPNFuncs.getvalueforurl) return;
+    if (!NPNFuncs.getvalueforurl) {
+        LOG_ONCE( gnash::log_debug("Browser doesn't support reading cookies") 
+        return;
+    }
     // Cookie appear to drop anything past the domain, so we strip
     // that off.


Summary of changes:
 plugin/npapi/plugin.cpp |   12 +++++++++++-
 1 files changed, 11 insertions(+), 1 deletions(-)


reply via email to

[Prev in Thread] Current Thread [Next in Thread]