Add metrics for usage of permission features from iframes in blink

third_party/WebKit/Source/core/dom/Element.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Peter Kelly (pmk@post.com)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved.
  *           (C) 2007 Eric Seidel (eric@webkit.org)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 3023 matching lines @@
         return;
     setElementFlag(IsInTopLayer, inTopLayer);
 
     // We must ensure a reattach occurs so the layoutObject is inserted in the correct sibling order under LayoutView according to its
     // top layer position, or in its usual place if not in the top layer.
     lazyReattachIfAttached();
 }
 
 void Element::requestPointerLock()
 {
+    UseCounter::countCrossOriginIframe(document(), UseCounter::ElementRequestPointerLockIframe);

[mlamouri (slow - plz ping), 2016/01/19 17:54:48]

Maybe you could move this into `PointerLockController::requestPointerLock`?

[raymes, 2016/01/20 00:13:12]

Done. Is that just to keep code out of Element.cpp?

[mlamouri (slow - plz ping), 2016/01/21 13:53:18]

Yes.

     if (document().page())
         document().page()->pointerLockController().requestPointerLock(this);
 }
 
 SpellcheckAttributeState Element::spellcheckAttributeState() const
 {
     const AtomicString& value = fastGetAttribute(spellcheckAttr);
     if (value == nullAtom)
         return SpellcheckAttributeDefault;
     if (equalIgnoringCase(value, "true") || equalIgnoringCase(value, ""))
@@ skipping 595 matching lines @@
 {
 #if ENABLE(OILPAN)
     if (hasRareData())
         visitor->trace(elementRareData());
     visitor->trace(m_elementData);
 #endif
     ContainerNode::trace(visitor);
 }
 
 } // namespace blink