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

Issue 603125 link

Starred by 0 users

Issue metadata

Status: Duplicate
Merged: issue 600384
Owner:
Last visit > 30 days ago
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

mojom code generator not running when building blink_platform_unittests

Project Member Reported by drott@chromium.org, Apr 13 2016

Issue description

Version: ToT 04/13/2016
OS: OS X

What steps will reproduce the problem?
(1) Generate a new gn folder, say gn args out/gnasan 
(2) Configure for asan build (but I don't think this matters), i.e.

is_asan = true
enable_nacl = false  # Necessary until NaCl GN build is more complete.
is_debug = false  # Release build.

(3) Run ninja -C out/gnasan blink_platform_unittests

What is the expected output?

Builds and links successfully.

What do you see instead?

In file included from ../../third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTService.cpp:12:
In file included from ../../third_party/WebKit/Source/modules/bluetooth/BluetoothError.h:9:
../../third_party/WebKit/public/platform/modules/bluetooth/WebBluetoothError.h:8:10: fatal error: 'public/platform/modules/bluetooth/web_bluetooth.mojom.h' file not found
#include "public/platform/modules/bluetooth/web_bluetooth.mojom.h"
         ^

When I add "chrome" as a build target, everything completes and the header is generated from the mojom file. I am guessing there is a missing dependency to the mojo code generation somewhere?

 

Comment 1 by drott@chromium.org, Apr 13 2016

Labels: -OS-Mac
Sorry, the OS was Linux.

Comment 2 by drott@chromium.org, Apr 13 2016

Labels: OS-Linux

Comment 3 by roc...@chromium.org, Apr 13 2016

Owner: ortuno@chromium.org
Status: Assigned (was: Unconfirmed)
ortuno@ could you look into this when you have time? Does blink_platform_unittests not depend on blink_headers?

Comment 4 by ortuno@chromium.org, Apr 13 2016

Mergedinto: 600384
Status: Duplicate (was: Assigned)
It does. But blink_headers doesn't generate bindings yet.

Sign in to add a comment