Last Comment Bug 717203 - Remove old speculative HTML parsing code
: Remove old speculative HTML parsing code
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: HTML: Parser (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla12
Assigned To: Henri Sivonen (:hsivonen) (Not doing reviews or reading bugmail until 2016-08-01)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-11 03:51 PST by Henri Sivonen (:hsivonen) (Not doing reviews or reading bugmail until 2016-08-01)
Modified: 2012-01-12 08:33 PST (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Remove obsolete speculation code (26.09 KB, patch)
2012-01-11 03:53 PST, Henri Sivonen (:hsivonen) (Not doing reviews or reading bugmail until 2016-08-01)
mrbkap: review+
Details | Diff | Splinter Review

Description Henri Sivonen (:hsivonen) (Not doing reviews or reading bugmail until 2016-08-01) 2012-01-11 03:51:29 PST
Get rid of the old speculative HTML parsing code. It's not used for XML anyway and it doesn't make sense to run it for about:blank.
Comment 1 Henri Sivonen (:hsivonen) (Not doing reviews or reading bugmail until 2016-08-01) 2012-01-11 03:53:49 PST
Created attachment 587651 [details] [diff] [review]
Remove obsolete speculation code
Comment 2 Blake Kaplan (:mrbkap) 2012-01-11 04:40:47 PST
Comment on attachment 587651 [details] [diff] [review]
Remove obsolete speculation code

It makes me sad to review this, but it's time to move on. That being said, the Holder class (just above nsSpeculativeScriptThread) can be deleted as well.
Comment 3 Henri Sivonen (:hsivonen) (Not doing reviews or reading bugmail until 2016-08-01) 2012-01-11 07:52:41 PST
Thanks. Landed with Holder removed, too.
https://hg.mozilla.org/integration/mozilla-inbound/rev/39e49fcaed22
Comment 4 Matt Brubeck (:mbrubeck) 2012-01-12 08:33:52 PST
https://hg.mozilla.org/mozilla-central/rev/39e49fcaed22

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