Just found a couple of things that needed cleanup during
the investigation of the bug below. So, this CL:
- Removes unnecessary CHECK(env); after
JNIEnv* env = AttachCurrentThread();
- s/__PRETTY_FUNCTION__/__FUNCTION__/ because the former
is illegible in adb logcat or any other log, really.
And concretely in GpuVideoEncodeAccelerator this CL:
- moves the static methods
out of the class and into anonymous namespace of the .cc
file, since they don't need to be in the class at all.
- nukes (*CreateVEAFp)() in that class and uses instead a
base::Callback(), bound to the static method mentioned above.
- uses for-range loops.
TEST=all unittests. content_browsertests, browser_tests etc
working, and tested by hand in N7.