Fixed problem in Thunderbird resulting from Yahoo-specific change las…

…t night
SHAREVIEW · Feb 5, 2013 · dfcecf6 · dfcecf6
1 parent 5db0583
commit dfcecf6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/common/markdown-render.js b/src/common/markdown-render.js
@@ -112,7 +112,7 @@
     // Yahoo seems to often/always/sometimes (only in Chrome?) use <p> instead
     // of <div>. We'll replace the former with the latter so that our other rules work.
     // TODO: Figure out if it's more than Yahoo that's a problem.
-    if (url.match(/\.yahoo\./i)) {
+    if (url && url.match(/\.yahoo\./i)) {
       preprocessInfo.html =
         preprocessInfo.html
           .replace(/<p\b[^>]*>/ig, '<div>')

diff --git a/src/firefox/chrome/content/ff-overlay.js b/src/firefox/chrome/content/ff-overlay.js
@@ -174,7 +174,7 @@ var markdown_here = {
           hljs.hljs,
           html,
           targetDocument,
-          targetDocument.location.href),
+          targetDocument.location ? targetDocument.location.href : null),
         prefs['main-css'] + prefs['syntax-css']);
     });
   },