History log of /external/skia/src/ports/SkOSFile_posix.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
d5e4d85843f92921660e704333df8afe8c582dc8 16-Jan-2018 Brian Osman <brianosman@google.com> Fix file loading from bundles on iOS

Bug: skia:
Change-Id: I9b9a7dcbdeb1630794b04d2a5a00d5f496aae85d
Reviewed-on: https://skia-review.googlesource.com/94963
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/src/ports/SkOSFile_posix.cpp
329c5a6d826b7b90e5e75ec0a7824c75ed5f8077 29-Nov-2017 Jim Van Verth <jvanverth@google.com> Bundle resources and skps directories into iOS app.

Will bundle resources/ for viewer (and skps/ if
that directory exists in the main Skia directory).
Also updates file code on iOS to fall back to bundle directory.

Docs-Preview: https://skia.org/?cl=76803
Bug: skia:7339
Change-Id: I244f67559c866451a6d02c3f1c4948d89457ec84
Reviewed-on: https://skia-review.googlesource.com/76803
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>
/external/skia/src/ports/SkOSFile_posix.cpp
a93a14a99816d25b773f0b12868143702baf44bf 28-Aug-2017 Ben Wagner <bungeman@google.com> Convert NULL and 0 to nullptr.

This was created by looking at warnings produced by clang's
-Wzero-as-null-pointer-constant. This updates most issues in
Skia code. However, there are places where GL and Vulkan want
pointer values which are explicitly 0, external headers which
use NULL directly, and possibly more uses in un-compiled
sources (for other platforms).

Change-Id: Id22fbac04d5c53497a53d734f0896b4f06fe8345
Reviewed-on: https://skia-review.googlesource.com/39521
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
/external/skia/src/ports/SkOSFile_posix.cpp
564ad05f065297658e691315993610e8055295c1 23-May-2017 Leon Scroggins <scroggo@google.com> Revert "sk_fgetsize to not use ftell."

This reverts commit eefa289a214ea3917e5e00554f231adb5ad6f712.

Reason for revert: Causing failures in BitmapFactoryTest. b/38233042

Original change's description:
> sk_fgetsize to not use ftell.
>
> The previous version of sk_fgetsize used ftell and fseek to compute
> the size of a file. There are so many issues with this that it is called
> out by securecoding.cert.org as FIO19-C as a thing not to do. We already
> have correct code for computing the size of a file in the mmap code, so
> use that instead.
>
> Change-Id: I1d771124989d0ec1523f6d858814ee563263213a
> Reviewed-on: https://skia-review.googlesource.com/9860
> Reviewed-by: Leon Scroggins <scroggo@google.com>
> Commit-Queue: Ben Wagner <bungeman@google.com>
>

TBR=bungeman@google.com,scroggo@google.com
# Not skipping CQ checks because original CL landed > 1 day ago.

Change-Id: Ie9392dde8747ae7c74ebfa00153705e316e841a2
Reviewed-on: https://skia-review.googlesource.com/17705
Reviewed-by: Leon Scroggins <scroggo@google.com>
Commit-Queue: Leon Scroggins <scroggo@google.com>
/external/skia/src/ports/SkOSFile_posix.cpp
eefa289a214ea3917e5e00554f231adb5ad6f712 17-Mar-2017 Ben Wagner <bungeman@google.com> sk_fgetsize to not use ftell.

The previous version of sk_fgetsize used ftell and fseek to compute
the size of a file. There are so many issues with this that it is called
out by securecoding.cert.org as FIO19-C as a thing not to do. We already
have correct code for computing the size of a file in the mmap code, so
use that instead.

Change-Id: I1d771124989d0ec1523f6d858814ee563263213a
Reviewed-on: https://skia-review.googlesource.com/9860
Reviewed-by: Leon Scroggins <scroggo@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
/external/skia/src/ports/SkOSFile_posix.cpp
4d1955c43aaab045511b74a495dfbea4ef0057c5 10-Mar-2017 Ben Wagner <bungeman@google.com> Fix SkFILEStream.

Change-Id: I8c66e4e3e857227aed3d0bc497982f4c0d96d917
Reviewed-on: https://skia-review.googlesource.com/9498
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Leon Scroggins <scroggo@google.com>
/external/skia/src/ports/SkOSFile_posix.cpp
d76be9c79baa1530d3dc95c58022e83607a1bb2a 20-Nov-2015 halcanary <halcanary@google.com> Eliminate SkFILE: it always is the same as FILE.

Review URL: https://codereview.chromium.org/1467533003
/external/skia/src/ports/SkOSFile_posix.cpp
96fcdcc219d2a0d3579719b84b28bede76efba64 27-Aug-2015 halcanary <halcanary@google.com> Style Change: NULL->nullptr
DOCS_PREVIEW= https://skia.org/?cl=1316233002

Review URL: https://codereview.chromium.org/1316233002
/external/skia/src/ports/SkOSFile_posix.cpp
385fe4d4b62d7d1dd76116dd570df3290a2f487b 26-Aug-2015 halcanary <halcanary@google.com> Style Change: SkNEW->new; SkDELETE->delete
DOCS_PREVIEW= https://skia.org/?cl=1316123003

Review URL: https://codereview.chromium.org/1316123003
/external/skia/src/ports/SkOSFile_posix.cpp
99fe82260633fcf5d92cca38d12ef0937ecca61c 20-Aug-2015 bungeman <bungeman@google.com> Use static_assert instead of SK_COMPILE_ASSERT.

Now that static_assert is allowed, there is no need to use a non-
standard compile time assertion

Review URL: https://codereview.chromium.org/1306443004
/external/skia/src/ports/SkOSFile_posix.cpp
f20488b4f2139e6ca09fee7e39b731dd8ab467db 29-Jul-2015 bungeman <bungeman@google.com> Clean up a few includes, introduce iwyu.

The current include-what-you-use with current clang is much less
noisy and more useful than it has been in the past. This change
introduces a few IWYU directives (which are helpful documentation for
humans as well) and fixes a few sets of includes.

Review URL: https://codereview.chromium.org/1207893002
/external/skia/src/ports/SkOSFile_posix.cpp
e998b7ff3a398eaa2f359d4af1a1898ea2824938 12-Feb-2015 bungeman <bungeman@google.com> Move SkOSFile::Iter impls into ports.

This was one large ifdef in SkOSFile.cpp in utils.
This moves the code to existing ports files.

Review URL: https://codereview.chromium.org/920593002
/external/skia/src/ports/SkOSFile_posix.cpp
bf0b9ced0b93d9684b044e0880691768f9aa4394 11-Jul-2014 bungeman <bungeman@google.com> Add file access modes to sk_exists.

Both Windows and Posix 'access' calls take a mode parameter which,
in addition to checking existence, checks access modes.
This change exposes this functionality.

R=mtklein@google.com, reed@google.com

Author: bungeman@google.com

Review URL: https://codereview.chromium.org/384903002
/external/skia/src/ports/SkOSFile_posix.cpp
f5cc5b140c1c00c536e02b5cfbe158bb2d5c2c15 12-Jul-2013 bungeman@google.com <bungeman@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> R=mtklein@google.com

Review URL: https://codereview.chromium.org/18503009

git-svn-id: http://skia.googlecode.com/svn/trunk@10050 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/ports/SkOSFile_posix.cpp
11f2b444500d552031fcae0b381a0770600400fd 04-Jun-2013 skia.committer@gmail.com <skia.committer@gmail.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Sanitizing source files in Housekeeper-Nightly

git-svn-id: http://skia.googlecode.com/svn/trunk@9423 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/ports/SkOSFile_posix.cpp
11c9a55afd95078d14ab8cd7c1c5c0032af2a498 03-Jun-2013 bungeman@google.com <bungeman@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Add SkData::NewFromFD.

Chromium needs a SkStream backed by a file descriptor.
Skia already has the code and can do the work, this change exposes the
functionality in Skia in a clean way.

https://codereview.chromium.org/15941025/


git-svn-id: http://skia.googlecode.com/svn/trunk@9408 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/ports/SkOSFile_posix.cpp
6cab1a4b6a68aa81237731308ff37a646d48f51c 29-May-2013 bungeman@google.com <bungeman@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Change SkStream.
https://codereview.chromium.org/15298009/


git-svn-id: http://skia.googlecode.com/svn/trunk@9312 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/ports/SkOSFile_posix.cpp