Toolbar: Report offline state through icon and verbose text.

components/toolbar/BUILD.gn

Index: components/toolbar/BUILD.gn
diff --git a/components/toolbar/BUILD.gn b/components/toolbar/BUILD.gn
index 94163637c2b524e11718ef6e49a19ac62cef49f6..6f2271d708502eeb2698cb724b24e7eb02b030cf 100644
--- a/components/toolbar/BUILD.gn
+++ b/components/toolbar/BUILD.gn
@@ -24,6 +24,7 @@ aggregate_vector_icons("toolbar_vector_icons") {
     "https_valid.icon",
     "https_valid_in_chip.1x.icon",
     "https_valid_in_chip.icon",
+    "offline_pin.icon",

[Peter Kasting, 2017/07/07 01:21:55]

You should likely add distinct 1x and 2x versions, as we've done for these
others.

[cjgrant, 2017/07/07 15:08:00]

My understanding of 1x is that they're hand-tuned to render nicely onto a 16x16
canvas, if the icon has features that warrant it.  As of now, there's no
foreseeable need for a 1x icon, as only VR uses it.  I've messaged Terry
Anderson offline about this - he mentioned adding a mechanism for catching
misuse of non-1x icons, I believe.

[cjgrant, 2017/07/07 17:25:47]

After talking to Terry, I still think this should be okay. The icon is usable at
any resolution, and a 1x icon could be introduced if required later.  However,
the check Terry mentioned is different - it ensures that a 1x icon isn't
rendered at another size (not applicable here).

     "product.1x.icon",
     "product.icon",
     "star_active.icon",