OOR-CORS: call Resource::SetSourceOrigin on each Resource creation

This is a preparation to merge the source origin and the fetcher origin
in Resource. Now the source origin is set when the Resource is cacheable,
or for preloads.

With this logic, in most cases, SetSourceOrigin is called twice, or
isn't called for some cases that do not need the origin for managing
the memory cache.

This patch makes the origin available on each Resource creation
so that all Resource instances can have it to be used for another
purpose, e.g. caching meta information for SW, and so on.

Bug: 803766
Change-Id: Ib03fbe52f473d6cf70a5920c94f4bf421d851334
Reviewed-on: https://chromium-review.googlesource.com/940364Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541392}

third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.cpp

@@ -466,6 +466,7 @@ Resource* ResourceFetcher::ResourceForStaticData(
     resource->SetResourceBuffer(data);
   resource->SetIdentifier(CreateUniqueIdentifier());
   resource->SetCacheIdentifier(cache_identifier);
+  resource->SetSourceOrigin(GetSourceOrigin(params.Options()));
   resource->Finish(0.0, Context().GetLoadingTaskRunner().get());
 
   if (!substitute_data.IsValid())
@@ -482,6 +483,7 @@ Resource* ResourceFetcher::ResourceForBlockedRequest(
       params.GetResourceRequest(), params.Options(), params.DecoderOptions());
   resource->SetStatus(ResourceStatus::kPending);
   resource->NotifyStartLoad();
+  resource->SetSourceOrigin(GetSourceOrigin(params.Options()));
   resource->FinishAsError(ResourceError::CancelledDueToAccessCheckError(
                               params.Url(), blocked_reason),
                           Context().GetLoadingTaskRunner().get());
@@ -875,8 +877,6 @@ void ResourceFetcher::AddToMemoryCacheIfNeeded(const FetchParameters& params,
   if (!ShouldResourceBeAddedToMemoryCache(params, resource))
     return;
 
-  resource->SetSourceOrigin(GetSourceOrigin(params.Options()));
-
   GetMemoryCache()->Add(resource);
 }
 
@@ -898,6 +898,7 @@ Resource* ResourceFetcher::CreateResourceForLoading(
     resource->SetPreloadDiscoveryTime(params.PreloadDiscoveryTime());
   }
   resource->SetCacheIdentifier(cache_identifier);
+  resource->SetSourceOrigin(GetSourceOrigin(params.Options()));
 
   AddToMemoryCacheIfNeeded(params, resource);
   return resource;
@@ -1011,8 +1012,6 @@ void ResourceFetcher::InsertAsPreloadIfNecessary(Resource* resource,
   if (preloads_.find(key) != preloads_.end())
     return;
 
-  resource->SetSourceOrigin(GetSourceOrigin(params.Options()));
-
   preloads_.insert(key, resource);
   resource->MarkAsPreload();
   if (preloaded_urls_for_test_)