`Wrap return type in...` Quick Fix should not wrap types that are already wrapped

[sbdchd]

Running Wrap return type in... Option:

fn foo(x: i64) -> i64 {
    if x > 10 {
        Some(1)
    } else {
        None
    }
}

Gives:

fn foo(x: i64) -> Option<i64> {
    if x > 10 {
        Some(Some(1))
    } else {
        Some(None)
    }
}

But it would be better if it recognized the return values are already Option<T> so they don't need to be wrapped again:

fn foo(x: i64) -> Option<i64> {
    if x > 10 {
        Some(1)
    } else {
        None
    }
}