<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>217927</bug_id>
          
          <creation_ts>2020-10-19 16:00:23 -0700</creation_ts>
          <short_desc>Remove iOS-only NSUserDefaults-backed UA override</short_desc>
          <delta_ts>2023-06-09 14:05:17 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore Misc.</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Tim Horton">thorton</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ahmad.saleem792</cc>
    
    <cc>a_protyasha</cc>
    
    <cc>ap</cc>
    
    <cc>bfulgham</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>wenson_hsieh</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1699498</commentid>
    <comment_count>0</comment_count>
    <who name="Tim Horton">thorton</who>
    <bug_when>2020-10-19 16:00:23 -0700</bug_when>
    <thetext>We should remove the code in UserAgentIOS.mm that reads an override UA from the NSUserDefault [com.apple.WebFoundation UserAgent].

It is incompatible with the modern need to compose the UA from various bits of information these days (e.g. desktop vs. mobile). Clients should use the API to set the application name or UA instead.

I have stumbled upon one client (com.fark.hey), and there are likely others, so it should be a linked-on-or-after change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1699501</commentid>
    <comment_count>1</comment_count>
    <who name="Tim Horton">thorton</who>
    <bug_when>2020-10-19 16:04:40 -0700</bug_when>
    <thetext>I left a reference here next to the code in https://trac.webkit.org/changeset/268697/webkit</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1701663</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-10-26 16:01:21 -0700</bug_when>
    <thetext>&lt;rdar://problem/70699433&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1898009</commentid>
    <comment_count>3</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-09-12 14:37:01 -0700</bug_when>
    <thetext>Just to update that this FIXME is still present:

https://github.com/WebKit/WebKit/blob/6ead5274db5f92656360fa1fbae3e0091481fc4f/Source/WebCore/platform/ios/UserAgentIOS.mm#L90</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1960712</commentid>
    <comment_count>4</comment_count>
    <who name="Abrar Rahman Protyasha">a_protyasha</who>
    <bug_when>2023-06-09 00:24:06 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/14805</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1960832</commentid>
    <comment_count>5</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2023-06-09 14:05:15 -0700</bug_when>
    <thetext>Committed 265034@main (6a68b0c65d6c): &lt;https://commits.webkit.org/265034@main&gt;

Reviewed commits have been landed. Closing PR #14805 and removing active labels.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>