ResourceFetcher::requestPreload should set its requestContext

Source/core/fetch/ResourceFetcher.cpp

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
     Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
     Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
     Copyright (C) 2009 Torch Mobile Inc. http://www.torchmobile.com/
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
@@ skipping 1224 matching lines @@
 {
     requestPreload(type, request, charset);
 }
 
 void ResourceFetcher::requestPreload(Resource::Type type, FetchRequest& request, const String& charset)
 {
     // Ensure main resources aren't preloaded, since the cache can't actually reuse the preload.
     if (type == Resource::MainResource)
         return;
 
+    ASSERT(type == Resource::Script || type == Resource::CSSStyleSheet || type == Resource::Image);

[Mike West, 2014/11/11 13:34:18]

Is this an exhaustive list of what goes through this method? We preload a lot of
things...

+
     String encoding;
-    if (type == Resource::Script || type == Resource::CSSStyleSheet)
+    if (type == Resource::Script || type == Resource::CSSStyleSheet) {
         encoding = charset.isEmpty() ? m_document->charset().string() : charset;
 
+        // RequestContext for Resource::Image is set in fetchImage below.
+        determineRequestContext(request.mutableResourceRequest(), type);
+    }
+
     request.setCharset(encoding);
     request.setForPreload(true);
 
     ResourcePtr<Resource> resource;
     // Loading images involves several special cases, so use dedicated fetch method instead.
     if (type == Resource::Image)
         resource = fetchImage(request);
+
     if (!resource)
         resource = requestResource(type, request);
     if (!resource || (m_preloads && m_preloads->contains(resource.get())))
         return;
     TRACE_EVENT_ASYNC_STEP_INTO0("net", "Resource", resource.get(), "Preload");
     resource->increasePreloadCount();
 
     if (!m_preloads)
         m_preloads = adoptPtr(new ListHashSet<Resource*>);
     m_preloads->add(resource.get());
@@ skipping 297 matching lines @@
 
 void ResourceFetcher::trace(Visitor* visitor)
 {
     visitor->trace(m_document);
     visitor->trace(m_loaders);
     visitor->trace(m_multipartLoaders);
     ResourceLoaderHost::trace(visitor);
 }
 
 }