Last Comment Bug 569646 - remove URL parsing backup code in CSS parser
: remove URL parsing backup code in CSS parser
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: CSS Parsing and Computation (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla1.9.3a5
Assigned To: David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-02 11:04 PDT by David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
Modified: 2010-06-03 13:24 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (6.20 KB, patch)
2010-06-02 17:48 PDT, David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
zackw: review+
Details | Diff | Splinter Review

Description David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2010-06-02 11:04:27 PDT
We should remove the URL parsing backup code in the CSS parser per today's working group resolution of CSS 2.1 issue 129.

This basically means reverting http://hg.mozilla.org/mozilla-central/rev/0ff87d9d2d73 , adjusting tests to match, and recontributing the revised test to the CSS 2.1 test suite.
Comment 1 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2010-06-02 17:48:35 PDT
Created attachment 448913 [details] [diff] [review]
patch

This is mostly just backout, although not quite a straight backout.
Comment 2 Zack Weinberg (:zwol) 2010-06-02 18:07:44 PDT
Comment on attachment 448913 [details] [diff] [review]
patch

r=zwol
Comment 3 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2010-06-03 13:24:05 PDT
http://hg.mozilla.org/mozilla-central/rev/fb1e226d7c11

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