start shaping up VCA assign process
[ardour.git] / gtk2_ardour / mixer_ui.cc
index 6384a7bf6ac9188f9d7037b4d3c513f8b30cbcc4..a321085e23cd5218a67b35ca61ca7271b701e164 100644 (file)
 #include <gtkmm/accelmap.h>
 
 #include "pbd/convert.h"
+#include "pbd/stacktrace.h"
 #include "pbd/unwind.h"
 
 #include <glibmm/threads.h>
 
 #include <gtkmm2ext/gtk_ui.h>
+#include <gtkmm2ext/keyboard.h>
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/tearoff.h>
 #include <gtkmm2ext/window_title.h>
+#include <gtkmm2ext/doi.h>
 
 #include "ardour/amp.h"
 #include "ardour/debug.h"
@@ -44,6 +47,8 @@
 #include "ardour/route_group.h"
 #include "ardour/route_sorters.h"
 #include "ardour/session.h"
+#include "ardour/vca.h"
+#include "ardour/vca_manager.h"
 
 #include "keyboard.h"
 #include "mixer_ui.h"
@@ -60,6 +65,7 @@
 #include "mixer_group_tabs.h"
 #include "timers.h"
 #include "ui_config.h"
+#include "vca_master_strip.h"
 
 #include "i18n.h"
 
@@ -87,9 +93,7 @@ Mixer_UI::instance ()
 }
 
 Mixer_UI::Mixer_UI ()
-       : Window (Gtk::WINDOW_TOPLEVEL)
-       , VisibilityTracker (*((Gtk::Window*) this))
-       , _visible (false)
+       : Tabbable (_content, _("Mixer"))
        , no_track_list_redisplay (false)
        , in_group_row_change (false)
        , track_menu (0)
@@ -103,18 +107,26 @@ Mixer_UI::Mixer_UI ()
        , _maximised (false)
        , _show_mixer_list (true)
 {
-       /* allow this window to become the key focus window */
-       set_flags (CAN_FOCUS);
-
        Route::SyncOrderKeys.connect (*this, invalidator (*this), boost::bind (&Mixer_UI::sync_treeview_from_order_keys, this), gui_context());
 
+       /* bindings was already set in MixerActor constructor */
+
+       _content.set_data ("ardour-bindings", bindings);
+
        scroller.set_can_default (true);
-       set_default (scroller);
+       // set_default (scroller);
 
        scroller_base.set_flags (Gtk::CAN_FOCUS);
        scroller_base.add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
        scroller_base.set_name ("MixerWindow");
        scroller_base.signal_button_release_event().connect (sigc::mem_fun(*this, &Mixer_UI::strip_scroller_button_release));
+
+       /* set up drag-n-drop */
+       vector<TargetEntry> target_table;
+       target_table.push_back (TargetEntry ("PluginFavoritePtr"));
+       scroller_base.drag_dest_set (target_table);
+       scroller_base.signal_drag_data_received().connect (sigc::mem_fun(*this, &Mixer_UI::scroller_drag_data_received));
+
        // add as last item of strip packer
        strip_packer.pack_end (scroller_base, true, true);
 
@@ -194,6 +206,10 @@ Mixer_UI::Mixer_UI ()
        group_display_frame.set_shadow_type (Gtk::SHADOW_IN);
        group_display_frame.add (group_display_vbox);
 
+
+       list<TargetEntry> target_list;
+       target_list.push_back (TargetEntry ("PluginPresetPtr"));
+
        favorite_plugins_model = PluginTreeStore::create (favorite_plugins_columns);
        favorite_plugins_display.set_model (favorite_plugins_model);
        favorite_plugins_display.append_column (_("Favorite Plugins"), favorite_plugins_columns.name);
@@ -203,11 +219,15 @@ Mixer_UI::Mixer_UI ()
        favorite_plugins_display.set_headers_visible (true);
        favorite_plugins_display.set_rules_hint (true);
        favorite_plugins_display.set_can_focus (false);
-       favorite_plugins_display.add_object_drag (favorite_plugins_columns.plugin.index(), "PluginPresetPtr");
+       favorite_plugins_display.add_object_drag (favorite_plugins_columns.plugin.index(), "PluginFavoritePtr");
        favorite_plugins_display.set_drag_column (favorite_plugins_columns.name.index());
+       favorite_plugins_display.add_drop_targets (target_list);
        favorite_plugins_display.signal_row_activated().connect (sigc::mem_fun (*this, &Mixer_UI::plugin_row_activated));
        favorite_plugins_display.signal_button_press_event().connect (sigc::mem_fun (*this, &Mixer_UI::plugin_row_button_press), false);
        favorite_plugins_display.signal_drop.connect (sigc::mem_fun (*this, &Mixer_UI::plugin_drop));
+       favorite_plugins_display.signal_row_expanded().connect (sigc::mem_fun (*this, &Mixer_UI::save_favorite_ui_state));
+       favorite_plugins_display.signal_row_collapsed().connect (sigc::mem_fun (*this, &Mixer_UI::save_favorite_ui_state));
+       favorite_plugins_model->signal_row_has_child_toggled().connect (sigc::mem_fun (*this, &Mixer_UI::sync_treeview_favorite_ui_state));
 
        favorite_plugins_scroller.add (favorite_plugins_display);
        favorite_plugins_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
@@ -224,7 +244,8 @@ Mixer_UI::Mixer_UI ()
        list_vpacker.pack_start (rhs_pane2, true, true);
 
        global_hpacker.pack_start (scroller, true, true);
-       global_hpacker.pack_start (out_packer, false, false, 0);
+       global_hpacker.pack_start (vca_packer, false, false);
+       global_hpacker.pack_start (out_packer, false, false);
 
        list_hpane.pack1(list_vpacker, false, true);
        list_hpane.pack2(global_hpacker, true, false);
@@ -236,26 +257,18 @@ Mixer_UI::Mixer_UI ()
        list_hpane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun(*this, &Mixer_UI::pane_allocation_handler),
                                                         static_cast<Gtk::Paned*> (&list_hpane)));
 
-       global_vpacker.pack_start (list_hpane, true, true);
-
-       add (global_vpacker);
-       set_name ("MixerWindow");
+       _content.pack_start (list_hpane, true, true);
 
        update_title ();
 
-       set_wmclass (X_("ardour_mixer"), PROGRAM_NAME);
-
-       signal_delete_event().connect (sigc::mem_fun (*this, &Mixer_UI::hide_window));
-       add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK);
-
-       signal_configure_event().connect (sigc::mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::configure_handler));
-
        route_group_display_button_box->show();
        route_group_add_button->show();
        route_group_remove_button->show();
 
+       _content.show ();
+       _content.set_name ("MixerWindow");
+
        global_hpacker.show();
-       global_vpacker.show();
        scroller.show();
        scroller_base.show();
        scroller_hpacker.show();
@@ -271,6 +284,7 @@ Mixer_UI::Mixer_UI ()
        rhs_pane1.show();
        rhs_pane2.show();
        strip_packer.show();
+       vca_packer.show();
        out_packer.show();
        list_hpane.show();
        group_display.show();
@@ -303,52 +317,76 @@ Mixer_UI::track_editor_selection ()
        PublicEditor::instance().get_selection().TracksChanged.connect (sigc::mem_fun (*this, &Mixer_UI::follow_editor_selection));
 }
 
-
-void
-Mixer_UI::ensure_float (Window& win)
+Gtk::Window*
+Mixer_UI::use_own_window (bool and_fill_it)
 {
-       win.set_transient_for (*this);
+       bool new_window = !own_window();
+
+       Gtk::Window* win = Tabbable::use_own_window (and_fill_it);
+
+
+       if (win && new_window) {
+               win->set_name ("MixerWindow");
+               ARDOUR_UI::instance()->setup_toplevel_window (*win, _("Mixer"), this);
+               win->signal_scroll_event().connect (sigc::mem_fun (*this, &Mixer_UI::on_scroll_event), false);
+               win->signal_event().connect (sigc::bind (sigc::ptr_fun (&Keyboard::catch_user_event_for_pre_dialog_focus), win));
+               win->set_data ("ardour-bindings", bindings);
+               update_title ();
+       }
+
+       return win;
 }
 
 void
 Mixer_UI::show_window ()
 {
-       present ();
-       if (!_visible) {
-               set_window_pos_and_size ();
+       Tabbable::show_window ();
 
-               /* show/hide group tabs as required */
-               parameter_changed ("show-group-tabs");
+       /* show/hide group tabs as required */
+       parameter_changed ("show-group-tabs");
 
-               /* now reset each strips width so the right widgets are shown */
-               MixerStrip* ms;
+       /* now reset each strips width so the right widgets are shown */
+       MixerStrip* ms;
 
-               TreeModel::Children rows = track_model->children();
-               TreeModel::Children::iterator ri;
+       TreeModel::Children rows = track_model->children();
+       TreeModel::Children::iterator ri;
 
-               for (ri = rows.begin(); ri != rows.end(); ++ri) {
-                       ms = (*ri)[track_columns.strip];
-                       ms->set_width_enum (ms->get_width_enum (), ms->width_owner());
-                       /* Fix visibility of mixer strip stuff */
-                       ms->parameter_changed (X_("mixer-element-visibility"));
+       for (ri = rows.begin(); ri != rows.end(); ++ri) {
+               ms = (*ri)[track_columns.strip];
+               if (!ms) {
+                       continue;
                }
+               ms->set_width_enum (ms->get_width_enum (), ms->width_owner());
+               /* Fix visibility of mixer strip stuff */
+               ms->parameter_changed (X_("mixer-element-visibility"));
        }
 
        /* force focus into main area */
        scroller_base.grab_focus ();
-
-       _visible = true;
 }
 
-bool
-Mixer_UI::hide_window (GdkEventAny *ev)
+void
+Mixer_UI::add_masters (VCAList& vcas)
 {
-       get_window_pos_and_size ();
+       cerr << vcas.size() << " VCAs added\n";
+
+       for (VCAList::iterator v = vcas.begin(); v != vcas.end(); ++v) {
+
+               VCAMasterStrip* vms = new VCAMasterStrip (_session, *v);
+
+               TreeModel::Row row = *(track_model->append());
+               row[track_columns.text] = (*v)->name();
+               row[track_columns.visible] = true;
+               row[track_columns.vca] = vms;
+       }
 
-       _visible = false;
-       return just_hide_it(ev, static_cast<Gtk::Window *>(this));
+       redisplay_track_list ();
 }
 
+void
+Mixer_UI::remove_master (VCAMasterStrip* vms)
+{
+}
 
 void
 Mixer_UI::add_strips (RouteList& routes)
@@ -359,6 +397,10 @@ Mixer_UI::add_strips (RouteList& routes)
        for (Gtk::TreeModel::Children::iterator it = track_model->children().begin(); it != track_model->children().end(); ++it) {
                boost::shared_ptr<Route> r = (*it)[track_columns.route];
 
+               if (!r) {
+                       continue;
+               }
+
                if (r->order_key() == (routes.front()->order_key() + routes.size())) {
                        insert_iter = it;
                        break;
@@ -425,6 +467,7 @@ Mixer_UI::add_strips (RouteList& routes)
                        row[track_columns.visible] = strip->route()->is_master() ? true : strip->marked_for_display();
                        row[track_columns.route] = route;
                        row[track_columns.strip] = strip;
+                       row[track_columns.vca] = 0;
 
                        if (!from_scratch) {
                                _selection.add (strip);
@@ -436,7 +479,8 @@ Mixer_UI::add_strips (RouteList& routes)
                        strip->signal_button_release_event().connect (sigc::bind (sigc::mem_fun(*this, &Mixer_UI::strip_button_release_event), strip));
                }
 
-       } catch (...) {
+       } catch (const std::exception& e) {
+               error << string_compose (_("Error adding GUI elements for new tracks/busses %1"), e.what()) << endmsg;
        }
 
        no_track_list_redisplay = false;
@@ -530,6 +574,10 @@ Mixer_UI::reset_remote_control_ids ()
                boost::shared_ptr<Route> route = (*ri)[track_columns.route];
                bool visible = (*ri)[track_columns.visible];
 
+               if (!route) {
+                       continue;
+               }
+
                if (!route->is_master() && !route->is_monitor()) {
 
                        uint32_t new_rid = (visible ? rid : invisible_key--);
@@ -577,6 +625,10 @@ Mixer_UI::sync_order_keys_from_treeview ()
                boost::shared_ptr<Route> route = (*ri)[track_columns.route];
                bool visible = (*ri)[track_columns.visible];
 
+               if (!route) {
+                       continue;
+               }
+
                uint32_t old_key = route->order_key ();
 
                if (order != old_key) {
@@ -640,6 +692,9 @@ Mixer_UI::sync_treeview_from_order_keys ()
 
        for (TreeModel::Children::iterator ri = rows.begin(); ri != rows.end(); ++ri, ++old_order) {
                boost::shared_ptr<Route> route = (*ri)[track_columns.route];
+               if (!route) {
+                       continue;
+               }
                sorted_routes.push_back (RoutePlusOrderKey (route, old_order, route->order_key ()));
        }
 
@@ -799,8 +854,10 @@ Mixer_UI::set_session (Session* sess)
                return;
        }
 
+       refill_favorite_plugins();
+
        XMLNode* node = ARDOUR_UI::instance()->mixer_settings();
-       set_state (*node);
+       set_state (*node, 0);
 
        update_title ();
 
@@ -814,6 +871,8 @@ Mixer_UI::set_session (Session* sess)
        _session->DirtyChanged.connect (_session_connections, invalidator (*this), boost::bind (&Mixer_UI::update_title, this), gui_context());
        _session->StateSaved.connect (_session_connections, invalidator (*this), boost::bind (&Mixer_UI::update_title, this), gui_context());
 
+       _session->vca_manager().VCAAdded.connect (_session_connections, invalidator (*this), boost::bind (&Mixer_UI::add_masters, this, _1), gui_context());
+
        Config->ParameterChanged.connect (*this, invalidator (*this), boost::bind (&Mixer_UI::parameter_changed, this, _1), gui_context ());
 
        route_groups_changed ();
@@ -821,8 +880,6 @@ Mixer_UI::set_session (Session* sess)
        if (_visible) {
                show_window();
        }
-
-       refill_favorite_plugins();
        start_updating ();
 }
 
@@ -890,7 +947,9 @@ Mixer_UI::update_track_visibility ()
 
                for (i = rows.begin(); i != rows.end(); ++i) {
                        MixerStrip *strip = (*i)[track_columns.strip];
-                       (*i)[track_columns.visible] = strip->marked_for_display ();
+                       if (strip) {
+                               (*i)[track_columns.visible] = strip->marked_for_display ();
+                       }
                }
 
                /* force route order keys catch up with visibility changes
@@ -953,7 +1012,7 @@ Mixer_UI::stop_updating ()
 void
 Mixer_UI::fast_update_strips ()
 {
-       if (is_mapped () && _session) {
+       if (_content.is_mapped () && _session) {
                for (list<MixerStrip *>::iterator i = strips.begin(); i != strips.end(); ++i) {
                        (*i)->fast_update ();
                }
@@ -974,7 +1033,7 @@ Mixer_UI::set_all_strips_visibility (bool yn)
                        TreeModel::Row row = (*i);
                        MixerStrip* strip = row[track_columns.strip];
 
-                       if (strip == 0) {
+                       if (!strip) {
                                continue;
                        }
 
@@ -1003,7 +1062,7 @@ Mixer_UI::set_all_audio_midi_visibility (int tracks, bool yn)
                        TreeModel::Row row = (*i);
                        MixerStrip* strip = row[track_columns.strip];
 
-                       if (strip == 0) {
+                       if (!strip) {
                                continue;
                        }
 
@@ -1124,11 +1183,22 @@ Mixer_UI::redisplay_track_list ()
                return;
        }
 
+       container_clear (vca_packer);
+
        for (i = rows.begin(); i != rows.end(); ++i) {
 
+               VCAMasterStrip* vms = (*i)[track_columns.vca];
+
+               if (vms) {
+                       vca_packer.pack_start (*vms, false, false);
+                       vms->show ();
+                       cerr << "Packed vca " << vms->vca()->number() << " into vca_packer\n";
+                       continue;
+               }
+
                MixerStrip* strip = (*i)[track_columns.strip];
 
-               if (strip == 0) {
+               if (!strip) {
                        /* we're in the middle of changing a row, don't worry */
                        continue;
                }
@@ -1180,7 +1250,7 @@ Mixer_UI::strip_width_changed ()
 {
        _group_tabs->set_dirty ();
 
-#ifdef GTKOSX
+#ifdef __APPLE__
        TreeModel::Children rows = track_model->children();
        TreeModel::Children::iterator i;
        long order;
@@ -1216,6 +1286,8 @@ Mixer_UI::initial_track_display ()
                Unwinder<bool> uw2 (ignore_reorder, true);
 
                track_model->clear ();
+               VCAList vcas = _session->vca_manager().vcas();
+               add_masters (vcas);
                add_strips (copy);
        }
 
@@ -1319,7 +1391,7 @@ Mixer_UI::group_display_button_press (GdkEventButton* ev)
                if (Keyboard::is_edit_event (ev)) {
                        if (group) {
                                // edit_route_group (group);
-#ifdef GTKOSX
+#ifdef __APPLE__
                                group_display.queue_draw();
 #endif
                                return true;
@@ -1331,7 +1403,7 @@ Mixer_UI::group_display_button_press (GdkEventButton* ev)
        {
                bool visible = (*iter)[group_columns.visible];
                (*iter)[group_columns.visible] = !visible;
-#ifdef GTKOSX
+#ifdef __APPLE__
                group_display.queue_draw();
 #endif
                return true;
@@ -1476,15 +1548,15 @@ Mixer_UI::show_mixer_list (bool yn)
 {
        if (yn) {
                list_vpacker.show ();
-               
-               //if user wants to show the pane, we should make sure that it is wide enough to be visible 
+
+               //if user wants to show the pane, we should make sure that it is wide enough to be visible
                int width = list_hpane.get_position();
                if (width < 40)
                        list_hpane.set_position(40);
        } else {
                list_vpacker.hide ();
        }
-       
+
        _show_mixer_list = yn;
 }
 
@@ -1613,7 +1685,7 @@ Mixer_UI::strip_scroller_button_release (GdkEventButton* ev)
        using namespace Menu_Helpers;
 
        if (Keyboard::is_context_menu_event (ev)) {
-               ARDOUR_UI::instance()->add_route (this);
+               ARDOUR_UI::instance()->add_route ();
                return true;
        }
 
@@ -1621,29 +1693,48 @@ Mixer_UI::strip_scroller_button_release (GdkEventButton* ev)
 }
 
 void
-Mixer_UI::set_strip_width (Width w, bool save)
+Mixer_UI::scroller_drag_data_received (const Glib::RefPtr<Gdk::DragContext>& context, int x, int y, const Gtk::SelectionData& data, guint info, guint time)
 {
-       _strip_width = w;
+       printf ("Mixer_UI::scroller_drag_data_received\n");
+       if (data.get_target() != "PluginFavoritePtr") {
+               context->drag_finish (false, false, time);
+               return;
+       }
 
-       for (list<MixerStrip*>::iterator i = strips.begin(); i != strips.end(); ++i) {
-               (*i)->set_width_enum (w, save ? (*i)->width_owner() : this);
+       const void * d = data.get_data();
+       const Gtkmm2ext::DnDTreeView<ARDOUR::PluginPresetPtr>* tv = reinterpret_cast<const Gtkmm2ext::DnDTreeView<ARDOUR::PluginPresetPtr>*>(d);                                                   
+
+       PluginPresetList nfos;
+       TreeView* source;
+       tv->get_object_drag_data (nfos, &source);
+
+       Route::ProcessorList pl;
+       bool ok = false;
+
+       for (list<PluginPresetPtr>::const_iterator i = nfos.begin(); i != nfos.end(); ++i) {
+               PluginPresetPtr ppp = (*i);
+               PluginInfoPtr pip = ppp->_pip;
+               if (!pip->is_instrument ()) {
+                       continue;
+               }
+               ARDOUR_UI::instance()->session_add_midi_track (NULL, 1, _("MIDI"), Config->get_strict_io (), pip, ppp->_preset.valid ? &ppp->_preset : 0);
+               ok = true;
        }
-}
 
-void
-Mixer_UI::set_window_pos_and_size ()
-{
-       resize (m_width, m_height);
-       move (m_root_x, m_root_y);
+       context->drag_finish (ok, false, time);
 }
 
 void
-Mixer_UI::get_window_pos_and_size ()
+Mixer_UI::set_strip_width (Width w, bool save)
 {
-       get_position(m_root_x, m_root_y);
-       get_size(m_width, m_height);
+       _strip_width = w;
+
+       for (list<MixerStrip*>::iterator i = strips.begin(); i != strips.end(); ++i) {
+               (*i)->set_width_enum (w, save ? (*i)->width_owner() : this);
+       }
 }
 
+
 struct PluginStateSorter {
 public:
        bool operator() (PluginInfoPtr a, PluginInfoPtr b) const {
@@ -1667,49 +1758,11 @@ private:
 };
 
 int
-Mixer_UI::set_state (const XMLNode& node)
+Mixer_UI::set_state (const XMLNode& node, int version)
 {
-       const XMLProperty* prop;
-       XMLNode* geometry;
-
-       m_width = default_width;
-       m_height = default_height;
-       m_root_x = 1;
-       m_root_y = 1;
-
-       if ((geometry = find_named_node (node, "geometry")) != 0) {
-
-               XMLProperty* prop;
-
-               if ((prop = geometry->property("x_size")) == 0) {
-                       prop = geometry->property ("x-size");
-               }
-               if (prop) {
-                       m_width = atoi(prop->value());
-               }
-               if ((prop = geometry->property("y_size")) == 0) {
-                       prop = geometry->property ("y-size");
-               }
-               if (prop) {
-                       m_height = atoi(prop->value());
-               }
-
-               if ((prop = geometry->property ("x_pos")) == 0) {
-                       prop = geometry->property ("x-pos");
-               }
-               if (prop) {
-                       m_root_x = atoi (prop->value());
-
-               }
-               if ((prop = geometry->property ("y_pos")) == 0) {
-                       prop = geometry->property ("y-pos");
-               }
-               if (prop) {
-                       m_root_y = atoi (prop->value());
-               }
-       }
+       XMLProperty const * prop;
 
-       set_window_pos_and_size ();
+       Tabbable::set_state (node, version);
 
        if ((prop = node.property ("narrow-strips"))) {
                if (string_is_affirmative (prop->value())) {
@@ -1732,8 +1785,7 @@ Mixer_UI::set_state (const XMLNode& node)
                Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
                bool fs = tact && tact->get_active();
                if (yn ^ fs) {
-                       ActionManager::do_action ("Common",
-                                       "ToggleMaximalMixer");
+                       ActionManager::do_action ("Common", "ToggleMaximalMixer");
                }
        }
 
@@ -1768,78 +1820,49 @@ Mixer_UI::set_state (const XMLNode& node)
                favorite_order.sort (cmp);
                sync_treeview_from_favorite_order ();
        }
-
        return 0;
 }
 
 XMLNode&
-Mixer_UI::get_state (void)
+Mixer_UI::get_state ()
 {
-       XMLNode* node = new XMLNode ("Mixer");
-
-       if (is_realized()) {
-               Glib::RefPtr<Gdk::Window> win = get_window();
-
-               get_window_pos_and_size ();
-
-               XMLNode* geometry = new XMLNode ("geometry");
-               char buf[32];
-               snprintf(buf, sizeof(buf), "%d", m_width);
-               geometry->add_property(X_("x_size"), string(buf));
-               snprintf(buf, sizeof(buf), "%d", m_height);
-               geometry->add_property(X_("y_size"), string(buf));
-               snprintf(buf, sizeof(buf), "%d", m_root_x);
-               geometry->add_property(X_("x_pos"), string(buf));
-               snprintf(buf, sizeof(buf), "%d", m_root_y);
-               geometry->add_property(X_("y_pos"), string(buf));
+       XMLNode* node = new XMLNode (X_("Mixer"));
+       char buf[128];
 
-               // written only for compatibility, they are not used.
-               snprintf(buf, sizeof(buf), "%d", 0);
-               geometry->add_property(X_("x_off"), string(buf));
-               snprintf(buf, sizeof(buf), "%d", 0);
-               geometry->add_property(X_("y_off"), string(buf));
+       node->add_child_nocopy (Tabbable::get_state());
 
-               snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&rhs_pane1)->gobj()));
-               geometry->add_property(X_("mixer_rhs_pane1_pos"), string(buf));
-               snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&rhs_pane2)->gobj()));
-               geometry->add_property(X_("mixer_rhs_pane2_pos"), string(buf));
-               snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (static_cast<Paned*>(&list_hpane)->gobj()));
-               geometry->add_property(X_("mixer_list_hpane_pos"), string(buf));
-
-               node->add_child_nocopy (*geometry);
-       }
+       snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (const_cast<GtkPaned*>(static_cast<const Paned*>(&rhs_pane1)->gobj())));
+       node->add_property(X_("mixer_rhs_pane1_pos"), string(buf));
+       snprintf(buf,sizeof(buf), "%d",gtk_paned_get_position (const_cast<GtkPaned*>(static_cast<const Paned*>(&list_hpane)->gobj())));
+       node->add_property(X_("mixer_list_hpane_pos"), string(buf));
 
        node->add_property ("narrow-strips", _strip_width == Narrow ? "yes" : "no");
-
        node->add_property ("show-mixer", _visible ? "yes" : "no");
-
        node->add_property ("show-mixer-list", _show_mixer_list ? "yes" : "no");
-
        node->add_property ("maximised", _maximised ? "yes" : "no");
 
        store_current_favorite_order ();
        XMLNode* plugin_order = new XMLNode ("PluginOrder");
        int cnt = 0;
        for (PluginInfoList::const_iterator i = favorite_order.begin(); i != favorite_order.end(); ++i, ++cnt) {
-                       XMLNode* p = new XMLNode ("PluginInfo");
-                       p->add_property ("sort", cnt);
-                       p->add_property ("unique-id", (*i)->unique_id);
-                       if (favorite_ui_state.find ((*i)->unique_id) != favorite_ui_state.end ()) {
-                               p->add_property ("expanded", favorite_ui_state[(*i)->unique_id]);
-                       }
-                       plugin_order->add_child_nocopy (*p);
-               ;
+               XMLNode* p = new XMLNode ("PluginInfo");
+               p->add_property ("sort", cnt);
+               p->add_property ("unique-id", (*i)->unique_id);
+               if (favorite_ui_state.find ((*i)->unique_id) != favorite_ui_state.end ()) {
+                       p->add_property ("expanded", favorite_ui_state[(*i)->unique_id]);
+               }
+               plugin_order->add_child_nocopy (*p);
        }
        node->add_child_nocopy (*plugin_order);
+
        return *node;
 }
 
-
 void
 Mixer_UI::pane_allocation_handler (Allocation&, Gtk::Paned* which)
 {
        int pos;
-       XMLProperty* prop = 0;
+       XMLProperty const * prop = 0;
        XMLNode* node = ARDOUR_UI::instance()->mixer_settings();
        XMLNode* geometry;
        int height;
@@ -1922,44 +1945,6 @@ Mixer_UI::scroll_right ()
        scroller.get_hscrollbar()->set_value (min (adj->get_upper(), adj->get_value() + adj->get_step_increment()));
 }
 
-bool
-Mixer_UI::on_key_press_event (GdkEventKey* ev)
-{
-        /* focus widget gets first shot, then bindings, otherwise
-           forward to main window
-        */
-
-       if (gtk_window_propagate_key_event (GTK_WINDOW(gobj()), ev)) {
-               return true;
-       }
-
-       KeyboardKey k (ev->state, ev->keyval);
-
-       if (bindings.activate (k, Bindings::Press)) {
-               return true;
-       }
-
-        return forward_key_press (ev);
-}
-
-bool
-Mixer_UI::on_key_release_event (GdkEventKey* ev)
-{
-       if (gtk_window_propagate_key_event (GTK_WINDOW(gobj()), ev)) {
-               return true;
-       }
-
-       KeyboardKey k (ev->state, ev->keyval);
-
-       if (bindings.activate (k, Bindings::Release)) {
-               return true;
-       }
-
-        /* don't forward releases */
-
-        return true;
-}
-
 bool
 Mixer_UI::on_scroll_event (GdkEventScroll* ev)
 {
@@ -2008,7 +1993,7 @@ Mixer_UI::parameter_changed (string const & p)
        } else if (p == "remote-model") {
                reset_remote_control_ids ();
        } else if (p == "use-monitor-bus") {
-               if (!_session->monitor_out()) {
+               if (_session && !_session->monitor_out()) {
                        monitor_section_detached ();
                }
        }
@@ -2088,13 +2073,17 @@ Mixer_UI::setup_track_display ()
 void
 Mixer_UI::new_track_or_bus ()
 {
-       ARDOUR_UI::instance()->add_route (this);
+       ARDOUR_UI::instance()->add_route ();
 }
 
 
 void
 Mixer_UI::update_title ()
 {
+       if (!own_window()) {
+               return;
+       }
+
        if (_session) {
                string n;
 
@@ -2111,13 +2100,13 @@ Mixer_UI::update_title ()
                WindowTitle title (n);
                title += S_("Window|Mixer");
                title += Glib::get_application_name ();
-               set_title (title.get_string());
+               own_window()->set_title (title.get_string());
 
        } else {
 
                WindowTitle title (S_("Window|Mixer"));
                title += Glib::get_application_name ();
-               set_title (title.get_string());
+               own_window()->set_title (title.get_string());
        }
 }
 
@@ -2127,7 +2116,7 @@ Mixer_UI::strip_by_x (int x)
        for (list<MixerStrip*>::iterator i = strips.begin(); i != strips.end(); ++i) {
                int x1, x2, y;
 
-               (*i)->translate_coordinates (*this, 0, 0, x1, y);
+               (*i)->translate_coordinates (_content, 0, 0, x1, y);
                x2 = x1 + (*i)->get_width();
 
                if (x >= x1 && x <= x2) {
@@ -2187,24 +2176,30 @@ Mixer_UI::toggle_midi_input_active (bool flip_others)
 void
 Mixer_UI::maximise_mixer_space ()
 {
-       if (_maximised) {
+       if (!own_window()) {
                return;
        }
 
-       fullscreen ();
+       if (_maximised) {
+               return;
+       }
 
+       _window->fullscreen ();
        _maximised = true;
 }
 
 void
 Mixer_UI::restore_mixer_space ()
 {
-       if (!_maximised) {
+       if (!own_window()) {
                return;
        }
 
-       unfullscreen();
+       if (!_maximised) {
+               return;
+       }
 
+       own_window()->unfullscreen();
        _maximised = false;
 }
 
@@ -2240,6 +2235,15 @@ Mixer_UI::store_current_favorite_order ()
        }
 }
 
+void
+Mixer_UI::save_favorite_ui_state (const TreeModel::iterator& iter, const TreeModel::Path& path)
+{
+       Gtk::TreeModel::Row row = *iter;
+       ARDOUR::PluginPresetPtr ppp = row[favorite_plugins_columns.plugin];
+       assert (ppp);
+       favorite_ui_state[(*ppp->_pip).unique_id] = favorite_plugins_display.row_expanded (favorite_plugins_model->get_path(iter));
+}
+
 void
 Mixer_UI::refiller (PluginInfoList& result, const PluginInfoList& plugs)
 {
@@ -2255,11 +2259,13 @@ Mixer_UI::refiller (PluginInfoList& result, const PluginInfoList& plugs)
 struct PluginCustomSorter {
 public:
        bool operator() (PluginInfoPtr a, PluginInfoPtr b) const {
-               PluginInfoList::const_iterator aiter = std::find(_user.begin(), _user.end(), a);
-               PluginInfoList::const_iterator biter = std::find(_user.begin(), _user.end(), b);
+               PluginInfoList::const_iterator aiter = _user.begin();
+               PluginInfoList::const_iterator biter = _user.begin();
+               while (aiter != _user.end()) { if ((*aiter)->unique_id == a->unique_id) { break; } ++aiter; }
+               while (biter != _user.end()) { if ((*biter)->unique_id == b->unique_id) { break; } ++biter; }
 
                if (aiter != _user.end() && biter != _user.end()) {
-                       return std::distance (_user.begin(), aiter)  < std::distance (_user.begin(), biter);
+                       return std::distance (_user.begin(), aiter) < std::distance (_user.begin(), biter);
                }
                if (aiter != _user.end()) {
                        return true;
@@ -2293,6 +2299,7 @@ Mixer_UI::refill_favorite_plugins ()
        refiller (plugs, mgr.au_plugin_info ());
 #endif
        refiller (plugs, mgr.ladspa_plugin_info ());
+       refiller (plugs, mgr.lua_plugin_info ());
 
        store_current_favorite_order ();
 
@@ -2304,6 +2311,25 @@ Mixer_UI::refill_favorite_plugins ()
        sync_treeview_from_favorite_order ();
 }
 
+void
+Mixer_UI::sync_treeview_favorite_ui_state (const TreeModel::Path& path, const TreeModel::iterator&)
+{
+       TreeIter iter;
+       if (!(iter = favorite_plugins_model->get_iter (path))) {
+               return;
+       }
+       ARDOUR::PluginPresetPtr ppp = (*iter)[favorite_plugins_columns.plugin];
+       if (!ppp) {
+               return;
+       }
+       PluginInfoPtr pip = ppp->_pip;
+       if (favorite_ui_state.find (pip->unique_id) != favorite_ui_state.end ()) {
+               if (favorite_ui_state[pip->unique_id]) {
+                       favorite_plugins_display.expand_row (path, true);
+               }
+       }
+}
+
 void
 Mixer_UI::sync_treeview_from_favorite_order ()
 {
@@ -2318,7 +2344,7 @@ Mixer_UI::sync_treeview_from_favorite_order ()
                        continue;
                }
 
-               vector<ARDOUR::Plugin::PresetRecord> presets = (*i)->get_presets(*_session);
+               vector<ARDOUR::Plugin::PresetRecord> presets = (*i)->get_presets (true);
                for (vector<ARDOUR::Plugin::PresetRecord>::const_iterator j = presets.begin(); j != presets.end(); ++j) {
                        Gtk::TreeModel::Row child_row = *(favorite_plugins_model->append (newrow.children()));
                        child_row[favorite_plugins_columns.name] = (*j).label;
@@ -2359,7 +2385,10 @@ Mixer_UI::popup_note_context_menu (GdkEventButton *ev)
 
        ARDOUR::PluginPresetPtr ppp = selected_plugin();
        if (ppp && ppp->_preset.valid && ppp->_preset.user) {
-               items.push_back (MenuElem (_("Delete Preset"), sigc::mem_fun (*this, &Mixer_UI::delete_selected_preset)));
+               // we cannot currently delete AU presets
+               if (!ppp->_pip || ppp->_pip->type != AudioUnit) {
+                       items.push_back (MenuElem (_("Delete Preset"), sigc::mem_fun (*this, &Mixer_UI::delete_selected_preset)));
+               }
        }
 
        m->popup (ev->button, ev->time);
@@ -2483,13 +2512,22 @@ Mixer_UI::add_favorite_processor (ARDOUR::PluginPresetPtr ppp, ProcessorPosition
                        case AddPostFader:
                                {
                                        int idx = 0;
+                                       int pos = 0;
                                        for (;;++idx) {
                                                boost::shared_ptr<Processor> np = rt->nth_processor (idx);
-                                               if (!np || boost::dynamic_pointer_cast<Amp> (np)) {
+                                               if (!np) {
+                                                       break;
+                                               }
+                                               if (!np->display_to_user()) {
+                                                       continue;
+                                               }
+                                               if (boost::dynamic_pointer_cast<Amp> (np) && // Fader, not Trim
+                                                               boost::dynamic_pointer_cast<Amp> (np)->gain_control()->parameter().type() == GainAutomation) {
                                                        break;
                                                }
+                                               ++pos;
                                        }
-                                       rt->add_processor_by_index (processor, ++idx, &err, Config->get_new_plugins_active ());
+                                       rt->add_processor_by_index (processor, ++pos, &err, Config->get_new_plugins_active ());
                                }
                                break;
                        case AddBottom:
@@ -2505,6 +2543,16 @@ PluginTreeStore::row_drop_possible_vfunc(const Gtk::TreeModel::Path& dest, const
        if (data.get_target() != "GTK_TREE_MODEL_ROW") {
                return false;
        }
+
+       // only allow to re-order top-level items
+       TreePath src;
+       if (TreePath::get_from_selection_data (data, src)) {
+               if (src.up() && src.up()) {
+                       return false;
+               }
+       }
+
+       // don't allow to drop as child-rows.
        Gtk::TreeModel::Path _dest = dest; // un const
        const bool is_child = _dest.up (); // explicit bool for clang
        if (!is_child || _dest.empty ()) {
@@ -2531,3 +2579,17 @@ Mixer_UI::plugin_drop (const Glib::RefPtr<Gdk::DragContext>&, const Gtk::Selecti
        manager.set_status (ppp->_pip->type, ppp->_pip->unique_id, status);
        manager.save_statuses ();
 }
+
+void
+Mixer_UI::do_vca_assign (boost::shared_ptr<VCA> vca)
+{
+       /* call protected MixerActor:: method */
+       vca_assign (vca);
+}
+
+void
+Mixer_UI::do_vca_unassign (boost::shared_ptr<VCA> vca)
+{
+       /* call protected MixerActor:: method */
+       vca_unassign (vca);
+}