Fix chrome://offline-internal page.

chrome/browser/resources/offline_pages/offline_internals.js

 // Copyright 2016 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.
 
 cr.define('offlineInternals', function() {
   'use strict';
 
   /** @type {!Array<OfflinePage>} */
   var offlinePages = [];
 
@@ skipping 154 matching lines @@
   function refreshLog() {
     browserProxy_.getEventLogs().then(fillEventLog);
     browserProxy_.getLoggingState().then(updateLogStatus);
   }
 
   function initialize() {
     $('clear-all').onclick = deleteAllPages;
     $('clear-selected').onclick = deleteSelectedPages;
     $('refresh').onclick = refreshAll;
     $('download').onclick = download;
+    browserProxy_ =

[dpapad, 2016/06/29 00:22:18]

Is there any good reason to not initialize this at line 15 along with the
declaration?

[Dmitry Titov, 2016/06/29 01:01:30]

Done.

+        offlineInternals.OfflineInternalsBrowserProxyImpl.getInstance();
     $('log-model-on').onclick =
         browserProxy_.setRecordPageModel.bind(browserProxy_, true);
     $('log-model-off').onclick =
         browserProxy_.setRecordPageModel.bind(browserProxy_, false);
     $('log-request-on').onclick =
         browserProxy_.setRecordRequestQueue.bind(browserProxy_, true);
     $('log-request-off').onclick =
         browserProxy_.setRecordRequestQueue.bind(browserProxy_, false);
     $('refresh-logs').onclick = refreshLog;
-    browserProxy_ =
-        offlineInternals.OfflineInternalsBrowserProxyImpl.getInstance();
     refreshAll();
   }
 
   // Return an object with all of the exports.
   return {
     initialize: initialize,
   };
 });
 
 document.addEventListener('DOMContentLoaded', offlineInternals.initialize);