Continue cleaning up `collections::binary_heap`

Some more tidying up.
I personally favour squashing these "forwarding" impls.

`#[inline] fn len(&self) -> uint { self.iter.len() }`.

Also I find it hard to believe that this is necessary?