Per discussion with WG, update table ordering to what OTS expects. Once browsers are on latest cut of OTS we will go back to spec order.

src/font.cc

@@ -49,8 +49,11 @@ std::vector<uint32_t> Font::OutputOrderedTags() const {
     output_order.push_back(table.tag);
   }
 
-  // alphabetize (not required), then put loca immediately after glyf (required)
+  // Alphabetize and do not put loca immediately after glyf
+  // This violates woff2 spec but results in a font that passes OTS
   std::sort(output_order.begin(), output_order.end());
+  // TODO(user): change to match spec once browsers are on newer OTS
+  /*
   auto glyf_loc = std::find(output_order.begin(), output_order.end(),
       kGlyfTableTag);
   auto loca_loc = std::find(output_order.begin(), output_order.end(),
@@ -59,7 +62,7 @@ std::vector<uint32_t> Font::OutputOrderedTags() const {
     output_order.erase(loca_loc);
     output_order.insert(std::find(output_order.begin(), output_order.end(),
       kGlyfTableTag) + 1, kLocaTableTag);
-  }
+  }*/
 
   return output_order;
 }