skia: Add runtime option to disable picture IO security precautions

src/core/SkPicture.cpp

 /*
  * Copyright 2007 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkAtomics.h"
 #include "SkMessageBus.h"
 #include "SkPicture.h"
 #include "SkPictureData.h"
 #include "SkPicturePlayback.h"
 #include "SkPictureRecord.h"
 #include "SkPictureRecorder.h"
 
+namespace {
+#if defined(SK_DISALLOW_CROSSPROCESS_PICTUREIMAGEFILTERS) || \
+    defined(SK_ENABLE_PICTURE_IO_SECURITY_PRECAUTIONS)
+bool gAllPictureIOSecurityPrecautionsEnabled = true;

[mtklein, 2015/06/16 15:57:49]

Skia tends to use static where possible instead of namespace {} and name them
g_like_this, so

static bool g_all_picture_io_security_precautions_enabled

[hendrikw, 2015/06/16 16:09:37]

Done.

+#else
+bool gAllPictureIOSecurityPrecautionsEnabled = false;
+#endif
+}  // namespac
+
 DECLARE_SKMESSAGEBUS_MESSAGE(SkPicture::DeletionMessage);
 
 /* SkPicture impl.  This handles generic responsibilities like unique IDs and serialization. */
 
 SkPicture::SkPicture() : fUniqueID(0) {}
 
 SkPicture::~SkPicture() {
     // TODO: move this to ~SkBigPicture() only?
 
     // If the ID is still zero, no one has read it, so no need to send this message.
@@ skipping 163 matching lines @@
     }
 }
 
 bool SkPicture::suitableForGpuRasterization(GrContext*, const char** whyNot) const {
     if (this->numSlowPaths() > 5) {
         if (whyNot) { *whyNot = "Too many slow paths (either concave or dashed)."; }
         return false;
     }
     return true;
 }
+
+// Global setting to disable security precautions for serialization.
+void SkPicture::setPictureIOSecurityPrecautionsEnabled_Dangerous(bool set) {
+    gAllPictureIOSecurityPrecautionsEnabled = set;
+}
+
+bool SkPicture::pictureIOSecurityPrecautionsEnabled() {
+    return gAllPictureIOSecurityPrecautionsEnabled;
+}