Fix popup widgets placement (e.g. <select>) in Tkinter example (cztomczak#255).

cztomczak · cztomczak · commit 0af803f455ca · 2016-10-11T14:06:08.000+02:00
Popup widgets appeared at wrong location. Also when window was moved
or resized their location was not updated.
diff --git a/api/Browser.md b/api/Browser.md
@@ -602,10 +602,11 @@ This is an alias for the `LoadUrl` method.
 | --- | --- |
 | __Return__ | void |
 
-Notify the browser of move or resize events so that popup windows are
-displayed in the correct location and dismissed when the window moves.
-Also so that drag&drop areas are updated accordingly. In upstream
-cefclient this method is being called only on Linux and Windows.
+Notify the browser of move or resize events so that popup widgets
+(e.g. `<select>`) are displayed in the correct location and dismissed
+when the window moves. Also so that drag&drop areas are updated
+accordingly. In upstream cefclient this method is being called
+only on Linux and Windows.
 
 
 ### NotifyScreenInfoChanged
diff --git a/examples/tkinter_.py b/examples/tkinter_.py
@@ -46,6 +46,7 @@ def __init__(self, master):
         tk.Frame.__init__(self, master)
         self.master.title("Tkinter example")
         self.master.protocol("WM_DELETE_WINDOW", self.on_close)
+        self.master.bind("<Configure>", self.on_root_configure)
         self.setup_icon()
         self.bind("<Configure>", self.on_configure)
         self.bind("<FocusIn>", self.on_focus_in)
@@ -68,10 +69,19 @@ def __init__(self, master):
         # Pack MainFrame
         self.pack(fill=tk.BOTH, expand=tk.YES)
 
+    def on_root_configure(self, _):
+        logger.debug("MainFrame.on_root_configure")
+        if self.browser_frame:
+            self.browser_frame.on_root_configure()
+
     def on_configure(self, event):
-        self.browser_frame.on_root_configure(
-                event.width,
-                event.height-self.navigation_bar.winfo_height())
+        logger.debug("MainFrame.on_configure")
+        if self.browser_frame:
+            width = event.width
+            height = event.height
+            if self.navigation_bar:
+                height = height - self.navigation_bar.winfo_height()
+            self.browser_frame.on_mainframe_configure(width, height)
 
     def on_focus_in(self, _):
         logger.debug("MainFrame.on_focus_in")
@@ -177,6 +187,7 @@ def on_url_focus_out(self, _):
 
     def on_load_url(self, _):
         if self.master.get_browser():
+            self.master.get_browser().StopLoad()
             self.master.get_browser().LoadUrl(self.url_entry.get())
 
     def on_button1(self, _):
@@ -252,9 +263,15 @@ def on_configure(self, _):
         if not self.browser:
             self.embed_browser()
 
-    def on_root_configure(self, width, height):
+    def on_root_configure(self):
+        # Root <Configure> event will be called when top window is moved
+        if self.browser:
+            self.browser.NotifyMoveOrResizeStarted()
+
+    def on_mainframe_configure(self, width, height):
         if self.browser:
             self.browser.SetBounds(0, 0, width, height)
+            self.browser.NotifyMoveOrResizeStarted()
 
     def on_focus_in(self, _):
         logger.debug("BrowserFrame.on_focus_in")
diff --git a/tools/automate.py b/tools/automate.py
@@ -1,6 +1,6 @@
 # Copyright (c) 2016 CEF Python, see the Authors file. All rights reserved.
 
-"""This tool automates building CEF from with CEF Python patches applied.
+"""Automates building CEF from sources with CEF Python patches applied.
 
 Usage:
     automate.py (--prebuilt-cef | --build-cef)
