Fix Presentation API mocks: do not eagerly drop clients.

content/shell/renderer/test_runner/mock_presentation_service.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/shell/renderer/test_runner/mock_presentation_service.h"
 
 #include "base/logging.h"
 
 namespace content {
 
@@ skipping 10 matching lines @@
     return;
   screen_availability_ = available;
 
   FOR_EACH_OBSERVER(
       MockPresentationClient,
       presentation_clients_,
       SetScreenAvailability(available));
 }
 
 void MockPresentationService::Reset() {
-  FOR_EACH_OBSERVER(MockPresentationClient, presentation_clients_, Reset());

[whywhat, 2015/02/19 14:30:35]

I think you need to leave this line as presentation_clients_ need to reset the
availability state between the tests.

[mlamouri (slow - plz ping), 2015/02/19 14:44:31]

Indeed. Didn't mean to remove it.

-  presentation_clients_.Clear();
   screen_availability_ = false;
 }
 
 void MockPresentationService::RegisterPresentationClientMock(
     MockPresentationClient* client) {
   DCHECK(client);
   presentation_clients_.AddObserver(client);
 }
 
 void MockPresentationService::UnregisterPresentationClientMock(
     MockPresentationClient* client) {
   DCHECK(client);
   presentation_clients_.RemoveObserver(client);
 }
 
 }  // namespace content