[WIP] Add WebCore.UseCounter UMA to supercede FeatureObserver

third_party/WebKit/Source/core/page/Page.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All Rights Reserved.
  * Copyright (C) 2008 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 447 matching lines @@
 void Page::updateAcceleratedCompositingSettings()
 {
     for (Frame* frame = mainFrame(); frame; frame = frame->tree().traverseNext()) {
         if (!frame->isLocalFrame())
             continue;
         if (FrameView* view = toLocalFrame(frame)->view())
             view->updateAcceleratedCompositingSettings();
     }
 }
 
-void Page::didCommitLoad(LocalFrame* frame)
+void Page::didCommitLoad()
 {
+    useCounter().didCommitLoad();
+    frameHost().visualViewport().sendUMAMetrics();
+}
+
+void Page::deprecatedDidCommitFrameLoad(LocalFrame* frame)
+{
+    // This method can be invoked at surprising times.  Most logic probably wants to move
+    // into didCommitLoad.
     notifyDidCommitLoad(frame);
     if (m_mainFrame == frame) {
         frame->console().clearMessages();
-        useCounter().didCommitLoad();
+        useCounter().deprecatedDidCommitFrameLoad();
         deprecation().clearSuppression();
-        frameHost().visualViewport().sendUMAMetrics();
         m_hostsUsingFeatures.updateMeasurementsAndClear();
         UserGestureIndicator::clearProcessedUserGestureSinceLoad();
     }
 }
 
 void Page::acceptLanguagesChanged()
 {
     HeapVector<Member<LocalFrame>> frames;
 
     // Even though we don't fire an event from here, the LocalDOMWindow's will fire
@@ skipping 87 matching lines @@
 {
 }
 
 Page::PageClients::~PageClients()
 {
 }
 
 template class CORE_TEMPLATE_EXPORT Supplement<Page>;
 
 } // namespace blink