chore: might_have_observers has been removed

MarshallOfSound · MarshallOfSound · commit e900271beaaf · 2021-03-04T16:25:33.000-08:00
Refs: https://chromium-review.googlesource.com/c/chromium/src/+/2667839
diff --git a/shell/browser/api/electron_api_web_contents.h b/shell/browser/api/electron_api_web_contents.h
@@ -338,7 +338,7 @@ class WebContents : public gin::Wrappable<WebContents>,
   }
   void RemoveObserver(ExtendedWebContentsObserver* obs) {
     // Trying to remove from an empty collection leads to an access violation
-    if (observers_.might_have_observers())
+    if (!observers_.empty())
       observers_.RemoveObserver(obs);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -338,7 +338,7 @@ class WebContents : public gin::Wrappable<WebContents>,`
`338`	`338`	`}`
`339`	`339`	`void RemoveObserver(ExtendedWebContentsObserver* obs) {`
`340`	`340`	`// Trying to remove from an empty collection leads to an access violation`
`341`		`- if (observers_.might_have_observers())`
	`341`	`+ if (!observers_.empty())`
`342`	`342`	`observers_.RemoveObserver(obs);`
`343`	`343`	`}`
`344`	`344`