merge with master, primarily for adrian's maximise-mixer change
[ardour.git] / gtk2_ardour / region_view.cc
index 534c7ef0d6d984c219d24efa47fd5c61a9040759..f468eedb60b643bec604dd2856abc1a69204cb81 100644 (file)
 #include <gtkmm2ext/gtk_ui.h>
 
 #include "ardour/playlist.h"
-#include "ardour/audioregion.h"
-#include "ardour/audiosource.h"
-#include "ardour/audio_diskstream.h"
 #include "ardour/session.h"
 
+#include "canvas/polygon.h"
+#include "canvas/debug.h"
+#include "canvas/pixbuf.h"
+#include "canvas/text.h"
+#include "canvas/line.h"
+
 #include "ardour_ui.h"
 #include "global_signals.h"
-#include "canvas-noevent-text.h"
 #include "streamview.h"
 #include "region_view.h"
 #include "automation_region_view.h"
 #include "route_time_axis.h"
-#include "simplerect.h"
-#include "simpleline.h"
-#include "waveview.h"
 #include "public_editor.h"
 #include "region_editor.h"
 #include "ghostregion.h"
@@ -70,8 +69,9 @@ RegionView::RegionView (ArdourCanvas::Group*              parent,
                         Gdk::Color const &                basic_color,
                        bool                              automation)
        : TimeAxisViewItem (r->name(), *parent, tv, spu, basic_color, r->position(), r->length(), false, automation,
-                           TimeAxisViewItem::Visibility (TimeAxisViewItem::ShowNameText|
-                                                         TimeAxisViewItem::ShowNameHighlight| TimeAxisViewItem::ShowFrame))
+                           (automation ? TimeAxisViewItem::ShowFrame :
+                            TimeAxisViewItem::Visibility (TimeAxisViewItem::ShowNameText|
+                                                          TimeAxisViewItem::ShowNameHighlight| TimeAxisViewItem::ShowFrame)))
        , _region (r)
        , sync_mark(0)
        , sync_line(0)
@@ -86,7 +86,7 @@ RegionView::RegionView (ArdourCanvas::Group*              parent,
        , _region_relative_time_converter(r->session().tempo_map(), r->position())
        , _source_relative_time_converter(r->session().tempo_map(), r->position() - r->start())
 {
-       GhostRegion::CatchDeletion.connect (*this, invalidator (*this), ui_bind (&RegionView::remove_ghost, this, _1), gui_context());
+       GhostRegion::CatchDeletion.connect (*this, invalidator (*this), boost::bind (&RegionView::remove_ghost, this, _1), gui_context());
 }
 
 RegionView::RegionView (const RegionView& other)
@@ -103,7 +103,7 @@ RegionView::RegionView (const RegionView& other)
        valid = false;
        _pixel_width = other._pixel_width;
 
-       GhostRegion::CatchDeletion.connect (*this, invalidator (*this), ui_bind (&RegionView::remove_ghost, this, _1), gui_context());
+       GhostRegion::CatchDeletion.connect (*this, invalidator (*this), boost::bind (&RegionView::remove_ghost, this, _1), gui_context());
 }
 
 RegionView::RegionView (const RegionView& other, boost::shared_ptr<Region> other_region)
@@ -124,7 +124,7 @@ RegionView::RegionView (const RegionView& other, boost::shared_ptr<Region> other
        valid = false;
        _pixel_width = other._pixel_width;
 
-       GhostRegion::CatchDeletion.connect (*this, invalidator (*this), ui_bind (&RegionView::remove_ghost, this, _1), gui_context());
+       GhostRegion::CatchDeletion.connect (*this, invalidator (*this), boost::bind (&RegionView::remove_ghost, this, _1), gui_context());
 }
 
 RegionView::RegionView (ArdourCanvas::Group*         parent,
@@ -167,26 +167,26 @@ RegionView::init (Gdk::Color const & basic_color, bool wfd)
 
        if (name_highlight) {
                name_highlight->set_data ("regionview", this);
-               name_highlight->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_name_highlight_event), name_highlight, this));
+               name_highlight->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_name_highlight_event), name_highlight, this));
 
                if (frame_handle_start) {
                        frame_handle_start->set_data ("regionview", this);
                        frame_handle_start->set_data ("isleft", (void*) 1);
-                       frame_handle_start->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_frame_handle_event), frame_handle_start, this));
+                       frame_handle_start->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_frame_handle_event), frame_handle_start, this));
                        frame_handle_start->raise_to_top();
                }
 
                if (frame_handle_end) {
                        frame_handle_end->set_data ("regionview", this);
                        frame_handle_end->set_data ("isleft", (void*) 0);
-                       frame_handle_end->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_frame_handle_event), frame_handle_end, this));
+                       frame_handle_end->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_frame_handle_event), frame_handle_end, this));
                        frame_handle_end->raise_to_top();
                }
        }
 
-       if (name_pixbuf) {
-               name_pixbuf->set_data ("regionview", this);
-               name_pixbuf->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_name_event), name_pixbuf, this));
+       if (name_text) {
+               name_text->set_data ("regionview", this);
+               name_text->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_name_event), name_text, this));
        }
 
        if (wfd) {
@@ -195,9 +195,7 @@ RegionView::init (Gdk::Color const & basic_color, bool wfd)
 
        set_height (trackview.current_height());
 
-       _region->PropertyChanged.connect (*this, invalidator (*this), ui_bind (&RegionView::region_changed, this, _1), gui_context());
-
-       group->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_event), group, this));
+       _region->PropertyChanged.connect (*this, invalidator (*this), boost::bind (&RegionView::region_changed, this, _1), gui_context());
 
        set_colors ();
 
@@ -214,7 +212,7 @@ RegionView::~RegionView ()
                delete *g;
        }
 
-       for (list<ArdourCanvas::SimpleRect*>::iterator i = _coverage_frames.begin (); i != _coverage_frames.end (); ++i) {
+       for (list<ArdourCanvas::Rectangle*>::iterator i = _coverage_frames.begin (); i != _coverage_frames.end (); ++i) {
                delete *i;
        }
 
@@ -223,8 +221,14 @@ RegionView::~RegionView ()
        delete editor;
 }
 
+bool
+RegionView::canvas_group_event (GdkEvent* event)
+{
+       return trackview.editor().canvas_region_view_event (event, group, this);
+}
+
 void
-RegionView::set_silent_frames (const AudioIntervalResult& silences, double threshold)
+RegionView::set_silent_frames (const AudioIntervalResult& silences, double /*threshold*/)
 {
         framecnt_t shortest = max_framecnt;
 
@@ -235,21 +239,22 @@ RegionView::set_silent_frames (const AudioIntervalResult& silences, double thres
                 return;
         }
 
-        uint32_t const color = ARDOUR_UI::config()->canvasvar_Silence.get();
+        uint32_t const color = ARDOUR_UI::config()->get_canvasvar_Silence();
 
        for (AudioIntervalResult::const_iterator i = silences.begin(); i != silences.end(); ++i) {
 
-               ArdourCanvas::SimpleRect* cr = new ArdourCanvas::SimpleRect (*group);
+               ArdourCanvas::Rectangle* cr = new ArdourCanvas::Rectangle (group);
+               cr->set_ignore_events (true);
                _silent_frames.push_back (cr);
 
                /* coordinates for the rect are relative to the regionview origin */
 
-               cr->property_x1() = trackview.editor().frame_to_pixel (i->first - _region->start());
-               cr->property_x2() = trackview.editor().frame_to_pixel (i->second - _region->start());
-               cr->property_y1() = 1;
-               cr->property_y2() = _height - 2;
-               cr->property_outline_pixels() = 0;
-               cr->property_fill_color_rgba () = color;
+               cr->set_x0 (trackview.editor().sample_to_pixel (i->first - _region->start()));
+               cr->set_x1 (trackview.editor().sample_to_pixel (i->second - _region->start()));
+               cr->set_y0 (1);
+               cr->set_y1 (_height - 2);
+               cr->set_outline (false);
+               cr->set_fill_color (color);
 
                shortest = min (shortest, i->second - i->first);
        }
@@ -272,15 +277,15 @@ RegionView::set_silent_frames (const AudioIntervalResult& silences, double thres
                shortest_audible = min (shortest_audible, dur);
        }
 
-        _silence_text = new ArdourCanvas::NoEventText (*group);
-        _silence_text->property_font_desc() = get_font_for_style (N_("SilenceText"));
-        _silence_text->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_SilenceText.get();
-        _silence_text->property_anchor() = ANCHOR_NW;
+        _silence_text = new ArdourCanvas::Text (group);
+       _silence_text->set_ignore_events (true);
+        _silence_text->set_font_description (get_font_for_style (N_("SilenceText")));
+        _silence_text->set_color (ARDOUR_UI::config()->get_canvasvar_SilenceText());
 
         /* both positions are relative to the region start offset in source */
 
-        _silence_text->property_x() = trackview.editor().frame_to_pixel (silences.front().first - _region->start()) + 10.0;
-        _silence_text->property_y() = 20.0;
+        _silence_text->set_x_position (trackview.editor().sample_to_pixel (silences.front().first - _region->start()) + 10.0);
+        _silence_text->set_y_position (20.0);
 
         double ms = (float) shortest/_region->session().frame_rate();
 
@@ -320,13 +325,13 @@ RegionView::set_silent_frames (const AudioIntervalResult& silences, double thres
                text += string_compose (_("\n  (shortest audible segment = %1 %2)"), ma, aunits);
        }
 
-       _silence_text->property_text() = text.c_str ();
+       _silence_text->set (text);
 }
 
 void
 RegionView::hide_silent_frames ()
 {
-       for (list<ArdourCanvas::SimpleRect*>::iterator i = _silent_frames.begin (); i != _silent_frames.end (); ++i) {
+       for (list<ArdourCanvas::Rectangle*>::iterator i = _silent_frames.begin (); i != _silent_frames.end (); ++i) {
                 (*i)->hide ();
        }
         _silence_text->hide();
@@ -335,7 +340,7 @@ RegionView::hide_silent_frames ()
 void
 RegionView::drop_silent_frames ()
 {
-       for (list<ArdourCanvas::SimpleRect*>::iterator i = _silent_frames.begin (); i != _silent_frames.end (); ++i) {
+       for (list<ArdourCanvas::Rectangle*>::iterator i = _silent_frames.begin (); i != _silent_frames.end (); ++i) {
                delete *i;
        }
         _silent_frames.clear ();
@@ -388,6 +393,10 @@ RegionView::region_changed (const PropertyChange& what_changed)
        if (what_changed.contains (ARDOUR::Properties::locked)) {
                region_locked ();
        }
+       if (what_changed.contains (ARDOUR::Properties::locked)) {
+               /* name will show locked status */
+               region_renamed ();
+       }
 }
 
 void
@@ -419,7 +428,7 @@ RegionView::region_resized (const PropertyChange& what_changed)
 
                set_duration (_region->length(), 0);
 
-               unit_length = _region->length() / samples_per_unit;
+               unit_length = _region->length() / samples_per_pixel;
 
                for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
 
@@ -477,7 +486,7 @@ RegionView::set_position (framepos_t pos, void* /*src*/, double* ignored)
 
        if (delta) {
                for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-                       (*i)->group->move (delta, 0.0);
+                       (*i)->group->move (ArdourCanvas::Duple (delta, 0.0));
                }
        }
 
@@ -485,13 +494,13 @@ RegionView::set_position (framepos_t pos, void* /*src*/, double* ignored)
 }
 
 void
-RegionView::set_samples_per_unit (gdouble spu)
+RegionView::set_samples_per_pixel (double fpp)
 {
-       TimeAxisViewItem::set_samples_per_unit (spu);
+       TimeAxisViewItem::set_samples_per_pixel (fpp);
 
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-               (*i)->set_samples_per_unit (spu);
-               (*i)->set_duration (_region->length() / samples_per_unit);
+               (*i)->set_samples_per_pixel (fpp);
+               (*i)->set_duration (_region->length() / fpp);
        }
 
        region_sync_changed ();
@@ -505,7 +514,7 @@ RegionView::set_duration (framecnt_t frames, void *src)
        }
 
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-               (*i)->set_duration (_region->length() / samples_per_unit);
+               (*i)->set_duration (_region->length() / samples_per_pixel);
        }
 
        return true;
@@ -517,8 +526,9 @@ RegionView::set_colors ()
        TimeAxisViewItem::set_colors ();
 
        if (sync_mark) {
-               sync_mark->property_fill_color_rgba() = RGBA_TO_UINT(0,255,0,255);     // fill_color;                   // FIXME make a themeable colour
-               sync_line->property_fill_color_rgba() = RGBA_TO_UINT(0,255,0,255);     // fill_color;                   // FIXME make a themeable colour
+               /* XXX: make these colours themable */
+               sync_mark->set_fill_color (RGBA_TO_UINT (0, 255, 0, 255));
+               sync_line->set_outline_color (RGBA_TO_UINT (0, 255, 0, 255));
        }
 }
 
@@ -580,6 +590,10 @@ RegionView::make_name () const
                str += '{';
                str += _region->name();
                str += '}';
+       } else if (_region->video_locked()) {
+               str += '[';
+               str += _region->name();
+               str += ']';
        } else {
                str = _region->name();
        }
@@ -622,15 +636,16 @@ RegionView::region_sync_changed ()
 
                /* points set below */
 
-               sync_mark = new ArdourCanvas::Polygon (*group);
-               sync_mark->property_fill_color_rgba() = RGBA_TO_UINT(0,255,0,255);     // fill_color;                   // FIXME make a themeable colour
+               sync_mark = new ArdourCanvas::Polygon (group);
+               CANVAS_DEBUG_NAME (sync_mark, string_compose ("sync mark for %1", get_item_name()));
+               sync_mark->set_fill_color (RGBA_TO_UINT(0,255,0,255));    // FIXME make a themeable colour
 
-               sync_line = new ArdourCanvas::Line (*group);
-               sync_line->property_fill_color_rgba() = RGBA_TO_UINT(0,255,0,255);     // fill_color;                   // FIXME make a themeable colour
-               sync_line->property_width_pixels() = 1;
+               sync_line = new ArdourCanvas::Line (group);
+               CANVAS_DEBUG_NAME (sync_line, string_compose ("sync mark for %1", get_item_name()));
+               sync_line->set_outline_color (RGBA_TO_UINT(0,255,0,255)); // FIXME make a themeable colour
        }
 
-       /* this has to handle both a genuine change of position, a change of samples_per_unit,
+       /* this has to handle both a genuine change of position, a change of samples_per_pixel
           and a change in the bounds of the _region->
         */
 
@@ -658,19 +673,15 @@ RegionView::region_sync_changed ()
 
                        //points = sync_mark->property_points().get_value();
 
-                       double offset = sync_offset / samples_per_unit;
-                       points.push_back (Gnome::Art::Point (offset - ((sync_mark_width-1)/2), 1));
-                       points.push_back (Gnome::Art::Point (offset + ((sync_mark_width-1)/2), 1));
-                       points.push_back (Gnome::Art::Point (offset, sync_mark_width - 1));
-                       points.push_back (Gnome::Art::Point (offset - ((sync_mark_width-1)/2), 1));
-                       sync_mark->property_points().set_value (points);
+                       double offset = sync_offset / samples_per_pixel;
+                       points.push_back (ArdourCanvas::Duple (offset - ((sync_mark_width-1)/2), 1));
+                       points.push_back (ArdourCanvas::Duple (offset + ((sync_mark_width-1)/2), 1));
+                       points.push_back (ArdourCanvas::Duple (offset, sync_mark_width - 1));
+                       points.push_back (ArdourCanvas::Duple (offset - ((sync_mark_width-1)/2), 1));
+                       sync_mark->set (points);
                        sync_mark->show ();
 
-                       points.clear ();
-                       points.push_back (Gnome::Art::Point (offset, 0));
-                       points.push_back (Gnome::Art::Point (offset, trackview.current_height() - NAME_HIGHLIGHT_SIZE));
-
-                       sync_line->property_points().set_value (points);
+                       sync_line->set (ArdourCanvas::Duple (offset, 0), ArdourCanvas::Duple (offset, trackview.current_height() - NAME_HIGHLIGHT_SIZE));
                        sync_line->show ();
                }
        }
@@ -683,12 +694,12 @@ RegionView::move (double x_delta, double y_delta)
                return;
        }
 
-       get_canvas_group()->move (x_delta, y_delta);
+       get_canvas_group()->move (ArdourCanvas::Duple (x_delta, y_delta));
 
        /* note: ghosts never leave their tracks so y_delta for them is always zero */
 
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-               (*i)->group->move (x_delta, 0.0);
+               (*i)->group->move (ArdourCanvas::Duple (x_delta, 0.0));
        }
 }
 
@@ -734,19 +745,20 @@ RegionView::set_height (double h)
                int sync_dir;
                framecnt_t sync_offset;
                sync_offset = _region->sync_offset (sync_dir);
-               double offset = sync_offset / samples_per_unit;
+               double offset = sync_offset / samples_per_pixel;
 
-               points.push_back (Gnome::Art::Point (offset, 0));
-               points.push_back (Gnome::Art::Point (offset, h - NAME_HIGHLIGHT_SIZE));
-               sync_line->property_points().set_value (points);
+               sync_line->set (
+                       ArdourCanvas::Duple (offset, 0),
+                       ArdourCanvas::Duple (offset, h - NAME_HIGHLIGHT_SIZE)
+                       );
        }
 
-       for (list<ArdourCanvas::SimpleRect*>::iterator i = _coverage_frames.begin(); i != _coverage_frames.end(); ++i) {
-               (*i)->property_y2() = h + 1;
+       for (list<ArdourCanvas::Rectangle*>::iterator i = _coverage_frames.begin(); i != _coverage_frames.end(); ++i) {
+               (*i)->set_y1 (h + 1);
        }
 
-       for (list<ArdourCanvas::SimpleRect*>::iterator i = _silent_frames.begin(); i != _silent_frames.end(); ++i) {
-               (*i)->property_y2() = h + 1;
+       for (list<ArdourCanvas::Rectangle*>::iterator i = _silent_frames.begin(); i != _silent_frames.end(); ++i) {
+               (*i)->set_y1 (h + 1);
        }
 
 }
@@ -757,7 +769,7 @@ void
 RegionView::update_coverage_frames (LayerDisplay d)
 {
        /* remove old coverage frames */
-       for (list<ArdourCanvas::SimpleRect*>::iterator i = _coverage_frames.begin (); i != _coverage_frames.end (); ++i) {
+       for (list<ArdourCanvas::Rectangle*>::iterator i = _coverage_frames.begin (); i != _coverage_frames.end (); ++i) {
                delete *i;
        }
 
@@ -777,11 +789,11 @@ RegionView::update_coverage_frames (LayerDisplay d)
        framepos_t t = position;
        framepos_t const end = _region->last_frame ();
 
-       ArdourCanvas::SimpleRect* cr = 0;
+       ArdourCanvas::Rectangle* cr = 0;
        bool me = false;
 
-       uint32_t const color = frame->property_fill_color_rgba ();
-       uint32_t const base_alpha = UINT_RGBA_A (color);
+       /* the color that will be used to show parts of regions that will not be heard */
+       uint32_t const non_playing_color = ARDOUR_UI::config()->get_canvasvar_CoveredRegion ();
 
        while (t < end) {
 
@@ -792,23 +804,23 @@ RegionView::update_coverage_frames (LayerDisplay d)
 
                /* finish off any old rect, if required */
                if (cr && me != new_me) {
-                       cr->property_x2() = trackview.editor().frame_to_pixel (t - position);
+                       cr->set_x1 (trackview.editor().sample_to_pixel (t - position));
                }
 
                /* start off any new rect, if required */
                if (cr == 0 || me != new_me) {
-                       cr = new ArdourCanvas::SimpleRect (*group);
+                       cr = new ArdourCanvas::Rectangle (group);
                        _coverage_frames.push_back (cr);
-                       cr->property_x1() = trackview.editor().frame_to_pixel (t - position);
-                       cr->property_y1() = 1;
-                       cr->property_y2() = _height + 1;
-                       cr->property_outline_pixels() = 0;
-                       /* areas that will be played get a lower alpha */
-                       uint32_t alpha = base_alpha;
+                       cr->set_x0 (trackview.editor().sample_to_pixel (t - position));
+                       cr->set_y0 (1);
+                       cr->set_y1 (_height + 1);
+                       cr->set_outline (false);
+                       cr->set_ignore_events (true);
                        if (new_me) {
-                               alpha /= 2;
+                               cr->set_fill_color (UINT_RGBA_CHANGE_A (non_playing_color, 0));
+                       } else {
+                               cr->set_fill_color (non_playing_color);
                        }
-                       cr->property_fill_color_rgba () = UINT_RGBA_CHANGE_A (color, alpha);
                }
 
                t = pl->find_next_region_boundary (t, 1);
@@ -817,7 +829,7 @@ RegionView::update_coverage_frames (LayerDisplay d)
 
        if (cr) {
                /* finish off the last rectangle */
-               cr->property_x2() = trackview.editor().frame_to_pixel (end - position);
+               cr->set_x1 (trackview.editor().sample_to_pixel (end - position));
        }
 
        if (frame_handle_start) {
@@ -832,16 +844,16 @@ RegionView::update_coverage_frames (LayerDisplay d)
                name_highlight->raise_to_top ();
        }
 
-       if (name_pixbuf) {
-               name_pixbuf->raise_to_top ();
+       if (name_text) {
+               name_text->raise_to_top ();
        }
 }
 
-void
+bool
 RegionView::trim_front (framepos_t new_bound, bool no_overlap)
 {
        if (_region->locked()) {
-               return;
+               return false;
        }
 
        RouteTimeAxisView& rtv = dynamic_cast<RouteTimeAxisView&> (trackview);
@@ -869,13 +881,15 @@ RegionView::trim_front (framepos_t new_bound, bool no_overlap)
        }
 
        region_changed (ARDOUR::bounds_change);
+
+       return (pre_trim_first_frame != _region->first_frame());  //return true if we actually changed something
 }
 
-void
+bool
 RegionView::trim_end (framepos_t new_bound, bool no_overlap)
 {
        if (_region->locked()) {
-               return;
+               return false;
        }
 
        RouteTimeAxisView& rtv = dynamic_cast<RouteTimeAxisView&> (trackview);
@@ -906,6 +920,8 @@ RegionView::trim_end (framepos_t new_bound, bool no_overlap)
        } else {
                region_changed (PropertyChange (ARDOUR::Properties::length));
        }
+
+       return (pre_trim_last_frame != _region->last_frame());  //return true if we actually changed something
 }
 
 
@@ -921,32 +937,12 @@ RegionView::thaw_after_trim ()
 
 
 void
-RegionView::trim_contents (framepos_t frame_delta, bool left_direction, bool swap_direction)
+RegionView::move_contents (frameoffset_t distance)
 {
        if (_region->locked()) {
                return;
        }
-
-       framepos_t new_bound;
-
-       RouteTimeAxisView& rtv = dynamic_cast<RouteTimeAxisView&> (trackview);
-       double const speed = rtv.track()->speed ();
-
-       if (left_direction) {
-               if (swap_direction) {
-                       new_bound = (framepos_t) (_region->position() / speed) + frame_delta;
-               } else {
-                       new_bound = (framepos_t) (_region->position() / speed) - frame_delta;
-               }
-       } else {
-               if (swap_direction) {
-                       new_bound = (framepos_t) (_region->position() / speed) - frame_delta;
-               } else {
-                       new_bound = (framepos_t) (_region->position() / speed) + frame_delta;
-               }
-       }
-
-       _region->trim_start ((framepos_t) (new_bound * speed));
+       _region->move_start (distance);
        region_changed (PropertyChange (ARDOUR::Properties::start));
 }