fix check for order max in PresentationInfo::parse (string&)
[ardour.git] / gtk2_ardour / editor_routes.cc
index 327841f4ad40cd5c97acba8d5aabebab7d9a85d2..d28eb5d0067908388c47bd6f39f8059cdf396713 100644 (file)
 #include "pbd/unwind.h"
 
 #include "ardour/debug.h"
+#include "ardour/audio_track.h"
 #include "ardour/midi_track.h"
 #include "ardour/route.h"
 #include "ardour/session.h"
+#include "ardour/solo_isolate_control.h"
+#include "ardour/utils.h"
 
 #include "gtkmm2ext/cell_renderer_pixbuf_multi.h"
 #include "gtkmm2ext/cell_renderer_pixbuf_toggle.h"
@@ -46,7 +49,9 @@
 #include "keyboard.h"
 #include "midi_time_axis.h"
 #include "mixer_strip.h"
+#include "plugin_setup_dialog.h"
 #include "route_sorter.h"
+#include "tooltips.h"
 #include "utils.h"
 
 #include "i18n.h"
@@ -72,12 +77,13 @@ EditorRoutes::EditorRoutes (Editor* e)
        , _no_redisplay (false)
        , _adding_routes (false)
        , _route_deletion_in_progress (false)
+       , _redisplay_on_resume (false)
        , _redisplay_active (0)
        , _queue_tv_update (0)
        , _menu (0)
        , old_focus (0)
        , selection_countdown (0)
-               , name_editable (0)
+       , name_editable (0)
 {
        static const int column_width = 22;
 
@@ -106,6 +112,23 @@ EditorRoutes::EditorRoutes (Editor* e)
        rec_state_column->set_expand(false);
        rec_state_column->set_fixed_width(column_width);
 
+
+       // Record safe toggle
+       CellRendererPixbufMulti* rec_safe_renderer = manage (new CellRendererPixbufMulti ());
+
+       rec_safe_renderer->set_pixbuf (0, ::get_icon("rec-safe-disabled"));
+       rec_safe_renderer->set_pixbuf (1, ::get_icon("rec-safe-enabled"));
+       rec_safe_renderer->signal_changed().connect (sigc::mem_fun (*this, &EditorRoutes::on_tv_rec_safe_toggled));
+
+       TreeViewColumn* rec_safe_column = manage (new TreeViewColumn(_("RS"), *rec_safe_renderer));
+       rec_safe_column->add_attribute(rec_safe_renderer->property_state(), _columns.rec_safe);
+       rec_safe_column->add_attribute(rec_safe_renderer->property_visible(), _columns.is_track);
+       rec_safe_column->set_sizing(TREE_VIEW_COLUMN_FIXED);
+       rec_safe_column->set_alignment(ALIGN_CENTER);
+       rec_safe_column->set_expand(false);
+       rec_safe_column->set_fixed_width(column_width);
+
+
        // MIDI Input Active
 
        CellRendererPixbufMulti* input_active_col_renderer = manage (new CellRendererPixbufMulti());
@@ -193,6 +216,7 @@ EditorRoutes::EditorRoutes (Editor* e)
 
        _display.append_column (*input_active_column);
        _display.append_column (*rec_state_column);
+       _display.append_column (*rec_safe_column);
        _display.append_column (*mute_state_column);
        _display.append_column (*solo_state_column);
        _display.append_column (*solo_isolate_state_column);
@@ -208,17 +232,18 @@ EditorRoutes::EditorRoutes (Editor* e)
                { 2, S_("Active|A"),    _("Track/Bus active ?") },
                { 3, S_("MidiInput|I"), _("MIDI input enabled") },
                { 4, S_("Rec|R"),       _("Record enabled") },
-               { 5, S_("Mute|M"),      _("Muted") },
-               { 6, S_("Solo|S"),      _("Soloed") },
-               { 7, S_("SoloIso|SI"),  _("Solo Isolated") },
-               { 8, S_("SoloLock|SS"), _("Solo Safe (Locked)") },
+               { 5, S_("Rec|RS"),      _("Record Safe") },
+               { 6, S_("Mute|M"),      _("Muted") },
+               { 7, S_("Solo|S"),      _("Soloed") },
+               { 8, S_("SoloIso|SI"),  _("Solo Isolated") },
+               { 9, S_("SoloLock|SS"), _("Solo Safe (Locked)") },
                { -1, 0, 0 }
        };
 
        for (int i = 0; ci[i].index >= 0; ++i) {
                col = _display.get_column (ci[i].index);
                l = manage (new Label (ci[i].label));
-               ARDOUR_UI::instance()->set_tip (*l, ci[i].tooltip);
+               set_tooltip (*l, ci[i].tooltip);
                col->set_widget (*l);
                l->show ();
        }
@@ -289,7 +314,8 @@ EditorRoutes::EditorRoutes (Editor* e)
 
        _display.set_enable_search (false);
 
-       Route::SyncOrderKeys.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::sync_treeview_from_order_keys, this), gui_context());
+       Route::PluginSetup.connect_same_thread (*this, boost::bind (&EditorRoutes::plugin_setup, this, _1, _2, _3));
+       Stripable::PresentationInfoChange.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::sync_treeview_from_presentation_info, this), gui_context());
 }
 
 bool
@@ -360,6 +386,13 @@ EditorRoutes::set_session (Session* s)
        if (_session) {
                _session->SoloChanged.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::solo_changed_so_update_mute, this), gui_context());
                _session->RecordStateChanged.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_rec_display, this), gui_context());
+
+               /* TODO: check if these needs to be tied in with DisplaySuspender
+                * Given that the UI is single-threaded and DisplaySuspender is only used
+                * in loops in the UI thread all should be fine.
+                */
+               _session->BatchUpdateStart.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::suspend_redisplay, this), gui_context());
+               _session->BatchUpdateEnd.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::resume_redisplay, this), gui_context());
        }
 }
 
@@ -384,7 +417,6 @@ EditorRoutes::on_input_active_changed (std::string const & path_string)
 void
 EditorRoutes::on_tv_rec_enable_changed (std::string const & path_string)
 {
-       DisplaySuspender ds;
        // Get the model row that has been toggled.
        Gtk::TreeModel::Row row = *_model->get_iter (Gtk::TreeModel::Path (path_string));
 
@@ -392,9 +424,23 @@ EditorRoutes::on_tv_rec_enable_changed (std::string const & path_string)
        RouteTimeAxisView *rtv = dynamic_cast<RouteTimeAxisView*> (tv);
 
        if (rtv && rtv->track()) {
+               // TODO check rec-safe and ...
+               _session->set_control (rtv->track()->rec_enable_control(), !rtv->track()->rec_enable_control()->get_value(), Controllable::UseGroup);
+       }
+}
+
+void
+EditorRoutes::on_tv_rec_safe_toggled (std::string const & path_string)
+{
+       Gtk::TreeModel::Row row = *_model->get_iter (Gtk::TreeModel::Path (path_string));
+       TimeAxisView* tv = row[_columns.tv];
+       RouteTimeAxisView *rtv = dynamic_cast<RouteTimeAxisView*> (tv);
+
+       if (rtv && rtv->track() && !rtv->track()->record_enabled()) {
+               DisplaySuspender ds;
                boost::shared_ptr<RouteList> rl (new RouteList);
                rl->push_back (rtv->route());
-               _session->set_record_enabled (rl, !rtv->track()->record_enabled(), Session::rt_cleanup);
+               _session->set_record_safe (rl, !rtv->track()->record_safe(), Session::rt_cleanup);
        }
 }
 
@@ -408,9 +454,7 @@ EditorRoutes::on_tv_mute_enable_toggled (std::string const & path_string)
        RouteTimeAxisView *rtv = dynamic_cast<RouteTimeAxisView*> (tv);
 
        if (rtv != 0) {
-               boost::shared_ptr<RouteList> rl (new RouteList);
-               rl->push_back (rtv->route());
-               _session->set_mute (rl, !rtv->route()->muted(), Session::rt_cleanup);
+               _session->set_control (rtv->route()->mute_control(), rtv->route()->mute_control()->get_value() ? 0.0 : 1.0, Controllable::UseGroup);
        }
 }
 
@@ -424,13 +468,7 @@ EditorRoutes::on_tv_solo_enable_toggled (std::string const & path_string)
        RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (tv);
 
        if (rtv != 0) {
-               boost::shared_ptr<RouteList> rl (new RouteList);
-               rl->push_back (rtv->route());
-               if (Config->get_solo_control_is_listen_control()) {
-                       _session->set_listen (rl, !rtv->route()->listening_via_monitor(), Session::rt_cleanup);
-               } else {
-                       _session->set_solo (rl, !rtv->route()->self_soloed(), Session::rt_cleanup);
-               }
+               rtv->route()->solo_control()->set_value (rtv->route()->soloed() ? 0.0 : 1.0, Controllable::UseGroup);
        }
 }
 
@@ -444,7 +482,7 @@ EditorRoutes::on_tv_solo_isolate_toggled (std::string const & path_string)
        RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (tv);
 
        if (rtv) {
-               rtv->route()->set_solo_isolated (!rtv->route()->solo_isolated(), this);
+               rtv->route()->solo_isolate_control()->set_value (rtv->route()->solo_isolate_control()->get_value() ? 0.0 : 1.0, Controllable::UseGroup);
        }
 }
 
@@ -458,7 +496,7 @@ EditorRoutes::on_tv_solo_safe_toggled (std::string const & path_string)
        RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (tv);
 
        if (rtv) {
-               rtv->route()->set_solo_safe (!rtv->route()->solo_safe(), this);
+               rtv->route()->solo_safe_control()->set_value (rtv->route()->solo_safe_control()->get_value() ? 0.0 : 1.0, Controllable::UseGroup);
        }
 }
 
@@ -481,7 +519,7 @@ EditorRoutes::build_menu ()
        items.push_back (MenuElem (_("Hide All Audio Busses"), sigc::mem_fun (*this, &EditorRoutes::hide_all_audiobus)));
        items.push_back (MenuElem (_("Show All Midi Tracks"), sigc::mem_fun (*this, &EditorRoutes::show_all_miditracks)));
        items.push_back (MenuElem (_("Hide All Midi Tracks"), sigc::mem_fun (*this, &EditorRoutes::hide_all_miditracks)));
-       items.push_back (MenuElem (_("Show Tracks With Regions Under Playhead"), sigc::mem_fun (*this, &EditorRoutes::show_tracks_with_regions_at_playhead)));
+       items.push_back (MenuElem (_("Only Show Tracks with Regions Under Playhead"), sigc::mem_fun (*this, &EditorRoutes::show_tracks_with_regions_at_playhead)));
 }
 
 void
@@ -516,7 +554,7 @@ EditorRoutes::redisplay_real ()
                }
 
                bool visible = tv->marked_for_display ();
-               
+
                /* show or hide the TimeAxisView */
                if (visible) {
                        position += tv->show_at (position, n, &_editor->edit_controls_vbox);
@@ -549,20 +587,28 @@ EditorRoutes::redisplay_real ()
 void
 EditorRoutes::redisplay ()
 {
-       if (_no_redisplay || !_session || _session->deletion_in_progress()) {
+       if (!_session || _session->deletion_in_progress()) {
+               return;
+       }
+
+       if (_no_redisplay) {
+               _redisplay_on_resume = true;
                return;
        }
 
        // model deprecated g_atomic_int_exchange_and_add(, 1)
-       g_atomic_int_inc(&_redisplay_active);
-       if (!g_atomic_int_compare_and_exchange (&_redisplay_active, 1, 1)) {
+       g_atomic_int_inc(const_cast<gint*>(&_redisplay_active));
+       if (!g_atomic_int_compare_and_exchange (const_cast<gint*>(&_redisplay_active), 1, 1)) {
+               /* recursive re-display can happen if redisplay shows/hides a TrackView
+                * which has children and their display status changes as result.
+                */
                return;
        }
 
        redisplay_real ();
 
-       while (!g_atomic_int_compare_and_exchange (&_redisplay_active, 1, 0)) {
-               g_atomic_int_set(&_redisplay_active, 1);
+       while (!g_atomic_int_compare_and_exchange (const_cast<gint*>(&_redisplay_active), 1, 0)) {
+               g_atomic_int_set(const_cast<gint*>(&_redisplay_active), 1);
                redisplay_real ();
        }
 }
@@ -570,31 +616,35 @@ EditorRoutes::redisplay ()
 void
 EditorRoutes::row_deleted (Gtk::TreeModel::Path const &)
 {
+       if (!_session || _session->deletion_in_progress()) {
+               return;
+       }
        /* this happens as the second step of a DnD within the treeview, and
         * when a route is actually removed. we don't differentiate between
         * the two cases.
         *
-        * note that the sync_orders_keys() step may not actually change any
-        * RID's (e.g. the last track may be removed, so all other tracks keep
-        * the same RID), which means that no redisplay would happen. so we
-        * have to force a redisplay.
+        * note that the sync_presentation_info_from_treeview() step may not
+        * actually change any presentation info (e.g. the last track may be
+        * removed, so all other tracks keep the same presentation info), which
+        * means that no redisplay would happen. so we have to force a
+        * redisplay.
         */
 
        DEBUG_TRACE (DEBUG::OrderKeys, "editor routes treeview row deleted\n");
 
        DisplaySuspender ds;
-       sync_order_keys_from_treeview ();
+       sync_presentation_info_from_treeview ();
 }
 
 void
 EditorRoutes::reordered (TreeModel::Path const &, TreeModel::iterator const &, int* /*what*/)
 {
-       /* reordering implies that RID's will change, so sync_order_keys() will
-          cause a redisplay.
+       /* reordering implies that RID's will change, so
+          sync_presentation_info_from_treeview() will cause a redisplay.
        */
 
        DEBUG_TRACE (DEBUG::OrderKeys, "editor routes treeview reordered\n");
-       sync_order_keys_from_treeview ();
+       sync_presentation_info_from_treeview ();
 }
 
 void
@@ -642,7 +692,7 @@ EditorRoutes::routes_added (list<RouteTimeAxisView*> routes)
        for (Gtk::TreeModel::Children::iterator it = _model->children().begin(); it != _model->children().end(); ++it) {
                boost::shared_ptr<Route> r = (*it)[_columns.route];
 
-               if (r->order_key() == (routes.front()->route()->order_key() + routes.size())) {
+               if (r->presentation_info().group_order() == (routes.front()->route()->presentation_info().group_order() + routes.size())) {
                        insert_iter = it;
                        break;
                }
@@ -676,8 +726,8 @@ EditorRoutes::routes_added (list<RouteTimeAxisView*> routes)
                row[_columns.mute_state] = (*x)->route()->muted() ? Gtkmm2ext::ExplicitActive : Gtkmm2ext::Off;
                row[_columns.solo_state] = RouteUI::solo_active_state ((*x)->route());
                row[_columns.solo_visible] = !(*x)->route()->is_master ();
-               row[_columns.solo_isolate_state] = (*x)->route()->solo_isolated();
-               row[_columns.solo_safe_state] = (*x)->route()->solo_safe();
+               row[_columns.solo_isolate_state] = (*x)->route()->solo_isolate_control()->solo_isolated();
+               row[_columns.solo_safe_state] = (*x)->route()->solo_safe_control()->solo_safe();
                row[_columns.name_editable] = true;
 
                boost::weak_ptr<Route> wr ((*x)->route());
@@ -687,7 +737,8 @@ EditorRoutes::routes_added (list<RouteTimeAxisView*> routes)
 
                if ((*x)->is_track()) {
                        boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track> ((*x)->route());
-                       t->RecordEnableChanged.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_rec_display, this), gui_context());
+                       t->rec_enable_control()->Changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_rec_display, this), gui_context());
+                       t->rec_safe_control()->Changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_rec_display, this), gui_context());
                }
 
                if ((*x)->is_midi_track()) {
@@ -696,18 +747,18 @@ EditorRoutes::routes_added (list<RouteTimeAxisView*> routes)
                        t->InputActiveChanged.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_input_active_display, this), gui_context());
                }
 
-               (*x)->route()->mute_changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_mute_display, this), gui_context());
-               (*x)->route()->solo_changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_solo_display, this, _1), gui_context());
-               (*x)->route()->listen_changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_solo_display, this, _1), gui_context());
-               (*x)->route()->solo_isolated_changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_solo_isolate_display, this), gui_context());
-               (*x)->route()->solo_safe_changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_solo_safe_display, this), gui_context());
+               (*x)->route()->mute_control()->Changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_mute_display, this), gui_context());
+               (*x)->route()->solo_control()->Changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_solo_display, this), gui_context());
+               (*x)->route()->solo_isolate_control()->Changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_solo_isolate_display, this), gui_context());
+               (*x)->route()->solo_safe_control()->Changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_solo_safe_display, this), gui_context());
+
                (*x)->route()->active_changed.connect (*this, MISSING_INVALIDATOR, boost::bind (&EditorRoutes::update_active_display, this), gui_context ());
 
        }
 
        update_rec_display ();
        update_mute_display ();
-       update_solo_display (true);
+       update_solo_display ();
        update_solo_isolate_display ();
        update_solo_safe_display ();
        update_input_active_display ();
@@ -717,7 +768,7 @@ EditorRoutes::routes_added (list<RouteTimeAxisView*> routes)
 
        /* now update route order keys from the treeview/track display order */
        if (!from_scratch) {
-               sync_order_keys_from_treeview ();
+               sync_presentation_info_from_treeview ();
        }
 }
 
@@ -791,7 +842,7 @@ EditorRoutes::route_property_changed (const PropertyChange& what_changed, boost:
 void
 EditorRoutes::update_active_display ()
 {
-       if (g_atomic_int_compare_and_exchange (&_queue_tv_update, 0, 1)) {
+       if (g_atomic_int_compare_and_exchange (const_cast<gint*>(&_queue_tv_update), 0, 1)) {
                Glib::signal_idle().connect (sigc::mem_fun (*this, &EditorRoutes::idle_update_mute_rec_solo_etc));
        }
 }
@@ -812,7 +863,7 @@ EditorRoutes::update_visibility ()
        /* force route order keys catch up with visibility changes
         */
 
-       sync_order_keys_from_treeview ();
+       sync_presentation_info_from_treeview ();
 }
 
 void
@@ -855,60 +906,10 @@ EditorRoutes::reset_remote_control_ids ()
                return;
        }
 
-       TreeModel::Children rows = _model->children();
-
-       if (rows.empty()) {
-               return;
-       }
-
-
-       DEBUG_TRACE (DEBUG::OrderKeys, "editor reset remote control ids\n");
-
-       TreeModel::Children::iterator ri;
-       bool rid_change = false;
-       uint32_t rid = 1;
-       uint32_t invisible_key = UINT32_MAX;
-
-       for (ri = rows.begin(); ri != rows.end(); ++ri) {
-
-               /* skip two special values */
-               
-               if (rid == Route::MasterBusRemoteControlID) {
-                       rid++;
-               }
-               
-               if (rid == Route::MonitorBusRemoteControlID) {
-                       rid++;
-               }
-
-               boost::shared_ptr<Route> route = (*ri)[_columns.route];
-               bool visible = (*ri)[_columns.visible];
-
-               if (!route->is_master() && !route->is_monitor()) {
-
-                       uint32_t new_rid = (visible ? rid : invisible_key--);
-
-                       if (new_rid != route->remote_control_id()) {
-                               route->set_remote_control_id_explicit (new_rid);
-                               rid_change = true;
-                       }
-
-                       if (visible) {
-                               rid++;
-                       }
-
-               }
-       }
-
-       if (rid_change) {
-               /* tell the world that we changed the remote control IDs */
-               _session->notify_remote_id_change ();
-       }
+       sync_presentation_info_from_treeview ();
 }
-
-
 void
-EditorRoutes::sync_order_keys_from_treeview ()
+EditorRoutes::sync_presentation_info_from_treeview ()
 {
        if (_ignore_reorder || !_session || _session->deletion_in_progress()) {
                return;
@@ -920,60 +921,46 @@ EditorRoutes::sync_order_keys_from_treeview ()
                return;
        }
 
-
        DEBUG_TRACE (DEBUG::OrderKeys, "editor sync order keys from treeview\n");
 
        TreeModel::Children::iterator ri;
-       bool changed = false;
-       bool rid_change = false;
-       uint32_t order = 0;
-       uint32_t rid = 1;
-       uint32_t invisible_key = UINT32_MAX;
+       bool change = false;
+       PresentationInfo::order_t order = 0;
+
+       /* hmm, problem ... editor doesn't represent all Stripables... we can't
+          reset the whole presentation order from here.
+       */
 
        for (ri = rows.begin(); ri != rows.end(); ++ri) {
 
                boost::shared_ptr<Route> route = (*ri)[_columns.route];
                bool visible = (*ri)[_columns.visible];
 
-               uint32_t old_key = route->order_key ();
-
-               if (order != old_key) {
-                       route->set_order_key (order);
-
-                       changed = true;
+               if (route->presentation_info().special ()) {
+                       continue;
                }
 
-               if ((Config->get_remote_model() == MixerOrdered) && !route->is_master() && !route->is_monitor()) {
-
-                       uint32_t new_rid = (visible ? rid : invisible_key--);
-
-                       if (new_rid != route->remote_control_id()) {
-                               route->set_remote_control_id_explicit (new_rid);
-                               rid_change = true;
-                       }
-
-                       if (visible) {
-                               rid++;
-                       }
+               if (!visible) {
+                       route->presentation_info().set_flag (PresentationInfo::Hidden);
+               } else {
+                       route->presentation_info().unset_flag (PresentationInfo::Hidden);
+               }
 
+               if (order != route->presentation_info().group_order()) {
+                       route->set_presentation_group_order_explicit (order);
+                       change = true;
                }
 
                ++order;
        }
 
-       if (changed) {
-               /* tell the world that we changed the editor sort keys */
-               _session->sync_order_keys ();
-       }
-
-       if (rid_change) {
-               /* tell the world that we changed the remote control IDs */
-               _session->notify_remote_id_change ();
+       if (change) {
+               Stripable::PresentationInfoChange(); /* EMIT SIGNAL */
        }
 }
 
 void
-EditorRoutes::sync_treeview_from_order_keys ()
+EditorRoutes::sync_treeview_from_presentation_info ()
 {
        /* Some route order key(s) have been changed, make sure that
           we update out tree/list model and GUI to reflect the change.
@@ -983,7 +970,7 @@ EditorRoutes::sync_treeview_from_order_keys ()
                return;
        }
 
-       DEBUG_TRACE (DEBUG::OrderKeys, "editor sync model from order keys.\n");
+       DEBUG_TRACE (DEBUG::OrderKeys, "editor sync model from presentation info.\n");
 
        /* we could get here after either a change in the Mixer or Editor sort
         * order, but either way, the mixer order keys reflect the intended
@@ -999,21 +986,25 @@ EditorRoutes::sync_treeview_from_order_keys ()
                return;
        }
 
-       OrderKeySortedRoutes sorted_routes;
+       OrderingKeys sorted;
 
        for (TreeModel::Children::iterator ri = rows.begin(); ri != rows.end(); ++ri, ++old_order) {
                boost::shared_ptr<Route> route = (*ri)[_columns.route];
-               sorted_routes.push_back (RoutePlusOrderKey (route, old_order, route->order_key ()));
+               sorted.push_back (OrderKeys (old_order, route->presentation_info().group_order()));
+               DEBUG_TRACE (DEBUG::OrderKeys, string_compose ("build new order: route %3  old = %1 new = %1\n",
+                                                              old_order,
+                                                              route->presentation_info().group_order(),
+                                                              route->name()));
        }
 
        SortByNewDisplayOrder cmp;
 
-       sort (sorted_routes.begin(), sorted_routes.end(), cmp);
-       neworder.assign (sorted_routes.size(), 0);
+       sort (sorted.begin(), sorted.end(), cmp);
+       neworder.assign (sorted.size(), 0);
 
        uint32_t n = 0;
 
-       for (OrderKeySortedRoutes::iterator sr = sorted_routes.begin(); sr != sorted_routes.end(); ++sr, ++n) {
+       for (OrderingKeys::iterator sr = sorted.begin(); sr != sorted.end(); ++sr, ++n) {
 
                neworder[n] = sr->old_display_order;
 
@@ -1021,8 +1012,8 @@ EditorRoutes::sync_treeview_from_order_keys ()
                        changed = true;
                }
 
-               DEBUG_TRACE (DEBUG::OrderKeys, string_compose ("EDITOR change order for %1 from %2 to %3\n",
-                                                              sr->route->name(), sr->old_display_order, n));
+               DEBUG_TRACE (DEBUG::OrderKeys, string_compose ("EDITOR change order from %1 to %2\n",
+                                                              sr->old_display_order, n));
        }
 
        if (changed) {
@@ -1078,7 +1069,7 @@ EditorRoutes::set_all_tracks_visibility (bool yn)
        /* force route order keys catch up with visibility changes
         */
 
-       sync_order_keys_from_treeview ();
+       sync_presentation_info_from_treeview ();
 }
 
 void
@@ -1104,17 +1095,20 @@ EditorRoutes::set_all_audio_midi_visibility (int tracks, bool yn)
                if ((atv = dynamic_cast<AudioTimeAxisView*>(tv)) != 0) {
                        switch (tracks) {
                        case 0:
+                               atv->set_marked_for_display (yn);
                                (*i)[_columns.visible] = yn;
                                break;
 
                        case 1:
                                if (atv->is_audio_track()) {
+                                       atv->set_marked_for_display (yn);
                                        (*i)[_columns.visible] = yn;
                                }
                                break;
 
                        case 2:
                                if (!atv->is_audio_track()) {
+                                       atv->set_marked_for_display (yn);
                                        (*i)[_columns.visible] = yn;
                                }
                                break;
@@ -1123,11 +1117,13 @@ EditorRoutes::set_all_audio_midi_visibility (int tracks, bool yn)
                else if ((mtv = dynamic_cast<MidiTimeAxisView*>(tv)) != 0) {
                        switch (tracks) {
                        case 0:
+                               mtv->set_marked_for_display (yn);
                                (*i)[_columns.visible] = yn;
                                break;
 
                        case 3:
                                if (mtv->is_midi_track()) {
+                                       mtv->set_marked_for_display (yn);
                                        (*i)[_columns.visible] = yn;
                                }
                                break;
@@ -1138,7 +1134,7 @@ EditorRoutes::set_all_audio_midi_visibility (int tracks, bool yn)
        /* force route order keys catch up with visibility changes
         */
 
-       sync_order_keys_from_treeview ();
+       sync_presentation_info_from_treeview ();
 }
 
 void
@@ -1216,25 +1212,27 @@ EditorRoutes::key_press (GdkEventKey* ev)
 
                case 'm':
                        if (get_relevant_routes (rl)) {
-                               _session->set_mute (rl, !rl->front()->muted(), Session::rt_cleanup);
+                               _session->set_controls (route_list_to_control_list (rl, &Route::mute_control), rl->front()->muted() ? 0.0 : 1.0, Controllable::NoGroup);
                        }
                        return true;
                        break;
 
                case 's':
                        if (get_relevant_routes (rl)) {
-                               if (Config->get_solo_control_is_listen_control()) {
-                                       _session->set_listen (rl, !rl->front()->listening_via_monitor(), Session::rt_cleanup);
-                               } else {
-                                       _session->set_solo (rl, !rl->front()->self_soloed(), Session::rt_cleanup);
-                               }
+                               _session->set_controls (route_list_to_control_list (rl, &Route::solo_control), rl->front()->self_soloed() ? 0.0 : 1.0, Controllable::NoGroup);
                        }
                        return true;
                        break;
 
                case 'r':
                        if (get_relevant_routes (rl)) {
-                               _session->set_record_enabled (rl, !rl->front()->record_enabled(), Session::rt_cleanup);
+                               for (RouteList::const_iterator r = rl->begin(); r != rl->end(); ++r) {
+                                       boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track> (*r);
+                                       if (t) {
+                                               _session->set_controls (route_list_to_control_list (rl, &Track::rec_enable_control), !t->rec_enable_control()->get_value(), Controllable::NoGroup);
+                                               break;
+                                       }
+                               }
                        }
                        break;
 
@@ -1370,7 +1368,7 @@ EditorRoutes::selection_filter (Glib::RefPtr<TreeModel> const &, TreeModel::Path
        return true;
 }
 
-struct EditorOrderRouteSorter
+struct PresentationInfoRouteSorter
 {
        bool operator() (boost::shared_ptr<Route> a, boost::shared_ptr<Route> b) {
                if (a->is_master()) {
@@ -1380,7 +1378,7 @@ struct EditorOrderRouteSorter
                        /* everything else before master */
                        return false;
                }
-               return a->order_key () < b->order_key ();
+               return a->presentation_info().global_order () < b->presentation_info().global_order ();
        }
 };
 
@@ -1395,8 +1393,8 @@ EditorRoutes::initial_display ()
        }
 
        RouteList r (*_session->get_routes());
-               
-       r.sort (EditorOrderRouteSorter ());
+
+       r.sort (PresentationInfoRouteSorter ());
        _editor->add_routes (r);
 }
 
@@ -1518,7 +1516,7 @@ EditorRoutes::move_selected_tracks (bool up)
        }
 
        for (leading = view_routes.begin(); leading != view_routes.end(); ++leading) {
-               uint32_t order = leading->second->order_key ();
+               uint32_t order = (uint32_t) leading->second->presentation_info().group_order ();
                neworder.push_back (order);
        }
 
@@ -1562,7 +1560,7 @@ EditorRoutes::update_input_active_display ()
 void
 EditorRoutes::update_rec_display ()
 {
-       if (g_atomic_int_compare_and_exchange (&_queue_tv_update, 0, 1)) {
+       if (g_atomic_int_compare_and_exchange (const_cast<gint*>(&_queue_tv_update), 0, 1)) {
                Glib::signal_idle().connect (sigc::mem_fun (*this, &EditorRoutes::idle_update_mute_rec_solo_etc));
        }
 }
@@ -1570,7 +1568,7 @@ EditorRoutes::update_rec_display ()
 bool
 EditorRoutes::idle_update_mute_rec_solo_etc()
 {
-       g_atomic_int_set (&_queue_tv_update, 0);
+       g_atomic_int_set (const_cast<gint*>(&_queue_tv_update), 0);
        TreeModel::Children rows = _model->children();
        TreeModel::Children::iterator i;
 
@@ -1581,10 +1579,13 @@ EditorRoutes::idle_update_mute_rec_solo_etc()
                (*i)[_columns.solo_isolate_state] = RouteUI::solo_isolate_active_state (route) ? 1 : 0;
                (*i)[_columns.solo_safe_state] = RouteUI::solo_safe_active_state (route) ? 1 : 0;
                (*i)[_columns.active] = route->active ();
-               if (boost::dynamic_pointer_cast<Track> (route)) {
+
+               boost::shared_ptr<Track> trk (boost::dynamic_pointer_cast<Track>(route));
+
+               if (trk) {
                        boost::shared_ptr<MidiTrack> mt = boost::dynamic_pointer_cast<MidiTrack> (route);
-                       
-                       if (route->record_enabled()) {
+
+                       if (trk->rec_enable_control()->get_value()) {
                                if (_session->record_status() == Session::Recording) {
                                        (*i)[_columns.rec_state] = 1;
                                } else {
@@ -1595,8 +1596,9 @@ EditorRoutes::idle_update_mute_rec_solo_etc()
                        } else {
                                (*i)[_columns.rec_state] = 0;
                        }
-                       
-                       (*i)[_columns.name_editable] = !route->record_enabled ();
+
+                       (*i)[_columns.rec_safe] = !trk->rec_safe_control()->get_value();
+                       (*i)[_columns.name_editable] = !trk->rec_enable_control()->get_value();
                }
        }
 
@@ -1607,15 +1609,15 @@ EditorRoutes::idle_update_mute_rec_solo_etc()
 void
 EditorRoutes::update_mute_display ()
 {
-       if (g_atomic_int_compare_and_exchange (&_queue_tv_update, 0, 1)) {
+       if (g_atomic_int_compare_and_exchange (const_cast<gint*>(&_queue_tv_update), 0, 1)) {
                Glib::signal_idle().connect (sigc::mem_fun (*this, &EditorRoutes::idle_update_mute_rec_solo_etc));
        }
 }
 
 void
-EditorRoutes::update_solo_display (bool /* selfsoloed */)
+EditorRoutes::update_solo_display ()
 {
-       if (g_atomic_int_compare_and_exchange (&_queue_tv_update, 0, 1)) {
+       if (g_atomic_int_compare_and_exchange (const_cast<gint*>(&_queue_tv_update), 0, 1)) {
                Glib::signal_idle().connect (sigc::mem_fun (*this, &EditorRoutes::idle_update_mute_rec_solo_etc));
        }
 }
@@ -1623,7 +1625,7 @@ EditorRoutes::update_solo_display (bool /* selfsoloed */)
 void
 EditorRoutes::update_solo_isolate_display ()
 {
-       if (g_atomic_int_compare_and_exchange (&_queue_tv_update, 0, 1)) {
+       if (g_atomic_int_compare_and_exchange (const_cast<gint*>(&_queue_tv_update), 0, 1)) {
                Glib::signal_idle().connect (sigc::mem_fun (*this, &EditorRoutes::idle_update_mute_rec_solo_etc));
        }
 }
@@ -1631,7 +1633,7 @@ EditorRoutes::update_solo_isolate_display ()
 void
 EditorRoutes::update_solo_safe_display ()
 {
-       if (g_atomic_int_compare_and_exchange (&_queue_tv_update, 0, 1)) {
+       if (g_atomic_int_compare_and_exchange (const_cast<gint*>(&_queue_tv_update), 0, 1)) {
                Glib::signal_idle().connect (sigc::mem_fun (*this, &EditorRoutes::idle_update_mute_rec_solo_etc));
        }
 }
@@ -1711,6 +1713,21 @@ EditorRoutes::show_tracks_with_regions_at_playhead ()
        TreeModel::Children rows = _model->children ();
        for (TreeModel::Children::iterator i = rows.begin(); i != rows.end(); ++i) {
                TimeAxisView* tv = (*i)[_columns.tv];
-               (*i)[_columns.visible] = (show.find (tv) != show.end());
+               bool to_show = (show.find (tv) != show.end());
+
+               tv->set_marked_for_display (to_show);
+               (*i)[_columns.visible] = to_show;
        }
+
+       /* force route order keys catch up with visibility changes
+        */
+
+       sync_presentation_info_from_treeview ();
+}
+
+int
+EditorRoutes::plugin_setup (boost::shared_ptr<Route> r, boost::shared_ptr<PluginInsert> pi, ARDOUR::Route::PluginSetupOptions flags)
+{
+       PluginSetupDialog psd (r, pi, flags);
+       return psd.run ();
 }