ProcessMemoryDumpTest.CountResidentBytesInSharedMemory is failing on iOS 9 devices |
||||||
Issue descriptionThe test has been introduced in https://chromium-review.googlesource.com/c/575802/. The test is consistently failing on iOS 9 only devices. Log: ProcessMemoryDumpTest.CountResidentBytesInSharedMemory: ../../base/trace_event/process_memory_dump_unittest.cc:465: Failure Expected: res3.value() Which is: 8126464 To be equal to: kTouchedMemorySize Which is: 7340032
,
Jul 25 2017
Isn't this a problem on iOS 10? As a temporary fix, disabling the test when base::ios::IsRunningOnIOS10OrLater() is false sounds good. erikchen@, wdyt?
,
Jul 25 2017
No, it is a problem only on iOS 9, and only on devices (simulators run fine).
,
Jul 25 2017
gambard@, thank you!
,
Jul 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9dd9c24af2feb7ef0222326ba6b3f794711b4b9b commit 9dd9c24af2feb7ef0222326ba6b3f794711b4b9b Author: gambard <gambard@chromium.org> Date: Wed Jul 26 22:02:07 2017 Disable ProcessMemoryDumpTest.CountResidentBytesInSharedMemory for iOS 9 TBR=primiano Bug: 748410 Change-Id: Ie1f5e37772e918b9986faf1f3feb4e1bb2eebc97 Reviewed-on: https://chromium-review.googlesource.com/584843 Commit-Queue: Gregory Chatzinoff <gchatz@chromium.org> Reviewed-by: Gregory Chatzinoff <gchatz@chromium.org> Reviewed-by: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#489770} [modify] https://crrev.com/9dd9c24af2feb7ef0222326ba6b3f794711b4b9b/base/trace_event/process_memory_dump_unittest.cc
,
Aug 7 2017
,
Jan 24 2018
,
Jan 24 2018
,
Mar 20 2018
Can we close this out since we no longer support ios9?
,
Mar 20 2018
Sure. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by hajimehoshi@chromium.org
, Jul 25 2017Owner: hajimehoshi@chromium.org
Status: Started (was: Untriaged)
Summary: ProcessMemoryDumpTest.CountResidentBytesInSharedMemory is failing on iOS 9 devices (was: ProcessMemoryDumpTest.CountResidentBytes is failing on iOS 9 devices)