[webkit-reviews] review denied: [Bug 24625] Accessor to the selected index needed in PopupMenuChromium : [Attachment 28657] Adds the selectedIndex accessor to PopupMenuChromium

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 17 15:38:09 PDT 2009


Dimitri Glazkov (Google) <dglazkov at chromium.org> has denied Jay Campan
<jcampan at google.com>'s request for review:
Bug 24625: Accessor to the selected index needed in PopupMenuChromium
https://bugs.webkit.org/show_bug.cgi?id=24625

Attachment 28657: Adds the selectedIndex accessor to PopupMenuChromium
https://bugs.webkit.org/attachment.cgi?id=28657&action=review

------- Additional Comments from Dimitri Glazkov (Google)
<dglazkov at chromium.org>
> +++ WebCore/platform/chromium/PopupMenuChromium.h	(working copy)
> @@ -107,6 +107,10 @@ namespace WebCore {
>  
>	   PopupListBox* listBox() const { return m_listBox.get(); }
>  
> +	// Gets the index of the item that the user is currently moused-over or

> +	// has selected with the keyboard up/down arrows.
> +	int selectedIndex() const;
> +

ident fail? :)


More information about the webkit-reviews mailing list