• ben@chromium.org's avatar
    Fix destruction tests. · 3b149b73
    ben@chromium.org authored
    I had to find a way to destroy a ViewManagerConnection from the client lib.
    In the new world, the ViewManagerSynchronizer is the InterfaceImpl, and lifetime of all other client objects hangs off it.
    ViewManagerSynchronizer is owned by the ServiceConnector which is owned by the Application object, so I had to find a way to delete that. In the test, I simply reset the ServiceLoader by URL. This tears down the synchronizer & corresponding ViewManagerConnection, triggering the correct notifications.
    
    Dave, I had to add a NULL check in the service manager code because it hadn't considered there could be a NULL loader for a URL. LMK if you have a different idea.
    
    R=davemoore@chromium.org, sky@chromium.org
    http://crbug.com/365012
    
    Review URL: https://codereview.chromium.org/315863002
    
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@274841 0039d316-1c4b-4281-b951-d872f2087c98
    3b149b73
service_manager.cc 5.69 KB