When we read from a RemoteFontStream (might happen maybe with other streams), stream's data might b…

src/ports/SkFontHost_fontconfig.cpp

 /*
  * Copyright 2008 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkFontConfigInterface.h"
 #include "SkFontDescriptor.h"
 #include "SkFontHost.h"
@@ skipping 229 matching lines @@
     }
 
     return SkFontHost::CreateTypeface(NULL, familyName, style);
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
 static SkStream* open_stream(const FontConfigTypeface* face, int* ttcIndex) {
     SkStream* stream = face->getLocalStream();
     if (stream) {
+        // TODO: fix issue 1176.
+        // As of now open_stream will return a stream and unwind it, but the
+        // SkStream is not thread safe, and if two threads use the stream they
+        // may collide and print preview for example could still fail,
+        // or there could be some failures in rendering if this stream is used
+        // there.
+        stream->rewind();
         stream->ref();
         // should have been provided by CreateFromStream()
         *ttcIndex = 0;
     } else {
         SkAutoTUnref<SkFontConfigInterface> fci(RefFCI());
         if (NULL == fci.get()) {
             return NULL;
         }
         stream = fci->openStream(face->getIdentity());
         *ttcIndex = face->getIdentity().fTTCIndex;
@@ skipping 49 matching lines @@
     return stream.get()
                 ? SkFontStream::GetTableData(stream, ttcIndex,
                                              tag, offset, length, data)
                 : 0;
 }
 
 void FontConfigTypeface::onGetFontDescriptor(SkFontDescriptor* desc) const {
     desc->setStyle(this->style());
     desc->setFamilyName(this->getFamilyName());
 }