the endless quest to plug memory leaks -- episode 379
[ardour.git] / gtk2_ardour / route_ui.cc
index 5dedca92fb80addd9ebff61d9b40a9e479d27397..3f2f39d0cdc59d5e3aadc7f1a9e149e665eb2d13 100644 (file)
 #include <gtkmm2ext/gtk_ui.h>
 #include <gtkmm2ext/utils.h>
 
-#include "ardour/route_group.h"
-#include "ardour/dB.h"
 #include "pbd/memento_command.h"
 #include "pbd/stacktrace.h"
 #include "pbd/controllable.h"
 #include "pbd/enumwriter.h"
 
-#include "ardour_ui.h"
-#include "editor.h"
-#include "route_ui.h"
-#include "ardour_button.h"
-#include "keyboard.h"
-#include "utils.h"
-#include "prompter.h"
-#include "gui_thread.h"
-#include "ardour_dialog.h"
-#include "latency_gui.h"
-#include "mixer_strip.h"
-#include "automation_time_axis.h"
-#include "route_time_axis.h"
-#include "group_tabs.h"
-#include "timers.h"
-#include "ui_config.h"
-
+#include "ardour/dB.h"
+#include "ardour/route_group.h"
+#include "ardour/solo_isolate_control.h"
+#include "ardour/vca.h"
+#include "ardour/vca_manager.h"
 #include "ardour/audio_track.h"
 #include "ardour/audioengine.h"
 #include "ardour/filename_extensions.h"
 #include "ardour/midi_track.h"
+#include "ardour/monitor_control.h"
 #include "ardour/internal_send.h"
 #include "ardour/profile.h"
+#include "ardour/phase_control.h"
 #include "ardour/send.h"
 #include "ardour/route.h"
 #include "ardour/session.h"
 #include "ardour/template_utils.h"
 
-#include "i18n.h"
+#include "ardour_button.h"
+#include "ardour_dialog.h"
+#include "ardour_ui.h"
+#include "automation_time_axis.h"
+#include "editor.h"
+#include "group_tabs.h"
+#include "gui_object.h"
+#include "gui_thread.h"
+#include "keyboard.h"
+#include "latency_gui.h"
+#include "mixer_strip.h"
+#include "plugin_pin_dialog.h"
+#include "prompter.h"
+#include "rgb_macros.h"
+#include "route_time_axis.h"
+#include "route_ui.h"
+#include "timers.h"
+#include "ui_config.h"
+#include "utils.h"
+
+
+#include "pbd/i18n.h"
 using namespace Gtk;
 using namespace Gtkmm2ext;
 using namespace ARDOUR;
@@ -76,7 +85,8 @@ boost::weak_ptr<Route> RouteUI::_showing_sends_to;
 std::string RouteUI::program_port_prefix;
 
 RouteUI::RouteUI (ARDOUR::Session* sess)
-       : AxisView(sess)
+       : monitor_input_button (0)
+       , monitor_disk_button (0)
        , mute_menu(0)
        , solo_menu(0)
        , sends_menu(0)
@@ -93,13 +103,16 @@ RouteUI::RouteUI (ARDOUR::Session* sess)
                boost::to_lower (lpn);
                program_port_prefix = lpn + ":"; // e.g. "ardour:"
        }
-       if (sess) init ();
+
+       if (sess) {
+               init ();
+       }
 }
 
 RouteUI::~RouteUI()
 {
        if (_route) {
-               gui_object_state().remove_node (route_state_id());
+               ARDOUR_UI::instance()->gui_object_state->remove_node (route_state_id());
        }
 
        _route.reset (); /* drop reference to route, so that it can be cleaned up */
@@ -108,10 +121,12 @@ RouteUI::~RouteUI()
        delete solo_menu;
        delete mute_menu;
        delete sends_menu;
-        delete record_menu;
+       delete record_menu;
        delete comment_window;
        delete input_selector;
        delete output_selector;
+       delete monitor_input_button;
+       delete monitor_disk_button;
        delete _invert_menu;
 
        send_blink_connection.disconnect ();
@@ -125,20 +140,21 @@ RouteUI::init ()
        mute_menu = 0;
        solo_menu = 0;
        sends_menu = 0;
-        record_menu = 0;
+       record_menu = 0;
        _invert_menu = 0;
        pre_fader_mute_check = 0;
        post_fader_mute_check = 0;
        listen_mute_check = 0;
        main_mute_check = 0;
-        solo_safe_check = 0;
-        solo_isolated_check = 0;
-        solo_isolated_led = 0;
-        solo_safe_led = 0;
+       solo_safe_check = 0;
+       solo_isolated_check = 0;
+       solo_isolated_led = 0;
+       solo_safe_led = 0;
        _solo_release = 0;
        _mute_release = 0;
        denormal_menu_item = 0;
-        step_edit_item = 0;
+       step_edit_item = 0;
+       rec_safe_item = 0;
        multiple_mute_change = false;
        multiple_solo_change = false;
        _i_am_the_modifier = 0;
@@ -170,13 +186,13 @@ RouteUI::init ()
        show_sends_button->set_name ("send alert button");
        UI::instance()->set_tip (show_sends_button, _("make mixer strips show sends to this bus"), "");
 
-       monitor_input_button = manage (new ArdourButton (ArdourButton::default_elements));
+       monitor_input_button = new ArdourButton (ArdourButton::default_elements);
        monitor_input_button->set_name ("monitor button");
        monitor_input_button->set_text (_("In"));
        UI::instance()->set_tip (monitor_input_button, _("Monitor input"), "");
        monitor_input_button->set_no_show_all (true);
 
-       monitor_disk_button = manage (new ArdourButton (ArdourButton::default_elements));
+       monitor_disk_button = new ArdourButton (ArdourButton::default_elements);
        monitor_disk_button->set_name ("monitor button");
        monitor_disk_button->set_text (_("Disk"));
        UI::instance()->set_tip (monitor_disk_button, _("Monitor playback"), "");
@@ -240,7 +256,7 @@ RouteUI::set_route (boost::shared_ptr<Route> rp)
        _route = rp;
 
        if (set_color_from_route()) {
-               set_color (unique_random_color());
+               set_color (gdk_color_to_rgba (AxisView::unique_random_color ()));
        }
 
        if (self_destruct) {
@@ -257,29 +273,32 @@ RouteUI::set_route (boost::shared_ptr<Route> rp)
        solo_button->set_controllable (_route->solo_control());
 
        _route->active_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_active_changed, this), gui_context());
-       _route->mute_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_mute_display, this), gui_context());
 
        _route->comment_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::comment_changed, this), gui_context());
 
-       _route->solo_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
-       _route->solo_safe_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
-       _route->listen_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
-       _route->solo_isolated_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
+       _route->mute_control()->Changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_mute_display, this), gui_context());
+       _route->solo_control()->Changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
+       _route->solo_safe_control()->Changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
+       _route->solo_isolate_control()->Changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_solo_display, this), gui_context());
+       _route->phase_control()->Changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::polarity_changed, this), gui_context());
+
        if (is_track()) {
+               track()->FreezeChange.connect (*this, invalidator (*this), boost::bind (&RouteUI::map_frozen, this), gui_context());
                track()->TrackModeChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::track_mode_changed, this), gui_context());
                track_mode_changed();
        }
 
-       _route->phase_invert_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::polarity_changed, this), gui_context());
-       _route->PropertyChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::property_changed, this, _1), gui_context());
+
+       _route->PropertyChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_property_changed, this, _1), gui_context());
+       _route->presentation_info().PropertyChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_gui_changed, this, _1), gui_context ());
 
        _route->io_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::setup_invert_buttons, this), gui_context ());
-       _route->gui_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_gui_changed, this, _1), gui_context ());
 
        if (_session->writable() && is_track()) {
                boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track>(_route);
 
-               t->RecordEnableChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_rec_enable_changed, this), gui_context());
+               t->rec_enable_control()->Changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_rec_enable_changed, this), gui_context());
+               t->rec_safe_control()->Changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_rec_enable_changed, this), gui_context());
 
                rec_enable_button->show();
                rec_enable_button->set_controllable (t->rec_enable_control());
@@ -297,7 +316,7 @@ RouteUI::set_route (boost::shared_ptr<Route> rp)
 
        if (is_track()) {
                boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track>(_route);
-               t->MonitoringChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::monitoring_changed, this), gui_context());
+               t->monitoring_control()->Changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_monitoring_display, this), gui_context());
 
                update_monitoring_display ();
        }
@@ -328,7 +347,10 @@ RouteUI::set_route (boost::shared_ptr<Route> rp)
                blink_rec_display(true); // set initial rec-en button state
        }
 
+       check_rec_enable_sensitivity ();
+       maybe_add_route_print_mgr ();
        route_color_changed();
+       route_gui_changed (PropertyChange (Properties::selected));
 }
 
 void
@@ -369,7 +391,7 @@ RouteUI::mute_press (GdkEventButton* ev)
                if (Keyboard::is_button2_event (ev)) {
                        // button2-click is "momentary"
 
-                       _mute_release = new SoloMuteRelease (_route->muted ());
+                       _mute_release = new SoloMuteRelease (_route->mute_control()->muted ());
                }
 
                if (ev->button == 1 || Keyboard::is_button2_event (ev)) {
@@ -401,7 +423,7 @@ RouteUI::mute_press (GdkEventButton* ev)
                                }
 
                                DisplaySuspender ds;
-                               _session->set_mute (copy, !_route->muted());
+                               _session->set_controls (route_list_to_control_list (copy, &Stripable::mute_control), _route->muted_by_self() ? 0.0 : 1.0, Controllable::UseGroup);
 
                        } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
 
@@ -430,7 +452,7 @@ RouteUI::mute_press (GdkEventButton* ev)
                                        }
 
                                        DisplaySuspender ds;
-                                       _session->set_mute (rl, !_route->muted(), Session::rt_cleanup, Controllable::InverseGroup);
+                                       _session->set_controls (route_list_to_control_list (rl, &Stripable::mute_control), _route->muted_by_self() ? 0.0 : 1.0, Controllable::InverseGroup);
                                }
 
                        } else {
@@ -444,8 +466,7 @@ RouteUI::mute_press (GdkEventButton* ev)
                                        _mute_release->routes = rl;
                                }
 
-                               _session->set_mute (rl, !_route->muted());
-
+                               _route->mute_control()->set_value (!_route->muted_by_self(), Controllable::UseGroup);
                        }
                }
        }
@@ -458,7 +479,7 @@ RouteUI::mute_release (GdkEventButton* /*ev*/)
 {
        if (_mute_release){
                DisplaySuspender ds;
-               _session->set_mute (_mute_release->routes, _mute_release->active, Session::rt_cleanup, Controllable::UseGroup);
+               _session->set_controls (route_list_to_control_list (_mute_release->routes, &Stripable::mute_control), _mute_release->active, Controllable::UseGroup);
                delete _mute_release;
                _mute_release = 0;
        }
@@ -558,11 +579,7 @@ RouteUI::solo_press(GdkEventButton* ev)
                                }
 
                                DisplaySuspender ds;
-                               if (Config->get_solo_control_is_listen_control()) {
-                                       _session->set_listen (_session->get_routes(), !_route->listening_via_monitor(),  Session::rt_cleanup, Controllable::NoGroup);
-                               } else {
-                                       _session->set_solo (_session->get_routes(), !_route->self_soloed(),  Session::rt_cleanup, Controllable::NoGroup);
-                               }
+                               _session->set_controls (route_list_to_control_list (_session->get_routes(), &Stripable::solo_control), !_route->solo_control()->get_value(), Controllable::UseGroup);
 
                        } else if (Keyboard::modifier_state_contains (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::SecondaryModifier))) {
 
@@ -586,14 +603,14 @@ RouteUI::solo_press(GdkEventButton* ev)
                                        /* ??? we need a just_one_listen() method */
                                } else {
                                        DisplaySuspender ds;
-                                       _session->set_just_one_solo (_route, true);
+                                       _route->solo_control()->set_value (1.0, Controllable::NoGroup);
                                }
 
                        } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::TertiaryModifier)) {
 
                                // shift-click: toggle solo isolated status
 
-                               _route->set_solo_isolated (!_route->solo_isolated(), Controllable::UseGroup);
+                               _route->solo_isolate_control()->set_value (_route->solo_isolate_control()->get_value() ? 0.0 : 1.0, Controllable::UseGroup);
                                delete _solo_release;
                                _solo_release = 0;
 
@@ -633,11 +650,7 @@ RouteUI::solo_press(GdkEventButton* ev)
 
                                        DisplaySuspender ds;
 
-                                       if (Config->get_solo_control_is_listen_control()) {
-                                               _session->set_listen (rl, !_route->listening_via_monitor(),  Session::rt_cleanup, Controllable::InverseGroup);
-                                       } else {
-                                               _session->set_solo (rl, !_route->self_soloed(),  Session::rt_cleanup, Controllable::InverseGroup);
-                                       }
+                                       _session->set_controls (route_list_to_control_list (rl, &Stripable::solo_control), !_route->self_soloed(), Controllable::InverseGroup);
                                }
 
                                delete _solo_release;
@@ -655,11 +668,7 @@ RouteUI::solo_press(GdkEventButton* ev)
                                }
 
                                DisplaySuspender ds;
-                               if (Config->get_solo_control_is_listen_control()) {
-                                       _session->set_listen (rl, !_route->listening_via_monitor());
-                               } else {
-                                       _session->set_solo (rl, !_route->self_soloed());
-                               }
+                               _session->set_controls (route_list_to_control_list (rl, &Stripable::solo_control), !_route->self_soloed(), Controllable::UseGroup);
                        }
                }
        }
@@ -676,11 +685,7 @@ RouteUI::solo_release (GdkEventButton* /*ev*/)
 
                } else {
                        DisplaySuspender ds;
-                       if (Config->get_solo_control_is_listen_control()) {
-                               _session->set_listen (_solo_release->routes, _solo_release->active, Session::rt_cleanup, Controllable::UseGroup);
-                       } else {
-                               _session->set_solo (_solo_release->routes, _solo_release->active, Session::rt_cleanup, Controllable::UseGroup);
-                       }
+                       _session->set_controls (route_list_to_control_list (_solo_release->routes, &Stripable::solo_control), _solo_release->active ? 1.0 : 0.0, Controllable::UseGroup);
                }
 
                delete _solo_release;
@@ -727,7 +732,7 @@ RouteUI::rec_enable_press(GdkEventButton* ev)
                } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::TertiaryModifier))) {
 
                        DisplaySuspender ds;
-                       _session->set_record_enabled (_session->get_routes(), !_route->record_enabled());
+                       _session->set_controls (route_list_to_control_list (_session->get_routes(), &Stripable::rec_enable_control), !track()->rec_enable_control()->get_value(), Controllable::NoGroup);
 
                } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
 
@@ -739,17 +744,11 @@ RouteUI::rec_enable_press(GdkEventButton* ev)
 
                                boost::shared_ptr<RouteList> rl;
 
-                               if (_route->route_group()) {
-
-                                       rl = _route->route_group()->route_list();
-
-                               } else {
-                                       rl.reset (new RouteList);
-                                       rl->push_back (_route);
-                               }
+                               rl.reset (new RouteList);
+                               rl->push_back (_route);
 
                                DisplaySuspender ds;
-                               _session->set_record_enabled (rl, !_route->record_enabled(), Session::rt_cleanup, Controllable::InverseGroup);
+                               _session->set_controls (route_list_to_control_list (rl, &Stripable::rec_enable_control), !track()->rec_enable_control()->get_value(), Controllable::InverseGroup);
                        }
 
                } else if (Keyboard::is_context_menu_event (ev)) {
@@ -758,22 +757,14 @@ RouteUI::rec_enable_press(GdkEventButton* ev)
 
                } else {
 
-                       boost::shared_ptr<RouteList> rl (new RouteList);
-                       rl->push_back (route());
-                       DisplaySuspender ds;
-                       _session->set_record_enabled (rl, !_route->record_enabled());
+                       boost::shared_ptr<Track> trk = track();
+                       trk->rec_enable_control()->set_value (!trk->rec_enable_control()->get_value(), Controllable::UseGroup);
                }
        }
 
        return false;
 }
 
-void
-RouteUI::monitoring_changed ()
-{
-       update_monitoring_display ();
-}
-
 void
 RouteUI::update_monitoring_display ()
 {
@@ -789,7 +780,7 @@ RouteUI::update_monitoring_display ()
 
        MonitorState ms = t->monitoring_state();
 
-       if (t->monitoring_choice() & MonitorInput) {
+       if (t->monitoring_control()->monitoring_choice() & MonitorInput) {
                monitor_input_button->set_active_state (Gtkmm2ext::ExplicitActive);
        } else {
                if (ms & MonitoringInput) {
@@ -799,7 +790,7 @@ RouteUI::update_monitoring_display ()
                }
        }
 
-       if (t->monitoring_choice() & MonitorDisk) {
+       if (t->monitoring_control()->monitoring_choice() & MonitorDisk) {
                monitor_disk_button->set_active_state (Gtkmm2ext::ExplicitActive);
        } else {
                if (ms & MonitoringDisk) {
@@ -855,8 +846,8 @@ RouteUI::monitor_release (GdkEventButton* ev, MonitorChoice monitor_choice)
           signal together, which requires yet more buffers.
        */
 
-       if (t->monitoring_choice() & monitor_choice) {
-               mc = MonitorChoice (t->monitoring_choice() & ~monitor_choice);
+       if (t->monitoring_control()->monitoring_choice() & monitor_choice) {
+               mc = MonitorChoice (t->monitoring_control()->monitoring_choice() & ~monitor_choice);
        } else {
                /* this line will change when the options are non-orthogonal */
                // mc = MonitorChoice (t->monitoring_choice() | monitor_choice);
@@ -879,7 +870,7 @@ RouteUI::monitor_release (GdkEventButton* ev, MonitorChoice monitor_choice)
        }
 
        DisplaySuspender ds;
-       _session->set_monitoring (rl, mc, Session::rt_cleanup, Controllable::UseGroup);
+       _session->set_controls (route_list_to_control_list (rl, &Stripable::monitoring_control), (double) mc, Controllable::UseGroup);
 
        return false;
 }
@@ -887,41 +878,64 @@ RouteUI::monitor_release (GdkEventButton* ev, MonitorChoice monitor_choice)
 void
 RouteUI::build_record_menu ()
 {
-       if (record_menu) {
-               return;
-       }
-
-       /* no rec-button context menu for non-MIDI tracks
-        */
-
-       if (is_midi_track()) {
+       if (!record_menu) {
                record_menu = new Menu;
                record_menu->set_name ("ArdourContextMenu");
-
                using namespace Menu_Helpers;
                MenuList& items = record_menu->items();
 
-               items.push_back (CheckMenuElem (_("Step Entry"), sigc::mem_fun (*this, &RouteUI::toggle_step_edit)));
-               step_edit_item = dynamic_cast<Gtk::CheckMenuItem*> (&items.back());
+               items.push_back (CheckMenuElem (_("Rec-Safe"), sigc::mem_fun (*this, &RouteUI::toggle_rec_safe)));
+               rec_safe_item = dynamic_cast<Gtk::CheckMenuItem*> (&items.back());
 
-               if (_route->record_enabled()) {
-                       step_edit_item->set_sensitive (false);
+               if (is_midi_track()) {
+                       items.push_back (SeparatorElem());
+                       items.push_back (CheckMenuElem (_("Step Entry"), sigc::mem_fun (*this, &RouteUI::toggle_step_edit)));
+                       step_edit_item = dynamic_cast<Gtk::CheckMenuItem*> (&items.back());
                }
+       }
 
+       if (step_edit_item) {
+               if (track()->rec_enable_control()->get_value()) {
+                       step_edit_item->set_sensitive (false);
+               }
                step_edit_item->set_active (midi_track()->step_editing());
        }
+       if (rec_safe_item) {
+               rec_safe_item->set_sensitive (!_route->rec_enable_control()->get_value());
+               rec_safe_item->set_active (_route->rec_safe_control()->get_value());
+       }
 }
 
 void
 RouteUI::toggle_step_edit ()
 {
-       if (!is_midi_track() || _route->record_enabled()) {
+       if (!is_midi_track() || track()->rec_enable_control()->get_value()) {
                return;
        }
 
        midi_track()->set_step_editing (step_edit_item->get_active());
 }
 
+void
+RouteUI::toggle_rec_safe ()
+{
+       boost::shared_ptr<AutomationControl> rs = _route->rec_safe_control();
+
+       if (!rs) {
+               return;
+       }
+
+       /* This check is made inside the control too, but dong it here can't
+        * hurt.
+        */
+
+       if (_route->rec_enable_control()->get_value()) {
+               return;
+       }
+
+       rs->set_value (rec_safe_item->get_active (), Controllable::UseGroup);
+}
+
 void
 RouteUI::step_edit_changed (bool yn)
 {
@@ -1113,41 +1127,42 @@ RouteUI::send_blink (bool onoff)
 }
 
 Gtkmm2ext::ActiveState
-RouteUI::solo_active_state (boost::shared_ptr<Route> r)
+RouteUI::solo_active_state (boost::shared_ptr<Stripable> s)
 {
-       if (r->is_master() || r->is_monitor()) {
+       boost::shared_ptr<SoloControl> sc = s->solo_control();
+
+       if (!sc) {
                return Gtkmm2ext::Off;
        }
 
-       if (Config->get_solo_control_is_listen_control()) {
-
-               if (r->listening_via_monitor()) {
-                       return Gtkmm2ext::ExplicitActive;
-               } else {
-                       return Gtkmm2ext::Off;
-               }
-
+       if (!sc->can_solo()) {
+               return Gtkmm2ext::Off;
        }
 
-       if (r->soloed()) {
-                if (!r->self_soloed()) {
-                        return Gtkmm2ext::ImplicitActive;
-                } else {
-                        return Gtkmm2ext::ExplicitActive;
-                }
+
+       if (sc->self_soloed()) {
+               return Gtkmm2ext::ExplicitActive;
+       } else if (sc->soloed_by_others()) {
+               return Gtkmm2ext::ImplicitActive;
        } else {
                return Gtkmm2ext::Off;
        }
 }
 
 Gtkmm2ext::ActiveState
-RouteUI::solo_isolate_active_state (boost::shared_ptr<Route> r)
+RouteUI::solo_isolate_active_state (boost::shared_ptr<Stripable> s)
 {
-       if (r->is_master() || r->is_monitor()) {
+       boost::shared_ptr<SoloIsolateControl> sc = s->solo_isolate_control();
+
+       if (!sc) {
+               return Gtkmm2ext::Off;
+       }
+
+       if (s->is_master() || s->is_monitor()) {
                return Gtkmm2ext::Off;
        }
 
-       if (r->solo_isolated()) {
+       if (sc->solo_isolated()) {
                return Gtkmm2ext::ExplicitActive;
        } else {
                return Gtkmm2ext::Off;
@@ -1155,13 +1170,19 @@ RouteUI::solo_isolate_active_state (boost::shared_ptr<Route> r)
 }
 
 Gtkmm2ext::ActiveState
-RouteUI::solo_safe_active_state (boost::shared_ptr<Route> r)
+RouteUI::solo_safe_active_state (boost::shared_ptr<Stripable> s)
 {
-       if (r->is_master() || r->is_monitor()) {
+       boost::shared_ptr<SoloSafeControl> sc = s->solo_safe_control();
+
+       if (!sc) {
+               return Gtkmm2ext::Off;
+       }
+
+       if (s->is_master() || s->is_monitor()) {
                return Gtkmm2ext::Off;
        }
 
-       if (r->solo_safe()) {
+       if (sc->solo_safe()) {
                return Gtkmm2ext::ExplicitActive;
        } else {
                return Gtkmm2ext::Off;
@@ -1171,13 +1192,13 @@ RouteUI::solo_safe_active_state (boost::shared_ptr<Route> r)
 void
 RouteUI::update_solo_display ()
 {
-       bool yn = _route->solo_safe ();
+       bool yn = _route->solo_safe_control()->solo_safe ();
 
        if (solo_safe_check && solo_safe_check->get_active() != yn) {
                solo_safe_check->set_active (yn);
        }
 
-       yn = _route->solo_isolated ();
+       yn = _route->solo_isolate_control()->solo_isolated ();
 
        if (solo_isolated_check && solo_isolated_check->get_active() != yn) {
                solo_isolated_check->set_active (yn);
@@ -1186,7 +1207,7 @@ RouteUI::update_solo_display ()
         set_button_names ();
 
         if (solo_isolated_led) {
-               if (_route->solo_isolated()) {
+               if (_route->solo_isolate_control()->solo_isolated()) {
                        solo_isolated_led->set_active_state (Gtkmm2ext::ExplicitActive);
                } else {
                        solo_isolated_led->unset_active_state ();
@@ -1194,7 +1215,7 @@ RouteUI::update_solo_display ()
         }
 
         if (solo_safe_led) {
-               if (_route->solo_safe()) {
+               if (_route->solo_safe_control()->solo_safe()) {
                        solo_safe_led->set_active_state (Gtkmm2ext::ExplicitActive);
                } else {
                        solo_safe_led->unset_active_state ();
@@ -1206,7 +1227,7 @@ RouteUI::update_solo_display ()
         /* some changes to solo status can affect mute display, so catch up
          */
 
-        update_mute_display ();
+       update_mute_display ();
 }
 
 void
@@ -1216,19 +1237,25 @@ RouteUI::solo_changed_so_update_mute ()
 }
 
 ActiveState
-RouteUI::mute_active_state (Session* s, boost::shared_ptr<Route> r)
+RouteUI::mute_active_state (Session*, boost::shared_ptr<Stripable> s)
 {
-       if (r->is_monitor()) {
-               return ActiveState(0);
+       boost::shared_ptr<MuteControl> mc = s->mute_control();
+
+       if (s->is_monitor()) {
+               return Gtkmm2ext::Off;
        }
 
+       if (!mc) {
+               return Gtkmm2ext::Off;
+       }
 
        if (Config->get_show_solo_mutes() && !Config->get_solo_control_is_listen_control ()) {
 
-               if (r->muted ()) {
+               if (mc->muted_by_self ()) {
                        /* full mute */
                        return Gtkmm2ext::ExplicitActive;
-               } else if (r->muted_by_others()) {
+               } else if (mc->muted_by_others_soloing () || mc->muted_by_masters ()) {
+                       /* this will reflect both solo mutes AND master mutes */
                        return Gtkmm2ext::ImplicitActive;
                } else {
                        /* no mute at all */
@@ -1237,9 +1264,12 @@ RouteUI::mute_active_state (Session* s, boost::shared_ptr<Route> r)
 
        } else {
 
-               if (r->muted()) {
+               if (mc->muted_by_self()) {
                        /* full mute */
                        return Gtkmm2ext::ExplicitActive;
+               } else if (mc->muted_by_masters ()) {
+                       /* this shows only master mutes, not mute-by-others-soloing */
+                       return Gtkmm2ext::ImplicitActive;
                } else {
                        /* no mute at all */
                        return Gtkmm2ext::Off;
@@ -1259,18 +1289,17 @@ RouteUI::update_mute_display ()
         mute_button->set_active_state (mute_active_state (_session, _route));
 }
 
+
 void
 RouteUI::route_rec_enable_changed ()
 {
-       blink_rec_display(true);  //this lets the button change "immediately" rather than wait for the next blink
-       update_monitoring_display ();
+       blink_rec_display (true);  //this lets the button change "immediately" rather than wait for the next blink
 }
 
 void
 RouteUI::session_rec_enable_changed ()
 {
-       blink_rec_display(true);  //this lets the button change "immediately" rather than wait for the next blink
-       update_monitoring_display ();
+       blink_rec_display (true);  //this lets the button change "immediately" rather than wait for the next blink
 }
 
 void
@@ -1279,11 +1308,16 @@ RouteUI::blink_rec_display (bool blinkOn)
        if (!rec_enable_button || !_route) {
                return;
        }
+
        if (boost::dynamic_pointer_cast<Send>(_current_delivery)) {
                return;
        }
 
-       if (_route->record_enabled()) {
+       if (!is_track()) {
+               return;
+       }
+
+       if (track()->rec_enable_control()->get_value()) {
                 switch (_session->record_status ()) {
                 case Session::Recording:
                         rec_enable_button->set_active_state (Gtkmm2ext::ExplicitActive);
@@ -1291,12 +1325,12 @@ RouteUI::blink_rec_display (bool blinkOn)
 
                 case Session::Disabled:
                 case Session::Enabled:
-                        if ( UIConfiguration::instance().get_blink_rec_arm() )
-                                                       rec_enable_button->set_active_state ( blinkOn ? Gtkmm2ext::ExplicitActive : Gtkmm2ext::Off );
-                                               else
-                                                       rec_enable_button->set_active_state ( ImplicitActive );
+                       if (UIConfiguration::instance().get_blink_rec_arm()) {
+                               rec_enable_button->set_active_state ( blinkOn ? Gtkmm2ext::ExplicitActive : Gtkmm2ext::Off );
+                       } else {
+                               rec_enable_button->set_active_state ( ImplicitActive );
+                       }
                         break;
-
                 }
 
                 if (step_edit_item) {
@@ -1311,7 +1345,6 @@ RouteUI::blink_rec_display (bool blinkOn)
                 }
        }
 
-
        check_rec_enable_sensitivity ();
 }
 
@@ -1326,14 +1359,14 @@ RouteUI::build_solo_menu (void)
        Gtk::CheckMenuItem* check;
 
        check = new Gtk::CheckMenuItem(_("Solo Isolate"));
-       check->set_active (_route->solo_isolated());
+       check->set_active (_route->solo_isolate_control()->solo_isolated());
        check->signal_toggled().connect (sigc::bind (sigc::mem_fun (*this, &RouteUI::toggle_solo_isolated), check));
        items.push_back (CheckMenuElem(*check));
         solo_isolated_check = dynamic_cast<Gtk::CheckMenuItem*>(&items.back());
        check->show_all();
 
        check = new Gtk::CheckMenuItem(_("Solo Safe"));
-       check->set_active (_route->solo_safe());
+       check->set_active (_route->solo_safe_control()->solo_safe());
        check->signal_toggled().connect (sigc::bind (sigc::mem_fun (*this, &RouteUI::toggle_solo_safe), check));
        items.push_back (CheckMenuElem(*check));
         solo_safe_check = dynamic_cast<Gtk::CheckMenuItem*>(&items.back());
@@ -1387,16 +1420,16 @@ RouteUI::build_mute_menu(void)
 void
 RouteUI::init_mute_menu(MuteMaster::MutePoint mp, Gtk::CheckMenuItem* check)
 {
-       check->set_active (_route->mute_points() & mp);
+       check->set_active (_route->mute_control()->mute_points() & mp);
 }
 
 void
 RouteUI::toggle_mute_menu(MuteMaster::MutePoint mp, Gtk::CheckMenuItem* check)
 {
        if (check->get_active()) {
-               _route->set_mute_points (MuteMaster::MutePoint (_route->mute_points() | mp));
+               _route->mute_control()->set_mute_points (MuteMaster::MutePoint (_route->mute_control()->mute_points() | mp));
        } else {
-               _route->set_mute_points (MuteMaster::MutePoint (_route->mute_points() & ~mp));
+               _route->mute_control()->set_mute_points (MuteMaster::MutePoint (_route->mute_control()->mute_points() & ~mp));
        }
 }
 
@@ -1406,7 +1439,7 @@ RouteUI::muting_change ()
        ENSURE_GUI_THREAD (*this, &RouteUI::muting_change)
 
        bool yn;
-       MuteMaster::MutePoint current = _route->mute_points ();
+       MuteMaster::MutePoint current = _route->mute_control()->mute_points ();
 
        yn = (current & MuteMaster::PreFader);
 
@@ -1441,7 +1474,7 @@ RouteUI::solo_isolate_button_release (GdkEventButton* ev)
        }
 
        bool view = solo_isolated_led->active_state();
-       bool model = _route->solo_isolated();
+       bool model = _route->solo_isolate_control()->solo_isolated();
 
        /* called BEFORE the view has changed */
 
@@ -1451,11 +1484,11 @@ RouteUI::solo_isolate_button_release (GdkEventButton* ev)
                        if (model) {
                                /* disable isolate for all routes */
                                DisplaySuspender ds;
-                               _session->set_solo_isolated (_session->get_routes(), false, Session::rt_cleanup, Controllable::NoGroup);
+                               _session->set_controls (route_list_to_control_list (_session->get_routes(), &Stripable::solo_isolate_control), 0.0, Controllable::NoGroup);
                        } else {
                                /* enable isolate for all routes */
                                DisplaySuspender ds;
-                               _session->set_solo_isolated (_session->get_routes(), true, Session::rt_cleanup, Controllable::NoGroup);
+                               _session->set_controls (route_list_to_control_list (_session->get_routes(), &Stripable::solo_isolate_control), 1.0, Controllable::NoGroup);
                        }
 
                } else {
@@ -1467,7 +1500,7 @@ RouteUI::solo_isolate_button_release (GdkEventButton* ev)
                                boost::shared_ptr<RouteList> rl (new RouteList);
                                rl->push_back (_route);
                                DisplaySuspender ds;
-                               _session->set_solo_isolated (rl, !view, Session::rt_cleanup, Controllable::NoGroup);
+                               _session->set_controls (route_list_to_control_list (rl, &Stripable::solo_isolate_control), view ? 0.0 : 1.0, Controllable::NoGroup);
                        }
                }
        }
@@ -1483,7 +1516,7 @@ RouteUI::solo_safe_button_release (GdkEventButton* ev)
        }
 
        bool view = solo_safe_led->active_state();
-       bool model = _route->solo_safe();
+       bool model = _route->solo_safe_control()->solo_safe();
 
        if (ev->button == 1) {
                if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::TertiaryModifier))) {
@@ -1492,20 +1525,20 @@ RouteUI::solo_safe_button_release (GdkEventButton* ev)
                                /* disable solo safe for all routes */
                                DisplaySuspender ds;
                                for (RouteList::iterator i = rl->begin(); i != rl->end(); ++i) {
-                                       (*i)->set_solo_safe (false, Controllable::NoGroup);
+                                       (*i)->solo_safe_control()->set_value (0.0, Controllable::NoGroup);
                                }
                        } else {
                                /* enable solo safe for all routes */
                                DisplaySuspender ds;
                                for (RouteList::iterator i = rl->begin(); i != rl->end(); ++i) {
-                                       (*i)->set_solo_safe (true, Controllable::NoGroup);
+                                       (*i)->solo_safe_control()->set_value (1.0, Controllable::NoGroup);
                                }
                        }
                }
                else {
                        if (model == view) {
                                /* flip just this route */
-                               _route->set_solo_safe (!view, Controllable::NoGroup);
+                               _route->solo_safe_control()->set_value (view ? 0.0 : 1.0, Controllable::NoGroup);
                        }
                }
        }
@@ -1517,19 +1550,19 @@ void
 RouteUI::toggle_solo_isolated (Gtk::CheckMenuItem* check)
 {
         bool view = check->get_active();
-        bool model = _route->solo_isolated();
+        bool model = _route->solo_isolate_control()->solo_isolated();
 
         /* called AFTER the view has changed */
 
         if (model != view) {
-               _route->set_solo_isolated (view, Controllable::UseGroup);
+               _route->solo_isolate_control()->set_value (view ? 1.0 : 0.0, Controllable::UseGroup);
         }
 }
 
 void
 RouteUI::toggle_solo_safe (Gtk::CheckMenuItem* check)
 {
-       _route->set_solo_safe (check->get_active(), Controllable::UseGroup);
+       _route->solo_safe_control()->set_value (check->get_active() ? 1.0 : 0.0, Controllable::UseGroup);
 }
 
 /** Ask the user to choose a colour, and then apply that color to my route
@@ -1538,10 +1571,11 @@ void
 RouteUI::choose_color ()
 {
        bool picked;
-       Gdk::Color const color = Gtkmm2ext::UI::instance()->get_color (_("Color Selection"), picked, &_color);
+       Gdk::Color c (gdk_color_from_rgb (_route->presentation_info().color()));
+       Gdk::Color const color = Gtkmm2ext::UI::instance()->get_color (_("Color Selection"), picked, &c);
 
        if (picked) {
-               set_color(color);
+               set_color (gdk_color_to_rgba (color));
        }
 }
 
@@ -1549,22 +1583,9 @@ RouteUI::choose_color ()
  *  the route is in a group which shares its color with its routes.
  */
 void
-RouteUI::set_color (const Gdk::Color & c)
+RouteUI::set_color (uint32_t c)
 {
-       /* leave _color alone in the group case so that tracks can retain their
-        * own pre-group colors.
-        */
-
-       char buf[64];
-       _color = c;
-       snprintf (buf, sizeof (buf), "%d:%d:%d", c.get_red(), c.get_green(), c.get_blue());
-
-       /* note: we use the route state ID here so that color is the same for both
-          the time axis view and the mixer strip
-       */
-
-       gui_object_state().set_property<string> (route_state_id(), X_("color"), buf);
-       _route->gui_changed ("color", (void *) 0); /* EMIT_SIGNAL */
+       _route->presentation_info().set_color (c);
 }
 
 /** @return GUI state ID for things that are common to the route in all its representations */
@@ -1577,21 +1598,11 @@ RouteUI::route_state_id () const
 int
 RouteUI::set_color_from_route ()
 {
-       const string str = gui_object_state().get_string (route_state_id(), X_("color"));
-
-       if (str.empty()) {
-               return 1;
+       if (_route->presentation_info().color_set()) {
+               return 0; /* nothing to do */
        }
 
-       int r, g, b;
-
-       sscanf (str.c_str(), "%d:%d:%d", &r, &g, &b);
-
-       _color.set_red (r);
-       _color.set_green (g);
-       _color.set_blue (b);
-
-       return 0;
+       return 1; /* pick a color */
 }
 
 /** @return true if this name should be used for the route, otherwise false */
@@ -1659,14 +1670,6 @@ RouteUI::route_rename ()
 
 }
 
-void
-RouteUI::property_changed (const PropertyChange& what_changed)
-{
-       if (what_changed.contains (ARDOUR::Properties::name)) {
-               name_label.set_text (_route->name());
-       }
-}
-
 void
 RouteUI::toggle_comment_editor ()
 {
@@ -1831,12 +1834,6 @@ RouteUI::has_audio_outputs () const
        return (_route->n_outputs().n_audio() > 0);
 }
 
-string
-RouteUI::name() const
-{
-       return _route->name();
-}
-
 void
 RouteUI::map_frozen ()
 {
@@ -1845,14 +1842,7 @@ RouteUI::map_frozen ()
        AudioTrack* at = dynamic_cast<AudioTrack*>(_route.get());
 
        if (at) {
-               switch (at->freeze_state()) {
-               case AudioTrack::Frozen:
-                       rec_enable_button->set_sensitive (false);
-                       break;
-               default:
-                       rec_enable_button->set_sensitive (true);
-                       break;
-               }
+               check_rec_enable_sensitivity ();
        }
 }
 
@@ -1925,12 +1915,27 @@ RouteUI::save_as_template ()
 void
 RouteUI::check_rec_enable_sensitivity ()
 {
+       if (!rec_enable_button) {
+               assert (0); // This should not happen
+               return;
+       }
+       if (!_session->writable()) {
+               rec_enable_button->set_sensitive (false);
+               return;
+       }
+
        if (_session->transport_rolling() && rec_enable_button->active_state() && Config->get_disable_disarm_during_roll()) {
                rec_enable_button->set_sensitive (false);
+       } else if (is_audio_track ()  && track()->freeze_state() == AudioTrack::Frozen) {
+               rec_enable_button->set_sensitive (false);
        } else {
                rec_enable_button->set_sensitive (true);
        }
-
+       if (_route && _route->rec_safe_control () && _route->rec_safe_control()->get_value()) {
+               rec_enable_button->set_visual_state (Gtkmm2ext::VisualState (solo_button->visual_state() | Gtkmm2ext::Insensitive));
+       } else {
+               rec_enable_button->set_visual_state (Gtkmm2ext::VisualState (solo_button->visual_state() & ~Gtkmm2ext::Insensitive));
+       }
        update_monitoring_display ();
 }
 
@@ -1959,80 +1964,25 @@ RouteUI::parameter_changed (string const & p)
 void
 RouteUI::step_gain_up ()
 {
-       _route->set_gain (dB_to_coefficient (accurate_coefficient_to_dB (_route->gain_control()->get_value()) + 0.1), Controllable::UseGroup);
+       _route->gain_control()->set_value (dB_to_coefficient (accurate_coefficient_to_dB (_route->gain_control()->get_value()) + 0.1), Controllable::UseGroup);
 }
 
 void
 RouteUI::page_gain_up ()
 {
-       _route->set_gain (dB_to_coefficient (accurate_coefficient_to_dB (_route->gain_control()->get_value()) + 0.5), Controllable::UseGroup);
+       _route->gain_control()->set_value (dB_to_coefficient (accurate_coefficient_to_dB (_route->gain_control()->get_value()) + 0.5), Controllable::UseGroup);
 }
 
 void
 RouteUI::step_gain_down ()
 {
-       _route->set_gain (dB_to_coefficient (accurate_coefficient_to_dB (_route->gain_control()->get_value()) - 0.1), Controllable::UseGroup);
+       _route->gain_control()->set_value (dB_to_coefficient (accurate_coefficient_to_dB (_route->gain_control()->get_value()) - 0.1), Controllable::UseGroup);
 }
 
 void
 RouteUI::page_gain_down ()
 {
-       _route->set_gain (dB_to_coefficient (accurate_coefficient_to_dB (_route->gain_control()->get_value()) - 0.5), Controllable::UseGroup);
-}
-
-void
-RouteUI::open_remote_control_id_dialog ()
-{
-       ArdourDialog dialog (_("Remote Control ID"));
-       SpinButton* spin = 0;
-
-       dialog.get_vbox()->set_border_width (18);
-
-       if (Config->get_remote_model() == UserOrdered) {
-               uint32_t const limit = _session->ntracks() + _session->nbusses () + 4;
-
-               HBox* hbox = manage (new HBox);
-               hbox->set_spacing (6);
-               hbox->pack_start (*manage (new Label (_("Remote control ID:"))));
-               spin = manage (new SpinButton);
-               spin->set_digits (0);
-               spin->set_increments (1, 10);
-               spin->set_range (0, limit);
-               spin->set_value (_route->remote_control_id());
-               hbox->pack_start (*spin);
-               dialog.get_vbox()->pack_start (*hbox);
-
-               dialog.add_button (Stock::CANCEL, RESPONSE_CANCEL);
-               dialog.add_button (Stock::APPLY, RESPONSE_ACCEPT);
-       } else {
-               Label* l = manage (new Label());
-               if (_route->is_master() || _route->is_monitor()) {
-                       l->set_markup (string_compose (_("The remote control ID of %1 is: %2\n\n\n"
-                                                        "The remote control ID of %3 cannot be changed."),
-                                                      Gtkmm2ext::markup_escape_text (_route->name()),
-                                                      _route->remote_control_id(),
-                                                      (_route->is_master() ? _("the master bus") : _("the monitor bus"))));
-               } else {
-                       l->set_markup (string_compose (_("The remote control ID of %5 is: %2\n\n\n"
-                                                        "Remote Control IDs are currently determined by track/bus ordering in %6.\n\n"
-                                                        "%3Use the User Interaction tab of the Preferences window if you want to change this%4"),
-                                                      (is_track() ? _("track") : _("bus")),
-                                                      _route->remote_control_id(),
-                                                      "<span size=\"small\" style=\"italic\">",
-                                                      "</span>",
-                                                      Gtkmm2ext::markup_escape_text (_route->name()),
-                                                      PROGRAM_NAME));
-               }
-               dialog.get_vbox()->pack_start (*l);
-               dialog.add_button (Stock::OK, RESPONSE_CANCEL);
-       }
-
-       dialog.show_all ();
-       int const r = dialog.run ();
-
-       if (r == RESPONSE_ACCEPT && spin) {
-               _route->set_remote_control_id (spin->get_value_as_int ());
-       }
+       _route->gain_control()->set_value (dB_to_coefficient (accurate_coefficient_to_dB (_route->gain_control()->get_value()) - 0.5), Controllable::UseGroup);
 }
 
 void
@@ -2070,9 +2020,9 @@ RouteUI::setup_invert_buttons ()
                }
 
                if (N <= _max_invert_buttons) {
-                       UI::instance()->set_tip (*b, string_compose (_("Left-click to invert (phase reverse) channel %1 of this track.  Right-click to show menu."), i + 1));
+                       UI::instance()->set_tip (*b, string_compose (_("Left-click to invert polarity of channel %1 of this track. Right-click to show menu."), i + 1));
                } else {
-                       UI::instance()->set_tip (*b, _("Click to show a menu of channels for inversion (phase reverse)"));
+                       UI::instance()->set_tip (*b, _("Click to show a menu of channels to invert polarity"));
                }
 
                _invert_buttons.push_back (b);
@@ -2095,9 +2045,9 @@ RouteUI::set_invert_button_state ()
 
                ArdourButton* b = _invert_buttons.front ();
 
-               if (_route->phase_invert().count() == _route->phase_invert().size()) {
+               if (_route->phase_control()->count() == _route->phase_control()->size()) {
                        b->set_active_state (Gtkmm2ext::ExplicitActive);
-               } else if (_route->phase_invert().any()) {
+               } else if (_route->phase_control()->any()) {
                        b->set_active_state (Gtkmm2ext::ImplicitActive);
                } else {
                        b->set_active_state (Gtkmm2ext::Off);
@@ -2109,7 +2059,7 @@ RouteUI::set_invert_button_state ()
 
                int j = 0;
                for (vector<ArdourButton*>::iterator i = _invert_buttons.begin(); i != _invert_buttons.end(); ++i, ++j) {
-                       (*i)->set_active (_route->phase_invert (j));
+                       (*i)->set_active (_route->phase_control()->inverted (j));
                }
 
        }
@@ -2122,7 +2072,7 @@ RouteUI::invert_release (GdkEventButton* ev, uint32_t i)
                uint32_t const N = _route->input()->n_ports().n_audio ();
                if (N <= _max_invert_buttons) {
                        /* left-click inverts phase so long as we have a button per channel */
-                       _route->set_phase_invert (i, !_invert_buttons[i]->get_active());
+                       _route->phase_control()->set_phase_invert (i, !_invert_buttons[i]->get_active());
                        return false;
                }
        }
@@ -2153,7 +2103,7 @@ RouteUI::invert_press (GdkEventButton* ev)
                items.push_back (CheckMenuElem (string_compose (X_("Ø%1"), i + 1), sigc::bind (sigc::mem_fun (*this, &RouteUI::invert_menu_toggled), i)));
                Gtk::CheckMenuItem* e = dynamic_cast<Gtk::CheckMenuItem*> (&items.back ());
                ++_i_am_the_modifier;
-               e->set_active (_route->phase_invert (i));
+               e->set_active (_route->phase_control()->inverted (i));
                --_i_am_the_modifier;
        }
 
@@ -2169,7 +2119,8 @@ RouteUI::invert_menu_toggled (uint32_t c)
                return;
        }
 
-       _route->set_phase_invert (c, !_route->phase_invert (c));
+
+       _route->phase_control()->set_phase_invert (c, !_route->phase_control()->inverted (c));
 }
 
 void
@@ -2190,9 +2141,9 @@ RouteUI::request_redraw ()
 
 /** The Route's gui_changed signal has been emitted */
 void
-RouteUI::route_gui_changed (string what_changed)
+RouteUI::route_gui_changed (PropertyChange const& what_changed)
 {
-       if (what_changed == "color") {
+       if (what_changed.contains (Properties::color)) {
                if (set_color_from_route () == 0) {
                        route_color_changed ();
                }
@@ -2218,18 +2169,55 @@ RouteUI::track_mode_changed (void)
 /** @return the color that this route should use; it maybe its own,
     or it maybe that of its route group.
 */
+
 Gdk::Color
-RouteUI::color () const
+RouteUI::route_color () const
 {
+       Gdk::Color c;
        RouteGroup* g = _route->route_group ();
+       string p;
 
        if (g && g->is_color()) {
-               Gdk::Color c;
                set_color_from_rgba (c, GroupTabs::group_color (g));
-               return c;
+       } else {
+
+               /* deal with older 4.x color, which was stored in the GUI object state */
+
+               string p = ARDOUR_UI::instance()->gui_object_state->get_string (route_state_id(), X_("color"));
+
+               if (!p.empty()) {
+
+                       /* old v4.x or earlier session. Use this information */
+
+                       int red, green, blue;
+                       char colon;
+
+                       stringstream ss (p);
+
+                       /* old color format version was:
+
+                          16bit value for red:16 bit value for green:16 bit value for blue
+
+                          decode to rgb ..
+                       */
+
+                       ss >> red;
+                       ss >> colon;
+                       ss >> green;
+                       ss >> colon;
+                       ss >> blue;
+
+                       red >>= 2;
+                       green >>= 2;
+                       blue >>= 2;
+
+                       _route->presentation_info().set_color (RGBA_TO_UINT (red, green, blue, 255));
+               }
+
+               set_color_from_rgba (c, _route->presentation_info().color());
        }
 
-       return _color;
+       return c;
 }
 
 void
@@ -2256,3 +2244,108 @@ RouteUI::route_group() const
 {
        return _route->route_group();
 }
+
+
+RoutePinWindowProxy::RoutePinWindowProxy(std::string const &name, boost::shared_ptr<ARDOUR::Route> route)
+       : WM::ProxyBase (name, string())
+       , _route (boost::weak_ptr<Route> (route))
+{
+       route->DropReferences.connect (going_away_connection, MISSING_INVALIDATOR, boost::bind (&RoutePinWindowProxy::route_going_away, this), gui_context());
+}
+
+RoutePinWindowProxy::~RoutePinWindowProxy()
+{
+       _window = 0;
+}
+
+ARDOUR::SessionHandlePtr*
+RoutePinWindowProxy::session_handle ()
+{
+       ArdourWindow* aw = dynamic_cast<ArdourWindow*> (_window);
+       if (aw) { return aw; }
+       return 0;
+}
+
+Gtk::Window*
+RoutePinWindowProxy::get (bool create)
+{
+       boost::shared_ptr<Route> r = _route.lock ();
+       if (!r) {
+               return 0;
+       }
+
+       if (!_window) {
+               if (!create) {
+                       return 0;
+               }
+               _window = new PluginPinDialog (r);
+               ArdourWindow* aw = dynamic_cast<ArdourWindow*> (_window);
+               if (aw) {
+                       aw->set_session (_session);
+               }
+               _window->show_all ();
+       }
+       return _window;
+}
+
+void
+RoutePinWindowProxy::route_going_away ()
+{
+       delete _window;
+       _window = 0;
+       WM::Manager::instance().remove (this);
+       going_away_connection.disconnect();
+}
+
+void
+RouteUI::maybe_add_route_print_mgr ()
+{
+       if (_route->pinmgr_proxy ()) {
+               return;
+       }
+       RoutePinWindowProxy* wp = new RoutePinWindowProxy (
+                       string_compose ("RPM-%1", _route->id()), _route);
+       wp->set_session (_session);
+
+       const XMLNode* ui_xml = _session->extra_xml (X_("UI"));
+       if (ui_xml) {
+               wp->set_state (*ui_xml, 0);
+       }
+
+#if 0
+       void* existing_ui = _route->pinmgr_proxy ();
+       if (existing_ui) {
+               wp->use_window (*(reinterpret_cast<Gtk::Window*>(existing_ui)));
+       }
+#endif
+       _route->set_pingmgr_proxy (wp);
+
+       WM::Manager::instance().register_window (wp);
+}
+
+void
+RouteUI::manage_pins ()
+{
+       RoutePinWindowProxy* proxy = _route->pinmgr_proxy ();
+       if (proxy) {
+               proxy->get (true);
+               proxy->present();
+       }
+}
+
+bool
+RouteUI::mark_hidden (bool yn)
+{
+       if (yn != _route->presentation_info().hidden()) {
+               _route->presentation_info().set_hidden (yn);
+               return true; // things changed
+       }
+       return false;
+}
+
+boost::shared_ptr<Stripable>
+RouteUI::stripable () const
+{
+       return _route;
+}
+