The values in /etc/machine-info can be quoteed like `CHASSIS="handset"`. In this case the check will fail because `"handset"` isn't same as `handset`. We should unquote it.
Created attachment 410108 [details] Patch
Comment on attachment 410108 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410108&action=review > Source/WTF/wtf/glib/ChassisType.cpp:51 > + if (error.get() != nullptr) Do not compare to nullptr, and you don't need the .get() either: if (error) g_warning();
Created attachment 410111 [details] Patch
Sure.
Committed r267789: <https://trac.webkit.org/changeset/267789> All reviewed patches have been landed. Closing bug and clearing flags on attachment 410111 [details].