commit 921e6ac4b7610a178285898d191eb0e3afe906c0
author Mathias Agopian <mathias@google.com> Mon Jul 23 23:11:29 2012 -0700
committer Mathias Agopian <mathias@google.com> Tue Jul 24 00:09:35 2012 -0700
tree dbf63e51bc59ea67d63fd17092bcb3e8d188f8e8
parent 36b285eac98cccdb9394c14ae2e6ace9f3d90966

SurfaceFlinger cleanup

mostly refactored SurfaceFlinger.h, but also removed dead code.
cleaned-up a few includes as well.

Change-Id: Ib15f4ffe567912b61ee98aa076c6a283b72811b5

services/surfaceflinger/Client.cpp

diff --git a/services/surfaceflinger/Client.cpp b/services/surfaceflinger/Client.cpp
index 6af1943..d5d551e 100644
--- a/services/surfaceflinger/Client.cpp
+++ b/services/surfaceflinger/Client.cpp
@@ -22,6 +22,7 @@
 #include <private/android_filesystem_config.h>
 
 #include "Client.h"
+#include "Layer.h"
 #include "LayerBase.h"
 #include "SurfaceFlinger.h"
 
@@ -118,7 +119,7 @@
      * have access to the GL context.
      */
 
-    class MessageCreateSurface : public MessageBase {
+    class MessageCreateLayer : public MessageBase {
         sp<ISurface> result;
         SurfaceFlinger* flinger;
         ISurfaceComposerClient::surface_data_t* params;
@@ -129,7 +130,7 @@
         PixelFormat format;
         uint32_t flags;
     public:
-        MessageCreateSurface(SurfaceFlinger* flinger,
+        MessageCreateLayer(SurfaceFlinger* flinger,
                 ISurfaceComposerClient::surface_data_t* params,
                 const String8& name, Client* client,
                 DisplayID display, uint32_t w, uint32_t h, PixelFormat format,
@@ -140,19 +141,19 @@
         }
         sp<ISurface> getResult() const { return result; }
         virtual bool handler() {
-            result = flinger->createSurface(params, name, client,
+            result = flinger->createLayer(params, name, client,
                     display, w, h, format, flags);
             return true;
         }
     };
 
-    sp<MessageBase> msg = new MessageCreateSurface(mFlinger.get(),
+    sp<MessageBase> msg = new MessageCreateLayer(mFlinger.get(),
             params, name, this, display, w, h, format, flags);
     mFlinger->postMessageSync(msg);
-    return static_cast<MessageCreateSurface*>( msg.get() )->getResult();
+    return static_cast<MessageCreateLayer*>( msg.get() )->getResult();
 }
 status_t Client::destroySurface(SurfaceID sid) {
-    return mFlinger->removeSurface(this, sid);
+    return mFlinger->onLayerRemoved(this, sid);
 }
 
 // ---------------------------------------------------------------------------