Add UMAs for SharedBuffer::lock()/unlock()

third_party/WebKit/Source/core/fetch/Resource.h

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller <mueller@kde.org>
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
 
     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.
@@ skipping 39 matching lines @@
 class ResourceLoader;
 class SecurityOrigin;
 class SharedBuffer;
 
 // A resource that is held in the cache. Classes who want to use this object should derive
 // from ResourceClient, to get the function calls in case the requested data has arrived.
 // This class also does the actual communication with the loader to obtain the resource from the network.
 class CORE_EXPORT Resource : public GarbageCollectedFinalized<Resource> {
     WTF_MAKE_NONCOPYABLE(Resource);
 public:
+    // |Type| enum values are used in UMAs, so do not change the values of
+    // existing |Type|. When adding a new |Type|, append it at the last and

[Ilya Sherman, 2016/07/12 21:23:17]

nit: s/at the last/at the end

[hiroshige, 2016/07/15 07:47:05]

Done.

+    // update |kLastResourceType|.
     enum Type {
         MainResource,
         Image,
         CSSStyleSheet,
         Script,
         Font,
         Raw,
         SVGDocument,
         XSLStyleSheet,
         LinkPrefetch,
         LinkPreload,
         TextTrack,
         ImportResource,
         Media, // Audio or video file requested by a HTML5 media element
         Manifest
     };
+    static const int kLastResourceType = Manifest + 1;
 
     enum Status {
         NotStarted,
         Pending, // load in progress
         Cached, // load completed successfully
         LoadError,
         DecodeError
     };
 
     // Exposed for testing.
@@ skipping 275 matching lines @@
 
     Resource::Type m_type;
 };
 
 #define DEFINE_RESOURCE_TYPE_CASTS(typeName) \
     DEFINE_TYPE_CASTS(typeName##Resource, Resource, resource, resource->getType() == Resource::typeName, resource.getType() == Resource::typeName);
 
 } // namespace blink
 
 #endif