Added |load_completion_status| to WebStateObserver's |PageLoaded|.

ios/web/public/web_state/web_state_observer.h

 // Copyright 2014 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.
 
 #ifndef IOS_WEB_PUBLIC_WEB_STATE_WEB_STATE_OBSERVER_H_
 #define IOS_WEB_PUBLIC_WEB_STATE_WEB_STATE_OBSERVER_H_
 
 #include "base/macros.h"
 
 namespace web {
@@ skipping 10 matching lines @@
   WebState* web_state() const { return web_state_; }
 
   // This method is invoked when a new non-pending navigation item is created.
   // This corresponds to one NavigationManager item being created
   // (in the case of new navigations) or renavigated to (for back/forward
   // navigations).
   virtual void NavigationItemCommitted(
       const LoadCommittedDetails& load_details) {}
 
   // Called when the current page is loaded.
-  virtual void PageLoaded() {}
+  virtual void PageLoaded(bool load_success) {}

[stuartmorgan, 2015/01/06 19:29:27]

Since PageLoaded(false) isn't quite obvious (it sounds like the page hasn't
loaded), how about an enum instead of a bool?

 
   // Called on URL hash change events.
   virtual void URLHashChanged() {}
 
   // Called on history state change events.
   virtual void HistoryStateChanged() {}
 
   // Invoked when the WebState is being destroyed. Gives subclasses a chance
   // to cleanup.
   virtual void WebStateDestroyed() {}
@@ skipping 20 matching lines @@
   void ResetWebState();
 
   WebState* web_state_;
 
   DISALLOW_COPY_AND_ASSIGN(WebStateObserver);
 };
 
 }  // namespace web
 
 #endif  // IOS_WEB_PUBLIC_WEB_STATE_WEB_STATE_OBSERVER_H_