Capture/Replay: Don't force built-in attribs to have a location
Capture/Replay: Don't force built-in attribs to have a location

CaptureMidExecutionSetup() hits an assert while capturing "World War
Doh":

  ANGLE   : FATAL: FrameCapture.cpp:2876 (CaptureMidExecutionSetup):
    ! Assert failed [[[...]]] attrib.location != -1

This is due to CaptureMidExecutionSetup() enforcing that all attributes,
including built-ins, have a location. However, in the case of "World War
Doh", the built-in 'gl_GlobalInvocationID' does not have a location.

Moving the assert to after the gl::IsBuiltInName() check allows the
capture to complete without hitting any asserts.

Bug: angleproject:4048
Test: Capture "World War Doh"
Change-Id: Ice96dec629b1477b11a710283c4ec4bf1c95c6e2
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2606807
Reviewed-by: Jamie Madill <7e492b4f1c8458024932de3ba475cbf015424c30@chromium.org>
Reviewed-by: Cody Northrop <94770d7fe1cce6ea0791fa9927c1107de4d2e37b@google.com>
Commit-Queue: Tim Van Patten <c7814369a8d3d5671c484acc7b3215f1bf4221be@google.com>

ppy/angle,ppy/angle,ppy/angle,ppy/angle