Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(36)

Issue 1180713004: Remove unused code from WTF TypedArrays (Closed)

Created:
4 years, 10 months ago by adamk
Modified:
4 years, 10 months ago
CC:
blink-reviews, blink-reviews-wtf_chromium.org, Dmitry Lomov (no reviews), Jeffrey Yasskin, Mikhail
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

Remove unused code from WTF TypedArrays These look to be leftovers from before V8 implemented TypedArrays. Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196958

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -273 lines) Patch
M Source/wtf/ArrayBuffer.h View 1 chunk +1 line, -2 lines 0 comments Download
M Source/wtf/ArrayBufferView.h View 4 chunks +0 lines, -43 lines 0 comments Download
M Source/wtf/Float32Array.h View 3 chunks +0 lines, -22 lines 0 comments Download
M Source/wtf/Float64Array.h View 2 chunks +0 lines, -18 lines 0 comments Download
M Source/wtf/Int16Array.h View 2 chunks +0 lines, -22 lines 0 comments Download
M Source/wtf/Int32Array.h View 2 chunks +0 lines, -22 lines 0 comments Download
M Source/wtf/Int8Array.h View 2 chunks +0 lines, -22 lines 0 comments Download
M Source/wtf/TypedArrayBase.h View 2 chunks +0 lines, -24 lines 0 comments Download
M Source/wtf/Uint16Array.h View 2 chunks +0 lines, -22 lines 0 comments Download
M Source/wtf/Uint32Array.h View 2 chunks +0 lines, -22 lines 0 comments Download
M Source/wtf/Uint8Array.h View 2 chunks +0 lines, -22 lines 0 comments Download
M Source/wtf/Uint8ClampedArray.h View 3 chunks +0 lines, -32 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
adamk
I think there's more dead code around here, but this was the first bit I ...
4 years, 10 months ago (2015-06-11 01:25:31 UTC) #2
jochen (gone - plz use gerrit)
lgtm
4 years, 10 months ago (2015-06-11 12:43:43 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1180713004/1
4 years, 10 months ago (2015-06-11 14:56:52 UTC) #5
commit-bot: I haz the power
4 years, 10 months ago (2015-06-11 15:54:10 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196958

Powered by Google App Engine
This is Rietveld 408576698