[Regression] mw.loader.addEmbeddedCSS throws cssText.indexOf is not a function
Closed, ResolvedPublic

Description

(Quote from bug 46401 comment #2)

http://commons.wikimedia.org/wiki/Commons:
FORUM#komprimiertes_JavaScript_kaputt.3F
https://commons.wikimedia.org/wiki/Commons:
Village_pump#Strange_problem_with_CSS
https://en.wikipedia.org/wiki/Wikipedia:
Village_pump_%28technical%29#CSS_errors_with_search_box.2C_etc..3F
https://en.wikipedia.org/wiki/Wikipedia:
Village_pump_%28technical%29#Errors_with_editing_interface_and_sidebar

Only affects Firefox.


Version: 1.21.x
Severity: critical

bzimport set Reference to bz46575.
Krinkle created this task.Via LegacyMar 26 2013, 5:57 PM
Krinkle added a comment.Via ConduitMar 26 2013, 6:01 PM

Caused by passing addEmbeddedCSS to setTimeout directly.

Which is a problem in Firefox (below version 13) due its non-standard
behaviour of passing a numerical "lateness" value as first argument to this
callback.

See also
http://benalman.com/news/2009/07/the-mysterious-firefox-settime/

Change-Id: I80af730daa815f0c273fe942c570d1f0144bbbb1

Krinkle added a comment.Via ConduitMar 26 2013, 6:18 PM

(In reply to comment #1)

Change-Id: I80af730daa815f0c273fe942c570d1f0144bbbb1

Merged and deployed to 1.21wmf12.

gerritbot added a comment.Via ConduitApr 17 2013, 1:24 AM

Related URL: https://gerrit.wikimedia.org/r/59571 (Gerrit Change I80af730daa815f0c273fe942c570d1f0144bbbb1)

Add Comment

Column Prototype
This is a very early prototype of a persistent column. It is not expected to work yet, and leaving it open will activate other new features which will break things. Press "\" (backslash) on your keyboard to close it now.