[ServiceWorker] Cleanup to remove old focus() code kept to not break Chromium.

public/web/WebServiceWorkerContextClient.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 15 matching lines @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef WebServiceWorkerContextClient_h
 #define WebServiceWorkerContextClient_h
 
 #include "WebWorkerPermissionClientProxy.h"
 #include "public/platform/WebMessagePortChannel.h"
-#include "public/platform/WebServiceWorkerClientFocusCallback.h"
 #include "public/platform/WebServiceWorkerClientsClaimCallbacks.h"
 #include "public/platform/WebServiceWorkerClientsInfo.h"
 #include "public/platform/WebServiceWorkerEventResult.h"
 #include "public/platform/WebServiceWorkerSkipWaitingCallbacks.h"
 #include "public/platform/WebURL.h"
 
 namespace blink {
 
 struct WebCrossOriginServiceWorkerClient;
 class WebDataSource;
@@ skipping 118 matching lines @@
     // Ownership of the passed callbacks is transferred to the callee, callee
     // should delete the callbacks after run.
     virtual void skipWaiting(WebServiceWorkerSkipWaitingCallbacks*) { BLINK_ASSERT_NOT_REACHED(); }
 
     // Ownership of the passed callbacks is transferred to the callee, callee
     // should delete the callbacks after run.
     virtual void claim(WebServiceWorkerClientsClaimCallbacks*) { BLINK_ASSERT_NOT_REACHED(); }
 
     // Ownership of the passed callbacks is transferred to the callee, callee
     // should delete the callback after calling either onSuccess or onError.
-    // FIXME: delete this method as soon as Chromium stop calling it.
-    virtual void focus(int clientID, WebServiceWorkerClientFocusCallback*) { BLINK_ASSERT_NOT_REACHED(); }
-
-    // Ownership of the passed callbacks is transferred to the callee, callee

[dominicc (has gone to gerrit), 2015/02/02 18:25:36]

Why not keep the comment about ownership?

[mlamouri (slow - plz ping), 2015/02/06 14:05:47]

The comment is still, it's just a bit above. The diff make it look like I
removed it though.

-    // should delete the callback after calling either onSuccess or onError.
     virtual void focus(int cliendID, WebServiceWorkerClientCallbacks*) { BLINK_ASSERT_NOT_REACHED(); }
 };
 
 } // namespace blink
 
 #endif // WebServiceWorkerContextClient_h