Closed Bug 1636384 Opened 5 years ago Closed 2 years ago

Runtime.disable has to send Runtime.executionContextDestroyed events for all registered contexts

Categories

(Remote Protocol :: CDP, enhancement, P2)

enhancement

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: whimboo, Assigned: canadahonk)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Whiteboard: [puppeteer-beta2-mvp])

Attachments

(1 obsolete file)

Priority: P3 → P2
Whiteboard: [puppeteer-beta-reserve] → [puppeteer-beta2-mvp]
Component: CDP: Runtime → CDP

On Runtime.disable, Runtime.executionContextDestroyed events should be
sent for all registered contexts. Also adds new test to check this
(with a todo marker as the test needs to be updated when another bug is
fixed in the future.)

Assignee: nobody → oj
Status: NEW → ASSIGNED

The review process is currently blocked on https://bugs.chromium.org/p/chromium/issues/detail?id=1407315.

(In reply to Henrik Skupin [:whimboo][⌚️UTC+1] from comment #2)

The review process is currently blocked on https://bugs.chromium.org/p/chromium/issues/detail?id=1407315.

As the discussion on that issue revealed this it not wanted. As such we can close this bug as invalid.

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
Attachment #9311863 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: