Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(144)

Side by Side Diff: src/utils/win/SkDWriteFontFileStream.cpp

Issue 861413002: Remove staging SkStream::unref(). (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Fix stragglers. Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright 2012 Google Inc. 2 * Copyright 2012 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkTypes.h" 8 #include "SkTypes.h"
9 #include "SkDWriteFontFileStream.h" 9 #include "SkDWriteFontFileStream.h"
10 #include "SkHRESULT.h" 10 #include "SkHRESULT.h"
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 size_t length = this->getLength(); 96 size_t length = this->getLength();
97 fPos = (position > length) ? length : position; 97 fPos = (position > length) ? length : position;
98 return true; 98 return true;
99 } 99 }
100 100
101 bool SkDWriteFontFileStream::move(long offset) { 101 bool SkDWriteFontFileStream::move(long offset) {
102 return seek(fPos + offset); 102 return seek(fPos + offset);
103 } 103 }
104 104
105 SkDWriteFontFileStream* SkDWriteFontFileStream::fork() const { 105 SkDWriteFontFileStream* SkDWriteFontFileStream::fork() const {
106 SkAutoTUnref<SkDWriteFontFileStream> that(this->duplicate()); 106 SkAutoTDelete<SkDWriteFontFileStream> that(this->duplicate());
107 that->seek(fPos); 107 that->seek(fPos);
108 return that.detach(); 108 return that.detach();
109 } 109 }
110 110
111 size_t SkDWriteFontFileStream::getLength() const { 111 size_t SkDWriteFontFileStream::getLength() const {
112 HRESULT hr = S_OK; 112 HRESULT hr = S_OK;
113 UINT64 realFileSize = 0; 113 UINT64 realFileSize = 0;
114 hr = fFontFileStream->GetFileSize(&realFileSize); 114 hr = fFontFileStream->GetFileSize(&realFileSize);
115 if (!SkTFitsIn<size_t>(realFileSize)) { 115 if (!SkTFitsIn<size_t>(realFileSize)) {
116 return 0; 116 return 0;
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
223 HRESULT STDMETHODCALLTYPE SkDWriteFontFileStreamWrapper::GetFileSize(UINT64* fil eSize) { 223 HRESULT STDMETHODCALLTYPE SkDWriteFontFileStreamWrapper::GetFileSize(UINT64* fil eSize) {
224 *fileSize = fStream->getLength(); 224 *fileSize = fStream->getLength();
225 return S_OK; 225 return S_OK;
226 } 226 }
227 227
228 HRESULT STDMETHODCALLTYPE SkDWriteFontFileStreamWrapper::GetLastWriteTime(UINT64 * lastWriteTime) { 228 HRESULT STDMETHODCALLTYPE SkDWriteFontFileStreamWrapper::GetLastWriteTime(UINT64 * lastWriteTime) {
229 // The concept of last write time does not apply to this loader. 229 // The concept of last write time does not apply to this loader.
230 *lastWriteTime = 0; 230 *lastWriteTime = 0;
231 return E_NOTIMPL; 231 return E_NOTIMPL;
232 } 232 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698