Skip to content

Instantly share code, notes, and snippets.

@pier-oliviert

pier-oliviert/bug.js

Last active August 29, 2015 14:13
Show Gist options

  • Select an option

    Learn more about clone URLs
  • Save pier-oliviert/ae0320dce453b414309e to your computer and use it in GitHub Desktop.

Select an option

Learn more about clone URLs
Save pier-oliviert/ae0320dce453b414309e to your computer and use it in GitHub Desktop.
Download ZIP
Raw
bug.js
var div = document.createElement('div')
div.innerHTML = "<script></script>"
var s = div.children[0]
s.src = "//connect.facebook.net/en_US/all.js#xfbml=1"
// true
s instanceof HTMLScriptElement
// Will NOT load the script
document.head.appendChild(s)
Raw
expected.js
var s = document.createElement('script')
s.src = "//connect.facebook.net/en_US/all.js#xfbml=1"
// true
s instanceof HTMLScriptElement
// Will load the script
document.head.appendChild(s)
@pier-oliviert
Copy link
Author

pier-oliviert commented Jan 14, 2015

@david-treblig you are right I don't think it's a webkit bug either (tested on FF & safari too and I actually made that distinction when I submitted the bug over to chromium.)

I also think that the node is internally marked differently. I really think this is unintended behavior, but as my bug report are usually dismissed, I wouldn't be surprised if they dismiss it.

@pier-oliviert
Copy link
Author

pier-oliviert commented Jan 14, 2015

Thanks for the follow up David, I'm going to add your findings to the bug report https://code.google.com/p/chromium/issues/detail?id=448783&colspec=ID%20Pri%20M%20Week%20ReleaseBlock%20Cr%20Status%20Owner%20Summary%20OS%20Modified#makechanges

@pier-oliviert
Copy link
Author

pier-oliviert commented Jan 14, 2015

Dah, just found this http://www.w3.org/TR/2008/WD-html5-20080610/dom.html#innerhtml0

That's why.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment