Fix DCHECK() when updating instruments with no phone number.

components/autofill/content/browser/wallet/wallet_client.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_AUTOFILL_CONTENT_BROWSER_WALLET_WALLET_CLIENT_H_
 #define COMPONENTS_AUTOFILL_CONTENT_BROWSER_WALLET_WALLET_CLIENT_H_
 
 #include <queue>
 #include <string>
 #include <vector>
@@ skipping 147 matching lines @@
   // |card_verification_number|. |delegate_| is notified when the request is
   // complete. Used to respond to Risk challenges.
   virtual void AuthenticateInstrument(
       const std::string& instrument_id,
       const std::string& card_verification_number);
 
   // GetFullWallet retrieves the a FullWallet for the user.
   virtual void GetFullWallet(const FullWalletRequest& full_wallet_request);
 
   // Saves the data in |instrument| and/or |address| to Wallet. |instrument|
-  // does not have to be complete if its being used to update an existing
+  // does not have to be complete if it's being used to update an existing
   // instrument, like in the case of expiration date or address only updates.
-  virtual void SaveToWallet(scoped_ptr<Instrument> instrument,
-                            scoped_ptr<Address> address);
+  // |reference_instrument| and |reference_address| are the original instrument
+  // and address to be updated on the server (and should be NULL if |instrument|
+  // or |address| are new data).
+  virtual void SaveToWallet(
+      scoped_ptr<Instrument> instrument,
+      scoped_ptr<Address> address,
+      const WalletItems::MaskedInstrument* reference_instrument,
+      const Address* reference_address);
 
   bool HasRequestInProgress() const;
 
   // Cancels and clears the current |request_| and |pending_requests_| (if any).
   void CancelRequests();
 
   size_t user_index() const { return user_index_; }
   void set_user_index(size_t user_index) {
     user_index_ = user_index;
   }
@@ skipping 13 matching lines @@
 
   // Like AcceptLegalDocuments, but takes a vector of document ids.
   void DoAcceptLegalDocuments(
       const std::vector<std::string>& document_ids,
       const std::string& google_transaction_id);
 
   // Posts |post_body| to |url| with content type |mime_type| and notifies
   // |delegate_| when the request is complete.
   void MakeWalletRequest(const GURL& url,
                          const std::string& post_body,
-                         const std::string& mime_type);
+                         const std::string& mime_type,
+                         RequestType request_type);
 
   // Performs bookkeeping tasks for any invalid requests.
   void HandleMalformedResponse(RequestType request_type,
                                net::URLFetcher* request);
   void HandleNetworkError(int response_code);
   void HandleWalletError(ErrorType error_type);
 
   // Start the next pending request (if any).
   void StartNextPendingRequest();
 
@@ skipping 41 matching lines @@
 
   base::WeakPtrFactory<WalletClient> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WalletClient);
 };
 
 }  // namespace wallet
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CONTENT_BROWSER_WALLET_WALLET_CLIENT_H_