fix load+save of plugin parameter automation
[ardour.git] / gtk2_ardour / route_time_axis.cc
index 1315ec40a2aadd4aac9937dcacd2be881614b9bd..2c64615ea18f48b1d32391aae9f2db5320563d5d 100644 (file)
@@ -23,6 +23,7 @@
 #include <algorithm>
 #include <string>
 #include <vector>
+#include  <map>
 #include <utility>
 
 #include <sigc++/bind.h>
 #include "pbd/whitespace.h"
 #include "pbd/memento_command.h"
 #include "pbd/enumwriter.h"
+#include "pbd/stateful_diff_command.h"
 
 #include <gtkmm/menu.h>
 #include <gtkmm/menuitem.h>
 #include <gtkmm2ext/gtk_ui.h>
 #include <gtkmm2ext/selector.h>
-#include <gtkmm2ext/stop_signal.h>
 #include <gtkmm2ext/bindable_button.h>
 #include <gtkmm2ext/utils.h>
 
@@ -88,7 +89,6 @@ using namespace PBD;
 using namespace Gtkmm2ext;
 using namespace Gtk;
 using namespace Editing;
-using namespace sigc;
 using namespace std;
 
 Glib::RefPtr<Gdk::Pixbuf> RouteTimeAxisView::slider;
@@ -101,7 +101,7 @@ RouteTimeAxisView::setup_slider_pix ()
        }
 }
 
-RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::shared_ptr<Route> rt, Canvas& canvas)
+RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session* sess, boost::shared_ptr<Route> rt, Canvas& canvas)
        : AxisView(sess)
        , RouteUI(rt, sess)
        , TimeAxisView(sess,ed,(TimeAxisView*) 0, canvas)
@@ -109,13 +109,10 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
        , button_table (3, 3)
        , route_group_button (_("g"))
        , playlist_button (_("p"))
-       , size_button (_("h"))
        , automation_button (_("a"))
-       , visual_button (_("v"))
        , gm (sess, slider, true, 115)
+       , _ignore_track_mode_change (false)
 {
-       set_button_names ();
-
        gm.set_controls (_route, _route->shared_peak_meter(), _route->amp());
        gm.get_level_meter().set_no_show_all();
        gm.get_level_meter().setup_meters(50);
@@ -124,6 +121,7 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
        playlist_menu = 0;
        playlist_action_menu = 0;
        automation_action_menu = 0;
+       plugins_submenu_item = 0;
        mode_menu = 0;
        _view = 0;
 
@@ -131,6 +129,9 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
                _marked_for_display = true;
        }
 
+       mute_changed (0);
+        update_solo_display ();
+
        timestretch_rect = 0;
        no_redraw = false;
        destructive_track_mode_item = 0;
@@ -142,31 +143,14 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
        route_group_button.set_name ("TrackGroupButton");
        playlist_button.set_name ("TrackPlaylistButton");
        automation_button.set_name ("TrackAutomationButton");
-       size_button.set_name ("TrackSizeButton");
-       visual_button.set_name ("TrackVisualButton");
-       hide_button.set_name ("TrackRemoveButton");
 
        route_group_button.unset_flags (Gtk::CAN_FOCUS);
        playlist_button.unset_flags (Gtk::CAN_FOCUS);
        automation_button.unset_flags (Gtk::CAN_FOCUS);
-       size_button.unset_flags (Gtk::CAN_FOCUS);
-       visual_button.unset_flags (Gtk::CAN_FOCUS);
-       hide_button.unset_flags (Gtk::CAN_FOCUS);
-
-       hide_button.add (*(manage (new Image (::get_icon("hide")))));
-       hide_button.show_all ();
-
-       route_group_button.signal_button_release_event().connect (mem_fun(*this, &RouteTimeAxisView::edit_click), false);
-       playlist_button.signal_clicked().connect (mem_fun(*this, &RouteTimeAxisView::playlist_click));
-       automation_button.signal_clicked().connect (mem_fun(*this, &RouteTimeAxisView::automation_click));
-       size_button.signal_button_release_event().connect (mem_fun(*this, &RouteTimeAxisView::size_click), false);
-       visual_button.signal_clicked().connect (mem_fun(*this, &RouteTimeAxisView::visual_click));
-       hide_button.signal_clicked().connect (mem_fun(*this, &RouteTimeAxisView::hide_click));
 
-       solo_button->signal_button_press_event().connect (mem_fun(*this, &RouteUI::solo_press), false);
-       solo_button->signal_button_release_event().connect (mem_fun(*this, &RouteUI::solo_release), false);
-       mute_button->signal_button_press_event().connect (mem_fun(*this, &RouteUI::mute_press), false);
-       mute_button->signal_button_release_event().connect (mem_fun(*this, &RouteUI::mute_release), false);
+       route_group_button.signal_button_release_event().connect (sigc::mem_fun(*this, &RouteTimeAxisView::route_group_click), false);
+       playlist_button.signal_clicked().connect (sigc::mem_fun(*this, &RouteTimeAxisView::playlist_click));
+       automation_button.signal_clicked().connect (sigc::mem_fun(*this, &RouteTimeAxisView::automation_click));
 
        if (is_track()) {
 
@@ -185,49 +169,40 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
                }
                rec_enable_button->show_all ();
 
-               rec_enable_button->signal_button_press_event().connect (mem_fun(*this, &RouteUI::rec_enable_press), false);
-               rec_enable_button->signal_button_release_event().connect (mem_fun(*this, &RouteUI::rec_enable_release), false);
                controls_table.attach (*rec_enable_button, 5, 6, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
-               ARDOUR_UI::instance()->tooltips().set_tip(*rec_enable_button, _("Record"));
 
-               rec_enable_button->set_sensitive (_session.writable());
+                if (is_midi_track()) {
+                        ARDOUR_UI::instance()->set_tip(*rec_enable_button, _("Record (Right-click for Step Edit)"));
+                } else {
+                        ARDOUR_UI::instance()->set_tip(*rec_enable_button, _("Record"));
+                }
+
+               rec_enable_button->set_sensitive (_session->writable());
        }
 
        controls_hbox.pack_start(gm.get_level_meter(), false, false);
-       _route->meter_change.connect (mem_fun(*this, &RouteTimeAxisView::meter_changed));
-       _route->input()->changed.connect (mem_fun(*this, &RouteTimeAxisView::io_changed));
-       _route->output()->changed.connect (mem_fun(*this, &RouteTimeAxisView::io_changed));
+       _route->meter_change.connect (*this, invalidator (*this), bind (&RouteTimeAxisView::meter_changed, this), gui_context());
+       _route->input()->changed.connect (*this, invalidator (*this), ui_bind (&RouteTimeAxisView::io_changed, this, _1, _2), gui_context());
+       _route->output()->changed.connect (*this, invalidator (*this), ui_bind (&RouteTimeAxisView::io_changed, this, _1, _2), gui_context());
 
        controls_table.attach (*mute_button, 6, 7, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
-       controls_table.attach (*solo_button, 7, 8, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
+
+        if (!_route->is_master()) {
+                controls_table.attach (*solo_button, 7, 8, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
+        }
 
        controls_table.attach (route_group_button, 7, 8, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
        controls_table.attach (gm.get_gain_slider(), 0, 5, 1, 2, Gtk::SHRINK, Gtk::SHRINK, 0, 0);
 
-       ARDOUR_UI::instance()->tooltips().set_tip(*solo_button,_("Solo"));
-       ARDOUR_UI::instance()->tooltips().set_tip(*mute_button,_("Mute"));
-       ARDOUR_UI::instance()->tooltips().set_tip(route_group_button, _("Route Group"));
-       ARDOUR_UI::instance()->tooltips().set_tip(size_button,_("Display Height"));
-       ARDOUR_UI::instance()->tooltips().set_tip(playlist_button,_("Playlist"));
-       ARDOUR_UI::instance()->tooltips().set_tip(automation_button, _("Automation"));
-       ARDOUR_UI::instance()->tooltips().set_tip(visual_button, _("Visual options"));
-       ARDOUR_UI::instance()->tooltips().set_tip(hide_button, _("Hide this track"));
+       ARDOUR_UI::instance()->set_tip(*solo_button,_("Solo"));
+       ARDOUR_UI::instance()->set_tip(*mute_button,_("Mute"));
+       ARDOUR_UI::instance()->set_tip(route_group_button, _("Route Group"));
+       ARDOUR_UI::instance()->set_tip(playlist_button,_("Playlist"));
+       ARDOUR_UI::instance()->set_tip(automation_button, _("Automation"));
 
        label_view ();
 
-       if (0) {
-
-               /* old school - when we used to put an extra row of buttons in place */
-
-               controls_table.attach (hide_button, 0, 1, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-               controls_table.attach (visual_button, 1, 2, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-               controls_table.attach (size_button, 2, 3, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-               controls_table.attach (automation_button, 3, 4, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-
-       } else {
-
-               controls_table.attach (automation_button, 6, 7, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-       }
+       controls_table.attach (automation_button, 6, 7, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
 
        if (is_track() && track()->mode() == ARDOUR::Normal) {
                controls_table.attach (playlist_button, 5, 6, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
@@ -235,42 +210,39 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
 
        _y_position = -1;
 
-       _route->mute_changed.connect (mem_fun(*this, &RouteUI::mute_changed));
-       _route->solo_changed.connect (mem_fun(*this, &RouteUI::solo_changed));
-       _route->processors_changed.connect (mem_fun(*this, &RouteTimeAxisView::processors_changed));
-       _route->NameChanged.connect (mem_fun(*this, &RouteTimeAxisView::route_name_changed));
-       _route->solo_isolated_changed.connect (mem_fun(*this, &RouteUI::solo_changed));
-
+       _route->processors_changed.connect (*this, invalidator (*this), ui_bind (&RouteTimeAxisView::processors_changed, this, _1), gui_context());
+       _route->PropertyChanged.connect (*this, invalidator (*this), ui_bind (&RouteTimeAxisView::route_property_changed, this, _1), gui_context());
 
        if (is_track()) {
 
-               track()->TrackModeChanged.connect (mem_fun(*this, &RouteTimeAxisView::track_mode_changed));
-               track()->FreezeChange.connect (mem_fun(*this, &RouteTimeAxisView::map_frozen));
-               track()->DiskstreamChanged.connect (mem_fun(*this, &RouteTimeAxisView::diskstream_changed));
-               get_diskstream()->SpeedChanged.connect (mem_fun(*this, &RouteTimeAxisView::speed_changed));
+               track()->TrackModeChanged.connect (*this, invalidator (*this), boost::bind (&RouteTimeAxisView::track_mode_changed, this), gui_context());
+               track()->FreezeChange.connect (*this, invalidator (*this), boost::bind (&RouteTimeAxisView::map_frozen, this), gui_context());
+               track()->SpeedChanged.connect (*this, invalidator (*this), boost::bind (&RouteTimeAxisView::speed_changed, this), gui_context());
 
                /* pick up the correct freeze state */
                map_frozen ();
 
        }
 
-       _editor.ZoomChanged.connect (mem_fun(*this, &RouteTimeAxisView::reset_samples_per_unit));
-       ColorsChanged.connect (mem_fun (*this, &RouteTimeAxisView::color_handler));
+       _editor.ZoomChanged.connect (sigc::mem_fun(*this, &RouteTimeAxisView::reset_samples_per_unit));
+       ColorsChanged.connect (sigc::mem_fun (*this, &RouteTimeAxisView::color_handler));
 
-       route_group_menu = new RouteGroupMenu (
-               _session,
-               (RouteGroup::Property) (RouteGroup::Mute | RouteGroup::Solo | RouteGroup::Edit)
-               );
+       PropertyList* plist = new PropertyList();
+       
+       plist->add (ARDOUR::Properties::edit, true);
+       plist->add (ARDOUR::Properties::mute, true);
+       plist->add (ARDOUR::Properties::solo, true);
+       
+       route_group_menu = new RouteGroupMenu (_session, plist);
+       route_group_menu->GroupSelected.connect (sigc::mem_fun (*this, &RouteTimeAxisView::set_route_group_from_menu));
 
-       route_group_menu->GroupSelected.connect (mem_fun (*this, &RouteTimeAxisView::set_route_group_from_menu));
-
-       gm.get_gain_slider().signal_scroll_event().connect(mem_fun(*this, &RouteTimeAxisView::controls_ebox_scroll), false);
+       gm.get_gain_slider().signal_scroll_event().connect(sigc::mem_fun(*this, &RouteTimeAxisView::controls_ebox_scroll), false);
        gm.get_gain_slider().set_name ("TrackGainFader");
 }
 
 RouteTimeAxisView::~RouteTimeAxisView ()
 {
-       GoingAway (); /* EMIT_SIGNAL */
+       CatchDeletion (this);
 
        for (list<ProcessorAutomationInfo*>::iterator i = processor_automation.begin(); i != processor_automation.end(); ++i) {
                delete *i;
@@ -285,10 +257,6 @@ RouteTimeAxisView::~RouteTimeAxisView ()
        delete _view;
        _view = 0;
 
-       for (AutomationTracks::iterator i = _automation_tracks.begin(); i != _automation_tracks.end(); ++i) {
-               delete i->second;
-       }
-
        _automation_tracks.clear ();
 
        delete route_group_menu;
@@ -301,30 +269,39 @@ RouteTimeAxisView::post_construct ()
 
        update_diskstream_display ();
 
+       _subplugin_menu_map.clear ();
        subplugin_menu.items().clear ();
-       _route->foreach_processor (mem_fun (*this, &RouteTimeAxisView::add_processor_to_subplugin_menu));
-       _route->foreach_processor (mem_fun (*this, &RouteTimeAxisView::add_existing_processor_automation_curves));
+       _route->foreach_processor (sigc::mem_fun (*this, &RouteTimeAxisView::add_processor_to_subplugin_menu));
+       _route->foreach_processor (sigc::mem_fun (*this, &RouteTimeAxisView::add_existing_processor_automation_curves));
        reset_processor_automation_curves ();
 }
 
 gint
-RouteTimeAxisView::edit_click (GdkEventButton *ev)
+RouteTimeAxisView::route_group_click (GdkEventButton *ev)
 {
        if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
-               _route->set_route_group (0, this);
-               return FALSE;
+               if (_route->route_group()) {
+                       _route->route_group()->remove (_route);
+               }
+               return false;
        }
 
        route_group_menu->rebuild (_route->route_group ());
        route_group_menu->popup (ev->button, ev->time);
 
-       return FALSE;
+       return false;
 }
 
 void
 RouteTimeAxisView::set_route_group_from_menu (RouteGroup *eg)
 {
-       _route->set_route_group (eg, this);
+       if (eg) {
+               eg->add (_route);
+       } else {
+               if (_route->route_group()) {
+                       _route->route_group()->remove (_route);
+               }
+       }
 }
 
 void
@@ -342,13 +319,15 @@ RouteTimeAxisView::label_view ()
                name_entry.set_text (x);
        }
 
-       ARDOUR_UI::instance()->tooltips().set_tip (name_entry, x);
+       ARDOUR_UI::instance()->set_tip (name_entry, x);
 }
 
 void
-RouteTimeAxisView::route_name_changed ()
+RouteTimeAxisView::route_property_changed (const PropertyChange& what_changed)
 {
-       label_view ();
+       if (what_changed.contains (ARDOUR::Properties::name)) {
+               label_view ();
+       }
 }
 
 void
@@ -363,15 +342,7 @@ RouteTimeAxisView::take_name_changed (void *src)
 void
 RouteTimeAxisView::playlist_click ()
 {
-       // always build a new action menu
-
-       delete playlist_action_menu;
-
-       playlist_action_menu = new Menu;
-       playlist_action_menu->set_name ("ArdourContextMenu");
-
-       build_playlist_menu (playlist_action_menu);
-
+       build_playlist_menu ();
        conditionally_add_to_selection ();
        playlist_action_menu->popup (1, gtk_get_current_event_time());
 }
@@ -418,40 +389,36 @@ RouteTimeAxisView::build_automation_action_menu ()
 {
        using namespace Menu_Helpers;
 
-       automation_action_menu = manage (new Menu);
-       MenuList& automation_items = automation_action_menu->items();
-       automation_action_menu->set_name ("ArdourContextMenu");
-
-       automation_items.push_back (MenuElem (_("Show all automation"),
-                                             mem_fun(*this, &RouteTimeAxisView::show_all_automation)));
-
-       automation_items.push_back (MenuElem (_("Show existing automation"),
-                                             mem_fun(*this, &RouteTimeAxisView::show_existing_automation)));
-
-       automation_items.push_back (MenuElem (_("Hide all automation"),
-                                             mem_fun(*this, &RouteTimeAxisView::hide_all_automation)));
-
-       if (subplugin_menu.get_attach_widget())
-               subplugin_menu.detach();
-
-       automation_items.push_back (MenuElem (_("Plugins"), subplugin_menu));
-       automation_items.back().set_sensitive (!subplugin_menu.items().empty());
+       /* detach subplugin_menu from automation_action_menu before we delete automation_action_menu,
+          otherwise bad things happen (see comment for similar case in MidiTimeAxisView::build_automation_action_menu)
+       */
 
-       map<Evoral::Parameter, RouteAutomationNode*>::iterator i;
-       for (i = _automation_tracks.begin(); i != _automation_tracks.end(); ++i) {
+       detach_menu (subplugin_menu);
 
-               automation_items.push_back (SeparatorElem());
+       _main_automation_menu_map.clear ();
+       delete automation_action_menu;
+       automation_action_menu = new Menu;
 
-               delete i->second->menu_item;
+       MenuList& items = automation_action_menu->items();
 
-               automation_items.push_back(CheckMenuElem (_route->describe_parameter(i->second->param),
-                               bind (mem_fun(*this, &RouteTimeAxisView::toggle_automation_track), i->second->param)));
+       automation_action_menu->set_name ("ArdourContextMenu");
+       
+       items.push_back (MenuElem (_("Show All Automation"),
+                                  sigc::mem_fun(*this, &RouteTimeAxisView::show_all_automation)));
+       
+       items.push_back (MenuElem (_("Show Existing Automation"),
+                                  sigc::mem_fun(*this, &RouteTimeAxisView::show_existing_automation)));
+       
+       items.push_back (MenuElem (_("Hide All Automation"),
+                                  sigc::mem_fun(*this, &RouteTimeAxisView::hide_all_automation)));
 
-               i->second->menu_item = static_cast<Gtk::CheckMenuItem*>(&automation_items.back());
+       items.push_back (SeparatorElem ());
+       
+       /* Attach the plugin submenu. It may have previously been used elsewhere,
+          so it was detached above */
 
-               i->second->menu_item->set_active(show_automation(i->second->param));
-               //i->second->menu_item->set_active(false);
-       }
+       items.push_back (MenuElem (_("Plugins"), subplugin_menu));
+       items.back().set_sensitive (!subplugin_menu.items().empty());
 }
 
 void
@@ -459,10 +426,6 @@ RouteTimeAxisView::build_display_menu ()
 {
        using namespace Menu_Helpers;
 
-       /* get the size menu ready */
-
-       build_size_menu ();
-
        /* prepare it */
 
        TimeAxisView::build_display_menu ();
@@ -472,15 +435,15 @@ RouteTimeAxisView::build_display_menu ()
        MenuList& items = display_menu->items();
        display_menu->set_name ("ArdourContextMenu");
 
-       items.push_back (MenuElem (_("Height"), *size_menu));
-       items.push_back (MenuElem (_("Color"), mem_fun(*this, &RouteTimeAxisView::select_track_color)));
+       items.push_back (MenuElem (_("Color..."), sigc::mem_fun(*this, &RouteTimeAxisView::select_track_color)));
 
        items.push_back (SeparatorElem());
 
        if (!Profile->get_sae()) {
-               items.push_back (MenuElem (_("Remote Control ID..."), mem_fun (*this, &RouteUI::open_remote_control_id_dialog)));
+               items.push_back (MenuElem (_("Remote Control ID..."), sigc::mem_fun (*this, &RouteUI::open_remote_control_id_dialog)));
                /* rebuild this every time */
                build_automation_action_menu ();
+               detach_menu (*automation_action_menu);
                items.push_back (MenuElem (_("Automation"), *automation_action_menu));
                items.push_back (SeparatorElem());
        }
@@ -498,52 +461,60 @@ RouteTimeAxisView::build_display_menu ()
                RadioMenuItem::Group layers_group;
 
                layers_items.push_back(RadioMenuElem (layers_group, _("Overlaid"),
-                               bind (mem_fun (*this, &RouteTimeAxisView::set_layer_display), Overlaid)));
+                               sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::set_layer_display), Overlaid)));
                layers_items.push_back(RadioMenuElem (layers_group, _("Stacked"),
-                               bind (mem_fun (*this, &RouteTimeAxisView::set_layer_display), Stacked)));
+                               sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::set_layer_display), Stacked)));
 
                items.push_back (MenuElem (_("Layers"), *layers_menu));
 
-               Menu* alignment_menu = manage (new Menu);
-               MenuList& alignment_items = alignment_menu->items();
-               alignment_menu->set_name ("ArdourContextMenu");
-
-               RadioMenuItem::Group align_group;
-
-               alignment_items.push_back (RadioMenuElem (align_group, _("Align with existing material"),
-                                       bind (mem_fun(*this, &RouteTimeAxisView::set_align_style), ExistingMaterial)));
-               align_existing_item = dynamic_cast<RadioMenuItem*>(&alignment_items.back());
-               if (get_diskstream()->alignment_style() == ExistingMaterial)
-                       align_existing_item->set_active();
-
-               alignment_items.push_back (RadioMenuElem (align_group, _("Align with capture time"),
-                                       bind (mem_fun(*this, &RouteTimeAxisView::set_align_style), CaptureTime)));
-               align_capture_item = dynamic_cast<RadioMenuItem*>(&alignment_items.back());
-               if (get_diskstream()->alignment_style() == CaptureTime)
-                       align_capture_item->set_active();
-
                if (!Profile->get_sae()) {
+
+                       Menu* alignment_menu = manage (new Menu);
+                       MenuList& alignment_items = alignment_menu->items();
+                       alignment_menu->set_name ("ArdourContextMenu");
+                       
+                       RadioMenuItem::Group align_group;
+                       
+                       alignment_items.push_back (RadioMenuElem (align_group, _("Align With Existing Material"),
+                                                                 sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::set_align_style), ExistingMaterial)));
+                       align_existing_item = dynamic_cast<RadioMenuItem*>(&alignment_items.back());
+                       if (track()->alignment_style() == ExistingMaterial) {
+                               align_existing_item->set_active();
+                       }
+                       
+                       alignment_items.push_back (RadioMenuElem (align_group, _("Align With Capture Time"),
+                                                                 sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::set_align_style), CaptureTime)));
+                       align_capture_item = dynamic_cast<RadioMenuItem*>(&alignment_items.back());
+                       if (track()->alignment_style() == CaptureTime) {
+                               align_capture_item->set_active();
+                       }
+                       
                        items.push_back (MenuElem (_("Alignment"), *alignment_menu));
-                       get_diskstream()->AlignmentStyleChanged.connect (
-                                       mem_fun(*this, &RouteTimeAxisView::align_style_changed));
+                       track()->AlignmentStyleChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteTimeAxisView::align_style_changed, this), gui_context());
+
+                       Menu* mode_menu = manage (new Menu);
+                       MenuList& mode_items = mode_menu->items ();
+                       mode_menu->set_name ("ArdourContextMenu");
 
                        RadioMenuItem::Group mode_group;
-                       items.push_back (RadioMenuElem (mode_group, _("Normal mode"), bind (
-                                       mem_fun (*this, &RouteTimeAxisView::set_track_mode),
+
+                       mode_items.push_back (RadioMenuElem (mode_group, _("Normal Mode"), sigc::bind (
+                                       sigc::mem_fun (*this, &RouteTimeAxisView::set_track_mode),
                                        ARDOUR::Normal)));
-                       normal_track_mode_item = dynamic_cast<RadioMenuItem*>(&items.back());
+                       normal_track_mode_item = dynamic_cast<RadioMenuItem*>(&mode_items.back());
 
-                       items.push_back (RadioMenuElem (mode_group, _("Tape mode"), bind (
-                                       mem_fun (*this, &RouteTimeAxisView::set_track_mode),
+                       mode_items.push_back (RadioMenuElem (mode_group, _("Tape Mode"), sigc::bind (
+                                       sigc::mem_fun (*this, &RouteTimeAxisView::set_track_mode),
                                        ARDOUR::Destructive)));
-                       destructive_track_mode_item = dynamic_cast<RadioMenuItem*>(&items.back());
-
-                       items.push_back (RadioMenuElem (mode_group, _("No layering mode"),
-                                                       bind (mem_fun (*this, &RouteTimeAxisView::set_track_mode), ARDOUR::NonLayered)));
-                       non_layered_track_mode_item = dynamic_cast<RadioMenuItem*>(&items.back());
+                       destructive_track_mode_item = dynamic_cast<RadioMenuItem*>(&mode_items.back());
 
+                       mode_items.push_back (RadioMenuElem (mode_group, _("Non-Layered Mode"),
+                                                       sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::set_track_mode), ARDOUR::NonLayered)));
+                       non_layered_track_mode_item = dynamic_cast<RadioMenuItem*>(&mode_items.back());
 
 
+                       _ignore_track_mode_change = true;
+                       
                        switch (track()->mode()) {
                        case ARDOUR::Destructive:
                                destructive_track_mode_item->set_active ();
@@ -555,33 +526,42 @@ RouteTimeAxisView::build_display_menu ()
                                non_layered_track_mode_item->set_active ();
                                break;
                        }
-               }
-
-               get_diskstream()->AlignmentStyleChanged.connect (
-                               mem_fun(*this, &RouteTimeAxisView::align_style_changed));
+                       
+                       _ignore_track_mode_change = false;
 
-               mode_menu = build_mode_menu();
-               if (mode_menu)
                        items.push_back (MenuElem (_("Mode"), *mode_menu));
+               }
 
                color_mode_menu = build_color_mode_menu();
-               if (color_mode_menu)
+               if (color_mode_menu) {
                        items.push_back (MenuElem (_("Color Mode"), *color_mode_menu));
+               }
+
+               items.push_back (SeparatorElem());
+
+               build_playlist_menu ();
+               items.push_back (MenuElem (_("Playlist"), *playlist_action_menu));
+
+               route_group_menu->rebuild (_route->route_group ());
+               items.push_back (MenuElem (_("Route Group"), *route_group_menu));
+
+               build_automation_action_menu ();
+               items.push_back (MenuElem (_("Automation"), *automation_action_menu));
 
                items.push_back (SeparatorElem());
        }
 
-       items.push_back (CheckMenuElem (_("Active"), mem_fun(*this, &RouteUI::toggle_route_active)));
+       items.push_back (CheckMenuElem (_("Active"), sigc::mem_fun(*this, &RouteUI::toggle_route_active)));
        route_active_menu_item = dynamic_cast<CheckMenuItem *> (&items.back());
        route_active_menu_item->set_active (_route->active());
 
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Hide"), mem_fun(*this, &RouteTimeAxisView::hide_click)));
+       items.push_back (MenuElem (_("Hide"), sigc::bind (sigc::mem_fun(_editor, &PublicEditor::hide_track_in_display), this, false)));
        if (!Profile->get_sae()) {
-               items.push_back (MenuElem (_("Remove"), mem_fun(*this, &RouteUI::remove_this_route)));
+               items.push_back (MenuElem (_("Remove"), sigc::mem_fun(*this, &RouteUI::remove_this_route)));
        } else {
                items.push_front (SeparatorElem());
-               items.push_front (MenuElem (_("Delete"), mem_fun(*this, &RouteUI::remove_this_route)));
+               items.push_front (MenuElem (_("Delete"), sigc::mem_fun(*this, &RouteUI::remove_this_route)));
        }
 }
 
@@ -595,6 +575,10 @@ static bool __reset_item (RadioMenuItem* item, RadioMenuItem* item_2)
 void
 RouteTimeAxisView::set_track_mode (TrackMode mode)
 {
+       if (_ignore_track_mode_change) {
+               return;
+       }
+       
        RadioMenuItem* item;
        RadioMenuItem* other_item;
        RadioMenuItem* other_item_2;
@@ -621,7 +605,7 @@ RouteTimeAxisView::set_track_mode (TrackMode mode)
                return;
        }
 
-       if (item && other_item && other_item_2 && item->get_active() && track()->mode() != mode) {
+       if (item && other_item && other_item_2 && track()->mode() != mode) {
                _set_track_mode (track().get(), mode, other_item, other_item_2);
        }
 }
@@ -635,10 +619,11 @@ RouteTimeAxisView::_set_track_mode (Track* track, TrackMode mode, RadioMenuItem*
 
                if (!needs_bounce) {
                        /* cannot be done */
-                       Glib::signal_idle().connect (bind (sigc::ptr_fun (__reset_item), reset_item, reset_item_2));
+                       Glib::signal_idle().connect (sigc::bind (sigc::ptr_fun (__reset_item), reset_item, reset_item_2));
                        return;
                } else {
                        cerr << "would bounce this one\n";
+                       /* XXX: radio menu item becomes inconsistent with track state in this case */
                        return;
                }
        }
@@ -789,14 +774,14 @@ RouteTimeAxisView::set_height (uint32_t h)
        snprintf (buf, sizeof (buf), "%u", height);
        xml_node->add_property ("height", buf);
 
-       if (height >= hNormal) {
+       if (height >= preset_height (HeightNormal)) {
                reset_meter();
                show_name_entry ();
                hide_name_label ();
 
                gm.get_gain_slider().show();
                mute_button->show();
-               if (!_route || _route->is_control()) {
+               if (!_route || _route->is_monitor()) {
                        solo_button->hide();
                } else {
                        solo_button->show();
@@ -805,16 +790,13 @@ RouteTimeAxisView::set_height (uint32_t h)
                        rec_enable_button->show();
 
                route_group_button.show();
-               hide_button.show();
-               visual_button.show();
-               size_button.show();
                automation_button.show();
 
                if (is_track() && track()->mode() == ARDOUR::Normal) {
                        playlist_button.show();
                }
 
-       } else if (height >= hSmaller) {
+       } else if (height >= preset_height (HeightSmaller)) {
 
                reset_meter();
                show_name_entry ();
@@ -822,7 +804,7 @@ RouteTimeAxisView::set_height (uint32_t h)
 
                gm.get_gain_slider().hide();
                mute_button->show();
-               if (!_route || _route->is_control()) {
+               if (!_route || _route->is_monitor()) {
                        solo_button->hide();
                } else {
                        solo_button->show();
@@ -831,9 +813,6 @@ RouteTimeAxisView::set_height (uint32_t h)
                        rec_enable_button->show();
 
                route_group_button.hide ();
-               hide_button.hide ();
-               visual_button.hide ();
-               size_button.hide ();
                automation_button.hide ();
 
                if (is_track() && track()->mode() == ARDOUR::Normal) {
@@ -864,15 +843,12 @@ RouteTimeAxisView::set_height (uint32_t h)
                        rec_enable_button->hide();
 
                route_group_button.hide ();
-               hide_button.hide ();
-               visual_button.hide ();
-               size_button.hide ();
                automation_button.hide ();
                playlist_button.hide ();
                name_label.set_text (_route->name());
        }
 
-       if (height_changed) {
+       if (height_changed && !no_redraw) {
                /* only emit the signal if the height really changed */
                 _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
        }
@@ -900,8 +876,8 @@ RouteTimeAxisView::set_samples_per_unit (double spu)
 {
        double speed = 1.0;
 
-       if (get_diskstream() != 0) {
-               speed = get_diskstream()->speed();
+       if (track()) {
+               speed = track()->speed();
        }
 
        if (_view) {
@@ -914,7 +890,7 @@ RouteTimeAxisView::set_samples_per_unit (double spu)
 void
 RouteTimeAxisView::align_style_changed ()
 {
-       switch (get_diskstream()->alignment_style()) {
+       switch (track()->alignment_style()) {
        case ExistingMaterial:
                if (!align_existing_item->get_active()) {
                        align_existing_item->set_active();
@@ -947,7 +923,7 @@ RouteTimeAxisView::set_align_style (AlignStyle style)
        }
 
        if (item->get_active()) {
-               get_diskstream()->set_align_style (style);
+               track()->set_align_style (style);
        }
 }
 
@@ -957,15 +933,18 @@ RouteTimeAxisView::rename_current_playlist ()
        ArdourPrompter prompter (true);
        string name;
 
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
-       if (!ds || ds->destructive())
+       boost::shared_ptr<Track> tr = track();
+       if (!tr || tr->destructive()) {
                return;
+       }
 
-       boost::shared_ptr<Playlist> pl = ds->playlist();
-       if (!pl)
+       boost::shared_ptr<Playlist> pl = tr->playlist();
+       if (!pl) {
                return;
+       }
 
-       prompter.set_prompt (_("Name for playlist"));
+       prompter.set_title (_("Rename Playlist"));
+       prompter.set_prompt (_("New name for playlist:"));
        prompter.set_initial_text (pl->name());
        prompter.add_button (_("Rename"), Gtk::RESPONSE_ACCEPT);
        prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, false);
@@ -1023,13 +1002,15 @@ RouteTimeAxisView::use_copy_playlist (bool prompt, vector<boost::shared_ptr<Play
 {
        string name;
 
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
-       if (!ds || ds->destructive())
+       boost::shared_ptr<Track> tr = track ();
+       if (!tr || tr->destructive()) {
                return;
+       }
 
-       boost::shared_ptr<const Playlist> pl = ds->playlist();
-       if (!pl)
+       boost::shared_ptr<const Playlist> pl = tr->playlist();
+       if (!pl) {
                return;
+       }
 
        name = pl->name();
 
@@ -1037,8 +1018,8 @@ RouteTimeAxisView::use_copy_playlist (bool prompt, vector<boost::shared_ptr<Play
                name = resolve_new_group_playlist_name(name, playlists_before_op);
        }
 
-       while (_session.playlists.by_name(name)) {
-               name = Playlist::bump_name (name, _session);
+       while (_session->playlists->by_name(name)) {
+               name = Playlist::bump_name (name, *_session);
        }
 
        // TODO: The prompter "new" button should be de-activated if the user
@@ -1048,7 +1029,8 @@ RouteTimeAxisView::use_copy_playlist (bool prompt, vector<boost::shared_ptr<Play
 
                ArdourPrompter prompter (true);
 
-               prompter.set_prompt (_("Name for Playlist"));
+               prompter.set_title (_("New Copy Playlist"));
+               prompter.set_prompt (_("Name for new playlist:"));
                prompter.set_initial_text (name);
                prompter.add_button (Gtk::Stock::NEW, Gtk::RESPONSE_ACCEPT);
                prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, true);
@@ -1065,8 +1047,8 @@ RouteTimeAxisView::use_copy_playlist (bool prompt, vector<boost::shared_ptr<Play
        }
 
        if (name.length()) {
-               ds->use_copy_playlist ();
-               ds->playlist()->set_name (name);
+               tr->use_copy_playlist ();
+               tr->playlist()->set_name (name);
        }
 }
 
@@ -1075,13 +1057,15 @@ RouteTimeAxisView::use_new_playlist (bool prompt, vector<boost::shared_ptr<Playl
 {
        string name;
 
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
-       if (!ds || ds->destructive())
+       boost::shared_ptr<Track> tr = track ();
+       if (!tr || tr->destructive()) {
                return;
+       }
 
-       boost::shared_ptr<const Playlist> pl = ds->playlist();
-       if (!pl)
+       boost::shared_ptr<const Playlist> pl = tr->playlist();
+       if (!pl) {
                return;
+       }
 
        name = pl->name();
 
@@ -1089,8 +1073,8 @@ RouteTimeAxisView::use_new_playlist (bool prompt, vector<boost::shared_ptr<Playl
                name = resolve_new_group_playlist_name(name,playlists_before_op);
        }
 
-       while (_session.playlists.by_name(name)) {
-               name = Playlist::bump_name (name, _session);
+       while (_session->playlists->by_name(name)) {
+               name = Playlist::bump_name (name, *_session);
        }
 
 
@@ -1098,7 +1082,8 @@ RouteTimeAxisView::use_new_playlist (bool prompt, vector<boost::shared_ptr<Playl
 
                ArdourPrompter prompter (true);
 
-               prompter.set_prompt (_("Name for Playlist"));
+               prompter.set_title (_("New Playlist"));
+               prompter.set_prompt (_("Name for new playlist:"));
                prompter.set_initial_text (name);
                prompter.add_button (Gtk::Stock::NEW, Gtk::RESPONSE_ACCEPT);
                prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, true);
@@ -1114,21 +1099,23 @@ RouteTimeAxisView::use_new_playlist (bool prompt, vector<boost::shared_ptr<Playl
        }
 
        if (name.length()) {
-               ds->use_new_playlist ();
-               ds->playlist()->set_name (name);
+               tr->use_new_playlist ();
+               tr->playlist()->set_name (name);
        }
 }
 
 void
 RouteTimeAxisView::clear_playlist ()
 {
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
-       if (!ds || ds->destructive())
+       boost::shared_ptr<Track> tr = track ();
+       if (!tr || tr->destructive()) {
                return;
+       }
 
-       boost::shared_ptr<Playlist> pl = ds->playlist();
-       if (!pl)
+       boost::shared_ptr<Playlist> pl = tr->playlist();
+       if (!pl) {
                return;
+       }
 
        _editor.clear_playlist (pl);
 }
@@ -1136,20 +1123,15 @@ RouteTimeAxisView::clear_playlist ()
 void
 RouteTimeAxisView::speed_changed ()
 {
-       Gtkmm2ext::UI::instance()->call_slot (mem_fun(*this, &RouteTimeAxisView::reset_samples_per_unit));
-}
-
-void
-RouteTimeAxisView::diskstream_changed ()
-{
-       Gtkmm2ext::UI::instance()->call_slot (mem_fun(*this, &RouteTimeAxisView::update_diskstream_display));
+       Gtkmm2ext::UI::instance()->call_slot (invalidator (*this), boost::bind (&RouteTimeAxisView::reset_samples_per_unit, this));
 }
 
 void
 RouteTimeAxisView::update_diskstream_display ()
 {
-       if (!get_diskstream()) // bus
+       if (!track()) {
                return;
+       }
 
        map_frozen ();
 }
@@ -1169,33 +1151,23 @@ RouteTimeAxisView::selection_click (GdkEventButton* ev)
                return;
        }
 
-       PublicEditor::TrackViewList* tracks = _editor.get_valid_views (this, _route->route_group());
-
-       switch (Keyboard::selection_type (ev->state)) {
+       switch (ArdourKeyboard::selection_type (ev->state)) {
        case Selection::Toggle:
-               _editor.get_selection().toggle (*tracks);
+               _editor.get_selection().toggle (this);
                break;
 
        case Selection::Set:
-               _editor.get_selection().set (*tracks);
+               _editor.get_selection().set (this);
                break;
 
        case Selection::Extend:
-               if (tracks->size() > 1) {
-                       /* add each one, do not "extend" */
-                       _editor.get_selection().add (*tracks);
-               } else {
-                       /* extend to the single track */
-                       _editor.extend_selection_to_track (*tracks->front());
-               }
+               _editor.extend_selection_to_track (*this);
                break;
 
        case Selection::Add:
-               _editor.get_selection().add (*tracks);
+               _editor.get_selection().add (this);
                break;
        }
-
-       delete tracks;
 }
 
 void
@@ -1218,12 +1190,12 @@ RouteTimeAxisView::set_selected_regionviews (RegionSelection& regions)
  * @param results List to add things to.
  */
 void
-RouteTimeAxisView::get_selectables (nframes_t start, nframes_t end, double top, double bot, list<Selectable*>& results)
+RouteTimeAxisView::get_selectables (framepos_t start, framepos_t end, double top, double bot, list<Selectable*>& results)
 {
        double speed = 1.0;
 
-       if (get_diskstream() != 0) {
-               speed = get_diskstream()->speed();
+       if (track() != 0) {
+               speed = track()->speed();
        }
 
        nframes_t start_adjusted = session_frame_to_track_frame(start, speed);
@@ -1258,34 +1230,6 @@ RouteTimeAxisView::get_inverted_selectables (Selection& sel, list<Selectable*>&
        return;
 }
 
-bool
-RouteTimeAxisView::show_automation(Evoral::Parameter param)
-{
-       return (_show_automation.find(param) != _show_automation.end());
-}
-
-/** Retuns 0 if track for \a param doesn't exist.
- */
-RouteTimeAxisView::RouteAutomationNode*
-RouteTimeAxisView::automation_track (Evoral::Parameter param)
-{
-       map<Evoral::Parameter, RouteAutomationNode*>::iterator i = _automation_tracks.find (param);
-
-       if (i != _automation_tracks.end()) {
-               return i->second;
-       } else {
-               return 0;
-       }
-}
-
-/** Shorthand for GainAutomation, etc.
- */
-RouteTimeAxisView::RouteAutomationNode*
-RouteTimeAxisView::automation_track (AutomationType type)
-{
-       return automation_track (Evoral::Parameter(type));
-}
-
 RouteGroup*
 RouteTimeAxisView::route_group () const
 {
@@ -1301,10 +1245,10 @@ RouteTimeAxisView::name() const
 boost::shared_ptr<Playlist>
 RouteTimeAxisView::playlist () const
 {
-       boost::shared_ptr<Diskstream> ds;
+       boost::shared_ptr<Track> tr;
 
-       if ((ds = get_diskstream()) != 0) {
-               return ds->playlist();
+       if ((tr = track()) != 0) {
+               return tr->playlist();
        } else {
                return boost::shared_ptr<Playlist> ();
        }
@@ -1328,42 +1272,25 @@ RouteTimeAxisView::name_entry_changed ()
                return;
        }
 
-       if (!_session.route_name_unique (x)) {
+       if (!_session->route_name_unique (x)) {
                ARDOUR_UI::instance()->popup_error (_("A track already exists with that name"));
                name_entry.set_text (_route->name());
-       } else if (_session.route_name_internal (x)) {
-               ARDOUR_UI::instance()->popup_error (_("You cannot create a track with that name as it is reserved for Ardour"));
+       } else if (_session->route_name_internal (x)) {
+               ARDOUR_UI::instance()->popup_error (string_compose (_("You cannot create a track with that name as it is reserved for %1"),
+                                                                    PROGRAM_NAME));
                name_entry.set_text (_route->name());
        } else {
                _route->set_name (x);
        }
 }
 
-void
-RouteTimeAxisView::visual_click ()
-{
-       popup_display_menu (0);
-}
-
-void
-RouteTimeAxisView::hide_click ()
-{
-       // LAME fix for hide_button refresh fix
-       hide_button.set_sensitive(false);
-
-       _editor.hide_track_in_display (*this);
-
-       hide_button.set_sensitive(true);
-}
-
 boost::shared_ptr<Region>
 RouteTimeAxisView::find_next_region (nframes_t pos, RegionPoint point, int32_t dir)
 {
-       boost::shared_ptr<Diskstream> stream;
-       boost::shared_ptr<Playlist> playlist;
+       boost::shared_ptr<Playlist> pl = playlist ();
 
-       if ((stream = get_diskstream()) != 0 && (playlist = stream->playlist()) != 0) {
-               return playlist->find_next_region (pos, point, dir);
+       if (pl) {
+               return pl->find_next_region (pos, point, dir);
        }
 
        return boost::shared_ptr<Region> ();
@@ -1372,33 +1299,31 @@ RouteTimeAxisView::find_next_region (nframes_t pos, RegionPoint point, int32_t d
 nframes64_t
 RouteTimeAxisView::find_next_region_boundary (nframes64_t pos, int32_t dir)
 {
-       boost::shared_ptr<Diskstream> stream;
-       boost::shared_ptr<Playlist> playlist;
+       boost::shared_ptr<Playlist> pl = playlist ();
 
-       if ((stream = get_diskstream()) != 0 && (playlist = stream->playlist()) != 0) {
-               return playlist->find_next_region_boundary (pos, dir);
+       if (pl) {
+               return pl->find_next_region_boundary (pos, dir);
        }
 
        return -1;
 }
 
-bool
+void
 RouteTimeAxisView::cut_copy_clear (Selection& selection, CutCopyOp op)
 {
        boost::shared_ptr<Playlist> what_we_got;
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
+       boost::shared_ptr<Track> tr = track ();
        boost::shared_ptr<Playlist> playlist;
-       bool ret = false;
 
-       if (ds == 0) {
+       if (tr == 0) {
                /* route is a bus, not a track */
-               return false;
+               return;
        }
 
-       playlist = ds->playlist();
+       playlist = tr->playlist();
 
        TimeSelection time (selection.time);
-       float speed = ds->speed();
+       float const speed = tr->speed();
        if (speed != 1.0f) {
                for (TimeSelection::iterator i = time.begin(); i != time.end(); ++i) {
                        (*i).start = session_frame_to_track_frame((*i).start, speed);
@@ -1406,13 +1331,22 @@ RouteTimeAxisView::cut_copy_clear (Selection& selection, CutCopyOp op)
                }
        }
 
-       XMLNode &before = playlist->get_state();
+        playlist->clear_history ();
+        playlist->clear_owned_history ();
+
        switch (op) {
        case Cut:
                if ((what_we_got = playlist->cut (time)) != 0) {
                        _editor.get_cut_buffer().add (what_we_got);
-                       _session.add_command( new MementoCommand<Playlist>(*playlist.get(), &before, &playlist->get_state()));
-                       ret = true;
+
+                        vector<StatefulDiffCommand*> cmds;
+                        
+                        playlist->rdiff (cmds);
+                        
+                        for (vector<StatefulDiffCommand*>::iterator c = cmds.begin(); c != cmds.end(); ++c) {
+                                _session->add_command (*c);
+                        }
+                        _session->add_command (new StatefulDiffCommand (playlist));
                }
                break;
        case Copy:
@@ -1423,24 +1357,28 @@ RouteTimeAxisView::cut_copy_clear (Selection& selection, CutCopyOp op)
 
        case Clear:
                if ((what_we_got = playlist->cut (time)) != 0) {
-                       _session.add_command( new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
+                        vector<StatefulDiffCommand*> cmds;
+                        
+                        playlist->rdiff (cmds);
+                        
+                        for (vector<StatefulDiffCommand*>::iterator c = cmds.begin(); c != cmds.end(); ++c) {
+                                _session->add_command (*c);
+                        }
+                        _session->add_command (new StatefulDiffCommand (playlist));
                        what_we_got->release ();
-                       ret = true;
                }
                break;
        }
-
-       return ret;
 }
 
 bool
-RouteTimeAxisView::paste (nframes_t pos, float times, Selection& selection, size_t nth)
+RouteTimeAxisView::paste (framepos_t pos, float times, Selection& selection, size_t nth)
 {
        if (!is_track()) {
                return false;
        }
 
-       boost::shared_ptr<Playlist> playlist = get_diskstream()->playlist();
+       boost::shared_ptr<Playlist> pl = playlist ();
        PlaylistSelection::iterator p;
 
        for (p = selection.playlists.begin(); p != selection.playlists.end() && nth; ++p, --nth) {}
@@ -1449,100 +1387,104 @@ RouteTimeAxisView::paste (nframes_t pos, float times, Selection& selection, size
                return false;
        }
 
-       if (get_diskstream()->speed() != 1.0f) {
-               pos = session_frame_to_track_frame(pos, get_diskstream()->speed() );
+       if (track()->speed() != 1.0f) {
+               pos = session_frame_to_track_frame (pos, track()->speed());
        }
 
-       XMLNode &before = playlist->get_state();
-       playlist->paste (*p, pos, times);
-       _session.add_command( new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
+        pl->clear_history ();
+       pl->paste (*p, pos, times);
+       _session->add_command (new StatefulDiffCommand (pl));
 
        return true;
 }
 
 
-TimeAxisView::Children
-RouteTimeAxisView::get_child_list()
-{
-       TimeAxisView::Children redirect_children;
-
-       for (Children::iterator i = children.begin(); i != children.end(); ++i) {
-               if (!(*i)->hidden()) {
-                       redirect_children.push_back(*i);
-               }
-       }
-       return redirect_children;
-}
-
+struct PlaylistSorter {
+    bool operator() (boost::shared_ptr<Playlist> a, boost::shared_ptr<Playlist> b) const {
+            return a->sort_id() < b->sort_id();
+    }
+};
 
 void
-RouteTimeAxisView::build_playlist_menu (Gtk::Menu * menu)
+RouteTimeAxisView::build_playlist_menu ()
 {
        using namespace Menu_Helpers;
 
-       if (!menu || !is_track()) {
+       if (!is_track()) {
                return;
        }
 
-       MenuList& playlist_items = menu->items();
-       menu->set_name ("ArdourContextMenu");
+       delete playlist_action_menu;
+       playlist_action_menu = new Menu;
+       playlist_action_menu->set_name ("ArdourContextMenu");
+
+       MenuList& playlist_items = playlist_action_menu->items();
+       playlist_action_menu->set_name ("ArdourContextMenu");
        playlist_items.clear();
 
        delete playlist_menu;
 
-       playlist_menu = new Menu;
-       playlist_menu->set_name ("ArdourContextMenu");
-
-       vector<boost::shared_ptr<Playlist> > playlists;
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
+        vector<boost::shared_ptr<Playlist> > playlists, playlists_tr;
+       boost::shared_ptr<Track> tr = track();
        RadioMenuItem::Group playlist_group;
 
-       _session.playlists.get (playlists);
-
-       for (vector<boost::shared_ptr<Playlist> >::iterator i = playlists.begin(); i != playlists.end(); ++i) {
-
-               if ((*i)->get_orig_diskstream_id() == ds->id()) {
-                       playlist_items.push_back (RadioMenuElem (playlist_group, (*i)->name(), bind (mem_fun (*this, &RouteTimeAxisView::use_playlist),
-                                                                                                    boost::weak_ptr<Playlist> (*i))));
-
-                       if (ds->playlist()->id() == (*i)->id()) {
-                               static_cast<RadioMenuItem*>(&playlist_items.back())->set_active();
-                       }
-               } else if (ds->playlist()->id() == (*i)->id()) {
-                       playlist_items.push_back (RadioMenuElem (playlist_group, (*i)->name(), bind (mem_fun (*this, &RouteTimeAxisView::use_playlist),
-                                                                                                    boost::weak_ptr<Playlist>(*i))));
-                       static_cast<RadioMenuItem*>(&playlist_items.back())->set_active();
-
+       _session->playlists->get (playlists);
+
+        /* find the playlists for this diskstream */
+        for (vector<boost::shared_ptr<Playlist> >::iterator i = playlists.begin(); i != playlists.end(); ++i) {
+                if (((*i)->get_orig_diskstream_id() == tr->diskstream_id()) || (tr->playlist()->id() == (*i)->id())) {
+                        playlists_tr.push_back(*i);
+                }
+        }
+
+        /* sort the playlists */
+        PlaylistSorter cmp;
+        sort (playlists_tr.begin(), playlists_tr.end(), cmp);
+        
+        /* add the playlists to the menu */
+        for (vector<boost::shared_ptr<Playlist> >::iterator i = playlists_tr.begin(); i != playlists_tr.end(); ++i) {
+                playlist_items.push_back (RadioMenuElem (playlist_group, (*i)->name()));
+                RadioMenuItem *item = static_cast<RadioMenuItem*>(&playlist_items.back());
+                item->signal_toggled().connect(sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::use_playlist), item, boost::weak_ptr<Playlist> (*i)));
+                
+                if (tr->playlist()->id() == (*i)->id()) {
+                        item->set_active();
+                        
                }
        }
-
+        
        playlist_items.push_back (SeparatorElem());
-       playlist_items.push_back (MenuElem (_("Rename"), mem_fun(*this, &RouteTimeAxisView::rename_current_playlist)));
+       playlist_items.push_back (MenuElem (_("Rename..."), sigc::mem_fun(*this, &RouteTimeAxisView::rename_current_playlist)));
        playlist_items.push_back (SeparatorElem());
 
        if (!route_group() || !route_group()->is_active()) {
-               playlist_items.push_back (MenuElem (_("New"), bind(mem_fun(_editor, &PublicEditor::new_playlists), this)));
-               playlist_items.push_back (MenuElem (_("New Copy"), bind(mem_fun(_editor, &PublicEditor::copy_playlists), this)));
+               playlist_items.push_back (MenuElem (_("New..."), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::new_playlists), this)));
+               playlist_items.push_back (MenuElem (_("New Copy..."), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::copy_playlists), this)));
 
        } else {
                // Use a label which tells the user what is happening
-               playlist_items.push_back (MenuElem (_("New Take"), bind(mem_fun(_editor, &PublicEditor::new_playlists), this)));
-               playlist_items.push_back (MenuElem (_("Copy Take"), bind(mem_fun(_editor, &PublicEditor::copy_playlists), this)));
+               playlist_items.push_back (MenuElem (_("New Take"), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::new_playlists), this)));
+               playlist_items.push_back (MenuElem (_("Copy Take"), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::copy_playlists), this)));
 
        }
 
        playlist_items.push_back (SeparatorElem());
-       playlist_items.push_back (MenuElem (_("Clear Current"), bind(mem_fun(_editor, &PublicEditor::clear_playlists), this)));
+       playlist_items.push_back (MenuElem (_("Clear Current"), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::clear_playlists), this)));
        playlist_items.push_back (SeparatorElem());
 
-       playlist_items.push_back (MenuElem(_("Select from all ..."), mem_fun(*this, &RouteTimeAxisView::show_playlist_selector)));
+       playlist_items.push_back (MenuElem(_("Select from all..."), sigc::mem_fun(*this, &RouteTimeAxisView::show_playlist_selector)));
 }
 
 void
-RouteTimeAxisView::use_playlist (boost::weak_ptr<Playlist> wpl)
+RouteTimeAxisView::use_playlist (RadioMenuItem *item, boost::weak_ptr<Playlist> wpl)
 {
        assert (is_track());
 
+        // exit if we were triggered by deactivating the old playlist
+        if (!item->get_active()) {
+                return;
+        }
+
        boost::shared_ptr<Playlist> pl (wpl.lock());
 
        if (!pl) {
@@ -1552,16 +1494,14 @@ RouteTimeAxisView::use_playlist (boost::weak_ptr<Playlist> wpl)
        boost::shared_ptr<AudioPlaylist> apl = boost::dynamic_pointer_cast<AudioPlaylist> (pl);
 
        if (apl) {
-               if (get_diskstream()->playlist() == apl) {
-                       // radio button cotnrols mean this function is called for both the
-                       // old and new playlist
+               if (track()->playlist() == apl) {
+                        // exit when use_playlist is called by the creation of the playlist menu
+                        // or the playlist choice is unchanged
                        return;
                }
-               get_diskstream()->use_playlist (apl);
-
+               track()->use_playlist (apl);
 
                if (route_group() && route_group()->is_active()) {
-                       //PBD::stacktrace(cerr, 20);
                        std::string group_string = "."+route_group()->name()+".";
 
                        std::string take_name = apl->name();
@@ -1572,29 +1512,29 @@ RouteTimeAxisView::use_playlist (boost::weak_ptr<Playlist> wpl)
 
                        take_name = take_name.substr(idx + group_string.length()); // find the bit containing the take number / name
 
-                       for (list<Route*>::const_iterator i = route_group()->route_list().begin(); i != route_group()->route_list().end(); ++i) {
-                               if ( (*i) == this->route().get()) {
+                       boost::shared_ptr<RouteList> rl (route_group()->route_list());
+
+                       for (RouteList::const_iterator i = rl->begin(); i != rl->end(); ++i) {
+                               if ( (*i) == this->route()) {
                                        continue;
                                }
 
                                std::string playlist_name = (*i)->name()+group_string+take_name;
 
-                               Track *track = dynamic_cast<Track *>(*i);
+                               boost::shared_ptr<Track> track = boost::dynamic_pointer_cast<Track>(*i);
                                if (!track) {
                                        std::cerr << "route " << (*i)->name() << " is not a Track" << std::endl;
                                        continue;
                                }
 
-                               boost::shared_ptr<Playlist> ipl = session().playlists.by_name(playlist_name);
+                               boost::shared_ptr<Playlist> ipl = session()->playlists->by_name(playlist_name);
                                if (!ipl) {
                                        // No playlist for this track for this take yet, make it
-                                       track->diskstream()->use_new_playlist();
-                                       track->diskstream()->playlist()->set_name(playlist_name);
+                                       track->use_new_playlist();
+                                       track->playlist()->set_name(playlist_name);
                                } else {
-                                       track->diskstream()->use_playlist(ipl);
+                                       track->use_playlist(ipl);
                                }
-
-                               //(*i)->get_dis
                        }
                }
        }
@@ -1613,7 +1553,7 @@ RouteTimeAxisView::map_frozen ()
                return;
        }
 
-       ENSURE_GUI_THREAD (mem_fun(*this, &RouteTimeAxisView::map_frozen));
+       ENSURE_GUI_THREAD (*this, &RouteTimeAxisView::map_frozen)
 
        switch (track()->freeze_state()) {
        case Track::Frozen:
@@ -1642,31 +1582,30 @@ RouteTimeAxisView::color_handler ()
        reset_meter();
 }
 
+/** Toggle an automation track for a fully-specified Parameter (type,channel,id)
+ *  Will add track if necessary.
+ */
 void
-RouteTimeAxisView::toggle_automation_track (Evoral::Parameter param)
+RouteTimeAxisView::toggle_automation_track (const Evoral::Parameter& param)
 {
-       RouteAutomationNode* node = automation_track(param);
-
-       if (!node)
-               return;
-
-       bool showit = node->menu_item->get_active();
-
-       if (showit != node->track->marked_for_display()) {
-               if (showit) {
-                       node->track->set_marked_for_display (true);
-                       node->track->canvas_display()->show();
-                       node->track->get_state_node()->add_property ("shown", X_("yes"));
-               } else {
-                       node->track->set_marked_for_display (false);
-                       node->track->hide ();
-                       node->track->get_state_node()->add_property ("shown", X_("no"));
-               }
-
-               /* now trigger a redisplay */
-
-               if (!no_redraw) {
-                        _route->gui_changed (X_("track_height"), (void *) 0); /* EMIT_SIGNAL */
+       boost::shared_ptr<AutomationTimeAxisView> track = automation_child (param);
+       Gtk::CheckMenuItem* menu = automation_child_menu_item (param);
+       
+       if (!track) {
+               /* it doesn't exist yet, so we don't care about the button state: just add it */
+               create_automation_child (param, true);
+       } else {
+               assert (menu);
+               bool yn = menu->get_active();
+               if (track->set_visibility (menu->get_active()) && yn) {
+                       
+                       /* we made it visible, now trigger a redisplay. if it was hidden, then automation_track_hidden()
+                          will have done that for us.
+                       */
+                       
+                       if (!no_redraw) {
+                               _route->gui_changed (X_("track_height"), (void *) 0); /* EMIT_SIGNAL */
+                       } 
                }
        }
 }
@@ -1674,20 +1613,26 @@ RouteTimeAxisView::toggle_automation_track (Evoral::Parameter param)
 void
 RouteTimeAxisView::automation_track_hidden (Evoral::Parameter param)
 {
-       RouteAutomationNode* ran = automation_track(param);
-       if (!ran) {
+       boost::shared_ptr<AutomationTimeAxisView> track = automation_child (param);
+
+       if (!track) {
                return;
        }
 
+       Gtk::CheckMenuItem* menu = automation_child_menu_item (param);
+
        // if Evoral::Parameter::operator< doesn't obey strict weak ordering, we may crash here....
-       _show_automation.erase(param);
-       ran->track->get_state_node()->add_property (X_("shown"), X_("no"));
+       track->get_state_node()->add_property (X_("shown"), X_("no"));
 
-       if (ran->menu_item && !_hidden) {
-               ran->menu_item->set_active (false);
+       if (menu && !_hidden) {
+               ignore_toggle = true;
+               menu->set_active (false);
+               ignore_toggle = false;
        }
 
-        _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
+       if (_route && !no_redraw) {
+               _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
+       }
 }
 
 
@@ -1698,12 +1643,16 @@ RouteTimeAxisView::show_all_automation ()
 
        /* Show our automation */
 
-       map<Evoral::Parameter, RouteAutomationNode*>::iterator i;
-       for (i = _automation_tracks.begin(); i != _automation_tracks.end(); ++i) {
-               i->second->track->set_marked_for_display (true);
-               i->second->track->canvas_display()->show();
-               i->second->track->get_state_node()->add_property ("shown", X_("yes"));
-               i->second->menu_item->set_active(true);
+       for (AutomationTracks::iterator i = _automation_tracks.begin(); i != _automation_tracks.end(); ++i) {
+               i->second->set_marked_for_display (true);
+               i->second->canvas_display()->show();
+               i->second->get_state_node()->add_property ("shown", X_("yes"));
+
+               Gtk::CheckMenuItem* menu = automation_child_menu_item (i->first);
+               
+               if (menu) {
+                       menu->set_active(true);
+               }
        }
 
 
@@ -1723,7 +1672,7 @@ RouteTimeAxisView::show_all_automation ()
 
        /* Redraw */
 
-        _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
+        _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
 }
 
 void
@@ -1733,13 +1682,16 @@ RouteTimeAxisView::show_existing_automation ()
 
        /* Show our automation */
 
-       map<Evoral::Parameter, RouteAutomationNode*>::iterator i;
-       for (i = _automation_tracks.begin(); i != _automation_tracks.end(); ++i) {
-               if (i->second->track->line() && i->second->track->line()->npoints() > 0) {
-                       i->second->track->set_marked_for_display (true);
-                       i->second->track->canvas_display()->show();
-                       i->second->track->get_state_node()->add_property ("shown", X_("yes"));
-                       i->second->menu_item->set_active(true);
+       for (AutomationTracks::iterator i = _automation_tracks.begin(); i != _automation_tracks.end(); ++i) {
+               if (i->second->has_automation()) {
+                       i->second->set_marked_for_display (true);
+                       i->second->canvas_display()->show();
+                       i->second->get_state_node()->add_property ("shown", X_("yes"));
+
+                       Gtk::CheckMenuItem* menu = automation_child_menu_item (i->first);
+                       if (menu) {
+                               menu->set_active(true);
+                       }
                }
        }
 
@@ -1748,7 +1700,7 @@ RouteTimeAxisView::show_existing_automation ()
 
        for (list<ProcessorAutomationInfo*>::iterator i = processor_automation.begin(); i != processor_automation.end(); ++i) {
                for (vector<ProcessorAutomationNode*>::iterator ii = (*i)->lines.begin(); ii != (*i)->lines.end(); ++ii) {
-                       if ((*ii)->view != 0 && (*i)->processor->data().control((*ii)->what)->list()->size() > 0) {
+                       if ((*ii)->view != 0 && (*i)->processor->control((*ii)->what)->list()->size() > 0) {
                                (*ii)->menu_item->set_active (true);
                        }
                }
@@ -1756,7 +1708,7 @@ RouteTimeAxisView::show_existing_automation ()
 
        no_redraw = false;
 
-       _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
+       _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
 }
 
 void
@@ -1766,11 +1718,16 @@ RouteTimeAxisView::hide_all_automation ()
 
        /* Hide our automation */
 
-       for (map<Evoral::Parameter, RouteAutomationNode*>::iterator i = _automation_tracks.begin(); i != _automation_tracks.end(); ++i) {
-               i->second->track->set_marked_for_display (false);
-               i->second->track->hide ();
-               i->second->track->get_state_node()->add_property ("shown", X_("no"));
-               i->second->menu_item->set_active (false);
+       for (AutomationTracks::iterator i = _automation_tracks.begin(); i != _automation_tracks.end(); ++i) {
+               i->second->set_marked_for_display (false);
+               i->second->hide ();
+               i->second->get_state_node()->add_property ("shown", X_("no"));
+
+               Gtk::CheckMenuItem* menu = automation_child_menu_item (i->first);
+               
+               if (menu) {
+                       menu->set_active (false);
+               }
        }
 
        /* Hide processor automation */
@@ -1781,10 +1738,8 @@ RouteTimeAxisView::hide_all_automation ()
                }
        }
 
-       _show_automation.clear();
-
        no_redraw = false;
-        _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
+        _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
 }
 
 
@@ -1792,7 +1747,7 @@ void
 RouteTimeAxisView::region_view_added (RegionView* rv)
 {
        /* XXX need to find out if automation children have automationstreamviews. If yes, no ghosts */
-       if(is_audio_track()) {
+       if (is_audio_track()) {
                for (Children::iterator i = children.begin(); i != children.end(); ++i) {
                        boost::shared_ptr<AutomationTimeAxisView> atv;
 
@@ -1872,9 +1827,10 @@ RouteTimeAxisView::add_processor_automation_curve (boost::shared_ptr<Processor>
        ProcessorAutomationNode* pan;
 
        if ((pan = find_processor_automation_node (processor, what)) == 0) {
-               error << _("programming error: ")
-                     << string_compose (X_("processor automation curve for %1:%2 not registered with track!"),
-                                 processor->name(), what)
+                /* session state may never have been saved with new plugin */
+                error << _("programming error: ")
+                     << string_compose (X_("processor automation curve for %1:%2/%3/%4 not registered with track!"),
+                                         processor->name(), what.type(), (int) what.channel(), what.id() )
                      << endmsg;
                /*NOTREACHED*/
                return;
@@ -1891,16 +1847,16 @@ RouteTimeAxisView::add_processor_automation_curve (boost::shared_ptr<Processor>
        /* FIXME: ew */
 
        char state_name[256];
-       snprintf (state_name, sizeof (state_name), "Redirect-%s-%" PRIu32, legalize_for_xml_node (processor->name()).c_str(), what.id());
+       snprintf (state_name, sizeof (state_name), "%s-%" PRIu32, legalize_for_xml_node (processor->name()).c_str(), what.id());
 
        boost::shared_ptr<AutomationControl> control
-                       = boost::dynamic_pointer_cast<AutomationControl>(processor->data().control(what, true));
+                       = boost::dynamic_pointer_cast<AutomationControl>(processor->control(what, true));
 
        pan->view = boost::shared_ptr<AutomationTimeAxisView>(
                        new AutomationTimeAxisView (_session, _route, processor, control,
                                _editor, *this, false, parent_canvas, name, state_name));
 
-       pan->view->Hiding.connect (bind (mem_fun(*this, &RouteTimeAxisView::processor_automation_track_hidden), pan, processor));
+       pan->view->Hiding.connect (sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::processor_automation_track_hidden), pan, processor));
 
        if (!pan->view->marked_for_display()) {
                pan->view->hide ();
@@ -1911,7 +1867,7 @@ RouteTimeAxisView::add_processor_automation_curve (boost::shared_ptr<Processor>
        add_child (pan->view);
 
        if (_view) {
-               _view->foreach_regionview (mem_fun(*pan->view.get(), &TimeAxisView::add_ghost));
+               _view->foreach_regionview (sigc::mem_fun(*pan->view.get(), &TimeAxisView::add_ghost));
        }
 
        processor->mark_automation_visible (what, true);
@@ -1926,7 +1882,9 @@ RouteTimeAxisView::processor_automation_track_hidden (RouteTimeAxisView::Process
 
        i->mark_automation_visible (pan->what, false);
 
-        _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
+       if (!no_redraw) {
+               _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
+       }
 }
 
 void
@@ -1953,7 +1911,7 @@ RouteTimeAxisView::add_existing_processor_automation_curves (boost::weak_ptr<Pro
 }
 
 void
-RouteTimeAxisView::add_automation_child(Evoral::Parameter param, boost::shared_ptr<AutomationTimeAxisView> track, bool show)
+RouteTimeAxisView::add_automation_child (Evoral::Parameter param, boost::shared_ptr<AutomationTimeAxisView> track, bool show)
 {
        using namespace Menu_Helpers;
 
@@ -1962,7 +1920,7 @@ RouteTimeAxisView::add_automation_child(Evoral::Parameter param, boost::shared_p
 
        add_child (track);
 
-       track->Hiding.connect (bind (mem_fun (*this, &RouteTimeAxisView::automation_track_hidden), param));
+       track->Hiding.connect (sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::automation_track_hidden), param));
 
        bool hideit = (!show);
 
@@ -1974,23 +1932,25 @@ RouteTimeAxisView::add_automation_child(Evoral::Parameter param, boost::shared_p
                }
        }
 
-       _automation_tracks.insert(std::make_pair(param, new RouteAutomationNode(param, NULL, track)));
-
-       if (hideit) {
-               track->hide ();
-       } else {
-               _show_automation.insert (param);
+       _automation_tracks[param] = track;
 
+       track->set_visibility (!hideit);
 
-               if (!no_redraw) {
-                       _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
-               }
+       if (!no_redraw) {
+               _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
        }
 
-       build_display_menu();
+       if (!EventTypeMap::instance().is_midi_parameter(param)) {
+               /* MIDI-related parameters are always in the menu, there's no
+                  reason to rebuild the menu just because we added a automation
+                  lane for one of them. But if we add a non-MIDI automation
+                  lane, then we need to invalidate the display menu.
+               */
+               delete display_menu;
+               display_menu = 0;
+       }
 }
 
-
 void
 RouteTimeAxisView::add_processor_to_subplugin_menu (boost::weak_ptr<Processor> p)
 {
@@ -2050,6 +2010,8 @@ RouteTimeAxisView::add_processor_to_subplugin_menu (boost::weak_ptr<Processor> p
                items.push_back (CheckMenuElem (name));
                mitem = dynamic_cast<CheckMenuItem*> (&items.back());
 
+               _subplugin_menu_map[*i] = mitem;
+
                if (has_visible_automation.find((*i)) != has_visible_automation.end()) {
                        mitem->set_active(true);
                }
@@ -2068,7 +2030,7 @@ RouteTimeAxisView::add_processor_to_subplugin_menu (boost::weak_ptr<Processor> p
 
                }
 
-               mitem->signal_toggled().connect (bind (mem_fun(*this, &RouteTimeAxisView::processor_menu_item_toggled), rai, pan));
+               mitem->signal_toggled().connect (sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::processor_menu_item_toggled), rai, pan));
        }
 
        /* add the menu for this processor, because the subplugin
@@ -2110,10 +2072,7 @@ RouteTimeAxisView::processor_menu_item_toggled (RouteTimeAxisView::ProcessorAuto
        }
 
        if (redraw && !no_redraw) {
-
-               /* now trigger a redisplay */
-
-                _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
+                _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
 
        }
 }
@@ -2132,10 +2091,11 @@ RouteTimeAxisView::processors_changed (RouteProcessorChange c)
                (*i)->valid = false;
        }
 
+       _subplugin_menu_map.clear ();
        subplugin_menu.items().clear ();
 
-       _route->foreach_processor (mem_fun (*this, &RouteTimeAxisView::add_processor_to_subplugin_menu));
-       _route->foreach_processor (mem_fun (*this, &RouteTimeAxisView::add_existing_processor_automation_curves));
+       _route->foreach_processor (sigc::mem_fun (*this, &RouteTimeAxisView::add_processor_to_subplugin_menu));
+       _route->foreach_processor (sigc::mem_fun (*this, &RouteTimeAxisView::add_existing_processor_automation_curves));
 
        bool deleted_processor_automation = false;
 
@@ -2157,8 +2117,8 @@ RouteTimeAxisView::processors_changed (RouteProcessorChange c)
                i = tmp;
        }
 
-       if (deleted_processor_automation) {
-               _route->gui_changed ("visible_tracks", this);
+       if (deleted_processor_automation && !no_redraw) {
+               _route->gui_changed ("track_height", this);
        }
 }
 
@@ -2219,10 +2179,11 @@ boost::shared_ptr<AutomationTimeAxisView>
 RouteTimeAxisView::automation_child(Evoral::Parameter param)
 {
        AutomationTracks::iterator i = _automation_tracks.find(param);
-       if (i != _automation_tracks.end())
-               return i->second->track;
-       else
+       if (i != _automation_tracks.end()) {
+               return i->second;
+       } else {
                return boost::shared_ptr<AutomationTimeAxisView>();
+       }
 }
 
 void
@@ -2261,9 +2222,9 @@ RouteTimeAxisView::clear_meter ()
 }
 
 void
-RouteTimeAxisView::meter_changed (void *src)
+RouteTimeAxisView::meter_changed ()
 {
-       ENSURE_GUI_THREAD (bind (mem_fun(*this, &RouteTimeAxisView::meter_changed), src));
+       ENSURE_GUI_THREAD (*this, &RouteTimeAxisView::meter_changed)
        reset_meter();
 }
 
@@ -2274,10 +2235,11 @@ RouteTimeAxisView::io_changed (IOChange /*change*/, void */*src*/)
 }
 
 void
-RouteTimeAxisView::build_underlay_menu(Gtk::Menu* parent_menu) {
+RouteTimeAxisView::build_underlay_menu(Gtk::Menu* parent_menu)
+{
        using namespace Menu_Helpers;
 
-       if(!_underlay_streams.empty()) {
+       if (!_underlay_streams.empty()) {
                MenuList& parent_items = parent_menu->items();
                Menu* gs_menu = manage (new Menu);
                gs_menu->set_name ("ArdourContextMenu");
@@ -2287,7 +2249,7 @@ RouteTimeAxisView::build_underlay_menu(Gtk::Menu* parent_menu) {
 
                for(UnderlayList::iterator it = _underlay_streams.begin(); it != _underlay_streams.end(); ++it) {
                        gs_items.push_back(MenuElem(string_compose(_("Remove \"%1\""), (*it)->trackview().name()),
-                                                   bind(mem_fun(*this, &RouteTimeAxisView::remove_underlay), *it)));
+                                                   sigc::bind(sigc::mem_fun(*this, &RouteTimeAxisView::remove_underlay), *it)));
                }
        }
 }
@@ -2295,7 +2257,7 @@ RouteTimeAxisView::build_underlay_menu(Gtk::Menu* parent_menu) {
 bool
 RouteTimeAxisView::set_underlay_state()
 {
-       if(!underlay_xml_node) {
+       if (!underlay_xml_node) {
                return false;
        }
 
@@ -2306,7 +2268,7 @@ RouteTimeAxisView::set_underlay_state()
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
                child_node = *niter;
 
-               if(child_node->name() != "Underlay") {
+               if (child_node->name() != "Underlay") {
                        continue;
                }
 
@@ -2314,7 +2276,7 @@ RouteTimeAxisView::set_underlay_state()
                if (prop) {
                        PBD::ID id (prop->value());
 
-                       RouteTimeAxisView* v = _editor.get_route_view_by_id (id);
+                       RouteTimeAxisView* v = _editor.get_route_view_by_route_id (id);
 
                        if (v) {
                                add_underlay(v->view(), false);
@@ -2326,16 +2288,16 @@ RouteTimeAxisView::set_underlay_state()
 }
 
 void
-RouteTimeAxisView::add_underlay(StreamView* v, bool update_xml)
+RouteTimeAxisView::add_underlay (StreamView* v, bool update_xml)
 {
-       if(!v) {
+       if (!v) {
                return;
        }
 
        RouteTimeAxisView& other = v->trackview();
 
-       if(find(_underlay_streams.begin(), _underlay_streams.end(), v) == _underlay_streams.end()) {
-               if(find(other._underlay_mirrors.begin(), other._underlay_mirrors.end(), this) != other._underlay_mirrors.end()) {
+       if (find(_underlay_streams.begin(), _underlay_streams.end(), v) == _underlay_streams.end()) {
+               if (find(other._underlay_mirrors.begin(), other._underlay_mirrors.end(), this) != other._underlay_mirrors.end()) {
                        fatal << _("programming error: underlay reference pointer pairs are inconsistent!") << endmsg;
                        /*NOTREACHED*/
                }
@@ -2343,10 +2305,10 @@ RouteTimeAxisView::add_underlay(StreamView* v, bool update_xml)
                _underlay_streams.push_back(v);
                other._underlay_mirrors.push_back(this);
 
-               v->foreach_regionview(mem_fun(*this, &RouteTimeAxisView::add_ghost));
+               v->foreach_regionview(sigc::mem_fun(*this, &RouteTimeAxisView::add_ghost));
 
-               if(update_xml) {
-                       if(!underlay_xml_node) {
+               if (update_xml) {
+                       if (!underlay_xml_node) {
                                ensure_xml_node();
                                underlay_xml_node = xml_node->add_child("Underlays");
                        }
@@ -2359,29 +2321,29 @@ RouteTimeAxisView::add_underlay(StreamView* v, bool update_xml)
 }
 
 void
-RouteTimeAxisView::remove_underlay(StreamView* v)
+RouteTimeAxisView::remove_underlay (StreamView* v)
 {
-       if(!v) {
+       if (!v) {
                return;
        }
 
        UnderlayList::iterator it = find(_underlay_streams.begin(), _underlay_streams.end(), v);
        RouteTimeAxisView& other = v->trackview();
 
-       if(it != _underlay_streams.end()) {
+       if (it != _underlay_streams.end()) {
                UnderlayMirrorList::iterator gm = find(other._underlay_mirrors.begin(), other._underlay_mirrors.end(), this);
 
-               if(gm == other._underlay_mirrors.end()) {
+               if (gm == other._underlay_mirrors.end()) {
                        fatal << _("programming error: underlay reference pointer pairs are inconsistent!") << endmsg;
                        /*NOTREACHED*/
                }
 
-               v->foreach_regionview(mem_fun(*this, &RouteTimeAxisView::remove_ghost));
+               v->foreach_regionview(sigc::mem_fun(*this, &RouteTimeAxisView::remove_ghost));
 
                _underlay_streams.erase(it);
                other._underlay_mirrors.erase(gm);
 
-               if(underlay_xml_node) {
+               if (underlay_xml_node) {
                        underlay_xml_node->remove_nodes_and_delete("id", v->trackview().route()->id().to_s());
                }
        }
@@ -2392,18 +2354,37 @@ RouteTimeAxisView::set_button_names ()
 {
        rec_enable_button_label.set_text (_("r"));
 
-       if (Config->get_solo_control_is_listen_control()) {
-               switch (Config->get_listen_position()) {
-               case AfterFaderListen:
-                       solo_button_label.set_text (_("A"));
-                       break;
-               case PreFaderListen:
-                       solo_button_label.set_text (_("P"));
-                       break;
-               }
-       } else {
-               solo_button_label.set_text (_("s"));
+        if (_route && _route->solo_safe()) {
+                solo_button_label.set_text (X_("!"));
+        } else {
+                if (Config->get_solo_control_is_listen_control()) {
+                        switch (Config->get_listen_position()) {
+                        case AfterFaderListen:
+                                solo_button_label.set_text (_("A"));
+                                break;
+                        case PreFaderListen:
+                                solo_button_label.set_text (_("P"));
+                                break;
+                        }
+                } else {
+                        solo_button_label.set_text (_("s"));
+                }
+        }
+       mute_button_label.set_text (_("m"));
+}
+
+Gtk::CheckMenuItem*
+RouteTimeAxisView::automation_child_menu_item (Evoral::Parameter param)
+{
+       ParameterMenuMap::iterator i = _main_automation_menu_map.find (param);
+       if (i != _main_automation_menu_map.end()) {
+               return i->second;
+       }
+       
+       i = _subplugin_menu_map.find (param);
+       if (i != _subplugin_menu_map.end()) {
+               return i->second;
        }
 
-       mute_button_label.set_text (_("m"));
+       return 0;
 }