allow renaming of json serialization related things

Summary:
After D3983103 and D4246140 we can rename Foo <-> FooSerializer <-> FooDeserializer related classes.

One class that's a hold out to the various machinery that allows this is FeedUnit and its serdes. So, blacklist them specifically in proguard.

Reviewed By: int3

Differential Revision: D4326199

fbshipit-source-id: 492e1975814a4e0936c4906f92fd2f7f6c7d3292

opt/renameclasses/RenameClassesV2.cpp

@@ -585,13 +585,13 @@ void RenameClassesPassV2::eval_classes(
       continue;
     }
 
-    if (dont_rename_serde_relationships.count(clazz->get_type()) > 0) {
-      m_dont_rename_reasons[clazz] = { DontRenameReasonCode::SerdeRelationships, norule };
+    if (!can_rename_if_ignoring_blanket_keep(clazz)) {
+      m_dont_rename_reasons[clazz] = { DontRenameReasonCode::ProguardCantRename, norule };
       continue;
     }
 
-    if (!can_rename_if_ignoring_blanket_keep(clazz)) {
-      m_dont_rename_reasons[clazz] = { DontRenameReasonCode::ProguardCantRename, norule };
+    if (dont_rename_serde_relationships.count(clazz->get_type()) > 0) {
+      m_dont_rename_reasons[clazz] = { DontRenameReasonCode::SerdeRelationships, norule };
       continue;
     }
   }