Currently, PaymentRequestItemList holds a vector of unique_ptrs to the items being listed. This is a problem, since we track the currently-selected item with a raw pointer which could be stale. This was the cause of http://www. crbug.com/749815 , a crasher.

We should change this to use shared_ptr for the vector and weak_ptr for selected_item.