From 3aeda5c5c47ae29197e8b8b665749cb6cb449e19 Mon Sep 17 00:00:00 2001
From: Ben Wagner <bungeman@google.com>
Date: Fri, 17 Mar 2017 17:22:01 -0400
Subject: [PATCH] Use access directly instead of sk_exists.

Skia would like to move SkOSFile and make it private spi. This appears
to be the only place SkOSFile is used outside Skia, and using access
here directly is more apropriate.

Test: Simple inline refactor, built and ran. Changes test code only.
Change-Id: Ib76c180bb3bccc54c20dcadc842837e3e8270929
---
 libs/hwui/pipeline/skia/SkiaPipeline.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/libs/hwui/pipeline/skia/SkiaPipeline.cpp b/libs/hwui/pipeline/skia/SkiaPipeline.cpp
index 11dc1f4c2161d..10c1865ac50c7 100644
--- a/libs/hwui/pipeline/skia/SkiaPipeline.cpp
+++ b/libs/hwui/pipeline/skia/SkiaPipeline.cpp
@@ -19,7 +19,6 @@
 #include "utils/TraceUtils.h"
 #include <SkImageEncoder.h>
 #include <SkImagePriv.h>
-#include <SkOSFile.h>
 #include <SkOverdrawCanvas.h>
 #include <SkOverdrawColorFilter.h>
 #include <SkPicture.h>
@@ -27,6 +26,8 @@
 #include <SkPixelSerializer.h>
 #include <SkStream.h>
 
+#include <unistd.h>
+
 using namespace android::uirenderer::renderthread;
 
 namespace android {
@@ -190,7 +191,7 @@ void SkiaPipeline::renderFrame(const LayerUpdateQueue& layers, const SkRect& cli
     char prop[PROPERTY_VALUE_MAX];
     if (skpCaptureEnabled()) {
         property_get("debug.hwui.capture_frame_as_skp", prop, "0");
-        recordingPicture = prop[0] != '0' && !sk_exists(prop);
+        recordingPicture = prop[0] != '0' && access(prop, F_OK) != 0;
         if (recordingPicture) {
             recorder.reset(new SkPictureRecorder());
             canvas = recorder->beginRecording(surface->width(), surface->height(),