From 4c86d66b5642726325636ecfe7677f0139ce06c1 Mon Sep 17 00:00:00 2001
From: Bob Zeleznik <zzzman@gmail.com>
Date: Mon, 18 May 2020 22:42:05 -0400
Subject: added a long delay and an attempt to click on the skip ads button to
 make youtube snapshots.

---
 src/server/ApiManagers/UploadManager.ts | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/server/ApiManagers/UploadManager.ts b/src/server/ApiManagers/UploadManager.ts
index 60c52bcfc..ad165e3fe 100644
--- a/src/server/ApiManagers/UploadManager.ts
+++ b/src/server/ApiManagers/UploadManager.ts
@@ -277,7 +277,9 @@ export default class UploadManager extends ApiManager {
     }
 
 }
-
+function delay(ms: number) {
+    return new Promise(resolve => setTimeout(resolve, ms));
+}
 /**
  * On success, returns a buffer containing the bytes of a screenshot
  * of the video specified by @param url at timecode @param t.
@@ -289,9 +291,15 @@ async function captureYoutubeScreenshot(targetUrl: string): Promise<Opt<Buffer>>
     const page = await browser.newPage();
     await page.setViewport({ width: 1920, height: 1080 });
 
-    await page.goto(targetUrl, { waitUntil: 'networkidle2' as any });
+    await page.goto(targetUrl, { waitUntil: 'domcontentloaded' as any });
 
     const videoPlayer = await page.$('.html5-video-player');
+    videoPlayer && await page.focus("video");
+    await delay(7000);
+    const ad = await page.$('.ytp-ad-skip-button-text');
+    await ad?.click();
+    await videoPlayer?.click();
+    await delay(1000);
     // hide youtube player controls.
     await page.evaluate(() =>
         (document.querySelector('.ytp-chrome-bottom') as any).style.display = 'none');
-- 
cgit v1.2.3-70-g09d2


From 039892dd122f7958662e1cfe27b0e3296c2780b9 Mon Sep 17 00:00:00 2001
From: Sam Wilkins <samwilkins333@gmail.com>
Date: Mon, 18 May 2020 22:25:31 -0700
Subject: commenting cleanup

---
 src/client/views/nodes/VideoBox.tsx     | 2 +-
 src/server/ApiManagers/UploadManager.ts | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/client/views/nodes/VideoBox.tsx b/src/client/views/nodes/VideoBox.tsx
index a3abaedfa..6b1e6ae18 100644
--- a/src/client/views/nodes/VideoBox.tsx
+++ b/src/client/views/nodes/VideoBox.tsx
@@ -111,7 +111,7 @@ export class VideoBox extends ViewBoxAnnotatableComponent<FieldViewProps, VideoD
             this._videoRef && ctx.drawImage(this._videoRef, 0, 0, canvas.width, canvas.height);
         }
 
-        if (!this._videoRef) { // can't find a way to take snapshots of videos
+        if (!this._videoRef) {
             const b = Docs.Create.LabelDocument({
                 x: (this.layoutDoc.x || 0) + width, y: (this.layoutDoc.y || 1),
                 _width: 150, _height: 50, title: (this.layoutDoc.currentTimecode || 0).toString(),
diff --git a/src/server/ApiManagers/UploadManager.ts b/src/server/ApiManagers/UploadManager.ts
index 1ab81803f..10d7e3a34 100644
--- a/src/server/ApiManagers/UploadManager.ts
+++ b/src/server/ApiManagers/UploadManager.ts
@@ -280,7 +280,7 @@ export default class UploadManager extends ApiManager {
 
 /**
  * On success, returns a buffer containing the bytes of a screenshot
- * of the video specified by @param url at timecode @param t.
+ * of the video (optionally, at a timecode) specified by @param targetUrl.
  * 
  * On failure, returns undefined.
  */
-- 
cgit v1.2.3-70-g09d2