Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Maciej Stachowiak
On Sep 14, 2010, at 9:58 PM, Jian Li wrote: > Indeed when I implement BlobResourceHandle, I have taken the range request > support into consideration and got it work. The problem is that current media > element implementation does not route through ResourceHandle. > > However, introducing a Re

Re: [webkit-dev] Canvas component in Bugzilla

2010-09-14 Thread Darin Adler
On Sep 14, 2010, at 5:54 PM, Andreas Kling wrote: > I propose we add a "Canvas" component in Bugzilla. Thoughts? Sounds OK to me. -- Darin ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Darin Fisher
On Tue, Sep 14, 2010 at 10:09 PM, Adam Barth wrote: > One thing that Maciej suggested in #webkit was to interpose in the > same way that appcache interposes. One advantage of that approach is > economy of mechanism. > I think we have mostly done this. See BlobRegistry. > > (By the way, round

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Darin Fisher
I think it makes sense to extend ResourceHandle.cpp to access the BlobRegistry singleton in order to support blob URLs. This will work for all ports that use ResourceHandle.cpp (everyone except Chromium AFAIK). I don't think we need to overabstract this. When designing this system, we took a har

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Adam Barth
One thing that Maciej suggested in #webkit was to interpose in the same way that appcache interposes. One advantage of that approach is economy of mechanism. (By the way, roundtrips to WebCore are cheap for the WebKit layer; there isn't much reason to avoid them.) Adam On Tue, Sep 14, 2010 at

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Darin Fisher
On Tue, Sep 14, 2010 at 5:52 PM, Maciej Stachowiak wrote: > > On Sep 14, 2010, at 5:22 PM, Adam Barth wrote: > > > > > In some sense, this is analogous to adding an HTTPResourceHandle and > > implementing the HTTP protocol inside of WebCore. While its true that > > most (all?) users of WebKit wi

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Jian Li
Indeed when I implement BlobResourceHandle, I have taken the range request support into consideration and got it work. The problem is that current media element implementation does not route through ResourceHandle. However, introducing a ResourceHandle subclass does not sound like an elegant solut

Re: [webkit-dev] Can node->renderer()->node() == node ever be false?

2010-09-14 Thread Dimitri Glazkov
Ok, after following down a few rabbit holes, I can with some degree of certainty say that node->renderer()->node() == 0 can only at the end of RenderWidget::destroy(), but that is clearly not the case we're fishing for. Even if we modify the loop to start at the parent, we're jumping into the rend

Re: [webkit-dev] bugs.webkit.org autocomplete

2010-09-14 Thread Aaron Boodman
Ojan could also autoupdate the extension to be a noop. - a On Tue, Sep 14, 2010 at 7:10 PM, Adam Barth wrote: > Hi webkit-dev, > > I just landed a change to bugs.webkit.org to autocomplete email > address for committers and reviewers.  If you've been using Ojan's > awesome extension, it's the sa

Re: [webkit-dev] bugs.webkit.org autocomplete

2010-09-14 Thread Alexey Proskuryakov
14.09.2010, в 19:10, Adam Barth написал(а): > Let me know if you have feedback on the feature. Yes, I do. It's awesome! - WBR, Alexey Proskuryakov ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/w

Re: [webkit-dev] Can node->renderer()->node() == node ever be false?

2010-09-14 Thread Dimitri Glazkov
Sounds good, I will look into this. FWIW, when I remove this loop, I see no regressions in layout tests, including the tests added with the same commit as the loop. :DG< On Tue, Sep 14, 2010 at 5:55 PM, Maciej Stachowiak wrote: > > On Sep 14, 2010, at 5:41 PM, Dimitri Glazkov wrote: > >> Sorry,

[webkit-dev] bugs.webkit.org autocomplete

2010-09-14 Thread Adam Barth
Hi webkit-dev, I just landed a change to bugs.webkit.org to autocomplete email address for committers and reviewers. If you've been using Ojan's awesome extension, it's the same code, just now natively part of bugs.webkit.org thanks to William Siegrist being awesome. (Note that if you have the e

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread David Levin
On Tue, Sep 14, 2010 at 6:53 PM, Oliver Hunt wrote: > > On Sep 14, 2010, at 5:56 PM, David Levin wrote: > > On Tue, Sep 14, 2010 at 5:42 PM, Adam Barth wrote: > >> What do you think of the idea of having a re-useable BlobCore module >> that all the ports can share? >> > > I don't think this is a

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Oliver Hunt
On Sep 14, 2010, at 5:56 PM, David Levin wrote: > On Tue, Sep 14, 2010 at 5:42 PM, Adam Barth wrote: > What do you think of the idea of having a re-useable BlobCore module > that all the ports can share? > > I don't think this is a good idea. This "re-usable module" would only be used > by the

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Maciej Stachowiak
On Sep 14, 2010, at 6:02 PM, Adam Barth wrote: > On Tue, Sep 14, 2010 at 5:56 PM, David Levin wrote: >> On Tue, Sep 14, 2010 at 5:42 PM, Adam Barth wrote: >>> What do you think of the idea of having a re-useable BlobCore module >>> that all the ports can share? >> >> I don't think this is a go

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Jian Li
On Tue, Sep 14, 2010 at 6:02 PM, Adam Barth wrote: > On Tue, Sep 14, 2010 at 5:56 PM, David Levin wrote: > > On Tue, Sep 14, 2010 at 5:42 PM, Adam Barth wrote: > >> What do you think of the idea of having a re-useable BlobCore module > >> that all the ports can share? > > > > I don't think this

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Adam Barth
On Tue, Sep 14, 2010 at 5:56 PM, David Levin wrote: > On Tue, Sep 14, 2010 at 5:42 PM, Adam Barth wrote: >> What do you think of the idea of having a re-useable BlobCore module >> that all the ports can share? > > I don't think this is a good idea. This "re-usable module" would only be > used by

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread David Levin
On Tue, Sep 14, 2010 at 5:42 PM, Adam Barth wrote: > What do you think of the idea of having a re-useable BlobCore module > that all the ports can share? > I don't think this is a good idea. This "re-usable module" would only be used by the Safari WebKit port. As I understand it, Chromium wouldn

Re: [webkit-dev] Can node->renderer()->node() == node ever be false?

2010-09-14 Thread Maciej Stachowiak
On Sep 14, 2010, at 5:41 PM, Dimitri Glazkov wrote: > Sorry, I meant node->renderer()->node() != 0. My bad. This loop will > always exit in in the first iteration. It definitely is possible for renderers to have a null result from node(). I do not know for sure that it's impossible for node->re

[webkit-dev] Canvas component in Bugzilla

2010-09-14 Thread Andreas Kling
Hey folks! I propose we add a "Canvas" component in Bugzilla. Thoughts? -Kling ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Maciej Stachowiak
On Sep 14, 2010, at 5:22 PM, Adam Barth wrote: > > In some sense, this is analogous to adding an HTTPResourceHandle and > implementing the HTTP protocol inside of WebCore. While its true that > most (all?) users of WebKit will need to wire WebCore up to an HTTP > library, that doesn't necessari

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Jian Li
I think it is a good idea but it will involve lots of work to figure out how to build and test such module that could be used by all the ports. My concern is that I do not have enough resource to speak of for all other ports. Thanks. On Tue, Sep 14, 2010 at 5:42 PM, Adam Barth wrote: > What do

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Adam Barth
What do you think of the idea of having a re-useable BlobCore module that all the ports can share? Adam On Tue, Sep 14, 2010 at 5:39 PM, Jian Li wrote: > When I implemented the blob scheme handling, I intentionally tried to have > some common implementation that could be used by all applicable

Re: [webkit-dev] Can node->renderer()->node() == node ever be false?

2010-09-14 Thread Dimitri Glazkov
Sorry, I meant node->renderer()->node() != 0. My bad. This loop will always exit in in the first iteration. :DG< On Tue, Sep 14, 2010 at 4:45 PM, Adam Roben wrote: > On Sep 14, 2010, at 6:46 PM, Dimitri Glazkov wrote: > >> I've been looking at this line here and it doesn't seem to make sense >>

Re: [webkit-dev] Blob scheme implementation

2010-09-14 Thread Jian Li
When I implemented the blob scheme handling, I intentionally tried to have some common implementation that could be used by all applicable platforms. But it seems that introducing BlobResourceHandle derived from ResourceHandle might not be a good hook up point because ResourceHandle is only a wrapp

[webkit-dev] Blob scheme implementation

2010-09-14 Thread Adam Barth
Jian Li just had a conversation in #webkit about where the code for implementing the Blob URL scheme should live. I thought I'd open the discussion to webkit-dev in case folks had a different perspective. As part of the fileapi, we're introducing a new URL scheme, called blob, which represents a

Re: [webkit-dev] Can node->renderer()->node() == node ever be false?

2010-09-14 Thread Adam Roben
On Sep 14, 2010, at 6:46 PM, Dimitri Glazkov wrote: > I've been looking at this line here and it doesn't seem to make sense > to me: > http://trac.webkit.org/browser/trunk/WebCore/page/EventHandler.cpp#L2153 > > It looks like the loop in question will always exit early, because it > short-circui

Re: [webkit-dev] Canvas performance and memory usage

2010-09-14 Thread Jakob Petsovits
On August 16, 2010, David Hyatt wrote: > There is a temporary method implemented by all ports called: > > bool drawsUsingCopy() const; // If the image buffer has to > render using a > copied image, it will return true. > > This method only exists right now because some of the port

[webkit-dev] Can node->renderer()->node() == node ever be false?

2010-09-14 Thread Dimitri Glazkov
Dear WebKit, I've been looking at this line here and it doesn't seem to make sense to me: http://trac.webkit.org/browser/trunk/WebCore/page/EventHandler.cpp#L2153 It looks like the loop in question will always exit early, because it short-circuits to node->renderer()->node() == node, which seems

[webkit-dev] Build.webkit.org Downtime: Tomorrow, Sep 15, 8am PDT

2010-09-14 Thread William Siegrist
Build.webkit.org will be down tomorrow for a software upgrade starting around 8am PDT. Total downtime should be less than 1 hour. This is fairly short notice, so I apologize for that, but we're trying to fix the performance issues we've been experiencing. You may have trouble running webkit-pa

Re: [webkit-dev] Bug system: Platform and OS fields don't quite align

2010-09-14 Thread Ryosuke Niwa
On Tue, Sep 14, 2010 at 10:38 AM, Jeremy Orlow wrote: > > It's worth noting that Android also uses v8 and skia. They only have one > reviewer at the moment, but it still might be best to not associate Chromium > with skia/v8 changes. > > As for the original questions: does anyone actually use any

Re: [webkit-dev] Bug system: Platform and OS fields don't quite align

2010-09-14 Thread Jeremy Orlow
On Tue, Sep 14, 2010 at 6:32 PM, David Levin wrote: > > > On Tue, Sep 14, 2010 at 10:28 AM, Adam Barth wrote: > >> We're not very good at using these fields in Bugzilla. In this >> situation, folks will usually prefix the summary of the bug with >> "[Chromium]". This is a signal to reviewers t

Re: [webkit-dev] Bug system: Platform and OS fields don't quite align

2010-09-14 Thread David Levin
On Tue, Sep 14, 2010 at 10:28 AM, Adam Barth wrote: > We're not very good at using these fields in Bugzilla. In this > situation, folks will usually prefix the summary of the bug with > "[Chromium]". This is a signal to reviewers that they might be more > or less interested in reviewing the pat

Re: [webkit-dev] Bug system: Platform and OS fields don't quite align

2010-09-14 Thread Adam Barth
We're not very good at using these fields in Bugzilla. In this situation, folks will usually prefix the summary of the bug with "[Chromium]". This is a signal to reviewers that they might be more or less interested in reviewing the patch depending on whether they like/feel comfortably with review

[webkit-dev] Bug system: Platform and OS fields don't quite align

2010-09-14 Thread Mike Belshe
Hi, I tend to hit code which is often chromium-platform specific. It's hard to know the appropriate "Platform" and "OS" fields for such a bug. For instance, I am working on a small change to WebKit/chromium/src/WebKit.cpp. It's not a Mac bug, its not a PC bug, its a Chromium bug. Chromium is a