add (bezier) curves to canvas, use for automation lines; fix issue with rectangles...
[ardour.git] / gtk2_ardour / region_view.cc
index 85770d731eac8a32ab8f17deaec2098a65498a9c..824026ff6e1be3e9498c239e5d06631bcf5f2816 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"
@@ -83,16 +82,18 @@ RegionView::RegionView (ArdourCanvas::Group*              parent,
        , in_destructor(false)
        , wait_for_data(false)
         , _silence_text (0)
-       , _time_converter(r->session().tempo_map(), r->position())
+       , _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)
        : sigc::trackable(other)
        , TimeAxisViewItem (other)
         , _silence_text (0)
-       , _time_converter(other._time_converter)
+       , _region_relative_time_converter(other.region_relative_time_converter())
+       , _source_relative_time_converter(other.source_relative_time_converter())
 {
        /* derived concrete type will call init () */
 
@@ -101,14 +102,15 @@ 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)
        : sigc::trackable(other)
        , TimeAxisViewItem (other)
         , _silence_text (0)
-       , _time_converter(other._time_converter)
+       , _region_relative_time_converter(other_region->session().tempo_map(), other_region->position())
+       , _source_relative_time_converter(other_region->session().tempo_map(), other_region->position() - other_region->start())
 {
        /* this is a pseudo-copy constructor used when dragging regions
           around on the canvas.
@@ -121,7 +123,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,
@@ -143,7 +145,8 @@ RegionView::RegionView (ArdourCanvas::Group*         parent,
        , in_destructor(false)
        , wait_for_data(false)
         , _silence_text (0)
-       , _time_converter(r->session().tempo_map(), r->position())
+       , _region_relative_time_converter(r->session().tempo_map(), r->position())
+       , _source_relative_time_converter(r->session().tempo_map(), r->position() - r->start())
 {
 }
 
@@ -163,26 +166,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));
+               name_pixbuf->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_name_event), name_pixbuf, this));
        }
 
        if (wfd) {
@@ -191,9 +194,9 @@ 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());
+
+       group->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_event), group, this));
 
        set_colors ();
 
@@ -210,7 +213,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;
        }
 
@@ -220,123 +223,110 @@ RegionView::~RegionView ()
 }
 
 void
-RegionView::set_silent_frames (const AudioIntervalResult& silences)
+RegionView::set_silent_frames (const AudioIntervalResult& silences, double /*threshold*/)
 {
         framecnt_t shortest = max_framecnt;
-        framecnt_t shortest_audible = max_framecnt;
-        bool seen_audible = false;
 
        /* remove old silent frames */
         drop_silent_frames ();
 
-        if (!silences.empty()) {
+        if (silences.empty()) {
+                return;
+        }
 
-                uint32_t const color = ARDOUR_UI::config()->canvasvar_Silence.get();
-                framecnt_t last_end;
+        uint32_t const color = ARDOUR_UI::config()->canvasvar_Silence.get();
 
-                if (silences.front().first != 0) {
-                        /* use initial non-silent segment as shortest */
-                        shortest_audible = silences.front().first;
-                        seen_audible = true;
-                }
-                
-                for (AudioIntervalResult::const_iterator i = silences.begin(); i != silences.end(); ++i) {
+       for (AudioIntervalResult::const_iterator i = silences.begin(); i != silences.end(); ++i) {
 
-                        if ((*i).first > last_end) {
-                                /* (audible) gap between the end of the last interval and this one */
-                                shortest_audible = min (shortest_audible, (*i).first - last_end);
-                                seen_audible = true;
-                        }
+               ArdourCanvas::Rectangle* cr = new ArdourCanvas::Rectangle (group);
+               cr->set_ignore_events (true);
+               _silent_frames.push_back (cr);
 
-                        
-                        ArdourCanvas::SimpleRect* cr = new ArdourCanvas::SimpleRect (*group);
-                        _silent_frames.push_back (cr);
+               /* coordinates for the rect are relative to the regionview origin */
 
-                        /* coordinates for the rect are relative to the regionview origin */
+               cr->set_x0 (trackview.editor().frame_to_pixel (i->first - _region->start()));
+               cr->set_x1 (trackview.editor().frame_to_pixel (i->second - _region->start()));
+               cr->set_y0 (1);
+               cr->set_y1 (_height - 2);
+               cr->set_outline (false);
+               cr->set_fill_color (color);
 
-                        cr->property_x1() = trackview.editor().frame_to_pixel ((*i).first - _region->start());
-                        cr->property_y1() = 1;
-                        cr->property_y2() = _height - 2;
-                        cr->property_outline_pixels() = 0;
-                        cr->property_fill_color_rgba () = color;
+               shortest = min (shortest, i->second - i->first);
+       }
 
-                        last_end = (*i).second;
+       /* Find shortest audible segment */
+        framecnt_t shortest_audible = max_framecnt;
 
-                        cr->property_x2() = trackview.editor().frame_to_pixel ((*i).second - _region->start());
+       framecnt_t s = _region->start();
+       for (AudioIntervalResult::const_iterator i = silences.begin(); i != silences.end(); ++i) {
+               framecnt_t const dur = i->first - s;
+               if (dur > 0) {
+                       shortest_audible = min (shortest_audible, dur);
+               }
 
-                        if (((*i).second - (*i).first) < shortest) {
-                                shortest= (*i).second;
-                        }
-                }
-                
-                if (last_end != _region->length()) {
-                        shortest_audible = min (shortest_audible, _region->last_frame() - last_end);
-                        seen_audible = true;
-                }
+               s = i->second;
+       }
 
-                _silence_text = new ArdourCanvas::NoEventText (*group);
-                _silence_text->property_font_desc() = *(get_font_for_style (N_("VerboseCanvasCusor")));
-                _silence_text->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_SilenceText.get();                                                
-                _silence_text->property_anchor() = ANCHOR_NW;
-
-                /* 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;
-
-                double ms;
-                char const * sunits;
-                char const * noun;
-                
-                if (silences.size() > 1) {
-                        noun = _("silent segments");
-                } else {
-                        noun = _("silent segment");
-                }
+       framecnt_t const dur = _region->start() + _region->length() - 1 - s;
+       if (dur > 0) {
+               shortest_audible = min (shortest_audible, dur);
+       }
 
-                ms = (float) shortest/_region->session().frame_rate();
-                
-                /* ms are now in seconds */
+        _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()->canvasvar_SilenceText.get());
 
-                if (ms >= 60.0) {
-                        sunits = _("minutes");
-                        ms /= 60.0;
-                } else if (ms < 1.0) {
-                        sunits = _("msecs");
-                        ms *= 1000.0;
-                } else {
-                        sunits = _("secs");
-                }
+        /* both positions are relative to the region start offset in source */
+
+        _silence_text->set_x_position (trackview.editor().frame_to_pixel (silences.front().first - _region->start()) + 10.0);
+        _silence_text->set_y_position (20.0);
+
+        double ms = (float) shortest/_region->session().frame_rate();
+
+        /* ms are now in seconds */
+
+        char const * sunits;
+
+        if (ms >= 60.0) {
+                sunits = _("minutes");
+                ms /= 60.0;
+        } else if (ms < 1.0) {
+                sunits = _("msecs");
+                ms *= 1000.0;
+        } else {
+                sunits = _("secs");
+        }
+
+       string text = string_compose (ngettext ("%1 silent segment", "%1 silent segments", silences.size()), silences.size())
+               + ", "
+               + string_compose (_("shortest = %1 %2"), ms, sunits);
 
-                if (seen_audible) {
-                        /* ms are now in seconds */
-                        double ma = shortest_audible / _region->session().frame_rate();
-                        char const * aunits;
-
-                        if (ma >= 60.0) {
-                                aunits = _("minutes");
-                                ma /= 60.0;
-                        } else if (ma < 1.0) {
-                                aunits = _("msecs");
-                                ma *= 1000.0;
-                        } else {
-                                aunits = _("secs");
-                        }
-
-                        _silence_text->property_text() = string_compose (_("%1 %2, shortest = %3 %4\n  (shortest audible segment = %5 %6)"),
-                                                                         silences.size(), noun, 
-                                                                         ms, sunits, ma, aunits).c_str();
+        if (shortest_audible != max_framepos) {
+                /* ms are now in seconds */
+                double ma = (float) shortest_audible / _region->session().frame_rate();
+                char const * aunits;
+
+                if (ma >= 60.0) {
+                        aunits = _("minutes");
+                        ma /= 60.0;
+                } else if (ma < 1.0) {
+                        aunits = _("msecs");
+                        ma *= 1000.0;
                 } else {
-                        _silence_text->property_text() = string_compose (_("%1 %2, shortest = %3 %4"),
-                                                                         silences.size(), noun, ms, sunits).c_str();
+                        aunits = _("secs");
                 }
-        } 
+
+               text += string_compose (_("\n  (shortest audible segment = %1 %2)"), ma, aunits);
+       }
+
+       _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();
@@ -345,7 +335,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 ();
@@ -354,15 +344,6 @@ RegionView::drop_silent_frames ()
         _silence_text = 0;
 }
 
-void
-RegionView::show_silent_frames ()
-{
-       for (list<ArdourCanvas::SimpleRect*>::iterator i = _silent_frames.begin (); i != _silent_frames.end (); ++i) {
-                (*i)->show ();
-       }
-        _silence_text->show ();
-}
-
 gint
 RegionView::_lock_toggle (ArdourCanvas::Item*, GdkEvent* ev, void* arg)
 {
@@ -407,6 +388,12 @@ RegionView::region_changed (const PropertyChange& what_changed)
        if (what_changed.contains (ARDOUR::Properties::locked)) {
                region_locked ();
        }
+#ifdef WITH_VIDEOTIMELINE
+       if (what_changed.contains (ARDOUR::Properties::locked)) {
+               /* name will show locked status */
+               region_renamed ();
+       }
+#endif
 }
 
 void
@@ -423,7 +410,11 @@ RegionView::region_resized (const PropertyChange& what_changed)
 
        if (what_changed.contains (ARDOUR::Properties::position)) {
                set_position (_region->position(), 0);
-               _time_converter.set_origin_b (_region->position());
+               _region_relative_time_converter.set_origin_b (_region->position());
+       }
+
+       if (what_changed.contains (ARDOUR::Properties::start) || what_changed.contains (ARDOUR::Properties::position)) {
+               _source_relative_time_converter.set_origin_b (_region->position() - _region->start());
        }
 
        PropertyChange s_and_l;
@@ -434,7 +425,7 @@ RegionView::region_resized (const PropertyChange& what_changed)
 
                set_duration (_region->length(), 0);
 
-               unit_length = _region->length() / samples_per_unit;
+               unit_length = _region->length() / frames_per_pixel;
 
                for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
 
@@ -492,7 +483,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));
                }
        }
 
@@ -500,13 +491,13 @@ RegionView::set_position (framepos_t pos, void* /*src*/, double* ignored)
 }
 
 void
-RegionView::set_samples_per_unit (gdouble spu)
+RegionView::set_frames_per_pixel (double fpp)
 {
-       TimeAxisViewItem::set_samples_per_unit (spu);
+       TimeAxisViewItem::set_frames_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_frames_per_pixel (fpp);
+               (*i)->set_duration (_region->length() / fpp);
        }
 
        region_sync_changed ();
@@ -520,7 +511,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() / frames_per_pixel);
        }
 
        return true;
@@ -532,8 +523,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));
        }
 }
 
@@ -595,6 +587,12 @@ RegionView::make_name () const
                str += '{';
                str += _region->name();
                str += '}';
+#ifdef WITH_VIDEOTIMELINE
+       } else if (_region->video_locked()) {
+               str += '[';
+               str += _region->name();
+               str += ']';
+#endif
        } else {
                str = _region->name();
        }
@@ -637,15 +635,15 @@ 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);
+               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);
+               sync_line->set_outline_color (RGBA_TO_UINT(0,255,0,255)); // FIXME make a themeable colour
+               sync_line->set_outline_width  (1);
        }
 
-       /* 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 frames_per_pixel
           and a change in the bounds of the _region->
         */
 
@@ -673,19 +671,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 / frames_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 ();
                }
        }
@@ -698,12 +692,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));
        }
 }
 
@@ -749,19 +743,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 / frames_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);
        }
 
 }
@@ -772,7 +767,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;
        }
 
@@ -792,11 +787,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()->canvasvar_CoveredRegion.get ();
 
        while (t < end) {
 
@@ -807,23 +802,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().frame_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().frame_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);
@@ -832,7 +827,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().frame_to_pixel (end - position));
        }
 
        if (frame_handle_start) {
@@ -852,11 +847,11 @@ RegionView::update_coverage_frames (LayerDisplay d)
        }
 }
 
-void
+bool
 RegionView::trim_front (framepos_t new_bound, bool no_overlap)
 {
        if (_region->locked()) {
-               return;
+               return false;
        }
 
        RouteTimeAxisView& rtv = dynamic_cast<RouteTimeAxisView&> (trackview);
@@ -864,7 +859,7 @@ RegionView::trim_front (framepos_t new_bound, bool no_overlap)
 
        framepos_t const pre_trim_first_frame = _region->first_frame();
 
-       _region->trim_front ((framepos_t) (new_bound * speed), this);
+       _region->trim_front ((framepos_t) (new_bound * speed));
 
        if (no_overlap) {
                // Get the next region on the left of this region and shrink/expand it.
@@ -879,18 +874,20 @@ RegionView::trim_front (framepos_t new_bound, bool no_overlap)
 
                // Only trim region on the left if the first frame has gone beyond the left region's last frame.
                if (region_left != 0 && (region_left->last_frame() > _region->first_frame() || regions_touching)) {
-                       region_left->trim_end (_region->first_frame() - 1, this);
+                       region_left->trim_end (_region->first_frame() - 1);
                }
        }
 
        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);
@@ -898,7 +895,7 @@ RegionView::trim_end (framepos_t new_bound, bool no_overlap)
 
        framepos_t const pre_trim_last_frame = _region->last_frame();
 
-       _region->trim_end ((framepos_t) (new_bound * speed), this);
+       _region->trim_end ((framepos_t) (new_bound * speed));
 
        if (no_overlap) {
                // Get the next region on the right of this region and shrink/expand it.
@@ -913,14 +910,16 @@ RegionView::trim_end (framepos_t new_bound, bool no_overlap)
 
                // Only trim region on the right if the last frame has gone beyond the right region's first frame.
                if (region_right != 0 && (region_right->first_frame() < _region->last_frame() || regions_touching)) {
-                       region_right->trim_front (_region->last_frame() + 1, this);
+                       region_right->trim_front (_region->last_frame() + 1);
                }
 
                region_changed (ARDOUR::bounds_change);
-                       
+
        } else {
                region_changed (PropertyChange (ARDOUR::Properties::length));
        }
+
+       return (pre_trim_last_frame != _region->last_frame());  //return true if we actually changed something
 }
 
 
@@ -961,6 +960,32 @@ RegionView::trim_contents (framepos_t frame_delta, bool left_direction, bool swa
                }
        }
 
-       _region->trim_start ((framepos_t) (new_bound * speed), this);
+       _region->trim_start ((framepos_t) (new_bound * speed));
        region_changed (PropertyChange (ARDOUR::Properties::start));
 }
+
+/** Snap a frame offset within our region using the current snap settings.
+ *  @param x Frame offset from this region's position.
+ *  @return Snapped frame offset from this region's position.
+ */
+frameoffset_t
+RegionView::snap_frame_to_frame (frameoffset_t x) const
+{
+       PublicEditor& editor = trackview.editor();
+
+       /* x is region relative, convert it to global absolute frames */
+       framepos_t const session_frame = x + _region->position();
+
+       /* try a snap in either direction */
+       framepos_t frame = session_frame;
+       editor.snap_to (frame, 0);
+
+       /* if we went off the beginning of the region, snap forwards */
+       if (frame < _region->position ()) {
+               frame = session_frame;
+               editor.snap_to (frame, 1);
+       }
+
+       /* back to region relative */
+       return frame - _region->position();
+}