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

bugzilla-daemon at opendarwin.org bugzilla-daemon at opendarwin.org
Thu Sep 28 09:08:27 PDT 2006


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.

More information about the webkit-unassigned mailing list