Skip to content

Firefox Bug 565031

April 12, 2011

Over the past 2 weeks, David Humphrey asked the class to work on a firefox bug collaboratively.  The bug is located here.  It basically states that the is_space() function in nsImageMap.cpp accepts characters other then spaces when it shouldn’t.  Before any patching, is_space() currently accepts characters such as tabs, vertical tabs, newline characters, carriage feeds, and also doesn’t work with semicolons, which it should.  One of my classmates, Matt Postill, came up with a patch last week that fixed this.  It now accepts only the correct input.  Matt’s patch can be found here.

Since a patch had been created, the next thing that needed to be done was write unit tests, as no patch would be accepted without them.  After writing a unit test, it is run using mozillas Mochitest.  My first time using mochitest I accidentally ran all the tests at once, resulting in an extremely long wait, don’t make the same mistake (unless you want to of course), you can also run tests individually by doing the following:

TEST_PATH=path/to/your/test/here.html make -C (whatever your objdir-ff is) mochitest-plain

Doing so allows you to run just a single test at a time, which is much more convenient for us.  My test for the bug can be found here.

Advertisements

From → school

Leave a Comment

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out / Change )

Twitter picture

You are commenting using your Twitter account. Log Out / Change )

Facebook photo

You are commenting using your Facebook account. Log Out / Change )

Google+ photo

You are commenting using your Google+ account. Log Out / Change )

Connecting to %s

%d bloggers like this: