Skip to content

Commit 018549e

Browse files
pixelflingerAndroid (Google) Code Review
authored andcommitted
Merge "remove dependency on android_native{s_priv|buffer}.h"
2 parents 5265466 + af01fea commit 018549e

File tree

20 files changed

+29
-150
lines changed

20 files changed

+29
-150
lines changed

core/jni/android_app_NativeActivity.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
#include <android_runtime/android_app_NativeActivity.h>
2727
#include <android_runtime/android_util_AssetManager.h>
2828
#include <surfaceflinger/Surface.h>
29-
#include <ui/egl/android_natives.h>
29+
#include <system/window.h>
3030
#include <androidfw/InputTransport.h>
3131
#include <utils/Looper.h>
3232

core/jni/com_google_android_gles_jni_EGLImpl.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,8 @@
3030
#include <SkBitmap.h>
3131
#include <SkPixelRef.h>
3232

33+
#include <ui/ANativeObjectBase.h>
34+
3335
#include <gui/SurfaceTexture.h>
3436
#include <gui/SurfaceTextureClient.h>
3537

include/gui/SurfaceTextureClient.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
#include <gui/ISurfaceTexture.h>
2121
#include <gui/SurfaceTexture.h>
2222

23-
#include <ui/egl/android_natives.h>
23+
#include <ui/ANativeObjectBase.h>
2424
#include <ui/Region.h>
2525

2626
#include <utils/RefBase.h>
@@ -31,7 +31,7 @@ namespace android {
3131
class Surface;
3232

3333
class SurfaceTextureClient
34-
: public EGLNativeBase<ANativeWindow, SurfaceTextureClient, RefBase>
34+
: public ANativeObjectBase<ANativeWindow, SurfaceTextureClient, RefBase>
3535
{
3636
public:
3737
SurfaceTextureClient(const sp<ISurfaceTexture>& surfaceTexture);

include/media/stagefright/HardwareAPI.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
#define HARDWARE_API_H_
2020

2121
#include <media/stagefright/OMXPluginBase.h>
22-
#include <ui/android_native_buffer.h>
22+
#include <system/window.h>
2323
#include <utils/RefBase.h>
2424

2525
#include <OMX_Component.h>

include/private/ui/android_natives_priv.h

Lines changed: 0 additions & 17 deletions
This file was deleted.

include/surfaceflinger/Surface.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@
2626

2727
#include <ui/PixelFormat.h>
2828
#include <ui/Region.h>
29-
#include <ui/egl/android_natives.h>
3029

3130
#include <gui/SurfaceTextureClient.h>
3231

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,7 @@
2222

2323
#include <hardware/gralloc.h>
2424
#include <system/window.h>
25-
// FIXME: remove this header, it's for legacy use. native_window is pulled from frameworks/base/native/include/android/
26-
#include <android/native_window.h>
25+
2726
// ---------------------------------------------------------------------------
2827

2928
/* FIXME: this is legacy for pixmaps */
@@ -52,11 +51,11 @@ typedef struct egl_native_pixmap_t
5251
namespace android {
5352

5453
/*
55-
* This helper class turns an EGL android_native_xxx type into a C++
54+
* This helper class turns a ANativeXXX object type into a C++
5655
* reference-counted object; with proper type conversions.
5756
*/
5857
template <typename NATIVE_TYPE, typename TYPE, typename REF>
59-
class EGLNativeBase : public NATIVE_TYPE, public REF
58+
class ANativeObjectBase : public NATIVE_TYPE, public REF
6059
{
6160
public:
6261
// Disambiguate between the incStrong in REF and NATIVE_TYPE
@@ -68,8 +67,8 @@ class EGLNativeBase : public NATIVE_TYPE, public REF
6867
}
6968

7069
protected:
71-
typedef EGLNativeBase<NATIVE_TYPE, TYPE, REF> BASE;
72-
EGLNativeBase() : NATIVE_TYPE(), REF() {
70+
typedef ANativeObjectBase<NATIVE_TYPE, TYPE, REF> BASE;
71+
ANativeObjectBase() : NATIVE_TYPE(), REF() {
7372
NATIVE_TYPE::common.incRef = incRef;
7473
NATIVE_TYPE::common.decRef = decRef;
7574
}
@@ -86,11 +85,11 @@ class EGLNativeBase : public NATIVE_TYPE, public REF
8685
return getSelf(reinterpret_cast<NATIVE_TYPE const*>(base));
8786
}
8887
static void incRef(android_native_base_t* base) {
89-
EGLNativeBase* self = getSelf(base);
88+
ANativeObjectBase* self = getSelf(base);
9089
self->incStrong(self);
9190
}
9291
static void decRef(android_native_base_t* base) {
93-
EGLNativeBase* self = getSelf(base);
92+
ANativeObjectBase* self = getSelf(base);
9493
self->decStrong(self);
9594
}
9695
};

include/ui/EGLNativeSurface.h

Lines changed: 0 additions & 55 deletions
This file was deleted.

include/ui/FramebufferNativeWindow.h

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,9 @@
2424

2525
#include <utils/threads.h>
2626
#include <utils/String8.h>
27-
#include <ui/Rect.h>
28-
29-
#include <pixelflinger/pixelflinger.h>
3027

31-
#include <ui/egl/android_natives.h>
28+
#include <ui/ANativeObjectBase.h>
29+
#include <ui/Rect.h>
3230

3331
#define NUM_FRAME_BUFFERS 2
3432

@@ -44,7 +42,7 @@ class NativeBuffer;
4442
// ---------------------------------------------------------------------------
4543

4644
class FramebufferNativeWindow
47-
: public EGLNativeBase<
45+
: public ANativeObjectBase<
4846
ANativeWindow,
4947
FramebufferNativeWindow,
5048
LightRefBase<FramebufferNativeWindow> >

include/ui/GraphicBuffer.h

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,11 @@
2020
#include <stdint.h>
2121
#include <sys/types.h>
2222

23-
#include <ui/android_native_buffer.h>
23+
#include <ui/ANativeObjectBase.h>
2424
#include <ui/PixelFormat.h>
2525
#include <ui/Rect.h>
2626
#include <utils/Flattenable.h>
27-
#include <pixelflinger/pixelflinger.h>
27+
2828

2929
struct ANativeWindowBuffer;
3030

@@ -37,7 +37,7 @@ class GraphicBufferMapper;
3737
// ===========================================================================
3838

3939
class GraphicBuffer
40-
: public EGLNativeBase<
40+
: public ANativeObjectBase<
4141
ANativeWindowBuffer,
4242
GraphicBuffer,
4343
LightRefBase<GraphicBuffer> >, public Flattenable
@@ -93,7 +93,6 @@ class GraphicBuffer
9393

9494
status_t lock(uint32_t usage, void** vaddr);
9595
status_t lock(uint32_t usage, const Rect& rect, void** vaddr);
96-
status_t lock(GGLSurface* surface, uint32_t usage);
9796
status_t unlock();
9897

9998
ANativeWindowBuffer* getNativeBuffer() const;

0 commit comments

Comments
 (0)