HWC2: Kill logspam for non-HWC virtual displays am: 179533d107 am: 9853a795ab am: 860ff9c846
am: e9a23ea018

Change-Id: I8f83ce11003cd878b8efe7259276e8ba4dff1c4e

HWC2: Fix virtual displays am: 5450fe5268 am: d6fb6ea07b am: aa23ebe03e  -s ours
am: 327490c8c8

Change-Id: I5a370186699fd7b800b3892d9a47f41e0447669c

HWC2: Don't set null handle for client layers am: 6be002845f am: 0fac961a58 am: 64e18c2bfe  -s ours
am: aa51ef39c9

Change-Id: Ic6163f655443bf32173f043127b00f2d4ac3b603

HWC2: Hook up to BoardConfig variable am: 8beb15f220 am: 608852e318 am: 607bb29a98
am: 4131cca88b

Change-Id: I44b1c16742abc57b815c4456ffecc487ae5d1b94

HWC2: Skip validate/present if display is off am: 00ac359a3b am: de79392408 am: ebec74a215
am: 092e03be27

Change-Id: Ia90f061aafc0c93e160152776d7252d178c12aeb

HWC2: Kill logspam for non-HWC virtual displays am: 179533d107 am: 9853a795ab
am: 860ff9c846

Change-Id: Ibc40214c27890b0224918449818b2195dff5de71

HWC2: Fix virtual displays am: 5450fe5268 am: d6fb6ea07b
am: aa23ebe03e  -s ours

Change-Id: I72c0ca4d6dff8e295eaeddc472cb3b3cc49b4498

HWC2: Don't set null handle for client layers am: 6be002845f am: 0fac961a58
am: 64e18c2bfe  -s ours

Change-Id: Ib001496809231f89c31a790617d4aa32e43b6825

HWC2: Hook up to BoardConfig variable am: 8beb15f220 am: 608852e318
am: 607bb29a98

Change-Id: Ica9191d91e58f8526ddf0f7e245bab1ab0ebfefd

HWC2: Skip validate/present if display is off am: 00ac359a3b am: de79392408
am: ebec74a215

Change-Id: Ic7168a2c94ec5c4d93e8b2507c0570d97434829f

Merge "Fix constructor parameters shadowing member variables"

Merge "Add missing liblog dependency"

Add missing liblog dependency

Bug: http://b/31289077
Bug: http://b/27171986
Test: remove liblog from libcutils and recompile
Change-Id: Ib46726fe6b330375bc925f409dfe6b1ce4cfe599

Rename dump methods to dumpState
am: dc782511dc

Change-Id: I9803c375b16a9564bd664eb70e83efccab4672e9

Fix logging implicit sign conversions
am: 6e7e2b44ef

Change-Id: I45daa0ed37856b39e2168859b2c5c93f05ad90fc

Merge "Fix struct vs. class mismatch in forward definitions" am: 1d6c149d89 am: 8806e2a789 am: 132b7491bc
am: 64b463a69b

Change-Id: I85efdbc7a80f9851d67aa72142285b30ed7ace09

Merge "Add "ip xfrm policy" to dumpstate output" into nyc-mr1-dev am: a4bb5c7173 am: 9b60d83c2c
am: efca26a34d

Change-Id: Icc9cbe3a1ebcf23d5f6e61f52aacc3861802b621

Add "ip xfrm policy" to dumpstate output am: 53ca94ab34 am: 5159997ffa
am: 19d86fdcdc  -s ours

Change-Id: Icc691b6504813d40ad15052cf759579a83690174

Merge "Add "ip xfrm policy" to dumpstate output"

Merge "Added initial tests for dumpstate."

Add "ip xfrm policy" to dumpstate output

Test: local build, treehugger

Bug: 30869013
Change-Id: I1f0e5d820f0153484c38ecb0f9c764fca02d786c

Fix constructor parameters shadowing member variables

Using the same name for the constructor parameter and the member
variable causes a warning that was being hidden by the use of -isystem
to include frameworks/native/include.  Prefix the parameter with an
underscore.

Bug: 31752268
Test: m -j
Change-Id: I60c7dc388999bafbb97cb42f5e0af5361df68d56

Rename dump methods to dumpState

The dump methods shadow the IBinder dump method.  The warnings were being
hidden by the use of -isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: Iafc64da43032d5d9d84b64640e70d93fd7051bcf
(cherry picked from commit 3d1d280fa3cbc30700f728a4b13e5308315433d3)

Merge "Rename dump methods to dumpState"

Fix logging implicit sign conversions

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I2ce11db524ee9d8f846fdf3ed6fb882d5ef57956
(cherry picked from commit 152c3b749a8ec634afc66dddef7d33130472aeb1)

Merge "Fix struct vs. class mismatch in forward definitions" am: 1d6c149d89 am: 8806e2a789
am: 132b7491bc

Change-Id: Ibb73e33e1517a724dda605e5d38a57615e48c4e1

HWC2: Kill logspam for non-HWC virtual displays am: 179533d107
am: 9853a795ab

Change-Id: I68c6c6a0bdefb7b196eacd95e486f275c21377df

HWC2: Fix virtual displays am: 5450fe5268
am: d6fb6ea07b

Change-Id: I11166b2d968bf8a4ba23b5ba86bf307a457e8ba9

HWC2: Don't set null handle for client layers am: 6be002845f
am: 0fac961a58

Change-Id: I259c37bd7ba6cf57f07f494cce4bfda7f837d76c

HWC2: Hook up to BoardConfig variable am: 8beb15f220
am: 608852e318

Change-Id: I91252f79de7035b958329a68e4172edb5b2c302a

HWC2: Skip validate/present if display is off am: 00ac359a3b
am: de79392408

Change-Id: I36810f4c2623591942bac447b2b118edc3104f9d

HWC2: Kill logspam for non-HWC virtual displays
am: 179533d107

Change-Id: Ib8398383cb606990920033538938855eda248071

HWC2: Fix virtual displays
am: 5450fe5268

Change-Id: I6710be1bb5c5fb998143b4ad898c94a6eee98382

HWC2: Don't set null handle for client layers
am: 6be002845f

Change-Id: I1fc9959536c40511c4fc49fa613d6fd7a97d43ad

HWC2: Hook up to BoardConfig variable
am: 8beb15f220

Change-Id: I012809dac02ab2d87ac1650be2fc619f2b68fbbf

HWC2: Skip validate/present if display is off
am: 00ac359a3b

Change-Id: I31fad8a631fc6b52cb4b6b4cfc26185ebb0214f2

Merge "Fix struct vs. class mismatch in forward definitions" am: 1d6c149d89
am: 8806e2a789

Change-Id: Ib37b329e85dd3f72953b4f9b4eada5733976bead

Merge "Fix struct vs. class mismatch in forward definitions"
am: 1d6c149d89

Change-Id: Iba83f822a72416e0b63b796f16065e8a672d66d9

Merge "Fix logging implicit sign conversions"

Merge "Fix libgui warnings"

Merge "Fix struct vs. class mismatch in forward definitions"

Merge "Add "ip xfrm policy" to dumpstate output" into nyc-mr1-dev am: a4bb5c7173
am: 9b60d83c2c

Change-Id: I52f750d205ba0671d43553baee040dd3af8bf1c8

Add "ip xfrm policy" to dumpstate output am: 53ca94ab34
am: 5159997ffa

Change-Id: I01a70f51e5c8df9d8421381f58246d49e69e8c52

Merge "Add "ip xfrm policy" to dumpstate output" into nyc-mr1-dev
am: a4bb5c7173

Change-Id: Iad51735819ce9e2d57072e657aa99d471b5baedc

Add "ip xfrm policy" to dumpstate output
am: 53ca94ab34

Change-Id: I0b7b5f39acd68f0e176b4c684c108c7d906e5515

Merge "Add "ip xfrm policy" to dumpstate output" into nyc-mr1-dev

Fix struct vs. class mismatch in forward definitions

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I46208cec43599f775f7e2cb22b39d19b0ea762e1

Rename dump methods to dumpState

The dump methods shadow the IBinder dump method.  The warnings were being
hidden by the use of -isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: Iafc64da43032d5d9d84b64640e70d93fd7051bcf

Merge "Fix weak vtable warnings" am: d93a40c920 am: 11a2bcad06 am: cd77a5a218
am: 97514a25d6

Change-Id: Id5af70ceadd584f0dae7aa4d9bb4811ad822cf28

Merge "Rewrite multicharacter constant using constexpr" am: 991667ba8c am: 3f6292ea81 am: 2ef0bdafc1
am: 860f72fff5

Change-Id: I89f0b62dac94e9617214ff40709e3b6c5954eddd

Merge "Fix rule of three warnings" am: dfd195160a am: 52b3fe2fcc am: 71b953d0a3
am: 90478a1e9e

Change-Id: If7314d5be257dc6221ef0b9222147db7b2710bb8

Merge "Fix implicit sign casts in Parcel.h" am: 9984fa0ed0 am: 4901e7c070 am: 7600934e17
am: 87109469b0

Change-Id: I4de63e308760a6c079bb07763cd05b3ba9608f5c

Fix logging implicit sign conversions

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I2ce11db524ee9d8f846fdf3ed6fb882d5ef57956

Fix libgui warnings

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: Idba8278bc0a73886ecb7978617854b433c0b5875

Merge "Installd: Add a delete_odex command" am: d6df7e33ad am: a0c21336ab am: ab5550ce1b
am: 3d8094e9aa

Change-Id: I139fc3cb1bae5d81cce380138ab28ab67bfb0614

Merge "Fix weak vtable warnings" am: d93a40c920 am: 11a2bcad06
am: cd77a5a218

Change-Id: Id487011570e506e49a23a793b13e3a173c618a4c

Merge "Rewrite multicharacter constant using constexpr" am: 991667ba8c am: 3f6292ea81
am: 2ef0bdafc1

Change-Id: Ib0fb4e1d2ab3203ba0e21b3a71de12d5176613cb

Merge "Fix rule of three warnings" am: dfd195160a am: 52b3fe2fcc
am: 71b953d0a3

Change-Id: I7a18515d52e3137b3fef304f6c69a6c2a7c64264

Merge "Fix implicit sign casts in Parcel.h" am: 9984fa0ed0 am: 4901e7c070
am: 7600934e17

Change-Id: I40be1101c9bd1cb77571e1c26ac8000e96293232

Merge "Fix weak vtable warnings" am: d93a40c920
am: 11a2bcad06

Change-Id: I1fd134d6eedb2f324de36b8e066b7d942fccd2eb

Merge "Rewrite multicharacter constant using constexpr" am: 991667ba8c
am: 3f6292ea81

Change-Id: I8ccf4b34cb9dc76c91ceac5f928918fa2e0b7ec0

Merge "Fix rule of three warnings" am: dfd195160a
am: 52b3fe2fcc

Change-Id: I9737cdfceb1ba74211efc3fb7fb4d8657578cd25

Merge "Fix weak vtable warnings"
am: d93a40c920

Change-Id: I54d2cad55131ac3a8786012d692d4627f188a0a7

Merge "Rewrite multicharacter constant using constexpr"
am: 991667ba8c

Change-Id: Iecaee1af6b787187f971e57d8ca3d4d223c79627

Merge "Fix rule of three warnings"
am: dfd195160a

Change-Id: Ifcc0ef0c71f465ada5c62cbcccd40580df802a51

Merge "Installd: Add a delete_odex command" am: d6df7e33ad am: a0c21336ab
am: ab5550ce1b

Change-Id: I7e11ae9a4b601a9540a2dfc43aa27a324a732797

Merge "Fix weak vtable warnings"

Fix weak vtable warnings

Move virtual destructors into cpp file so that the compiler knows which
translation unit to put the vtable into.  Hide the warning for
DeathRecipient, which has no virtual methods to move.  The warnings were
being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I25329d66bfc1f6a5064d05ce7d12ad9b090601f8

Merge "Rewrite multicharacter constant using constexpr"

Rewrite multicharacter constant using constexpr

Multi character constants are implementation defined behavior, use a
constexpr instead.  The warnings were being hidden by the use of
-isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I22435d1f66a073c303202da7a1940cb14b0b9fd8

Merge "Fix rule of three warnings"

Fix rule of three warnings

Sensor and BufferItem have destructors, the copy constructor and
assignment operators should not be implicit. The warnings were being
hidden by the use of -isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: If375d85cd3d6ded4ee6b27b841bf10f563254682

HWC2: Kill logspam for non-HWC virtual displays

Removes some logspam and fixes some non-errors which were reported as
errors when running a virtual display that is not backed by hardware
composer.

Test: Cherry-pick from internal branch
Bug: 30022738
Change-Id: Ie966e37d95f53bcc050eef064d24bbe26748b7fd

HWC2: Fix virtual displays

Close the output buffer release fence per the contract in
hwcomposer2.h.

Also changes the HWC2 device to hold a weak pointer to the display,
which allows it to properly remove it when it gets torn down.

Test: Cherry-pick from internal branch
Bug: 29945684
Change-Id: Ib1b5d81e42a63d0e65b189f7c6c62e661e0ab6ae

HWC2: Don't set null handle for client layers

Removes the code that explicitly set a null buffer handle for client
layers.

Test: Cherry-pick from internal branch
Bug: 29463310
Change-Id: I570e6041ba72d64971bb6c364e8e7b6f84eeff08

HWC2: Hook up to BoardConfig variable

Uses TARGET_USES_HWC2 to enable the HWC2 code path in SurfaceFlinger

Test: Cherry-pick from internal branch
Bug: 27211042
Change-Id: I7b7e3e3f7a918e74df0be27418196c81b4332564

HWC2: Skip validate/present if display is off

Adds checks to ensure that the display is still on before attempting
to call into prepareFrame/commit (which in turn call into
validate/present).

Test: Cherry-pick from internal branch
Bug: 28596387
Change-Id: Iad87f02c3e44e86db43a15c258ee66b0cde14faa

Merge "Introduce new command callback to shell."

Introduce new command callback to shell.

This introduces a new feature of the IBinder command protocol
to allow the shell command implementation to call back into
its caller to ask it to open files in the calling context.  This
is needed so that commands that have arguments specifying files
can open those files as the calling shell, not the system (or
whatever) process.

Test: Manual

Change-Id: Idd5b49ea21057864cc9cef816b3e4afbf01948fc

Merge "Fix implicit sign casts in Parcel.h" am: 9984fa0ed0
am: 4901e7c070

Change-Id: Ic291dbf2eb8d6244b883c696756183ace21cfa46

Merge "Fix implicit sign casts in Parcel.h"
am: 9984fa0ed0

Change-Id: I7a5fb0652ccfd88094ceb2e48a8ff2291166f7f7

Added initial tests for dumpstate.

BUG: 31807540
Test: mmm -j32 frameworks/native/cmds/dumpstate/ && adb push ${ANDROID_PRODUCT_OUT}/data/nativetest/dumpstate_test* /data/nativetest && adb shell /data/nativetest/dumpstate_test/dumpstate_test

Change-Id: If5497784052b8d13d7c856f9400dbcd8c2015d05

Merge "Fix implicit sign casts in Parcel.h"

Fix implicit sign casts in Parcel.h

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I20d9a5712c77894f9048ef78264d7dc9f59208ea

Merge "Installd: Add a delete_odex command" am: d6df7e33ad
am: a0c21336ab

Change-Id: Ia246906318cdee0c4653c75e79ecf188bb572ca2

Merge "Installd: Add a delete_odex command"
am: d6df7e33ad

Change-Id: Ife9a2c8c7ee8339dc4e416a397519590fca2cdf6

Deprecate run_command() and dump_file().

This change will break dumpstate_board() implementations that were not
refactored to use the equivalent functions in the Dumpstate
class. For example:

void dumpstate_board() {
  dump_file("INTERRUPTS", "/proc/interrupts");
  run_command("SUBSYSTEM TOMBSTONES", 5, SU_PATH, "root",
              "ls", "-l", "/data/tombstones/ramdump", NULL);

}

Should be refactored to:

void dumpstate_board(){
  Dumpstate& ds = Dumpstate::GetInstance();

  ds.DumpFile("INTERRUPTS", "/proc/interrupts");
  ds.RunCommand("SUBSYSTEM TOMBSTONES",
                {"ls", "-l", "/data/tombstones/ramdump"},
                CommandOptions::AS_ROOT_5);
}

BUG: 26379932
Test: manual / refactored code

Change-Id: Ia74515cc57abc18bc6966a5aed71dd679422fd0e

Merge "Installd: Add a delete_odex command"

Merge "Fix libinput warnings" am: a4767442e3 am: a149886878 am: 35d869c15e
am: fef00db3e5

Change-Id: I558fa030c995220658d79c96fd8ee451363891f5

Merge "Fix libinput warnings" am: a4767442e3 am: a149886878
am: 35d869c15e

Change-Id: I6461ef056bf41d66d391f52cfb25a680dd61b14e

Merge "Fix libinput warnings" am: a4767442e3
am: a149886878

Change-Id: Ie7aaee558da27d52cc20574dfea4eb61f68c2901

Merge "Fix libinput warnings"
am: a4767442e3

Change-Id: I2bf4442e4785a3a15331d7c0482925326a9f38c4

Merge "Fix google-explicit-constructor warnings in  HardwareAPI." am: c8de4a06ab am: 9285335221 am: 64e47fabae
am: 14664fb503

Change-Id: Ifa5345bfaef9b1fc03491b7dc249ecd8ce191247

Merge "Fix google-explicit-constructor warnings in gui and ui." am: 170454b0fa am: 5a283bbd40 am: e7f4cb46c4
am: d2c6b7fd1e

Change-Id: I557d2a036bb1200349157c0cf872b10dc28f0164

Merge "Fix google-explicit-constructor warnings in  opengl." am: 9ccba3c090 am: 512fb374d2 am: ba50990a04
am: e021dd660e

Change-Id: I1e269ee04a286818b0336461efcdd18130d1ec56

EGL: check that display is still valid am: d6e9946cdd am: 3169437feb
am: 4bf4908aab

Change-Id: I80edf530a75ac48fa13d3138a631d2ecb6e7abd1

Merge "Fix libinput warnings"

Fix libinput warnings

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I13f03892c75ea3c3139ca676666b0d403871d241

Merge "Fix google-explicit-constructor warnings in  HardwareAPI." am: c8de4a06ab am: 9285335221
am: 64e47fabae

Change-Id: Ia27b9704d92149284e010a85b95e3bb8fbae6e51

Merge "Fix google-explicit-constructor warnings in gui and ui." am: 170454b0fa am: 5a283bbd40
am: e7f4cb46c4

Change-Id: I46d36b040e143dd5eedae7597ec7875ab920ac7b