Fixed wx.chromectrl crash during shutdown when no browser was created during session.

cztomczak · cztomczak · commit 6e398e5a8145 · 2014-08-01T16:20:16.000+02:00
There is something wrong with the wx.chromectrl sample2.py example. CPU usage is 100%
all the time on Linux. Added comment about this in the source.
diff --git a/cefpython/cef3/wx-subpackage/chromectrl.py b/cefpython/cef3/wx-subpackage/chromectrl.py
@@ -45,8 +45,12 @@ def MessageLoopTimer(event):
 def DestroyMessageLoopTimer():
     global g_messageLoopTimer
     Debug("DestroyMessageLoopTimer")
-    g_messageLoopTimer.Stop()
-    g_messageLoopTimer = None
+    if g_messageLoopTimer:
+        g_messageLoopTimer.Stop()
+        g_messageLoopTimer = None
+    else:
+        # There was no browser created during session.
+        Debug("DestroyMessageLoopTimer: timer not started")
 
 #-------------------------------------------------------------------------------
 
diff --git a/cefpython/cef3/wx-subpackage/examples/sample2.py b/cefpython/cef3/wx-subpackage/examples/sample2.py
@@ -1,6 +1,13 @@
 # Slightly more advanced sample illustrating the usage of CEFWindow class
 # __author__ = "Greg Kacy <grkacy@gmail.com>"
 
+# --WARNING--
+# TODO: There is something wrong with this example. On Linux
+#       CPU usage for the python process is 100% all the time.
+#       The other examples sample1.py / sample2.py do not
+#       show such behavior. It must have something to do
+#       with the wx controls used in this example.
+
 import wx
 import wx.lib.agw.flatnotebook as fnb
 import cefpython3.wx.chromectrl as chrome
