search

rdmenezes / cefpython

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

Debug flag not set in C++ code for the Browser process #98

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
In cefpython_app.cpp the g_debug variable is set only for the Render process in 
CefPythonApp::OnRenderThreadCreated(). This flag also needs to be set when 
CefPythonApp class is instantiated in the Browser process.

Original issue reported on code.google.com by czarek.t...@gmail.com on 18 Dec 2013 at 1:42

GoogleCodeExporter commented 9 years ago 
When g_debug is set in the browser proces, it won't be required to call 
BrowserProcessHandler_OnRenderProcessThreadCreated(). It's better get rid of 
this call as it might cause a deadlock 
(CefBrowserProcessHandler::OnRenderProcessThreadCreated called on the IO 
thread, then it calls Python on the UI thread). Deadlock issue reported by 
Michael Goffioul when porting CEF to Linux/Aura.

Original comment by czarek.t...@gmail.com on 8 Jan 2014 at 9:40

GoogleCodeExporter commented 9 years ago 
Fixed in revision fd0ed30fb387.

Original comment by czarek.t...@gmail.com on 15 Jan 2014 at 9:39