← Back to branch summary

~ubuntu-branches/ubuntu/karmic/webkit/karmic-proposed

« back to all changes in this revision

Viewing changes to WebCore/dom/ScriptElement.cpp

  • Committer: Bazaar Package Importer
  • Author(s): Gustavo Noronha Silva
  • Date: 2009-05-15 18:30:58 UTC
  • mfrom: (1.1.8 upstream)
  • Revision ID: james.westby@ubuntu.com-20090515183058-50q5exjo9b1kxy9s
Tags: 1.1.7-1
http://bugs.debian.org/526683
* New upstream release
* debian/libwebkit-1.0-2.symbols:
- updated with the new symbols in 1.1.7
* debian/libwebkit-dev.install, debian/libwebkit-dev.links,
  debian/rules:
- Build, and ship gtk-doc documentation (Closes: #526683)
* debian/copyright:
- updated.

Show diffs side-by-side

added added

removed removed

Lines of Context:
WebCore/dom/ScriptElement.cpp
232
232
    if (!language.isEmpty())
233
233
        return isSupportedJavaScriptLanguage(language);
234
234
 
235
 
    // No type or language is specified, so we assume the script to be JavaScript
236
 
    return true;
 
235
    // No type or language is specified, so we assume the script to be JavaScript.
 
236
    // We don't yet support setting event listeners via the 'for' attribute for scripts.
 
237
    // If there is such an attribute it's likely better to not execute the script than to do so
 
238
    // immediately and unconditionally.
 
239
    // FIXME: After <rdar://problem/4471751> / https://bugs.webkit.org/show_bug.cgi?id=16915 are resolved 
 
240
    // and we support the for syntax in script tags, this check can be removed and we should just
 
241
    // return 'true' here.
 
242
    String forAttribute = m_scriptElement->forAttributeValue();
 
243
    return forAttribute.isEmpty();
237
244
}
238
245
 
239
246
String ScriptElementData::scriptCharset() const