use new syntax for connecting to backend signals that enforces explicit connection...
[ardour.git] / gtk2_ardour / editor.cc
index ec351be787e33bfab7c7502ab148e2d64977d78c..57d1ef092220f2d70594b1a60660452788fc9e23 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2000-2009 Paul Davis 
+    Copyright (C) 2000-2009 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 #include <algorithm>
 #include <map>
 
+#include "ardour_ui.h"
+/*
+ * ardour_ui.h include was moved to the top of the list
+ * due to a conflicting definition of 'Style' between
+ * Apple's MacTypes.h and BarController.
+ */
+
 #include <boost/none.hpp>
 
 #include <sigc++/bind.h>
 #include "ardour/session_state_utils.h"
 #include "ardour/tempo.h"
 #include "ardour/utils.h"
+#include "ardour/session_playlists.h"
 
 #include "control_protocol/control_protocol.h"
 
-#include "ardour_ui.h"
 #include "editor.h"
 #include "keyboard.h"
 #include "marker.h"
@@ -81,6 +88,7 @@
 #include "audio_time_axis.h"
 #include "utils.h"
 #include "crossfade_view.h"
+#include "canvas-noevent-text.h"
 #include "editing.h"
 #include "public_editor.h"
 #include "crossfade_edit.h"
 #include "mixer_strip.h"
 #include "editor_route_groups.h"
 #include "editor_regions.h"
+#include "editor_locations.h"
 #include "editor_snapshots.h"
 
 #include "i18n.h"
 #endif
 
 using namespace std;
-using namespace sigc;
 using namespace ARDOUR;
 using namespace PBD;
 using namespace Gtk;
@@ -122,6 +130,7 @@ using namespace Editing;
 
 using PBD::internationalize;
 using PBD::atoi;
+using Gtkmm2ext::Keyboard;
 
 const double Editor::timebar_height = 15.0;
 
@@ -129,9 +138,9 @@ const double Editor::timebar_height = 15.0;
 
 static const gchar *_snap_type_strings[] = {
        N_("CD Frames"),
-       N_("SMPTE Frames"),
-       N_("SMPTE Seconds"),
-       N_("SMPTE Minutes"),
+       N_("Timecode Frames"),
+       N_("Timecode Seconds"),
+       N_("Timecode Minutes"),
        N_("Seconds"),
        N_("Minutes"),
        N_("Beats/32"),
@@ -169,7 +178,7 @@ static const gchar *_zoom_focus_strings[] = {
        N_("Center"),
        N_("Playhead"),
        N_("Mouse"),
-       N_("Active Mark"),
+       N_("Edit point"),
        0
 };
 
@@ -214,7 +223,7 @@ Editor::Editor ()
          /* time display buttons */
        : minsec_label (_("Mins:Secs"))
        , bbt_label (_("Bars:Beats"))
-       , smpte_label (_("Timecode"))
+       , timecode_label (_("Timecode"))
        , frame_label (_("Samples"))
        , tempo_label (_("Tempo"))
        , meter_label (_("Meter"))
@@ -227,29 +236,28 @@ Editor::Editor ()
          /* the values here don't matter: layout widgets
             reset them as needed.
          */
-         
+
        , vertical_adjustment (0.0, 0.0, 10.0, 400.0)
        , horizontal_adjustment (0.0, 0.0, 20.0, 1200.0)
-         
+
          /* tool bar related */
 
-       , edit_point_clock (X_("editpoint"), false, X_("EditPointClock"), true)
-       , zoom_range_clock (X_("zoomrange"), false, X_("ZoomRangeClock"), true, true)
-         
+       , zoom_range_clock (X_("zoomrange"), false, X_("ZoomRangeClock"), true, false, true)
+
        , toolbar_selection_clock_table (2,3)
-         
+
        , automation_mode_button (_("mode"))
        , global_automation_button (_("automation"))
-         
+
        , midi_panic_button (_("Panic"))
-         
+
 #ifdef WITH_CMT
        , image_socket_listener(0)
 #endif
-         
+
          /* nudge */
-         
-       , nudge_clock (X_("nudge"), false, X_("NudgeClock"), true, true)
+
+       , nudge_clock (X_("nudge"), false, X_("NudgeClock"), true, false, true)
        , meters_running(false)
        , _pending_locate_request (false)
 
@@ -260,7 +268,6 @@ Editor::Editor ()
 
        PublicEditor::_instance = this;
 
-       session = 0;
        _have_idled = false;
 
        selection = new Selection (this);
@@ -276,7 +283,7 @@ Editor::Editor ()
        current_mixer_strip = 0;
        current_bbt_points = 0;
        tempo_lines = 0;
-       
+
        snap_type_strings =  I18N (_snap_type_strings);
        snap_mode_strings =  I18N (_snap_mode_strings);
        zoom_focus_strings = I18N (_zoom_focus_strings);
@@ -284,7 +291,7 @@ Editor::Editor ()
 #ifdef USE_RUBBERBAND
        rb_opt_strings = I18N (_rb_opt_strings);
 #endif
-       
+
        snap_threshold = 5.0;
        bbt_beat_subdivision = 4;
        _canvas_width = 0;
@@ -336,9 +343,7 @@ Editor::Editor ()
        button_release_can_deselect = true;
        _dragging_playhead = false;
        _dragging_edit_point = false;
-       _dragging_hscrollbar = false;
        select_new_marker = false;
-       zoomed_to_region = false;
        rhythm_ferret = 0;
        _bundle_manager = 0;
        for (ARDOUR::DataType::iterator i = ARDOUR::DataType::begin(); i != ARDOUR::DataType::end(); ++i) {
@@ -363,11 +368,10 @@ Editor::Editor ()
        current_canvas_cursor = 0;
 
        frames_per_unit = 2048; /* too early to use reset_zoom () */
-       reset_hscrollbar_stepping ();
-       
+
        zoom_focus = ZoomFocusLeft;
        set_zoom_focus (ZoomFocusLeft);
-       zoom_range_clock.ValueChanged.connect (mem_fun(*this, &Editor::zoom_adjustment_changed));
+       zoom_range_clock.ValueChanged.connect (sigc::mem_fun(*this, &Editor::zoom_adjustment_changed));
 
        bbt_label.set_name ("EditorTimeButton");
        bbt_label.set_size_request (-1, (int)timebar_height);
@@ -381,12 +385,12 @@ Editor::Editor ()
        minsec_label.set_padding (5,0);
        minsec_label.hide ();
        minsec_label.set_no_show_all();
-       smpte_label.set_name ("EditorTimeButton");
-       smpte_label.set_size_request (-1, (int)timebar_height);
-       smpte_label.set_alignment (1.0, 0.5);
-       smpte_label.set_padding (5,0);
-       smpte_label.hide ();
-       smpte_label.set_no_show_all();
+       timecode_label.set_name ("EditorTimeButton");
+       timecode_label.set_size_request (-1, (int)timebar_height);
+       timecode_label.set_alignment (1.0, 0.5);
+       timecode_label.set_padding (5,0);
+       timecode_label.hide ();
+       timecode_label.set_no_show_all();
        frame_label.set_name ("EditorTimeButton");
        frame_label.set_size_request (-1, (int)timebar_height);
        frame_label.set_alignment (1.0, 0.5);
@@ -432,49 +436,38 @@ Editor::Editor ()
        transport_mark_label.set_no_show_all();
 
        initialize_rulers ();
-       _summary = new EditorSummary (this);
        initialize_canvas ();
+       _summary = new EditorSummary (this);
 
-       selection->TimeChanged.connect (mem_fun(*this, &Editor::time_selection_changed));
-       selection->TracksChanged.connect (mem_fun(*this, &Editor::track_selection_changed));
-       editor_regions_selection_changed_connection = selection->RegionsChanged.connect (mem_fun(*this, &Editor::region_selection_changed));
-       selection->PointsChanged.connect (mem_fun(*this, &Editor::point_selection_changed));
-       selection->MarkersChanged.connect (mem_fun(*this, &Editor::marker_selection_changed));
+       selection->TimeChanged.connect (sigc::mem_fun(*this, &Editor::time_selection_changed));
+       selection->TracksChanged.connect (sigc::mem_fun(*this, &Editor::track_selection_changed));
+       editor_regions_selection_changed_connection = selection->RegionsChanged.connect (sigc::mem_fun(*this, &Editor::region_selection_changed));
+       selection->PointsChanged.connect (sigc::mem_fun(*this, &Editor::point_selection_changed));
+       selection->MarkersChanged.connect (sigc::mem_fun(*this, &Editor::marker_selection_changed));
 
        edit_controls_vbox.set_spacing (0);
-       horizontal_adjustment.signal_value_changed().connect (mem_fun(*this, &Editor::scroll_canvas_horizontally), false);
-       vertical_adjustment.signal_value_changed().connect (mem_fun(*this, &Editor::tie_vertical_scrolling), true);
-       track_canvas->signal_map_event().connect (mem_fun (*this, &Editor::track_canvas_map_handler));
+       horizontal_adjustment.signal_value_changed().connect (sigc::mem_fun(*this, &Editor::scroll_canvas_horizontally), false);
+       vertical_adjustment.signal_value_changed().connect (sigc::mem_fun(*this, &Editor::tie_vertical_scrolling), true);
+       track_canvas->signal_map_event().connect (sigc::mem_fun (*this, &Editor::track_canvas_map_handler));
 
        HBox* h = manage (new HBox);
        _group_tabs = new EditorGroupTabs (this);
        h->pack_start (*_group_tabs, PACK_SHRINK);
        h->pack_start (edit_controls_vbox);
        controls_layout.add (*h);
-       
-       controls_layout.set_name ("EditControlsBase");
-       controls_layout.add_events (Gdk::SCROLL_MASK);
-       controls_layout.signal_scroll_event().connect (mem_fun(*this, &Editor::control_layout_scroll), false);
-       
-       controls_layout.add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::ENTER_NOTIFY_MASK|Gdk::LEAVE_NOTIFY_MASK);
-       controls_layout.signal_button_release_event().connect (mem_fun(*this, &Editor::edit_controls_button_release));
-       controls_layout_size_request_connection = controls_layout.signal_size_request().connect (mem_fun (*this, &Editor::controls_layout_size_request));
 
-       edit_vscrollbar.set_adjustment (vertical_adjustment);
-       edit_hscrollbar.set_adjustment (horizontal_adjustment);
+       ARDOUR_UI::instance()->tooltips().set_tip (*_group_tabs, _("Groups: context-click for possible operations"));
 
-       edit_hscrollbar.signal_button_press_event().connect (mem_fun(*this, &Editor::hscrollbar_button_press), false);
-       edit_hscrollbar.signal_button_release_event().connect (mem_fun(*this, &Editor::hscrollbar_button_release), false);
-       edit_hscrollbar.signal_size_allocate().connect (mem_fun(*this, &Editor::hscrollbar_allocate));
+       controls_layout.set_name ("EditControlsBase");
+       controls_layout.add_events (Gdk::SCROLL_MASK);
+       controls_layout.signal_scroll_event().connect (sigc::mem_fun(*this, &Editor::control_layout_scroll), false);
 
-       edit_hscrollbar.set_name ("EditorHScrollbar");
+       controls_layout.add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::ENTER_NOTIFY_MASK|Gdk::LEAVE_NOTIFY_MASK);
+       controls_layout.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::edit_controls_button_release));
+       controls_layout_size_request_connection = controls_layout.signal_size_request().connect (sigc::mem_fun (*this, &Editor::controls_layout_size_request));
 
        build_cursors ();
-       setup_toolbar ();
-       setup_midi_toolbar ();
 
-       edit_point_clock.ValueChanged.connect (mem_fun(*this, &Editor::edit_point_clock_changed));
-       
        ArdourCanvas::Canvas* time_pad = manage(new ArdourCanvas::Canvas());
        ArdourCanvas::SimpleLine* pad_line_1 = manage(new ArdourCanvas::SimpleLine(*time_pad->root(),
                        0.0, 1.0, 100.0, 1.0));
@@ -485,15 +478,15 @@ Editor::Editor ()
        time_canvas_vbox.set_size_request (-1, (int)(timebar_height * visible_timebars) + 2);
        time_canvas_vbox.set_size_request (-1, -1);
 
-       ruler_label_event_box.add (ruler_label_vbox);   
+       ruler_label_event_box.add (ruler_label_vbox);
        ruler_label_event_box.set_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
-       ruler_label_event_box.signal_button_release_event().connect (mem_fun(*this, &Editor::ruler_label_button_release));
+       ruler_label_event_box.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_label_button_release));
 
        time_button_event_box.add (time_button_vbox);
        time_button_event_box.set_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
-       time_button_event_box.signal_button_release_event().connect (mem_fun(*this, &Editor::ruler_label_button_release));
+       time_button_event_box.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_label_button_release));
 
-       /* these enable us to have a dedicated window (for cursor setting, etc.) 
+       /* these enable us to have a dedicated window (for cursor setting, etc.)
           for the canvas areas.
        */
 
@@ -527,22 +520,7 @@ Editor::Editor ()
        _routes = new EditorRoutes (this);
        _regions = new EditorRegions (this);
        _snapshots = new EditorSnapshots (this);
-
-       named_selection_scroller.add (named_selection_display);
-       named_selection_scroller.set_policy (POLICY_NEVER, POLICY_AUTOMATIC);
-
-       named_selection_model = TreeStore::create (named_selection_columns);
-       named_selection_display.set_model (named_selection_model);
-       named_selection_display.append_column (_("Chunks"), named_selection_columns.text);
-       named_selection_display.set_headers_visible (false);
-       named_selection_display.set_size_request (100, -1);
-       named_selection_display.set_name ("NamedSelectionDisplay");
-       
-       named_selection_display.get_selection()->set_mode (SELECTION_SINGLE);
-       named_selection_display.set_size_request (100, -1);
-       named_selection_display.signal_button_release_event().connect (mem_fun(*this, &Editor::named_selection_display_button_release), false);
-       named_selection_display.signal_key_release_event().connect (mem_fun(*this, &Editor::named_selection_display_key_release), false);
-       named_selection_display.get_selection()->signal_changed().connect (mem_fun (*this, &Editor::named_selection_display_selection_changed));
+       _locations = new EditorLocations (this);
 
        Gtk::Label* nlabel;
 
@@ -558,30 +536,58 @@ Editor::Editor ()
        nlabel = manage (new Label (_("Route Groups")));
        nlabel->set_angle (-90);
        the_notebook.append_page (_route_groups->widget (), *nlabel);
-       
-       if (!Profile->get_sae()) {
-               nlabel = manage (new Label (_("Chunks")));
-               nlabel->set_angle (-90);
-               the_notebook.append_page (named_selection_scroller, *nlabel);
-       }
+       nlabel = manage (new Label (_("Ranges & Marks")));
+       nlabel->set_angle (-90);
+       the_notebook.append_page (_locations->widget (), *nlabel);
 
        the_notebook.set_show_tabs (true);
        the_notebook.set_scrollable (true);
-       the_notebook.popup_enable ();
+       the_notebook.popup_disable ();
        the_notebook.set_tab_pos (Gtk::POS_RIGHT);
        the_notebook.show_all ();
-
+       
        post_maximal_editor_width = 0;
        post_maximal_pane_position = 0;
 
        VPaned *editor_summary_pane = manage(new VPaned());
        editor_summary_pane->pack1(edit_packer);
-       editor_summary_pane->pack2(*_summary);
+
+       Button* summary_arrows_left_left = manage (new Button);
+       summary_arrows_left_left->add (*manage (new Arrow (ARROW_LEFT, SHADOW_NONE)));
+       summary_arrows_left_left->signal_clicked().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_left));
+       Button* summary_arrows_left_right = manage (new Button);
+       summary_arrows_left_right->add (*manage (new Arrow (ARROW_RIGHT, SHADOW_NONE)));
+       summary_arrows_left_right->signal_clicked().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_right));
+       VBox* summary_arrows_left = manage (new VBox);
+       summary_arrows_left->pack_start (*summary_arrows_left_left);
+       summary_arrows_left->pack_start (*summary_arrows_left_right);
+
+       Button* summary_arrows_right_left = manage (new Button);
+       summary_arrows_right_left->add (*manage (new Arrow (ARROW_LEFT, SHADOW_NONE)));
+       summary_arrows_right_left->signal_clicked().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_left));
+       Button* summary_arrows_right_right = manage (new Button);
+       summary_arrows_right_right->add (*manage (new Arrow (ARROW_RIGHT, SHADOW_NONE)));
+       summary_arrows_right_right->signal_clicked().connect (sigc::mem_fun (*this, &Editor::horizontal_scroll_right));
+       VBox* summary_arrows_right = manage (new VBox);
+       summary_arrows_right->pack_start (*summary_arrows_right_left);
+       summary_arrows_right->pack_start (*summary_arrows_right_right);
+
+       Frame* summary_frame = manage (new Frame);
+       summary_frame->set_shadow_type (Gtk::SHADOW_ETCHED_IN);
+       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);
+       
+       editor_summary_pane->pack2(*summary_hbox);
 
        edit_pane.pack1 (*editor_summary_pane, true, true);
        edit_pane.pack2 (the_notebook, false, true);
-       
-       edit_pane.signal_size_allocate().connect (bind (mem_fun(*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&edit_pane)));
+
+       edit_pane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun(*this, &Editor::pane_allocation_handler), static_cast<Paned*> (&edit_pane)));
 
        top_hbox.pack_start (toolbar_frame, false, true);
 
@@ -602,23 +608,26 @@ Editor::Editor ()
        vpacker.pack_end (global_hpacker, true, true);
 
        /* register actions now so that set_state() can find them and set toggles/checks etc */
-       
+
        register_actions ();
 
-       snap_type = SnapToBeat;
-       set_snap_to (snap_type);
-       snap_mode = SnapOff;
-       set_snap_mode (snap_mode);
+       setup_toolbar ();
+       setup_midi_toolbar ();
+
+       _snap_type = SnapToBeat;
+       set_snap_to (_snap_type);
+       _snap_mode = SnapOff;
+       set_snap_mode (_snap_mode);
        set_mouse_mode (MouseObject, true);
        set_edit_point_preference (EditAtMouse, true);
 
        XMLNode* node = ARDOUR_UI::instance()->editor_settings();
-       set_state (*node);
+       set_state (*node, Stateful::loading_state_version);
 
        _playlist_selector = new PlaylistSelector();
-       _playlist_selector->signal_delete_event().connect (bind (sigc::ptr_fun (just_hide_it), static_cast<Window *> (_playlist_selector)));
+       _playlist_selector->signal_delete_event().connect (sigc::bind (sigc::ptr_fun (just_hide_it), static_cast<Window *> (_playlist_selector)));
 
-       RegionView::RegionViewGoingAway.connect (mem_fun(*this, &Editor::catch_vanishing_regionview));
+       RegionView::RegionViewGoingAway.connect (*this, boost::bind (&Editor::catch_vanishing_regionview, this, _1));
 
        /* nudge stuff */
 
@@ -663,18 +672,18 @@ Editor::Editor ()
        add (vpacker);
        add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK);
 
-       signal_configure_event().connect (mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::configure_handler));
-       signal_delete_event().connect (mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::exit_on_main_window_close));
+       signal_configure_event().connect (sigc::mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::configure_handler));
+       signal_delete_event().connect (sigc::mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::exit_on_main_window_close));
 
        /* allow external control surfaces/protocols to do various things */
 
-       ControlProtocol::ZoomToSession.connect (mem_fun (*this, &Editor::temporal_zoom_session));
-       ControlProtocol::ZoomIn.connect (bind (mem_fun (*this, &Editor::temporal_zoom_step), false));
-       ControlProtocol::ZoomOut.connect (bind (mem_fun (*this, &Editor::temporal_zoom_step), true));
-       ControlProtocol::ScrollTimeline.connect (mem_fun (*this, &Editor::control_scroll));
-       BasicUI::AccessAction.connect (mem_fun (*this, &Editor::access_action));
+       ControlProtocol::ZoomToSession.connect (*this, boost::bind (&Editor::temporal_zoom_session, this));
+       ControlProtocol::ZoomIn.connect (*this, boost::bind (&Editor::temporal_zoom_step, this, false));
+       ControlProtocol::ZoomOut.connect (*this, boost::bind (&Editor::temporal_zoom_step, this, true));
+       ControlProtocol::ScrollTimeline.connect (*this, boost::bind (&Editor::control_scroll, this, _1));
+       BasicUI::AccessAction.connect (*this, boost::bind (&Editor::access_action, this, _1, _2));
 
-       Config->ParameterChanged.connect (mem_fun (*this, &Editor::parameter_changed));
+       Config->ParameterChanged.connect (*this, boost::bind (&Editor::parameter_changed, this, _1));
 
        _last_normalization_value = 0;
 
@@ -690,7 +699,7 @@ Editor::~Editor()
                {
                        image_socket_listener->close_connection() ;
                }
-               
+
                delete image_socket_listener ;
                image_socket_listener = 0 ;
        }
@@ -778,7 +787,7 @@ Editor::show_window ()
                */
 
                TimeAxisView *tv;
-       
+
                for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                        tv = (static_cast<TimeAxisView*>(*i));
                        tv->reset_height ();
@@ -795,38 +804,17 @@ 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());
        }
 }
 
-void
-Editor::edit_point_clock_changed()
-{
-       if (_dragging_edit_point) {
-               return;
-       }
-
-       if (selection->markers.empty()) {
-               return;
-       }
-
-       bool ignored;
-       Location* loc = find_location_from_marker (selection->markers.front(), ignored);
-
-       if (!loc) {
-               return;
-       }
-
-       loc->move_to (edit_point_clock.current_time());
-}
-
 void
 Editor::zoom_adjustment_changed ()
 {
-       if (session == 0) {
+       if (_session == 0) {
                return;
        }
 
@@ -835,20 +823,20 @@ 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));
        }
-       
+
        temporal_zoom (fpu);
 }
 
 void
 Editor::control_scroll (float fraction)
 {
-       ENSURE_GUI_THREAD(bind (mem_fun (*this, &Editor::control_scroll), fraction));
+       ENSURE_GUI_THREAD (*this, &Editor::control_scroll, fraction)
 
-       if (!session) {
+       if (!_session) {
                return;
        }
 
@@ -856,7 +844,7 @@ Editor::control_scroll (float fraction)
 
        /*
                _control_scroll_target is an optional<T>
-       
+
                it acts like a pointer to an nframes64_t, with
                a operator conversion to boolean to check
                that it has a value could possibly use
@@ -866,7 +854,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;
        }
 
@@ -882,7 +870,7 @@ Editor::control_scroll (float fraction)
 
        playhead_cursor->set_position (*_control_scroll_target);
        UpdateAllTransportClocks (*_control_scroll_target);
-       
+
        if (*_control_scroll_target > (current_page_frames() / 2)) {
                /* try to center PH in window */
                reset_x_origin (*_control_scroll_target - (current_page_frames()/2));
@@ -902,13 +890,13 @@ Editor::control_scroll (float fraction)
 
        /* add the next timeout */
 
-       control_scroll_connection = Glib::signal_timeout().connect (bind (mem_fun (*this, &Editor::deferred_control_scroll), *_control_scroll_target), 250);
+       control_scroll_connection = Glib::signal_timeout().connect (sigc::bind (sigc::mem_fun (*this, &Editor::deferred_control_scroll), *_control_scroll_target), 250);
 }
 
 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;
@@ -918,11 +906,11 @@ Editor::deferred_control_scroll (nframes64_t /*target*/)
 void
 Editor::access_action (std::string action_group, std::string action_item)
 {
-       if (!session) {
+       if (!_session) {
                return;
        }
 
-       ENSURE_GUI_THREAD (bind (mem_fun (*this, &Editor::access_action), action_group, action_item));
+       ENSURE_GUI_THREAD (*this, &Editor::access_action, action_group, action_item)
 
        RefPtr<Action> act;
        act = ActionManager::get_action( action_group.c_str(), action_item.c_str() );
@@ -930,7 +918,7 @@ Editor::access_action (std::string action_group, std::string action_item)
        if (act) {
                act->activate();
        }
-               
+
 
 }
 
@@ -944,8 +932,8 @@ Editor::on_realize ()
 void
 Editor::start_scrolling ()
 {
-       scroll_connection = ARDOUR_UI::instance()->SuperRapidScreenUpdate.connect 
-               (mem_fun(*this, &Editor::update_current_screen));
+       scroll_connection = ARDOUR_UI::instance()->SuperRapidScreenUpdate.connect
+               (sigc::mem_fun(*this, &Editor::update_current_screen));
 
 }
 
@@ -958,15 +946,15 @@ Editor::stop_scrolling ()
 void
 Editor::map_position_change (nframes64_t frame)
 {
-       ENSURE_GUI_THREAD (bind (mem_fun(*this, &Editor::map_position_change), frame));
+       ENSURE_GUI_THREAD (*this, &Editor::map_position_change, frame)
 
-       if (session == 0 || !_follow_playhead) {
+       if (_session == 0 || !_follow_playhead) {
                return;
        }
 
        center_screen (frame);
        playhead_cursor->set_position (frame);
-}      
+}
 
 void
 Editor::center_screen (nframes64_t frame)
@@ -975,7 +963,7 @@ Editor::center_screen (nframes64_t frame)
 
        /* if we're off the page, then scroll.
         */
-       
+
        if (frame < leftmost_frame || frame >= leftmost_frame + page) {
                center_screen_internal (frame, page);
        }
@@ -985,7 +973,7 @@ void
 Editor::center_screen_internal (nframes64_t frame, float page)
 {
        page /= 2;
-               
+
        if (frame > page) {
                frame -= (nframes64_t) page;
        } else {
@@ -998,12 +986,12 @@ Editor::center_screen_internal (nframes64_t frame, float page)
 void
 Editor::handle_new_duration ()
 {
-       if (!session) {
+       if (!_session) {
                return;
        }
 
-       ENSURE_GUI_THREAD (mem_fun (*this, &Editor::handle_new_duration));
-       nframes64_t new_end = session->current_end_frame() + (nframes64_t) floorf (current_page_frames() * 0.10f);
+       ENSURE_GUI_THREAD (*this, &Editor::handle_new_duration)
+       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);
@@ -1014,28 +1002,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(bind (mem_fun(*this, &Editor::update_title_s), snap_name));
-       
-       update_title ();
-}
-
 void
 Editor::update_title ()
 {
-       ENSURE_GUI_THREAD (mem_fun(*this, &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) {
@@ -1049,9 +1029,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 ();
 
@@ -1060,115 +1067,96 @@ Editor::connect_to_session (Session *t)
        sensitize_the_right_region_actions (false);
 
        XMLNode* node = ARDOUR_UI::instance()->editor_settings();
-       set_state (*node);
+       set_state (*node, Stateful::loading_state_version);
 
        /* catch up with the playhead */
 
-       session->request_locate (playhead_cursor->current_frame);
+       _session->request_locate (playhead_cursor->current_frame);
 
        update_title ();
 
-       session->GoingAway.connect (mem_fun(*this, &Editor::session_going_away));
-       session->history().Changed.connect (mem_fun (*this, &Editor::history_changed));
+       _session->history().Changed.connect (_session_connections, boost::bind (&Editor::history_changed, this));
 
        /* 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 (mem_fun(*this, &Editor::map_transport_state)));
-       session_connections.push_back (session->PositionChanged.connect (mem_fun(*this, &Editor::map_position_change)));
-       session_connections.push_back (session->RouteAdded.connect (mem_fun(*this, &Editor::handle_new_route)));
-       session_connections.push_back (session->DurationChanged.connect (mem_fun(*this, &Editor::handle_new_duration)));
-       session_connections.push_back (session->NamedSelectionAdded.connect (mem_fun(*this, &Editor::handle_new_named_selection)));
-       session_connections.push_back (session->NamedSelectionRemoved.connect (mem_fun(*this, &Editor::handle_new_named_selection)));
-       session_connections.push_back (session->DirtyChanged.connect (mem_fun(*this, &Editor::update_title)));
-       session_connections.push_back (session->StateSaved.connect (mem_fun(*this, &Editor::update_title_s)));
-       session_connections.push_back (session->AskAboutPlaylistDeletion.connect (mem_fun(*this, &Editor::playlist_deletion_dialog)));
+       _session->TransportStateChange.connect (_session_connections, boost::bind (&Editor::map_transport_state, this));
+       _session->PositionChanged.connect (_session_connections, boost::bind (&Editor::map_position_change, this, _1));
+       _session->RouteAdded.connect (_session_connections, boost::bind (&Editor::handle_new_route, this, _1));
+       _session->DurationChanged.connect (_session_connections, boost::bind (&Editor::handle_new_duration, this));
+       _session->DirtyChanged.connect (_session_connections, boost::bind (&Editor::update_title, this));
+       _session->StateSaved.connect (_session_connections, boost::bind (&Editor::update_title, this));
+       _session->AskAboutPlaylistDeletion.connect (_session_connections, boost::bind (&Editor::playlist_deletion_dialog, this, _1));
+       _session->TimecodeOffsetChanged.connect (_session_connections, boost::bind (&Editor::update_just_timecode, this));
+       _session->tempo_map().StateChanged.connect (_session_connections, boost::bind (&Editor::tempo_map_changed, this, _1));
+       _session->Located.connect (_session_connections, boost::bind (&Editor::located, this));
+       _session->config.ParameterChanged.connect (_session_connections, boost::bind (&Editor::parameter_changed, this, _1));
 
-       session_connections.push_back (session->SMPTEOffsetChanged.connect (mem_fun(*this, &Editor::update_just_smpte)));
-
-       session_connections.push_back (session->tempo_map().StateChanged.connect (mem_fun(*this, &Editor::tempo_map_changed)));
-
-       session_connections.push_back (session->Located.connect (mem_fun (*this, &Editor::located)));
-       session_connections.push_back (session->config.ParameterChanged.connect (mem_fun (*this, &Editor::parameter_changed)));
-
-       edit_point_clock.set_mode(AudioClock::BBT);
-       edit_point_clock.set_session (session);
-       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::SMPTE); // 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 (mem_fun (*this, &Editor::parameter_changed));
-       session->config.map_parameters (mem_fun (*this, &Editor::parameter_changed));
-       
-       session->StateSaved.connect (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 (mem_fun(*this, &Editor::add_new_location));
-       session->locations()->removed.connect (mem_fun(*this, &Editor::location_gone));
-       session->locations()->changed.connect (mem_fun(*this, &Editor::refresh_location_display));
-       session->locations()->StateChanged.connect (mem_fun(*this, &Editor::refresh_location_display_s));
-       session->locations()->end_location()->changed.connect (mem_fun(*this, &Editor::end_location_changed));
 
-       if (sfbrowser) {
-               sfbrowser->set_session (session);
-       }
+       /* static signal - no need to drop connection when session is deleted (XXX or we are?)*/
 
-       handle_new_duration ();
+       _session->StateSaved.connect (*this, boost::bind (&Editor::session_state_saved, this, _1));
 
-       redisplay_named_selections ();
+       _session->locations()->added.connect (_session_connections, sigc::mem_fun(*this, &Editor::add_new_location));
+       _session->locations()->removed.connect (_session_connections, sigc::mem_fun(*this, &Editor::location_gone));
+       _session->locations()->changed.connect (_session_connections, sigc::mem_fun(*this, &Editor::refresh_location_display));
+       _session->locations()->StateChanged.connect (_session_connections, sigc::mem_fun(*this, &Editor::refresh_location_display_s));
+       _session->locations()->end_location()->changed.connect (_session_connections, sigc::mem_fun(*this, &Editor::end_location_changed));
+
+       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);
@@ -1176,7 +1164,7 @@ Editor::connect_to_session (Session *t)
 
        start_scrolling ();
 
-       switch (snap_type) {
+       switch (_snap_type) {
        case SnapToRegionStart:
        case SnapToRegionEnd:
        case SnapToRegionSync:
@@ -1189,15 +1177,8 @@ 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);
-       
+       _session->register_with_memento_command_factory(_id, this);
+
        start_updating ();
 }
 
@@ -1205,7 +1186,7 @@ void
 Editor::build_cursors ()
 {
        using namespace Gdk;
-       
+
        Gdk::Color mbg ("#000000" ); /* Black */
        Gdk::Color mfg ("#0000ff" ); /* Blue. */
 
@@ -1218,23 +1199,23 @@ Editor::build_cursors ()
 
        Gdk::Color fbg ("#ffffff" );
        Gdk::Color ffg  ("#000000" );
-       
+
        {
                RefPtr<Bitmap> source, mask;
-               
+
                source = Bitmap::create (fader_cursor_bits, fader_cursor_width, fader_cursor_height);
                mask = Bitmap::create (fader_cursor_mask_bits, fader_cursor_width, fader_cursor_height);
                fader_cursor = new Gdk::Cursor (source, mask, ffg, fbg, fader_cursor_x_hot, fader_cursor_y_hot);
        }
-       
-       { 
+
+       {
                RefPtr<Bitmap> source, mask;
                source = Bitmap::create (speaker_cursor_bits, speaker_cursor_width, speaker_cursor_height);
                mask = Bitmap::create (speaker_cursor_mask_bits, speaker_cursor_width, speaker_cursor_height);
                speaker_cursor = new Gdk::Cursor (source, mask, ffg, fbg, speaker_cursor_x_hot, speaker_cursor_y_hot);
        }
-       
-       { 
+
+       {
                RefPtr<Bitmap> bits;
                char pix[4] = { 0, 0, 0, 0 };
                bits = Bitmap::create (pix, 2, 2);
@@ -1242,17 +1223,17 @@ Editor::build_cursors ()
                transparent_cursor = new Gdk::Cursor (bits, bits, c, c, 0, 0);
        }
 
-       { 
+       {
                RefPtr<Bitmap> bits;
                char pix[4] = { 0, 0, 0, 0 };
                bits = Bitmap::create (pix, 2, 2);
                Gdk::Color c;
                transparent_cursor = new Gdk::Cursor (bits, bits, c, c, 0, 0);
        }
-       
+
 
        grabber_cursor = new Gdk::Cursor (HAND2);
-       
+
        {
                Glib::RefPtr<Gdk::Pixbuf> grabber_edit_point_pixbuf (::get_icon ("grabber_edit_point"));
                grabber_edit_point_cursor = new Gdk::Cursor (Gdk::Display::get_default(), grabber_edit_point_pixbuf, 5, 17);
@@ -1290,22 +1271,22 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
        case FadeInItem:
        case FadeInHandleItem:
                if (arv->audio_region()->fade_in_active()) {
-                       items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*this, &Editor::set_fade_in_active), false)));
+                       items.push_back (MenuElem (_("Deactivate"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_active), false)));
                } else {
-                       items.push_back (MenuElem (_("Activate"), bind (mem_fun (*this, &Editor::set_fade_in_active), true)));
+                       items.push_back (MenuElem (_("Activate"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_active), true)));
                }
-               
+
                items.push_back (SeparatorElem());
 
                if (Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
+                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
+                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
                } else {
-                       items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
-                       items.push_back (MenuElem (_("Slow"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogB)));
-                       items.push_back (MenuElem (_("Fast"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogA)));
-                       items.push_back (MenuElem (_("Fastest"), bind (mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Slow)));
+                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Linear)));
+                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Fast)));
+                       items.push_back (MenuElem (_("Slow"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogB)));
+                       items.push_back (MenuElem (_("Fast"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::LogA)));
+                       items.push_back (MenuElem (_("Fastest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_in_shape), AudioRegion::Slow)));
                }
 
                break;
@@ -1313,22 +1294,22 @@ Editor::popup_fade_context_menu (int button, int32_t time, ArdourCanvas::Item* i
        case FadeOutItem:
        case FadeOutHandleItem:
                if (arv->audio_region()->fade_out_active()) {
-                       items.push_back (MenuElem (_("Deactivate"), bind (mem_fun (*this, &Editor::set_fade_out_active), false)));
+                       items.push_back (MenuElem (_("Deactivate"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_active), false)));
                } else {
-                       items.push_back (MenuElem (_("Activate"), bind (mem_fun (*this, &Editor::set_fade_out_active), true)));
+                       items.push_back (MenuElem (_("Activate"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_active), true)));
                }
-               
+
                items.push_back (SeparatorElem());
-               
+
                if (Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
+                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
+                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
                } else {
-                       items.push_back (MenuElem (_("Linear"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
-                       items.push_back (MenuElem (_("Slowest"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
-                       items.push_back (MenuElem (_("Slow"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogA)));
-                       items.push_back (MenuElem (_("Fast"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogB)));
-                       items.push_back (MenuElem (_("Fastest"), bind (mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Fast)));
+                       items.push_back (MenuElem (_("Linear"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Linear)));
+                       items.push_back (MenuElem (_("Slowest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Slow)));
+                       items.push_back (MenuElem (_("Slow"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogA)));
+                       items.push_back (MenuElem (_("Fast"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::LogB)));
+                       items.push_back (MenuElem (_("Fastest"), sigc::bind (sigc::mem_fun (*this, &Editor::set_fade_out_shape), AudioRegion::Fast)));
                }
 
                break;
@@ -1388,7 +1369,7 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
 
        menu = (this->*build_menu_function)(frame);
        menu->set_name ("ArdourContextMenu");
-       
+
        /* now handle specific situations */
 
        switch (item_type) {
@@ -1433,23 +1414,23 @@ Editor::popup_track_context_menu (int button, int32_t time, ItemType item_type,
        if (item_type != SelectionItem && clicked_routeview && clicked_routeview->audio_track()) {
 
                /* Bounce to disk */
-               
+
                using namespace Menu_Helpers;
                MenuList& edit_items  = menu->items();
-               
+
                edit_items.push_back (SeparatorElem());
 
                switch (clicked_routeview->audio_track()->freeze_state()) {
                case AudioTrack::NoFreeze:
-                       edit_items.push_back (MenuElem (_("Freeze"), mem_fun(*this, &Editor::freeze_route)));
+                       edit_items.push_back (MenuElem (_("Freeze"), sigc::mem_fun(*this, &Editor::freeze_route)));
                        break;
 
                case AudioTrack::Frozen:
-                       edit_items.push_back (MenuElem (_("Unfreeze"), mem_fun(*this, &Editor::unfreeze_route)));
+                       edit_items.push_back (MenuElem (_("Unfreeze"), sigc::mem_fun(*this, &Editor::unfreeze_route)));
                        break;
-                       
+
                case AudioTrack::UnFrozen:
-                       edit_items.push_back (MenuElem (_("Freeze"), mem_fun(*this, &Editor::freeze_route)));
+                       edit_items.push_back (MenuElem (_("Freeze"), sigc::mem_fun(*this, &Editor::freeze_route)));
                        break;
                }
 
@@ -1498,16 +1479,16 @@ Editor::build_track_region_context_menu (nframes64_t frame)
        if (rtv) {
                boost::shared_ptr<Diskstream> ds;
                boost::shared_ptr<Playlist> pl;
-               
+
                if ((ds = rtv->get_diskstream()) && ((pl = ds->playlist()))) {
                        Playlist::RegionList* regions = pl->regions_at ((nframes64_t) floor ( (double)frame * ds->speed()));
 
                        if (selection->regions.size() > 1) {
-                               // there's already a multiple selection: just add a 
-                               // single region context menu that will act on all 
+                               // there's already a multiple selection: just add a
+                               // single region context menu that will act on all
                                // selected regions
-                               boost::shared_ptr<Region> dummy_region; // = NULL               
-                               add_region_context_items (rtv->view(), dummy_region, edit_items);                       
+                               boost::shared_ptr<Region> dummy_region; // = NULL
+                               add_region_context_items (rtv->view(), dummy_region, edit_items);
                        } else {
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
                                        add_region_context_items (rtv->view(), (*i), edit_items);
@@ -1551,11 +1532,11 @@ Editor::build_track_crossfade_context_menu (nframes64_t frame)
                        }
 
                        if (selection->regions.size() > 1) {
-                               // there's already a multiple selection: just add a 
-                               // single region context menu that will act on all 
+                               // there's already a multiple selection: just add a
+                               // single region context menu that will act on all
                                // selected regions
-                               boost::shared_ptr<Region> dummy_region; // = NULL               
-                               add_region_context_items (atv->audio_view(), dummy_region, edit_items);                 
+                               boost::shared_ptr<Region> dummy_region; // = NULL
+                               add_region_context_items (atv->audio_view(), dummy_region, edit_items);
                        } else {
                                for (Playlist::RegionList::iterator i = regions->begin(); i != regions->end(); ++i) {
                                        add_region_context_items (atv->audio_view(), (*i), edit_items);
@@ -1576,15 +1557,15 @@ 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();
        }
 
        analysis_window->set_regionmode();
        analysis_window->analyze();
-       
+
        analysis_window->present();
 }
 
@@ -1594,15 +1575,15 @@ 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();
        }
 
        analysis_window->set_rangemode();
        analysis_window->analyze();
-       
+
        analysis_window->present();
 }
 
@@ -1634,12 +1615,12 @@ Editor::add_crossfade_context_items (AudioStreamView* /*view*/, boost::shared_pt
 
        if (xfade->active()) {
                str = _("Mute");
-       } else { 
+       } else {
                str = _("Unmute");
        }
 
-       items.push_back (MenuElem (str, bind (mem_fun(*this, &Editor::toggle_xfade_active), boost::weak_ptr<Crossfade> (xfade))));
-       items.push_back (MenuElem (_("Edit"), bind (mem_fun(*this, &Editor::edit_xfade), boost::weak_ptr<Crossfade> (xfade))));
+       items.push_back (MenuElem (str, sigc::bind (sigc::mem_fun(*this, &Editor::toggle_xfade_active), boost::weak_ptr<Crossfade> (xfade))));
+       items.push_back (MenuElem (_("Edit"), sigc::bind (sigc::mem_fun(*this, &Editor::edit_xfade), boost::weak_ptr<Crossfade> (xfade))));
 
        if (xfade->can_follow_overlap()) {
 
@@ -1649,7 +1630,7 @@ Editor::add_crossfade_context_items (AudioStreamView* /*view*/, boost::shared_pt
                        str = _("Convert to full");
                }
 
-               items.push_back (MenuElem (str, bind (mem_fun(*this, &Editor::toggle_xfade_length), xfade)));
+               items.push_back (MenuElem (str, sigc::bind (sigc::mem_fun(*this, &Editor::toggle_xfade_length), xfade)));
        }
 
        if (many) {
@@ -1696,30 +1677,37 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                ar = boost::dynamic_pointer_cast<AudioRegion> (region);
                mr = boost::dynamic_pointer_cast<MidiRegion> (region);
 
-               /* when this particular menu pops up, make the relevant region 
+               /* when this particular menu pops up, make the relevant region
                   become selected.
                */
 
                region_menu->signal_map_event().connect (
-                       bind (mem_fun(*this, &Editor::set_selected_regionview_from_map_event), sv, boost::weak_ptr<Region>(region)));
-               
-               items.push_back (MenuElem (_("Rename"), mem_fun(*this, &Editor::rename_region)));
-               items.push_back (MenuElem (_("Popup region editor"), mem_fun(*this, &Editor::edit_region)));
+                       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)));
+               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 (_("Raise to top layer"), mem_fun(*this, &Editor::raise_region_to_top)));
-       items.push_back (MenuElem (_("Lower to bottom layer"), mem_fun  (*this, &Editor::lower_region_to_bottom)));
+       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"), mem_fun(*this, &Editor::set_region_sync_from_edit_point)));
-       items.push_back (MenuElem (_("Remove sync point"), mem_fun(*this, &Editor::remove_region_sync)));
+       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 (SeparatorElem());
 
-       items.push_back (MenuElem (_("Audition"), mem_fun(*this, &Editor::play_selected_region)));
-       items.push_back (MenuElem (_("Export"), mem_fun(*this, &Editor::export_region)));
-       items.push_back (MenuElem (_("Bounce"), mem_fun(*this, &Editor::bounce_region_selection)));
+       items.push_back (MenuElem (_("Audition"), sigc::mem_fun(*this, &Editor::play_selected_region)));
+       items.push_back (MenuElem (_("Export"), sigc::mem_fun(*this, &Editor::export_region)));
+       items.push_back (MenuElem (_("Bounce"), sigc::mem_fun(*this, &Editor::bounce_region_selection)));
 
        if (ar) {
-               items.push_back (MenuElem (_("Spectral Analysis"), mem_fun(*this, &Editor::analyze_region_selection)));
+               items.push_back (MenuElem (_("Spectral Analysis"), sigc::mem_fun(*this, &Editor::analyze_region_selection)));
        }
 
        items.push_back (SeparatorElem());
@@ -1738,11 +1726,11 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
        if (region_to_check->locked()) {
                region_lock_item->set_active();
        }
-       region_lock_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_region_lock));
-       
+       region_lock_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_lock));
+
        items.push_back (CheckMenuElem (_("Glue to Bars&Beats")));
        CheckMenuItem* bbt_glue_item = static_cast<CheckMenuItem*>(&items.back());
-       
+
        switch (region_to_check->positional_lock_style()) {
        case Region::MusicTime:
                bbt_glue_item->set_active (true);
@@ -1751,57 +1739,57 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                bbt_glue_item->set_active (false);
                break;
        }
-       
-       bbt_glue_item->signal_activate().connect (bind (mem_fun (*this, &Editor::set_region_lock_style), Region::MusicTime));
-       
+
+       bbt_glue_item->signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &Editor::set_region_lock_style), Region::MusicTime));
+
        items.push_back (CheckMenuElem (_("Mute")));
        CheckMenuItem* region_mute_item = static_cast<CheckMenuItem*>(&items.back());
-       fooc = region_mute_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_region_mute));
+       fooc = region_mute_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_mute));
        if (region_to_check->muted()) {
                fooc.block (true);
                region_mute_item->set_active();
                fooc.block (false);
        }
-       
+
        if (!Profile->get_sae()) {
                items.push_back (CheckMenuElem (_("Opaque")));
                CheckMenuItem* region_opaque_item = static_cast<CheckMenuItem*>(&items.back());
-               fooc = region_opaque_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_region_opaque));
+               fooc = region_opaque_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_region_opaque));
                if (region_to_check->opaque()) {
                        fooc.block (true);
                        region_opaque_item->set_active();
                        fooc.block (false);
                }
        }
-       
-       items.push_back (CheckMenuElem (_("Original position"), 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);
        }
-       
+
        items.push_back (SeparatorElem());
-       
+
        if (ar) {
-               
+
                RegionView* rv = sv->find_view (ar);
                AudioRegionView* arv = dynamic_cast<AudioRegionView*>(rv);
-               
+
                if (!Profile->get_sae()) {
-                       items.push_back (MenuElem (_("Reset Envelope"), mem_fun(*this, &Editor::reset_region_gain_envelopes)));
+                       items.push_back (MenuElem (_("Reset Envelope"), sigc::mem_fun(*this, &Editor::reset_region_gain_envelopes)));
 
                        items.push_back (CheckMenuElem (_("Envelope Visible")));
                        CheckMenuItem* region_envelope_visible_item = static_cast<CheckMenuItem*> (&items.back());
-                       fooc = region_envelope_visible_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_gain_envelope_visibility));
+                       fooc = region_envelope_visible_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_gain_envelope_visibility));
                        if (arv->envelope_visible()) {
                                fooc.block (true);
                                region_envelope_visible_item->set_active (true);
                                fooc.block (false);
                        }
-               
+
                        items.push_back (CheckMenuElem (_("Envelope Active")));
                        CheckMenuItem* region_envelope_active_item = static_cast<CheckMenuItem*> (&items.back());
-                       fooc = region_envelope_active_item->signal_activate().connect (mem_fun(*this, &Editor::toggle_gain_envelope_active));
-                       
+                       fooc = region_envelope_active_item->signal_activate().connect (sigc::mem_fun(*this, &Editor::toggle_gain_envelope_active));
+
                        if (ar->envelope_active()) {
                                fooc.block (true);
                                region_envelope_active_item->set_active (true);
@@ -1811,42 +1799,41 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                        items.push_back (SeparatorElem());
                }
 
-               if (ar->scale_amplitude() != 1.0f) {
-                       items.push_back (MenuElem (_("DeNormalize"), mem_fun(*this, &Editor::denormalize_region)));
-               } else {
-                       items.push_back (MenuElem (_("Normalize"), mem_fun(*this, &Editor::normalize_region)));
+               items.push_back (MenuElem (_("Normalize"), sigc::mem_fun(*this, &Editor::normalize_region)));
+               if (ar->scale_amplitude() != 1) {
+                       items.push_back (MenuElem (_("Reset Gain"), sigc::mem_fun(*this, &Editor::reset_region_scale_amplitude)));
                }
 
        } else if (mr) {
-               items.push_back (MenuElem (_("Quantize"), mem_fun(*this, &Editor::quantize_region)));
+               items.push_back (MenuElem (_("Quantize"), sigc::mem_fun(*this, &Editor::quantize_region)));
                items.push_back (SeparatorElem());
        }
 
-       items.push_back (MenuElem (_("Strip silence..."), mem_fun (*this, &Editor::strip_region_silence)));
-       items.push_back (MenuElem (_("Reverse"), mem_fun(*this, &Editor::reverse_region)));
+       items.push_back (MenuElem (_("Strip Silence..."), sigc::mem_fun (*this, &Editor::strip_region_silence)));
+       items.push_back (MenuElem (_("Reverse"), sigc::mem_fun(*this, &Editor::reverse_region)));
        items.push_back (SeparatorElem());
 
        /* range related stuff */
 
-       items.push_back (MenuElem (_("Add Single Range"), mem_fun (*this, &Editor::add_location_from_audio_region)));
-       items.push_back (MenuElem (_("Add Range Markers"), mem_fun (*this, &Editor::add_locations_from_audio_region)));
+       items.push_back (MenuElem (_("Add Single Range"), sigc::mem_fun (*this, &Editor::add_location_from_audio_region)));
+       items.push_back (MenuElem (_("Add Range Markers"), sigc::mem_fun (*this, &Editor::add_locations_from_audio_region)));
        if (selection->regions.size() < 2) {
                items.back().set_sensitive (false);
        }
 
-       items.push_back (MenuElem (_("Set Range Selection"), mem_fun (*this, &Editor::set_selection_from_region)));
+       items.push_back (MenuElem (_("Set Range Selection"), sigc::mem_fun (*this, &Editor::set_selection_from_region)));
        items.push_back (SeparatorElem());
-                        
+
        /* Nudge region */
 
        Menu *nudge_menu = manage (new Menu());
        MenuList& nudge_items = nudge_menu->items();
        nudge_menu->set_name ("ArdourContextMenu");
-       
-       nudge_items.push_back (MenuElem (_("Nudge fwd"), (bind (mem_fun(*this, &Editor::nudge_forward), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge bwd"), (bind (mem_fun(*this, &Editor::nudge_backward), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge fwd by capture offset"), (mem_fun(*this, &Editor::nudge_forward_capture_offset))));
-       nudge_items.push_back (MenuElem (_("Nudge bwd by capture offset"), (mem_fun(*this, &Editor::nudge_backward_capture_offset))));
+
+       nudge_items.push_back (MenuElem (_("Nudge fwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_forward), false, false))));
+       nudge_items.push_back (MenuElem (_("Nudge bwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_backward), false, false))));
+       nudge_items.push_back (MenuElem (_("Nudge fwd by capture offset"), (sigc::mem_fun(*this, &Editor::nudge_forward_capture_offset))));
+       nudge_items.push_back (MenuElem (_("Nudge bwd by capture offset"), (sigc::mem_fun(*this, &Editor::nudge_backward_capture_offset))));
 
        items.push_back (MenuElem (_("Nudge"), *nudge_menu));
        items.push_back (SeparatorElem());
@@ -1854,38 +1841,38 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
        Menu *trim_menu = manage (new Menu);
        MenuList& trim_items = trim_menu->items();
        trim_menu->set_name ("ArdourContextMenu");
-       
-       trim_items.push_back (MenuElem (_("Start to edit point"), mem_fun(*this, &Editor::trim_region_from_edit_point)));
+
+       trim_items.push_back (MenuElem (_("Start to edit point"), sigc::mem_fun(*this, &Editor::trim_region_from_edit_point)));
        foo_item = &trim_items.back();
        if (_edit_point == EditAtMouse) {
                foo_item->set_sensitive (false);
        }
-       trim_items.push_back (MenuElem (_("Edit point to end"), mem_fun(*this, &Editor::trim_region_to_edit_point)));
+       trim_items.push_back (MenuElem (_("Edit point to end"), sigc::mem_fun(*this, &Editor::trim_region_to_edit_point)));
        foo_item = &trim_items.back();
        if (_edit_point == EditAtMouse) {
                foo_item->set_sensitive (false);
        }
-       trim_items.push_back (MenuElem (_("Trim To Loop"), mem_fun(*this, &Editor::trim_region_to_loop)));
-       trim_items.push_back (MenuElem (_("Trim To Punch"), mem_fun(*this, &Editor::trim_region_to_punch)));
-                            
+       trim_items.push_back (MenuElem (_("Trim To Loop"), sigc::mem_fun(*this, &Editor::trim_region_to_loop)));
+       trim_items.push_back (MenuElem (_("Trim To Punch"), sigc::mem_fun(*this, &Editor::trim_region_to_punch)));
+
        items.push_back (MenuElem (_("Trim"), *trim_menu));
        items.push_back (SeparatorElem());
 
-       items.push_back (MenuElem (_("Split"), (mem_fun(*this, &Editor::split))));
+       items.push_back (MenuElem (_("Split"), (sigc::mem_fun(*this, &Editor::split))));
        region_edit_menu_split_item = &items.back();
-       
+
        if (_edit_point == EditAtMouse) {
                region_edit_menu_split_item->set_sensitive (false);
        }
 
-       items.push_back (MenuElem (_("Make mono regions"), (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"), (bind (mem_fun(*this, &Editor::duplicate_dialog), false))));
-       items.push_back (MenuElem (_("Multi-Duplicate"), (bind (mem_fun(*this, &Editor::duplicate_dialog), true))));
-       items.push_back (MenuElem (_("Fill Track"), (mem_fun(*this, &Editor::region_fill_track))));
+       items.push_back (MenuElem (_("Duplicate"), (sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), false))));
+       items.push_back (MenuElem (_("Multi-Duplicate"), (sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), true))));
+       items.push_back (MenuElem (_("Fill Track"), (sigc::mem_fun(*this, &Editor::region_fill_track))));
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Remove"), mem_fun(*this, &Editor::remove_selected_regions)));
+       items.push_back (MenuElem (_("Remove"), sigc::mem_fun(*this, &Editor::remove_selected_regions)));
 
        /* OK, stick the region submenu at the top of the list, and then add
           the standard items.
@@ -1902,7 +1889,7 @@ Editor::add_region_context_items (StreamView* sv, boost::shared_ptr<Region> regi
                menu_item_name.replace (pos, 1, "__");
                pos += 2;
        }
-       
+
        edit_items.push_back (MenuElem (menu_item_name, *region_menu));
        edit_items.push_back (SeparatorElem());
 }
@@ -1915,47 +1902,46 @@ Editor::add_selection_context_items (Menu_Helpers::MenuList& edit_items)
 {
        using namespace Menu_Helpers;
 
-       edit_items.push_back (MenuElem (_("Play Range"), mem_fun(*this, &Editor::play_selection)));
-       edit_items.push_back (MenuElem (_("Loop Range"), bind (mem_fun(*this, &Editor::set_loop_from_selection), true)));
+       edit_items.push_back (MenuElem (_("Play Range"), sigc::mem_fun(*this, &Editor::play_selection)));
+       edit_items.push_back (MenuElem (_("Loop Range"), sigc::bind (sigc::mem_fun(*this, &Editor::set_loop_from_selection), true)));
 
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Spectral Analysis"), mem_fun(*this, &Editor::analyze_range_selection)));
+       edit_items.push_back (MenuElem (_("Spectral Analysis"), sigc::mem_fun(*this, &Editor::analyze_range_selection)));
 
        if (!selection->regions.empty()) {
                edit_items.push_back (SeparatorElem());
-               edit_items.push_back (MenuElem (_("Extend Range to End of Region"), bind (mem_fun(*this, &Editor::extend_selection_to_end_of_region), false)));
-               edit_items.push_back (MenuElem (_("Extend Range to Start of Region"), bind (mem_fun(*this, &Editor::extend_selection_to_start_of_region), false)));
+               edit_items.push_back (MenuElem (_("Extend Range to End of Region"), sigc::bind (sigc::mem_fun(*this, &Editor::extend_selection_to_end_of_region), false)));
+               edit_items.push_back (MenuElem (_("Extend Range to Start of Region"), sigc::bind (sigc::mem_fun(*this, &Editor::extend_selection_to_start_of_region), false)));
        }
 
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Silence Range"), mem_fun(*this, &Editor::separate_region_from_selection)));
-       edit_items.push_back (MenuElem (_("Convert to Region in Region List"), mem_fun(*this, &Editor::new_region_from_selection)));
-       
+       edit_items.push_back (MenuElem (_("Silence Range"), sigc::mem_fun(*this, &Editor::separate_region_from_selection)));
+       edit_items.push_back (MenuElem (_("Convert to Region in Region List"), sigc::mem_fun(*this, &Editor::new_region_from_selection)));
+
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Select All in Range"), mem_fun(*this, &Editor::select_all_selectables_using_time_selection)));
+       edit_items.push_back (MenuElem (_("Select All in Range"), sigc::mem_fun(*this, &Editor::select_all_selectables_using_time_selection)));
 
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Set Loop from Range"), bind (mem_fun(*this, &Editor::set_loop_from_selection), false)));
-       edit_items.push_back (MenuElem (_("Set Punch from Range"), mem_fun(*this, &Editor::set_punch_from_selection)));
-       
+       edit_items.push_back (MenuElem (_("Set Loop from Range"), sigc::bind (sigc::mem_fun(*this, &Editor::set_loop_from_selection), false)));
+       edit_items.push_back (MenuElem (_("Set Punch from Range"), sigc::mem_fun(*this, &Editor::set_punch_from_selection)));
+
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Add Range Markers"), mem_fun (*this, &Editor::add_location_from_selection)));
-       
+       edit_items.push_back (MenuElem (_("Add Range Markers"), sigc::mem_fun (*this, &Editor::add_location_from_selection)));
+
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Crop Region to Range"), mem_fun(*this, &Editor::crop_region_to_selection)));
-       edit_items.push_back (MenuElem (_("Fill Range with Region"), mem_fun(*this, &Editor::region_fill_selection)));
-       edit_items.push_back (MenuElem (_("Duplicate Range"), bind (mem_fun(*this, &Editor::duplicate_dialog), false)));
-       edit_items.push_back (MenuElem (_("Create Chunk from Range"), mem_fun(*this, &Editor::create_named_selection)));
-       
+       edit_items.push_back (MenuElem (_("Crop Region to Range"), sigc::mem_fun(*this, &Editor::crop_region_to_selection)));
+       edit_items.push_back (MenuElem (_("Fill Range with Region"), sigc::mem_fun(*this, &Editor::region_fill_selection)));
+       edit_items.push_back (MenuElem (_("Duplicate Range"), sigc::bind (sigc::mem_fun(*this, &Editor::duplicate_dialog), false)));
+
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Consolidate Range"), bind (mem_fun(*this, &Editor::bounce_range_selection), true, false)));
-       edit_items.push_back (MenuElem (_("Consolidate Range With Processing"), bind (mem_fun(*this, &Editor::bounce_range_selection), true, true)));
-       edit_items.push_back (MenuElem (_("Bounce Range to Region List"), bind (mem_fun(*this, &Editor::bounce_range_selection), false, false)));
-       edit_items.push_back (MenuElem (_("Bounce Range to Region List With Processing"), bind (mem_fun(*this, &Editor::bounce_range_selection), false, true)));
-       edit_items.push_back (MenuElem (_("Export Range"), mem_fun(*this, &Editor::export_range)));
+       edit_items.push_back (MenuElem (_("Consolidate Range"), sigc::bind (sigc::mem_fun(*this, &Editor::bounce_range_selection), true, false)));
+       edit_items.push_back (MenuElem (_("Consolidate Range With Processing"), sigc::bind (sigc::mem_fun(*this, &Editor::bounce_range_selection), true, true)));
+       edit_items.push_back (MenuElem (_("Bounce Range to Region List"), sigc::bind (sigc::mem_fun(*this, &Editor::bounce_range_selection), false, false)));
+       edit_items.push_back (MenuElem (_("Bounce Range to Region List With Processing"), sigc::bind (sigc::mem_fun(*this, &Editor::bounce_range_selection), false, true)));
+       edit_items.push_back (MenuElem (_("Export Range"), sigc::mem_fun(*this, &Editor::export_range)));
 }
 
-       
+
 void
 Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
 {
@@ -1966,13 +1952,13 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        Menu *play_menu = manage (new Menu);
        MenuList& play_items = play_menu->items();
        play_menu->set_name ("ArdourContextMenu");
-       
-       play_items.push_back (MenuElem (_("Play from edit point"), mem_fun(*this, &Editor::play_from_edit_point)));
-       play_items.push_back (MenuElem (_("Play from start"), mem_fun(*this, &Editor::play_from_start)));
-       play_items.push_back (MenuElem (_("Play region"), mem_fun(*this, &Editor::play_selected_region)));
+
+       play_items.push_back (MenuElem (_("Play from edit point"), sigc::mem_fun(*this, &Editor::play_from_edit_point)));
+       play_items.push_back (MenuElem (_("Play from start"), sigc::mem_fun(*this, &Editor::play_from_start)));
+       play_items.push_back (MenuElem (_("Play region"), sigc::mem_fun(*this, &Editor::play_selected_region)));
        play_items.push_back (SeparatorElem());
-       play_items.push_back (MenuElem (_("Loop Region"), mem_fun(*this, &Editor::loop_selected_region)));
-       
+       play_items.push_back (MenuElem (_("Loop Region"), sigc::mem_fun(*this, &Editor::loop_selected_region)));
+
        edit_items.push_back (MenuElem (_("Play"), *play_menu));
 
        /* Selection */
@@ -1980,24 +1966,22 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        Menu *select_menu = manage (new Menu);
        MenuList& select_items = select_menu->items();
        select_menu->set_name ("ArdourContextMenu");
-       
-       select_items.push_back (MenuElem (_("Select All in track"), bind (mem_fun(*this, &Editor::select_all_in_track), Selection::Set)));
-       select_items.push_back (MenuElem (_("Select All"), bind (mem_fun(*this, &Editor::select_all), Selection::Set)));
-       select_items.push_back (MenuElem (_("Invert selection in track"), mem_fun(*this, &Editor::invert_selection_in_track)));
-       select_items.push_back (MenuElem (_("Invert selection"), mem_fun(*this, &Editor::invert_selection)));
-       select_items.push_back (SeparatorElem());
-       select_items.push_back (MenuElem (_("Set range to loop range"), mem_fun(*this, &Editor::set_selection_from_loop)));
-       select_items.push_back (MenuElem (_("Set range to punch range"), mem_fun(*this, &Editor::set_selection_from_punch)));
-       select_items.push_back (SeparatorElem());
-       select_items.push_back (MenuElem (_("Select All After Edit Point"), bind (mem_fun(*this, &Editor::select_all_selectables_using_edit), true)));
-       select_items.push_back (MenuElem (_("Select All Before Edit Point"), bind (mem_fun(*this, &Editor::select_all_selectables_using_edit), false)));
-       select_items.push_back (MenuElem (_("Select All After Playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, true)));
-       select_items.push_back (MenuElem (_("Select All Before Playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, false)));
-       select_items.push_back (MenuElem (_("Select All Between Playhead & Edit Point"), bind (mem_fun(*this, &Editor::select_all_selectables_between), false)));
-       select_items.push_back (MenuElem (_("Select All Within Playhead & Edit Point"), bind (mem_fun(*this, &Editor::select_all_selectables_between), true)));
-       select_items.push_back (MenuElem (_("Select Range Between Playhead & Edit Point"), mem_fun(*this, &Editor::select_range_between)));
 
+       select_items.push_back (MenuElem (_("Select All in track"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_in_track), Selection::Set)));
+       select_items.push_back (MenuElem (_("Select All"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all), Selection::Set)));
+       select_items.push_back (MenuElem (_("Invert selection in track"), sigc::mem_fun(*this, &Editor::invert_selection_in_track)));
+       select_items.push_back (MenuElem (_("Invert selection"), sigc::mem_fun(*this, &Editor::invert_selection)));
        select_items.push_back (SeparatorElem());
+       select_items.push_back (MenuElem (_("Set range to loop range"), sigc::mem_fun(*this, &Editor::set_selection_from_loop)));
+       select_items.push_back (MenuElem (_("Set range to punch range"), sigc::mem_fun(*this, &Editor::set_selection_from_punch)));
+       select_items.push_back (SeparatorElem());
+       select_items.push_back (MenuElem (_("Select All After Edit Point"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_using_edit), true)));
+       select_items.push_back (MenuElem (_("Select All Before Edit Point"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_using_edit), false)));
+       select_items.push_back (MenuElem (_("Select All After Playhead"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, true)));
+       select_items.push_back (MenuElem (_("Select All Before Playhead"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, false)));
+       select_items.push_back (MenuElem (_("Select All Between Playhead & Edit Point"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_between), false)));
+       select_items.push_back (MenuElem (_("Select All Within Playhead & Edit Point"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_between), true)));
+       select_items.push_back (MenuElem (_("Select Range Between Playhead & Edit Point"), sigc::mem_fun(*this, &Editor::select_range_between)));
 
        edit_items.push_back (MenuElem (_("Select"), *select_menu));
 
@@ -2006,39 +1990,37 @@ Editor::add_dstream_context_items (Menu_Helpers::MenuList& edit_items)
        Menu *cutnpaste_menu = manage (new Menu);
        MenuList& cutnpaste_items = cutnpaste_menu->items();
        cutnpaste_menu->set_name ("ArdourContextMenu");
-       
-       cutnpaste_items.push_back (MenuElem (_("Cut"), mem_fun(*this, &Editor::cut)));
-       cutnpaste_items.push_back (MenuElem (_("Copy"), mem_fun(*this, &Editor::copy)));
-       cutnpaste_items.push_back (MenuElem (_("Paste"), bind (mem_fun(*this, &Editor::paste), 1.0f)));
+
+       cutnpaste_items.push_back (MenuElem (_("Cut"), sigc::mem_fun(*this, &Editor::cut)));
+       cutnpaste_items.push_back (MenuElem (_("Copy"), sigc::mem_fun(*this, &Editor::copy)));
+       cutnpaste_items.push_back (MenuElem (_("Paste"), sigc::bind (sigc::mem_fun(*this, &Editor::paste), 1.0f)));
 
        cutnpaste_items.push_back (SeparatorElem());
 
-       cutnpaste_items.push_back (MenuElem (_("Align"), bind (mem_fun(*this, &Editor::align), ARDOUR::SyncPoint)));
-       cutnpaste_items.push_back (MenuElem (_("Align Relative"), bind (mem_fun(*this, &Editor::align_relative), ARDOUR::SyncPoint)));
+       cutnpaste_items.push_back (MenuElem (_("Align"), sigc::bind (sigc::mem_fun(*this, &Editor::align), ARDOUR::SyncPoint)));
+       cutnpaste_items.push_back (MenuElem (_("Align Relative"), sigc::bind (sigc::mem_fun(*this, &Editor::align_relative), ARDOUR::SyncPoint)));
 
        cutnpaste_items.push_back (SeparatorElem());
 
-       cutnpaste_items.push_back (MenuElem (_("Insert chunk"), bind (mem_fun(*this, &Editor::paste_named_selection), 1.0f)));
-
        edit_items.push_back (MenuElem (_("Edit"), *cutnpaste_menu));
 
        /* Adding new material */
-       
+
        edit_items.push_back (SeparatorElem());
-       edit_items.push_back (MenuElem (_("Insert Selected Region"), bind (mem_fun(*this, &Editor::insert_region_list_selection), 1.0f)));
-       edit_items.push_back (MenuElem (_("Insert Existing Media"), bind (mem_fun(*this, &Editor::add_external_audio_action), ImportToTrack)));
+       edit_items.push_back (MenuElem (_("Insert Selected Region"), sigc::bind (sigc::mem_fun(*this, &Editor::insert_region_list_selection), 1.0f)));
+       edit_items.push_back (MenuElem (_("Insert Existing Media"), sigc::bind (sigc::mem_fun(*this, &Editor::add_external_audio_action), ImportToTrack)));
 
        /* Nudge track */
 
        Menu *nudge_menu = manage (new Menu());
        MenuList& nudge_items = nudge_menu->items();
        nudge_menu->set_name ("ArdourContextMenu");
-       
+
        edit_items.push_back (SeparatorElem());
-       nudge_items.push_back (MenuElem (_("Nudge entire track fwd"), (bind (mem_fun(*this, &Editor::nudge_track), false, true))));
-       nudge_items.push_back (MenuElem (_("Nudge track after edit point fwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, true))));
-       nudge_items.push_back (MenuElem (_("Nudge entire track bwd"), (bind (mem_fun(*this, &Editor::nudge_track), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge track after edit point bwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, false))));
+       nudge_items.push_back (MenuElem (_("Nudge entire track fwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, true))));
+       nudge_items.push_back (MenuElem (_("Nudge track after edit point fwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, true))));
+       nudge_items.push_back (MenuElem (_("Nudge entire track bwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, false))));
+       nudge_items.push_back (MenuElem (_("Nudge track after edit point bwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, false))));
 
        edit_items.push_back (MenuElem (_("Nudge"), *nudge_menu));
 }
@@ -2053,9 +2035,9 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
        Menu *play_menu = manage (new Menu);
        MenuList& play_items = play_menu->items();
        play_menu->set_name ("ArdourContextMenu");
-       
-       play_items.push_back (MenuElem (_("Play from edit point"), mem_fun(*this, &Editor::play_from_edit_point)));
-       play_items.push_back (MenuElem (_("Play from start"), mem_fun(*this, &Editor::play_from_start)));
+
+       play_items.push_back (MenuElem (_("Play from edit point"), sigc::mem_fun(*this, &Editor::play_from_edit_point)));
+       play_items.push_back (MenuElem (_("Play from start"), sigc::mem_fun(*this, &Editor::play_from_start)));
        edit_items.push_back (MenuElem (_("Play"), *play_menu));
 
        /* Selection */
@@ -2063,16 +2045,16 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
        Menu *select_menu = manage (new Menu);
        MenuList& select_items = select_menu->items();
        select_menu->set_name ("ArdourContextMenu");
-       
-       select_items.push_back (MenuElem (_("Select All in track"), bind (mem_fun(*this, &Editor::select_all_in_track), Selection::Set)));
-       select_items.push_back (MenuElem (_("Select All"), bind (mem_fun(*this, &Editor::select_all), Selection::Set)));
-       select_items.push_back (MenuElem (_("Invert selection in track"), mem_fun(*this, &Editor::invert_selection_in_track)));
-       select_items.push_back (MenuElem (_("Invert selection"), mem_fun(*this, &Editor::invert_selection)));
+
+       select_items.push_back (MenuElem (_("Select All in track"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_in_track), Selection::Set)));
+       select_items.push_back (MenuElem (_("Select All"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all), Selection::Set)));
+       select_items.push_back (MenuElem (_("Invert selection in track"), sigc::mem_fun(*this, &Editor::invert_selection_in_track)));
+       select_items.push_back (MenuElem (_("Invert selection"), sigc::mem_fun(*this, &Editor::invert_selection)));
        select_items.push_back (SeparatorElem());
-       select_items.push_back (MenuElem (_("Select all after edit point"), bind (mem_fun(*this, &Editor::select_all_selectables_using_edit), true)));
-       select_items.push_back (MenuElem (_("Select all before edit point"), bind (mem_fun(*this, &Editor::select_all_selectables_using_edit), false)));
-       select_items.push_back (MenuElem (_("Select all after playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, true)));
-       select_items.push_back (MenuElem (_("Select all before playhead"), bind (mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, false)));
+       select_items.push_back (MenuElem (_("Select all after edit point"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_using_edit), true)));
+       select_items.push_back (MenuElem (_("Select all before edit point"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_using_edit), false)));
+       select_items.push_back (MenuElem (_("Select all after playhead"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, true)));
+       select_items.push_back (MenuElem (_("Select all before playhead"), sigc::bind (sigc::mem_fun(*this, &Editor::select_all_selectables_using_cursor), playhead_cursor, false)));
 
        edit_items.push_back (MenuElem (_("Select"), *select_menu));
 
@@ -2081,36 +2063,48 @@ Editor::add_bus_context_items (Menu_Helpers::MenuList& edit_items)
        Menu *cutnpaste_menu = manage (new Menu);
        MenuList& cutnpaste_items = cutnpaste_menu->items();
        cutnpaste_menu->set_name ("ArdourContextMenu");
-       
-       cutnpaste_items.push_back (MenuElem (_("Cut"), mem_fun(*this, &Editor::cut)));
-       cutnpaste_items.push_back (MenuElem (_("Copy"), mem_fun(*this, &Editor::copy)));
-       cutnpaste_items.push_back (MenuElem (_("Paste"), bind (mem_fun(*this, &Editor::paste), 1.0f)));
+
+       cutnpaste_items.push_back (MenuElem (_("Cut"), sigc::mem_fun(*this, &Editor::cut)));
+       cutnpaste_items.push_back (MenuElem (_("Copy"), sigc::mem_fun(*this, &Editor::copy)));
+       cutnpaste_items.push_back (MenuElem (_("Paste"), sigc::bind (sigc::mem_fun(*this, &Editor::paste), 1.0f)));
 
        Menu *nudge_menu = manage (new Menu());
        MenuList& nudge_items = nudge_menu->items();
        nudge_menu->set_name ("ArdourContextMenu");
-       
+
        edit_items.push_back (SeparatorElem());
-       nudge_items.push_back (MenuElem (_("Nudge entire track fwd"), (bind (mem_fun(*this, &Editor::nudge_track), false, true))));
-       nudge_items.push_back (MenuElem (_("Nudge track after edit point fwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, true))));
-       nudge_items.push_back (MenuElem (_("Nudge entire track bwd"), (bind (mem_fun(*this, &Editor::nudge_track), false, false))));
-       nudge_items.push_back (MenuElem (_("Nudge track after edit point bwd"), (bind (mem_fun(*this, &Editor::nudge_track), true, false))));
+       nudge_items.push_back (MenuElem (_("Nudge entire track fwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, true))));
+       nudge_items.push_back (MenuElem (_("Nudge track after edit point fwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, true))));
+       nudge_items.push_back (MenuElem (_("Nudge entire track bwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), false, false))));
+       nudge_items.push_back (MenuElem (_("Nudge track after edit point bwd"), (sigc::bind (sigc::mem_fun(*this, &Editor::nudge_track), true, false))));
 
        edit_items.push_back (MenuElem (_("Nudge"), *nudge_menu));
 }
 
+SnapType
+Editor::snap_type() const
+{
+       return _snap_type;
+}
+
+SnapMode
+Editor::snap_mode() const
+{
+       return _snap_mode;
+}
+
 void
 Editor::set_snap_to (SnapType st)
 {
        unsigned int snap_ind = (unsigned int)st;
 
-       snap_type = st;
-       
+       _snap_type = st;
+
        if (snap_ind > snap_type_strings.size() - 1) {
                snap_ind = 0;
-               snap_type = (SnapType)snap_ind;
+               _snap_type = (SnapType)snap_ind;
        }
-       
+
        string str = snap_type_strings[snap_ind];
 
        if (str != snap_type_selector.get_active_text()) {
@@ -2119,7 +2113,7 @@ Editor::set_snap_to (SnapType st)
 
        instant_save ();
 
-       switch (snap_type) {
+       switch (_snap_type) {
        case SnapToAThirtysecondBeat:
        case SnapToASixteenthBeat:
        case SnapToAEighthBeat:
@@ -2145,7 +2139,7 @@ Editor::set_snap_to (SnapType st)
 void
 Editor::set_snap_mode (SnapMode mode)
 {
-       snap_mode = mode;
+       _snap_mode = mode;
        string str = snap_mode_strings[(int)mode];
 
        if (str != snap_mode_selector.get_active_text ()) {
@@ -2172,26 +2166,6 @@ Editor::set_edit_point_preference (EditPoint ep, bool force)
                return;
        }
 
-       switch (zoom_focus) {
-       case ZoomFocusMouse:
-       case ZoomFocusPlayhead:
-       case ZoomFocusEdit:
-               switch (_edit_point) {
-               case EditAtMouse:
-                       set_zoom_focus (ZoomFocusMouse);
-                       break;
-               case EditAtPlayhead:
-                       set_zoom_focus (ZoomFocusPlayhead);
-                       break;
-               case EditAtSelectedMarker:
-                       set_zoom_focus (ZoomFocusEdit);
-                       break;
-               }
-               break;
-       default:
-               break;
-       }
-
        const char* action=NULL;
 
        switch (_edit_point) {
@@ -2224,7 +2198,7 @@ Editor::set_edit_point_preference (EditPoint ep, bool force)
 }
 
 int
-Editor::set_state (const XMLNode& node)
+Editor::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
        XMLNode* geometry;
@@ -2290,21 +2264,13 @@ Editor::set_state (const XMLNode& node)
        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 {
                playhead_cursor->set_position (0);
-
-               /* reset_x_origin() doesn't work right here, since the old
-                  position may be zero already, and it does nothing in such
-                  circumstances.
-               */
-               
-               leftmost_frame = 0;
-               horizontal_adjustment.set_value (0);
        }
-
+       
        if ((prop = node.property ("mixer-width"))) {
                editor_mixer_strip_width = Width (string_2_enum (prop->value(), editor_mixer_strip_width));
        }
@@ -2334,12 +2300,31 @@ Editor::set_state (const XMLNode& node)
                set_mouse_mode (MouseObject, true);
        }
 
+       if ((prop = node.property ("left-frame")) != 0){
+               nframes64_t pos;
+               if (sscanf (prop->value().c_str(), "%" PRId64, &pos) == 1) {
+                       reset_x_origin (pos);
+                       /* this hack prevents the initial call to update_current_screen() from doing re-centering on the playhead */
+                       last_update_frame = pos;
+               }
+       }
+
+       if ((prop = node.property ("internal-edit"))) {
+               bool yn = string_is_affirmative (prop->value());
+               RefPtr<Action> act = ActionManager::get_action (X_("MouseMode"), X_("toggle-internal-edit"));
+               if (act) {
+                       RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic(act);
+                       tact->set_active (!yn);
+                       tact->set_active (yn);
+               }
+       }
+
        if ((prop = node.property ("edit-point"))) {
                set_edit_point_preference ((EditPoint) string_2_enum (prop->value(), _edit_point), true);
        }
 
        if ((prop = node.property ("show-waveforms-recording"))) {
-               bool yn = (prop->value() == "yes");
+               bool yn = string_is_affirmative (prop->value());
                _show_waveforms_recording = !yn;
                RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("ToggleWaveformsWhileRecording"));
                if (act) {
@@ -2349,9 +2334,9 @@ Editor::set_state (const XMLNode& node)
                        tact->set_active (yn);
                }
        }
-       
+
        if ((prop = node.property ("show-measures"))) {
-               bool yn = (prop->value() == "yes");
+               bool yn = string_is_affirmative (prop->value());
                _show_measures = !yn;
                RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("ToggleMeasureVisibility"));
                if (act) {
@@ -2363,7 +2348,7 @@ Editor::set_state (const XMLNode& node)
        }
 
        if ((prop = node.property ("follow-playhead"))) {
-               bool yn = (prop->value() == "yes");
+               bool yn = string_is_affirmative (prop->value());
                set_follow_playhead (yn);
                RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("toggle-follow-playhead"));
                if (act) {
@@ -2379,7 +2364,7 @@ Editor::set_state (const XMLNode& node)
        }
 
        if ((prop = node.property ("xfades-visible"))) {
-               bool yn = (prop->value() == "yes");
+               bool yn = string_is_affirmative (prop->value());
                _xfade_visibility = !yn;
                // set_xfade_visibility (yn);
        }
@@ -2390,15 +2375,15 @@ Editor::set_state (const XMLNode& node)
                if (act) {
 
                        Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-                       bool yn = (prop->value() == X_("yes"));
+                       bool yn = string_is_affirmative (prop->value());
 
                        /* do it twice to force the change */
-                       
+
                        tact->set_active (!yn);
                        tact->set_active (yn);
                }
        }
-       
+
        if ((prop = node.property ("show-editor-list"))) {
 
                Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-editor-list"));
@@ -2406,10 +2391,10 @@ Editor::set_state (const XMLNode& node)
                if (act) {
 
                        Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-                       bool yn = (prop->value() == X_("yes"));
+                       bool yn = string_is_affirmative (prop->value());
 
                        /* do it twice to force the change */
-                       
+
                        tact->set_active (!yn);
                        tact->set_active (yn);
                }
@@ -2427,15 +2412,15 @@ Editor::get_state ()
 
        _id.print (buf, sizeof (buf));
        node->add_property ("id", buf);
-       
+
        if (is_realized()) {
                Glib::RefPtr<Gdk::Window> win = get_window();
-               
+
                int x, y, xoff, yoff, width, height;
                win->get_root_origin(x, y);
                win->get_position(xoff, yoff);
                win->get_size(width, height);
-               
+
                XMLNode* geometry = new XMLNode ("geometry");
 
                snprintf(buf, sizeof(buf), "%d", width);
@@ -2457,20 +2442,22 @@ Editor::get_state ()
        }
 
        maybe_add_mixer_strip_width (*node);
-       
+
        snprintf (buf, sizeof(buf), "%d", (int) zoom_focus);
        node->add_property ("zoom-focus", buf);
        snprintf (buf, sizeof(buf), "%f", frames_per_unit);
        node->add_property ("zoom", buf);
-       snprintf (buf, sizeof(buf), "%d", (int) snap_type);
+       snprintf (buf, sizeof(buf), "%d", (int) _snap_type);
        node->add_property ("snap-to", buf);
-       snprintf (buf, sizeof(buf), "%d", (int) snap_mode);
+       snprintf (buf, sizeof(buf), "%d", (int) _snap_mode);
        node->add_property ("snap-mode", buf);
 
        node->add_property ("edit-point", enum_2_string (_edit_point));
 
        snprintf (buf, sizeof (buf), "%" PRIi64, playhead_cursor->current_frame);
        node->add_property ("playhead", buf);
+       snprintf (buf, sizeof (buf), "%" PRIi64, leftmost_frame);
+       node->add_property ("left-frame", buf);
 
        node->add_property ("show-waveforms-recording", _show_waveforms_recording ? "yes" : "no");
        node->add_property ("show-measures", _show_measures ? "yes" : "no");
@@ -2478,13 +2465,14 @@ Editor::get_state ()
        node->add_property ("xfades-visible", _xfade_visibility ? "yes" : "no");
        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");
+
        Glib::RefPtr<Action> act = ActionManager::get_action (X_("Editor"), X_("show-editor-mixer"));
        if (act) {
                Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
                node->add_property (X_("show-editor-mixer"), tact->get_active() ? "yes" : "no");
        }
-       
+
        act = ActionManager::get_action (X_("Editor"), X_("show-editor-list"));
        if (act) {
                Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
@@ -2515,86 +2503,123 @@ Editor::trackview_by_y_position (double y)
        return std::make_pair ( (TimeAxisView *) 0, 0);
 }
 
+/** 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.
+ */
 void
-Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
+Editor::snap_to_with_modifier (nframes64_t& start, GdkEvent const * event, int32_t direction, bool for_mark)
 {
-       if (!session || snap_mode == SnapOff) {
+       if (!_session) {
                return;
        }
 
-       snap_to_internal (start, direction, for_mark);
+       if (Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
+               if (_snap_mode == SnapOff) {
+                       snap_to_internal (start, direction, for_mark);
+               }
+       } else {
+               if (_snap_mode != SnapOff) {
+                       snap_to_internal (start, direction, for_mark);
+               }
+       }
 }
 
 void
-Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
+Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
 {
-       Location* before = 0;
-       Location* after = 0;
+       if (!_session || _snap_mode == SnapOff) {
+               return;
+       }
 
-       const nframes64_t one_second = session->frame_rate();
-       const nframes64_t one_minute = session->frame_rate() * 60;
-       const nframes64_t one_smpte_second = (nframes64_t)(rint(session->smpte_frames_per_second()) * session->frames_per_smpte_frame());
-       nframes64_t one_smpte_minute = (nframes64_t)(rint(session->smpte_frames_per_second()) * session->frames_per_smpte_frame() * 60);
-       nframes64_t presnap = start;
+       snap_to_internal (start, direction, for_mark);
+}
 
-       switch (snap_type) {
-       case SnapToCDFrame:
-               if (((direction == 0) && (start % (one_second/75) > (one_second/75) / 2)) || (direction > 0)) {
-                       start = (nframes64_t) ceil ((double) start / (one_second / 75)) * (one_second / 75);
-               } else {
-                       start = (nframes64_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
-               }
-               break;
+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);
 
-       case SnapToSMPTEFrame:
-               if (((direction == 0) && (fmod((double)start, (double)session->frames_per_smpte_frame()) > (session->frames_per_smpte_frame() / 2))) || (direction > 0)) {
-                       start = (nframes64_t) (ceil ((double) start / session->frames_per_smpte_frame()) * session->frames_per_smpte_frame());
+       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());
                } else {
-                       start = (nframes64_t) (floor ((double) start / session->frames_per_smpte_frame()) *  session->frames_per_smpte_frame());
+                       start = (nframes64_t) (floor ((double) start / _session->frames_per_timecode_frame()) *  _session->frames_per_timecode_frame());
                }
                break;
 
-       case SnapToSMPTESeconds:
-               if (session->smpte_offset_negative())
+       case SnapToTimecodeSeconds:
+               if (_session->timecode_offset_negative())
                {
-                       start += session->smpte_offset ();
+                       start += _session->timecode_offset ();
                } else {
-                       start -= session->smpte_offset ();
-               }    
-               if (((direction == 0) && (start % one_smpte_second > one_smpte_second / 2)) || direction > 0) {
-                       start = (nframes64_t) ceil ((double) start / one_smpte_second) * one_smpte_second;
+                       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;
                } else {
-                       start = (nframes64_t) floor ((double) start / one_smpte_second) * one_smpte_second;
+                       start = (nframes64_t) floor ((double) start / one_timecode_second) * one_timecode_second;
                }
-               
-               if (session->smpte_offset_negative())
+
+               if (_session->timecode_offset_negative())
                {
-                       start -= session->smpte_offset ();
+                       start -= _session->timecode_offset ();
                } else {
-                       start += session->smpte_offset ();
+                       start += _session->timecode_offset ();
                }
                break;
-               
-       case SnapToSMPTEMinutes:
-               if (session->smpte_offset_negative())
+
+       case SnapToTimecodeMinutes:
+               if (_session->timecode_offset_negative())
                {
-                       start += session->smpte_offset ();
+                       start += _session->timecode_offset ();
                } else {
-                       start -= session->smpte_offset ();
+                       start -= _session->timecode_offset ();
                }
-               if (((direction == 0) && (start % one_smpte_minute > one_smpte_minute / 2)) || direction > 0) {
-                       start = (nframes64_t) ceil ((double) start / one_smpte_minute) * one_smpte_minute;
+               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_smpte_minute) * one_smpte_minute;
+                       start = (nframes64_t) floor ((double) start / one_timecode_minute) * one_timecode_minute;
                }
-               if (session->smpte_offset_negative())
+               if (_session->timecode_offset_negative())
                {
-                       start -= session->smpte_offset ();
+                       start -= _session->timecode_offset ();
                } else {
-                       start += session->smpte_offset ();
+                       start += _session->timecode_offset ();
                }
                break;
-               
+       default:
+               fatal << "Editor::smpte_snap_to_internal() called with non-timecode snap type!" << endmsg;
+               /*NOTREACHED*/
+       }
+}
+
+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;
+       nframes64_t presnap = start;
+       nframes64_t before;
+       nframes64_t after;
+
+       switch (_snap_type) {
+       case SnapToTimecodeFrame:
+       case SnapToTimecodeSeconds:
+       case SnapToTimecodeMinutes:
+               return timecode_snap_to_internal (start, direction, for_mark);
+
+       case SnapToCDFrame:
+               if (((direction == 0) && (start % (one_second/75) > (one_second/75) / 2)) || (direction > 0)) {
+                       start = (nframes64_t) ceil ((double) start / (one_second / 75)) * (one_second / 75);
+               } else {
+                       start = (nframes64_t) floor ((double) start / (one_second / 75)) * (one_second / 75);
+               }
+               break;
+
        case SnapToSeconds:
                if (((direction == 0) && (start % one_second > one_second / 2)) || (direction > 0)) {
                        start = (nframes64_t) ceil ((double) start / one_second) * one_second;
@@ -2602,7 +2627,7 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
                        start = (nframes64_t) floor ((double) start / one_second) * one_second;
                }
                break;
-               
+
        case SnapToMinutes:
                if (((direction == 0) && (start % one_minute > one_minute / 2)) || (direction > 0)) {
                        start = (nframes64_t) ceil ((double) start / one_minute) * one_minute;
@@ -2612,31 +2637,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);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 32, direction);
                break;
 
        case SnapToASixteenthBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 16);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 16, direction);
                break;
 
        case SnapToAEighthBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 8);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 8, direction);
                break;
 
        case SnapToAQuarterBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 4);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 4, direction);
                break;
 
        case SnapToAThirdBeat:
-               start = session->tempo_map().round_to_beat_subdivision (start, 3);
+               start = _session->tempo_map().round_to_beat_subdivision (start, 3, direction);
                break;
 
        case SnapToMark:
@@ -2644,39 +2669,21 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
                        return;
                }
 
-               before = session->locations()->first_location_before (start);
-               after = session->locations()->first_location_after (start);
+               _session->locations()->marks_either_side (start, before, after);
 
-               if (direction < 0) {
-                       if (before) {
-                               start = before->start();
+               if (before == max_frames) {
+                       start = after;
+               } else if (after == max_frames) {
+                       start = before;
+               } else if (before != max_frames && after != max_frames) {
+                       /* have before and after */
+                       if ((start - before) < (after - start)) {
+                               start = before;
                        } else {
-                               start = 0;
-                       }
-               } else if (direction > 0) {
-                       if (after) {
-                               start = after->start();
-                       } else {
-                               start = session->current_end_frame();
-                       }
-               } else {
-                       if (before) {
-                               if (after) {
-                                       /* find nearest of the two */
-                                       if ((start - before->start()) < (after->start() - start)) {
-                                               start = before->start();
-                                       } else {
-                                               start = after->start();
-                                       }
-                               } else {
-                                       start = before->start();
-                               }
-                       } else if (after) {
-                               start = after->start();
-                       } else {
-                               /* relax */
+                               start = after;
                        }
                }
+
                break;
 
        case SnapToRegionStart:
@@ -2687,13 +2694,13 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
 
                        vector<nframes64_t>::iterator prev = region_boundary_cache.end ();
                        vector<nframes64_t>::iterator next = region_boundary_cache.end ();
-                       
+
                        if (direction > 0) {
                                next = std::upper_bound (region_boundary_cache.begin(), region_boundary_cache.end(), start);
                        } else {
                                next = std::lower_bound (region_boundary_cache.begin(), region_boundary_cache.end(), start);
                        }
-                       
+
                        if (next != region_boundary_cache.begin ()) {
                                prev = next;
                                prev--;
@@ -2707,74 +2714,35 @@ Editor::snap_to_internal (nframes64_t& start, int32_t direction, bool for_mark)
                        } else {
                                start = p;
                        }
-               } 
+               }
                break;
        }
 
-       switch (snap_mode) {
+       switch (_snap_mode) {
        case SnapNormal:
-               return;                 
-               
+               return;
+
        case SnapMagnetic:
-               
+
                if (presnap > start) {
                        if (presnap > (start + unit_to_frame(snap_threshold))) {
                                start = presnap;
                        }
-                       
+
                } else if (presnap < start) {
                        if (presnap < (start - unit_to_frame(snap_threshold))) {
                                start = presnap;
                        }
                }
-               
+
        default:
                /* handled at entry */
                return;
-               
-       }
-}
-
-double
-Editor::snap_length_beats (nframes64_t start)
-{
-       if (!session) {
-               return 1.0;
-       }
-
-       /* FIXME: This could/should also work with non-tempo based snap settings (ie seconds) */
-
-       switch (snap_type) {
-       case SnapToBar:
-               return session->tempo_map().meter_at(start).beats_per_bar();
-
-       case SnapToBeat:
-               return 1.0;
-
-       case SnapToAThirtysecondBeat:
-               return 1.0 / (double)32.0;
-               break;
-
-       case SnapToASixteenthBeat:
-               return 1.0 / (double)16.0;
-               break;
-
-       case SnapToAEighthBeat:
-               return 1.0 / (double)8.0;
-               break;
-
-       case SnapToAQuarterBeat:
-               return 1.0 / (double)4.0;
-               break;
 
-       case SnapToAThirdBeat:
-               return 1.0 / (double)3.0;
-
-       default:
-               return 1.0;
        }
 }
 
+
 void
 Editor::setup_toolbar ()
 {
@@ -2816,25 +2784,26 @@ Editor::setup_toolbar ()
 
        edit_mode_selector.set_name ("EditModeSelector");
        set_popdown_strings (edit_mode_selector, edit_mode_strings, true);
-       edit_mode_selector.signal_changed().connect (mem_fun(*this, &Editor::edit_mode_selection_done));
+       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);
-       
+
        mouse_mode_tearoff = manage (new TearOff (*mode_box));
        mouse_mode_tearoff->set_name ("MouseModeBase");
+       mouse_mode_tearoff->tearoff_window().signal_key_press_event().connect (sigc::bind (sigc::ptr_fun (relay_key_press), &mouse_mode_tearoff->tearoff_window()), false);
 
        if (Profile->get_sae()) {
                mouse_mode_tearoff->set_can_be_torn_off (false);
        }
 
-       mouse_mode_tearoff->Detach.connect (bind (mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox), 
+       mouse_mode_tearoff->Detach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
                                                  &mouse_mode_tearoff->tearoff_window()));
-       mouse_mode_tearoff->Attach.connect (bind (mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox), 
+       mouse_mode_tearoff->Attach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
                                                  &mouse_mode_tearoff->tearoff_window(), 1));
-       mouse_mode_tearoff->Hidden.connect (bind (mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox), 
+       mouse_mode_tearoff->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
                                                  &mouse_mode_tearoff->tearoff_window()));
-       mouse_mode_tearoff->Visible.connect (bind (mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox), 
+       mouse_mode_tearoff->Visible.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
                                                   &mouse_mode_tearoff->tearoff_window(), 1));
 
        mouse_move_button.set_mode (false);
@@ -2862,61 +2831,77 @@ Editor::setup_toolbar ()
        internal_edit_button.unset_flags (CAN_FOCUS);
 
        /* Zoom */
-       
+
        zoom_box.set_spacing (1);
        zoom_box.set_border_width (0);
 
        zoom_in_button.set_name ("EditorTimeButton");
-       zoom_in_button.set_size_request(-1,16);
-       zoom_in_button.add (*(manage (new Image (::get_icon("zoom_in")))));
-       zoom_in_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::temporal_zoom_step), false));
+       zoom_in_button.set_image (*(manage (new Image (Stock::ZOOM_IN, Gtk::ICON_SIZE_BUTTON))));
+       zoom_in_button.signal_clicked().connect (sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_step), false));
        ARDOUR_UI::instance()->tooltips().set_tip (zoom_in_button, _("Zoom In"));
-       
+
        zoom_out_button.set_name ("EditorTimeButton");
-       zoom_out_button.set_size_request(-1,16);
-       zoom_out_button.add (*(manage (new Image (::get_icon("zoom_out")))));
-       zoom_out_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::temporal_zoom_step), true));
+       zoom_out_button.set_image (*(manage (new Image (Stock::ZOOM_OUT, Gtk::ICON_SIZE_BUTTON))));
+       zoom_out_button.signal_clicked().connect (sigc::bind (sigc::mem_fun(*this, &Editor::temporal_zoom_step), true));
        ARDOUR_UI::instance()->tooltips().set_tip (zoom_out_button, _("Zoom Out"));
 
        zoom_out_full_button.set_name ("EditorTimeButton");
-       zoom_out_full_button.set_size_request(-1,16);
-       zoom_out_full_button.add (*(manage (new Image (::get_icon("zoom_full")))));
-       zoom_out_full_button.signal_clicked().connect (mem_fun(*this, &Editor::temporal_zoom_session));
+       zoom_out_full_button.set_image (*(manage (new Image (Stock::ZOOM_100, Gtk::ICON_SIZE_BUTTON))));
+       zoom_out_full_button.signal_clicked().connect (sigc::mem_fun(*this, &Editor::temporal_zoom_session));
        ARDOUR_UI::instance()->tooltips().set_tip (zoom_out_full_button, _("Zoom to Session"));
 
        zoom_focus_selector.set_name ("ZoomFocusSelector");
        set_popdown_strings (zoom_focus_selector, zoom_focus_strings, true);
-       zoom_focus_selector.signal_changed().connect (mem_fun(*this, &Editor::zoom_focus_selection_done));
+       zoom_focus_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::zoom_focus_selection_done));
        ARDOUR_UI::instance()->tooltips().set_tip (zoom_focus_selector, _("Zoom focus"));
 
        zoom_box.pack_start (zoom_out_button, false, false);
        zoom_box.pack_start (zoom_in_button, false, false);
        zoom_box.pack_start (zoom_out_full_button, false, false);
 
+       /* Track zoom buttons */
+       tav_expand_button.set_name ("TrackHeightButton");
+       tav_expand_button.set_size_request(-1,20);
+       tav_expand_button.add (*(manage (new Image (::get_icon("tav_exp")))));
+       tav_expand_button.signal_clicked().connect (sigc::bind (sigc::mem_fun(*this, &Editor::tav_zoom_step), true));
+       ARDOUR_UI::instance()->tooltips().set_tip (tav_expand_button, _("Expand Tracks"));
+
+       tav_shrink_button.set_name ("TrackHeightButton");
+       tav_shrink_button.set_size_request(-1,20);
+       tav_shrink_button.add (*(manage (new Image (::get_icon("tav_shrink")))));
+       tav_shrink_button.signal_clicked().connect (sigc::bind (sigc::mem_fun(*this, &Editor::tav_zoom_step), false));
+       ARDOUR_UI::instance()->tooltips().set_tip (tav_shrink_button, _("Shrink Tracks"));
+
+       track_zoom_box.set_spacing (1);
+       track_zoom_box.set_border_width (0);
+
+       track_zoom_box.pack_start (tav_shrink_button, false, false);
+       track_zoom_box.pack_start (tav_expand_button, false, false);
+
        HBox* zbc = manage (new HBox);
        zbc->pack_start (zoom_focus_selector, PACK_SHRINK);
        zoom_vbox.pack_start (*zbc, PACK_SHRINK);
        zoom_vbox.pack_start (zoom_box, PACK_SHRINK);
+       zoom_vbox.pack_start (track_zoom_box, PACK_SHRINK);
 
        snap_box.set_spacing (1);
        snap_box.set_border_width (2);
 
        snap_type_selector.set_name ("SnapTypeSelector");
        set_popdown_strings (snap_type_selector, snap_type_strings, true);
-       snap_type_selector.signal_changed().connect (mem_fun(*this, &Editor::snap_type_selection_done));
+       snap_type_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::snap_type_selection_done));
        ARDOUR_UI::instance()->tooltips().set_tip (snap_type_selector, _("Snap/Grid Units"));
 
        snap_mode_selector.set_name ("SnapModeSelector");
        set_popdown_strings (snap_mode_selector, snap_mode_strings, true);
-       snap_mode_selector.signal_changed().connect (mem_fun(*this, &Editor::snap_mode_selection_done));
+       snap_mode_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::snap_mode_selection_done));
        ARDOUR_UI::instance()->tooltips().set_tip (snap_mode_selector, _("Snap/Grid Mode"));
 
        edit_point_selector.set_name ("EditPointSelector");
        set_popdown_strings (edit_point_selector, edit_point_strings, true);
-       edit_point_selector.signal_changed().connect (mem_fun(*this, &Editor::edit_point_selection_done));
+       edit_point_selector.signal_changed().connect (sigc::mem_fun(*this, &Editor::edit_point_selection_done));
        ARDOUR_UI::instance()->tooltips().set_tip (edit_point_selector, _("Edit point"));
 
-       snap_box.pack_start (edit_point_clock, false, false);
        snap_box.pack_start (snap_mode_selector, false, false);
        snap_box.pack_start (snap_type_selector, false, false);
        snap_box.pack_start (edit_point_selector, false, false);
@@ -2927,8 +2912,8 @@ Editor::setup_toolbar ()
        nudge_box->set_spacing(1);
        nudge_box->set_border_width (2);
 
-       nudge_forward_button.signal_button_release_event().connect (mem_fun(*this, &Editor::nudge_forward_release), false);
-       nudge_backward_button.signal_button_release_event().connect (mem_fun(*this, &Editor::nudge_backward_release), false);
+       nudge_forward_button.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::nudge_forward_release), false);
+       nudge_backward_button.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::nudge_backward_release), false);
 
        nudge_box->pack_start (nudge_backward_button, false, false);
        nudge_box->pack_start (nudge_forward_button, false, false);
@@ -2942,18 +2927,19 @@ Editor::setup_toolbar ()
 
        tools_tearoff = manage (new TearOff (*hbox));
        tools_tearoff->set_name ("MouseModeBase");
+       tools_tearoff->tearoff_window().signal_key_press_event().connect (sigc::bind (sigc::ptr_fun (relay_key_press), &tools_tearoff->tearoff_window()), false);
 
        if (Profile->get_sae()) {
                tools_tearoff->set_can_be_torn_off (false);
        }
 
-       tools_tearoff->Detach.connect (bind (mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox), 
+       tools_tearoff->Detach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
                                             &tools_tearoff->tearoff_window()));
-       tools_tearoff->Attach.connect (bind (mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox), 
+       tools_tearoff->Attach.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
                                             &tools_tearoff->tearoff_window(), 0));
-       tools_tearoff->Hidden.connect (bind (mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox), 
+       tools_tearoff->Hidden.connect (sigc::bind (sigc::mem_fun(*this, &Editor::detach_tearoff), static_cast<Box*>(&toolbar_hbox),
                                             &tools_tearoff->tearoff_window()));
-       tools_tearoff->Visible.connect (bind (mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox), 
+       tools_tearoff->Visible.connect (sigc::bind (sigc::mem_fun(*this, &Editor::reattach_tearoff), static_cast<Box*> (&toolbar_hbox),
                                              &tools_tearoff->tearoff_window(), 0));
 
        toolbar_hbox.set_spacing (10);
@@ -2961,13 +2947,13 @@ Editor::setup_toolbar ()
 
        toolbar_hbox.pack_start (*mouse_mode_tearoff, false, false);
        toolbar_hbox.pack_start (*tools_tearoff, false, false);
-       
+
        hbox->pack_start (snap_box, false, false);
        hbox->pack_start (*nudge_box, false, false);
        hbox->pack_start (panic_box, false, false);
 
        hbox->show_all ();
-       
+
        toolbar_base.set_name ("ToolBarBase");
        toolbar_base.add (toolbar_hbox);
 
@@ -2977,46 +2963,32 @@ Editor::setup_toolbar ()
 }
 
 void
-Editor::midi_panic_button_pressed ()
+Editor::midi_panic ()
 {
-       if (session) {
-               session->midi_panic();
+       cerr << "MIDI panic\n";
+
+       if (_session) {
+               _session->midi_panic();
        }
 }
 
 void
 Editor::setup_midi_toolbar ()
 {
-#if 0
-       midi_tool_pencil_button.add (*(manage (new Image (::get_icon("midi_tool_pencil")))));
-       midi_tool_pencil_button.set_relief(Gtk::RELIEF_NONE);
-       midi_tool_buttons.push_back (&midi_tool_pencil_button);
-       midi_tool_select_button.add (*(manage (new Image (::get_icon("midi_tool_select")))));
-       midi_tool_select_button.set_relief(Gtk::RELIEF_NONE);
-       midi_tool_buttons.push_back (&midi_tool_select_button);
-       midi_tool_resize_button.add (*(manage (new Image (::get_icon("strip_width")))));
-       midi_tool_resize_button.set_relief(Gtk::RELIEF_NONE);
-       midi_tool_buttons.push_back (&midi_tool_resize_button);
-       midi_tool_erase_button.add (*(manage (new Image (::get_icon("midi_tool_erase")))));
-       midi_tool_erase_button.set_relief(Gtk::RELIEF_NONE);
-       midi_tool_buttons.push_back (&midi_tool_erase_button);
-
-       ARDOUR_UI::instance()->tooltips().set_tip (midi_tool_pencil_button, _("Add/Move/Stretch Notes"));
-       ARDOUR_UI::instance()->tooltips().set_tip (midi_tool_select_button, _("Select/Move Notes"));
-       ARDOUR_UI::instance()->tooltips().set_tip (midi_tool_resize_button, _("Resize Notes"));
-       ARDOUR_UI::instance()->tooltips().set_tip (midi_tool_erase_button,  _("Erase Notes"));
-#endif
-       
+       RefPtr<Action> act;
+
        /* Midi sound notes */
        midi_sound_notes.add (*(manage (new Image (::get_icon("midi_sound_notes")))));
        midi_sound_notes.set_relief(Gtk::RELIEF_NONE);
        ARDOUR_UI::instance()->tooltips().set_tip (midi_sound_notes, _("Sound Notes"));
        midi_sound_notes.unset_flags (CAN_FOCUS);
-       
+
        /* Panic */
-       
+
+       act = ActionManager::get_action (X_("MIDI"), X_("panic"));
        midi_panic_button.set_name("MidiPanicButton");
-       midi_panic_button.signal_pressed().connect (mem_fun(this, &Editor::midi_panic_button_pressed));
+       ARDOUR_UI::instance()->tooltips().set_tip (midi_panic_button, _("Send note off and reset controller messages on all MIDI channels"));
+       act->connect_proxy (midi_panic_button);
 
        panic_box.pack_start (midi_sound_notes , true, true);
        panic_box.pack_start (midi_panic_button, true, true);
@@ -3024,15 +2996,15 @@ Editor::setup_midi_toolbar ()
 
 int
 Editor::convert_drop_to_paths (
-               vector<ustring>&                paths, 
+               vector<ustring>&                paths,
                const RefPtr<Gdk::DragContext>& /*context*/,
                gint                            /*x*/,
                gint                            /*y*/,
                const SelectionData&            data,
                guint                           /*info*/,
                guint                           /*time*/)
-{      
-       if (session == 0) {
+{
+       if (_session == 0) {
                return -1;
        }
 
@@ -3047,8 +3019,8 @@ Editor::convert_drop_to_paths (
                if (data.get_target() != "text/plain") {
                        return -1;
                }
-  
-               /* Parse the "uri-list" format that Nautilus provides, 
+
+               /* Parse the "uri-list" format that Nautilus provides,
                   where each pathname is delimited by \r\n.
 
                   THERE MAY BE NO NULL TERMINATING CHAR!!!
@@ -3068,18 +3040,18 @@ Editor::convert_drop_to_paths (
                        {
                                while (g_ascii_isspace (*p))
                                        p++;
-                               
+
                                q = p;
                                while (*q && (*q != '\n') && (*q != '\r')) {
                                        q++;
                                }
-                               
+
                                if (q > p)
                                {
                                        q--;
                                        while (q > p && g_ascii_isspace (*q))
                                                q--;
-                                       
+
                                        if (q > p)
                                        {
                                                uris.push_back (ustring (p, q - p + 1));
@@ -3092,21 +3064,21 @@ Editor::convert_drop_to_paths (
                }
 
                free ((void*)p);
-               
+
                if (uris.empty()) {
                        return -1;
                }
        }
-       
+
        for (vector<ustring>::iterator i = uris.begin(); i != uris.end(); ++i) {
 
                if ((*i).substr (0,7) == "file://") {
-                       
+
                        ustring p = *i;
                        PBD::url_decode (p);
 
                        // scan forward past three slashes
-                       
+
                        ustring::size_type slashcnt = 0;
                        ustring::size_type n = 0;
                        ustring::iterator x = p.begin();
@@ -3142,9 +3114,9 @@ Editor::new_tempo_section ()
 void
 Editor::map_transport_state ()
 {
-       ENSURE_GUI_THREAD (mem_fun(*this, &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;
        }
 
@@ -3163,40 +3135,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()));
        }
 }
 
@@ -3223,26 +3175,26 @@ Editor::set_route_group_mute (Route& route, bool yn)
                route.set_mute (yn, this);
        }
 }
-               
+
 void
 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;
        }
@@ -3261,7 +3213,7 @@ Editor::duplicate_dialog (bool with_dialog)
 
        RegionSelection rs;
        get_regions_for_action (rs);
-       
+
        if (mouse_mode != MouseRange) {
 
                if (rs.empty()) {
@@ -3271,44 +3223,44 @@ Editor::duplicate_dialog (bool with_dialog)
 
        if (with_dialog) {
 
-               ArdourDialog win ("Duplication Dialog");
+               ArdourDialog win ("Duplicate");
                Label  label (_("Number of Duplications:"));
                Adjustment adjustment (1.0, 1.0, 1000000.0, 1.0, 5.0);
                SpinButton spinner (adjustment, 0.0, 1);
                HBox hbox;
-               
+
                win.get_vbox()->set_spacing (12);
                win.get_vbox()->pack_start (hbox);
                hbox.set_border_width (6);
                hbox.pack_start (label, PACK_EXPAND_PADDING, 12);
-               
+
                /* dialogs have ::add_action_widget() but that puts the spinner in the wrong
                   place, visually. so do this by hand.
                */
-               
+
                hbox.pack_start (spinner, PACK_EXPAND_PADDING, 12);
-               spinner.signal_activate().connect (sigc::bind (mem_fun (win, &ArdourDialog::response), RESPONSE_ACCEPT));
+               spinner.signal_activate().connect (sigc::bind (sigc::mem_fun (win, &ArdourDialog::response), RESPONSE_ACCEPT));
                spinner.grab_focus();
 
                hbox.show ();
                label.show ();
                spinner.show ();
-               
+
                win.add_button (Stock::CANCEL, RESPONSE_CANCEL);
                win.add_button (_("Duplicate"), RESPONSE_ACCEPT);
                win.set_default_response (RESPONSE_ACCEPT);
-               
+
                win.set_position (WIN_POS_MOUSE);
-               
+
                spinner.grab_focus ();
-               
+
                switch (win.run ()) {
                case RESPONSE_ACCEPT:
                        break;
                default:
                        return;
                }
-               
+
                times = adjustment.get_value();
        }
 
@@ -3356,6 +3308,24 @@ Editor::clamp_verbose_cursor_y (double y)
        return y;
 }
 
+void
+Editor::show_verbose_canvas_cursor_with (const string & txt)
+{
+       verbose_canvas_cursor->property_text() = txt.c_str();
+
+       int x, y;
+       double wx, wy;
+
+       track_canvas->get_pointer (x, y);
+       track_canvas->window_to_world (x, y, wx, wy);
+
+       /* don't get too close to the edge */
+       verbose_canvas_cursor->property_x() = clamp_verbose_cursor_x (wx);
+       verbose_canvas_cursor->property_y() = clamp_verbose_cursor_y (wy);
+
+       show_verbose_canvas_cursor ();
+}
+
 void
 Editor::set_verbose_canvas_cursor (const string & txt, double x, double y)
 {
@@ -3400,7 +3370,7 @@ Editor::cycle_edit_mode ()
 void
 Editor::edit_mode_selection_done ()
 {
-       if (session == 0) {
+       if (_session == 0) {
                return;
        }
 
@@ -3416,7 +3386,7 @@ Editor::edit_mode_selection_done ()
        }
 
        Config->set_edit_mode (mode);
-}      
+}
 
 void
 Editor::snap_type_selection_done ()
@@ -3450,12 +3420,12 @@ Editor::snap_type_selection_done ()
                snaptype = SnapToRegionSync;
        } else if (choice == _("CD Frames")) {
                snaptype = SnapToCDFrame;
-       } else if (choice == _("SMPTE Frames")) {
-               snaptype = SnapToSMPTEFrame;
-       } else if (choice == _("SMPTE Seconds")) {
-               snaptype = SnapToSMPTESeconds;
-       } else if (choice == _("SMPTE Minutes")) {
-               snaptype = SnapToSMPTEMinutes;
+       } else if (choice == _("Timecode Frames")) {
+               snaptype = SnapToTimecodeFrame;
+       } else if (choice == _("Timecode Seconds")) {
+               snaptype = SnapToTimecodeSeconds;
+       } else if (choice == _("Timecode Minutes")) {
+               snaptype = SnapToTimecodeMinutes;
        } else if (choice == _("Seconds")) {
                snaptype = SnapToSeconds;
        } else if (choice == _("Minutes")) {
@@ -3466,7 +3436,7 @@ Editor::snap_type_selection_done ()
        if (ract) {
                ract->set_active ();
        }
-}      
+}
 
 void
 Editor::snap_mode_selection_done ()
@@ -3542,24 +3512,20 @@ Editor::zoom_focus_selection_done ()
                focus_type = ZoomFocusRight;
        } else if (choice == _("Center")) {
                focus_type = ZoomFocusCenter;
-       } else if (choice == _("Play")) {
+       } else if (choice == _("Playhead")) {
                focus_type = ZoomFocusPlayhead;
-       } else if (choice == _("Edit")) {
-               focus_type = ZoomFocusEdit;
-       } else if (choice == _("Active Mark")) {
-               focus_type = ZoomFocusEdit;
-       } else if (choice == _("Active Mark")) {
-               focus_type = ZoomFocusEdit;
-       } else {
+       } else if (choice == _("Mouse")) {
                focus_type = ZoomFocusMouse;
-       } 
-       
+       } else if (choice == _("Edit point")) {
+               focus_type = ZoomFocusEdit;
+       }
+
        RefPtr<RadioAction> ract = zoom_focus_action (focus_type);
 
        if (ract) {
                ract->set_active ();
        }
-}      
+}
 
 gint
 Editor::edit_controls_button_release (GdkEventButton* ev)
@@ -3582,43 +3548,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)
 {
@@ -3627,7 +3556,7 @@ Editor::set_zoom_focus (ZoomFocus f)
        if (str != zoom_focus_selector.get_active_text()) {
                zoom_focus_selector.set_active_text (str);
        }
-       
+
        if (zoom_focus != f) {
                zoom_focus = f;
 
@@ -3643,7 +3572,7 @@ Editor::ensure_float (Window& win)
        win.set_transient_for (*this);
 }
 
-void 
+void
 Editor::pane_allocation_handler (Allocation &alloc, Paned* which)
 {
        /* recover or initialize pane positions. do this here rather than earlier because
@@ -3703,7 +3632,7 @@ Editor::detach_tearoff (Box* /*b*/, Window* /*w*/)
 {
        cerr << "remove tearoff\n";
 
-       if (tools_tearoff->torn_off() && 
+       if (tools_tearoff->torn_off() &&
            mouse_mode_tearoff->torn_off()) {
                top_hbox.remove (toolbar_frame);
        }
@@ -3782,17 +3711,17 @@ 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);
-       
+
        switch (cew.run ()) {
        case RESPONSE_ACCEPT:
                break;
        default:
                return;
        }
-       
+
        cew.apply ();
        xfade->StateChanged (Change (~0));
 }
@@ -3803,6 +3732,60 @@ Editor::playlist_selector () const
        return *_playlist_selector;
 }
 
+Evoral::MusicalTime
+Editor::get_grid_type_as_beats (bool& success, nframes64_t position)
+{
+       success = true;
+
+       switch (_snap_type) {
+       case SnapToBeat:
+               return 1.0;
+               break;
+
+       case SnapToAThirtysecondBeat:
+               return 1.0/32.0;
+               break;
+
+       case SnapToASixteenthBeat:
+               return 1.0/16.0;
+               break;
+
+       case SnapToAEighthBeat:
+               return 1.0/8.0;
+               break;
+
+       case SnapToAQuarterBeat:
+               return 1.0/4.0;
+               break;
+
+       case SnapToAThirdBeat:
+               return 1.0/3.0;
+               break;
+
+       case SnapToBar:
+               if (_session) {
+                       return _session->tempo_map().meter_at (position).beats_per_bar();
+               }
+               break;
+
+       case SnapToCDFrame:
+       case SnapToTimecodeFrame:
+       case SnapToTimecodeSeconds:
+       case SnapToTimecodeMinutes:
+       case SnapToSeconds:
+       case SnapToMinutes:
+       case SnapToRegionStart:
+       case SnapToRegionEnd:
+       case SnapToRegionSync:
+       case SnapToRegionBoundary:
+       default:
+               success = false;
+               break;
+       }
+
+       return 0.0;
+}
+
 nframes64_t
 Editor::get_nudge_distance (nframes64_t pos, nframes64_t& next)
 {
@@ -3817,7 +3800,7 @@ Editor::get_nudge_distance (nframes64_t pos, nframes64_t& next)
 void
 Editor::end_location_changed (Location* location)
 {
-       ENSURE_GUI_THREAD (bind (mem_fun(*this, &Editor::end_location_changed), location));
+       ENSURE_GUI_THREAD (*this, &Editor::end_location_changed, location)
        //reset_scrolling_region ();
        nframes64_t session_span = location->start() + (nframes64_t) floorf (current_page_frames() * 0.10f);
        horizontal_adjustment.set_upper (session_span / frames_per_unit);
@@ -3831,7 +3814,7 @@ Editor::playlist_deletion_dialog (boost::shared_ptr<Playlist> pl)
                                        "If left alone, no audio files used by it will be cleaned.\n"
                                        "If deleted, audio files used by it alone by will cleaned."),
                                      pl->name()));
-       
+
        dialog.set_position (WIN_POS_CENTER);
        dialog.get_vbox()->pack_start (label);
 
@@ -3892,8 +3875,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;
        }
@@ -3902,8 +3885,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;
        }
@@ -3912,6 +3895,10 @@ Editor::transport_punch_location()
 bool
 Editor::control_layout_scroll (GdkEventScroll* ev)
 {
+       if (Keyboard::some_magic_widget_has_focus()) {
+               return false;
+       }
+
        switch (ev->direction) {
        case GDK_SCROLL_UP:
                scroll_tracks_up_line ();
@@ -3921,7 +3908,7 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
        case GDK_SCROLL_DOWN:
                scroll_tracks_down_line ();
                return true;
-               
+
        default:
                /* no left/right handling yet */
                break;
@@ -3933,8 +3920,8 @@ Editor::control_layout_scroll (GdkEventScroll* ev)
 void
 Editor::session_state_saved (string snap_name)
 {
-       ENSURE_GUI_THREAD (bind (mem_fun(*this, &Editor::session_state_saved), snap_name));
-       
+       ENSURE_GUI_THREAD (*this, &Editor::session_state_saved, snap_name)
+
        _snapshots->redisplay ();
 }
 
@@ -3954,7 +3941,7 @@ Editor::maximise_editing_space ()
        fullscreen();
 
        if(post_maximal_editor_width) {
-               edit_pane.set_position (post_maximal_pane_position - 
+               edit_pane.set_position (post_maximal_pane_position -
                        abs(post_maximal_editor_width - pre_maximal_editor_width));
        } else {
                edit_pane.set_position (post_maximal_pane_position);
@@ -3985,13 +3972,13 @@ Editor::restore_editing_space ()
  *  @param v Track.
  */
 
-void 
+void
 Editor::new_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("new playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
-       session->get_playlists(playlists);
-       mapover_tracks (bind (mem_fun (*this, &Editor::mapped_use_new_playlist), playlists), v, RouteGroup::Edit);
+       _session->playlists->get (playlists);
+       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_new_playlist), playlists), v, RouteGroup::Edit);
        commit_reversible_command ();
 }
 
@@ -4006,8 +3993,8 @@ Editor::copy_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("copy playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
-       session->get_playlists(playlists);
-       mapover_tracks (bind (mem_fun (*this, &Editor::mapped_use_copy_playlist), playlists), v, RouteGroup::Edit);
+       _session->playlists->get (playlists);
+       mapover_tracks (sigc::bind (sigc::mem_fun (*this, &Editor::mapped_use_copy_playlist), playlists), v, RouteGroup::Edit);
        commit_reversible_command ();
 }
 
@@ -4016,17 +4003,17 @@ Editor::copy_playlists (TimeAxisView* v)
  *  @param v Track.
  */
 
-void 
+void
 Editor::clear_playlists (TimeAxisView* v)
 {
        begin_reversible_command (_("clear playlists"));
        vector<boost::shared_ptr<ARDOUR::Playlist> > playlists;
-       session->get_playlists(playlists);
-       mapover_tracks (mem_fun (*this, &Editor::mapped_clear_playlist), v, RouteGroup::Edit);
+       _session->playlists->get (playlists);
+       mapover_tracks (sigc::mem_fun (*this, &Editor::mapped_clear_playlist), v, RouteGroup::Edit);
        commit_reversible_command ();
 }
 
-void 
+void
 Editor::mapped_use_new_playlist (RouteTimeAxisView& atv, uint32_t sz, vector<boost::shared_ptr<ARDOUR::Playlist> > const & playlists)
 {
        atv.use_new_playlist (sz > 1 ? false : true, playlists);
@@ -4038,7 +4025,7 @@ Editor::mapped_use_copy_playlist (RouteTimeAxisView& atv, uint32_t sz, vector<bo
        atv.use_copy_playlist (sz > 1 ? false : true, playlists);
 }
 
-void 
+void
 Editor::mapped_clear_playlist (RouteTimeAxisView& atv, uint32_t /*sz*/)
 {
        atv.clear_playlist ();
@@ -4096,14 +4083,13 @@ Editor::current_visual_state (bool with_tracks)
        vs->frames_per_unit = frames_per_unit;
        vs->leftmost_frame = leftmost_frame;
        vs->zoom_focus = zoom_focus;
-       vs->zoomed_to_region = zoomed_to_region;
 
        if (with_tracks) {
                for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
                        vs->track_states.push_back (TAVState ((*i), &(*i)->get_state()));
                }
        }
-       
+
        return vs;
 }
 
@@ -4154,22 +4140,21 @@ Editor::use_visual_state (VisualState& vs)
 
        set_zoom_focus (vs.zoom_focus);
        reposition_and_zoom (vs.leftmost_frame, vs.frames_per_unit);
-       zoomed_to_region = vs.zoomed_to_region;
-       
+
        for (list<TAVState>::iterator i = vs.track_states.begin(); i != vs.track_states.end(); ++i) {
                TrackViewList::iterator t;
 
                /* check if the track still exists - it could have been deleted */
 
                if ((t = find (track_views.begin(), track_views.end(), i->first)) != track_views.end()) {
-                       (*t)->set_state (*(i->second));
+                       (*t)->set_state (*(i->second), Stateful::loading_state_version);
                }
        }
 
 
        if (!vs.track_states.empty()) {
                _routes->update_visibility ();
-       } 
+       }
 
        _routes->resume_redisplay ();
 
@@ -4191,7 +4176,7 @@ Editor::set_frames_per_unit (double fpu)
                fpu = 2.0;
        }
 
-       
+
        /* don't allow zooms that fit more than the maximum number
           of frames into an 800 pixel wide space.
        */
@@ -4199,7 +4184,7 @@ Editor::set_frames_per_unit (double fpu)
        if (max_frames / fpu < 800.0) {
                return;
        }
-       
+
        if (tempo_lines)
                tempo_lines->tempo_map_changed();
 
@@ -4221,14 +4206,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);
                }
        }
 
@@ -4236,10 +4215,8 @@ Editor::post_zoom ()
 
        ZoomChanged (); /* EMIT_SIGNAL */
 
-       reset_hscrollbar_stepping ();
-
-       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);
 
@@ -4259,17 +4236,17 @@ void
 Editor::queue_visual_change (nframes64_t where)
 {
        pending_visual_change.add (VisualChange::TimeOrigin);
-       
+
        /* if we're moving beyond the end, make sure the upper limit of the horizontal adjustment
           can reach.
        */
-       
-       if (where > session->current_end_frame()) {
+
+       if (_session && (where > _session->current_end_frame())) {
                horizontal_adjustment.set_upper ((where + current_page_frames()) / frames_per_unit);
        }
-       
+
        pending_visual_change.time_origin = where;
-       
+
        ensure_visual_change_idle_handler ();
 }
 
@@ -4280,7 +4257,6 @@ Editor::queue_visual_change (double fpu)
        pending_visual_change.frames_per_unit = fpu;
 
        ensure_visual_change_idle_handler ();
-       
 }
 
 void
@@ -4312,9 +4288,7 @@ Editor::idle_visual_changer ()
        VisualChange::Type p = pending_visual_change.pending;
        pending_visual_change.pending = (VisualChange::Type) 0;
 
-#ifdef FIX_THIS_FOR_V3
        double last_time_origin = horizontal_adjustment.get_value();
-#endif
 
        if (p & VisualChange::ZoomLevel) {
                set_frames_per_unit (pending_visual_change.frames_per_unit);
@@ -4330,25 +4304,24 @@ Editor::idle_visual_changer ()
        if (p & VisualChange::YOrigin) {
                vertical_adjustment.set_value (pending_visual_change.y_origin);
        }
-       
+
        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 */
 
-#ifdef FIX_THIS_FOR_V3
+
        if (last_time_origin == horizontal_adjustment.get_value() ) {
                /* changed signal not emitted */
                update_fixed_rulers ();
                redisplay_tempo (true);
        }
-#endif
-       
+
        if (current_time_origin != pending_visual_change.time_origin) {
                cef += current_page_frames() / 10; // Add a little extra so we can see the end marker
                horizontal_adjustment.set_upper (cef / frames_per_unit);
@@ -4359,8 +4332,8 @@ Editor::idle_visual_changer ()
        }
 
        _summary->set_overlays_dirty ();
-       
-       //cerr << "Editor::idle_visual_changer () called ha v:l:u:ps:fpu = " << horizontal_adjustment.get_value() << ":" << horizontal_adjustment.get_lower() << ":" << horizontal_adjustment.get_upper() << ":" << horizontal_adjustment.get_page_size() << ":" << frames_per_unit << endl;//DEBUG
+
+       // cerr << "Editor::idle_visual_changer () called ha v:l:u:ps:fpu = " << horizontal_adjustment.get_value() << ":" << horizontal_adjustment.get_lower() << ":" << horizontal_adjustment.get_upper() << ":" << horizontal_adjustment.get_page_size() << ":" << frames_per_unit << endl;//DEBUG
        pending_visual_change.idle_handler_id = -1;
        return 0; /* this is always a one-shot call */
 }
@@ -4370,9 +4343,9 @@ struct EditorOrderTimeAxisSorter {
            return a->order () < b->order ();
     }
 };
-       
+
 void
-Editor::sort_track_selection (TrackSelection* sel)
+Editor::sort_track_selection (TrackViewList* sel)
 {
        EditorOrderTimeAxisSorter cmp;
 
@@ -4400,9 +4373,9 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
 
        switch (ep) {
        case EditAtPlayhead:
-               where = session->audible_frame();
+               where = _session->audible_frame();
                break;
-               
+
        case EditAtSelectedMarker:
                if (!selection->markers.empty()) {
                        bool is_start;
@@ -4415,9 +4388,9 @@ Editor::get_preferred_edit_position (bool ignore_playhead)
                                }
                                break;
                        }
-               } 
+               }
                /* fallthru */
-               
+
        default:
        case EditAtMouse:
                if (!mouse_frame (where, ignored)) {
@@ -4434,55 +4407,55 @@ 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);
-       
+
        Location* tll;
 
        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 ();
 }
 
 void
 Editor::set_punch_range (nframes64_t start, nframes64_t end, string cmd)
 {
-       if (!session) return;
+       if (!_session) return;
 
        begin_reversible_command (cmd);
-       
+
        Location* tpl;
 
        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 ();
 }
 
@@ -4492,9 +4465,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;
@@ -4502,12 +4475,12 @@ 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;
                        boost::shared_ptr<Playlist> pl;
-                       
+
                        if ((ds = rtv->get_diskstream()) && ((pl = ds->playlist()))) {
 
                                Playlist::RegionList* regions = pl->regions_at (
@@ -4527,9 +4500,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;
@@ -4537,12 +4510,12 @@ 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;
                        boost::shared_ptr<Playlist> pl;
-                       
+
                        if ((ds = rtv->get_diskstream()) && ((pl = ds->playlist()))) {
 
                                Playlist::RegionList* regions = pl->regions_touched (
@@ -4567,12 +4540,12 @@ Editor::get_regions_after (RegionSelection& rs, nframes64_t where, const TrackSe
  *      - selected or
  *      - the entered_regionview (if allow_entered == true) or
  *      - under the preferred edit position AND on a selected track, or on a track
- *        which is in the same active edit-enable route group as a selected region.
+ *        which is in the same active edit-enable route group as a selected region (if allow_edit_position == true)
  *  @param rs Returned region list.
  *  @param allow_entered true to include the entered_regionview in the list.
  */
 void
-Editor::get_regions_for_action (RegionSelection& rs, bool allow_entered)
+Editor::get_regions_for_action (RegionSelection& rs, bool allow_entered, bool allow_edit_position)
 {
        /* Start with selected regions */
        rs = selection->regions;
@@ -4582,25 +4555,27 @@ Editor::get_regions_for_action (RegionSelection& rs, bool allow_entered)
                rs.add (entered_regionview);
        }
 
-       TrackSelection tracks = selection->tracks;
+       if (allow_edit_position) {
 
-       RegionSelection to_map = rs;
+               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 = to_map.begin (); i != to_map.end(); ++i) {
+               /* 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 */
-       for (RegionSelection::iterator i = to_map.begin (); i != to_map.end(); ++i) {
-               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);
+               }
        }
 }
 
@@ -4608,34 +4583,34 @@ void
 Editor::get_regions_corresponding_to (boost::shared_ptr<Region> region, vector<RegionView*>& regions)
 {
        for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
-               
+
                RouteTimeAxisView* tatv;
-               
+
                if ((tatv = dynamic_cast<RouteTimeAxisView*> (*i)) != 0) {
-                       
+
                        boost::shared_ptr<Playlist> pl;
                        vector<boost::shared_ptr<Region> > results;
                        RegionView* marv;
                        boost::shared_ptr<Diskstream> ds;
-                       
+
                        if ((ds = tatv->get_diskstream()) == 0) {
                                /* bus */
                                continue;
                        }
-                       
+
                        if ((pl = (ds->playlist())) != 0) {
                                pl->get_region_list_equivalent_regions (region, results);
                        }
-                       
+
                        for (vector<boost::shared_ptr<Region> >::iterator ir = results.begin(); ir != results.end(); ++ir) {
                                if ((marv = tatv->view()->find_view (*ir)) != 0) {
                                        regions.push_back (marv);
                                }
                        }
-                       
+
                }
        }
-}      
+}
 
 void
 Editor::show_rhythm_ferret ()
@@ -4644,26 +4619,16 @@ 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 ();
 }
 
-void
-Editor::show_bundle_manager ()
-{
-       if (_bundle_manager == 0) {
-               _bundle_manager = new BundleManager (*session);
-       }
-
-       _bundle_manager->show ();
-}
-
 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 ();
@@ -4674,8 +4639,8 @@ Editor::first_idle ()
 {
        MessageDialog* dialog = 0;
 
-       if (track_views.size() > 1) { 
-               dialog = new MessageDialog (*this, 
+       if (track_views.size() > 1) {
+               dialog = new MessageDialog (*this,
                                            _("Please wait while Ardour loads visual data"),
                                            true,
                                            Gtk::MESSAGE_INFO,
@@ -4690,7 +4655,7 @@ Editor::first_idle ()
 
        // first idle adds route children (automation tracks), so we need to redisplay here
        _routes->redisplay ();
-       
+
        delete dialog;
 
        _have_idled = true;
@@ -4721,7 +4686,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);
                }
        }
@@ -4744,25 +4709,25 @@ 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);
                        }
                }
        }
-       
+
        flush_canvas ();
        _group_tabs->set_dirty ();
        resize_idle_id = -1;
-       
+
        return false;
 }
 
 void
 Editor::located ()
 {
-       ENSURE_GUI_THREAD (mem_fun (*this, &Editor::located));
+       ENSURE_GUI_THREAD (*this, &Editor::located)
 
        _pending_locate_request = false;
 }
@@ -4780,12 +4745,12 @@ Editor::streamview_height_changed ()
 }
 
 TimeAxisView*
-Editor::axis_view_from_route (Route* r) const
+Editor::axis_view_from_route (boost::shared_ptr<Route> r) const
 {
        TrackViewList::const_iterator j = track_views.begin ();
        while (j != track_views.end()) {
                RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (*j);
-               if (rtv && rtv->route().get() == r) {
+               if (rtv && rtv->route() == r) {
                        return rtv;
                }
                ++j;
@@ -4795,12 +4760,12 @@ Editor::axis_view_from_route (Route* r) const
 }
 
 
-TrackSelection
-Editor::axis_views_from_routes (list<Route*> r) const
+TrackViewList
+Editor::axis_views_from_routes (boost::shared_ptr<RouteList> r) const
 {
-       TrackSelection t;
-       
-       for (list<Route*>::const_iterator i = r.begin(); i != r.end(); ++i) {
+       TrackViewList t;
+
+       for (RouteList::const_iterator i = r->begin(); i != r->end(); ++i) {
                TimeAxisView* tv = axis_view_from_route (*i);
                if (tv) {
                        t.push_back (tv);
@@ -4814,8 +4779,8 @@ Editor::axis_views_from_routes (list<Route*> r) const
 void
 Editor::handle_new_route (RouteList& routes)
 {
-       ENSURE_GUI_THREAD (bind (mem_fun (*this, &Editor::handle_new_route), routes));
-       
+       ENSURE_GUI_THREAD (*this, &Editor::handle_new_route, routes)
+
        RouteTimeAxisView *rtv;
        list<RouteTimeAxisView*> new_views;
 
@@ -4829,22 +4794,22 @@ 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();
                }
 
                new_views.push_back (rtv);
                track_views.push_back (rtv);
-               
+
                rtv->effective_gain_display ();
-               
-               rtv->view()->RegionViewAdded.connect (mem_fun (*this, &Editor::region_view_added));
-               rtv->view()->HeightChanged.connect (mem_fun (*this, &Editor::streamview_height_changed));
-               
-               rtv->GoingAway.connect (bind (mem_fun(*this, &Editor::remove_route), rtv));
+
+               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 (*this, boost::bind (&Editor::remove_route, this, rtv));
        }
 
        _routes->routes_added (new_views);
@@ -4861,11 +4826,17 @@ Editor::handle_new_route (RouteList& routes)
 void
 Editor::remove_route (TimeAxisView *tv)
 {
-       ENSURE_GUI_THREAD(bind (mem_fun(*this, &Editor::remove_route), tv));
+       ENSURE_GUI_THREAD (*this, &Editor::remove_route, tv)
 
        TrackViewList::iterator i;
+
        boost::shared_ptr<Route> route;
-       TimeAxisView* next_tv;
+       RouteTimeAxisView* rtav = dynamic_cast<RouteTimeAxisView*> (tv);
+       if (rtav) {
+               route = rtav->route ();
+       }
+               
+       TimeAxisView* next_tv = 0;
 
        if (tv == entered_track) {
                entered_track = 0;
@@ -4883,7 +4854,7 @@ Editor::remove_route (TimeAxisView *tv)
                       next_tv = (*i);
                }
        }
-       
+
        if (current_mixer_strip && current_mixer_strip->route() == route) {
 
                if (next_tv) {
@@ -4895,7 +4866,7 @@ Editor::remove_route (TimeAxisView *tv)
 
                        ActionManager::uncheck_toggleaction ("<Actions>/Editor/show-editor-mixer");
                }
-       } 
+       }
 }
 
 void
@@ -4914,11 +4885,11 @@ 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 ();
-       
+
        return false; // do not call again (until needed)
 }
 
@@ -4949,7 +4920,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);
 }
 
@@ -4959,18 +4930,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;
 }
 
@@ -4984,18 +4955,59 @@ 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
 Editor::audition_region_from_region_list ()
 {
-       _regions->selection_mapover (mem_fun (*this, &Editor::consider_auditioning));
+       _regions->selection_mapover (sigc::mem_fun (*this, &Editor::consider_auditioning));
 }
 
 void
 Editor::hide_region_from_region_list ()
 {
-       _regions->selection_mapover (mem_fun (*this, &Editor::hide_a_region));
+       _regions->selection_mapover (sigc::mem_fun (*this, &Editor::hide_a_region));
 }
 
+void
+Editor::start_step_editing ()
+{
+       step_edit_connection = Glib::signal_timeout().connect (sigc::mem_fun (*this, &Editor::check_step_edit), 20);
+}
+
+void
+Editor::stop_step_editing ()
+{
+       step_edit_connection.disconnect ();
+}
+
+bool
+Editor::check_step_edit ()
+{
+       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+               MidiTimeAxisView* mtv = dynamic_cast<MidiTimeAxisView*> (*i);
+               if (mtv) {
+                       mtv->check_step_edit ();
+               }
+       }
+
+       return true; // do it again, till we stop
+}
+
+void
+Editor::horizontal_scroll_left ()
+{
+       double x = leftmost_position() - current_page_frames() / 5;
+       if (x < 0) {
+               x = 0;
+       }
+       
+       reset_x_origin (x);
+}
+
+void
+Editor::horizontal_scroll_right ()
+{
+       reset_x_origin (leftmost_position() + current_page_frames() / 5);
+}