Re: [webkit-dev] DocumentLoader::m_mainResourceData seems no need

2011-08-16 Thread Pavel Feldman
On Tue, Aug 16, 2011 at 12:44 PM, Xianzhu Wang wrote: > > > On Tue, Aug 16, 2011 at 3:41 PM, Pavel Feldman wrote: > >> On Tue, Aug 16, 2011 at 11:20 AM, Xianzhu Wang wrote: >> >>> Hi, >>> >>> I found that DocumentLoader will hold m_mainResourceData during its whole >>> life. I'm wondering if this

Re: [webkit-dev] DocumentLoader::m_mainResourceData seems no need

2011-08-16 Thread Xianzhu Wang
On Tue, Aug 16, 2011 at 3:41 PM, Pavel Feldman wrote: > On Tue, Aug 16, 2011 at 11:20 AM, Xianzhu Wang wrote: > >> Hi, >> >> I found that DocumentLoader will hold m_mainResourceData during its whole >> life. I'm wondering if this is necessary. I searched all callers of >> DocumentLoader::mainReso

Re: [webkit-dev] DocumentLoader::m_mainResourceData seems no need

2011-08-16 Thread Pavel Feldman
On Tue, Aug 16, 2011 at 11:20 AM, Xianzhu Wang wrote: > Hi, > > I found that DocumentLoader will hold m_mainResourceData during its whole > life. I'm wondering if this is necessary. I searched all callers of > DocumentLoader::mainResourceData() and DocumentLoader::mainResource() and > they seem a

[webkit-dev] DocumentLoader::m_mainResourceData seems no need

2011-08-16 Thread Xianzhu Wang
Hi, I found that DocumentLoader will hold m_mainResourceData during its whole life. I'm wondering if this is necessary. I searched all callers of DocumentLoader::mainResourceData() and DocumentLoader::mainResource() and they seem all called before or just after the document is loaded. If we remove