Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Commit

Permalink
Bug 811315, part 0: Fix setVisible() bug in the browser app and prepa…
Browse files Browse the repository at this point in the history
…re it for explicit mozasyncpanzoom-ness. r=benfrancis a=vingtetun
  • Loading branch information
joneschrisg committed Nov 22, 2012
1 parent 3a6fc76 commit f9fb021
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions apps/browser/js/browser.js
Expand Up @@ -568,7 +568,7 @@ var Browser = {
url = url.trim();
// If the address entered starts with a quote then search, if it
// contains a . or : then treat as a url, else search
return /^"|\'/.test(url) || !(/\.|\:/.test(url));
return /^"|\'/.test(url) || !(/\.|\:/.test(url)); //"
},

getUrlFromInput: function browser_getUrlFromInput(url) {
Expand Down Expand Up @@ -1131,8 +1131,8 @@ var Browser = {
tab.dom.style.top = '-999px';
return;
}
if (tab.dom.setActive) {
tab.dom.setActive(visible);
if (tab.dom.setVisible) {
tab.dom.setVisible(visible);
}
if (tab.crashed) {
this.showCrashScreen();
Expand Down Expand Up @@ -1167,6 +1167,7 @@ var Browser = {

iframe.style.top = '-999px';

iframe.setAttribute('mozasyncpanzoom', 'true');
// FIXME: content shouldn't control this directly
iframe.setAttribute('remote', 'true');

Expand Down Expand Up @@ -1706,7 +1707,7 @@ var Utils = {
span.innerHTML = span.innerHTML.replace(/\s/g, ' ');

if (escapeQuotes)
return span.innerHTML.replace(/"/g, '"').replace(/'/g, ''');
return span.innerHTML.replace(/"/g, '"').replace(/'/g, '''); //"
return span.innerHTML;
}
};
Expand Down

0 comments on commit f9fb021

Please sign in to comment.