• Julie Jeongeun Kim's avatar
    [printing] Reuse AwPrintManager with introducing UpdateParam() · 296bf0c3
    Julie Jeongeun Kim authored
    This CL introduces AwPrintManager::UpdateParam() which updates
    |settings_|, |fd_|, and |pdf_writing_done_callback_| in order
    to reuse AwPrintManager avoiding creating it every time.
    
    It fixes a regression issue by crrev.com/c/2326857. The
    problem is that WebContentsFrameReceiverSet is created always
    along with AwPrintManager but WebContentsImpl::AddReceiverSet()
    expects that it's called once for an each interface. So, this
    CL doesn't create AwPrintManager always like other print
    managers such as PrintViewManager or HeadlessPrintManager.
    
    Bug: 1137016
    Change-Id: I11c176009ccdad643ebf495cc4b983209ba8b1c6
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463270Reviewed-by: default avatarBo <boliu@chromium.org>
    Commit-Queue: Julie Kim <jkim@igalia.com>
    Cr-Commit-Position: refs/heads/master@{#816141}
    296bf0c3
aw_pdf_exporter.cc 4.9 KB