missing 'used before declaration' error within computed name of accessor or method

TypeScript Version: 3.3.0-dev.20181222

Search Terms:

Code

Taken from the tests added in https://github.com/Microsoft/TypeScript/pull/29136

class D {
    static A = class extends D.B {
        [D.D]() {} // should be an error
    }
    static B = class {}
    static C = {
        [D.D]: 1,
        ...{get [D.D]() {return 0;}} // should be an error
    };
    static D = '';

    [foo]() {} // should have an error
} 

let bar = {
    [foo]() {} // should have an error
}
let foo = 1;

Expected behavior:

The lines marked with // should be an error should have an error.

Actual behavior:

Currently there's no error at all. After #29136 lands the lines with the comments are missing an error.

Playground Link: https://agentcooper.github.io/typescript-play/#code/MYGwhgzhAEAi0G8BQ1XQgFzBglsaAgtALzSiQwCmAHhpQHYAmMsAdAEKIpo8DabsALoAKAJSIAvtAD009AAsA9gFcQjaACNK0MPWiUATgcUHuqCWfRZc+TqXJRJlzNjzQAwiS48+AwQC5oAEYAGkseVkiEAHNKDGh+ViExRAM45QM9AAYAbgkpWQUVNU1tXX0jE0sJHOdrN3hSAHIm2qkgA

Related Issues: #29135

Start by reviewing the tests added in pull request #29136 and the related issue #29135, then reproduce the example with the reported TypeScript version or the Playground link. Done means the marked computed-name cases produce the expected “used before declaration” errors, including accessor and method names.