[webkit-reviews] review granted: [Bug 199435] AX: CrashTracer: com.apple.WebKit.WebContent at WebKit: WebKit::WebSpeechSynthesisClient::speak : [Attachment 373396] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 3 15:14:57 PDT 2019

Ryosuke Niwa <rniwa at webkit.org> has granted chris fleizach
<cfleizach at apple.com>'s request for review:
Bug 199435: AX: CrashTracer: com.apple.WebKit.WebContent at WebKit:

Attachment 373396: patch


--- Comment #6 from Ryosuke Niwa <rniwa at webkit.org> ---
Comment on attachment 373396
  --> https://bugs.webkit.org/attachment.cgi?id=373396

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

>>>>> Source/WebKit/WebProcess/WebCoreSupport/WebSpeechSynthesisClient.cpp:57
>>>>> + 	   observer->didFinishSpeaking();
>>>> Could speechSynthesisClient() also possibly return nullptr?
>>> I looked into it and to my eyes it did not seem so. It looks like it's set
>> Perhaps it could return a reference, then? I don't think this is required in
this patch, though. It can be done in a follow-up patch.
> let me look into that

m_speechSynthesisClient(WTFMove(pageConfiguration.speechSynthesisClient)) so if
pageConfiguration.speechSynthesisClient is null, it could be null.
I think we should add an extra null check for now. We can turn it to a
reference in the future.

More information about the webkit-reviews mailing list