Add ASSERT() to avoid accidental leaking ServiceWorkerContainer to cross origin context.

BUG=522791

Review URL: https://codereview.chromium.org/1305903007

git-svn-id: svn://svn.chromium.org/blink/trunk@201889 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/modules/serviceworkers/NavigatorServiceWorker.cpp

@@ -54,8 +54,9 @@ const char* NavigatorServiceWorker::supplementName()
     return "NavigatorServiceWorker";
 }
 
-ServiceWorkerContainer* NavigatorServiceWorker::serviceWorker(Navigator& navigator, ExceptionState& exceptionState)
+ServiceWorkerContainer* NavigatorServiceWorker::serviceWorker(ExecutionContext* executionContext, Navigator& navigator, ExceptionState& exceptionState)
 {
+    ASSERT(!navigator.frame() || executionContext->securityOrigin()->canAccessCheckSuborigins(navigator.frame()->securityContext()->securityOrigin()));
     return NavigatorServiceWorker::from(navigator).serviceWorker(exceptionState);
 }
 

third_party/WebKit/Source/modules/serviceworkers/NavigatorServiceWorker.h

@@ -26,7 +26,7 @@ public:
 
     virtual ~NavigatorServiceWorker();
 
-    static ServiceWorkerContainer* serviceWorker(Navigator&, ExceptionState&);
+    static ServiceWorkerContainer* serviceWorker(ExecutionContext*, Navigator&, ExceptionState&);
 
     DECLARE_VIRTUAL_TRACE();
 

third_party/WebKit/Source/modules/serviceworkers/NavigatorServiceWorker.idl

@@ -4,5 +4,5 @@
 
 // https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#navigator-service-worker
 partial interface Navigator {
-    [RaisesException] readonly attribute ServiceWorkerContainer serviceWorker;
+    [RaisesException, CallWith=ExecutionContext] readonly attribute ServiceWorkerContainer serviceWorker;
 };