Remove ContextClient from NavigatorCredentialManager

third_party/WebKit/Source/modules/credentialmanager/NavigatorCredentials.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/credentialmanager/NavigatorCredentials.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/LocalDOMWindow.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Navigator.h"
 #include "modules/credentialmanager/CredentialsContainer.h"
 
 namespace blink {
 
 NavigatorCredentials::NavigatorCredentials(Navigator& navigator)
-    : ContextClient(navigator.frame()) {}
+    : Supplement<Navigator>(navigator) {}
 
 NavigatorCredentials& NavigatorCredentials::from(Navigator& navigator) {
   NavigatorCredentials* supplement = static_cast<NavigatorCredentials*>(
       Supplement<Navigator>::from(navigator, supplementName()));
   if (!supplement) {
     supplement = new NavigatorCredentials(navigator);
     provideTo(navigator, supplementName(), supplement);
   }
   return *supplement;
 }
 
 const char* NavigatorCredentials::supplementName() {
   return "NavigatorCredentials";
 }
 
 CredentialsContainer* NavigatorCredentials::credentials(Navigator& navigator) {
   return NavigatorCredentials::from(navigator).credentials();
 }
 
 CredentialsContainer* NavigatorCredentials::credentials() {
-  if (!m_credentialsContainer && frame())

[haraken, 2017/01/06 01:04:50]

I don't think the frame() check makes sense. So dropped.

[sof, 2017/01/06 06:25:57]

yes, no implicit dependency on there being a frame when creating that object
either.

+  if (!m_credentialsContainer)
     m_credentialsContainer = CredentialsContainer::create();
   return m_credentialsContainer.get();
 }
 
 DEFINE_TRACE(NavigatorCredentials) {
   visitor->trace(m_credentialsContainer);
   Supplement<Navigator>::trace(visitor);
-  ContextClient::trace(visitor);
 }
 
 }  // namespace blink