New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 793193 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Dec 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Complie error in components/exo/wayland/server.cc in m64

Project Member Reported by warx@chromium.org, Dec 8 2017

Issue description

../../components/exo/wayland/server.cc:2490:29: error: excess elements in struct initializer
    aura_surface_set_frame, aura_surface_set_parent};
                            ^~~~~~~~~~~~~~~~~~~~~~~
../../components/exo/wayland/server.cc:2513:20: error: use of undeclared identifier 'ZAURA_OUTPUT_SCALE_PROPERTY_PREFERRED'
          flags |= ZAURA_OUTPUT_SCALE_PROPERTY_PREFERRED;
                   ^
../../components/exo/wayland/server.cc:2515:20: error: use of undeclared identifier 'ZAURA_OUTPUT_SCALE_PROPERTY_CURRENT'
          flags |= ZAURA_OUTPUT_SCALE_PROPERTY_CURRENT;
                   ^
../../components/exo/wayland/server.cc:2517:9: error: use of undeclared identifier 'zaura_output_send_scale'
        zaura_output_send_scale(resource_, flags, mode.ui_scale() * 1000);
        ^
../../components/exo/wayland/server.cc:2521:31: error: use of undeclared identifier 'ZAURA_OUTPUT_SCALE_PROPERTY_CURRENT'
                              ZAURA_OUTPUT_SCALE_PROPERTY_CURRENT |
                              ^
../../components/exo/wayland/server.cc:2522:35: error: use of undeclared identifier 'ZAURA_OUTPUT_SCALE_PROPERTY_PREFERRED'
                                  ZAURA_OUTPUT_SCALE_PROPERTY_PREFERRED,
                                  ^
../../components/exo/wayland/server.cc:2523:31: error: use of undeclared identifier 'ZAURA_OUTPUT_SCALE_FACTOR_1000'
                              ZAURA_OUTPUT_SCALE_FACTOR_1000);
                              ^
../../components/exo/wayland/server.cc:2564:19: error: use of undeclared identifier 'ZAURA_SHELL_ERROR_AURA_OUTPUT_EXISTS'; did you mean 'ZAURA_SHELL_ERROR_AURA_SURFACE_EXISTS'?
        resource, ZAURA_SHELL_ERROR_AURA_OUTPUT_EXISTS,
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                  ZAURA_SHELL_ERROR_AURA_SURFACE_EXISTS
../../components/exo/wayland/aura-shell-server-protocol.h:94:2: note: 'ZAURA_SHELL_ERROR_AURA_SURFACE_EXISTS' declared here
        ZAURA_SHELL_ERROR_AURA_SURFACE_EXISTS = 0,
        ^
../../components/exo/wayland/server.cc:2570:16: error: use of undeclared identifier 'zaura_output_interface'; did you mean 'wl_output_interface'?
      client, &zaura_output_interface, wl_resource_get_version(resource), id);
               ^~~~~~~~~~~~~~~~~~~~~~
               wl_output_interface
../../third_party/wayland/include/protocol/wayland-server-protocol.h:633:34: note: 'wl_output_interface' declared here
extern const struct wl_interface wl_output_interface;
                                 ^
../../components/exo/wayland/server.cc:2579:34: error: excess elements in struct initializer
    aura_shell_get_aura_surface, aura_shell_get_aura_output};
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~
10 errors generated.
ninja: build stopped: subcommand failed.

cl here: https://chromium-review.googlesource.com/c/chromium/src/+/816514
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 8 2017

Labels: merge-merged-3282
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/843d616e23985127e65eb603c79bb037e661d2cd

commit 843d616e23985127e65eb603c79bb037e661d2cd
Author: Qiang(Joe) Xu <warx@chromium.org>
Date: Fri Dec 08 05:08:59 2017

Revert "exo: Implement aura shell version 2."

This reverts commit 92041623687a1f43333508d1ffc84ce8bd97bf2d.

Reason for revert: causing compile error in m64 builders,
see  crbug.com/793193 

Original change's description:
> exo: Implement aura shell version 2.
> 
> Version 2 provides clients with an interface to keep track
> of scale factors used by the shell. It provides a full list
> of supported scale factors and flags to indicate what the
> current and preferred scale factor is.
> 
> Version 2 also adds a new request for setting the parent
> of a top level shell surface and specifying an initial
> location relative to parent.
> 
> This also adds a new 'info' client that can be used for testing. The
> client connects to a compositor and prints all output related
> properties. Example output:
> 
> geometry:
>    x:                 0
>    y:                 0
>    physical width:    423 mm
>    physical height:   317 mm
>    subpixel:          unknown
>    make:              unknown
>    model:             unknown
>    transform:         normal
> 
> modes:
>    1600x1200:         60.00 Hz      current       preferred
> 
> device scale factor:  1
> 
> scales:
>    0.500:
>    0.800:
>    1.000:             preferred
>    1.200:
>    1.600:             current
> 
> TBR=reveman@chromium.org
> 
> (cherry picked from commit 32715093d78f468c77dc13324914ecbcb56f1219)
> 
> Test: exo_unittests --gtest_filter=ShellSurfaceTest.SetParent
> Test: wayland_info_client
> Bug:  791672 ,  791681 
> Change-Id: Iab6869f5b54a6b5970dea40025edea46d0113c2c
> Reviewed-on: https://chromium-review.googlesource.com/805358
> Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
> Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
> Cr-Original-Commit-Position: refs/heads/master@{#521752}
> Reviewed-on: https://chromium-review.googlesource.com/816514
> Reviewed-by: David Reveman <reveman@chromium.org>
> Cr-Commit-Position: refs/branch-heads/3282@{#90}
> Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}

TBR=reveman@chromium.org,dcastagna@chromium.org

Change-Id: I8724288a5837f2c994585170b2831a05520722e8
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  791672 ,  791681 
Bug:  793193 
Reviewed-on: https://chromium-review.googlesource.com/816614
Reviewed-by: Qiang(Joe) Xu <warx@chromium.org>
Cr-Commit-Position: refs/branch-heads/3282@{#92}
Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/shell_surface.cc
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/shell_surface.h
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/shell_surface_unittest.cc
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/sub_surface.h
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/surface.cc
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/surface.h
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/surface_delegate.h
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/surface_tree_host.h
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/wayland/BUILD.gn
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/wayland/clients/client_helper.cc
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/wayland/clients/client_helper.h
[delete] https://crrev.com/298fb263f122250540bcb69dcc00531f2e95bcba/components/exo/wayland/clients/info.cc
[modify] https://crrev.com/843d616e23985127e65eb603c79bb037e661d2cd/components/exo/wayland/server.cc

Comment 2 by warx@chromium.org, Dec 8 2017

Status: Fixed (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4

commit c9d9d4bd244c0e166781f9224fabbdf6b5c390c4
Author: David Reveman <reveman@chromium.org>
Date: Tue Dec 12 03:02:38 2017

Revert "Revert "exo: Implement aura shell version 2.""

This reverts commit 843d616e23985127e65eb603c79bb037e661d2cd.

Reason for revert: Dependency CL (https://chromium-review.googlesource.com/c/chromium/src/+/821371) has now been merge to branch.

Original change's description:
> Revert "exo: Implement aura shell version 2."
> 
> This reverts commit 92041623687a1f43333508d1ffc84ce8bd97bf2d.
> 
> Reason for revert: causing compile error in m64 builders,
> see  crbug.com/793193 
> 
> Original change's description:
> > exo: Implement aura shell version 2.
> > 
> > Version 2 provides clients with an interface to keep track
> > of scale factors used by the shell. It provides a full list
> > of supported scale factors and flags to indicate what the
> > current and preferred scale factor is.
> > 
> > Version 2 also adds a new request for setting the parent
> > of a top level shell surface and specifying an initial
> > location relative to parent.
> > 
> > This also adds a new 'info' client that can be used for testing. The
> > client connects to a compositor and prints all output related
> > properties. Example output:
> > 
> > geometry:
> >    x:                 0
> >    y:                 0
> >    physical width:    423 mm
> >    physical height:   317 mm
> >    subpixel:          unknown
> >    make:              unknown
> >    model:             unknown
> >    transform:         normal
> > 
> > modes:
> >    1600x1200:         60.00 Hz      current       preferred
> > 
> > device scale factor:  1
> > 
> > scales:
> >    0.500:
> >    0.800:
> >    1.000:             preferred
> >    1.200:
> >    1.600:             current
> > 
> > TBR=reveman@chromium.org
> > 
> > (cherry picked from commit 32715093d78f468c77dc13324914ecbcb56f1219)
> > 
> > Test: exo_unittests --gtest_filter=ShellSurfaceTest.SetParent
> > Test: wayland_info_client
> > Bug:  791672 ,  791681 
> > Change-Id: Iab6869f5b54a6b5970dea40025edea46d0113c2c
> > Reviewed-on: https://chromium-review.googlesource.com/805358
> > Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
> > Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
> > Cr-Original-Commit-Position: refs/heads/master@{#521752}
> > Reviewed-on: https://chromium-review.googlesource.com/816514
> > Reviewed-by: David Reveman <reveman@chromium.org>
> > Cr-Commit-Position: refs/branch-heads/3282@{#90}
> > Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}
> 
> TBR=reveman@chromium.org,dcastagna@chromium.org
> 
> Change-Id: I8724288a5837f2c994585170b2831a05520722e8
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug:  791672 ,  791681 
> Bug:  793193 
> Reviewed-on: https://chromium-review.googlesource.com/816614
> Reviewed-by: Qiang(Joe) Xu <warx@chromium.org>
> Cr-Commit-Position: refs/branch-heads/3282@{#92}
> Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}

TBR=reveman@chromium.org,dcastagna@chromium.org,warx@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  791672 ,  791681 ,  793193 
Change-Id: Ie0a0b4db78693e0ecaef8eee9a8b21806a38fae5
Reviewed-on: https://chromium-review.googlesource.com/821571
Reviewed-by: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/branch-heads/3282@{#166}
Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/shell_surface.cc
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/shell_surface.h
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/shell_surface_unittest.cc
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/sub_surface.h
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/surface.cc
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/surface.h
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/surface_delegate.h
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/surface_tree_host.h
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/wayland/BUILD.gn
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/wayland/clients/client_helper.cc
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/wayland/clients/client_helper.h
[add] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/wayland/clients/info.cc
[modify] https://crrev.com/c9d9d4bd244c0e166781f9224fabbdf6b5c390c4/components/exo/wayland/server.cc

Sign in to add a comment