Bug 227742 – Set Caption preferences in the UI process

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Per Arne Vollan]

Due to sandbox restrictions in the WebContent process, Caption preferences should be set in the UI process.

[Per Arne Vollan]

<rdar://79040526>

[Per Arne Vollan]

Created attachment 433026 [details]
Patch

[Per Arne Vollan]

Created attachment 433029 [details]
Patch

[Eric Carlson]

Comment on attachment 433029 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=433029&action=review

> Source/WebCore/WebCore.xcodeproj/project.pbxproj:5527
> -		FD3160BF12B0272A00C1A359 /* AudioFileReaderMac.h in Headers */ = {isa = PBXBuildFile; fileRef = FD3160B912B0272A00C1A359 /* AudioFileReaderMac.h */; };
> +		FD3160BF12B0272A00C1A359 /* (null) in Headers */ = {isa = PBXBuildFile; };

Nit: this seems unintentional

> Source/WebCore/WebCore.xcodeproj/project.pbxproj:31619
> +				FD3160BF12B0272A00C1A359 /* (null) in Headers */,

Ditto

[Per Arne Vollan]

(In reply to Eric Carlson from comment #4)
> Comment on attachment 433029 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=433029&action=review
> 
> > Source/WebCore/WebCore.xcodeproj/project.pbxproj:5527
> > -		FD3160BF12B0272A00C1A359 /* AudioFileReaderMac.h in Headers */ = {isa = PBXBuildFile; fileRef = FD3160B912B0272A00C1A359 /* AudioFileReaderMac.h */; };
> > +		FD3160BF12B0272A00C1A359 /* (null) in Headers */ = {isa = PBXBuildFile; };
> 
> Nit: this seems unintentional
> 
> > Source/WebCore/WebCore.xcodeproj/project.pbxproj:31619
> > +				FD3160BF12B0272A00C1A359 /* (null) in Headers */,
> 
> Ditto

Will fix.

Thanks for reviewing!

[Per Arne Vollan]

Created attachment 433123 [details]
Patch

[Per Arne Vollan]

(In reply to Per Arne Vollan from comment #5)
> (In reply to Eric Carlson from comment #4)
> > Comment on attachment 433029 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=433029&action=review
> > 
> > > Source/WebCore/WebCore.xcodeproj/project.pbxproj:5527
> > > -		FD3160BF12B0272A00C1A359 /* AudioFileReaderMac.h in Headers */ = {isa = PBXBuildFile; fileRef = FD3160B912B0272A00C1A359 /* AudioFileReaderMac.h */; };
> > > +		FD3160BF12B0272A00C1A359 /* (null) in Headers */ = {isa = PBXBuildFile; };
> > 
> > Nit: this seems unintentional
> > 
> > > Source/WebCore/WebCore.xcodeproj/project.pbxproj:31619
> > > +				FD3160BF12B0272A00C1A359 /* (null) in Headers */,
> > 
> > Ditto
> 
> Will fix.
> 

These files seem to have been removed earlier, and Xcode removed the references in the project file.

> Thanks for reviewing!

[EWS]

Committed r279720 (239508@main): <https://commits.webkit.org/239508@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 433123 [details].