New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ignore Meizu RecommendActivity leaked in Android Instrumentation #596
Conversation
@@ -442,6 +443,13 @@ | |||
excluded.instanceField("android.view.Choreographer$FrameDisplayEventReceiver", | |||
"mMessageQueue").alwaysExclude(); | |||
} | |||
}, | |||
|
|||
INSTRUMENTATION_RECOMMEND_ACTIVITY(MEIZU.equals(MANUFACTURER)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you move this closer to the top, next to the other manufacturer ignored leaks? Also please hide a version range check (SDK_INT)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean a version check should be added also?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes.
Sorry, I force updated commit after rebase with lastest remote code. it seems this pr closed. |
I have pushed new commits as you requested. Please check. |
@@ -305,6 +306,14 @@ | |||
|
|||
// ######## Manufacturer specific Excluded refs ######## | |||
|
|||
INSTRUMENTATION_RECOMMEND_ACTIVITY(MEIZU.equals(MANUFACTURER) && SDK_INT >= LOLLIPOP) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should not be SDK_INT >= LOLLIPOP
. Only target the versions you know for sure are impacted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. I will change this
Thx! Can you squash all commits into one? |
…ndroid 5.0 and 5.1
squash done. please check |
When running UI automation tests using Instrumentation on Meizu devices, some page would report a memory leak in
I digged and found
RecommendActivity
is a class inframework.jar
in Meizu FlymeOS (smali source). I do not know what it does and when it would be invoked. Maybe it relates to permission alerts prompted by FlymeOSBut I found many people got this memory leak.(google result) I also reported an issue #590 . and found a similar issue earlier #506