This is a common-enough need that the Emacs developers have you covered – you don’t need advice for this. From the release notes for Emacs 29:
https://www.gnu.org/savannah-checkouts/gnu/emacs/news/NEWS.29.1
‘scroll-other-window’ and ‘scroll-other-window-down’ now respect remapping.
These commands (bound to ‘C-M-v’ and ‘C-M-V’) used to scroll the other
windows without looking at customizations in that other window. These
functions now check whether they have been rebound in the buffer shown
in that other window, and then call the remapped function instead.
So you don’t need advice for this, you can just do
(keymap-set pdf-view-mode-map “<remap> <scroll-up-command>” pdf-view-scroll-up-or-next-page)
(keymap-set pdf-view-mode-map “<remap> <scroll-down-command>” pdf-view-scroll-down-or-previous-page)
I covered this in a write-up on window management a while ago: https://karthinks.com/software/emacs-window-management-almanac/#scroll-other-window--built-in
– karthink 2026-06-15 23:21 UTC