Merge "Treating all FileSection files as device specific."

cmds/incidentd/src/Section.cpp

@@ -255,7 +255,9 @@ status_t FileSection::Execute(ReportRequestSet* requests) const {
     unique_fd fd(open(mFilename, O_RDONLY | O_CLOEXEC));
     if (fd.get() == -1) {
         ALOGW("FileSection '%s' failed to open file", this->name.string());
-        return this->deviceSpecific ? NO_ERROR : -errno;
+        // There may be some devices/architectures that won't have the file.
+        // Just return here without an error.
+        return NO_ERROR;
     }
 
     FdBuffer buffer;

cmds/incidentd/tests/Section_test.cpp

@@ -145,7 +145,7 @@ TEST_F(SectionTest, FileSection) {
 
 TEST_F(SectionTest, FileSectionNotExist) {
     FileSection fs1(NOOP_PARSER, "notexist", false, QUICK_TIMEOUT_MS);
-    ASSERT_EQ(NAME_NOT_FOUND, fs1.Execute(&requests));
+    ASSERT_EQ(NO_ERROR, fs1.Execute(&requests));
 
     FileSection fs2(NOOP_PARSER, "notexist", true, QUICK_TIMEOUT_MS);
     ASSERT_EQ(NO_ERROR, fs2.Execute(&requests));