RFC: Remove #[crate_id]

Using `metadata()` like that also enables the ability to warn about unknown crate attributes, which I think is a useful property.

We also need a new name for this, "crate attributes" already refers to `#![foo]` constructs, so using it to also refer to the `name` and `type` of `#![crate(...)]` is a bit confusing.