Make Satisfy/Require callbacks pass Surface id and sequence by reference

cc/surfaces/surface_factory.h

Index: cc/surfaces/surface_factory.h
diff --git a/cc/surfaces/surface_factory.h b/cc/surfaces/surface_factory.h
index 99f0aeb54d1d4344775161248604dd4f1d684ea2..7eca9390834084b2894cc4f1bd308e8a38ba5289 100644
--- a/cc/surfaces/surface_factory.h
+++ b/cc/surfaces/surface_factory.h
@@ -85,7 +85,7 @@ class CC_SURFACES_EXPORT SurfaceFactory
 
   // SurfaceFactory's owner can call this when it finds out that SurfaceManager
   // is no longer alive during destruction.
-  void didDestroySurfaceManager() { manager_ = nullptr; }
+  void did_destroy_surface_manager() { manager_ = nullptr; }

[xlai (Olivia), 2016/06/28 21:16:19]

The previous function name uses Blink style instead of Chromium style.

[danakj, 2016/06/28 22:13:14]

I think I prefer DidDestroySurfaceManager instead.

hacker_style() that doesn't start with set_ looks like an accessor to me. I know
the style guide loosened the times when you're allowed to use it to any "cheap"
function, but I'd prefer that still here.

 
  private:
   SurfaceManager* manager_;