• yoshiki iguchi's avatar
    Make SystemTrayView owned by the view hierarchy · 3ae2446b
    yoshiki iguchi authored
    Previously, I have added SystemTrayView class which was owned by SystemTray class (crrev.com/727379). But the views in the tray are assumed to be destroyed with the parent view and the bubble, so that caused some issues.
    This patch changes that and makes SystemTrayView owned by the view hierarchy. Then the issues are solved.
    
    This patch also updates the newly-added code in NetworkStateListDetaiedView because the code didn't assume that lifetime of views.
    
    Bug: 790716, 789833
    Change-Id: I62eca328780cabb7c5d68967cc1ddee7a1e23b9a
    Reviewed-on: https://chromium-review.googlesource.com/813153
    Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
    Reviewed-by: default avatarYuki Awano <yawano@chromium.org>
    Reviewed-by: default avatarMitsuru Oshima <oshima@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#523701}
    3ae2446b
system_tray.h 8.64 KB