Step 12 of the "prepare a script" algorithm

[mbrubeck]

Implement step 12 of the prepare a script algorithm from the HTML specification, by adding code to HTMLScriptElementHelpers::prepare.

12. If the script element has an event attribute and a for attribute, then run these substeps:
13. Let for be the value of the for attribute.
14. Let event be the value of the event attribute.
15. Strip leading and trailing whitespace from event and for.
16. If for is not an ASCII case-insensitive match for the string "window", then the user agent must abort these steps at this point. The script is not executed.
17. If event is not an ASCII case-insensitive match for either the string "onload" or the string "onload()", then the user agent must abort these steps at this point. The script is not executed.

This is part of #4089.

[th0114nd]

I'm currently looking into this

[psdh]

Assuming that @th0114nd isn't still working on this(are you?).
I will try and fix this :)

[psdh]

@mbrubeck needed some help here:
I am new to servo and rust and am facing a tough time finding how to access say the 'for' attribute,
element.get_attribute(ns!(""), &atom("for")).root() ?
I was thinking something like http://pastebin.mozilla.org/8584728

Thanks :)

[mbrubeck]

element.get_attribute(ns!(""), &atom("for")).root() ?

Yes, this looks correct. After you unwrap the resulting Option, you'll need to call .r().Value() to convert it to a string, like in this example code. (If you're curious why, this page has some explanations.)

[psdh]

please check this, http://pastebin.mozilla.org/8585585
I have pasted my changes and the build error that I am getting.

[jdm]

Ah, that happens when the string isn't in a list in https://github.com/servo/string-cache/blob/master/macros/src/atom/data.rs. In that case, you can use Atom::from_slice("event") instead of atom!("event").

[psdh]

@mbrubeck: I think this can be closed, now that the PR has merged