Fix bug preventing app icons in notification settings view.

Fix bug preventing app icons in notification settings view.

Icons loaded asynchronously needed to cause a relayout instead of just
a change to the layout manager.

R=jianli@chromium.org
BUG=315244
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233532

[dewittj, 2013-11-05 21:49:45]

also, since I'm not really sure, should I also call SchedulePaint?

[jianli, 2013-11-05 21:55:19]

lgtm

[jianli, 2013-11-05 21:55:29]

I think it is not needed since GridLayout::Layout will update the bounds of
each subview which causes SchedulePaint to be triggered from
View::SetBounds.


On Tue, Nov 5, 2013 at 1:49 PM, <dewittj@chromium.org> wrote:

> also, since I'm not really sure, should I also call SchedulePaint?
>
> https://codereview.chromium.org/61003002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-11-05 22:22:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dewittj@chromium.org/61003002/1

[commit-bot: I haz the power, 2013-11-06 02:38:37]

Retried try job too often on linux_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[commit-bot: I haz the power, 2013-11-06 18:01:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dewittj@chromium.org/61003002/1

[commit-bot: I haz the power, 2013-11-07 07:43:14]

Change committed as 233532