Extend getUserMedia quirk to warbyparker.com
<rdar://problem/72839707>
Created attachment 417814 [details] Patch
Created attachment 417952 [details] Patch
Committed r271667: <https://trac.webkit.org/changeset/271667> All reviewed patches have been landed. Closing bug and clearing flags on attachment 417952 [details].
Comment on attachment 417952 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=417952&action=review > Source/WebCore/page/Quirks.cpp:869 > + m_shouldEnableLegacyGetUserMediaQuirk = host == "www.baidu.com" || host == "www.warbyparker.com"; Why did we stop using equalLettersIgnoringASCIICase? I don’t think a case sensitive check is right for a host name.
(In reply to Darin Adler from comment #5) > Comment on attachment 417952 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=417952&action=review > > > Source/WebCore/page/Quirks.cpp:869 > > + m_shouldEnableLegacyGetUserMediaQuirk = host == "www.baidu.com" || host == "www.warbyparker.com"; > > Why did we stop using equalLettersIgnoringASCIICase? I don’t think a case > sensitive check is right for a host name. This is an optimization since we are using SecurityOrigin::host, which is lower cased. I guess we could revert back to caseless checks in Quirks.cpp for consistency. Or introduce matching methods with SecurityOrigin instead for clarity.
Comment on attachment 417952 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=417952&action=review >>> Source/WebCore/page/Quirks.cpp:869 >>> + m_shouldEnableLegacyGetUserMediaQuirk = host == "www.baidu.com" || host == "www.warbyparker.com"; >> >> Why did we stop using equalLettersIgnoringASCIICase? I don’t think a case sensitive check is right for a host name. > > This is an optimization since we are using SecurityOrigin::host, which is lower cased. > I guess we could revert back to caseless checks in Quirks.cpp for consistency. > Or introduce matching methods with SecurityOrigin instead for clarity. Got it. I didn’t notice that this is calling a different host function now. Code seems fine. Thank you for answering my question.