refactor

Author: Donaldcwl

lib/image-compression.js

@@ -64,6 +64,8 @@ export default async function compress (file, options) {
 
   const origExceedMaxSize = tempFile.size > maxSizeByte
   const sizeBecomeLarger = tempFile.size > file.size
+  // console.log('original file size', file.size)
+  // console.log('current file size', tempFile.size)
 
   // check if we need to compress or resize
   if (!origExceedMaxSize && !sizeBecomeLarger) {
@@ -81,6 +83,8 @@ export default async function compress (file, options) {
   while (remainingTrials-- && (currentSize > maxSizeByte || currentSize > sourceSize)) {
     const newWidth = origExceedMaxSize ? canvas.width * 0.95 : canvas.width
     const newHeight = origExceedMaxSize ? canvas.height * 0.95 : canvas.height;
+    // console.log('current width', newWidth);
+    // console.log('current height', newHeight);
     [newCanvas, ctx] = getNewCanvasAndCtx(newWidth, newHeight)
 
     ctx.drawImage(canvas, 0, 0, newWidth, newHeight)
@@ -98,8 +102,6 @@ export default async function compress (file, options) {
     setProgress(Math.min(99, Math.floor((renderedSize - currentSize) / (renderedSize - maxSizeByte) * 100)))
   }
 
-  // garbage clean canvas for safari
-  // ref: https://bugs.webkit.org/show_bug.cgi?id=195325
   cleanupCanvasMemory(canvas)
   cleanupCanvasMemory(newCanvas)
   cleanupCanvasMemory(maxWidthOrHeightFixedCanvas)

lib/utils.js

@@ -304,6 +304,8 @@ export function getNewCanvasAndCtx (width, height) {
  * @returns null
  */
 export function cleanupCanvasMemory (canvas) {
+  // garbage clean canvas for safari
+  // ref: https://bugs.webkit.org/show_bug.cgi?id=195325
   canvas.width = 0
   canvas.height = 0
 }