[Offline Pages] Make prerenderer notify us of network progress.

chrome/browser/prerender/prerender_contents.cc

 // Copyright (c) 2012 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 "chrome/browser/prerender/prerender_contents.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <functional>
@@ skipping 760 matching lines @@
     prerender_canceler_binding_.Bind(std::move(request));
 }
 
 void PrerenderContents::AddResourceThrottle(
     const base::WeakPtr<PrerenderResourceThrottle>& throttle) {
   resource_throttles_.push_back(throttle);
 }
 
 void PrerenderContents::AddNetworkBytes(int64_t bytes) {
   network_bytes_ += bytes;
+  for (Observer& observer : observer_list_)

[pasko-google - do not use, 2017/02/22 13:32:44]

nit: auto&?

[Pete Williamson, 2017/02/22 18:46:30]

I think this is better as Observer&, auto is helpful when the type is really
long (like a const iterator over some huge type).

[Dmitry Titov, 2017/02/23 04:27:54]

Oh, opposite guidance! :)
I've looked through this file and for consistency with existing here code, the
Observer& seems to win... SO leaving it for now.

[pasko, 2017/02/24 15:10:28]

sgtm

+    observer.OnPrerenderNetworkBytesChanged(this);
 }
 
 }  // namespace prerender