Closed Bug 1549708 Opened 4 years ago Closed 4 years ago

Implement Page.reload's ignoreCache argument


(Remote Protocol :: CDP, enhancement, P1)



(firefox72 fixed)

Tracking Status
firefox72 --- fixed


(Reporter: ochameau, Assigned: ato)


(Blocks 1 open bug)


(Whiteboard: [puppeteer-alpha])


(2 files)

In bug 1544445 I introduced minimal support for Page.reload.
But Page.reload supports two option arguments, including ignoreCache:
Reloads given page optionally ignoring the cache.

  • ignoreCache
    boolean If true, browser cache is ignored (as if the user pressed Shift+refresh).
Assignee: nobody → ato
Priority: P3 → P1
Depends on: 1563533
Priority: P1 → P2
Depends on: 1589625

Why is this implementation blocked on my patch for bug 1589625?

Flags: needinfo?(ato)

Because it cleans up tabs in a more generic way than could be implemented here.

Flags: needinfo?(ato)

Sure, but what's unclear is why the dependency has been added a day after it actually was been landed. It's basically no blocker given that the feature is already available on central. I will remove the dependency.

No longer depends on: 1589625

Note that Page.reload in Puppeteer is called without any argument. So this is not blocking the alpha release.

Priority: P2 → P1
Whiteboard: [puppeteer-alpha]

Exposes a new nsIDocShell API, isForceReloading, to determine if
the loaded document was force-reloaded or not.

It relies on the underlying behaviour of nsDocShell::IsForceReloading(),
which again relies on nsDocShell::IsForceReloadType(mLoadType).

The getter is used in the remote agent to test that
Page.reload({ignoreCache: true}) works as intended.

Why is this blocking the alpha release? I cannot see that Puppeteer is sending parameters at all. See also my comment 4.

Also Gutenberg doesn't make use of it.

Flags: needinfo?(ato)

I had this patch lying around and I want it to be part of this milestone
so it doesn’t degrade beyond repair.

Flags: needinfo?(ato)
Pushed by
docshell: add nsIDocShell.isForceReloading; r=bzbarsky
remote: implement Page.reload's ignoreCache argument; r=remote-protocol-reviewers,whimboo
Closed: 4 years ago
Resolution: --- → FIXED
Component: CDP: Page → CDP
You need to log in before you can comment on or make changes to this bug.