Attention is currently required from: Edward O'Callaghan, Angel Pons, Nikolai Artemiev.

Patch set 6:Code-Review +1

View Change

2 comments:

• Commit Message:

  • Patch Set #6, Line 16: TEST=builds

    Have you done any testing with layouts that have multiple regions?

• Patchset:

  • Patch Set #2:

    That's a good question since this patch on its own doesn't have any extra context. […]

    I agree commit message needs more info on why the patch exist, one more paragraph would be great, something around "The patch allows for ABC to be done / XYZ to be improved / OPQ to be fixed later in the chain".
    As a reminder for Nikolai, b:260440773 is just a magic number, doesn't give any info.
    Thanks!

To view, visit change 69196. To unsubscribe, or for help writing mail filters, visit settings.