first stab at send+route panner link
[ardour.git] / gtk2_ardour / processor_box.cc
index 2cdf1030341d91f1c3dd49c54410ac34870ea90b..c2421dcabf560cd822750a8165dc438d8204e974 100644 (file)
 #include <gtkmm2ext/choice.h>
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/doi.h>
+#include <gtkmm2ext/rgb_macros.h>
 
 #include "ardour/amp.h"
-#include "ardour/ardour.h"
 #include "ardour/audio_track.h"
 #include "ardour/audioengine.h"
-#include "ardour/internal_send.h"
 #include "ardour/internal_return.h"
-#include "ardour/ladspa_plugin.h"
-#include "ardour/meter.h"
+#include "ardour/internal_send.h"
+#include "ardour/panner_shell.h"
 #include "ardour/plugin_insert.h"
 #include "ardour/port_insert.h"
 #include "ardour/profile.h"
@@ -54,7 +53,7 @@
 #include "ardour/route.h"
 #include "ardour/send.h"
 #include "ardour/session.h"
-#include "ardour/dB.h"
+#include "ardour/types.h"
 
 #include "actions.h"
 #include "ardour_dialog.h"
@@ -92,34 +91,89 @@ RefPtr<Action> ProcessorBox::paste_action;
 RefPtr<Action> ProcessorBox::cut_action;
 RefPtr<Action> ProcessorBox::rename_action;
 RefPtr<Action> ProcessorBox::edit_action;
-RefPtr<Action> ProcessorBox::controls_action;
-Glib::RefPtr<Gdk::Pixbuf> SendProcessorEntry::_slider;
+RefPtr<Action> ProcessorBox::edit_generic_action;
+
+static const uint32_t audio_port_color = 0x4A8A0EFF; // Green
+static const uint32_t midi_port_color = 0x960909FF; //Red
 
-ProcessorEntry::ProcessorEntry (boost::shared_ptr<Processor> p, Width w)
+ProcessorEntry::ProcessorEntry (ProcessorBox* parent, boost::shared_ptr<Processor> p, Width w)
        : _button (ArdourButton::led_default_elements)
        , _position (PreFader)
+       , _parent (parent)
        , _processor (p)
        , _width (w)
        , _visual_state (Gtk::STATE_NORMAL)
+       , _input_icon(true)
+       , _output_icon(false)
 {
-       _vbox.pack_start (_button, true, true);
        _vbox.show ();
        
-       if (_processor->active()) {
-               _button.set_active_state (Gtkmm2ext::Active);
-       }
        _button.set_diameter (3);
        _button.set_distinct_led_click (true);
        _button.set_led_left (true);
        _button.signal_led_clicked.connect (sigc::mem_fun (*this, &ProcessorEntry::led_clicked));
        _button.set_text (name (_width));
-       _button.show ();
 
-       _processor->ActiveChanged.connect (active_connection, invalidator (*this), boost::bind (&ProcessorEntry::processor_active_changed, this), gui_context());
-       _processor->PropertyChanged.connect (name_connection, invalidator (*this), ui_bind (&ProcessorEntry::processor_property_changed, this, _1), gui_context());
+       if (_processor) {
 
-       setup_tooltip ();
-       setup_visuals ();
+               _vbox.pack_start (_routing_icon);
+               _vbox.pack_start (_input_icon);
+               _vbox.pack_start (_button, true, true);
+               _vbox.pack_end (_output_icon);
+
+               _button.set_active (_processor->active());
+
+               _routing_icon.set_no_show_all(true);
+               _input_icon.set_no_show_all(true);
+
+               _button.show ();
+               _routing_icon.set_visible(false);
+               _input_icon.hide();
+               _output_icon.show();
+
+               _processor->ActiveChanged.connect (active_connection, invalidator (*this), boost::bind (&ProcessorEntry::processor_active_changed, this), gui_context());
+               _processor->PropertyChanged.connect (name_connection, invalidator (*this), boost::bind (&ProcessorEntry::processor_property_changed, this, _1), gui_context());
+               _processor->ConfigurationChanged.connect (config_connection, invalidator (*this), boost::bind (&ProcessorEntry::processor_configuration_changed, this, _1, _2), gui_context());
+
+               set<Evoral::Parameter> p = _processor->what_can_be_automated ();
+               for (set<Evoral::Parameter>::iterator i = p.begin(); i != p.end(); ++i) {
+                       
+                       std::string label = _processor->describe_parameter (*i);
+
+                       if (boost::dynamic_pointer_cast<Send> (_processor)) {
+                               label = _("Send");
+                       } else if (boost::dynamic_pointer_cast<Return> (_processor)) {
+                               label = _("Return");
+                       }
+
+                       Control* c = new Control (_processor->automation_control (*i), label);
+                       
+                       _controls.push_back (c);
+
+                       if (boost::dynamic_pointer_cast<Amp> (_processor) == 0) {
+                               /* Add non-Amp controls to the processor box */
+                               _vbox.pack_start (c->box);
+                       }
+               }
+
+               _input_icon.set_ports(_processor->input_streams());
+               _output_icon.set_ports(_processor->output_streams());
+
+               _routing_icon.set_sources(_processor->input_streams());
+               _routing_icon.set_sinks(_processor->output_streams());
+
+               setup_tooltip ();
+               setup_visuals ();
+       } else {
+               _vbox.set_size_request (-1, _button.size_request().height);
+       }
+}
+
+ProcessorEntry::~ProcessorEntry ()
+{
+       for (list<Control*>::iterator i = _controls.begin(); i != _controls.end(); ++i) {
+               delete *i;
+       }
 }
 
 EventBox&
@@ -141,9 +195,17 @@ ProcessorEntry::drag_text () const
 }
 
 void
-ProcessorEntry::set_position (Position p)
+ProcessorEntry::set_position (Position p, uint32_t num)
 {
        _position = p;
+       _position_num = num;
+
+       if (_position_num == 0 || _routing_icon.get_visible()) {
+               _input_icon.show();
+       } else {
+               _input_icon.hide();
+       }
+
        setup_visuals ();
 }
 
@@ -186,25 +248,26 @@ void
 ProcessorEntry::set_enum_width (Width w)
 {
        _width = w;
+       _button.set_text (name (_width));
 }
 
 void
 ProcessorEntry::led_clicked()
 {
-       if (_button.active_state() == Gtkmm2ext::Active) {
-               _processor->deactivate ();
-       } else {
-               _processor->activate ();
+       if (_processor) {
+               if (_button.get_active ()) {
+                       _processor->deactivate ();
+               } else {
+                       _processor->activate ();
+               }
        }
 }
 
 void
 ProcessorEntry::processor_active_changed ()
 {
-       if (_processor->active()) {
-               _button.set_active_state (Gtkmm2ext::Active);
-       } else {
-               _button.unset_active_state ();
+       if (_processor) {
+               _button.set_active (_processor->active());
        }
 }
 
@@ -217,10 +280,40 @@ ProcessorEntry::processor_property_changed (const PropertyChange& what_changed)
        }
 }
 
+void
+ProcessorEntry::processor_configuration_changed (const ChanCount in, const ChanCount out)
+{
+       _input_icon.set_ports(in);
+       _output_icon.set_ports(out);
+       _routing_icon.set_sources(in);
+       _routing_icon.set_sinks(out);
+       _input_icon.queue_draw();
+       _output_icon.queue_draw();
+       _routing_icon.queue_draw();
+}
+
 void
 ProcessorEntry::setup_tooltip ()
 {
-       ARDOUR_UI::instance()->set_tip (_button, name (Wide));
+       if (_processor) {
+               boost::shared_ptr<PluginInsert> pi = boost::dynamic_pointer_cast<PluginInsert> (_processor);
+               if (pi) {
+                       std::string postfix = "";
+                       uint32_t replicated;
+                       if ((replicated = pi->get_count()) > 1) {
+                               postfix = string_compose(_("\nThis mono plugin has been replicated %1 times."), replicated);
+                       }
+                       if (pi->plugin()->has_editor()) {
+                               ARDOUR_UI::instance()->set_tip (_button,
+                                               string_compose (_("<b>%1</b>\nDouble-click to show GUI.\nAlt+double-click to show generic GUI.%2"), name (Wide), postfix));
+                       } else {
+                               ARDOUR_UI::instance()->set_tip (_button,
+                                               string_compose (_("<b>%1</b>\nDouble-click to show generic GUI.%2"), name (Wide), postfix));
+                       }
+                       return;
+               }
+       }
+       ARDOUR_UI::instance()->set_tip (_button, string_compose ("<b>%1</b>", name (Wide)));
 }
 
 string
@@ -229,6 +322,10 @@ ProcessorEntry::name (Width w) const
        boost::shared_ptr<Send> send;
        string name_display;
 
+       if (!_processor) {
+               return string();
+       }
+
        if ((send = boost::dynamic_pointer_cast<Send> (_processor)) != 0 &&
            !boost::dynamic_pointer_cast<InternalSend>(_processor)) {
 
@@ -250,6 +347,13 @@ ProcessorEntry::name (Width w) const
                }
 
        } else {
+               boost::shared_ptr<ARDOUR::PluginInsert> pi;
+               uint32_t replicated;
+               if ((pi = boost::dynamic_pointer_cast<ARDOUR::PluginInsert> (_processor)) != 0
+                               && (replicated = pi->get_count()) > 1)
+               {
+                       name_display += string_compose(_("(%1x1) "), replicated);
+               }
 
                switch (w) {
                case Wide:
@@ -265,130 +369,319 @@ ProcessorEntry::name (Width w) const
        return name_display;
 }
 
-SendProcessorEntry::SendProcessorEntry (boost::shared_ptr<Send> s, Width w)
-       : ProcessorEntry (s, w)
-       , _send (s)
-       , _adjustment (gain_to_slider_position_with_max (1.0, Config->get_max_gain()), 0, 1, 0.01, 0.1)
-       , _fader (_slider, &_adjustment, 0, false)
-       , _ignore_gain_change (false)
-       , _data_type (DataType::AUDIO)
+void
+ProcessorEntry::show_all_controls ()
+{
+       for (list<Control*>::iterator i = _controls.begin(); i != _controls.end(); ++i) {
+               (*i)->set_visible (true);
+       }
+
+       _parent->update_gui_object_state (this);
+}
+
+void
+ProcessorEntry::hide_all_controls ()
+{
+       for (list<Control*>::iterator i = _controls.begin(); i != _controls.end(); ++i) {
+               (*i)->set_visible (false);
+       }
+
+       _parent->update_gui_object_state (this);
+}
+
+void
+ProcessorEntry::add_control_state (XMLNode* node) const
+{
+       for (list<Control*>::const_iterator i = _controls.begin(); i != _controls.end(); ++i) {
+               (*i)->add_state (node);
+       }
+}
+
+void
+ProcessorEntry::set_control_state (XMLNode const * node)
+{
+       for (list<Control*>::const_iterator i = _controls.begin(); i != _controls.end(); ++i) {
+               (*i)->set_state (node);
+       }
+}
+
+string
+ProcessorEntry::state_id () const
+{
+       return string_compose ("processor %1", _processor->id().to_s());
+}
+
+void
+ProcessorEntry::hide_things ()
 {
-       _fader.set_name ("SendFader");
-       _fader.set_controllable (_send->amp()->gain_control ());
-       _vbox.pack_start (_fader);
+       for (list<Control*>::iterator i = _controls.begin(); i != _controls.end(); ++i) {
+               (*i)->hide_things ();
+       }
+}
 
-       _fader.show ();
 
-       _adjustment.signal_value_changed().connect (sigc::mem_fun (*this, &SendProcessorEntry::gain_adjusted));
+Menu *
+ProcessorEntry::build_controls_menu ()
+{
+       using namespace Menu_Helpers;
+       Menu* menu = manage (new Menu);
+       MenuList& items = menu->items ();
 
-       _send->amp()->gain_control()->Changed.connect (
-               _send_connections, invalidator (*this), boost::bind (&SendProcessorEntry::show_gain, this), gui_context()
+       items.push_back (
+               MenuElem (_("Show All Controls"), sigc::mem_fun (*this, &ProcessorEntry::show_all_controls))
                );
-       
-       _send->amp()->ConfigurationChanged.connect (
-               _send_connections, invalidator (*this), ui_bind (&SendProcessorEntry::setup_gain_adjustment, this), gui_context ()
+               
+       items.push_back (
+               MenuElem (_("Hide All Controls"), sigc::mem_fun (*this, &ProcessorEntry::hide_all_controls))
                );
 
-       setup_gain_adjustment ();
-       show_gain ();
+       if (!_controls.empty ()) {
+               items.push_back (SeparatorElem ());
+       }
+       
+       for (list<Control*>::iterator i = _controls.begin(); i != _controls.end(); ++i) {
+               items.push_back (CheckMenuElem ((*i)->name ()));
+               CheckMenuItem* c = dynamic_cast<CheckMenuItem*> (&items.back ());
+               c->set_active ((*i)->visible ());
+               c->signal_toggled().connect (sigc::bind (sigc::mem_fun (*this, &ProcessorEntry::toggle_control_visibility), *i));
+       }
+
+       return menu;
+}
+
+void
+ProcessorEntry::toggle_control_visibility (Control* c)
+{
+       c->set_visible (!c->visible ());
+       _parent->update_gui_object_state (this);
+}
+
+Menu *
+ProcessorEntry::build_send_options_menu ()
+{
+       using namespace Menu_Helpers;
+       Menu* menu = manage (new Menu);
+       MenuList& items = menu->items ();
+
+       boost::shared_ptr<Send> send = boost::dynamic_pointer_cast<Send> (_processor);
+       if (send) {
+
+               items.push_back (CheckMenuElem (_("Link panner controls")));
+               CheckMenuItem* c = dynamic_cast<CheckMenuItem*> (&items.back ());
+               c->set_active (send->panner_shell()->is_linked_to_route());
+               c->signal_toggled().connect (sigc::mem_fun (*this, &ProcessorEntry::toggle_panner_link));
+
+       }
+       return menu;
 }
 
 void
-SendProcessorEntry::setup_gain_adjustment ()
+ProcessorEntry::toggle_panner_link ()
 {
-       if (_send->amp()->output_streams().n_midi() == 0) {
-               _data_type = DataType::AUDIO;
-               _adjustment.set_lower (0);
-               _adjustment.set_upper (1);
-               _adjustment.set_step_increment (0.01);
-               _adjustment.set_page_increment (0.1);
-               _fader.set_default_value (gain_to_slider_position (1));
+       boost::shared_ptr<Send> send = boost::dynamic_pointer_cast<Send> (_processor);
+       if (send) {
+               send->panner_shell()->set_linked_to_route(!send->panner_shell()->is_linked_to_route());
+       }
+}
+
+ProcessorEntry::Control::Control (boost::shared_ptr<AutomationControl> c, string const & n)
+       : _control (c)
+       , _adjustment (gain_to_slider_position_with_max (1.0, Config->get_max_gain()), 0, 1, 0.01, 0.1)
+       , _slider (&_adjustment, 0, 13, false)
+       , _slider_persistant_tooltip (&_slider)
+       , _button (ArdourButton::led_default_elements)
+       , _ignore_ui_adjustment (false)
+       , _visible (false)
+       , _name (n)
+{
+       _slider.set_controllable (c);
+       box.set_padding(0, 0, 4, 4);
+
+       if (c->toggled()) {
+               _button.set_text (_name);
+               _button.set_led_left (true);
+               _button.set_name ("processor control button");
+               box.add (_button);
+               _button.show ();
+
+               _button.signal_clicked.connect (sigc::mem_fun (*this, &Control::button_clicked));
+               _button.signal_led_clicked.connect (sigc::mem_fun (*this, &Control::button_clicked));
+               c->Changed.connect (_connection, MISSING_INVALIDATOR, boost::bind (&Control::control_changed, this), gui_context ());
+
        } else {
-               _data_type = DataType::MIDI;
-               _adjustment.set_lower (0);
-               _adjustment.set_upper (2);
-               _adjustment.set_step_increment (0.05);
-               _adjustment.set_page_increment (0.1);
-               _fader.set_default_value (1);
+               
+               _slider.set_name ("ProcessorControlSlider");
+               _slider.set_text (_name);
+
+               box.add (_slider);
+               _slider.show ();
+
+               double const lo = c->internal_to_interface (c->lower ());
+               double const up = c->internal_to_interface (c->upper ());
+               
+               _adjustment.set_lower (lo);
+               _adjustment.set_upper (up);
+               _adjustment.set_step_increment ((up - lo) / 100);
+               _adjustment.set_page_increment ((up - lo) / 10);
+               _slider.set_default_value (c->internal_to_interface (c->normal ()));
+               
+               _adjustment.signal_value_changed().connect (sigc::mem_fun (*this, &Control::slider_adjusted));
+               c->Changed.connect (_connection, MISSING_INVALIDATOR, boost::bind (&Control::control_changed, this), gui_context ());
        }
+
+       ARDOUR_UI::RapidScreenUpdate.connect (sigc::mem_fun (*this, &Control::control_changed));
+       
+       control_changed ();
+       set_tooltip ();
+
+       /* We're providing our own PersistentTooltip */
+       set_no_tooltip_whatsoever (_slider);
 }
 
 void
-SendProcessorEntry::setup_slider_pix ()
+ProcessorEntry::Control::set_tooltip ()
 {
-       _slider = ::get_icon ("fader_belt_h_thin");
-       assert (_slider);
+       boost::shared_ptr<AutomationControl> c = _control.lock ();
+
+       if (!c) {
+               return;
+       }
+       
+       stringstream s;
+       s << _name << ": ";
+       if (c->toggled ()) {
+               s << (c->get_value() > 0.5 ? _("on") : _("off"));
+       } else {
+               s << setprecision(2) << fixed;
+               s << c->internal_to_user (c->get_value ());
+       }
+
+       string sm = Glib::Markup::escape_text (s.str());
+       
+       _slider_persistant_tooltip.set_tip (sm);
+       ARDOUR_UI::instance()->set_tip (_button, sm);
 }
 
 void
-SendProcessorEntry::show_gain ()
+ProcessorEntry::Control::slider_adjusted ()
 {
-       gain_t value = 0;
+       if (_ignore_ui_adjustment) {
+               return;
+       }
        
-       switch (_data_type) {
-       case DataType::AUDIO:
-               value = gain_to_slider_position_with_max (_send->amp()->gain (), Config->get_max_gain());
-               break;
-       case DataType::MIDI:
-               value = _send->amp()->gain ();
-               break;
+       boost::shared_ptr<AutomationControl> c = _control.lock ();
+
+       if (!c) {
+               return;
+       }
+
+       c->set_value (c->interface_to_internal (_adjustment.get_value ()));
+       set_tooltip ();
+}
+
+void
+ProcessorEntry::Control::button_clicked ()
+{
+       boost::shared_ptr<AutomationControl> c = _control.lock ();
+
+       if (!c) {
+               return;
+       }
+
+       bool const n = _button.get_active ();
+
+       c->set_value (n ? 0 : 1);
+       _button.set_active (!n);
+       set_tooltip ();
+}
+
+void
+ProcessorEntry::Control::control_changed ()
+{
+       boost::shared_ptr<AutomationControl> c = _control.lock ();
+       if (!c) {
+               return;
        }
 
-       if (_adjustment.get_value() != value) {
-               _ignore_gain_change = true;
-               _adjustment.set_value (value);
-               _ignore_gain_change = false;
+       _ignore_ui_adjustment = true;
 
+       if (c->toggled ()) {
+
+               _button.set_active (c->get_value() > 0.5);
+               
+       } else {
+
+               _adjustment.set_value (c->internal_to_interface (c->get_value ()));
+               
                stringstream s;
                s.precision (1);
                s.setf (ios::fixed, ios::floatfield);
-               s << accurate_coefficient_to_dB (_send->amp()->gain ());
-               if (_data_type == DataType::AUDIO) {
-                       s << _("dB");
-               }
-               
-               _fader.set_tooltip_text (s.str ());
+               s << c->internal_to_user (c->get_value ());
        }
+       
+       _ignore_ui_adjustment = false;
 }
 
 void
-SendProcessorEntry::gain_adjusted ()
+ProcessorEntry::Control::add_state (XMLNode* node) const
 {
-       if (_ignore_gain_change) {
-               return;
-       }
+       XMLNode* c = new XMLNode (X_("Object"));
+       c->add_property (X_("id"), state_id ());
+       c->add_property (X_("visible"), _visible);
+       node->add_child_nocopy (*c);
+}
 
-       gain_t value = 0;
+void
+ProcessorEntry::Control::set_state (XMLNode const * node)
+{
+       XMLNode* n = GUIObjectState::get_node (node, state_id ());
+       if (n) {
+               XMLProperty* p = n->property (X_("visible"));
+               set_visible (p && string_is_affirmative (p->value ()));
+       } else {
+               set_visible (false);
+       }
+}
 
-       switch (_data_type) {
-       case DataType::AUDIO:
-               value = slider_position_to_gain_with_max (_adjustment.get_value(), Config->get_max_gain());
-               break;
-       case DataType::MIDI:
-               value = _adjustment.get_value ();
+void
+ProcessorEntry::Control::set_visible (bool v)
+{
+       if (v) {
+               box.show ();
+       } else {
+               box.hide ();
        }
        
-       _send->amp()->set_gain (value, this);
+       _visible = v;
 }
 
+/** Called when the Editor might have re-shown things that
+    we want hidden.
+*/
 void
-SendProcessorEntry::set_pixel_width (int p)
+ProcessorEntry::Control::hide_things ()
 {
-       _fader.set_fader_length (p);
+       if (!_visible) {
+               box.hide ();
+       }
 }
 
-PluginInsertProcessorEntry::PluginInsertProcessorEntry (boost::shared_ptr<ARDOUR::PluginInsert> p, Width w)
-       : ProcessorEntry (p, w)
-       , _plugin_insert (p)
+string
+ProcessorEntry::Control::state_id () const
 {
-       p->SplittingChanged.connect (
-               _splitting_connection, invalidator (*this), ui_bind (&PluginInsertProcessorEntry::plugin_insert_splitting_changed, this), gui_context()
-               );
+       boost::shared_ptr<AutomationControl> c = _control.lock ();
+       assert (c);
 
-       _splitting_icon.set_size_request (-1, 12);
+       return string_compose (X_("control %1"), c->id().to_s ());
+}
 
-       _vbox.pack_start (_splitting_icon);
-       _vbox.reorder_child (_splitting_icon, 0);
+PluginInsertProcessorEntry::PluginInsertProcessorEntry (ProcessorBox* b, boost::shared_ptr<ARDOUR::PluginInsert> p, Width w)
+       : ProcessorEntry (b, p, w)
+       , _plugin_insert (p)
+{
+       p->PluginIoReConfigure.connect (
+               _splitting_connection, invalidator (*this), boost::bind (&PluginInsertProcessorEntry::plugin_insert_splitting_changed, this), gui_context()
+               );
 
        plugin_insert_splitting_changed ();
 }
@@ -396,48 +689,127 @@ PluginInsertProcessorEntry::PluginInsertProcessorEntry (boost::shared_ptr<ARDOUR
 void
 PluginInsertProcessorEntry::plugin_insert_splitting_changed ()
 {
-       if (_plugin_insert->splitting ()) {
-               _splitting_icon.show ();
+       _output_icon.set_ports(_plugin_insert->output_streams());
+       _routing_icon.set_splitting(_plugin_insert->splitting ());
+
+       ChanCount sources = _plugin_insert->input_streams();
+       ChanCount sinks = _plugin_insert->natural_input_streams();
+
+       /* replicated instances */
+       if (!_plugin_insert->splitting () && _plugin_insert->get_count() > 1) {
+               for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
+                       sinks.set(*t, sinks.get(*t) * _plugin_insert->get_count());
+               }
+       }
+       /* MIDI bypass */
+       if (_plugin_insert->natural_output_streams().n_midi() == 0 &&
+                       _plugin_insert->output_streams().n_midi() == 1) {
+               sinks.set(DataType::MIDI, 1);
+               sources.set(DataType::MIDI, 1);
+       }
+
+       _input_icon.set_ports(sinks);
+       _routing_icon.set_sinks(sinks);
+       _routing_icon.set_sources(sources);
+
+       if (_plugin_insert->splitting () ||
+                       _plugin_insert->input_streams().n_audio() < _plugin_insert->natural_input_streams().n_audio()
+                )
+       {
+               _routing_icon.set_size_request (-1, 7);
+               _routing_icon.set_visible(true);
+               _input_icon.show();
        } else {
-               _splitting_icon.hide ();
+               _routing_icon.set_visible(false);
+               if (_position_num != 0) {
+                       _input_icon.hide();
+               }
        }
+
+       _input_icon.queue_draw();
+       _output_icon.queue_draw();
+       _routing_icon.queue_draw();
 }
 
 void
 PluginInsertProcessorEntry::hide_things ()
 {
+       ProcessorEntry::hide_things ();
        plugin_insert_splitting_changed ();
 }
 
-void
-PluginInsertProcessorEntry::setup_visuals ()
+
+bool
+ProcessorEntry::PortIcon::on_expose_event (GdkEventExpose* ev)
 {
-       switch (_position) {
-       case PreFader:
-               _splitting_icon.set_name ("ProcessorPreFader");
-               break;
+       cairo_t* cr = gdk_cairo_create (get_window()->gobj());
 
-       case Fader:
-               _splitting_icon.set_name ("ProcessorFader");
-               break;
+       cairo_rectangle (cr, ev->area.x, ev->area.y, ev->area.width, ev->area.height);
+       cairo_clip (cr);
 
-       case PostFader:
-               _splitting_icon.set_name ("ProcessorPostFader");
-               break;
+       Gtk::Allocation a = get_allocation();
+       double const width = a.get_width();
+       double const height = a.get_height();
+
+       Gdk::Color const bg = get_style()->get_bg (STATE_NORMAL);
+       cairo_set_source_rgb (cr, bg.get_red_p (), bg.get_green_p (), bg.get_blue_p ());
+
+       cairo_rectangle (cr, 0, 0, width, height);
+       cairo_fill (cr);
+
+       if (_ports.n_total() > 1) {
+               for (uint32_t i = 0; i < _ports.n_total(); ++i) {
+                       if (i < _ports.n_midi()) {
+                               cairo_set_source_rgb (cr,
+                                               UINT_RGBA_R_FLT(midi_port_color),
+                                               UINT_RGBA_G_FLT(midi_port_color),
+                                               UINT_RGBA_B_FLT(midi_port_color));
+                       } else {
+                               cairo_set_source_rgb (cr,
+                                               UINT_RGBA_R_FLT(audio_port_color),
+                                               UINT_RGBA_G_FLT(audio_port_color),
+                                               UINT_RGBA_B_FLT(audio_port_color));
+                       }
+                       const float x = rintf(width * (.2f + .6f * i / (_ports.n_total() - 1.f)));
+                       cairo_rectangle (cr, x-1, 0, 3, height);
+                       cairo_fill(cr);
+               }
+       } else if (_ports.n_total() == 1) {
+               if (_ports.n_midi() == 1) {
+                       cairo_set_source_rgb (cr,
+                                       UINT_RGBA_R_FLT(midi_port_color),
+                                       UINT_RGBA_G_FLT(midi_port_color),
+                                       UINT_RGBA_B_FLT(midi_port_color));
+               } else {
+                       cairo_set_source_rgb (cr,
+                                       UINT_RGBA_R_FLT(audio_port_color),
+                                       UINT_RGBA_G_FLT(audio_port_color),
+                                       UINT_RGBA_B_FLT(audio_port_color));
+               }
+               const float x = rintf(width * .5);
+               cairo_rectangle (cr, x-1, 0, 3, height);
+               cairo_fill(cr);
+               cairo_stroke(cr);
        }
 
-       ProcessorEntry::setup_visuals ();
+       cairo_destroy(cr);
+       return true;
 }
 
 bool
-PluginInsertProcessorEntry::SplittingIcon::on_expose_event (GdkEventExpose* ev)
+ProcessorEntry::RoutingIcon::on_expose_event (GdkEventExpose* ev)
 {
        cairo_t* cr = gdk_cairo_create (get_window()->gobj());
 
-       cairo_set_line_width (cr, 1);
+       cairo_rectangle (cr, ev->area.x, ev->area.y, ev->area.width, ev->area.height);
+       cairo_clip (cr);
 
-       double const width = ev->area.width;
-       double const height = ev->area.height;
+       cairo_set_line_width (cr, 1.0);
+       cairo_set_line_cap (cr, CAIRO_LINE_CAP_ROUND);
+
+       Gtk::Allocation a = get_allocation();
+       double const width = a.get_width();
+       double const height = a.get_height();
 
        Gdk::Color const bg = get_style()->get_bg (STATE_NORMAL);
        cairo_set_source_rgb (cr, bg.get_red_p (), bg.get_green_p (), bg.get_blue_p ());
@@ -448,31 +820,83 @@ PluginInsertProcessorEntry::SplittingIcon::on_expose_event (GdkEventExpose* ev)
        Gdk::Color const fg = get_style()->get_fg (STATE_NORMAL);
        cairo_set_source_rgb (cr, fg.get_red_p (), fg.get_green_p (), fg.get_blue_p ());
 
-       cairo_move_to (cr, width * 0.3, height);
-       cairo_line_to (cr, width * 0.3, height * 0.5);
-       cairo_line_to (cr, width * 0.7, height * 0.5);
-       cairo_line_to (cr, width * 0.7, height);
-       cairo_move_to (cr, width * 0.5, height * 0.5);
-       cairo_line_to (cr, width * 0.5, 0);
-       cairo_stroke (cr);
-
+       const uint32_t sources = _sources.n_total();
+       const uint32_t sinks = _sinks.n_total();
+
+       /* MIDI */
+       const uint32_t midi_sources = _sources.n_midi();
+       const uint32_t midi_sinks = _sinks.n_midi();
+
+       cairo_set_source_rgb (cr,
+                       UINT_RGBA_R_FLT(midi_port_color),
+                       UINT_RGBA_G_FLT(midi_port_color),
+                       UINT_RGBA_B_FLT(midi_port_color));
+       if (midi_sources > 0 && midi_sinks > 0 && sinks > 1 && sources > 1) {
+               for (uint32_t i = 0 ; i < midi_sources; ++i) {
+                       const float si_x  = rintf(width * (.2f + .6f * i  / (sinks - 1.f))) + .5f;
+                       const float si_x0 = rintf(width * (.2f + .6f * i / (sources - 1.f))) + .5f;
+                       cairo_move_to (cr, si_x, height);
+                       cairo_curve_to (cr, si_x, 0, si_x0, height, si_x0, 0);
+                       cairo_stroke (cr);
+               }
+       } else if (midi_sources == 1 && midi_sinks == 1 && sinks == 1 && sources == 1) {
+               const float si_x = rintf(width * .5f) + .5f;
+               cairo_move_to (cr, si_x, height);
+               cairo_line_to (cr, si_x, 0);
+               cairo_stroke (cr);
+       }
+
+       /* AUDIO */
+       const uint32_t audio_sources = _sources.n_audio();
+       const uint32_t audio_sinks = _sinks.n_audio();
+       cairo_set_source_rgb (cr,
+                       UINT_RGBA_R_FLT(audio_port_color),
+                       UINT_RGBA_G_FLT(audio_port_color),
+                       UINT_RGBA_B_FLT(audio_port_color));
+
+       if (_splitting) {
+               assert(audio_sinks > 1);
+               const float si_x0 = rintf(width * .5f) + .5f;
+               for (uint32_t i = midi_sinks; i < sinks; ++i) {
+                       const float si_x = rintf(width * (.2f + .6f * i / (sinks - 1.f))) + .5f;
+                       cairo_move_to (cr, si_x, height);
+                       cairo_curve_to (cr, si_x, 0, si_x0, height, si_x0, 0);
+                       cairo_stroke (cr);
+               }
+       } else if (audio_sources > 1) {
+               for (uint32_t i = 0 ; i < audio_sources; ++i) {
+                       const float si_x = rintf(width * (.2f + .6f * (i + midi_sinks) / (sinks - 1.f))) + .5f;
+                       const float si_x0 = rintf(width * (.2f + .6f * (i + midi_sources) / (sources - 1.f))) + .5f;
+                       cairo_move_to (cr, si_x, height);
+                       cairo_curve_to (cr, si_x, 0, si_x0, height, si_x0, 0);
+                       cairo_stroke (cr);
+               }
+       } else if (audio_sources == 1 && audio_sinks == 1) {
+               const float si_x = rintf(width * .5f) + .5f;
+               cairo_move_to (cr, si_x, height);
+               cairo_line_to (cr, si_x, 0);
+               cairo_stroke (cr);
+       }
+       cairo_destroy(cr);
        return true;
 }
 
 ProcessorBox::ProcessorBox (ARDOUR::Session* sess, boost::function<PluginSelector*()> get_plugin_selector,
-                           RouteRedirectSelection& rsel, MixerStrip* parent, bool owner_is_mixer)
+                           RouteProcessorSelection& rsel, MixerStrip* parent, bool owner_is_mixer)
        : _parent_strip (parent)
        , _owner_is_mixer (owner_is_mixer)
        , ab_direction (true)
        , _get_plugin_selector (get_plugin_selector)
        , _placement (-1)
+       , _visible_prefader_processors (0)
        , _rr_selection(rsel)
+       , _redisplay_pending (false)
+
 {
        set_session (sess);
 
        _width = Wide;
        processor_menu = 0;
-       send_action_menu = 0;
        no_processor_redisplay = false;
 
        processor_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
@@ -481,35 +905,42 @@ ProcessorBox::ProcessorBox (ARDOUR::Session* sess, boost::function<PluginSelecto
 
        processor_display.set_flags (CAN_FOCUS);
        processor_display.set_name ("ProcessorList");
-       processor_display.set_size_request (48, -1);
        processor_display.set_data ("processorbox", this);
-       processor_display.set_spacing (2);
+       processor_display.set_size_request (48, -1);
+       processor_display.set_spacing (0);
 
        processor_display.signal_enter_notify_event().connect (sigc::mem_fun(*this, &ProcessorBox::enter_notify), false);
        processor_display.signal_leave_notify_event().connect (sigc::mem_fun(*this, &ProcessorBox::leave_notify), false);
 
-       processor_display.signal_key_press_event().connect (sigc::mem_fun(*this, &ProcessorBox::processor_key_press_event));
-       processor_display.signal_key_release_event().connect (sigc::mem_fun(*this, &ProcessorBox::processor_key_release_event));
-
        processor_display.ButtonPress.connect (sigc::mem_fun (*this, &ProcessorBox::processor_button_press_event));
        processor_display.ButtonRelease.connect (sigc::mem_fun (*this, &ProcessorBox::processor_button_release_event));
 
        processor_display.Reordered.connect (sigc::mem_fun (*this, &ProcessorBox::reordered));
        processor_display.DropFromAnotherBox.connect (sigc::mem_fun (*this, &ProcessorBox::object_drop));
-       processor_display.SelectionChanged.connect (sigc::mem_fun (*this, &ProcessorBox::selection_changed));
 
        processor_scroller.show ();
        processor_display.show ();
 
        if (parent) {
                parent->DeliveryChanged.connect (
-                       _mixer_strip_connections, invalidator (*this), ui_bind (&ProcessorBox::mixer_strip_delivery_changed, this, _1), gui_context ()
+                       _mixer_strip_connections, invalidator (*this), boost::bind (&ProcessorBox::mixer_strip_delivery_changed, this, _1), gui_context ()
                        );
        }
+
+       ARDOUR_UI::instance()->set_tip (processor_display, _("Right-click to add/remove/edit\nplugins,inserts,sends and more"));
 }
 
 ProcessorBox::~ProcessorBox ()
 {
+       /* it may appear as if we should delete processor_menu but that is a
+        * pointer to a widget owned by the UI Manager, and has potentially
+        * be returned to many other ProcessorBoxes. GTK doesn't really make
+        * clear the ownership of this widget, which is a menu and thus is
+        * never packed into any container other than an implict GtkWindow.
+        *
+        * For now, until or if we ever get clarification over the ownership
+        * story just let it continue to exist. At worst, its a small memory leak.
+        */
 }
 
 void
@@ -526,7 +957,7 @@ ProcessorBox::set_route (boost::shared_ptr<Route> r)
        _route = r;
 
        _route->processors_changed.connect (
-               _route_connections, invalidator (*this), ui_bind (&ProcessorBox::route_processors_changed, this, _1), gui_context()
+               _route_connections, invalidator (*this), boost::bind (&ProcessorBox::route_processors_changed, this, _1), gui_context()
                );
 
        _route->DropReferences.connect (
@@ -534,7 +965,7 @@ ProcessorBox::set_route (boost::shared_ptr<Route> r)
                );
 
        _route->PropertyChanged.connect (
-               _route_connections, invalidator (*this), ui_bind (&ProcessorBox::route_property_changed, this, _1), gui_context()
+               _route_connections, invalidator (*this), boost::bind (&ProcessorBox::route_property_changed, this, _1), gui_context()
                );
 
        redisplay_processors ();
@@ -554,12 +985,25 @@ ProcessorBox::object_drop(DnDVBox<ProcessorEntry>* source, ProcessorEntry* posit
        boost::shared_ptr<Processor> p;
        if (position) {
                p = position->processor ();
+               if (!p) {
+                       /* dropped on the blank entry (which will be before the
+                          fader), so use the first non-blank child as our
+                          `dropped on' processor */
+                       list<ProcessorEntry*> c = processor_display.children ();
+                       list<ProcessorEntry*>::iterator i = c.begin ();
+
+                       assert (i != c.end ());
+                       p = (*i)->processor ();
+                       assert (p);
+               }
        }
 
        list<ProcessorEntry*> children = source->selection ();
        list<boost::shared_ptr<Processor> > procs;
        for (list<ProcessorEntry*>::const_iterator i = children.begin(); i != children.end(); ++i) {
-               procs.push_back ((*i)->processor ());
+               if ((*i)->processor ()) {
+                       procs.push_back ((*i)->processor ());
+               }
        }
 
        for (list<boost::shared_ptr<Processor> >::const_iterator i = procs.begin(); i != procs.end(); ++i) {
@@ -577,18 +1021,11 @@ ProcessorBox::object_drop(DnDVBox<ProcessorEntry>* source, ProcessorEntry* posit
        if ((context->get_suggested_action() == Gdk::ACTION_MOVE) && source) {
                ProcessorBox* other = reinterpret_cast<ProcessorBox*> (source->get_data ("processorbox"));
                if (other) {
-                       cerr << "source was another processor box, delete the selected items\n";
                        other->delete_dragged_processors (procs);
                }
        }
 }
 
-void
-ProcessorBox::update()
-{
-       redisplay_processors ();
-}
-
 void
 ProcessorBox::set_width (Width w)
 {
@@ -603,20 +1040,7 @@ ProcessorBox::set_width (Width w)
                (*i)->set_enum_width (w);
        }
 
-       redisplay_processors ();
-}
-
-void
-ProcessorBox::build_send_action_menu ()
-{
-       using namespace Menu_Helpers;
-
-       send_action_menu = new Menu;
-       send_action_menu->set_name ("ArdourContextMenu");
-       MenuList& items = send_action_menu->items();
-
-       items.push_back (MenuElem (_("New send"), sigc::mem_fun(*this, &ProcessorBox::new_send)));
-       items.push_back (MenuElem (_("Show send controls"), sigc::mem_fun(*this, &ProcessorBox::show_send_controls)));
+       queue_resize ();
 }
 
 Gtk::Menu*
@@ -625,6 +1049,7 @@ ProcessorBox::build_possible_aux_menu ()
        boost::shared_ptr<RouteList> rl = _session->get_routes_with_internal_returns();
 
        if (rl->empty()) {
+               /* No aux sends if there are no busses */
                return 0;
        }
 
@@ -641,16 +1066,6 @@ ProcessorBox::build_possible_aux_menu ()
        return menu;
 }
 
-void
-ProcessorBox::show_send_controls ()
-{
-}
-
-void
-ProcessorBox::new_send ()
-{
-}
-
 void
 ProcessorBox::show_processor_menu (int arg)
 {
@@ -659,12 +1074,16 @@ ProcessorBox::show_processor_menu (int arg)
                processor_menu->signal_unmap().connect (sigc::mem_fun (*this, &ProcessorBox::processor_menu_unmapped));
        }
 
+       /* Sort out the plugin submenu */
+
        Gtk::MenuItem* plugin_menu_item = dynamic_cast<Gtk::MenuItem*>(ActionManager::get_widget("/ProcessorMenu/newplugin"));
 
        if (plugin_menu_item) {
                plugin_menu_item->set_submenu (*_get_plugin_selector()->plugin_menu());
        }
 
+       /* And the aux submenu */
+
        Gtk::MenuItem* aux_menu_item = dynamic_cast<Gtk::MenuItem*>(ActionManager::get_widget("/ProcessorMenu/newaux"));
 
        if (aux_menu_item) {
@@ -679,9 +1098,63 @@ ProcessorBox::show_processor_menu (int arg)
                }
        }
 
+       ProcessorEntry* single_selection = 0;
+       if (processor_display.selection().size() == 1) {
+               single_selection = processor_display.selection().front();
+       }
+
+       /* And the controls submenu */
+
+       Gtk::MenuItem* controls_menu_item = dynamic_cast<Gtk::MenuItem*>(ActionManager::get_widget("/ProcessorMenu/controls"));
+
+       if (controls_menu_item) {
+               if (single_selection) {
+                       Menu* m = single_selection->build_controls_menu ();
+                       if (m && !m->items().empty()) {
+                               controls_menu_item->set_submenu (*m);
+                               controls_menu_item->set_sensitive (true);
+                       } else {
+                               gtk_menu_item_set_submenu (controls_menu_item->gobj(), 0);
+                               controls_menu_item->set_sensitive (false);
+                       }
+               }
+       }
+
+       Gtk::MenuItem* send_menu_item = dynamic_cast<Gtk::MenuItem*>(ActionManager::get_widget("/ProcessorMenu/send_options"));
+       if (send_menu_item) {
+               if (single_selection) {
+                       Menu* m = single_selection->build_send_options_menu ();
+                       if (m && !m->items().empty()) {
+                               send_menu_item->set_submenu (*m);
+                               send_menu_item->set_sensitive (true);
+                       } else {
+                               gtk_menu_item_set_submenu (send_menu_item->gobj(), 0);
+                               send_menu_item->set_sensitive (false);
+                       }
+               }
+       }
+
+       /* Sensitise actions as approprioate */
+
         cut_action->set_sensitive (can_cut());
        paste_action->set_sensitive (!_rr_selection.processors.empty());
 
+       const bool sensitive = !processor_display.selection().empty();
+       ActionManager::set_sensitive (ActionManager::plugin_selection_sensitive_actions, sensitive);
+       edit_action->set_sensitive (one_processor_can_be_edited ());
+       edit_generic_action->set_sensitive (one_processor_can_be_edited ());
+
+       boost::shared_ptr<PluginInsert> pi;
+       if (single_selection) {
+               pi = boost::dynamic_pointer_cast<PluginInsert> (single_selection->processor ());
+       }
+
+       /* allow editing with an Ardour-generated UI for plugin inserts with editors */
+       edit_action->set_sensitive (pi && pi->plugin()->has_editor ());
+
+       /* disallow rename for multiple selections, for plugin inserts and for the fader */
+       rename_action->set_sensitive (single_selection && !pi && !boost::dynamic_pointer_cast<Amp> (single_selection->processor ()));
+
        processor_menu->popup (1, arg);
 
        /* Add a placeholder gap to the processor list to indicate where a processor would be
@@ -690,56 +1163,28 @@ ProcessorBox::show_processor_menu (int arg)
        int x, y;
        processor_display.get_pointer (x, y);
        _placement = processor_display.add_placeholder (y);
+
+       if (_visible_prefader_processors == 0 && _placement > 0) {
+               --_placement;
+       }
 }
 
 bool
 ProcessorBox::enter_notify (GdkEventCrossing*)
 {
        _current_processor_box = this;
-       Keyboard::magic_widget_grab_focus ();
-       processor_display.grab_focus ();
-
        return false;
 }
 
 bool
-ProcessorBox::leave_notify (GdkEventCrossing* ev)
+ProcessorBox::leave_notify (GdkEventCrossing*)
 {
-       switch (ev->detail) {
-       case GDK_NOTIFY_INFERIOR:
-               break;
-       default:
-               Keyboard::magic_widget_drop_focus ();
-       }
-
        return false;
 }
 
-bool
-ProcessorBox::processor_key_press_event (GdkEventKey *ev)
-{
-       switch (ev->keyval) {
-       case GDK_a:
-       case GDK_c:
-       case GDK_x:
-       case GDK_v:
-       case GDK_Up:
-       case GDK_Down:
-       case GDK_Delete:
-       case GDK_BackSpace:
-       case GDK_Return:
-       case GDK_slash:
-               /* do real stuff on key release */
-               return true;
-       }
-       
-       return false;
-}
-
-bool
-ProcessorBox::processor_key_release_event (GdkEventKey *ev)
+void
+ProcessorBox::processor_operation (ProcessorOperation op) 
 {
-       bool ret = false;
        ProcSelection targets;
 
        get_selected_processors (targets);
@@ -751,62 +1196,37 @@ ProcessorBox::processor_key_release_event (GdkEventKey *ev)
 
                pair<ProcessorEntry *, double> const pointer = processor_display.get_child_at_position (y);
 
-               if (pointer.first) {
+               if (pointer.first && pointer.first->processor()) {
                        targets.push_back (pointer.first->processor ());
                }
        }
 
-
-       switch (ev->keyval) {
-       case GDK_a:
-               if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
-                       processor_display.select_all ();
-                       ret = true;
-               }
-               break;
-
-       case GDK_c:
-               if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
-                       copy_processors (targets);
-                       ret = true;
-               }
+       switch (op) {
+       case ProcessorsSelectAll:
+               processor_display.select_all ();
                break;
 
-       case GDK_x:
-               if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
-                       cut_processors (targets);
-                       ret = true;
-               }
+       case ProcessorsCopy:
+               copy_processors (targets);
                break;
 
-       case GDK_v:
-               if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
-                       if (targets.empty()) {
-                               paste_processors ();
-                       } else {
-                               paste_processors (targets.front());
-                       }
-                       ret = true;
-               }
-               break;
-
-       case GDK_Up:
-               processors_down ();
-               ret = true;
+       case ProcessorsCut:
+               cut_processors (targets);
                break;
 
-       case GDK_Down:
-               processors_up ();
-               ret = true;
+       case ProcessorsPaste:
+               if (targets.empty()) {
+                       paste_processors ();
+               } else {
+                       paste_processors (targets.front());
+               }
                break;
 
-       case GDK_Delete:
-       case GDK_BackSpace:
+       case ProcessorsDelete:
                delete_processors (targets);
-               ret = true;
                break;
 
-       case GDK_Return:
+       case ProcessorsToggleActive:
                for (ProcSelection::iterator i = targets.begin(); i != targets.end(); ++i) {
                        if ((*i)->active()) {
                                (*i)->deactivate ();
@@ -814,21 +1234,33 @@ ProcessorBox::processor_key_release_event (GdkEventKey *ev)
                                (*i)->activate ();
                        }
                }
-               ret = true;
                break;
 
-       case GDK_slash:
+       case ProcessorsAB:
                ab_plugins ();
-               ret = true;
                break;
 
        default:
                break;
        }
+}
 
-       return ret;
+ProcessorWindowProxy* 
+ProcessorBox::find_window_proxy (boost::shared_ptr<Processor> processor) const
+{
+       for (list<ProcessorWindowProxy*>::const_iterator i = _processor_window_info.begin(); i != _processor_window_info.end(); ++i) {
+               boost::shared_ptr<Processor> p = (*i)->processor().lock();
+
+               if (p && p == processor) {
+                       return (*i);
+               }
+       }
+
+       return 0;
 }
 
+
+
 bool
 ProcessorBox::processor_button_press_event (GdkEventButton *ev, ProcessorEntry* child)
 {
@@ -844,8 +1276,16 @@ ProcessorBox::processor_button_press_event (GdkEventButton *ev, ProcessorEntry*
 
                if (_session->engine().connected()) {
                        /* XXX giving an error message here is hard, because we may be in the midst of a button press */
-                       toggle_edit_processor (processor);
+
+                       if (!one_processor_can_be_edited ()) return true;
+
+                       if (Keyboard::modifier_state_equals (ev->state, Keyboard::SecondaryModifier)) {
+                               generic_edit_processor (processor);
+                       } else {
+                               edit_processor (processor);
+                       }
                }
+
                ret = true;
 
        } else if (processor && ev->button == 1 && selected) {
@@ -906,29 +1346,6 @@ ProcessorBox::build_processor_menu ()
        return processor_menu;
 }
 
-void
-ProcessorBox::selection_changed ()
-{
-       const bool sensitive = !processor_display.selection().empty();
-       ActionManager::set_sensitive(ActionManager::plugin_selection_sensitive_actions,
-                                    sensitive);
-       edit_action->set_sensitive(one_processor_can_be_edited());
-
-       const bool single_selection = (processor_display.selection().size() == 1);
-
-       boost::shared_ptr<PluginInsert> pi;
-       if (single_selection) {
-               pi = boost::dynamic_pointer_cast<PluginInsert>(
-                       processor_display.selection().front()->processor());
-       }
-
-       /* enable gui for plugin inserts with editors */
-       controls_action->set_sensitive(pi && pi->plugin()->has_editor());
-
-       /* disallow rename for multiple selections and for plugin inserts */
-       rename_action->set_sensitive(single_selection && pi);
-}
-
 void
 ProcessorBox::select_all_processors ()
 {
@@ -1011,7 +1428,6 @@ ProcessorBox::weird_plugin_dialog (Plugin& p, Route::ProcessorStreams streams)
        dialog.add_button (Stock::OK, RESPONSE_ACCEPT);
 
        dialog.set_name (X_("PluginIODialog"));
-       dialog.set_position (Gtk::WIN_POS_MOUSE);
        dialog.set_modal (true);
        dialog.show_all ();
 
@@ -1038,7 +1454,7 @@ ProcessorBox::choose_send ()
 
        /* XXX need processor lock on route */
        try {
-               Glib::Mutex::Lock lm (AudioEngine::instance()->process_lock());
+               Glib::Threads::Mutex::Lock lm (AudioEngine::instance()->process_lock());
                send->output()->ensure_io (outs, false, this);
        } catch (AudioEngine::PortRegistrationFailure& err) {
                error << string_compose (_("Cannot set up new send: %1"), err.what()) << endmsg;
@@ -1135,10 +1551,7 @@ ProcessorBox::choose_aux (boost::weak_ptr<Route> wr)
                return;
        }
 
-       boost::shared_ptr<RouteList> rlist (new RouteList);
-       rlist->push_back (_route);
-
-       _session->add_internal_sends (target, PreFader, rlist);
+       _session->add_internal_send (target, _placement, _route);
 }
 
 void
@@ -1155,7 +1568,8 @@ ProcessorBox::route_processors_changed (RouteProcessorChange c)
 void
 ProcessorBox::redisplay_processors ()
 {
-       ENSURE_GUI_THREAD (*this, &ProcessorBox::redisplay_processors)
+       ENSURE_GUI_THREAD (*this, &ProcessorBox::redisplay_processors);
+       bool     fader_seen;
 
        if (no_processor_redisplay) {
                return;
@@ -1163,9 +1577,15 @@ ProcessorBox::redisplay_processors ()
 
        processor_display.clear ();
 
+       _visible_prefader_processors = 0;
+       fader_seen = false;
+
+       _route->foreach_processor (sigc::bind (sigc::mem_fun (*this, &ProcessorBox::help_count_visible_prefader_processors), 
+                                              &_visible_prefader_processors, &fader_seen));
+
        _route->foreach_processor (sigc::mem_fun (*this, &ProcessorBox::add_processor_to_display));
 
-       for (list<ProcessorWindowProxy*>::iterator i = _processor_window_proxies.begin(); i != _processor_window_proxies.end(); ++i) {
+       for (ProcessorWindowProxies::iterator i = _processor_window_info.begin(); i != _processor_window_info.end(); ++i) {
                (*i)->marked = false;
        }
 
@@ -1173,16 +1593,43 @@ ProcessorBox::redisplay_processors ()
 
        /* trim dead wood from the processor window proxy list */
 
-       list<ProcessorWindowProxy*>::iterator i = _processor_window_proxies.begin();
-       while (i != _processor_window_proxies.end()) {
-               list<ProcessorWindowProxy*>::iterator j = i;
+       ProcessorWindowProxies::iterator i = _processor_window_info.begin();
+       while (i != _processor_window_info.end()) {
+               ProcessorWindowProxies::iterator j = i;
                ++j;
 
-               if (!(*i)->marked) {
-                       ARDOUR_UI::instance()->remove_window_proxy (*i);
+               if (!(*i)->valid()) {
+
+                       WM::Manager::instance().remove (*i);
                        delete *i;
-                       _processor_window_proxies.erase (i);
-               }
+                       _processor_window_info.erase (i);
+                       
+               } else if (!(*i)->marked) {
+
+                       /* this processor is no longer part of this processor
+                        * box.
+                        *
+                        * that could be because it was deleted or it could be
+                        * because the route being displayed in the parent
+                        * strip changed.
+                        *
+                        * The latter only happens with the editor mixer strip.
+                        */
+
+                       if (is_editor_mixer_strip()) {
+                               
+                               /* editor mixer strip .. DO NOTHING
+                                *
+                                * note: the processor window stays visible if
+                                * it is already visible
+                                */
+                       } else {
+                               (*i)->hide ();
+                               WM::Manager::instance().remove (*i);
+                               delete *i;
+                               _processor_window_info.erase (i);
+                       } 
+               } 
 
                i = j;
        }
@@ -1201,8 +1648,8 @@ ProcessorBox::maybe_add_processor_to_ui_list (boost::weak_ptr<Processor> w)
                return;
        }
 
-       list<ProcessorWindowProxy*>::iterator i = _processor_window_proxies.begin ();
-       while (i != _processor_window_proxies.end()) {
+       ProcessorWindowProxies::iterator i = _processor_window_info.begin ();
+       while (i != _processor_window_info.end()) {
 
                boost::shared_ptr<Processor> t = (*i)->processor().lock ();
 
@@ -1230,10 +1677,15 @@ ProcessorBox::maybe_add_processor_to_ui_list (boost::weak_ptr<Processor> w)
 
        ProcessorWindowProxy* wp = new ProcessorWindowProxy (
                string_compose ("%1-%2-%3", loc, _route->id(), p->id()),
-               _session->extra_xml (X_("UI")),
                this,
                w);
 
+       const XMLNode* ui_xml = _session->extra_xml (X_("UI"));
+
+       if (ui_xml) {
+               wp->set_state (*ui_xml);
+       }
+       
        wp->marked = true;
 
         /* if the processor already has an existing UI,
@@ -1243,11 +1695,28 @@ ProcessorBox::maybe_add_processor_to_ui_list (boost::weak_ptr<Processor> w)
         void* existing_ui = p->get_ui ();
 
         if (existing_ui) {
-                wp->set (static_cast<Gtk::Window*>(existing_ui));
+                wp->use_window (*(reinterpret_cast<Gtk::Window*>(existing_ui)));
         }
 
-       _processor_window_proxies.push_back (wp);
-       ARDOUR_UI::instance()->add_window_proxy (wp);
+       _processor_window_info.push_back (wp);
+       WM::Manager::instance().register_window (wp);
+}
+
+void
+ProcessorBox::help_count_visible_prefader_processors (boost::weak_ptr<Processor> p, uint32_t* cnt, bool* amp_seen)
+{
+       boost::shared_ptr<Processor> processor (p.lock ());
+
+       if (processor && processor->display_to_user()) {
+
+               if (boost::dynamic_pointer_cast<Amp>(processor)) {
+                       *amp_seen = true;
+               } else {
+                       if (!*amp_seen) {
+                               (*cnt)++;
+                       }
+               }
+       }
 }
 
 void
@@ -1259,34 +1728,26 @@ ProcessorBox::add_processor_to_display (boost::weak_ptr<Processor> p)
                return;
        }
 
-       boost::shared_ptr<Send> send = boost::dynamic_pointer_cast<Send> (processor);
        boost::shared_ptr<PluginInsert> plugin_insert = boost::dynamic_pointer_cast<PluginInsert> (processor);
        ProcessorEntry* e = 0;
-       if (send) {
-               e = new SendProcessorEntry (send, _width);
-       } else if (plugin_insert) {
-               e = new PluginInsertProcessorEntry (plugin_insert, _width);
+       if (plugin_insert) {
+               e = new PluginInsertProcessorEntry (this, plugin_insert, _width);
        } else {
-               e = new ProcessorEntry (processor, _width);
+               e = new ProcessorEntry (this, processor, _width);
        }
 
-       e->set_pixel_width (get_allocation().get_width());
-       processor_display.add_child (e);
-}
-
-
-void
-ProcessorBox::build_processor_tooltip (EventBox& box, string start)
-{
-       string tip(start);
-
-       list<ProcessorEntry*> children = processor_display.children ();
-       for (list<ProcessorEntry*>::iterator i = children.begin(); i != children.end(); ++i) {
-               tip += '\n';
-               tip += (*i)->processor()->name();
+       /* Set up this entry's state from the GUIObjectState */
+       XMLNode* proc = entry_gui_object_state (e);
+       if (proc) {
+               e->set_control_state (proc);
+       }
+       
+       if (boost::dynamic_pointer_cast<Send> (processor)) {
+               /* Always show send controls */
+               e->show_all_controls ();
        }
 
-       ARDOUR_UI::instance()->set_tip (box, tip);
+       processor_display.add_child (e);
 }
 
 void
@@ -1302,15 +1763,16 @@ ProcessorBox::setup_entry_positions ()
        list<ProcessorEntry*> children = processor_display.children ();
        bool pre_fader = true;
 
+       uint32_t num = 0;
        for (list<ProcessorEntry*>::iterator i = children.begin(); i != children.end(); ++i) {
                if (boost::dynamic_pointer_cast<Amp>((*i)->processor())) {
                        pre_fader = false;
-                       (*i)->set_position (ProcessorEntry::Fader);
+                       (*i)->set_position (ProcessorEntry::Fader, num++);
                } else {
                        if (pre_fader) {
-                               (*i)->set_position (ProcessorEntry::PreFader);
+                               (*i)->set_position (ProcessorEntry::PreFader, num++);
                        } else {
-                               (*i)->set_position (ProcessorEntry::PostFader);
+                               (*i)->set_position (ProcessorEntry::PostFader, num++);
                        }
                }
        }
@@ -1322,8 +1784,10 @@ ProcessorBox::compute_processor_sort_keys ()
        list<ProcessorEntry*> children = processor_display.children ();
        Route::ProcessorList our_processors;
 
-       for (list<ProcessorEntry*>::iterator iter = children.begin(); iter != children.end(); ++iter) {
-               our_processors.push_back ((*iter)->processor ());
+       for (list<ProcessorEntry*>::iterator i = children.begin(); i != children.end(); ++i) {
+               if ((*i)->processor()) {
+                       our_processors.push_back ((*i)->processor ());
+               }
        }
 
        if (_route->reorder_processors (our_processors)) {
@@ -1358,7 +1822,6 @@ outputs will not work correctly."));
        dialog.add_button (Stock::OK, RESPONSE_ACCEPT);
 
        dialog.set_name (X_("PluginIODialog"));
-       dialog.set_position (Gtk::WIN_POS_MOUSE);
        dialog.set_modal (true);
        dialog.show_all ();
 
@@ -1402,14 +1865,6 @@ ProcessorBox::can_cut () const
         return false;
 }
 
-void
-ProcessorBox::cut_processors ()
-{
-       ProcSelection to_be_removed;
-
-       get_selected_processors (to_be_removed);
-}
-
 void
 ProcessorBox::cut_processors (const ProcSelection& to_be_removed)
 {
@@ -1451,14 +1906,6 @@ ProcessorBox::cut_processors (const ProcSelection& to_be_removed)
        redisplay_processors ();
 }
 
-void
-ProcessorBox::copy_processors ()
-{
-       ProcSelection to_be_copied;
-       get_selected_processors (to_be_copied);
-       copy_processors (to_be_copied);
-}
-
 void
 ProcessorBox::copy_processors (const ProcSelection& to_be_copied)
 {
@@ -1480,27 +1927,6 @@ ProcessorBox::copy_processors (const ProcSelection& to_be_copied)
        _rr_selection.set (node);
 }
 
-void
-ProcessorBox::processors_up ()
-{
-       /* unimplemented */
-}
-
-void
-ProcessorBox::processors_down ()
-{
-       /* unimplemented */
-}
-       
-
-void
-ProcessorBox::delete_processors ()
-{
-       ProcSelection to_be_deleted;
-       get_selected_processors (to_be_deleted);
-       delete_processors (to_be_deleted);
-}
-
 void
 ProcessorBox::delete_processors (const ProcSelection& targets)
 {
@@ -1644,6 +2070,7 @@ ProcessorBox::paste_processor_state (const XMLNodeList& nlist, boost::shared_ptr
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
 
                XMLProperty const * type = (*niter)->property ("type");
+               XMLProperty const * role = (*niter)->property ("role");
                assert (type);
 
                boost::shared_ptr<Processor> p;
@@ -1651,15 +2078,28 @@ ProcessorBox::paste_processor_state (const XMLNodeList& nlist, boost::shared_ptr
                        if (type->value() == "meter" ||
                            type->value() == "main-outs" ||
                            type->value() == "amp" ||
-                           type->value() == "intsend" || type->value() == "intreturn") {
-                               /* do not paste meter, main outs, amp or internal send/returns */
+                           type->value() == "intreturn") {
+                               /* do not paste meter, main outs, amp or internal returns */
                                continue;
 
-                       } else if (type->value() == "send") {
+                       } else if (type->value() == "intsend") {
+                               
+                               /* aux sends are OK, but those used for
+                                * other purposes, are not.
+                                */
+                               
+                               assert (role);
+
+                               if (role->value() != "Aux") {
+                                       continue;
+                               }
 
                                XMLNode n (**niter);
-                               Send::make_unique (n, *_session);
-                                Send* s = new Send (*_session, _route->pannable(), _route->mute_master());
+                                InternalSend* s = new InternalSend (*_session, _route->pannable(), _route->mute_master(),
+                                                                   boost::shared_ptr<Route>(), Delivery::Aux); 
+
+                               IOProcessor::prepare_for_reset (n, s->name());
+
                                 if (s->set_state (n, Stateful::loading_state_version)) {
                                         delete s;
                                         return;
@@ -1667,13 +2107,27 @@ ProcessorBox::paste_processor_state (const XMLNodeList& nlist, boost::shared_ptr
 
                                p.reset (s);
 
+                       } else if (type->value() == "send") {
+
+                               XMLNode n (**niter);
+                               Send* s = new Send (*_session, _route->pannable(), _route->mute_master());
+
+                               IOProcessor::prepare_for_reset (n, s->name());
+                               
+                                if (s->set_state (n, Stateful::loading_state_version)) {
+                                        delete s;
+                                        return;
+                                }
+
+                               p.reset (s);
 
                        } else if (type->value() == "return") {
 
                                XMLNode n (**niter);
-                               Return::make_unique (n, *_session);
                                 Return* r = new Return (*_session);
 
+                               IOProcessor::prepare_for_reset (n, r->name());
+
                                 if (r->set_state (n, Stateful::loading_state_version)) {
                                         delete r;
                                         return;
@@ -1684,10 +2138,15 @@ ProcessorBox::paste_processor_state (const XMLNodeList& nlist, boost::shared_ptr
                        } else if (type->value() == "port") {
 
                                XMLNode n (**niter);
-                               p.reset (new PortInsert (*_session, _route->pannable (), _route->mute_master ()));
-                               if (p->set_state (n, Stateful::loading_state_version)) {
+                               PortInsert* pi = new PortInsert (*_session, _route->pannable (), _route->mute_master ());
+                               
+                               IOProcessor::prepare_for_reset (n, pi->name());
+                               
+                               if (pi->set_state (n, Stateful::loading_state_version)) {
                                        return;
                                }
+                               
+                               p.reset (pi);
 
                        } else {
                                /* XXX its a bit limiting to assume that everything else
@@ -1702,7 +2161,7 @@ ProcessorBox::paste_processor_state (const XMLNodeList& nlist, boost::shared_ptr
                }
 
                catch (...) {
-                       cerr << "plugin insert constructor failed\n";
+                       error << _("plugin insert constructor failed") << endmsg;
                }
        }
 
@@ -1721,18 +2180,6 @@ could not match the configuration of this track.");
        }
 }
 
-void
-ProcessorBox::activate_processor (boost::shared_ptr<Processor> r)
-{
-       r->activate ();
-}
-
-void
-ProcessorBox::deactivate_processor (boost::shared_ptr<Processor> r)
-{
-       r->deactivate ();
-}
-
 void
 ProcessorBox::get_selected_processors (ProcSelection& processors) const
 {
@@ -1752,10 +2199,9 @@ ProcessorBox::for_selected_processors (void (ProcessorBox::*method)(boost::share
 }
 
 void
-ProcessorBox::all_processors_active (bool state)
+ProcessorBox::all_visible_processors_active (bool state)
 {
-       _route->all_processors_active (PreFader, state);
-       _route->all_processors_active (PostFader, state);
+       _route->all_visible_processors_active (state);
 }
 
 void
@@ -1819,7 +2265,7 @@ ProcessorBox::processor_can_be_edited (boost::shared_ptr<Processor> processor)
        }
 
        if (
-               (boost::dynamic_pointer_cast<Send> (processor) && !boost::dynamic_pointer_cast<InternalSend> (processor))||
+               boost::dynamic_pointer_cast<Send> (processor) ||
                boost::dynamic_pointer_cast<Return> (processor) ||
                boost::dynamic_pointer_cast<PluginInsert> (processor) ||
                boost::dynamic_pointer_cast<PortInsert> (processor)
@@ -1842,8 +2288,8 @@ ProcessorBox::one_processor_can_be_edited ()
        return (i != selection.end());
 }
 
-void
-ProcessorBox::toggle_edit_processor (boost::shared_ptr<Processor> processor)
+Gtk::Window*
+ProcessorBox::get_editor_window (boost::shared_ptr<Processor> processor, bool use_custom)
 {
        boost::shared_ptr<Send> send;
        boost::shared_ptr<InternalSend> internal_send;
@@ -1852,10 +2298,22 @@ ProcessorBox::toggle_edit_processor (boost::shared_ptr<Processor> processor)
        boost::shared_ptr<PortInsert> port_insert;
        Window* gidget = 0;
 
+       /* This method may or may not return a Window, but if it does not it
+        * will modify the parent mixer strip appearance layout to allow
+        * "editing" the @param processor that was passed in.
+        *
+        * So for example, if the processor is an Amp (gain), the parent strip
+        * will be forced back into a model where the fader controls the main gain.
+        * If the processor is a send, then we map the send controls onto the
+        * strip.
+        * 
+        * Plugins and others will return a window for control.
+        */
+
        if (boost::dynamic_pointer_cast<AudioTrack>(_route) != 0) {
 
                if (boost::dynamic_pointer_cast<AudioTrack> (_route)->freeze_state() == AudioTrack::Frozen) {
-                       return;
+                       return 0;
                }
        }
 
@@ -1868,21 +2326,23 @@ ProcessorBox::toggle_edit_processor (boost::shared_ptr<Processor> processor)
        } else if ((send = boost::dynamic_pointer_cast<Send> (processor)) != 0) {
 
                if (!_session->engine().connected()) {
-                       return;
+                       return 0;
                }
 
-               SendUIWindow* w = new SendUIWindow (send, _session);
-               w->show ();
+               if (boost::dynamic_pointer_cast<InternalSend> (processor) == 0) {
+
+                       gidget = new SendUIWindow (send, _session);
+               }
 
        } else if ((retrn = boost::dynamic_pointer_cast<Return> (processor)) != 0) {
 
                if (boost::dynamic_pointer_cast<InternalReturn> (retrn)) {
                        /* no GUI for these */
-                       return;
+                       return 0;
                }
 
                if (!_session->engine().connected()) {
-                       return;
+                       return 0;
                }
 
                boost::shared_ptr<Return> retrn = boost::dynamic_pointer_cast<Return> (processor);
@@ -1908,20 +2368,15 @@ ProcessorBox::toggle_edit_processor (boost::shared_ptr<Processor> processor)
 
                /* these are both allowed to be null */
 
-               Container* toplevel = get_toplevel();
-               Window* win = dynamic_cast<Gtk::Window*>(toplevel);
-
                Window* w = get_processor_ui (plugin_insert);
 
                if (w == 0) {
-
-                       plugin_ui = new PluginUIWindow (win, plugin_insert);
+                       plugin_ui = new PluginUIWindow (plugin_insert, false, use_custom);
                        plugin_ui->set_title (generate_processor_title (plugin_insert));
                        set_processor_ui (plugin_insert, plugin_ui);
 
                } else {
                        plugin_ui = dynamic_cast<PluginUIWindow *> (w);
-                       plugin_ui->set_parent (win);
                }
 
                gidget = plugin_ui;
@@ -1929,9 +2384,9 @@ ProcessorBox::toggle_edit_processor (boost::shared_ptr<Processor> processor)
        } else if ((port_insert = boost::dynamic_pointer_cast<PortInsert> (processor)) != 0) {
 
                if (!_session->engine().connected()) {
-                       MessageDialog msg ( _("Not connected to JACK - no I/O changes are possible"));
+                       MessageDialog msg ( _("Not connected to audio engine - no I/O changes are possible"));
                        msg.run ();
-                       return;
+                       return 0;
                }
 
                PortInsertWindow *io_selector;
@@ -1949,36 +2404,23 @@ ProcessorBox::toggle_edit_processor (boost::shared_ptr<Processor> processor)
                gidget = io_selector;
        }
 
-       if (gidget) {
-               if (gidget->is_visible()) {
-                       gidget->hide ();
-               } else {
-                       gidget->show_all ();
-                       gidget->present ();
-               }
-       }
+       return gidget;
 }
 
-void
-ProcessorBox::toggle_processor_controls (boost::shared_ptr<Processor> processor)
+Gtk::Window*
+ProcessorBox::get_generic_editor_window (boost::shared_ptr<Processor> processor)
 {
        boost::shared_ptr<PluginInsert> plugin_insert
                = boost::dynamic_pointer_cast<PluginInsert>(processor);
+
        if (!plugin_insert) {
-               return;
+               return 0;
        }
 
-       Container*      toplevel  = get_toplevel();
-       Window*         win       = dynamic_cast<Gtk::Window*>(toplevel);
-       PluginUIWindow* plugin_ui = new PluginUIWindow(win, plugin_insert, true, false);
-       plugin_ui->set_title(generate_processor_title (plugin_insert));
+       PluginUIWindow* win = new PluginUIWindow (plugin_insert, true, false);
+       win->set_title (generate_processor_title (plugin_insert));
 
-       if (plugin_ui->is_visible()) {
-               plugin_ui->hide();
-       } else {
-               plugin_ui->show_all();
-               plugin_ui->present();
-       }
+       return win;
 }
 
 void
@@ -1993,13 +2435,16 @@ ProcessorBox::register_actions ()
 
        act = ActionManager::register_action (popup_act_grp, X_("newinsert"), _("New Insert"),
                        sigc::ptr_fun (ProcessorBox::rb_choose_insert));
-       ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_action (popup_act_grp, X_("newsend"), _("New Send ..."),
+       ActionManager::engine_sensitive_actions.push_back (act);
+       act = ActionManager::register_action (popup_act_grp, X_("newsend"), _("New External Send ..."),
                        sigc::ptr_fun (ProcessorBox::rb_choose_send));
-       ActionManager::jack_sensitive_actions.push_back (act);
+       ActionManager::engine_sensitive_actions.push_back (act);
 
        ActionManager::register_action (popup_act_grp, X_("newaux"), _("New Aux Send ..."));
 
+       ActionManager::register_action (popup_act_grp, X_("controls"), _("Controls"));
+       ActionManager::register_action (popup_act_grp, X_("send_options"), _("Send Options"));
+
        ActionManager::register_action (popup_act_grp, X_("clear"), _("Clear (all)"),
                        sigc::ptr_fun (ProcessorBox::rb_clear));
        ActionManager::register_action (popup_act_grp, X_("clear_pre"), _("Clear (pre-fader)"),
@@ -2030,9 +2475,9 @@ ProcessorBox::register_actions ()
 
        /* activation etc. */
 
-       ActionManager::register_action (popup_act_grp, X_("activate_all"), _("Activate all"),
+       ActionManager::register_action (popup_act_grp, X_("activate_all"), _("Activate All"),
                        sigc::ptr_fun (ProcessorBox::rb_activate_all));
-       ActionManager::register_action (popup_act_grp, X_("deactivate_all"), _("Deactivate all"),
+       ActionManager::register_action (popup_act_grp, X_("deactivate_all"), _("Deactivate All"),
                        sigc::ptr_fun (ProcessorBox::rb_deactivate_all));
        ActionManager::register_action (popup_act_grp, X_("ab_plugins"), _("A/B Plugins"),
                        sigc::ptr_fun (ProcessorBox::rb_ab_plugins));
@@ -2042,14 +2487,23 @@ ProcessorBox::register_actions ()
                popup_act_grp, X_("edit"), _("Edit..."),
                sigc::ptr_fun (ProcessorBox::rb_edit));
 
-       /* show plugin GUI */
-       controls_action = ActionManager::register_action (
-                       popup_act_grp, X_("controls"), _("Controls..."),
-                       sigc::ptr_fun (ProcessorBox::rb_controls));
+       edit_generic_action = ActionManager::register_action (
+               popup_act_grp, X_("edit-generic"), _("Edit with generic controls..."),
+               sigc::ptr_fun (ProcessorBox::rb_edit_generic));
 
        ActionManager::add_action_group (popup_act_grp);
 }
 
+void
+ProcessorBox::rb_edit_generic ()
+{
+       if (_current_processor_box == 0) {
+               return;
+       }
+
+       _current_processor_box->for_selected_processors (&ProcessorBox::generic_edit_processor);
+}
+
 void
 ProcessorBox::rb_ab_plugins ()
 {
@@ -2136,7 +2590,7 @@ ProcessorBox::rb_cut ()
                return;
        }
 
-       _current_processor_box->cut_processors ();
+       _current_processor_box->processor_operation (ProcessorsCut);
 }
 
 void
@@ -2146,7 +2600,7 @@ ProcessorBox::rb_delete ()
                return;
        }
 
-       _current_processor_box->delete_processors ();
+       _current_processor_box->processor_operation (ProcessorsDelete);
 }
 
 void
@@ -2155,7 +2609,7 @@ ProcessorBox::rb_copy ()
        if (_current_processor_box == 0) {
                return;
        }
-       _current_processor_box->copy_processors ();
+       _current_processor_box->processor_operation (ProcessorsCopy);
 }
 
 void
@@ -2165,7 +2619,7 @@ ProcessorBox::rb_paste ()
                return;
        }
 
-       _current_processor_box->paste_processors ();
+       _current_processor_box->processor_operation (ProcessorsPaste);
 }
 
 void
@@ -2184,7 +2638,7 @@ ProcessorBox::rb_select_all ()
                return;
        }
 
-       _current_processor_box->select_all_processors ();
+       _current_processor_box->processor_operation (ProcessorsSelectAll);
 }
 
 void
@@ -2204,7 +2658,7 @@ ProcessorBox::rb_activate_all ()
                return;
        }
 
-       _current_processor_box->all_processors_active (true);
+       _current_processor_box->all_visible_processors_active (true);
 }
 
 void
@@ -2213,7 +2667,7 @@ ProcessorBox::rb_deactivate_all ()
        if (_current_processor_box == 0) {
                return;
        }
-       _current_processor_box->all_processors_active (false);
+       _current_processor_box->all_visible_processors_active (false);
 }
 
 void
@@ -2223,17 +2677,61 @@ ProcessorBox::rb_edit ()
                return;
        }
 
-       _current_processor_box->for_selected_processors (&ProcessorBox::toggle_edit_processor);
+       _current_processor_box->for_selected_processors (&ProcessorBox::edit_processor);
+}
+
+bool
+ProcessorBox::edit_aux_send (boost::shared_ptr<Processor> processor)
+{
+       if (boost::dynamic_pointer_cast<InternalSend> (processor) == 0) {
+               return false;
+       }
+
+       if (_parent_strip) {
+               boost::shared_ptr<Send> send = boost::dynamic_pointer_cast<Send> (processor);
+               if (_parent_strip->current_delivery() == send) {
+                       _parent_strip->revert_to_default_display ();
+               } else {
+                       _parent_strip->show_send(send);
+               }
+       }
+       return true;
 }
 
 void
-ProcessorBox::rb_controls ()
+ProcessorBox::edit_processor (boost::shared_ptr<Processor> processor)
 {
-       if (_current_processor_box == 0) {
+       if (!processor) {
+               return;
+       }
+       if (edit_aux_send (processor)) {
+               return;
+       }
+
+       ProcessorWindowProxy* proxy = find_window_proxy (processor);
+
+       if (proxy) {
+               proxy->set_custom_ui_mode (true);
+               proxy->toggle ();
+       }
+}
+
+void
+ProcessorBox::generic_edit_processor (boost::shared_ptr<Processor> processor)
+{
+       if (!processor) {
+               return;
+       }
+       if (edit_aux_send (processor)) {
                return;
        }
 
-       _current_processor_box->for_selected_processors (&ProcessorBox::toggle_processor_controls);
+       ProcessorWindowProxy* proxy = find_window_proxy (processor);
+
+       if (proxy) {
+               proxy->set_custom_ui_mode (false);
+               proxy->toggle ();
+       }
 }
 
 void
@@ -2255,6 +2753,10 @@ ProcessorBox::route_property_changed (const PropertyChange& what_changed)
 
                processor = (*iter)->processor ();
 
+               if (!processor) {
+                       continue;
+               }
+
                Window* w = get_processor_ui (processor);
 
                if (!w) {
@@ -2286,17 +2788,12 @@ ProcessorBox::generate_processor_title (boost::shared_ptr<PluginInsert> pi)
                maker += " ...";
        }
 
-       return string_compose(_("%1: %2 (by %3)"), _route->name(), pi->name(), maker);
-}
-
-void
-ProcessorBox::on_size_allocate (Allocation& a)
-{
-       HBox::on_size_allocate (a);
+       SessionObject* owner = pi->owner();
 
-       list<ProcessorEntry*> children = processor_display.children ();
-       for (list<ProcessorEntry*>::const_iterator i = children.begin(); i != children.end(); ++i) {
-               (*i)->set_pixel_width (a.get_width ());
+       if (owner) {
+               return string_compose(_("%1: %2 (by %3)"), owner->name(), pi->name(), maker);
+       } else {
+               return string_compose(_("%1 (by %2)"), pi->name(), maker);
        }
 }
 
@@ -2306,8 +2803,8 @@ ProcessorBox::on_size_allocate (Allocation& a)
 Window *
 ProcessorBox::get_processor_ui (boost::shared_ptr<Processor> p) const
 {
-       list<ProcessorWindowProxy*>::const_iterator i = _processor_window_proxies.begin ();
-       while (i != _processor_window_proxies.end()) {
+       list<ProcessorWindowProxy*>::const_iterator i = _processor_window_info.begin ();
+       while (i != _processor_window_info.end()) {
                boost::shared_ptr<Processor> t = (*i)->processor().lock ();
                if (t && t == p) {
                        return (*i)->get ();
@@ -2316,10 +2813,6 @@ ProcessorBox::get_processor_ui (boost::shared_ptr<Processor> p) const
                ++i;
        }
 
-       /* we shouldn't get here, because the ProcessorUIList should always contain
-          an entry for each processor.
-       */
-       assert (false);
        return 0;
 }
 
@@ -2330,14 +2823,14 @@ ProcessorBox::get_processor_ui (boost::shared_ptr<Processor> p) const
 void
 ProcessorBox::set_processor_ui (boost::shared_ptr<Processor> p, Gtk::Window* w)
 {
-       list<ProcessorWindowProxy*>::iterator i = _processor_window_proxies.begin ();
+       list<ProcessorWindowProxy*>::iterator i = _processor_window_info.begin ();
 
        p->set_ui (w);
 
-       while (i != _processor_window_proxies.end()) {
+       while (i != _processor_window_info.end()) {
                boost::shared_ptr<Processor> t = (*i)->processor().lock ();
                if (t && t == p) {
-                       (*i)->set (w);
+                       (*i)->use_window (*w);
                        return;
                }
 
@@ -2371,6 +2864,7 @@ ProcessorBox::mixer_strip_delivery_changed (boost::weak_ptr<Delivery> w)
        }
 }
 
+/** Called to repair the damage of Editor::show_window doing a show_all() */
 void
 ProcessorBox::hide_things ()
 {
@@ -2386,29 +2880,158 @@ ProcessorBox::processor_menu_unmapped ()
        processor_display.remove_placeholder ();
 }
 
-ProcessorWindowProxy::ProcessorWindowProxy (
-       string const & name,
-       XMLNode const * node,
-       ProcessorBox* box,
-       boost::weak_ptr<Processor> processor
-       )
-       : WindowProxy<Gtk::Window> (name, node)
+XMLNode *
+ProcessorBox::entry_gui_object_state (ProcessorEntry* entry)
+{
+       if (!_parent_strip) {
+               return 0;
+       }
+
+       GUIObjectState& st = _parent_strip->gui_object_state ();
+       
+       XMLNode* strip = st.get_or_add_node (_parent_strip->state_id ());
+       assert (strip);
+       return st.get_or_add_node (strip, entry->state_id());
+}
+
+void
+ProcessorBox::update_gui_object_state (ProcessorEntry* entry)
+{
+       XMLNode* proc = entry_gui_object_state (entry);
+       if (!proc) {
+               return;
+       }
+
+       /* XXX: this is a bit inefficient; we just remove all child nodes and re-add them */
+       proc->remove_nodes_and_delete (X_("Object"));
+       entry->add_control_state (proc);
+}
+
+bool
+ProcessorBox::is_editor_mixer_strip() const
+{
+       return _parent_strip && !_parent_strip->mixer_owned();
+}
+
+ProcessorWindowProxy::ProcessorWindowProxy (string const & name, ProcessorBox* box, boost::weak_ptr<Processor> processor)
+       : WM::ProxyBase (name, string())
        , marked (false)
        , _processor_box (box)
        , _processor (processor)
+       , is_custom (false)
+       , want_custom (false)
+       , _valid (true)
+{
+       boost::shared_ptr<Processor> p = _processor.lock ();
+       if (!p) {
+               return;
+       }
+       p->DropReferences.connect (going_away_connection, MISSING_INVALIDATOR, boost::bind (&ProcessorWindowProxy::processor_going_away, this), gui_context());
+}
+
+ProcessorWindowProxy::~ProcessorWindowProxy()
+{
+       /* processor window proxies do not own the windows they create with
+        * ::get(), so set _window to null before the normal WindowProxy method
+        * deletes it.
+        */
+       _window = 0;
+}
+
+void
+ProcessorWindowProxy::processor_going_away ()
+{
+       delete _window;
+       _window = 0;
+       _valid = false;
+       /* should be no real reason to do this, since the object that would
+          send DropReferences is about to be deleted, but lets do it anyway.
+       */
+       going_away_connection.disconnect();
+}
+
+ARDOUR::SessionHandlePtr*
+ProcessorWindowProxy::session_handle() 
 {
+       /* we don't care */
+       return 0;
+}
 
+bool
+ProcessorWindowProxy::valid() const
+{
+       return _valid;
 }
 
+XMLNode&
+ProcessorWindowProxy::get_state () const
+{
+       XMLNode *node;
+       node = &ProxyBase::get_state();
+       node->add_property (X_("custom-ui"), is_custom? X_("yes") : X_("no"));
+       return *node;
+}
 
 void
-ProcessorWindowProxy::show ()
+ProcessorWindowProxy::set_state (const XMLNode& node)
+{
+       XMLNodeList children = node.children ();
+       XMLNodeList::const_iterator i = children.begin ();
+       while (i != children.end()) {
+               XMLProperty* prop = (*i)->property (X_("name"));
+               if ((*i)->name() == X_("Window") && prop && prop->value() == _name) {
+                       break;
+               }
+               ++i;
+       }
+
+       if (i != children.end()) {
+               XMLProperty* prop;
+               if ((prop = (*i)->property (X_("custom-ui"))) != 0) {
+                       want_custom = PBD::string_is_affirmative (prop->value ());
+               }
+       }
+
+       ProxyBase::set_state(node);
+}
+
+Gtk::Window*
+ProcessorWindowProxy::get (bool create)
 {
        boost::shared_ptr<Processor> p = _processor.lock ();
+
        if (!p) {
-               return;
+               return 0;
+       }
+       if (_window && (is_custom != want_custom)) {
+               /* drop existing window - wrong type */
+               drop_window ();
+       }
+
+       if (!_window) {
+               if (!create) {
+                       return 0;
+               }
+               
+               is_custom = want_custom;
+               _window = _processor_box->get_editor_window (p, is_custom);
+
+               if (_window) {
+                       setup ();
+               }
        }
 
-       _processor_box->toggle_edit_processor (p);
+       return _window;
 }
 
+void
+ProcessorWindowProxy::toggle ()
+{
+       if (_window && (is_custom != want_custom)) {
+               /* drop existing window - wrong type */
+               drop_window ();
+       }
+       is_custom = want_custom;
+
+       WM::ProxyBase::toggle ();
+}