Completely rewrite the PpdProvider/PpdCache to use the SCS backend. Along the way, clean it up a l…

components/sync/protocol/printer_specifics.proto

 // Copyright 2016 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.
 //
 // Sync protocol datatype extension for printer data.
 
 // Update proto_value_conversions{.h,.cc,_unittest.cc} if you change
 // any fields in this file.
 
 syntax = "proto2";
 
 option optimize_for = LITE_RUNTIME;
 
 package sync_pb;
 
 // User PPD configuration
 message PrinterPPDReference {
   // Url for user provided file.  Overrides other fields.
   optional string user_supplied_ppd_url = 1;
 
-  // Make and model to reference into PPD Server.
-  optional string effective_manufacturer = 2;
-  optional string effective_model = 3;
+  // FIELD 2 RETIRED

[skau, 2017/01/27 01:37:02]

If it compiles (it was supposed to be fixed) can we use reserved 2,3 here?

[Carlson, 2017/01/27 18:48:43]

Done.

+  // FIELD 3 RETIRED
+
+  // Key used to find a ppd for this printer from the ppd server.
+  optional string ppd_server_key = 4;
 }
 
 message PrinterSpecifics {
   // Printer record GUID
   optional string id = 1;
 
   // User visible name.  Any string.
   optional string display_name = 2;
 
   // User visible description.  Any string.
@@ skipping 13 matching lines @@
   // Universally Unique Identifier provided by the printer.  Used for unique
   // identification of printers in a zeroconf environment.
   optional string uuid = 7;
 
   // PPDData was deprecated in favor of PPDReference format.
   optional bytes ppd = 8 [deprecated = true];
 
   // Structure representing the user's ppd configuration.
   optional PrinterPPDReference ppd_reference = 9;
 }