⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 254601 in webkit


Ignore:
Timestamp:
Jan 15, 2020, 11:15:19 AM (7 years ago)
Author:
Alan Coon
Message:

Cherry-pick r254239. rdar://problem/58549100

Resource Load Statistics: Flip experimental cookie blocking setting from an enable to a disable
​https://bugs.webkit.org/show_bug.cgi?id=205963
<rdar://problem/58424136>

Reviewed by Brent Fulgham.

To get default on behavior, experimental features in the network process need to be
turned from enable flags to disable flags. This patch does that for the experimental
cookie blocking flag.

Source/WebCore:

No new tests. This change just reverses the interpretation of a flag.

  • page/Settings.yaml:

Source/WebKit:

This change also aligns the init values of the setting to match the default.

  • NetworkProcess/Classifier/ResourceLoadStatisticsStore.h:
  • NetworkProcess/NetworkSession.h:
  • NetworkProcess/NetworkSessionCreationParameters.h:
  • Shared/WebPreferences.yaml:
  • UIProcess/WebProcessPool.cpp: (WebKit::WebProcessPool::ensureNetworkProcess):
  • UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm: (WebKit::WebsiteDataStore::parameters):

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@254239 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Location:
branches/safari-609-branch/Source
Files:
9 edited

Legend:

Unmodified
Added
Removed

  • branches/safari-609-branch/Source/WebCore/ChangeLog

    r254598 r254601  
     12020-01-14  Alan Coon  <alancoon@apple.com>
     2
     3        Cherry-pick r254239. rdar://problem/58549100
     4
     5    Resource Load Statistics: Flip experimental cookie blocking setting from an enable to a disable
     6    https://bugs.webkit.org/show_bug.cgi?id=205963
     7    <rdar://problem/58424136>
     8   
     9    Reviewed by Brent Fulgham.
     10   
     11    To get default on behavior, experimental features in the network process need to be
     12    turned from enable flags to disable flags. This patch does that for the experimental
     13    cookie blocking flag.
     14   
     15    Source/WebCore:
     16   
     17    No new tests. This change just reverses the interpretation of a flag.
     18   
     19    * page/Settings.yaml:
     20   
     21    Source/WebKit:
     22   
     23    This change also aligns the init values of the setting to match the default.
     24   
     25    * NetworkProcess/Classifier/ResourceLoadStatisticsStore.h:
     26    * NetworkProcess/NetworkSession.h:
     27    * NetworkProcess/NetworkSessionCreationParameters.h:
     28    * Shared/WebPreferences.yaml:
     29    * UIProcess/WebProcessPool.cpp:
     30    (WebKit::WebProcessPool::ensureNetworkProcess):
     31    * UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:
     32    (WebKit::WebsiteDataStore::parameters):
     33   
     34   
     35    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@254239 268f45cc-cd09-0410-ab3c-d52691b4dbfc
     36
     37    2020-01-08  John Wilander  <wilander@apple.com>
     38
     39            Resource Load Statistics: Flip experimental cookie blocking setting from an enable to a disable
     40            https://bugs.webkit.org/show_bug.cgi?id=205963
     41            <rdar://problem/58424136>
     42
     43            Reviewed by Brent Fulgham.
     44
     45            To get default on behavior, experimental features in the network process need to be
     46            turned from enable flags to disable flags. This patch does that for the experimental
     47            cookie blocking flag.
     48
     49            No new tests. This change just reverses the interpretation of a flag.
     50
     51            * page/Settings.yaml:
     52
    1532020-01-14  Alan Coon  <alancoon@apple.com>
    254

  • branches/safari-609-branch/Source/WebCore/page/Settings.yaml

    r253898 r254601  
    887887  inspectorOverride: true
    888888
    889 isThirdPartyCookieBlockingEnabled:
    890   initial: true
     889isThirdPartyCookieBlockingDisabled:
     890  initial: false
    891891
    892892isFirstPartyWebsiteDataRemovalEnabled:

  • branches/safari-609-branch/Source/WebKit/ChangeLog

    r254599 r254601  
     12020-01-14  Alan Coon  <alancoon@apple.com>
     2
     3        Cherry-pick r254239. rdar://problem/58549100
     4
     5    Resource Load Statistics: Flip experimental cookie blocking setting from an enable to a disable
     6    https://bugs.webkit.org/show_bug.cgi?id=205963
     7    <rdar://problem/58424136>
     8   
     9    Reviewed by Brent Fulgham.
     10   
     11    To get default on behavior, experimental features in the network process need to be
     12    turned from enable flags to disable flags. This patch does that for the experimental
     13    cookie blocking flag.
     14   
     15    Source/WebCore:
     16   
     17    No new tests. This change just reverses the interpretation of a flag.
     18   
     19    * page/Settings.yaml:
     20   
     21    Source/WebKit:
     22   
     23    This change also aligns the init values of the setting to match the default.
     24   
     25    * NetworkProcess/Classifier/ResourceLoadStatisticsStore.h:
     26    * NetworkProcess/NetworkSession.h:
     27    * NetworkProcess/NetworkSessionCreationParameters.h:
     28    * Shared/WebPreferences.yaml:
     29    * UIProcess/WebProcessPool.cpp:
     30    (WebKit::WebProcessPool::ensureNetworkProcess):
     31    * UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:
     32    (WebKit::WebsiteDataStore::parameters):
     33   
     34   
     35    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@254239 268f45cc-cd09-0410-ab3c-d52691b4dbfc
     36
     37    2020-01-08  John Wilander  <wilander@apple.com>
     38
     39            Resource Load Statistics: Flip experimental cookie blocking setting from an enable to a disable
     40            https://bugs.webkit.org/show_bug.cgi?id=205963
     41            <rdar://problem/58424136>
     42
     43            Reviewed by Brent Fulgham.
     44
     45            To get default on behavior, experimental features in the network process need to be
     46            turned from enable flags to disable flags. This patch does that for the experimental
     47            cookie blocking flag.
     48
     49            This change also aligns the init values of the setting to match the default.
     50
     51            * NetworkProcess/Classifier/ResourceLoadStatisticsStore.h:
     52            * NetworkProcess/NetworkSession.h:
     53            * NetworkProcess/NetworkSessionCreationParameters.h:
     54            * Shared/WebPreferences.yaml:
     55            * UIProcess/WebProcessPool.cpp:
     56            (WebKit::WebProcessPool::ensureNetworkProcess):
     57            * UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:
     58            (WebKit::WebsiteDataStore::parameters):
     59
    1602020-01-14  Alan Coon  <alancoon@apple.com>
    261

  • branches/safari-609-branch/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.h

    r253484 r254601  
    275275    bool m_debugLoggingEnabled { false };
    276276    bool m_debugModeEnabled { false };
    277     WebCore::ThirdPartyCookieBlockingMode m_thirdPartyCookieBlockingMode { WebCore::ThirdPartyCookieBlockingMode::AllOnSitesWithoutUserInteraction };
     277    WebCore::ThirdPartyCookieBlockingMode m_thirdPartyCookieBlockingMode { WebCore::ThirdPartyCookieBlockingMode::All };
    278278    bool m_dataRecordsBeingRemoved { false };
    279279    ShouldIncludeLocalhost m_shouldIncludeLocalhost { ShouldIncludeLocalhost::Yes };

  • branches/safari-609-branch/Source/WebKit/NetworkProcess/NetworkSession.h

    r253277 r254601  
    154154    bool m_enableResourceLoadStatisticsLogTestingEvent;
    155155    bool m_downgradeReferrer { true };
    156     WebCore::ThirdPartyCookieBlockingMode m_thirdPartyCookieBlockingMode { WebCore::ThirdPartyCookieBlockingMode::AllOnSitesWithoutUserInteraction };
     156    WebCore::ThirdPartyCookieBlockingMode m_thirdPartyCookieBlockingMode { WebCore::ThirdPartyCookieBlockingMode::All };
    157157    WebCore::FirstPartyWebsiteDataRemovalMode m_firstPartyWebsiteDataRemovalMode { WebCore::FirstPartyWebsiteDataRemovalMode::None };
    158158#endif

  • branches/safari-609-branch/Source/WebKit/NetworkProcess/NetworkSessionCreationParameters.h

    r253277 r254601  
    8787    bool shouldIncludeLocalhostInResourceLoadStatistics { true };
    8888    bool enableResourceLoadStatisticsDebugMode { false };
    89     WebCore::ThirdPartyCookieBlockingMode thirdPartyCookieBlockingMode { WebCore::ThirdPartyCookieBlockingMode::AllOnSitesWithoutUserInteraction };
     89    WebCore::ThirdPartyCookieBlockingMode thirdPartyCookieBlockingMode { WebCore::ThirdPartyCookieBlockingMode::All };
    9090    WebCore::FirstPartyWebsiteDataRemovalMode firstPartyWebsiteDataRemovalMode { WebCore::FirstPartyWebsiteDataRemovalMode::None };
    9191    bool deviceManagementRestrictionsEnabled { false };

  • branches/safari-609-branch/Source/WebKit/Shared/WebPreferences.yaml

    r253925 r254601  
    18231823  category: experimental
    18241824
    1825 IsThirdPartyCookieBlockingEnabled:
    1826   type: bool
    1827   defaultValue: true
    1828   humanReadableName: "Block All 3rd-Party Cookies (ITP)"
    1829   humanReadableDescription: "Block all third-party cookies when Intelligent Tracking Prevention is enabled"
     1825IsThirdPartyCookieBlockingDisabled:
     1826  type: bool
     1827  defaultValue: false
     1828  humanReadableName: "Disable Full 3rd-Party Cookie Blocking (ITP)"
     1829  humanReadableDescription: "Disable full third-party cookie blocking when Intelligent Tracking Prevention is enabled"
    18301830  category: experimental
    18311831

  • branches/safari-609-branch/Source/WebKit/UIProcess/WebProcessPool.cpp

    r254474 r254601  
    577577    bool shouldIncludeLocalhost = true;
    578578    bool enableResourceLoadStatisticsDebugMode = false;
    579     WebCore::ThirdPartyCookieBlockingMode thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::AllOnSitesWithoutUserInteraction;
     579    WebCore::ThirdPartyCookieBlockingMode thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::All;
    580580    WebCore::FirstPartyWebsiteDataRemovalMode firstPartyWebsiteDataRemovalMode = WebCore::FirstPartyWebsiteDataRemovalMode::None;
    581581    WebCore::RegistrableDomain manualPrevalentResource { };

  • branches/safari-609-branch/Source/WebKit/UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm

    r253834 r254601  
    7373    bool shouldLogCookieInformation = false;
    7474    bool enableResourceLoadStatisticsDebugMode = false;
    75     auto thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::OnlyAccordingToPerDomainPolicy;
     75    auto thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::All;
    7676    auto firstPartyWebsiteDataRemovalMode = WebCore::FirstPartyWebsiteDataRemovalMode::None;
    7777    bool enableLegacyTLS = false;
    … …  
    8888#if ENABLE(RESOURCE_LOAD_STATISTICS)
    8989    enableResourceLoadStatisticsDebugMode = [defaults boolForKey:@"ITPDebugMode"];
    90     if ([defaults boolForKey:[NSString stringWithFormat:@"Experimental%@", WebPreferencesKey::isThirdPartyCookieBlockingEnabledKey().createCFString().get()]])
     90    if ([defaults boolForKey:[NSString stringWithFormat:@"Experimental%@", WebPreferencesKey::isThirdPartyCookieBlockingDisabledKey().createCFString().get()]])
     91        thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::AllOnSitesWithoutUserInteraction;
     92    else
    9193        thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::All;
    92     else
    93         thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::AllOnSitesWithoutUserInteraction;
    9494    if ([defaults boolForKey:[NSString stringWithFormat:@"Experimental%@", WebPreferencesKey::isFirstPartyWebsiteDataRemovalEnabledKey().createCFString().get()]]) {
    9595        if ([defaults boolForKey:[NSString stringWithFormat:@"InternalDebug%@", WebPreferencesKey::isFirstPartyWebsiteDataRemovalReproTestingEnabledKey().createCFString().get()]])
Note: See TracChangeset for help on using the changeset viewer.