Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in

Issue 2946933002: Revert of [api] Fix compilation error for `UNIMPLEMENTED()` method (Closed)

Can't Edit
Can't Publish+Mail
Start Review
3 months ago by Clemens Hammacher
2 months ago
Franzi, Anna Henningsen
Target Ref:


Revert of [api] Fix compilation error for `UNIMPLEMENTED()` method (patchset #1 id:1 of ) Reason for revert: not needed any more, and contradicts our cleanup efforts: Original issue's description: > [api] Fix compilation error for `UNIMPLEMENTED()` method > > Return `nullptr` from `ArrayBuffer::Allocator::Reserve` because > apparently not doing so results in compile errors for some people. > > BUG= > > Ref: > Review-Url: > Cr-Commit-Position: refs/heads/master@{#45886} > Committed:, Review-Url: Cr-Commit-Position: refs/heads/master@{#46739} Committed:

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -4 lines) Patch
M src/ View 1 chunk +1 line, -4 lines 0 comments Download
Trybot results:  v8_win64_rel_ng_triggered   v8_win_rel_ng_triggered   v8_win_nosnap_shared_rel_ng_triggered   v8_linux64_rel_ng_triggered   v8_linux64_asan_rel_ng_triggered   v8_linux_arm64_rel_ng_triggered   v8_linux64_avx2_rel_ng_triggered   v8_linux_rel_ng_triggered   v8_linux64_verify_csa_rel_ng_triggered   v8_linux_dbg_ng_triggered   v8_linux_verify_csa_rel_ng_triggered   v8_linux64_gyp_rel_ng_triggered   v8_linux_nodcheck_rel_ng_triggered   v8_linux_arm_rel_ng_triggered   v8_mac_rel_ng_triggered   v8_linux_arm_rel_ng   v8_linux64_gyp_rel_ng   v8_linux_rel_ng   v8_node_linux64_rel   v8_linux_mipsel_compile_rel   v8_android_arm_compile_rel   v8_mac_rel_ng   v8_win_compile_dbg   v8_linux_dbg_ng   v8_linux_arm64_rel_ng   v8_linux_mips64el_compile_rel   v8_linux_chromium_gn_rel   v8_linux64_rel_ng   v8_linux_nodcheck_rel_ng   v8_linux64_verify_csa_rel_ng   v8_win64_rel_ng   v8_linux64_gcc_compile_dbg   v8_win_rel_ng   v8_win_nosnap_shared_rel_ng   v8_linux64_asan_rel_ng   v8_presubmit   v8_linux_gcc_compile_rel   v8_linux64_avx2_rel_ng   v8_linux_verify_csa_rel_ng   v8_win64_rel_ng_triggered   v8_linux_arm_rel_ng_triggered   v8_linux_dbg_ng_triggered   v8_linux_rel_ng_triggered   v8_linux64_rel_ng_triggered   v8_win_rel_ng_triggered   v8_win_nosnap_shared_rel_ng_triggered   v8_linux64_gyp_rel_ng_triggered   v8_linux_arm64_rel_ng_triggered   v8_linux_nodcheck_rel_ng_triggered   v8_linux64_asan_rel_ng_triggered   v8_linux64_rel_ng   v8_linux_verify_csa_rel_ng_triggered   v8_linux64_avx2_rel_ng_triggered   v8_linux64_verify_csa_rel_ng_triggered   v8_mac_rel_ng_triggered   v8_linux_arm_rel_ng   v8_linux64_gyp_rel_ng   v8_linux_rel_ng   v8_node_linux64_rel   v8_linux_mipsel_compile_rel   v8_android_arm_compile_rel   v8_mac_rel_ng   v8_linux_arm64_rel_ng   v8_win_compile_dbg   v8_linux_dbg_ng   v8_linux_mips64el_compile_rel   v8_linux_chromium_gn_rel   v8_linux64_rel_ng   v8_linux_nodcheck_rel_ng   v8_win64_rel_ng   v8_linux64_verify_csa_rel_ng   v8_linux64_gcc_compile_dbg   v8_win_nosnap_shared_rel_ng   v8_win_rel_ng   v8_presubmit   v8_linux_gcc_compile_rel   v8_linux64_asan_rel_ng   v8_linux64_avx2_rel_ng   v8_linux_verify_csa_rel_ng 
Commit queue not available (can’t edit this change).


Total messages: 16 (10 generated)
Clemens Hammacher
Created Revert of [api] Fix compilation error for `UNIMPLEMENTED()` method
3 months ago (2017-06-20 17:41:21 UTC) #1
Anna Henningsen
lgtm lgtm lgtm
3 months ago (2017-06-20 17:45:08 UTC) #4
Clemens Hammacher
On 2017/06/20 at 17:45:08, addaleax wrote: > lgtm > > lgtm > > lgtm Ping ...
2 months ago (2017-07-17 10:00:56 UTC) #9
lgtm lgtm sorry for the delay, thought that had landed a long time ago.
2 months ago (2017-07-18 13:57:41 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at:
2 months ago (2017-07-18 13:57:48 UTC) #12
commit-bot: I haz the power
2 months ago (2017-07-18 14:25:38 UTC) #16
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld b40b6558b