Remove Rc usage from the crate

[nox]

Callers should use Rc<T> wherever they want, so that adapters such as ErrorCheckingGl can do their job without multiple layers of virtual calls.

---

This change is 

[nox]

Cc @asajeffrey @bholley

[bholley]

Looking at this patch, I'm not sure it's compatible with WebRender. That code needs to start using a trait object early on (to abstract over the difference between GL and GLES), and by the time we decide to apply the wrappers (e.g. ErrorReactingGl) we don't have the concrete type anymore.

ProfilingGl has a similar problem [1], which is sort of exacerbated by the fact that it can have four different inners (GL/GLES, with/without ErrorReactingGl wrapper).

[1] https://hg.mozilla.org/integration/mozilla-inbound/rev/1972b8836948

[kvark]

@bholley interesting point! Can't we solve this by simply:

impl<T: Gl> Gl for Rc<T> {...}

[nox]

Yes.

[kvark]

@nox so we are basically ready to land this, just waiting for a moment to do the breaking change?