Remove BLINK_ASSERT() and BLINK_ASSERT_NOT_REACHED() macros.

third_party/WebKit/public/web/WebFrameClient.h

Index: third_party/WebKit/public/web/WebFrameClient.h
diff --git a/third_party/WebKit/public/web/WebFrameClient.h b/third_party/WebKit/public/web/WebFrameClient.h
index 5fd21c9dc1ceb855c01633ff06731e3ecf9f89b1..89ff3aa91b2cb837744e957ac303c168e0e8650f 100644
--- a/third_party/WebKit/public/web/WebFrameClient.h
+++ b/third_party/WebKit/public/web/WebFrameClient.h
@@ -55,6 +55,7 @@
 #include "public/platform/WebStorageQuotaType.h"
 #include "public/platform/WebURLError.h"
 #include "public/platform/WebURLRequest.h"
+#include <memory>
 #include <v8.h>
 
 namespace blink {
@@ -721,7 +722,11 @@ public:
 
     // Checks that the given audio sink exists and is authorized. The result is provided via the callbacks.
     // This method takes ownership of the callbacks pointer.
-    virtual void checkIfAudioSinkExistsAndIsAuthorized(const WebString& sinkId, const WebSecurityOrigin&, WebSetSinkIdCallbacks*) { BLINK_ASSERT_NOT_REACHED(); }
+    virtual void checkIfAudioSinkExistsAndIsAuthorized(const WebString& sinkId, const WebSecurityOrigin&, WebSetSinkIdCallbacks* callbacks)
+    {
+        std::unique_ptr<WebSetSinkIdCallbacks> callback;
+        callback->onError(WebSetSinkIdError::NotSupported);

[esprehn, 2016/04/06 20:02:02]

this doesn't really do anything I think?

You don't use the callbacks argument, you access a nullptr on the stack?

I think you wanted to do callbacks-> here? Dunno what the unique_ptr is for.

[liberato (no reviews please), 2016/04/06 21:02:34]

i think that "std::unique... callback(callbacks)" was intended, to delete
|callbacks| on return.

putting in an explicit delete might be clearer in this case.

[kotenkov, 2016/04/07 06:03:28]

You are correct. Okay, I've changed it to explicit delete.

+    }
 
 protected:
     virtual ~WebFrameClient() { }