Add UMA histograms to MHTML save operations.

third_party/WebKit/Source/web/WebFrameSerializer.cpp

 /*
  * Copyright (C) 2009 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 25 matching lines @@
 #include "core/frame/Frame.h"
 #include "core/frame/FrameSerializer.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/RemoteFrame.h"
 #include "core/html/HTMLAllCollection.h"
 #include "core/html/HTMLFrameElementBase.h"
 #include "core/html/HTMLFrameOwnerElement.h"
 #include "core/html/HTMLInputElement.h"
 #include "core/html/HTMLTableElement.h"
 #include "core/loader/DocumentLoader.h"
+#include "platform/Histogram.h"
 #include "platform/SerializedResource.h"
 #include "platform/SharedBuffer.h"
 #include "platform/TraceEvent.h"
 #include "platform/mhtml/MHTMLArchive.h"
 #include "platform/mhtml/MHTMLParser.h"
 #include "platform/network/ResourceRequest.h"
 #include "platform/network/ResourceResponse.h"
 #include "platform/weborigin/KURL.h"
 #include "public/platform/WebString.h"
 #include "public/platform/WebURL.h"
@@ skipping 154 matching lines @@
     if (!frameShouldBeSerializedAsMHTML(webFrame, webDelegate->cacheControlPolicy()))
         return WebData();
 
     // Translate arguments from public to internal blink APIs.
     LocalFrame* frame = toWebLocalFrameImpl(webFrame)->frame();
     MHTMLArchive::EncodingPolicy encodingPolicy = webDelegate->useBinaryEncoding()
         ? MHTMLArchive::EncodingPolicy::UseBinaryEncoding
         : MHTMLArchive::EncodingPolicy::UseDefaultEncoding;
 
     // Serialize.
+    TRACE_EVENT_BEGIN0("page-serialization", "WebFrameSerializer::generateMHTMLParts serializing");
     Vector<SerializedResource> resources;
-    TRACE_EVENT_BEGIN0("page-serialization", "WebFrameSerializer::generateMHTMLParts serializing");
-    MHTMLFrameSerializerDelegate coreDelegate(*webDelegate);
-    FrameSerializer serializer(resources, coreDelegate);
-    serializer.serializeFrame(*frame);
+    {
+        SCOPED_BLINK_UMA_HISTOGRAM_TIMER("PageSerialization.MhtmlGeneration.SerializationTime.SingleFrame");
+        MHTMLFrameSerializerDelegate coreDelegate(*webDelegate);
+        FrameSerializer serializer(resources, coreDelegate);
+        serializer.serializeFrame(*frame);
+    }
     TRACE_EVENT_END1("page-serialization", "WebFrameSerializer::generateMHTMLParts serializing",
         "resource count", static_cast<unsigned long long>(resources.size()));
 
     // Get Content-ID for the frame being serialized.
     String frameContentID = webDelegate->getContentID(webFrame);
 
     // Encode serializer's output as MHTML.
     RefPtr<SharedBuffer> output = SharedBuffer::create();
-    bool isFirstResource = true;
-    for (const SerializedResource& resource : resources) {
-        TRACE_EVENT0("page-serialization", "WebFrameSerializer::generateMHTMLParts encoding");
-        // Frame is the 1st resource (see FrameSerializer::serializeFrame doc
-        // comment). Frames get a Content-ID header.
-        String contentID = isFirstResource ? frameContentID : String();
+    {
+        SCOPED_BLINK_UMA_HISTOGRAM_TIMER("PageSerialization.MhtmlGeneration.EncodingTime.SingleFrame");
+        bool isFirstResource = true;
+        for (const SerializedResource& resource : resources) {
+            TRACE_EVENT0("page-serialization", "WebFrameSerializer::generateMHTMLParts encoding");
+            // Frame is the 1st resource (see FrameSerializer::serializeFrame doc
+            // comment). Frames get a Content-ID header.
+            String contentID = isFirstResource ? frameContentID : String();
 
-        MHTMLArchive::generateMHTMLPart(
-            boundary, contentID, encodingPolicy, resource, *output);
+            MHTMLArchive::generateMHTMLPart(
+                boundary, contentID, encodingPolicy, resource, *output);
 
-        isFirstResource = false;
+            isFirstResource = false;
+        }
     }
     return output.release();
 }
 
 WebData WebFrameSerializer::generateMHTMLFooter(const WebString& boundary)
 {
     TRACE_EVENT0("page-serialization", "WebFrameSerializer::generateMHTMLFooter");
     RefPtr<SharedBuffer> buffer = SharedBuffer::create();
     MHTMLArchive::generateMHTMLFooter(boundary, *buffer);
     return buffer.release();
@@ skipping 27 matching lines @@
 WebString WebFrameSerializer::generateBaseTagDeclaration(const WebString& baseTarget)
 {
     // TODO(yosin) We should call |FrameSerializer::baseTagDeclarationOf()|.
     if (baseTarget.isEmpty())
         return String("<base href=\".\">");
     String baseString = "<base href=\".\" target=\"" + static_cast<const String&>(baseTarget) + "\">";
     return baseString;
 }
 
 } // namespace blink