[webkit-changes] [WebKit/WebKit] d8b27a: [UnifiedPDF] pdfLog() is a misnomer since it logs ...

Abrar Rahman Protyasha noreply at github.com
Thu Feb 1 10:08:19 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: d8b27a8ef37a2055cc7c96442de22f9ff44ef664
      https://github.com/WebKit/WebKit/commit/d8b27a8ef37a2055cc7c96442de22f9ff44ef664
  Author: Abrar Rahman Protyasha <a_protyasha at apple.com>
  Date:   2024-02-01 (Thu, 01 Feb 2024)

  Changed paths:
    M Source/WebKit/WebProcess/Plugins/PDF/PDFIncrementalLoader.h
    M Source/WebKit/WebProcess/Plugins/PDF/PDFIncrementalLoader.mm
    M Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.h
    M Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.mm

  Log Message:
  -----------
  [UnifiedPDF] pdfLog() is a misnomer since it logs for the incremental loader
https://bugs.webkit.org/show_bug.cgi?id=268493
rdar://122038661

Reviewed by Simon Fraser.

pdfLog() can be misleading for a caller since it logs exclusively for
the incremental loader. This patch addresses this ambiguity by renaming
it to incrementalLoaderLog() instead. This method also makes the
verboseLog method a static method in the PDFPluginBase translation unit,
since it only has one caller inside PDFPluginBase::incrementalLoaderLog.

* Source/WebKit/WebProcess/Plugins/PDF/PDFIncrementalLoader.h:
* Source/WebKit/WebProcess/Plugins/PDF/PDFIncrementalLoader.mm:
(WebKit::PDFIncrementalLoader::receivedNonLinearizedPDFSentinel):
(WebKit::PDFIncrementalLoader::incrementalPDFStreamDidFinishLoading):
(WebKit::PDFIncrementalLoader::getResourceBytesAtPosition):
(WebKit::PDFIncrementalLoader::streamLoaderDidStart):
(WebKit::PDFIncrementalLoader::forgetStreamLoader):
(WebKit::PDFIncrementalLoader::requestCompleteIfPossible):
(WebKit::PDFIncrementalLoader::requestDidCompleteWithBytes):
(WebKit::PDFIncrementalLoader::requestDidCompleteWithAccumulatedData):
(WebKit::PDFIncrementalLoader::dataProviderGetBytesAtPosition):
(WebKit::PDFIncrementalLoader::dataProviderGetByteRanges):
(WebKit::PDFIncrementalLoader::threadEntry):
(WebKit::PDFIncrementalLoader::incrementalLoaderLog):
(WebKit::PDFIncrementalLoader::pdfLog): Deleted.
* Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.h:
* Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.mm:
(WebKit::PDFPluginBase::streamDidReceiveData):
(WebKit::PDFPluginBase::adoptBackgroundThreadDocument):
(WebKit::PDFPluginBase::receivedNonLinearizedPDFSentinel):
(WebKit::verboseLog):
(WebKit::PDFPluginBase::incrementalLoaderLog):
(WebKit::PDFPluginBase::pdfLog): Deleted.
(WebKit::PDFPluginBase::verboseLog): Deleted.

Canonical link: https://commits.webkit.org/273920@main




More information about the webkit-changes mailing list