Use SkASSERTResult to avoid unused local variables

tools/viewer/sk_app/android/surface_glue_android.cpp

 /*
 * Copyright 2016 Google Inc.
 *
 * Use of this source code is governed by a BSD-style license that can be
 * found in the LICENSE file.
 */
 
 #include "surface_glue_android.h"
 
 #include <jni.h>
@@ skipping 59 matching lines @@
     fPThreadEnv->DeleteLocalRef(titleString);
 }
 
 void SkiaAndroidApp::paintIfNeeded() {
     if (fNativeWindow && fWindow) {
         fWindow->onPaint();
     }
 }
 
 void SkiaAndroidApp::postMessage(const Message& message) const {
-    auto writeSize = write(fPipes[1], &message, sizeof(message));
-    SkASSERT(writeSize == sizeof(message));
+    SkAssertResult(write(fPipes[1], &message, sizeof(message)) == sizeof(message));
 }
 
 void SkiaAndroidApp::readMessage(Message* message) const {
-    auto readSize = read(fPipes[0], message, sizeof(Message));
-    SkASSERT(readSize == sizeof(Message));
+    SkAssertResult(read(fPipes[0], message, sizeof(Message)) == sizeof(Message));

[egdaniel, 2016/05/20 18:50:23]

So just doing a fly by unsolicited review. Here would it be better to have just
surrounded the "auto read/writeSize =" in SkDEBUGCODE(...) so we don't do the
compare on release?

[liyuqian, 2016/05/20 18:53:00]

That's a good point. However, I've already committed the change... Shall I
create another CL?

 }
 
 void SkiaAndroidApp::inval() {
     SkAutoMutexAcquire ama(fMutex);
     if (!fIsContentInvalidated) {
         postMessage(Message(kContentInvalidated));
         fIsContentInvalidated = true;
     }
 }
 
@@ skipping 142 matching lines @@
     auto skiaAndroidApp = (SkiaAndroidApp*)handle;
     Message message(kTouched);
     message.fTouchOwner = owner;
     message.fTouchState = state;
     message.fTouchX = x;
     message.fTouchY = y;
     skiaAndroidApp->postMessage(message);
 }
 
 }  // namespace sk_app