1666529 - Add cookie storage resource

Status: RESOLVED FIXED

(DevTools :: Storage Inspector, enhancement, P3)

Description

[Belén [:ladybenko]]

+++ This bug was initially created as a clone of Bug #1666519 +++

Add a Fission-compatible cookie storage resource

Comment 1

[Chris Peterson [:cpeterson]]

Tracking for Fission Beta milestone (M7).

Comment 2

[Belén [:ladybenko]]

Attachment: Bug 1666529 - Add cookie storage resource for fission

work in progress patch for cookies

Comment 3

[Belén [:ladybenko]]

Attachment: WIP - cookies (handle navigation)

Depends on D96841

Comment 4

[Chris Peterson [:cpeterson]]

Tracking for Fission M8 milestone (Release experiment).

Comment 5

[Belén [:ladybenko]]

Attachment: Bug 1666529 - [devtools] Part 1: Add cookie storage resource

This adds a resource for the cookie storage type, that supports inspecting iframes, etc. on Fission.

Comment 6

[Belén [:ladybenko]]

Attachment: Bug 1666529 - [devtools] Part 2: Add mochitest for cookies resource in Fission

NOTE: this depends on D108040.

Adds a mochitest to test cookies in iframes under Fission.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3be87c868f09
[devtools] Part 1: Add cookie storage resource r=ochameau,jdescottes,devtools-backward-compat-reviewers
https://hg.mozilla.org/integration/autoland/rev/24a6a96f3536
[devtools] Part 2: Add mochitest for cookies resource in Fission r=jdescottes

Comment 8

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d1006497fd63
Backed out 2 changesets for causing lint failures.

Comment 9

[Atila Butkovits]

Backed out for causing lint failures.

Backout link: https://hg.mozilla.org/integration/autoland/rev/d1006497fd63719f9c16d96d346ba7bff70060d2

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&revision=24a6a96f35366a7d37a535528d520a2bb42ad5f0&searchStr=Linting%2Copt%2CJavaScript%2Cchecks%2Csource-test-mozlint-eslint%2CES&selectedTaskRun=Xrqc8HtMTouD8lu_b96LqA.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=334569232&repo=autoland&lineNumber=392

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/81f00b369188
[devtools] Part 1: Add cookie storage resource r=ochameau,jdescottes,devtools-backward-compat-reviewers
https://hg.mozilla.org/integration/autoland/rev/b0526fa68b7b
[devtools] Part 2: Add mochitest for cookies resource in Fission r=jdescottes

Comment 11

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/81f00b369188
https://hg.mozilla.org/mozilla-central/rev/b0526fa68b7b