[Webkit-unassigned] [Bug 99525] Spatial Navigation handling of space key in <select> appears to confuse listIndex and optionIndex

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Feb 14 05:30:08 PST 2014


https://bugs.webkit.org/show_bug.cgi?id=99525





--- Comment #16 from Csaba Osztrogonác <ossy at webkit.org>  2014-02-14 05:27:23 PST ---
(In reply to comment #15)
> (In reply to comment #14)
> > (From update of attachment 206469 [details] [details])
> > Cleared Joseph Pecoraro's review+ from obsolete attachment 206469 [details] [details] so that this bug does not appear in http://webkit.org/pending-commit.
> 
> I think you may have been able to keep the review+ marker, and instead move the bug to resolved / fixed. I personally find the r/cq flag useful documentation and easy to see at a glance what happened. I dislike when they are removed, that means I might have to read through all the comments to see what happened.

You can always check the history of the flags, for example: https://bugs.webkit.org/show_activity.cgi?id=99525

Tools/Scripts/webit-patch tool always remove flags when you (or commit queue)
land a patch. And it removes flags and obsolete old attachments if you upload
a new patch to a bug. The reason is not to have obsolete and/or already landed
patches in http://webkit.org/pending-commit and http://webkit.org/pending-review

webkit-patch has two commands to maintain these flags, I just ran them:
 - clean-pending-commit        Clear r+ on obsolete patches so they do not appear in the pending-commit list.
 - clean-review-queue          Clear r? on obsolete patches so they do not appear in the pending-review list.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list