Add captive portal interstitial icons.

Add captive portal interstitial icons.

These icons are displayed inside captive portal blocking page.

BUG=384667
Committed: https://crrev.com/6e3abef8679370042cb47d7e14fc5e99716fad15
Cr-Commit-Position: refs/heads/master@{#310720}

[meacer, 2014-12-19 23:34:29]

meacer@chromium.org changed reviewers:
+ oshima@chromium.org

[meacer, 2014-12-19 23:34:30]

oshima: Could you PTAL? These icons will replace the T-Rex in the screenshots:
https://goo.gl/cnLIXQ

[oshima, 2014-12-20 20:11:06]

Did you forget to upload grd change?

[meacer, 2014-12-22 20:40:13]

Thanks. The files are referred directly from a .css page. Do I still need to add
an identifier?

[meacer, 2015-01-08 14:28:34]

meacer@chromium.org changed reviewers:
+ bauerb@chromium.org

[meacer, 2015-01-08 14:28:35]

oshima: Ping?

Also +bauerb for chrome/browser/resources/security_warnings/interstitial_v2.css.

[oshima, 2015-01-08 17:45:06]

https://codereview.chromium.org/791083003/diff/20001/chrome/browser/resources...
File chrome/browser/resources/security_warnings/interstitial_v2.css (right):

https://codereview.chromium.org/791083003/diff/20001/chrome/browser/resources...
chrome/browser/resources/security_warnings/interstitial_v2.css:179:
url(../../../app/theme/default_100_percent/common/captive_portal_page_icon.png)
1x,
are you going to use these images from C++ as well?
If not, you should put them in c/b/resources/images like above.

[Bernhard Bauer, 2015-01-08 21:40:22]

CSS LGTM

[meacer, 2015-01-09 02:16:14]

https://codereview.chromium.org/791083003/diff/20001/chrome/browser/resources...
File chrome/browser/resources/security_warnings/interstitial_v2.css (right):

https://codereview.chromium.org/791083003/diff/20001/chrome/browser/resources...
chrome/browser/resources/security_warnings/interstitial_v2.css:179:
url(../../../app/theme/default_100_percent/common/captive_portal_page_icon.png)
1x,
On 2015/01/08 17:45:06, oshima wrote:
> are you going to use these images from C++ as well?
> If not, you should put them in c/b/resources/images like above.

Not planning to use in C++ yet, moved.

[oshima, 2015-01-09 06:00:58]

lgtm (though I'm not the owner)

[meacer, 2015-01-09 06:07:40]

Looks like Bernhard is, that's convenient :) Thanks!

[meacer, 2015-01-09 06:07:57]

The CQ bit was checked by meacer@chromium.org

[commit-bot: I haz the power, 2015-01-09 06:08:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/791083003/40001

[commit-bot: I haz the power, 2015-01-09 06:50:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-01-09 06:51:29]

Patchset 3 (id:??) landed as
https://crrev.com/6e3abef8679370042cb47d7e14fc5e99716fad15
Cr-Commit-Position: refs/heads/master@{#310720}