Insert pinch zoom virtual viewport layers to graphics layer tree.

Source/core/platform/graphics/GraphicsLayer.cpp

Index: Source/core/platform/graphics/GraphicsLayer.cpp
diff --git a/Source/core/platform/graphics/GraphicsLayer.cpp b/Source/core/platform/graphics/GraphicsLayer.cpp
index 461c2547c35e2854a9e7975e4a4a4fab0de097d9..b8a793fdc21e517e67003aced428af32efc166e3 100644
--- a/Source/core/platform/graphics/GraphicsLayer.cpp
+++ b/Source/core/platform/graphics/GraphicsLayer.cpp
@@ -720,7 +720,7 @@ void GraphicsLayer::resetTrackedRepaints()
 
 void GraphicsLayer::addRepaintRect(const FloatRect& repaintRect)
 {
-    if (m_client->isTrackingRepaints()) {
+    if (m_client && m_client->isTrackingRepaints()) {

[jamesr, 2013/06/14 20:55:46]

I think it'd be better to always create a GraphicsLayerClient, even if it's a
no-op.  The GraphicsLayerClient interface is small

[wjmaclean, 2013/06/17 17:45:27]

OK, done.

That being said, it seems weirdly inconsistent that we check in numerous other
places to see if we have a valid client, but not here. I would think either we
should always check, or never check. (It was primarily the existence of these
other checks that convinced me to code it with a null client ... a previous
version of this CL had a client ;-) )

         FloatRect largestRepaintRect(FloatPoint(), m_size);
         largestRepaintRect.intersect(repaintRect);
         RepaintMap::iterator repaintIt = repaintRectMap().find(this);
@@ -936,7 +936,8 @@ void GraphicsLayer::reportMemoryUsage(MemoryObjectInfo* memoryObjectInfo) const
     info.addMember(m_maskLayer, "maskLayer");
     info.addMember(m_replicaLayer, "replicaLayer");
     info.addMember(m_replicatedLayer, "replicatedLayer");
-    info.ignoreMember(m_client);
+    if (m_client)
+        info.ignoreMember(m_client);
     info.addMember(m_name, "name");
     info.addMember(m_nameBase, "nameBase");
     info.addMember(m_layer, "layer");