commit a50fa63b3382204408c44c0d3f6e182835c46529
author Amin Hassani <ahassani@chromium.org> Tue Oct 15 20:49:51 2019 -0700
committer Commit Bot <commit-bot@chromium.org> Wed Oct 16 21:16:35 2019 +0000
tree cd1c2fdc02bc7b0e19b9954703e49173bacdfc4d
parent 261c0df4cc3944b469b7dd79ffcad67a98f3e5c5

autoupdate: Use correct nebraska.Request attributes

event_type and event_results are properties of RequestApp not the
Request itself. This patch fixes it.

BUG=chromium:1014807
TEST=devserver_integration_test.py

Change-Id: Ic0847ea198a4b0058c0e8cd46bd1c5e2f8d3ccc6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/dev-util/+/1864310
Tested-by: Amin Hassani <ahassani@chromium.org>
Auto-Submit: Amin Hassani <ahassani@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>

autoupdate.py

diff --git a/autoupdate.py b/autoupdate.py
index 209ef4e..bbc4b6c 100644
--- a/autoupdate.py
+++ b/autoupdate.py
@@ -319,9 +319,10 @@
     self._LogRequest(request)
 
     if request.request_type == nebraska.Request.RequestType.EVENT:
-      if ((request.event_type ==
-           nebraska.Request.EVENT_TYPE_UPDATE_DOWNLOAD_STARTED) and
-          request.event_result == nebraska.Request.EVENT_RESULT_SUCCESS):
+      if (request.app_requests[0].event_type ==
+          nebraska.Request.EVENT_TYPE_UPDATE_DOWNLOAD_STARTED and
+          request.app_requests[0].event_result ==
+          nebraska.Request.EVENT_RESULT_SUCCESS):
         with self._update_count_lock:
           if self.max_updates == 0:
             _Log('Received too many download_started notifications. This '