ServiceWorker: Make a way to retrieve a service worker version id (Blink)

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 606 matching lines @@
 bool ResourceFetcher::isControlledByServiceWorker() const
 {
     LocalFrame* localFrame = frame();
     if (!localFrame)
         return false;
     if (!m_documentLoader)
         return false;
     return localFrame->loader().client()->isControlledByServiceWorker(*m_documentLoader);
 }
 
+int64_t ResourceFetcher::serviceWorkerID() const
+{
+    LocalFrame* localFrame = frame();
+    if (!localFrame)
+        return -1;
+    if (!m_documentLoader)

[horo, 2014/10/28 01:20:21]

ResourceFetcher of the imported HTML doesn't have m_documentLoader.
So we should check localFrame->loader().documentLoader().

See: https://codereview.chromium.org/655373003/

[nhiroki, 2014/10/28 05:21:22]

Thanks for your information! Done in the same way with you.

+        return -1;
+    return localFrame->loader().client()->serviceWorkerID(*m_documentLoader);
+}
+
 bool ResourceFetcher::shouldLoadNewResource(Resource::Type type) const
 {
     if (!frame())
         return false;
     if (!m_documentLoader)
         return true;
     if (type == Resource::MainResource)
         return m_documentLoader == frame()->loader().provisionalDocumentLoader();
     return m_documentLoader == frame()->loader().documentLoader();
 }
@@ skipping 907 matching lines @@
 
 void ResourceFetcher::trace(Visitor* visitor)
 {
     visitor->trace(m_document);
     visitor->trace(m_loaders);
     visitor->trace(m_multipartLoaders);
     ResourceLoaderHost::trace(visitor);
 }
 
 }