Remove BLINK_ASSERT() and BLINK_ASSERT_NOT_REACHED() macros.

third_party/WebKit/public/web/WebFrameClient.h

 /*
  * Copyright (C) 2011, 2012 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 37 matching lines @@
 #include "public/platform/BlameContext.h"
 #include "public/platform/WebCommon.h"
 #include "public/platform/WebFileSystem.h"
 #include "public/platform/WebFileSystemType.h"
 #include "public/platform/WebSecurityOrigin.h"
 #include "public/platform/WebSetSinkIdCallbacks.h"
 #include "public/platform/WebStorageQuotaCallbacks.h"
 #include "public/platform/WebStorageQuotaType.h"
 #include "public/platform/WebURLError.h"
 #include "public/platform/WebURLRequest.h"
+#include <memory>
 #include <v8.h>
 
 namespace blink {
 
 enum class WebTreeScopeType;
 class WebApplicationCacheHost;
 class WebApplicationCacheHostClient;
 class WebAppBannerClient;
 class WebBluetooth;
 class WebColorChooser;
@@ skipping 646 matching lines @@
     virtual WebBluetooth* bluetooth() { return 0; }
 
     // WebUSB --------------------------------------------------------------
     virtual WebUSBClient* usbClient() { return nullptr; }
 
 
     // Audio Output Devices API --------------------------------------------
 
     // 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() { }
 };
 
 } // namespace blink
 
 #endif