internal/lsp: return error if there is no builtin package

When we stopped returning an error from awaitInitialized, we didn't
handle this case in LookupBuiltin.

Fixes golang/go#36975.

Change-Id: I9668a7148f60ca1f9cad953d46caf6ec5500541e
Reviewed-on: https://go-review.googlesource.com/c/tools/+/217399
Run-TryBot: Rebecca Stambler <[email protected]>
TryBot-Result: Gobot Gobot <[email protected]>
Reviewed-by: Robert Findley <[email protected]>
(cherry picked from commit 2de6fe5)
Reviewed-on: https://go-review.googlesource.com/c/tools/+/217638
Reviewed-by: Heschi Kreinick <[email protected]>

Author: stamblerre

internal/lsp/cache/view.go

@@ -206,6 +206,9 @@ func (v *view) Rebuild(ctx context.Context) (source.Snapshot, error) {
 func (v *view) LookupBuiltin(ctx context.Context, name string) (*ast.Object, error) {
 	v.awaitInitialized(ctx)
 
+	if v.builtin == nil {
+		return nil, errors.Errorf("no builtin package for view %s", v.name)
+	}
 	data := v.builtin.handle.Get(ctx)
 	if ctx.Err() != nil {
 		return nil, ctx.Err()