[Webkit-unassigned] [Bug 10893] InsertRule can not handle @import statements

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=10893


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #10822|review?                     |review-
               Flag|                            |




------- Comment #6 from darin at apple.com  2006-09-28 09:08 PDT -------
(From update of attachment 10822)
The bug fix looks great.

But I don't understand why css-insert-import-rule.html ignores exceptions.
Either it should expect an exception, and fail if the expected one doesn't
happen, or expect no exception, and fail if an exception occurs.

Hiding whether an exception happened or not is not a good choice for a test.

Another way to do the grammar is to have a non-terminal that's either a ruleset
or an import, then we would not have to repeat the assignment statement.

I'm going to review- just for the exception issue in the test, but this change
otherwise looks great!


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.