Skip to content
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

Bug 1885351 - Add nsCOMPtr<T>:: and nsMaybeWeakPtr<T>:: to the irrele… #6557

Merged
merged 1 commit into from Mar 18, 2024

Conversation

amccreight
Copy link
Contributor

…vant signature list.

@amccreight amccreight requested a review from a team as a code owner March 14, 2024 14:55
Copy link
Collaborator

@willkg willkg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

app@socorro:/app$ ./mybin/test_signature_change.sh 'nsCOMPtr<T>::' 'nsMaybeWeakPtr<T>::'
Working on 'nsCOMPtr<T>::' ...
Crash id: 87085d63-5d53-45f0-a220-9005f0240311
Original: nsCOMPtr<T>::assign_from_query_referent | nsCOMPtr<T>::nsCOMPtr | mozilla::dom::XULBroadcastManager::AttributeChanged
New:      mozilla::dom::XULBroadcastManager::AttributeChanged
Same?:    False

Crash id: dec11def-9d58-43c0-8259-796720240311
Original: shutdownhang | nsCOMPtr<T>::nsCOMPtr | nsMaybeWeakPtr<T>::GetValue
New:      shutdownhang | nsObserverList::ReverseCloneObserverArray
Same?:    False

Working on 'nsMaybeWeakPtr<T>::' ...
Crash id: c6275273-395f-41d7-8492-f8c2b0240311
Original: shutdownhang | xul.dll | nsCOMPtr<T>::assign_from_query_referent | nsCOMPtr<T>::operator= | nsMaybeWeakPtr<T>::GetValue
New:      shutdownhang | xul.dll | nsObserverList::ReverseCloneObserverArray
Same?:    False

Crash id: dec11def-9d58-43c0-8259-796720240311
Original: shutdownhang | nsCOMPtr<T>::nsCOMPtr | nsMaybeWeakPtr<T>::GetValue
New:      shutdownhang | nsObserverList::ReverseCloneObserverArray
Same?:    False

@willkg willkg merged commit f95104f into mozilla-services:main Mar 18, 2024
1 check passed
@willkg
Copy link
Collaborator

willkg commented Mar 18, 2024

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants