// Copyright (c) 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#include "base/android/jni_registrar.h"

#include "base/logging.h"
#include "base/android/jni_android.h"

namespace base {
namespace android {

bool RegisterNativeMethods(JNIEnv* env,
                           const RegistrationMethod* method,
                           size_t count) {
  const RegistrationMethod* end = method + count;
  while (method != end) {
    if (!method->func(env) < 0) {
      DLOG(ERROR) << method->name << " failed registration!";
      return false;
    }
    method++;
  }
  return true;
}

}  // namespace android
}  // namespace base

// Copyright (c) 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#include "base/android/jni_registrar.h"

#include "base/logging.h"
#include "base/android/jni_android.h"

namespace base {
namespace android {

bool RegisterNativeMethods(JNIEnv* env,
                           const RegistrationMethod* method,
                           size_t count) {
  const RegistrationMethod* end = method + count;
  while (method != end) {
    if (!method->func(env)) {
      DLOG(ERROR) << method->name << " failed registration!";
      return false;
    }
    method++;
  }
  return true;
}

}  // namespace android
}  // namespace base

Fix Android RegisterNativeMethods failure logic
Fix Android RegisterNativeMethods failure logic

Function returns bool, so !func(env) should be enough. The < 0 check is
probably leftover crud when upstreamed.

BUG=


Review URL: https://chromiumcodereview.appspot.com/11833047

git-svn-id: de016e52bd170d2d4f2344f9bf92d50478b649e0@176266 0039d316-1c4b-4281-b951-d872f2087c98
