Add new API function: feedbackPrivate.readLogSource

chrome/test/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/chrome_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//chrome/common/features.gni")
@@ skipping 4106 matching lines @@
       # target.
       "//tools/json_schema_compiler/test:unit_tests",
     ]
     if (enable_service_discovery) {
       sources += [ "../browser/extensions/api/mdns/mdns_api_unittest.cc" ]
     }
     if (is_win) {
       sources +=
           [ "../browser/extensions/external_registry_loader_win_unittest.cc" ]
     }
-    if (!is_chromeos) {
+    if (is_chromeos) {
+      sources += [ "../browser/extensions/api/feedback_private/feedback_private_api_chromeos_unittest.cc" ]

[tbarzic, 2017/06/07 17:58:39]

I think you can move this to line 3953 (under if (enable_extensions) - chromeos
in the name should restrict the file to chromeos automatically)

[Simon Que, 2017/06/07 18:25:59]

Done.

+    } else {
       sources += [
         "../browser/extensions/api/messaging/native_message_process_host_unittest.cc",
         "../browser/extensions/api/messaging/native_messaging_host_manifest_unittest.cc",
         "../browser/extensions/api/messaging/native_messaging_policy_handler_unittest.cc",
       ]
     }
   }
   if (use_ash) {
     sources += [
       "../browser/media/webrtc/desktop_media_list_ash_unittest.cc",
@@ skipping 1003 matching lines @@
 }
 
 if (is_win) {
   loadable_module("conflicts_dll") {
     testonly = true
     sources = [
       "conflicts/conflicts_dll.cc",
     ]
   }
 }