Hide collection and lazy modules (use-ink#1111)

* Hide top level `collections` and `lazy` modules

* Ignore failing doc-tests

These use types from `collections` which are not available publicly
anymore.

* Stop using `Lazy` in `ERC-20` example

* Use `Mapping` re-export in `ERC-721` example

* Update UI tests to match updated examples

* Allow `dead_code` in `lazy` and `collections` modules

* Remove `Box` and `Pack` from top level `ink_storage` re-exports

* Stop using `Lazy` in `multisig` example

* Remove `Lazy` usage from DNS example

* Update path of `Mapping` in `ERC-1155` example

* Stop using `Lazy` in `trait-erc20` example

* Undo changes to `alloc` tests

* Add TODOs noting why we're ignoring certain doctests

* Remove references to `storage::Box` from doc comments

* Stop re-exporting `boxed::Box`

* Remove storage collection benches

It doesn't looks like there's a way to keep these compiling if we're
hiding the collection module, so the next best thing to do is to remove
them completely.

* Remove `Lazy` from `delegator` example

* Appease Clippy

* Mention tracking issue for code removal instead of blank TODO

crates/lang/macro/src/lib.rs

@@ -136,8 +136,11 @@ pub fn selector_bytes(input: TokenStream) -> TokenStream {
 ///
 ///     An example for this is the following type that could potentially be used
 ///     within a contract's storage struct definition:
-///     ```
-///     // A storage vector of storage vectors.
+///
+///
+///     ```ignore
+///     # // Tracking issue [#1119]: Right now we've hidden the `StorageVec` from public access so
+///     # // this doesn't compile.
 ///     # use ink_storage as storage;
 ///     # type _unused =
 ///     storage::Vec<storage::Vec<i32>>

crates/lang/tests/ui/contract/pass/example-erc20-works.rs

@@ -3,19 +3,16 @@ use ink_lang as ink;
 #[ink::contract]
 mod erc20 {
     use ink_storage::{
-        lazy::{
-            Lazy,
-            Mapping,
-        },
         traits::SpreadAllocate,
+        Mapping,
     };
 
     /// A simple ERC-20 contract.
     #[ink(storage)]
     #[derive(SpreadAllocate)]
     pub struct Erc20 {
         /// Total token supply.
-        total_supply: Lazy<Balance>,
+        total_supply: Balance,
         /// Mapping from owner to number of owned token.
         balances: Mapping<AccountId, Balance>,
         /// Mapping of the token amount which an account is allowed to withdraw
@@ -70,7 +67,7 @@ mod erc20 {
         fn new_init(&mut self, initial_supply: Balance) {
             let caller = Self::env().caller();
             self.balances.insert(&caller, &initial_supply);
-            Lazy::set(&mut self.total_supply, initial_supply);
+            self.total_supply = initial_supply;
             Self::env().emit_event(Transfer {
                 from: None,
                 to: Some(caller),
@@ -81,7 +78,7 @@ mod erc20 {
         /// Returns the total token supply.
         #[ink(message)]
         pub fn total_supply(&self) -> Balance {
-            *self.total_supply
+            self.total_supply
         }
 
         /// Returns the account balance for the specified `owner`.

crates/lang/tests/ui/contract/pass/example-erc721-works.rs

@@ -3,8 +3,8 @@ use ink_lang as ink;
 #[ink::contract]
 mod erc721 {
     use ink_storage::{
-        lazy::Mapping,
         traits::SpreadAllocate,
+        Mapping,
     };
 
     use scale::{

crates/storage/Cargo.toml

@@ -27,13 +27,6 @@ scale-info = { version = "1.0", default-features = false, features = ["derive"],
 cfg-if = "1.0"
 array-init = { version = "2.0", default-features = false }
 
-[target.'cfg(not(target_arch = "wasm32"))'.dependencies]
-# Workaround: we actually just need criterion as a dev-dependency, but
-# there is an issue with a doubly included std lib when executing
-# `cargo check --no-default-features --target wasm32-unknown-unknown`.
-# We haven't found a better solution yet.
-criterion = { version = "0.3", optional = true }
-
 [dev-dependencies]
 quickcheck = "1.0"
 quickcheck_macros = "1.0"
@@ -43,7 +36,6 @@ paste = "1.0"
 [features]
 default = ["std"]
 std = [
-    "criterion",
     "ink_metadata",
     "ink_metadata/std",
     "ink_env/std",
@@ -55,33 +47,3 @@ std = [
 ]
 ink-fuzz-tests = ["std"]
 ink-experimental-engine = ["ink_env/ink-experimental-engine"]
-
-[[bench]]
-name = "bench_lazy"
-path = "benches/bench_lazy.rs"
-harness = false
-
-[[bench]]
-name = "bench_vec"
-path = "benches/bench_vec.rs"
-harness = false
-
-[[bench]]
-name = "bench_stash"
-path = "benches/bench_stash.rs"
-harness = false
-
-[[bench]]
-name = "bench_bitstash"
-path = "benches/bench_bitstash.rs"
-harness = false
-
-[[bench]]
-name = "bench_hashmap"
-path = "benches/bench_hashmap.rs"
-harness = false
-
-[[bench]]
-name = "bench_binary_heap"
-path = "benches/bench_binary_heap.rs"
-harness = false