diff options
author | hans@chromium.org <hans@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-16 11:27:47 +0000 |
---|---|---|
committer | hans@chromium.org <hans@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-16 11:27:47 +0000 |
commit | 6f50481e509709864bd3294a28374ffc0a255199 (patch) | |
tree | 282d731dbf7dd7c7e3d2cb928f261a3a82430996 /android_webview/native/aw_autofill_manager_delegate.cc | |
parent | 2c7f8cf810963623e5d0f4fd6fe40ba8c96a94c8 (diff) | |
download | chromium_src-6f50481e509709864bd3294a28374ffc0a255199.zip chromium_src-6f50481e509709864bd3294a28374ffc0a255199.tar.gz chromium_src-6f50481e509709864bd3294a28374ffc0a255199.tar.bz2 |
WebView: RegisterNativesImpl() returns bool; don't compare it for >= 0.
A recent version of Clang warns about this:
../../android_webview/native/java_browser_view_renderer_helper.cc:133:35: error: comparison of constant 0 with expression of type 'bool' is always true [-Werror,-Wtautological-constant-out-of-range-compare]
return RegisterNativesImpl(env) >= 0;
~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~
BUG=363667
Review URL: https://codereview.chromium.org/238043003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264168 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'android_webview/native/aw_autofill_manager_delegate.cc')
-rw-r--r-- | android_webview/native/aw_autofill_manager_delegate.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/android_webview/native/aw_autofill_manager_delegate.cc b/android_webview/native/aw_autofill_manager_delegate.cc index e1107d6..070e25e 100644 --- a/android_webview/native/aw_autofill_manager_delegate.cc +++ b/android_webview/native/aw_autofill_manager_delegate.cc @@ -200,7 +200,7 @@ void AwAutofillManagerDelegate::ShowRequestAutocompleteDialog( } bool RegisterAwAutofillManagerDelegate(JNIEnv* env) { - return RegisterNativesImpl(env) >= 0; + return RegisterNativesImpl(env); } } // namespace android_webview |