On Wed, Dec 8, 2010 at 4:06 PM, Steven Roussey <[email protected]> wrote: > Oops, I didn't mean for that to be so short! > > This is probably my first time posting to this list (ok, now second), > so let me know what I might be doing correctly, or incorrectly. I want > to contribute to webkit and web inspector in the future, so hopefully > this will be a learning exercise. Though in this particular case, the > code is not mine. The original contributor to the Web Inspector bug > got Firebug people interested and it got deployed with us first. I had > thought that with Chrome's fast turn-arounds, that it was already > there, but I was wrong. Seems the contributor gave up: > > https://bugs.webkit.org/show_bug.cgi?id=38664#c52 > > I think the idea that the API needs wider review on the webkit mailing > list is now moot. > > What would I need to do to to move this forward? > > Remember, this is my first time!
Hi Steve, I would suggest manually finding qualified reviewers for the patch, and asking them via email whether they can look at it. A quick search through WebCore/ChangeLog for "inspector/" turns up a couple of reviewers of earlier patches. Also, a recent addition to the webkit-patch script is "webkit-patch upload --suggest-reviewers" which automates this process. -Ken > Thanks! > > -steve-- > > > > Steven Roussey > > > On Wed, Dec 8, 2010 at 3:45 PM, Steven Roussey <[email protected]> wrote: >> We released Firebug 1.6.0 with console.table(), might it now be time >> for Web Inspector to implement it? >> >> Steven Roussey >> > _______________________________________________ > webkit-dev mailing list > [email protected] > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev > _______________________________________________ webkit-dev mailing list [email protected] http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
