Add plumbing in blink to allow overriding the skia font manager.

skia/config/sk_ref_cnt_ext_release.h

Index: skia/config/sk_ref_cnt_ext_release.h
diff --git a/skia/config/sk_ref_cnt_ext_release.h b/skia/config/sk_ref_cnt_ext_release.h
index 7736c84cf7e1d4280a260f410a7864e7d4053d95..aec051283cb8cd9ddfee73a3a99a8e83f0768a06 100644
--- a/skia/config/sk_ref_cnt_ext_release.h
+++ b/skia/config/sk_ref_cnt_ext_release.h
@@ -15,5 +15,13 @@ public:
   void deref() const { SkRefCntBase::unref(); }
 };
 
+namespace WTF {

[bungeman-skia, 2016/02/12 15:45:11]

Is there a reason for adding any of this? WTF::adopted(void*) always exists and
already does nothing, and in release doesn't need to do anything.

[Ilya Kulshin, 2016/02/12 20:47:10]

I don't want to force callers to rely on a different component, just to call
these APIs. Since the debug build exposes these functions, I think the release
build should make them available as well. At the very least this makes it
clearer what happens in release vs debug.

+  inline void adopted(const SkRefCnt* object) {}
+  inline void requireAdoption(const SkRefCnt* object) {}
+};
+
+using WTF::adopted;

[bungeman-skia, 2016/02/12 15:45:11]

Are these needed? I would prefer these be removed and the corresponding ones in
the _debug.h version be removed as well, if possible. If this doesn't have a
demonstrated use case, it's just needlessly polluting the global namespace. I
can put together a CL separately to remove them from the _debug.h file, but I
wouldn't add this here if it isn't necessary.

[Ilya Kulshin, 2016/02/12 20:47:10]

I tried removing them, but that caused a bunch of breaks and would require
changes that should be in a separate change.

+using WTF::requireAdoption;
+
 #endif