[M108 Migration][VD] Fix office 365 Word/Excel show empty blank pages issue 64/289064/3
authorfeifei08.liu <feifei08.liu@samsung.com>
Tue, 28 Feb 2023 05:37:00 +0000 (13:37 +0800)
committerBot Blink <blinkbot@samsung.com>
Thu, 9 Mar 2023 03:15:05 +0000 (03:15 +0000)
1. Fix redirection issue in WebApp.

Now m85, exclude "file://" URLs frome default
SiteInstance.
So if webapp redirect url in file:///index.html.
It will create a new SiteInstance and RenderFrameHost
for the redirected url.

As WRT only get and store main frame once time (the first time
UIprocess communicated with RenderProcess).
So if frame changed, wrt cannot send message to the right frame.

So avoid this code that will not create new frame while redirection
in "file://" URLs.

2. [VD] Disable "OriginIsolationHeader" feature to fix office 365 Word/Excel
show empty blank pages issue.

"OriginIsolationHeader" enabled in M94 which is in development,
it will cause new webview created and
some information do not included in requests,
this feature disabled in M85, so we follow M85 logic disable this feature.

Reference:
- https://review.tizen.org/gerrit/286949/

Change-Id: Ic26c5a4ac80bfa246a3f1d205095093b503ae372
Signed-off-by: feifei08.liu <feifei08.liu@samsung.com>
content/browser/site_instance_impl.cc
content/public/common/content_features.cc

index f591ec3..e48f32d 100644 (file)
@@ -1204,11 +1204,13 @@ bool SiteInstanceImpl::CanBePlacedInDefaultSiteInstance(
     return false;
   }
 
+#if !BUILDFLAG(IS_TIZEN_TV)
   // Exclude "file://" URLs from the default SiteInstance to prevent the
   // default SiteInstance process from accumulating file access grants that
   // could be exploited by other non-isolated sites.
   if (url.SchemeIs(url::kFileScheme))
     return false;
+#endif
 
   // Don't use the default SiteInstance when
   // kProcessSharingWithStrictSiteInstances is enabled because we want each
index 2ce7fba..f6160a8 100644 (file)
@@ -741,9 +741,15 @@ BASE_FEATURE(kNotificationTriggers,
 //
 // The name is "OriginIsolationHeader" because that was the old name when the
 // feature was under development.
+#if BUILDFLAG(IS_TIZEN_TV)
+BASE_FEATURE(kOriginIsolationHeader,
+             "OriginIsolationHeader",
+             base::FEATURE_DISABLED_BY_DEFAULT);
+#else
 BASE_FEATURE(kOriginIsolationHeader,
              "OriginIsolationHeader",
              base::FEATURE_ENABLED_BY_DEFAULT);
+#endif
 
 // History navigation in response to horizontal overscroll (aka gesture-nav).
 BASE_FEATURE(kOverscrollHistoryNavigation,