apimall / chromiumembedded

Automatically exported from code.google.com/p/chromiumembedded
0 stars 1 forks source link

New notification from the request handler when the render view is ready #1564

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. This notification is needed to know when render view is ready to receive 
messages.
2.
3.

What is the expected output? What do you see instead?
N/A

What version of the product are you using? On what operating system?
CEF3, Branch 2295, r. 2040, linux64

Please provide any additional information below.
patch attached.

Original issue reported on code.google.com by amitkan...@gmail.com on 3 Mar 2015 at 10:31

Attachments:

GoogleCodeExporter commented 9 years ago
What is your use case for this notification?

Original comment by magreenb...@gmail.com on 3 Mar 2015 at 5:21

GoogleCodeExporter commented 9 years ago
We experienced cases where the renderrer didn't start after a new browser have 
been created. So we use a 'watchdog' to identify these cases. If the callback 
in the patch is not called within a <threshold> time - we restart the browser 
process.

Original comment by amitkan...@gmail.com on 3 Mar 2015 at 6:49

GoogleCodeExporter commented 9 years ago
@#2: Interesting. Does the problem reproduce with current branches (2171+?) Do 
you have any information (error messages, etc) on why the renderer process is 
failing to launch?

Original comment by magreenb...@gmail.com on 3 Mar 2015 at 6:56

GoogleCodeExporter commented 9 years ago
We didn't experience the problem on 2171+ (we'll keep you posted about these 
branches) - but it's crucial for us to have this ability, even if it'll never 
be used.

Original comment by amitkan...@gmail.com on 3 Mar 2015 at 7:02

GoogleCodeExporter commented 9 years ago
@#4: The reason I asked is that it would be good to fix the underlying problem 
if possible. If it still occurs for you please add a new CEF issue with 
whatever information you're able to gather.

Original comment by magreenb...@gmail.com on 3 Mar 2015 at 7:13

GoogleCodeExporter commented 9 years ago
Thanks for the patch. New CefRequestHandler::OnRenderViewReady method added in 
trunk revision 2055.

Original comment by magreenb...@gmail.com on 9 Mar 2015 at 9:05