[ServiceWorker] Update .focus() to return a WindowClient instead of bool.

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 154 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.
-    virtual void focus(int clientID, WebServiceWorkerClientFocusCallback* callback)
-    {
-        // FIXME: call BLINK_ASSERT_NOT_REACHED() when Chromium implementation
-        // is present. https://crbug.com/437149
-        bool result = true;
-        callback->onSuccess(&result);
-        delete callback;
-    }
+    // FIXME: delete this method as soon as Chromium stop calling it.
+    virtual void focus(int clientID, WebServiceWorkerClientFocusCallback*) { BLINK_ASSERT_NOT_REACHED(); }

[Mike West, 2015/02/02 18:11:18]

Is the assert what you want?

[mlamouri (slow - plz ping), 2015/02/03 16:55:56]

Yes :)

+
+    // Ownership of the passed callbacks is transferred to the callee, callee
+    // 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