Bug 186844 - [WPE] Add page for webkit.org
Summary: [WPE] Add page for webkit.org
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Website (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Manuel Rego Casasnovas
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-06-20 07:40 PDT by Manuel Rego Casasnovas
Modified: 2018-06-20 13:25 PDT (History)
6 users (show)

See Also:


Attachments
Patch (4.79 KB, patch)
2018-06-20 07:41 PDT, Manuel Rego Casasnovas
no flags Details | Formatted Diff | Diff
Patch (4.79 KB, patch)
2018-06-20 08:23 PDT, Manuel Rego Casasnovas
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Manuel Rego Casasnovas 2018-06-20 07:40:38 PDT
As we've been discussing with Jonathan Davis we're going to propose a WPE port page for webkit.org.
Comment 1 Manuel Rego Casasnovas 2018-06-20 07:41:58 PDT
Created attachment 343156 [details]
Patch
Comment 2 Michael Catanzaro 2018-06-20 07:53:06 PDT
Comment on attachment 343156 [details]
Patch

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

> Websites/webkit.org/wpe.md:6
> +accelerated-content rendering, and simplicity of deployment in mind,

accelerated content, no hyphen

> Websites/webkit.org/wpe.md:11
> +In times when embedded devices become more powerful and ad-hoc
> +to endless applications in a diversity of industries,

There is a missing verb somewhere. This needs rephrased.

> Websites/webkit.org/wpe.md:22
> +* **To provide a no-frills, straight to the point, web-runtime

web  runtime, no hyphen

> Websites/webkit.org/wpe.md:27
> +  and constrains of embedded devices in consideration,

constraints

> Websites/webkit.org/wpe.md:42
> +  are implement in WPE with our goals in mind.

implemented

> Websites/webkit.org/wpe.md:44
> +* **To use hardware-acceleration wherever its advantageous:

hardware acceleration, no hyphen

> Websites/webkit.org/wpe.md:52
> +  through a backend-architecture.**

Either "backend-based architecture" or else "backend architecture" without the hyphen

> Websites/webkit.org/wpe.md:54
> +  WPE has been designed with a backend architecture,

Um, now here the hyphen has disappeared. :P Be consistent!

> Websites/webkit.org/wpe.md:82
> +The WPE team is planning on making periodic stable releases and
> +these will be announced as soon as they are available.

These are nowadays available on https://wpewebkit.org/.
Comment 3 Philippe Normand 2018-06-20 07:58:37 PDT
Comment on attachment 343156 [details]
Patch

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

> Websites/webkit.org/wpe.md:69
> +* EME (V1 and V3, Clearkey, PlayReady, Widevine) supported

Please consider removing the mention to PlayReady and Widevine. Something like "other third-party DRM frameworks" would better fit here, IMHO...
Comment 4 Michael Catanzaro 2018-06-20 08:03:43 PDT
Comment on attachment 343156 [details]
Patch

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

>> Websites/webkit.org/wpe.md:11
>> +to endless applications in a diversity of industries,
> 
> There is a missing verb somewhere. This needs rephrased.

How about: "In times where embedded devices power endless ad-hoc applications in a diverse range of industries and the Web Continues to establish itself..."
Comment 5 Enrique Ocaña 2018-06-20 08:17:04 PDT
Comment on attachment 343156 [details]
Patch

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

> Websites/webkit.org/wpe.md:67
> +* MSE (MP4, WebM, VP9 and VP9, YouTube) supported and

MSE (MP4, WebM, VP9, Opus) supported and under constant development, optimized for YouTube and YouTube/tv.
Comment 6 Manuel Rego Casasnovas 2018-06-20 08:22:25 PDT
Comment on attachment 343156 [details]
Patch

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

Thank you all for the fast reviews. Uploading new patch with changes applied.

>> Websites/webkit.org/wpe.md:6
>> +accelerated-content rendering, and simplicity of deployment in mind,
> 
> accelerated content, no hyphen

ACK.

>>> Websites/webkit.org/wpe.md:11
>>> +to endless applications in a diversity of industries,
>> 
>> There is a missing verb somewhere. This needs rephrased.
> 
> How about: "In times where embedded devices power endless ad-hoc applications in a diverse range of industries and the Web Continues to establish itself..."

Thanks, that one sounds good.

>> Websites/webkit.org/wpe.md:22
>> +* **To provide a no-frills, straight to the point, web-runtime
> 
> web  runtime, no hyphen

ACK.

>> Websites/webkit.org/wpe.md:27
>> +  and constrains of embedded devices in consideration,
> 
> constraints

Fixed.

>> Websites/webkit.org/wpe.md:52
>> +  through a backend-architecture.**
> 
> Either "backend-based architecture" or else "backend architecture" without the hyphen

I think "backend architecture" is good enough.

>> Websites/webkit.org/wpe.md:54
>> +  WPE has been designed with a backend architecture,
> 
> Um, now here the hyphen has disappeared. :P Be consistent!

Yep. :-)

>> Websites/webkit.org/wpe.md:67
>> +* MSE (MP4, WebM, VP9 and VP9, YouTube) supported and
> 
> MSE (MP4, WebM, VP9, Opus) supported and under constant development, optimized for YouTube and YouTube/tv.

Updated.

>> Websites/webkit.org/wpe.md:69
>> +* EME (V1 and V3, Clearkey, PlayReady, Widevine) supported
> 
> Please consider removing the mention to PlayReady and Widevine. Something like "other third-party DRM frameworks" would better fit here, IMHO...

Updated too.

>> Websites/webkit.org/wpe.md:82
>> +these will be announced as soon as they are available.
> 
> These are nowadays available on https://wpewebkit.org/.

True thing, I've reworded that sentence.
Comment 7 Manuel Rego Casasnovas 2018-06-20 08:23:25 PDT
Created attachment 343158 [details]
Patch
Comment 8 WebKit Commit Bot 2018-06-20 08:51:57 PDT
Comment on attachment 343158 [details]
Patch

Clearing flags on attachment: 343158

Committed r233012: <https://trac.webkit.org/changeset/233012>
Comment 9 WebKit Commit Bot 2018-06-20 08:51:59 PDT
All reviewed patches have been landed.  Closing bug.
Comment 10 Radar WebKit Bug Importer 2018-06-20 08:52:18 PDT
<rdar://problem/41290804>
Comment 11 Manuel Rego Casasnovas 2018-06-20 13:25:05 PDT
Committed r233021: <https://trac.webkit.org/changeset/233021>