DescriptionSpeculative hardening of fullscreen infobar code.
While this probably isn't the principle infobar crash, it does have a
few bad smells:
1) InfoBarDelegate's already have an infobar() accessor. Use that
instead of a cached tab instance.
2) Many places in infobar code that handle user events check the
context to ensure they're still valid before handling them but this
didn't.
BUG=481758
Committed: https://crrev.com/4d978263c720d2accbf1ffa35f859bee9f8e3383
Cr-Commit-Position: refs/heads/master@{#347017}
Patch Set 1 #Patch Set 2 : don't cache tab #
Created: 5 years, 3 months ago
Messages
Total messages: 7 (2 generated)
|