[Pkg-mozext-commits] [firebug] 16/68: Simplify test for NamedNodeMap (related to the previous commit and issue 6315)

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to tag fbtest-1.11.4
in repository firebug.

commit 7dd02c019bb4eba164b2ac637b206cf16c7c325d
Author: Simon Lindholm <simon.lindholm10 at gmail.com>
Date:   Thu Mar 14 14:36:18 2013 +0100

    Simplify test for NamedNodeMap (related to the previous commit and issue 6315)
    
    http://code.google.com/p/fbug/issues/detail?id=6315
---
 extension/content/firebug/chrome/reps.js | 19 +++----------------
 1 file changed, 3 insertions(+), 16 deletions(-)

diff --git a/extension/content/firebug/chrome/reps.js b/extension/content/firebug/chrome/reps.js
index 0d8e1a8..0ef9f02 100644
--- a/extension/content/firebug/chrome/reps.js
+++ b/extension/content/firebug/chrome/reps.js
@@ -3184,22 +3184,9 @@ FirebugReps.NamedNodeMap = domplate(Firebug.Rep,
 
     supportsObject: function(object, type)
     {
-        try
-        {
-            return (object instanceof window.NamedNodeMap);
-        }
-        catch (err)
-        {
-            // Fails in Firefox 22
-        }
-
-        try
-        {
-            return (object instanceof window.MozNamedAttrMap);
-        }
-        catch (err)
-        {
-        }
+        // NamedNodeMap is no more since Fx 22 - see https://bugzilla.mozilla.org/show_bug.cgi?id=847195.
+        // The temporary Attr-only replacement is MozNamedAttrMap.
+        return (object instanceof (window.NamedNodeMap || window.MozNamedAttrMap));
     },
 
     // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * //

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/firebug.git