Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(868)

Unified Diff: content/browser/accessibility/browser_accessibility_com_win.h

Issue 2873683003: Use BrowserAccessibilityComWin::Destroy to clear delegate and owner. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | content/browser/accessibility/browser_accessibility_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/accessibility/browser_accessibility_com_win.h
diff --git a/content/browser/accessibility/browser_accessibility_com_win.h b/content/browser/accessibility/browser_accessibility_com_win.h
index ef16af35e9a667b84fdcb85de72e1af73f46c449..ffdcd4b34361b057de56bb6a95384db725ceafdd 100644
--- a/content/browser/accessibility/browser_accessibility_com_win.h
+++ b/content/browser/accessibility/browser_accessibility_com_win.h
@@ -745,6 +745,13 @@ class __declspec(uuid("562072fe-3390-43b1-9e2c-dd4118f5ac79"))
BrowserAccessibilityWin* GetOwner() const { return owner_; }
void SetOwner(BrowserAccessibilityWin* owner) { owner_ = owner; }
+ // Called when BrowserAccessibilityWin is about to go away.
+ // At this point, we detach from it.
+ void detach() {
dmazzoni 2017/05/10 06:15:50 I think this is the same as AXPlatformNode::Destro
+ owner_ = nullptr;
+ delegate_ = nullptr;
+ }
+
BrowserAccessibilityManager* owners_manager() const;
// Returns the IA2 text attributes for this object.
« no previous file with comments | « no previous file | content/browser/accessibility/browser_accessibility_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698