Get rid of Result in cookie method

[StevenDoesStuffs]

The cookie method is currently implemented as so:

pub fn cookie(&self, name: &str) -> Result<Option<Cookie<'static>>, Error> {
    let cookie_data = self
        .local::<CookieData>()
        .expect("should always be set by the cookies middleware");

    let locked_jar = cookie_data.content.read().unwrap();
    Ok(locked_jar.get(name).cloned())
}

It always returns Ok, so why bother wrapping it an a Result type?

[yoshuawuyts]

@StevenDoesStuffs that's a great point, let's get rid of that!