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

components/sync/protocol/printer_specifics.proto

Index: components/sync/protocol/printer_specifics.proto
diff --git a/components/sync/protocol/printer_specifics.proto b/components/sync/protocol/printer_specifics.proto
index e0881b71dd56b355ff42511fa9a22ba9d084f94e..e90813334db43c864244d3785dcb012bb51efa73 100644
--- a/components/sync/protocol/printer_specifics.proto
+++ b/components/sync/protocol/printer_specifics.proto
@@ -18,9 +18,12 @@ 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;
+  // Retired fields
+  reserved 2 to 4;

[skau, 2017/01/27 19:16:10]

You can use 4.  We never committed a CL that reserves 4.

https://cs.chromium.org/chromium/src/components/sync/protocol/printer_specifi...

[Carlson, 2017/01/27 22:56:22]

Indeed, that was used in a previous revision of *this* cl.  :)  done.

+
+  // String identifying the type of printer, used to look up a ppd to drive the
+  // printer.
+  optional string effective_make_and_model = 5;
 }
 
 message PrinterSpecifics {