Merge lp:~gero-bare/scratch/fix-1472377 into lp:~elementary-apps/scratch/scratch

[Gero.Bare]

Move tab to new window from main loop

Move tab to new window from main loop (lp:1472377)

[The Lemon Man (lemonboy)]

Code-wise LGTM, I'd just add a short comment explaining why it's being done in the main thread.

review: Approve (code)

[The Lemon Man (lemonboy)]

review: Approve (code)

=== modified file 'src/Widgets/DocumentView.vala'
--- src/Widgets/DocumentView.vala	2015-10-29 23:03:09 +0000
+++ src/Widgets/DocumentView.vala	2015-11-09 21:27:51 +0000
@@ -249,8 +249,14 @@
 
             DocumentView other_view = other_window.add_view ();
 
-            this.notebook.remove_tab (doc);
-            other_view.notebook.insert_tab (doc, -1);
+            // We need to make sure switch back to the main thread
+            // when we are modifiying Gtk widgets shared by two threads.
+            Idle.add (() => {
+                this.notebook.remove_tab (doc);
+                other_view.notebook.insert_tab (doc, -1);
+
+                return false;
+            });
         }
 
         private void on_doc_reordered (Granite.Widgets.Tab tab, int new_pos) {