As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 704034 - loopstart, loopend, playcount attributes in no longer exist in the spec
: loopstart, loopend, playcount attributes in no longer exis...
Product: Core
Classification: Components
Component: HTML: Parser (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla11
Assigned To: Henri Sivonen (:hsivonen)
: Andrew Overholt [:overholt]
Depends on: 704058
  Show dependency treegraph
Reported: 2011-11-20 18:11 PST by Matthew Gregan [:kinetik]
Modified: 2012-02-01 13:56 PST (History)
2 users (show)
hsivonen: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Add loop and remove the old ones (80.68 KB, patch)
2011-11-21 06:01 PST, Henri Sivonen (:hsivonen)
bugs: review+
Details | Diff | Splinter Review

Description User image Matthew Gregan [:kinetik] 2011-11-20 18:11:57 PST
The loopstart, loopend, and playcount attributes were removed when the media element's looping support was changed to a simple boolean attribute "loop". is also missing the new(er) loop attribute.

I didn't understand the process to modify and regenerate this code well enough to submit a patch, but I'm happy to do so if someone can walk me through it.
Comment 1 User image Henri Sivonen (:hsivonen) 2011-11-20 23:39:49 PST
Thanks. I can take care of the removal and regeneration.
Comment 2 User image Henri Sivonen (:hsivonen) 2011-11-21 06:01:39 PST
Created attachment 575841 [details] [diff] [review]
Add loop and remove the old ones
Comment 3 User image Henri Sivonen (:hsivonen) 2011-11-21 06:02:30 PST
There's no test, because these pre-interned tokens affect the number of allocations the parser does. Hence, the effect of this patch isn't really testable.
Comment 4 User image Olli Pettay [:smaug] (review request backlog because of a work week) 2011-11-21 07:18:00 PST
Comment on attachment 575841 [details] [diff] [review]
Add loop and remove the old ones

Comment 5 User image Henri Sivonen (:hsivonen) 2011-11-22 00:31:25 PST
Comment 6 User image Ed Morley [:emorley] 2011-11-22 09:07:51 PST

Note You need to log in before you can comment on or make changes to this bug.