Apply changes from original cefglue repo (OutSystems#85)

* Apply changes from original cefglue repo

* Apply PR suggestions

* Apply PR suggestions

CefGlue.BrowserProcess/Handlers/RenderProcessHandler.cs

@@ -29,13 +29,16 @@ protected override void OnWebKitInitialized()
 
         protected override bool OnProcessMessageReceived(CefBrowser browser, CefFrame frame, CefProcessId sourceProcess, CefProcessMessage message)
         {
+            using (message)
+
             WithErrorHandling(() =>
             {
                 using (CefObjectTracker.StartTracking())
                 {
                     _messageDispatcher.DispatchMessage(browser, frame, sourceProcess, message);
                 }
             }, frame);
+
             return base.OnProcessMessageReceived(browser, frame, sourceProcess, message);
         }
 

CefGlue.Common/CommonCefClient.cs

@@ -109,6 +109,7 @@ protected override bool OnProcessMessageReceived(CefBrowser browser, CefFrame fr
         {
             try
             {
+                using (message)
                 _messageDispatcher.DispatchMessage(browser, frame, sourceProcess, message);
                 return base.OnProcessMessageReceived(browser, frame, sourceProcess, message);
             }

CefGlue/Classes.Handlers/CefClient.cs

@@ -270,18 +270,20 @@ private int on_process_message_received(cef_client_t* self, cef_browser_t* brows
 
             var m_browser = CefBrowser.FromNative(browser);
             var m_frame = CefFrame.FromNative(frame);
-            using (var m_message = CefProcessMessage.FromNative(message))
-            {
-                var result = OnProcessMessageReceived(m_browser, m_frame, source_process, m_message);
-                return result ? 1 : 0;
-            }
+
+            // Client is responsible to call `Dispose()` on message when it no more needed.
+            var m_message = CefProcessMessage.FromNative(message);
+
+            var result = OnProcessMessageReceived(m_browser, m_frame, source_process, m_message);
+            return result ? 1 : 0;
         }
 
         /// <summary>
         /// Called when a new message is received from a different process. Return
         /// true if the message was handled or false otherwise.  It is safe to keep a
         /// reference to |message| outside of this callback.
         /// </summary>
+        /// <remarks>Client code is responsible to call CefProcessMessage::Dispose() when it no more needed.</remarks>
         protected virtual bool OnProcessMessageReceived(CefBrowser browser, CefFrame frame, CefProcessId sourceProcess, CefProcessMessage message)
         {
             return false;

CefGlue/Classes.Handlers/CefRenderProcessHandler.cs

@@ -184,18 +184,20 @@ private int on_process_message_received(cef_render_process_handler_t* self, cef_
 
             var m_browser = CefBrowser.FromNative(browser);
             var m_frame = CefFrame.FromNative(frame);
-            using (var m_message = CefProcessMessage.FromNative(message))
-            {
-                var result = OnProcessMessageReceived(m_browser, m_frame, source_process, m_message);
-                return result ? 1 : 0;
-            }
+
+            // Client is responsible to call `Dispose()` on message when it no more needed.
+            var m_message = CefProcessMessage.FromNative(message);
+
+            var result = OnProcessMessageReceived(m_browser, m_frame, source_process, m_message);
+            return result ? 1 : 0;
         }
 
         /// <summary>
         /// Called when a new message is received from a different process. Return
         /// true if the message was handled or false otherwise. It is safe to keep a
         /// reference to |message| outside of this callback.
         /// </summary>
+        /// <remarks>Client code is responsible to call CefProcessMessage::Dispose() when it no more needed.</remarks>
         protected virtual bool OnProcessMessageReceived(CefBrowser browser, CefFrame frame, CefProcessId sourceProcess, CefProcessMessage message)
         {
             return false;