arb_texture_storage: Use piglit idoms

This mostly reverts bd096a8a, but it keeps the subtest reporting.  The
non-idomatic way this test was implemented was causing confusion for at
least one new piglit developer.

v2: Rebase on top of 3c21a59.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Cc: Corey Richardson <corey@octayn.net>
Cc: Jon Ashburn <jon@lunarg.com>
Cc: Brian Paul <brianp@vmware.com>
arb_texture_storage: Update test requirements

The test uses 3D textures which were not part of OpenGL until 1.2.  This
doesn't matter much in practice because the Linux OpenGL ABI requires at
least OpenGL 1.2.  Change the minimum GL version from 1.0 to 1.2.

Also, require GL_ARB_texture_cube_map for "cube texture" subtest.

v2: Add the GL_ARB_texture_cube_map requirement.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Cc: Brian Paul <brianp@vmware.com>
arb_texture_storage: Comsetic fixes

Word wrap for 80-column displays.  Use the canonical form for spec
quotations.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
summary.py: align output of Summary.generate_text()

This lines up the colons for dmesg-* statuses with the other statuses.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
status.py: convert status object attributes from instance to class

This converts Status objects attributes to class attributes. This should
reduce lookup time and memory consumption of these classes.

Tested-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
summary.py: Replace buildDictionary function with a much simpler method

This patch produces the exact same output as the previous approach, just
in a much simpler way. It also moves some more of the status information
out of if branches and into the Status class in the status module.

There are a few side effects of using a defaultdict class, namely that
they do not generate a KeyError when one calls for a non existent key.
This means there are a couple of places where try/except blocks are
replaced by if checks.

v2: - fix HTMLIndex._groupResult()
    - don't overwrite the previous status with skip
    - Use a helper function to reduce code duplication

Tested-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>