cloudtrends / chromiumembedded

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

Added console message routing via event handler #90

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
This patch adds an callback to the event handler that routes console messages 
for the frame allowing the host application to display them in its own way.

This should be an enhancement 

Original issue reported on code.google.com by lodle...@gmail.com on 17 Jun 2010 at 7:37

Attachments:

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Fixed compile errors 2 (should let it finish compiling before submitting)

Original comment by lodle...@gmail.com on 17 Jun 2010 at 7:51

Attachments:

GoogleCodeExporter commented 9 years ago
The new CEF method should probably be called "HandleConsoleMessage" instead of 
"LogConsoleMsg". Other than that, the patch looks good -- I'll include it in 
the next CEF update.

Original comment by magreenb...@gmail.com on 17 Jun 2010 at 8:54

GoogleCodeExporter commented 9 years ago
Well the thing is it still does the default method for the console msg even if 
you "handle" it so the name is missleading.

Original comment by lodle...@gmail.com on 18 Jun 2010 at 6:49

GoogleCodeExporter commented 9 years ago
Modified implementation committed as revision 84.

Original comment by magreenb...@gmail.com on 22 Jun 2010 at 2:50