github-migration/prototype
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Rollback IFRAME error "feature test" for IE. Fix it another way.

Browse Source
This commit is contained in:
Andrew Dupont 2009-07-13 13:52:54 +02:00
parent 4586adff0b
commit 39f045d240
1 changed files with 19 additions and 35 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

54
src/dom/dom.js
View File

@ -593,43 +593,24 @@ Element.Methods = {
*
* Returns the value of `element`'s attribute with the given name.
**/
readAttribute: (function(){
var iframeGetAttributeThrowsError = (function(){
var el = document.createElement('iframe'),
isBuggy = false;
document.documentElement.appendChild(el);
try {
el.getAttribute('type', 2);
} catch(e) {
isBuggy = true;
}
document.documentElement.removeChild(el);
el = null;
return isBuggy;
})();
return function(element, name) {
element = $(element);
// check boolean first, to get out of expression faster
if (iframeGetAttributeThrowsError &&
name === 'type' &&
element.tagName.toUpperCase() == 'IFRAME') {
readAttribute: function(element, name) {
element = $(element);
if (Prototype.Browser.IE) {
// Circumvent issue in IE that causes crash.
if (name === 'type' &&
element.tagName.toUpperCase() == 'IFRAME') {
return element.getAttribute('type');
}
if (Prototype.Browser.IE) {
var t = Element._attributeTranslations.read;
if (t.values[name]) return t.values[name](element, name);
if (t.names[name]) name = t.names[name];
if (name.include(':')) {
return (!element.attributes || !element.attributes[name]) ? null :
element.attributes[name].value;
}
var t = Element._attributeTranslations.read;
if (t.values[name]) return t.values[name](element, name);
if (t.names[name]) name = t.names[name];
if (name.include(':')) {
return (!element.attributes || !element.attributes[name]) ? null :
element.attributes[name].value;
}
return element.getAttribute(name);
}
})(),
return element.getAttribute(name);
},
/**
* Element#writeAttribute(@element, attribute[, value = true]) -> Element
@ -1367,6 +1348,9 @@ else if (Prototype.Browser.IE) {
},
values: {
_getAttr: function(element, attribute) {
return element.getAttribute(attribute);
},
_getAttr2: function(element, attribute) {
return element.getAttribute(attribute, 2);
},
_getAttrNode: function(element, attribute) {
@ -1444,8 +1428,8 @@ else if (Prototype.Browser.IE) {
(function(v) {
Object.extend(v, {
href: v._getAttr,
src: v._getAttr,
href: v._getAttr2,
src: v._getAttr2,
type: v._getAttr,
action: v._getAttrNode,
disabled: v._flag,