Add a flag to change when android's progress bar completes

third_party/WebKit/Source/core/loader/ProgressTracker.h

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 19 matching lines @@
 #include "platform/heap/Handle.h"
 #include "wtf/Allocator.h"
 #include "wtf/Forward.h"
 #include "wtf/HashMap.h"
 #include "wtf/Noncopyable.h"
 #include "wtf/OwnPtr.h"
 
 namespace blink {
 
 class LocalFrame;
+class Resource;
 class ResourceResponse;
 struct ProgressItem;
 
 // FIXME: This is only used on Android. Android is the only Chrome
 // browser which shows a progress bar during loading.
 // We should find a better way for Android to get this data and remove this!
 class CORE_EXPORT ProgressTracker final : public GarbageCollectedFinalized<ProgressTracker> {
     WTF_MAKE_NONCOPYABLE(ProgressTracker);
 public:
     static ProgressTracker* create(LocalFrame*);
 
     ~ProgressTracker();
     DECLARE_TRACE();
     void dispose();
 
     double estimatedProgress() const;
 
     void progressStarted();
     void progressCompleted();
 
     void finishedParsing();
 
+    void willStartLoading(const Resource*);
     void incrementProgress(unsigned long identifier, const ResourceResponse&);
     void incrementProgress(unsigned long identifier, int);
     void completeProgress(unsigned long identifier);
 
-    long long totalPageAndResourceBytesToLoad() const { return m_totalPageAndResourceBytesToLoad; }
-    long long totalBytesReceived() const { return m_totalBytesReceived; }
-
 private:
     explicit ProgressTracker(LocalFrame*);
 
     void incrementProgressForMainResourceOnly(unsigned long identifier, int length);
+    void maybeSendProgress();
     void sendFinalProgress();
     void reset();
 
     Member<LocalFrame> m_frame;
-    unsigned long m_mainResourceIdentifier;
-    long long m_totalPageAndResourceBytesToLoad;
-    long long m_totalBytesReceived;

[Nate Chapin, 2016/05/06 22:19:08]

Maybe it should be in a separate patch, but I'm not sure it really helps
readability or is important enough for perf to stash these totals. I changed it
to just iterate over m_progressItems and calculate from scratch whenever
necessary, since I found that easier to reason about.

     double m_lastNotifiedProgressValue;
     double m_lastNotifiedProgressTime;
     double m_progressNotificationInterval;
     double m_progressNotificationTimeInterval;
+    bool m_finishedParsing;
     bool m_finalProgressChangedSent;
     double m_progressValue;
 
     HashMap<unsigned long, OwnPtr<ProgressItem>> m_progressItems;
 };
 
 } // namespace blink
 
 #endif