[webkit-dev] FrameLoaderClient::dispatchCreatePage

Maciej Stachowiak mjs at apple.com
Fri Feb 16 23:29:43 PST 2007


On Feb 16, 2007, at 10:49 PM, Morgan L wrote:

> I was wondering if there are any plans to replace
> FrameLoaderClient::dispatchCreatePage with a call to
> ChromeClient::createWindow.  It seems like
> FrameLoader::createWindow already calls
> ChromeClient::CreateWindow, and
> FrameLoaderClient::dispatchCreatePage is somewhat
> duplicating ChromeClient::createWindow.  Thoughts?
> Would a patch along these lines be welcomed?

You are right. The FrameLoaderClient call should be removed and the  
ChromeClient one should be used instead. There is no reason for them  
to be separate besides oversight, as far as I can tell.

Regards,
Maciej




More information about the webkit-dev mailing list