Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-02 15:15:23 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 155062. View background image fails/works incorrectly when things have a background attribute, but a different specified style. We should always get the computed background-image and ignore what

Browse Source 
the background attribute says.  r=bzbarsky sr=blake
This commit is contained in:
caillon%returnzero.com 2006-07-29 05:41:11 +00:00
parent d7d266c39f
commit 140cbf5772
1 changed files with 13 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
suite/common/nsContextMenu.js
View File

@ -277,12 +277,11 @@ nsContextMenu.prototype = {
// pages with multiple <body>s are lame. we'll teach them a lesson.
var bodyElt = this.target.ownerDocument.getElementsByTagName("body")[0];
if ( bodyElt ) {
var attr = bodyElt.getAttribute( "background" );
if ( attr ||
( attr = this.getComputedURL( bodyElt, "background-image" ) ) ) {
var computedURL = this.getComputedURL( bodyElt, "background-image" );
if ( computedURL ) {
this.hasBGImage = true;
this.bgImageURL = this.makeURLAbsolute( bodyElt.baseURI,
attr );
computedURL );
}
}
} else if ( "HTTPIndex" in _content &&
@ -382,17 +381,16 @@ nsContextMenu.prototype = {
}
}
// Background image? We don't bother if we've already found a background
// image further down the hierarchy. Otherwise, we look for background=
// attribute on html elements that support that, or, background-image style.
var bgImgUrl = null;
if ( !this.hasBGImage &&
( ( localname.search( /^(?:TD|TH|TABLE|BODY)$/ ) != -1 &&
( bgImgUrl = elem.getAttribute( "background" ) ) ) ||
( bgImgUrl = this.getComputedURL( elem, "background-image" ) ) ) ) {
this.hasBGImage = true;
this.bgImageURL = this.makeURLAbsolute( elem.baseURI,
bgImgUrl );
// Background image? Don't bother if we've already found a
// background image further down the hierarchy. Otherwise,
// we look for the computed background-image style.
if ( !this.hasBGImage ) {
var bgImgUrl = this.getComputedURL( elem, "background-image" );
if ( bgImgUrl ) {
this.hasBGImage = true;
this.bgImageURL = this.makeURLAbsolute( elem.baseURI,
bgImgUrl );
}
}
}
elem = elem.parentNode;