Forward-declare blink::mojom::EngagementLevel

third_party/WebKit/Source/core/dom/Document.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 45 matching lines @@
 #include "core/page/PageVisibilityState.h"
 #include "platform/Length.h"
 #include "platform/Timer.h"
 #include "platform/WebTaskRunner.h"
 #include "platform/loader/fetch/ClientHintsPreferences.h"
 #include "platform/scroll/ScrollTypes.h"
 #include "platform/weborigin/KURL.h"
 #include "platform/weborigin/ReferrerPolicy.h"
 #include "public/platform/WebFocusType.h"
 #include "public/platform/WebInsecureRequestPolicy.h"
-#include "public/platform/site_engagement.mojom-blink.h"
 #include "wtf/Compiler.h"
 #include "wtf/HashSet.h"
 #include "wtf/PassRefPtr.h"
 #include <memory>
 
 namespace blink {
 
+namespace mojom {
+enum class EngagementLevel;

[tapted, 2017/02/06 03:29:23]

This is implicitly `int` -- see "(3)" at
http://en.cppreference.com/w/cpp/language/enum#Scoped_enumerations -- but the
mojo header declares it as int32_t

They're "probably" the same size on the platforms Chrome supports, but I think
this should be 

enum class EngagementLevel : int32_t;

anyway, I'm the current chromium sheriff, and this has become Pri-0. This CL
needs a rebase and fails presubmits now that clang-format sorts headers for you,
so I've made https://codereview.chromium.org/2677003002/ and referenced the CL
here.

+}
+
 class AnimationClock;
 class DocumentTimeline;
 class AXObjectCache;
 class Attr;
 class CDATASection;
 class CSSStyleSheet;
 class CanvasFontCache;
 class CharacterData;
 class ChromeClient;
 class CompositorPendingAnimations;
@@ skipping 1110 matching lines @@
     return m_topLayerElements;
   }
   HTMLDialogElement* activeModalDialog() const;
 
   // A non-null m_templateDocumentHost implies that |this| was created by
   // ensureTemplateDocument().
   bool isTemplateDocument() const { return !!m_templateDocumentHost; }
   Document& ensureTemplateDocument();
   Document* templateDocumentHost() { return m_templateDocumentHost; }
 
-  mojom::blink::EngagementLevel getEngagementLevel() const {
+  mojom::EngagementLevel getEngagementLevel() const {
     return m_engagementLevel;
   }
-  void setEngagementLevel(mojom::blink::EngagementLevel level) {
+  void setEngagementLevel(mojom::EngagementLevel level) {
     m_engagementLevel = level;
   }
 
   // TODO(thestig): Rename these and related functions, since we can call them
   // for controls outside of forms as well.
   void didAssociateFormControl(Element*);
 
   void addConsoleMessage(ConsoleMessage*) final;
 
   LocalDOMWindow* executingWindow() const final;
@@ skipping 503 matching lines @@
 DEFINE_TYPE_CASTS(TreeScope, Document, document, true, true);
 
 }  // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 CORE_EXPORT void showLiveDocumentInstances();
 #endif
 
 #endif  // Document_h