Avoid element id collisions with the visual viewport

third_party/WebKit/Source/core/frame/VisualViewport.cpp

Index: third_party/WebKit/Source/core/frame/VisualViewport.cpp
diff --git a/third_party/WebKit/Source/core/frame/VisualViewport.cpp b/third_party/WebKit/Source/core/frame/VisualViewport.cpp
index 61572c8f2ff53a1aec04dea03dc4032b8cd09927..a92d1157a571db5ceabd44e24076cf3bf14d7c48 100644
--- a/third_party/WebKit/Source/core/frame/VisualViewport.cpp
+++ b/third_party/WebKit/Source/core/frame/VisualViewport.cpp
@@ -391,9 +391,10 @@ void VisualViewport::attachToLayerTree(GraphicsLayer* currentLayerTreeRoot) {
         m_innerViewportContainerLayer->platformLayer());
     m_innerViewportScrollLayer->platformLayer()->setUserScrollable(true, true);
     if (mainFrame()) {
-      if (Document* document = mainFrame()->document())
+      if (Document* document = mainFrame()->document()) {
         m_innerViewportScrollLayer->setElementId(createCompositorElementId(
             DOMNodeIds::idForNode(document), CompositorSubElementId::Scroll));

[bokan, 2016/10/21 18:32:28]

Should this be using the new Viewport subelement enum you added?

+      }
     }
 
     m_rootTransformLayer->addChild(m_innerViewportContainerLayer.get());