fix: WebContentsView removal should compare directly (#44672)

fix: `WebContentsView` removal should compare directly (#44656)

* fix: WebContentsView removal should compare directly

* fixup view comparision

* chore: use erase_if

* Apply review suggestions

---------

Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
This commit is contained in:
John Kleinschmidt 2024-11-15 07:09:42 -05:00 committed by GitHub
parent 43885b94d0
commit 8f88aae33c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 43 additions and 26 deletions

View file

@ -17,6 +17,8 @@
namespace electron::api {
using ChildPair = std::pair<raw_ptr<views::View>, v8::Global<v8::Object>>;
class View : public gin_helper::EventEmitter<View>,
private views::ViewObserver {
public:
@ -62,7 +64,7 @@ class View : public gin_helper::EventEmitter<View>,
private:
void ReorderChildView(gin::Handle<View> child, size_t index);
std::vector<v8::Global<v8::Object>> child_views_;
std::vector<ChildPair> child_views_;
bool delete_view_ = true;
raw_ptr<views::View> view_ = nullptr;