Bug 224843 – GPUProcess launches unnecessarily when loading the amazon front page

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Chris Dumez]

GPUProcess launches unnecessarily when load the amazon front page.

The Amazon front page is only using the HTMLMediaElement for feature detection, there should not be any need to launch the GPUProcess in this case:
```
l = {
                audio: function() {
                    return !!h.createElement("audio").canPlayType
                },
                video: function() {
                    return !!h.createElement("video").canPlayType
                },
```

[Chris Dumez]

Created attachment 426618 [details]
WIP Patch

Works on iPad with amazon.com (no GPUProcess). Will do more testing and polish the patch tomorrow.

[Chris Dumez]

I am currently A/B testing the change on PLUM3.

[Chris Dumez]

Created attachment 426630 [details]
WIP Patch

[Chris Dumez]

Created attachment 426634 [details]
WIP Patch

[Chris Dumez]

Created attachment 426690 [details]
WIP Patch

Should hopefully pass all the tests now. This seems to be a ~3.6% PLUM progression on iPhone so definitely worth pursuing.

[Chris Dumez]

Created attachment 426692 [details]
Patch

[Chris Dumez]

Created attachment 426706 [details]
Patch

[Chris Dumez]

Created attachment 426716 [details]
Patch

[Eric Carlson]

Comment on attachment 426716 [details]
Patch

This is great, thanks Chris!

[EWS]

Committed r276389 (236864@main): <https://commits.webkit.org/236864@main>

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

[Chris Dumez]

One of the last minute changes I made to fix a layout test must have messed thing up because this patch wasn't a progression when it landed. I had tested the earlier version of my patch and it was a progression so I will have to investigate tomorrow.

[Chris Dumez]

(In reply to Chris Dumez from comment #11)
> One of the last minute changes I made to fix a layout test must have messed
> thing up because this patch wasn't a progression when it landed. I had
> tested the earlier version of my patch and it was a progression so I will
> have to investigate tomorrow.

Looking deeper into A/B testing for this latest patch, I see that it did what I expected it to do. With the patch, there is no GPUProcess on amazon.com-front & amazon.com-search. Before the patch, there was a GPUProcess *before* the memory warning on both these pages. A/B testing indicates we get a ~1% progression (with 98% probability) on both of these pages. However, it doesn't seem this is sufficient to show a progression on the overall score (despite no other page regressing).

[Ling Ho]

<rdar://76977801>