Bug 189157

Summary: Web Share API compatible with AppleTV and WatchOS
Product: WebKit Reporter: Olivia Barnett <obarnett>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ddkilzer, jlewis3, realdawei, ryanhaddad, thorton, tsavell, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Description Olivia Barnett 2018-08-30 10:22:57 PDT 
Web Share API compatible with AppleTV and WatchOS
Comment 1 Olivia Barnett 2018-08-30 10:23:50 PDT 
Created attachment 348503 [details]
Patch
Comment 2 Tim Horton 2018-08-30 10:26:20 PDT 
Comment on attachment 348503 [details]
Patch

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

> Source/WebKit/Shared/WebPreferencesDefaultValues.h:81
> +#define DEFAULT_WEB_SHARE_ENABLED true

Now it’s not defined to anything on those platforms. Put this in its own section down near the bottom, with just two cases (iOS and not watch and not tv -> true, otherwise false).
Comment 3 Olivia Barnett 2018-08-30 10:38:33 PDT 
Created attachment 348505 [details]
Patch
Comment 4 Tim Horton 2018-08-30 10:42:00 PDT 
Comment on attachment 348505 [details]
Patch

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

> Source/WebKit/UIProcess/ios/forms/WKShareSheet.mm:26
> +#if PLATFORM(IOS) && !PLATFORM(WATCHOS) && !PLATFORM(APPLETV)

This went too far north! Config.h and the file’s own header always come first
Comment 5 Olivia Barnett 2018-08-30 10:42:11 PDT 
Created attachment 348506 [details]
Patch
Comment 6 Olivia Barnett 2018-08-30 10:44:26 PDT 
Created attachment 348507 [details]
Patch
Comment 7 Tim Horton 2018-08-30 11:12:00 PDT 
Comment on attachment 348507 [details]
Patch

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

> Source/WebKit/ChangeLog:3
> +        Web Share API compatible with AppleTV and WatchOS

For next time (but not this time since it's already in the CQ) the title ideally should either describe the /problem/ or say "Fix the build..."
Comment 8 WebKit Commit Bot 2018-08-30 11:30:06 PDT 
Comment on attachment 348507 [details]
Patch

Clearing flags on attachment: 348507

Committed r235512: <https://trac.webkit.org/changeset/235512>
Comment 9 WebKit Commit Bot 2018-08-30 11:30:08 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 10 David Kilzer (:ddkilzer) 2018-08-31 02:44:16 PDT 
(In reply to WebKit Commit Bot from comment #8)
> Comment on attachment 348507 [details]
> Patch
> 
> Clearing flags on attachment: 348507
> 
> Committed r235512: <https://trac.webkit.org/changeset/235512>

Follow-up build fix:

Committed r235520: <https://trac.webkit.org/changeset/235520>
Comment 11 Radar WebKit Bug Importer 2018-08-31 02:44:33 PDT 
<rdar://problem/43932330>
Comment 12 David Kilzer (:ddkilzer) 2018-08-31 02:55:43 PDT 
(In reply to David Kilzer (:ddkilzer) from comment #10)
> (In reply to WebKit Commit Bot from comment #8)
> > Comment on attachment 348507 [details]
> > Patch
> > 
> > Clearing flags on attachment: 348507
> > 
> > Committed r235512: <https://trac.webkit.org/changeset/235512>
> 
> Follow-up build fix:
> 
> Committed r235520: <https://trac.webkit.org/changeset/235520>

And I think:

Committed r235530: <https://trac.webkit.org/changeset/235530>
Comment 13 David Kilzer (:ddkilzer) 2018-08-31 02:58:35 PDT 
(In reply to David Kilzer (:ddkilzer) from comment #12)
> (In reply to David Kilzer (:ddkilzer) from comment #10)
> > (In reply to WebKit Commit Bot from comment #8)
> > > Comment on attachment 348507 [details]
> > > Patch
> > > 
> > > Clearing flags on attachment: 348507
> > > 
> > > Committed r235512: <https://trac.webkit.org/changeset/235512>
> > 
> > Follow-up build fix:
> > 
> > Committed r235520: <https://trac.webkit.org/changeset/235520>
> 
> And I think:
> 
> Committed r235530: <https://trac.webkit.org/changeset/235530>

And finally(?):

Committed r235534: <https://trac.webkit.org/changeset/235534>