Take care fall-through cases of switch in pdfium

fpdfsdk/src/javascript/JS_GlobalData.cpp

Index: fpdfsdk/src/javascript/JS_GlobalData.cpp
diff --git a/fpdfsdk/src/javascript/JS_GlobalData.cpp b/fpdfsdk/src/javascript/JS_GlobalData.cpp
index 474886c4846c85b1f287abca441d230d16d3fba1..4d6edc898d58152a93f7c5a03e4a570422ba33c0 100644
--- a/fpdfsdk/src/javascript/JS_GlobalData.cpp
+++ b/fpdfsdk/src/javascript/JS_GlobalData.cpp
@@ -51,13 +51,13 @@ void CJS_GlobalVariableArray::Copy(const CJS_GlobalVariableArray& array) {
         pNewObjData->nType = pOldObjData->nType;
         pNewObjData->objData.Copy(pOldObjData->objData);
         Add(pNewObjData);
-      }
+      } break;

[Lei Zhang, 2015/12/28 20:55:00]

This should be fixed, but it does not matter so much because this is dead code
and I'm debating whether to remove it or to implement code to call it.

       case JS_GLOBALDATA_TYPE_NULL: {
         CJS_KeyValue* pNewObjData = new CJS_KeyValue;
         pNewObjData->sKey = pOldObjData->sKey;
         pNewObjData->nType = pOldObjData->nType;
         Add(pNewObjData);
-      }
+      } break;
     }
   }
 }