Bug 206824 - Blob media loading does not work with GPUProcess
Summary: Blob media loading does not work with GPUProcess
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
Depends on: 206889
Blocks:
  Show dependency treegraph
 
Reported: 2020-01-27 07:18 PST by youenn fablet
Modified: 2020-01-29 02:35 PST (History)
5 users (show)

See Also:


Attachments
Patch (21.00 KB, patch)
2020-01-28 04:48 PST, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for relanding (20.99 KB, patch)
2020-01-29 01:52 PST, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2020-01-27 07:18:48 PST
Blob media loading does not work with GPUProcess
Comment 1 Radar WebKit Bug Importer 2020-01-27 07:19:12 PST
<rdar://problem/58917596>
Comment 2 youenn fablet 2020-01-28 04:48:34 PST
Created attachment 388981 [details]
Patch
Comment 3 WebKit Commit Bot 2020-01-28 06:54:34 PST
Comment on attachment 388981 [details]
Patch

Clearing flags on attachment: 388981

Committed r255237: <https://trac.webkit.org/changeset/255237>
Comment 4 WebKit Commit Bot 2020-01-28 06:54:35 PST
All reviewed patches have been landed.  Closing bug.
Comment 5 WebKit Commit Bot 2020-01-28 09:57:39 PST
Re-opened since this is blocked by bug 206889
Comment 6 youenn fablet 2020-01-28 09:58:29 PST
Breaking win build apparently:
https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Build%29/builds/10234

C:\cygwin\worker\win10-release\build\Source\WebCore\platform\graphics\avfoundation\cf\InbandTextTrackPrivateLegacyAVCF.cpp(47,32): error C2664: 'WebCore::InbandTextTrackPrivateAVF::InbandTextTrackPrivateAVF(const WebCore::InbandTextTrackPrivateAVF &)': cannot convert argument 1 from 'WebCore::MediaPlayerPrivateAVFoundationCF *' to 'const WebCore::InbandTextTrackPrivateAVF &' [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\worker\win10-release\build\Source\WebCore\platform\graphics\avfoundation\cf\InbandTextTrackPrivateLegacyAVCF.cpp(49,1): message : Reason: cannot convert from 'WebCore::MediaPlayerPrivateAVFoundationCF *' to 'const WebCore::InbandTextTrackPrivateAVF' [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\worker\win10-release\build\Source\WebCore\platform\graphics\avfoundation\cf\InbandTextTrackPrivateLegacyAVCF.cpp(49,1): message : No constructor could take the source type, or constructor overload resolution was ambiguous [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
  CookieStorageCFNet.cpp
  CredentialStorageCFNet.cpp
  DNSResolveQueueCFNet.cpp
C:\cygwin\worker\win10-release\build\Source\WebCore\platform\graphics\avfoundation\InbandTextTrackPrivateAVF.h(113,1): message : see declaration of 'WebCore::InbandTextTrackPrivateAVF::InbandTextTrackPrivateAVF' (compiling source file C:\cygwin\worker\win10-release\build\Source\WebCore\platform\graphics\avfoundation\cf\InbandTextTrackPrivateLegacyAVCF.cpp) [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\worker\win10-release\build\Source\WebCore\page\CaptionUserPreferencesMediaAF.cpp(131,9): error C2065: 'MTEnableCaption2015BehaviorPtrType': undeclared identifier [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\worker\win10-release\build\Source\WebCore\page\CaptionUserPreferencesMediaAF.cpp(131,44): error C2146: syntax error: missing ';' before identifier 'function' [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\worker\win10-release\build\Source\WebCore\page\CaptionUserPreferencesMediaAF.cpp(131,44): error C2065: 'function': undeclared identifier [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\worker\win10-release\build\Source\WebCore\page\CaptionUserPreferencesMediaAF.cpp(131,55): error C3861: 'MTEnableCaption2015BehaviorPtr': identifier not found [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
  FormDataStreamCFNet.cpp
C:\cygwin\worker\win10-release\build\Source\WebCore\page\CaptionUserPreferencesMediaAF.cpp(132,14): error C2065: 'function': undeclared identifier [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
  LoaderRunLoopCF.cpp
  NetworkStorageSessionCFNet.cpp
  NetworkStorageSessionCFNetWin.cpp
  ProtectionSpaceCFNet.cpp
  ProxyServerCFNet.cpp
  ResourceErrorCF.cpp
  ResourceHandleCFNet.cpp
C:\cygwin\worker\win10-release\build\Source\WebCore\page\CaptionUserPreferencesMediaAF.cpp(132,27): error C3861: 'function': identifier not found [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\worker\win10-release\build\Source\WebCore\platform\graphics\avfoundation\cf\MediaPlayerPrivateAVFoundationCF.cpp(1531,48): error C2065: 'AVCFURLAssetInheritURIQueryComponentFromReferencingURIKey': undeclared identifier [C:\cygwin\worker\win10-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
  ResourceHandleCFURLConnectionDelegate.cpp
Comment 7 youenn fablet 2020-01-29 01:46:20 PST
EWS win bot is fine and the files I modified in this patch are not compiled by Windows so this seems weird it causes a build issue.
Comment 8 youenn fablet 2020-01-29 01:50:10 PST
Bot was still failing to compile after the rollout for a few versions and then went green with https://trac.webkit.org/changeset/255258/webkit which is just a test expectation change.

https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Build%29/builds/10237 for instance is failing with the same build errors.

Let's try relanding it and see what bots think.
Comment 9 youenn fablet 2020-01-29 01:52:01 PST
Created attachment 389114 [details]
Patch for relanding
Comment 10 WebKit Commit Bot 2020-01-29 02:35:33 PST
Comment on attachment 389114 [details]
Patch for relanding

Clearing flags on attachment: 389114

Committed r255347: <https://trac.webkit.org/changeset/255347>
Comment 11 WebKit Commit Bot 2020-01-29 02:35:34 PST
All reviewed patches have been landed.  Closing bug.