fix(icon): remove automatic aria labelling and add a11y guidance (#4665)

The automatic application of aria-label for md-icon, in hindsight, was
not a great idea. Neither the ligature string nor the SVG filename are
likely to be meaningful descriptions. Even in cases where they *are*,
the icon may be used in such a way that the application of a label is
still the wrong thing. On top of that, adding `role="img"` is usually
not the right approach for an icon, as that role typically refers to
*content* images, rather than icons that are part of the UI.

Ultimately, it is up to the application developer to add the appropriate
meaning for icons based on how they're used. To this end, we add
guidance to the documentation for what to do in different situations.

Author: jelbourn

e2e/components/icon/icon.e2e.ts

@@ -11,11 +11,6 @@ describe('icon', () => {
       testIcon = element(by.id('test-icon'));
     });
 
-    it('should have the correct aria-label when used', () => {
-      expect(testIcon.getAttribute('aria-label')).toBe('favorite');
-      screenshot();
-    });
-
     it('should have the correct class when used', async () => {
       const attr = await testIcon.getAttribute('class');