Catch exceptions when parsing tile PBFs

Also improved the error message and updated the test case.

2 files changed, 9 insertions, 3 deletions

diff --git a/src/mbgl/map/vector_tile_data.cpp b/src/mbgl/map/vector_tile_data.cpp
index be26408492..bf4c0f5814 100644
--- a/src/mbgl/map/vector_tile_data.cpp
+++ b/src/mbgl/map/vector_tile_data.cpp
@@ -76,7 +76,9 @@ bool VectorTileData::reparse(Worker&, std::function<void()> callback) {
         if (result.is<State>()) {
             state = result.get<State>();
         } else {
-            error = result.get<std::string>();
+            std::stringstream message;
+            message <<  "Failed to parse [" << std::string(id) << "]: " << result.get<std::string>();
+            error = message.str();
             state = State::obsolete;
         }
 
diff --git a/src/mbgl/util/worker.cpp b/src/mbgl/util/worker.cpp
index 74408a7daf..0d83bc49f4 100644
--- a/src/mbgl/util/worker.cpp
+++ b/src/mbgl/util/worker.cpp
@@ -25,8 +25,12 @@ public:
     }
 
     TileParseResult parseVectorTile(TileWorker* worker, std::string data) {
-        VectorTile vectorTile(pbf(reinterpret_cast<const unsigned char *>(data.data()), data.size()));
-        return worker->parse(vectorTile);
+        try {
+            pbf tilePBF(reinterpret_cast<const unsigned char *>(data.data()), data.size());
+            return worker->parse(VectorTile(tilePBF));
+        } catch (const std::exception& ex) {
+            return TileParseResult(ex.what());
+        }
     }
 
     TileParseResult parseLiveTile(TileWorker* worker, const LiveTile* tile) {