Code review comment for lp:~abreu-alexandre/webbrowser-app/enable-per-app-ua-override

Revision history for this message
Olivier Tilloy (osomon) wrote :

30 + property string localUserAgentOverride
63 + property string localUserAgentOverride
79 + property string localUserAgentOverride

Can the default value of this property be explicitly set to "" ?
It would make sense, given that the getUAString function checks its length (and I’m not sure what the default value of a property is in QML if not explicitly defined, but in any case we shouldn’t rely on a behaviour that might change).

« Back to merge proposal