Remove usage of CHECK_NOTNULL

We want to switch to absl logging instead of glog.  gtest and ceres are
going there, and we already have absl as a dependency.  ABSL doesn't
have CHECK_NOTNULL, and we can move things over in an easier to review
fashion.

Change-Id: Ifd9a11ec34a2357cec43f88dba015db9c28ed2cf
Signed-off-by: Austin Schuh <austin.linux@gmail.com>
diff --git a/aos/flatbuffer_utils.h b/aos/flatbuffer_utils.h
index aa22e68..003f141 100644
--- a/aos/flatbuffer_utils.h
+++ b/aos/flatbuffer_utils.h
@@ -24,11 +24,8 @@
 class FlatbufferType final {
  public:
   // Implicit on purpose, to allow freely creating a FlatbufferType.
-  FlatbufferType(const flatbuffers::TypeTable *type_table)
-      : type_table_(CHECK_NOTNULL(type_table)) {}
-  FlatbufferType(const reflection::Schema *schema)
-      : schema_(CHECK_NOTNULL(schema)),
-        object_(DCHECK_NOTNULL(schema->root_table())) {}
+  FlatbufferType(const flatbuffers::TypeTable *type_table);
+  FlatbufferType(const reflection::Schema *schema);
 
   // This is deliberately copyable, for ease of memory management. It is cheap
   // to pass by value.
@@ -114,11 +111,9 @@
 
  private:
   explicit FlatbufferType(const reflection::Schema *schema,
-                          const reflection::Object *object)
-      : schema_(DCHECK_NOTNULL(schema)), object_(DCHECK_NOTNULL(object)) {}
+                          const reflection::Object *object);
   explicit FlatbufferType(const reflection::Schema *schema,
-                          const reflection::Enum *fb_enum)
-      : schema_(DCHECK_NOTNULL(schema)), enum_(DCHECK_NOTNULL(fb_enum)) {}
+                          const reflection::Enum *fb_enum);
 
   const reflection::Type *ReflectionType(int index) const;
   const reflection::Field *ReflectionObjectField(int index) const;