First cut of some Pro-tools inspired editing features; linked play/play range
[ardour.git] / gtk2_ardour / editor.cc
index d0be4161d1f6cdfff038f9affc71aa257ee2c411..fd6d8a991f5899fe4bbe56bbbe24794661f2439e 100644 (file)
 #endif
 
 using namespace std;
-using namespace sigc;
 using namespace ARDOUR;
 using namespace PBD;
 using namespace Gtk;
@@ -221,8 +220,10 @@ show_me_the_size (Requisition* r, const char* what)
 }
 
 Editor::Editor ()
+       : _join_object_range_state (JOIN_OBJECT_RANGE_NONE)
+
          /* time display buttons */
-       : minsec_label (_("Mins:Secs"))
+       , minsec_label (_("Mins:Secs"))
        , bbt_label (_("Bars:Beats"))
        , timecode_label (_("Timecode"))
        , frame_label (_("Samples"))
@@ -269,7 +270,6 @@ Editor::Editor ()
 
        PublicEditor::_instance = this;
 
-       session = 0;
        _have_idled = false;
 
        selection = new Selection (this);
@@ -312,7 +312,6 @@ Editor::Editor ()
        _show_waveforms_recording = true;
        show_gain_after_trim = false;
        verbose_cursor_on = true;
-       route_removal = false;
        last_item_entered = 0;
        last_item_entered_n = 0;
 
@@ -438,8 +437,8 @@ Editor::Editor ()
        transport_mark_label.set_no_show_all();
 
        initialize_rulers ();
-       _summary = new EditorSummary (this);
        initialize_canvas ();
+       _summary = new EditorSummary (this);
 
        selection->TimeChanged.connect (sigc::mem_fun(*this, &Editor::time_selection_changed));
        selection->TracksChanged.connect (sigc::mem_fun(*this, &Editor::track_selection_changed));
@@ -579,12 +578,11 @@ Editor::Editor ()
        summary_frame->add (*_summary);
        summary_frame->show ();
 
-       HBox* summary_hbox = manage (new HBox);
-       summary_hbox->pack_start (*summary_arrows_left, false, false);
-       summary_hbox->pack_start (*summary_frame, true, true);
-       summary_hbox->pack_start (*summary_arrows_right, false, false);
+       _summary_hbox.pack_start (*summary_arrows_left, false, false);
+       _summary_hbox.pack_start (*summary_frame, true, true);
+       _summary_hbox.pack_start (*summary_arrows_right, false, false);
        
-       editor_summary_pane->pack2(*summary_hbox);
+       editor_summary_pane->pack2 (_summary_hbox);
 
        edit_pane.pack1 (*editor_summary_pane, true, true);
        edit_pane.pack2 (the_notebook, false, true);
@@ -629,7 +627,7 @@ Editor::Editor ()
        _playlist_selector = new PlaylistSelector();
        _playlist_selector->signal_delete_event().connect (sigc::bind (sigc::ptr_fun (just_hide_it), static_cast<Window *> (_playlist_selector)));
 
-       RegionView::RegionViewGoingAway.connect (sigc::mem_fun(*this, &Editor::catch_vanishing_regionview));
+       RegionView::RegionViewGoingAway.connect (*this, ui_bind (&Editor::catch_vanishing_regionview, this, _1), gui_context());
 
        /* nudge stuff */
 
@@ -679,13 +677,19 @@ Editor::Editor ()
 
        /* allow external control surfaces/protocols to do various things */
 
-       ControlProtocol::ZoomToSession.connect (sigc::mem_fun (*this, &Editor::temporal_zoom_session));
-       ControlProtocol::ZoomIn.connect (sigc::bind (sigc::mem_fun (*this, &Editor::temporal_zoom_step), false));
-       ControlProtocol::ZoomOut.connect (sigc::bind (sigc::mem_fun (*this, &Editor::temporal_zoom_step), true));
-       ControlProtocol::ScrollTimeline.connect (sigc::mem_fun (*this, &Editor::control_scroll));
-       BasicUI::AccessAction.connect (sigc::mem_fun (*this, &Editor::access_action));
+       ControlProtocol::ZoomToSession.connect (*this, boost::bind (&Editor::temporal_zoom_session, this), gui_context());
+       ControlProtocol::ZoomIn.connect (*this, boost::bind (&Editor::temporal_zoom_step, this, false), gui_context());
+       ControlProtocol::ZoomOut.connect (*this, boost::bind (&Editor::temporal_zoom_step, this, true), gui_context());
+       ControlProtocol::ScrollTimeline.connect (*this, ui_bind (&Editor::control_scroll, this, _1), gui_context());
+       BasicUI::AccessAction.connect (*this, ui_bind (&Editor::access_action, this, _1, _2), gui_context());
+       
+       /* problematic: has to return a value and thus cannot be x-thread */
+
+       Session::AskAboutPlaylistDeletion.connect_same_thread (*this, boost::bind (&Editor::playlist_deletion_dialog, this, _1));
+
+       Config->ParameterChanged.connect (*this, ui_bind (&Editor::parameter_changed, this, _1), gui_context());
 
-       Config->ParameterChanged.connect (sigc::mem_fun (*this, &Editor::parameter_changed));
+       TimeAxisView::CatchDeletion.connect (*this, ui_bind (&Editor::timeaxisview_deleted, this, _1), gui_context());
 
        _last_normalization_value = 0;
 
@@ -806,8 +810,8 @@ Editor::instant_save ()
                return;
        }
 
-       if (session) {
-               session->add_instant_xml(get_state());
+       if (_session) {
+               _session->add_instant_xml(get_state());
        } else {
                Config->add_instant_xml(get_state());
        }
@@ -816,7 +820,7 @@ Editor::instant_save ()
 void
 Editor::zoom_adjustment_changed ()
 {
-       if (session == 0) {
+       if (_session == 0) {
                return;
        }
 
@@ -825,8 +829,8 @@ Editor::zoom_adjustment_changed ()
        if (fpu < 1.0) {
                fpu = 1.0;
                zoom_range_clock.set ((nframes64_t) floor (fpu * _canvas_width));
-       } else if (fpu > session->current_end_frame() / _canvas_width) {
-               fpu = session->current_end_frame() / _canvas_width;
+       } else if (fpu > _session->current_end_frame() / _canvas_width) {
+               fpu = _session->current_end_frame() / _canvas_width;
                zoom_range_clock.set ((nframes64_t) floor (fpu * _canvas_width));
        }
 
@@ -838,7 +842,7 @@ Editor::control_scroll (float fraction)
 {
        ENSURE_GUI_THREAD (*this, &Editor::control_scroll, fraction)
 
-       if (!session) {
+       if (!_session) {
                return;
        }
 
@@ -856,7 +860,7 @@ Editor::control_scroll (float fraction)
        */
 
        if (!_control_scroll_target) {
-               _control_scroll_target = session->transport_frame();
+               _control_scroll_target = _session->transport_frame();
                _dragging_playhead = true;
        }
 
@@ -898,7 +902,7 @@ Editor::control_scroll (float fraction)
 bool
 Editor::deferred_control_scroll (nframes64_t /*target*/)
 {
-       session->request_locate (*_control_scroll_target, session->transport_rolling());
+       _session->request_locate (*_control_scroll_target, _session->transport_rolling());
        // reset for next stream
        _control_scroll_target = boost::none;
        _dragging_playhead = false;
@@ -908,7 +912,7 @@ Editor::deferred_control_scroll (nframes64_t /*target*/)
 void
 Editor::access_action (std::string action_group, std::string action_item)
 {
-       if (!session) {
+       if (!_session) {
                return;
        }
 
@@ -950,7 +954,7 @@ Editor::map_position_change (nframes64_t frame)
 {
        ENSURE_GUI_THREAD (*this, &Editor::map_position_change, frame)
 
-       if (session == 0 || !_follow_playhead) {
+       if (_session == 0 || !_follow_playhead) {
                return;
        }
 
@@ -988,12 +992,12 @@ Editor::center_screen_internal (nframes64_t frame, float page)
 void
 Editor::handle_new_duration ()
 {
-       if (!session) {
+       if (!_session) {
                return;
        }
 
        ENSURE_GUI_THREAD (*this, &Editor::handle_new_duration)
-       nframes64_t new_end = session->current_end_frame() + (nframes64_t) floorf (current_page_frames() * 0.10f);
+       nframes64_t new_end = _session->current_end_frame() + (nframes64_t) floorf (current_page_frames() * 0.10f);
 
        horizontal_adjustment.set_upper (new_end / frames_per_unit);
        horizontal_adjustment.set_page_size (current_page_frames()/frames_per_unit);
@@ -1004,28 +1008,20 @@ Editor::handle_new_duration ()
        //cerr << "Editor::handle_new_duration () called ha v:l:u:ps:lcf = " << horizontal_adjustment.get_value() << ":" << horizontal_adjustment.get_lower() << ":" << horizontal_adjustment.get_upper() << ":" << horizontal_adjustment.get_page_size() << ":" << endl;//DEBUG
 }
 
-void
-Editor::update_title_s (const string & snap_name)
-{
-       ENSURE_GUI_THREAD (*this, &Editor::update_title_s, snap_name)
-
-       update_title ();
-}
-
 void
 Editor::update_title ()
 {
        ENSURE_GUI_THREAD (*this, &Editor::update_title)
 
-       if (session) {
-               bool dirty = session->dirty();
+       if (_session) {
+               bool dirty = _session->dirty();
 
                string session_name;
 
-               if (session->snap_name() != session->name()) {
-                       session_name = session->snap_name();
+               if (_session->snap_name() != _session->name()) {
+                       session_name = _session->snap_name();
                } else {
-                       session_name = session->name();
+                       session_name = _session->name();
                }
 
                if (dirty) {
@@ -1039,9 +1035,36 @@ Editor::update_title ()
 }
 
 void
-Editor::connect_to_session (Session *t)
+Editor::set_session (Session *t)
 {
-       session = t;
+       SessionHandlePtr::set_session (t);
+
+       if (!_session) {
+               return;
+       }
+
+       zoom_range_clock.set_session (_session);
+       _playlist_selector->set_session (_session);
+       nudge_clock.set_session (_session);
+       _summary->set_session (_session);
+       _group_tabs->set_session (_session);
+       _route_groups->set_session (_session);
+       _regions->set_session (_session);
+       _snapshots->set_session (_session);
+       _routes->set_session (_session);
+       _locations->set_session (_session);
+
+       if (rhythm_ferret) {
+               rhythm_ferret->set_session (_session);
+       }
+
+       if (analysis_window) {
+               analysis_window->set_session (_session);
+       }
+
+       if (sfbrowser) {
+               sfbrowser->set_session (_session);
+       }
 
        compute_fixed_ruler_scale ();
 
@@ -1054,105 +1077,83 @@ Editor::connect_to_session (Session *t)
 
        /* catch up with the playhead */
 
-       session->request_locate (playhead_cursor->current_frame);
+       _session->request_locate (playhead_cursor->current_frame);
 
        update_title ();
 
-       session->GoingAway.connect (sigc::mem_fun(*this, &Editor::session_going_away));
-       session->history().Changed.connect (sigc::mem_fun (*this, &Editor::history_changed));
-
        /* These signals can all be emitted by a non-GUI thread. Therefore the
           handlers for them must not attempt to directly interact with the GUI,
           but use Gtkmm2ext::UI::instance()->call_slot();
        */
 
-       session_connections.push_back (session->TransportStateChange.connect (sigc::mem_fun(*this, &Editor::map_transport_state)));
-       session_connections.push_back (session->PositionChanged.connect (sigc::mem_fun(*this, &Editor::map_position_change)));
-       session_connections.push_back (session->RouteAdded.connect (sigc::mem_fun(*this, &Editor::handle_new_route)));
-       session_connections.push_back (session->DurationChanged.connect (sigc::mem_fun(*this, &Editor::handle_new_duration)));
-       session_connections.push_back (session->DirtyChanged.connect (sigc::mem_fun(*this, &Editor::update_title)));
-       session_connections.push_back (session->StateSaved.connect (sigc::mem_fun(*this, &Editor::update_title_s)));
-       session_connections.push_back (session->AskAboutPlaylistDeletion.connect (sigc::mem_fun(*this, &Editor::playlist_deletion_dialog)));
-
-       session_connections.push_back (session->TimecodeOffsetChanged.connect (sigc::mem_fun(*this, &Editor::update_just_timecode)));
-
-       session_connections.push_back (session->tempo_map().StateChanged.connect (sigc::mem_fun(*this, &Editor::tempo_map_changed)));
-
-       session_connections.push_back (session->Located.connect (sigc::mem_fun (*this, &Editor::located)));
-       session_connections.push_back (session->config.ParameterChanged.connect (sigc::mem_fun (*this, &Editor::parameter_changed)));
+       _session->TransportStateChange.connect (_session_connections, boost::bind (&Editor::map_transport_state, this), gui_context());
+       _session->PositionChanged.connect (_session_connections, ui_bind (&Editor::map_position_change, this, _1), gui_context());
+       _session->RouteAdded.connect (_session_connections, ui_bind (&Editor::handle_new_route, this, _1), gui_context());
+       _session->DurationChanged.connect (_session_connections, boost::bind (&Editor::handle_new_duration, this), gui_context());
+       _session->DirtyChanged.connect (_session_connections, boost::bind (&Editor::update_title, this), gui_context());
+       _session->TimecodeOffsetChanged.connect (_session_connections, boost::bind (&Editor::update_just_timecode, this), gui_context());
+       _session->tempo_map().StateChanged.connect (_session_connections, ui_bind (&Editor::tempo_map_changed, this, _1), gui_context());
+       _session->Located.connect (_session_connections, boost::bind (&Editor::located, this), gui_context());
+       _session->config.ParameterChanged.connect (_session_connections, ui_bind (&Editor::parameter_changed, this, _1), gui_context());
+       _session->StateSaved.connect (_session_connections, ui_bind (&Editor::session_state_saved, this, _1), gui_context());
+       _session->locations()->added.connect (_session_connections, ui_bind (&Editor::add_new_location, this, _1), gui_context());
+       _session->locations()->removed.connect (_session_connections, ui_bind (&Editor::location_gone, this, _1), gui_context());
+       _session->locations()->changed.connect (_session_connections, boost::bind (&Editor::refresh_location_display, this), gui_context());
+       _session->locations()->StateChanged.connect (_session_connections, ui_bind (&Editor::refresh_location_display_s, this, _1), gui_context());
+       _session->locations()->end_location()->changed.connect (_session_connections, ui_bind (&Editor::end_location_changed, this, _1), gui_context());
+       _session->history().Changed.connect (_session_connections, boost::bind (&Editor::history_changed, this), gui_context());
 
-       zoom_range_clock.set_session (session);
-       _playlist_selector->set_session (session);
-       nudge_clock.set_session (session);
        if (Profile->get_sae()) {
                BBT_Time bbt;
                bbt.bars = 0;
                bbt.beats = 0;
                bbt.ticks = 120;
-               nframes_t pos = session->tempo_map().bbt_duration_at (0, bbt, 1);
+               nframes_t pos = _session->tempo_map().bbt_duration_at (0, bbt, 1);
                nudge_clock.set_mode(AudioClock::BBT);
                nudge_clock.set (pos, true, 0, AudioClock::BBT);
 
        } else {
-               nudge_clock.set (session->frame_rate() * 5, true, 0, AudioClock::Timecode); // default of 5 seconds
+               nudge_clock.set (_session->frame_rate() * 5, true, 0, AudioClock::Timecode); // default of 5 seconds
        }
 
        playhead_cursor->canvas_item.show ();
 
-       if (rhythm_ferret) {
-               rhythm_ferret->set_session (session);
-       }
-
-       if (analysis_window != 0)
-               analysis_window->set_session (session);
-
-       Location* loc = session->locations()->auto_loop_location();
+       Location* loc = _session->locations()->auto_loop_location();
        if (loc == 0) {
-               loc = new Location (0, session->current_end_frame(), _("Loop"),(Location::Flags) (Location::IsAutoLoop | Location::IsHidden));
+               loc = new Location (0, _session->current_end_frame(), _("Loop"),(Location::Flags) (Location::IsAutoLoop | Location::IsHidden));
                if (loc->start() == loc->end()) {
                        loc->set_end (loc->start() + 1);
                }
-               session->locations()->add (loc, false);
-               session->set_auto_loop_location (loc);
+               _session->locations()->add (loc, false);
+               _session->set_auto_loop_location (loc);
        } else {
                // force name
                loc->set_name (_("Loop"));
        }
 
-       loc = session->locations()->auto_punch_location();
+       loc = _session->locations()->auto_punch_location();
        if (loc == 0) {
-               loc = new Location (0, session->current_end_frame(), _("Punch"), (Location::Flags) (Location::IsAutoPunch | Location::IsHidden));
+               loc = new Location (0, _session->current_end_frame(), _("Punch"), (Location::Flags) (Location::IsAutoPunch | Location::IsHidden));
                if (loc->start() == loc->end()) {
                        loc->set_end (loc->start() + 1);
                }
-               session->locations()->add (loc, false);
-               session->set_auto_punch_location (loc);
+               _session->locations()->add (loc, false);
+               _session->set_auto_punch_location (loc);
        } else {
                // force name
                loc->set_name (_("Punch"));
        }
 
-       Config->map_parameters (sigc::mem_fun (*this, &Editor::parameter_changed));
-       session->config.map_parameters (sigc::mem_fun (*this, &Editor::parameter_changed));
-
-       session->StateSaved.connect (sigc::mem_fun(*this, &Editor::session_state_saved));
+       boost::function<void (string)> pc (boost::bind (&Editor::parameter_changed, this, _1));
+       Config->map_parameters (pc);
+       _session->config.map_parameters (pc);
 
        refresh_location_display ();
-       session->locations()->added.connect (sigc::mem_fun(*this, &Editor::add_new_location));
-       session->locations()->removed.connect (sigc::mem_fun(*this, &Editor::location_gone));
-       session->locations()->changed.connect (sigc::mem_fun(*this, &Editor::refresh_location_display));
-       session->locations()->StateChanged.connect (sigc::mem_fun(*this, &Editor::refresh_location_display_s));
-       session->locations()->end_location()->changed.connect (sigc::mem_fun(*this, &Editor::end_location_changed));
-
-       if (sfbrowser) {
-               sfbrowser->set_session (session);
-       }
-
        handle_new_duration ();
 
        restore_ruler_visibility ();
        //tempo_map_changed (Change (0));
-       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
+       _session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 
        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                (static_cast<TimeAxisView*>(*i))->set_samples_per_unit (frames_per_unit);
@@ -1173,15 +1174,7 @@ Editor::connect_to_session (Session *t)
        }
 
        /* register for undo history */
-       session->register_with_memento_command_factory(_id, this);
-
-       _summary->connect_to_session (session);
-       _group_tabs->connect_to_session (session);
-       _route_groups->connect_to_session (session);
-       _regions->connect_to_session (session);
-       _snapshots->connect_to_session (session);
-       _routes->connect_to_session (session);
-       _locations->connect_to_session (session);
+       _session->register_with_memento_command_factory(_id, this);
 
        start_updating ();
 }
@@ -1561,8 +1554,8 @@ Editor::analyze_region_selection()
        if (analysis_window == 0) {
                analysis_window = new AnalysisWindow();
 
-               if (session != 0)
-                       analysis_window->set_session(session);
+               if (_session != 0)
+                       analysis_window->set_session(_session);
 
                analysis_window->show_all();
        }
@@ -1579,8 +1572,8 @@ Editor::analyze_range_selection()
        if (analysis_window == 0) {
                analysis_window = new AnalysisWindow();
 
-               if (session != 0)
-                       analysis_window->set_session(session);
+               if (_session != 0)
+                       analysis_window->set_session(_session);
 
                analysis_window->show_all();
        }
@@ -1688,22 +1681,22 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                region_menu->signal_map_event().connect (
                        sigc::bind (sigc::mem_fun(*this, &Editor::set_selected_regionview_from_map_event), sv, boost::weak_ptr<Region>(region)));
 
-               items.push_back (MenuElem (_("Rename"), sigc::mem_fun(*this, &Editor::rename_region)));
+               items.push_back (MenuElem (_("Rename..."), sigc::mem_fun(*this, &Editor::rename_region)));
                if (mr && internal_editing()) {
                        items.push_back (MenuElem (_("List editor..."), sigc::mem_fun(*this, &Editor::show_midi_list_editor)));
                } else {
-                       items.push_back (MenuElem (_("Region editor"), sigc::mem_fun(*this, &Editor::edit_region)));
+                       items.push_back (MenuElem (_("Region Properties..."), sigc::mem_fun(*this, &Editor::edit_region)));
                }
        }
 
        items.push_back (MenuElem (_("Raise to top layer"), sigc::mem_fun(*this, &Editor::raise_region_to_top)));
        items.push_back (MenuElem (_("Lower to bottom layer"), sigc::mem_fun  (*this, &Editor::lower_region_to_bottom)));
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Define sync point"), sigc::mem_fun(*this, &Editor::set_region_sync_from_edit_point)));
+       items.push_back (MenuElem (_("Define Sync Point"), sigc::mem_fun(*this, &Editor::set_region_sync_from_edit_point)));
        if (_edit_point == EditAtMouse) {
                items.back ().set_sensitive (false);
        }
-       items.push_back (MenuElem (_("Remove sync point"), sigc::mem_fun(*this, &Editor::remove_region_sync)));
+       items.push_back (MenuElem (_("Remove Sync Point"), sigc::mem_fun(*this, &Editor::remove_region_sync)));
        items.push_back (SeparatorElem());
 
        items.push_back (MenuElem (_("Audition"), sigc::mem_fun(*this, &Editor::play_selected_region)));
@@ -1732,7 +1725,7 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
        }
        region_lock_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_lock));
 
-       items.push_back (CheckMenuElem (_("Glue to Bars&Beats")));
+       items.push_back (CheckMenuElem (_("Glue to Bars & Beats")));
        CheckMenuItem* bbt_glue_item = static_cast<CheckMenuItem*>(&items.back());
 
        switch (region_to_check->positional_lock_style()) {
@@ -1766,7 +1759,7 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                }
        }
 
-       items.push_back (CheckMenuElem (_("Original position"), sigc::mem_fun(*this, &Editor::naturalize)));
+       items.push_back (CheckMenuElem (_("Original Position"), sigc::mem_fun(*this, &Editor::naturalize)));
        if (region_to_check->at_natural_position()) {
                items.back().set_sensitive (false);
        }
@@ -1869,7 +1862,7 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                region_edit_menu_split_item->set_sensitive (false);
        }
 
-       items.push_back (MenuElem (_("Make mono regions"), (sigc::mem_fun(*this, &Editor::split_multichannel_region))));
+       items.push_back (MenuElem (_("Make Mono Regions"), (sigc::mem_fun(*this, &Editor::split_multichannel_region))));
        region_edit_menu_split_multichannel_item = &items.back();
 
        items.push_back (MenuElem (_("Duplicate"), (sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), false))));
@@ -2268,7 +2261,7 @@ Editor::set_state (const XMLNode& node, int /*version*/)
        set_default_size (g.base_width, g.base_height);
        move (x, y);
 
-       if (session && (prop = node.property ("playhead"))) {
+       if (_session && (prop = node.property ("playhead"))) {
                nframes64_t pos = atol (prop->value().c_str());
                playhead_cursor->set_position (pos);
        } else {
@@ -2323,6 +2316,10 @@ Editor::set_state (const XMLNode& node, int /*version*/)
                }
        }
 
+       if ((prop = node.property ("join-object-range"))) {
+               join_object_range_button.set_active (string_is_affirmative (prop->value ()));
+       }
+
        if ((prop = node.property ("edit-point"))) {
                set_edit_point_preference ((EditPoint) string_2_enum (prop->value(), _edit_point), true);
        }
@@ -2470,6 +2467,7 @@ Editor::get_state ()
        node->add_property ("region-list-sort-type", enum2str (_regions->sort_type ()));
        node->add_property ("mouse-mode", enum2str(mouse_mode));
        node->add_property ("internal-edit", _internal_editing ? "yes" : "no");
+       node->add_property ("join-object-range", join_object_range_button.get_active () ? "yes" : "no");
 
        Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-editor-mixer"));
        if (act) {
@@ -2510,12 +2508,12 @@ Editor::trackview_by_y_position (double y)
 /** Snap a position to the grid, if appropriate, taking into account current
  *  grid settings and also the state of any snap modifier keys that may be pressed.
  *  @param start Position to snap.
- *  @param event Event to get current key modifier information from.
+ *  @param event Event to get current key modifier information from, or 0.
  */
 void
 Editor::snap_to_with_modifier (nframes64_t& start, GdkEvent const * event, int32_t direction, bool for_mark)
 {
-       if (!session) {
+       if (!_session || !event) {
                return;
        }
 
@@ -2533,7 +2531,7 @@ Editor::snap_to_with_modifier (nframes64_t& start, GdkEvent const * event, int32
 void
 Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
 {
-       if (!session || _snap_mode == SnapOff) {
+       if (!_session || _snap_mode == SnapOff) {
                return;
        }
 
@@ -2543,24 +2541,24 @@ Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
 void
 Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /*for_mark*/)
 {
-       const nframes64_t one_timecode_second = (nframes64_t)(rint(session->timecode_frames_per_second()) * session->frames_per_timecode_frame());
-       nframes64_t one_timecode_minute = (nframes64_t)(rint(session->timecode_frames_per_second()) * session->frames_per_timecode_frame() * 60);
+       const nframes64_t one_timecode_second = (nframes64_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame());
+       nframes64_t one_timecode_minute = (nframes64_t)(rint(_session->timecode_frames_per_second()) * _session->frames_per_timecode_frame() * 60);
 
        switch (_snap_type) {
        case SnapToTimecodeFrame:
-               if (((direction == 0) && (fmod((double)start, (double)session->frames_per_timecode_frame()) > (session->frames_per_timecode_frame() / 2))) || (direction > 0)) {
-                       start = (nframes64_t) (ceil ((double) start / session->frames_per_timecode_frame()) * session->frames_per_timecode_frame());
+               if (((direction == 0) && (fmod((double)start, (double)_session->frames_per_timecode_frame()) > (_session->frames_per_timecode_frame() / 2))) || (direction > 0)) {
+                       start = (nframes64_t) (ceil ((double) start / _session->frames_per_timecode_frame()) * _session->frames_per_timecode_frame());
                } else {
-                       start = (nframes64_t) (floor ((double) start / session->frames_per_timecode_frame()) *  session->frames_per_timecode_frame());
+                       start = (nframes64_t) (floor ((double) start / _session->frames_per_timecode_frame()) *  _session->frames_per_timecode_frame());
                }
                break;
 
        case SnapToTimecodeSeconds:
-               if (session->timecode_offset_negative())
+               if (_session->timecode_offset_negative())
                {
-                       start += session->timecode_offset ();
+                       start += _session->timecode_offset ();
                } else {
-                       start -= session->timecode_offset ();
+                       start -= _session->timecode_offset ();
                }
                if (((direction == 0) && (start % one_timecode_second > one_timecode_second / 2)) || direction > 0) {
                        start = (nframes64_t) ceil ((double) start / one_timecode_second) * one_timecode_second;
@@ -2568,31 +2566,31 @@ Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /
                        start = (nframes64_t) floor ((double) start / one_timecode_second) * one_timecode_second;
                }
 
-               if (session->timecode_offset_negative())
+               if (_session->timecode_offset_negative())
                {
-                       start -= session->timecode_offset ();
+                       start -= _session->timecode_offset ();
                } else {
-                       start += session->timecode_offset ();
+                       start += _session->timecode_offset ();
                }
                break;
 
        case SnapToTimecodeMinutes:
-               if (session->timecode_offset_negative())
+               if (_session->timecode_offset_negative())
                {
-                       start += session->timecode_offset ();
+                       start += _session->timecode_offset ();
                } else {
-                       start -= session->timecode_offset ();
+                       start -= _session->timecode_offset ();
                }
                if (((direction == 0) && (start % one_timecode_minute > one_timecode_minute / 2)) || direction > 0) {
                        start = (nframes64_t) ceil ((double) start / one_timecode_minute) * one_timecode_minute;
                } else {
                        start = (nframes64_t) floor ((double) start / one_timecode_minute) * one_timecode_minute;
                }
-               if (session->timecode_offset_negative())
+               if (_session->timecode_offset_negative())
                {
-                       start -= session->timecode_offset ();
+                       start -= _session->timecode_offset ();
                } else {
-                       start += session->timecode_offset ();
+                       start += _session->timecode_offset ();
                }
                break;
        default:
@@ -2604,8 +2602,8 @@ Editor::timecode_snap_to_internal (nframes64_t& start, int32_t direction, bool /
 void
 Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
 {
-       const nframes64_t one_second = session->frame_rate();
-       const nframes64_t one_minute = session->frame_rate() * 60;
+       const nframes64_t one_second = _session->frame_rate();
+       const nframes64_t one_minute = _session->frame_rate() * 60;
        nframes64_t presnap = start;
        nframes64_t before;
        nframes64_t after;
@@ -2641,31 +2639,31 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
                break;
 
        case SnapToBar:
-               start = session->tempo_map().round_to_bar (start, direction);
+               start = _session->tempo_map().round_to_bar (start, direction);
                break;
 
        case SnapToBeat:
-               start = session->tempo_map().round_to_beat (start, direction);
+               start = _session->tempo_map().round_to_beat (start, direction);
                break;
 
        case SnapToAThirtysecondBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 32, direction);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 32, direction);
                break;
 
        case SnapToASixteenthBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 16, direction);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 16, direction);
                break;
 
        case SnapToAEighthBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 8, direction);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 8, direction);
                break;
 
        case SnapToAQuarterBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 4, direction);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 4, direction);
                break;
 
        case SnapToAThirdBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 3, direction);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 3, direction);
                break;
 
        case SnapToMark:
@@ -2673,7 +2671,7 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
                        return;
                }
 
-               session->locations()->marks_either_side (start, before, after);
+               _session->locations()->marks_either_side (start, before, after);
 
                if (before == max_frames) {
                        start = after;
@@ -2761,24 +2759,45 @@ Editor::setup_toolbar ()
        mouse_timefx_button.set_relief(Gtk::RELIEF_NONE);
        mouse_audition_button.set_relief(Gtk::RELIEF_NONE);
        // internal_edit_button.set_relief(Gtk::RELIEF_NONE);
+       join_object_range_button.set_relief(Gtk::RELIEF_NONE);
 
        HBox* mode_box = manage(new HBox);
        mode_box->set_border_width (2);
        mode_box->set_spacing(4);
-       mouse_mode_button_box.set_spacing(1);
-       mouse_mode_button_box.pack_start(mouse_move_button, true, true);
-       if (!Profile->get_sae()) {
-               mouse_mode_button_box.pack_start(mouse_select_button, true, true);
+
+       /* table containing mode buttons */
+
+       Table* mouse_mode_button_table = manage (new Table (Profile->get_sae() ? 4 : 6, 2));
+
+       int c = 0;
+
+       if (Profile->get_sae()) {
+               mouse_mode_button_table->attach (mouse_move_button, c, c + 1, 0, 1);
+               ++c;
+       } else {
+               mouse_mode_button_table->attach (mouse_move_button, c, c + 1, 0, 1);
+               mouse_mode_button_table->attach (mouse_select_button, c + 1, c + 2, 0, 1);
+               mouse_mode_button_table->attach (join_object_range_button, c, c + 2, 1, 2);
+               c += 2;
        }
-       mouse_mode_button_box.pack_start(mouse_zoom_button, true, true);
+
+       mouse_mode_button_table->attach (mouse_zoom_button, c, c + 1, 0, 1);
+       ++c;
+       
        if (!Profile->get_sae()) {
-               mouse_mode_button_box.pack_start(mouse_gain_button, true, true);
+               mouse_mode_button_table->attach (mouse_gain_button, c, c + 1, 0, 1);
+               ++c;
        }
-       mouse_mode_button_box.pack_start(mouse_timefx_button, true, true);
-       mouse_mode_button_box.pack_start(mouse_audition_button, true, true);
-       mouse_mode_button_box.pack_start(internal_edit_button, true, true);
-       mouse_mode_button_box.set_homogeneous(true);
-
+       
+       mouse_mode_button_table->attach (mouse_timefx_button, c, c + 1, 0, 1);
+       ++c;
+       
+       mouse_mode_button_table->attach (mouse_audition_button, c, c + 1, 0, 1);
+       ++c;
+       
+       mouse_mode_button_table->attach (internal_edit_button, c, c + 1, 0, 1);
+       ++c;
+       
        vector<string> edit_mode_strings;
        edit_mode_strings.push_back (edit_mode_to_string (Slide));
        if (!Profile->get_sae()) {
@@ -2790,8 +2809,8 @@ Editor::setup_toolbar ()
        set_popdown_strings (edit_mode_selector, edit_mode_strings, true);
        edit_mode_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::edit_mode_selection_done));
 
-       mode_box->pack_start(edit_mode_selector);
-       mode_box->pack_start(mouse_mode_button_box);
+       mode_box->pack_start (edit_mode_selector);
+       mode_box->pack_start (*mouse_mode_button_table);
 
        mouse_mode_tearoff = manage (new TearOff (*mode_box));
        mouse_mode_tearoff->set_name ("MouseModeBase");
@@ -2816,6 +2835,7 @@ Editor::setup_toolbar ()
        mouse_zoom_button.set_mode (false);
        mouse_timefx_button.set_mode (false);
        mouse_audition_button.set_mode (false);
+       join_object_range_button.set_mode (false);
 
        mouse_move_button.set_name ("MouseModeButton");
        mouse_select_button.set_name ("MouseModeButton");
@@ -2823,8 +2843,8 @@ Editor::setup_toolbar ()
        mouse_zoom_button.set_name ("MouseModeButton");
        mouse_timefx_button.set_name ("MouseModeButton");
        mouse_audition_button.set_name ("MouseModeButton");
-
        internal_edit_button.set_name ("MouseModeButton");
+       join_object_range_button.set_name ("MouseModeButton");
 
        mouse_move_button.unset_flags (CAN_FOCUS);
        mouse_select_button.unset_flags (CAN_FOCUS);
@@ -2833,6 +2853,7 @@ Editor::setup_toolbar ()
        mouse_timefx_button.unset_flags (CAN_FOCUS);
        mouse_audition_button.unset_flags (CAN_FOCUS);
        internal_edit_button.unset_flags (CAN_FOCUS);
+       join_object_range_button.unset_flags (CAN_FOCUS);
 
        /* Zoom */
 
@@ -2971,8 +2992,8 @@ Editor::midi_panic ()
 {
        cerr << "MIDI panic\n";
 
-       if (session) {
-               session->midi_panic();
+       if (_session) {
+               _session->midi_panic();
        }
 }
 
@@ -3008,7 +3029,7 @@ Editor::convert_drop_to_paths (
                guint                           /*info*/,
                guint                           /*time*/)
 {
-       if (session == 0) {
+       if (_session == 0) {
                return -1;
        }
 
@@ -3120,7 +3141,7 @@ Editor::map_transport_state ()
 {
        ENSURE_GUI_THREAD (*this, &Editor::map_transport_state)
 
-       if (session->transport_stopped()) {
+       if (_session->transport_stopped()) {
                have_pending_keyboard_selection = false;
        }
 
@@ -3139,40 +3160,20 @@ Editor::State::~State ()
        delete selection;
 }
 
-void
-Editor::store_state (State& state) const
-{
-       *state.selection = *selection;
-}
-
-void
-Editor::restore_state (State *state)
-{
-       if (*selection == *state->selection) {
-               return;
-       }
-
-       *selection = *state->selection;
-       time_selection_changed ();
-       region_selection_changed ();
-
-       /* XXX other selection change handlers? */
-}
-
 void
 Editor::begin_reversible_command (string name)
 {
-       if (session) {
+       if (_session) {
                before = &get_state();
-               session->begin_reversible_command (name);
+               _session->begin_reversible_command (name);
        }
 }
 
 void
 Editor::commit_reversible_command ()
 {
-       if (session) {
-               session->commit_reversible_command (new MementoCommand<Editor>(*this, before, &get_state()));
+       if (_session) {
+               _session->commit_reversible_command (new MementoCommand<Editor>(*this, before, &get_state()));
        }
 }
 
@@ -3205,20 +3206,20 @@ Editor::history_changed ()
 {
        string label;
 
-       if (undo_action && session) {
-               if (session->undo_depth() == 0) {
+       if (undo_action && _session) {
+               if (_session->undo_depth() == 0) {
                        label = _("Undo");
                } else {
-                       label = string_compose(_("Undo (%1)"), session->next_undo());
+                       label = string_compose(_("Undo (%1)"), _session->next_undo());
                }
                undo_action->property_label() = label;
        }
 
-       if (redo_action && session) {
-               if (session->redo_depth() == 0) {
+       if (redo_action && _session) {
+               if (_session->redo_depth() == 0) {
                        label = _("Redo");
                } else {
-                       label = string_compose(_("Redo (%1)"), session->next_redo());
+                       label = string_compose(_("Redo (%1)"), _session->next_redo());
                }
                redo_action->property_label() = label;
        }
@@ -3394,7 +3395,7 @@ Editor::cycle_edit_mode ()
 void
 Editor::edit_mode_selection_done ()
 {
-       if (session == 0) {
+       if (_session == 0) {
                return;
        }
 
@@ -3572,43 +3573,6 @@ Editor::mouse_select_button_release (GdkEventButton* ev)
        return true;
 }
 
-Editor::TrackViewList *
-Editor::get_valid_views (TimeAxisView* track, RouteGroup* group)
-{
-       TrackViewList::iterator i;
-
-       TrackViewList* v = new TrackViewList;
-
-       if (track == 0 && group == 0) {
-
-               /* all views */
-
-               for (i = track_views.begin(); i != track_views.end (); ++i) {
-                       v->push_back (*i);
-               }
-
-       } else if ((track != 0 && group == 0) || (track != 0 && group != 0 && !group->active_property (RouteGroup::Select))) {
-
-               /* just the view for this track
-                */
-
-               v->push_back (track);
-
-       } else {
-
-               /* views for all tracks in the route group */
-
-               for (i = track_views.begin(); i != track_views.end (); ++i) {
-
-                       if (group == 0 || ((*i)->route_group() == group && group->active_property (RouteGroup::Select))) {
-                               v->push_back (*i);
-                       }
-               }
-       }
-
-       return v;
-}
-
 void
 Editor::set_zoom_focus (ZoomFocus f)
 {
@@ -3772,7 +3736,7 @@ Editor::edit_xfade (boost::weak_ptr<Crossfade> wxfade)
                return;
        }
 
-       CrossfadeEditor cew (*session, xfade, xfade->fade_in().get_min_y(), 1.0);
+       CrossfadeEditor cew (_session, xfade, xfade->fade_in().get_min_y(), 1.0);
 
        ensure_float (cew);
 
@@ -3824,8 +3788,8 @@ Editor::get_grid_type_as_beats (bool& success, nframes64_t position)
                break;
 
        case SnapToBar:
-               if (session) {
-                       return session->tempo_map().meter_at (position).beats_per_bar();
+               if (_session) {
+                       return _session->tempo_map().meter_at (position).beats_per_bar();
                }
                break;
 
@@ -3936,8 +3900,8 @@ Editor::finish_cleanup ()
 Location*
 Editor::transport_loop_location()
 {
-       if (session) {
-               return session->locations()->auto_loop_location();
+       if (_session) {
+               return _session->locations()->auto_loop_location();
        } else {
                return 0;
        }
@@ -3946,8 +3910,8 @@ Editor::transport_loop_location()
 Location*
 Editor::transport_punch_location()
 {
-       if (session) {
-               return session->locations()->auto_punch_location();
+       if (_session) {
+               return _session->locations()->auto_punch_location();
        } else {
                return 0;
        }
@@ -3981,8 +3945,9 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
 void
 Editor::session_state_saved (string snap_name)
 {
-       ENSURE_GUI_THREAD (*this, &Editor::session_state_saved, snap_name)
-
+       ENSURE_GUI_THREAD (*this, &Editor::session_state_saved, snap_name);
+       
+       update_title ();        
        _snapshots->redisplay ();
 }
 
@@ -4038,7 +4003,7 @@ Editor::new_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("new playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
-       session->playlists->get (playlists);
+       _session->playlists->get (playlists);
        mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_new_playlist), playlists), v, RouteGroup::Edit);
        commit_reversible_command ();
 }
@@ -4054,7 +4019,7 @@ Editor::copy_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("copy playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
-       session->playlists->get (playlists);
+       _session->playlists->get (playlists);
        mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_copy_playlist), playlists), v, RouteGroup::Edit);
        commit_reversible_command ();
 }
@@ -4069,7 +4034,7 @@ Editor::clear_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("clear playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
-       session->playlists->get (playlists);
+       _session->playlists->get (playlists);
        mapover_tracks (sigc::mem_fun (*this, &Editor::mapped_clear_playlist), v, RouteGroup::Edit);
        commit_reversible_command ();
 }
@@ -4267,14 +4232,8 @@ Editor::post_zoom ()
        }
 
        if (mouse_mode == MouseRange && selection->time.start () != selection->time.end_frame ()) {
-               if (!selection->tracks.empty()) {
-                       for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
-                               (*i)->reshow_selection (selection->time);
-                       }
-               } else {
-                       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-                               (*i)->reshow_selection (selection->time);
-                       }
+               for (TrackViewList::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
+                       (*i)->reshow_selection (selection->time);
                }
        }
 
@@ -4282,8 +4241,8 @@ Editor::post_zoom ()
 
        ZoomChanged (); /* EMIT_SIGNAL */
 
-       if (session) {
-               cef = session->current_end_frame() + (current_page_frames() / 10);// Add a little extra so we can see the end marker
+       if (_session) {
+               cef = _session->current_end_frame() + (current_page_frames() / 10);// Add a little extra so we can see the end marker
        }
        horizontal_adjustment.set_upper (cef / frames_per_unit);
 
@@ -4308,7 +4267,7 @@ Editor::queue_visual_change (nframes64_t where)
           can reach.
        */
 
-       if (session && (where > session->current_end_frame())) {
+       if (_session && (where > _session->current_end_frame())) {
                horizontal_adjustment.set_upper ((where + current_page_frames()) / frames_per_unit);
        }
 
@@ -4375,9 +4334,9 @@ Editor::idle_visual_changer ()
        nframes64_t csf=0, cef=0;
        nframes64_t current_time_origin = (nframes64_t) floor (horizontal_adjustment.get_value() * frames_per_unit);
 
-       if (session) {
-               csf = session->current_start_frame();
-               cef = session->current_end_frame();
+       if (_session) {
+               csf = _session->current_start_frame();
+               cef = _session->current_end_frame();
        }
 
        /* if we seek beyond the current end of the canvas, move the end */
@@ -4412,7 +4371,7 @@ struct EditorOrderTimeAxisSorter {
 };
 
 void
-Editor::sort_track_selection (TrackSelection* sel)
+Editor::sort_track_selection (TrackViewList* sel)
 {
        EditorOrderTimeAxisSorter cmp;
 
@@ -4440,7 +4399,7 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
 
        switch (ep) {
        case EditAtPlayhead:
-               where = session->audible_frame();
+               where = _session->audible_frame();
                break;
 
        case EditAtSelectedMarker:
@@ -4474,7 +4433,7 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
 void
 Editor::set_loop_range (nframes64_t start, nframes64_t end, string cmd)
 {
-       if (!session) return;
+       if (!_session) return;
 
        begin_reversible_command (cmd);
 
@@ -4482,17 +4441,17 @@ Editor::set_loop_range (nframes64_t start, nframes64_t end, string cmd)
 
        if ((tll = transport_loop_location()) == 0) {
                Location* loc = new Location (start, end, _("Loop"),  Location::IsAutoLoop);
-               XMLNode &before = session->locations()->get_state();
-               session->locations()->add (loc, true);
-               session->set_auto_loop_location (loc);
-               XMLNode &after = session->locations()->get_state();
-               session->add_command (new MementoCommand<Locations>(*(session->locations()), &before, &after));
+               XMLNode &before = _session->locations()->get_state();
+               _session->locations()->add (loc, true);
+               _session->set_auto_loop_location (loc);
+               XMLNode &after = _session->locations()->get_state();
+               _session->add_command (new MementoCommand<Locations>(*(_session->locations()), &before, &after));
        } else {
                XMLNode &before = tll->get_state();
                tll->set_hidden (false, this);
                tll->set (start, end);
                XMLNode &after = tll->get_state();
-               session->add_command (new MementoCommand<Location>(*tll, &before, &after));
+               _session->add_command (new MementoCommand<Location>(*tll, &before, &after));
        }
 
        commit_reversible_command ();
@@ -4501,7 +4460,7 @@ Editor::set_loop_range (nframes64_t start, nframes64_t end, string cmd)
 void
 Editor::set_punch_range (nframes64_t start, nframes64_t end, string cmd)
 {
-       if (!session) return;
+       if (!_session) return;
 
        begin_reversible_command (cmd);
 
@@ -4509,18 +4468,18 @@ Editor::set_punch_range (nframes64_t start, nframes64_t end, string cmd)
 
        if ((tpl = transport_punch_location()) == 0) {
                Location* loc = new Location (start, end, _("Loop"),  Location::IsAutoPunch);
-               XMLNode &before = session->locations()->get_state();
-               session->locations()->add (loc, true);
-               session->set_auto_loop_location (loc);
-               XMLNode &after = session->locations()->get_state();
-               session->add_command (new MementoCommand<Locations>(*(session->locations()), &before, &after));
+               XMLNode &before = _session->locations()->get_state();
+               _session->locations()->add (loc, true);
+               _session->set_auto_loop_location (loc);
+               XMLNode &after = _session->locations()->get_state();
+               _session->add_command (new MementoCommand<Locations>(*(_session->locations()), &before, &after));
        }
        else {
                XMLNode &before = tpl->get_state();
                tpl->set_hidden (false, this);
                tpl->set (start, end);
                XMLNode &after = tpl->get_state();
-               session->add_command (new MementoCommand<Location>(*tpl, &before, &after));
+               _session->add_command (new MementoCommand<Location>(*tpl, &before, &after));
        }
 
        commit_reversible_command ();
@@ -4532,9 +4491,9 @@ Editor::set_punch_range (nframes64_t start, nframes64_t end, string cmd)
  *  @param ts Tracks to look on; if this is empty, all tracks are examined.
  */
 void
-Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackSelection& ts) const
+Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackViewList& ts) const
 {
-       const TrackSelection* tracks;
+       const TrackViewList* tracks;
 
        if (ts.empty()) {
                tracks = &track_views;
@@ -4542,7 +4501,7 @@ Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackSelec
                tracks = &ts;
        }
 
-       for (TrackSelection::const_iterator t = tracks->begin(); t != tracks->end(); ++t) {
+       for (TrackViewList::const_iterator t = tracks->begin(); t != tracks->end(); ++t) {
                RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(*t);
                if (rtv) {
                        boost::shared_ptr<Diskstream> ds;
@@ -4567,9 +4526,9 @@ Editor::get_regions_at (RegionSelection& rs, nframes64_t where, const TrackSelec
 }
 
 void
-Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackSelection& ts) const
+Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackViewList& ts) const
 {
-       const TrackSelection* tracks;
+       const TrackViewList* tracks;
 
        if (ts.empty()) {
                tracks = &track_views;
@@ -4577,7 +4536,7 @@ Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackSe
                tracks = &ts;
        }
 
-       for (TrackSelection::const_iterator t = tracks->begin(); t != tracks->end(); ++t) {
+       for (TrackViewList::const_iterator t = tracks->begin(); t != tracks->end(); ++t) {
                RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(*t);
                if (rtv) {
                        boost::shared_ptr<Diskstream> ds;
@@ -4624,22 +4583,25 @@ Editor::get_regions_for_action (RegionSelection& rs, bool allow_entered, bool al
 
        if (allow_edit_position) {
 
-               TrackSelection tracks = selection->tracks;
+               TrackViewList tracks = selection->tracks;
 
                /* tracks is currently the set of selected tracks; add any other tracks that
                 * have regions that are in the same edit-activated route group as one of
                 * our regions */
                for (RegionSelection::iterator i = rs.begin (); i != rs.end(); ++i) {
 
-                       RouteGroup* g = (*i)->get_time_axis_view().route_group ();
-                       if (g && g->active_property (RouteGroup::Edit)) {
-                               tracks.add (axis_views_from_routes (g->route_list()));
-                       }
+                       RouteGroup* g = (*i)->get_time_axis_view().route_group ();
+                       if (g && g->active_property (RouteGroup::Edit)) {
+                               tracks.add (axis_views_from_routes (g->route_list()));
+                       }
+                       
                }
 
-               /* now find regions that are at the edit position on those tracks */
-               nframes64_t const where = get_preferred_edit_position ();
-               get_regions_at (rs, where, tracks);
+               if (!tracks.empty()) {
+                       /* now find regions that are at the edit position on those tracks */
+                       nframes64_t const where = get_preferred_edit_position ();
+                       get_regions_at (rs, where, tracks);
+               }
        }
 }
 
@@ -4683,7 +4645,7 @@ Editor::show_rhythm_ferret ()
                rhythm_ferret = new RhythmFerret(*this);
        }
 
-       rhythm_ferret->set_session (session);
+       rhythm_ferret->set_session (_session);
        rhythm_ferret->show ();
        rhythm_ferret->present ();
 }
@@ -4692,7 +4654,7 @@ void
 Editor::show_global_port_matrix (ARDOUR::DataType t)
 {
        if (_global_port_matrix[t] == 0) {
-               _global_port_matrix[t] = new GlobalPortMatrixWindow (session, t);
+               _global_port_matrix[t] = new GlobalPortMatrixWindow (_session, t);
        }
 
        _global_port_matrix[t]->show ();
@@ -4750,7 +4712,7 @@ Editor::add_to_idle_resize (TimeAxisView* view, int32_t h)
        min_resulting = min (min_resulting, int32_t (_pending_resize_view->current_height()) + _pending_resize_amount);
 
        if (selection->tracks.contains (_pending_resize_view)) {
-               for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
+               for (TrackViewList::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
                        min_resulting = min (min_resulting, int32_t ((*i)->current_height()) + _pending_resize_amount);
                }
        }
@@ -4774,7 +4736,7 @@ Editor::idle_resize ()
        if (dynamic_cast<AutomationTimeAxisView*> (_pending_resize_view) == 0 &&
            selection->tracks.contains (_pending_resize_view)) {
 
-               for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
+               for (TrackViewList::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
                        if (*i != _pending_resize_view) {
                                (*i)->idle_resize ((*i)->current_height() + _pending_resize_amount);
                        }
@@ -4824,10 +4786,10 @@ Editor::axis_view_from_route (boost::shared_ptr<Route> r) const
 }
 
 
-TrackSelection
+TrackViewList
 Editor::axis_views_from_routes (boost::shared_ptr<RouteList> r) const
 {
-       TrackSelection t;
+       TrackViewList t;
 
        for (RouteList::const_iterator i = r->begin(); i != r->end(); ++i) {
                TimeAxisView* tv = axis_view_from_route (*i);
@@ -4858,9 +4820,9 @@ Editor::handle_new_route (RouteList& routes)
                DataType dt = route->input()->default_type();
 
                if (dt == ARDOUR::DataType::AUDIO) {
-                       rtv = new AudioTimeAxisView (*this, *session, route, *track_canvas);
+                       rtv = new AudioTimeAxisView (*this, _session, route, *track_canvas);
                } else if (dt == ARDOUR::DataType::MIDI) {
-                       rtv = new MidiTimeAxisView (*this, *session, route, *track_canvas);
+                       rtv = new MidiTimeAxisView (*this, _session, route, *track_canvas);
                } else {
                        throw unknown_type();
                }
@@ -4872,8 +4834,6 @@ Editor::handle_new_route (RouteList& routes)
 
                rtv->view()->RegionViewAdded.connect (sigc::mem_fun (*this, &Editor::region_view_added));
                rtv->view()->HeightChanged.connect (sigc::mem_fun (*this, &Editor::streamview_height_changed));
-
-               rtv->GoingAway.connect (sigc::bind (sigc::mem_fun(*this, &Editor::remove_route), rtv));
        }
 
        _routes->routes_added (new_views);
@@ -4888,49 +4848,62 @@ Editor::handle_new_route (RouteList& routes)
 }
 
 void
-Editor::remove_route (TimeAxisView *tv)
+Editor::timeaxisview_deleted (TimeAxisView *tv)
 {
-       ENSURE_GUI_THREAD (*this, &Editor::remove_route, tv)
-
-       TrackViewList::iterator i;
-
-       boost::shared_ptr<Route> route;
-       RouteTimeAxisView* rtav = dynamic_cast<RouteTimeAxisView*> (tv);
-       if (rtav) {
-               route = rtav->route ();
+       if (_session && _session->deletion_in_progress()) {
+               /* the situation is under control */
+               return;
        }
+
+       ENSURE_GUI_THREAD (*this, &Editor::timeaxisview_deleted, tv);
                
-       TimeAxisView* next_tv = 0;
+
+       _routes->route_removed (tv);
 
        if (tv == entered_track) {
                entered_track = 0;
        }
+       
+       /* remove it from the list of track views */
+
+       TrackViewList::iterator i;
 
        if ((i = find (track_views.begin(), track_views.end(), tv)) != track_views.end()) {
+               i = track_views.erase (i);
+       }
 
-               i = track_views.erase (i);
+       /* update whatever the current mixer strip is displaying, if revelant */
 
-               if (track_views.empty()) {
-                       next_tv = 0;
-               } else if (i == track_views.end()) {
-                       next_tv = track_views.front();
-               } else {
-                      next_tv = (*i);
-               }
-       }
+       boost::shared_ptr<Route> route;
+       RouteTimeAxisView* rtav = dynamic_cast<RouteTimeAxisView*> (tv);
+
+       if (rtav) {
+               route = rtav->route ();
+       } 
 
        if (current_mixer_strip && current_mixer_strip->route() == route) {
 
-               if (next_tv) {
-                       set_selected_mixer_strip (*next_tv);
-               } else {
-                       /* make the editor mixer strip go away setting the
-                        * button to inactive (which also unticks the menu option)
-                        */
+               TimeAxisView* next_tv;
 
-                       ActionManager::uncheck_toggleaction ("<Actions>/Editor/show-editor-mixer");
-               }
-       }
+               if (track_views.empty()) {
+                       next_tv = 0;
+               } else if (i == track_views.end()) {
+                       next_tv = track_views.front();
+               } else {
+                       next_tv = (*i);
+               }
+               
+               
+               if (next_tv) {
+                       set_selected_mixer_strip (*next_tv);
+               } else {
+                       /* make the editor mixer strip go away setting the
+                        * button to inactive (which also unticks the menu option)
+                        */
+                       
+                       ActionManager::uncheck_toggleaction ("<Actions>/Editor/show-editor-mixer");
+               }
+       } 
 }
 
 void
@@ -4949,7 +4922,7 @@ Editor::hide_track_in_display (TimeAxisView& tv, bool /*temponly*/)
 bool
 Editor::sync_track_view_list_and_routes ()
 {
-       track_views = TrackSelection (_routes->views ());
+       track_views = TrackViewList (_routes->views ());
 
        _summary->set_dirty ();
        _group_tabs->set_dirty ();
@@ -4984,7 +4957,7 @@ Editor::get_route_view_by_id (PBD::ID& id)
 void
 Editor::fit_route_group (RouteGroup *g)
 {
-       TrackSelection ts = axis_views_from_routes (g->route_list ());
+       TrackViewList ts = axis_views_from_routes (g->route_list ());
        fit_tracks (ts);
 }
 
@@ -4994,18 +4967,18 @@ Editor::consider_auditioning (boost::shared_ptr<Region> region)
        boost::shared_ptr<AudioRegion> r = boost::dynamic_pointer_cast<AudioRegion> (region);
 
        if (r == 0) {
-               session->cancel_audition ();
+               _session->cancel_audition ();
                return;
        }
 
-       if (session->is_auditioning()) {
-               session->cancel_audition ();
+       if (_session->is_auditioning()) {
+               _session->cancel_audition ();
                if (r == last_audition_region) {
                        return;
                }
        }
 
-       session->audition_region (r);
+       _session->audition_region (r);
        last_audition_region = r;
 }
 
@@ -5019,7 +4992,7 @@ Editor::hide_a_region (boost::shared_ptr<Region> r)
 void
 Editor::remove_a_region (boost::shared_ptr<Region> r)
 {
-       session->remove_region_from_region_list (r);
+       _session->remove_region_from_region_list (r);
 }
 
 void