Hello,

On Tue, 2010-07-06 at 11:13 -0700, Alexey Proskuryakov wrote:
> We do sometimes land changes that are reviewed on IRC and that don't
> correspond to bugs. The main criteria for when this is appropriate
> are:
> - the change is simple, and doesn't need many eyes looking at it;
> - there is no historical trail to maintain, no one is going to look at
> svn blame and wonder why this change was made 10 years ago.
> 
> 
> A change that modifies cross platform code to fix a platform specific
> (?) crash, and that doesn't even include a test case definitely needs
> to be tracked in WebKit Bugzilla.

You are right. I at first believed the bug report I mentioned along with
the change description would suffice (since mentioning
non-webkit-bugzilla trackers is fairly common, and the trace is pretty
straightforward), but I believe for this change a webkit bug with a bit
more information on the problem we're addressing is in order indeed!

I have opened https://bugs.webkit.org/show_bug.cgi?id=41710 and will add
that as reference to the ChangeLog.

Thanks!

-- 
Gustavo Noronha Silva <[email protected]>
Collabora Ltd.

_______________________________________________
webkit-dev mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Reply via email to