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

skia/config/sk_ref_cnt_ext_release.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef SK_REF_CNT_EXT_RELEASE_H_
 #define SK_REF_CNT_EXT_RELEASE_H_
 
 #ifdef SK_REF_CNT_EXT_DEBUG_H_
 #error Only one SkRefCnt should be used.
 #endif
 
 // Alternate implementation of SkRefCnt for Chromium release builds
 class SK_API SkRefCnt : public SkRefCntBase {
 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