New issue
Advanced search Search tips

Issue 842413 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 836071



Sign in to add a comment

blink v8 bindings generator shouldn't produce include lines containing the name of the build dir

Project Member Reported by thakis@chromium.org, May 12 2018

Issue description

Sometimes the bindings generator produces include lines that look like so:

#include "../../../out/Debug/gen/third_party/blink/renderer/core/window_core_constructors.h"

(After https://chromium-review.googlesource.com/c/1054523/ , the v8 bindings generator is the only script with this behavior: https://cs.chromium.org/search/?q=%23include.*out%5C/debug&type=cs)

This is bad since this will usually be different on different machines and so goma's cache works less well.

It looks like just saying

#include "third_party/blink/renderer/core/window_core_constructors.h"

should have the same effect in practice (even though it admittedly looks confusing). Maybe the bindings should be compiled with the build dir root as -I flag and then the line could e.g. be

#include "gen/third_party/blink/renderer/core/window_core_constructors.h"

which makes it more clear that the header is generated.
 

Comment 1 by tkent@chromium.org, May 14 2018

Blocking: 836071

Comment 2 by tkent@chromium.org, May 14 2018

Probably this was caused by the Great Blink mv.

We have code like:

        match = re.search(r'/gen/blink/(.*)$', posixpath.abspath(include_path))
        if match:
            include_path = match.group(1)

and the event changed the abspath result to .../out/Debug/gen/third_party/blink/...

Comment 3 by tkent@chromium.org, May 14 2018

Owner: tkent@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, May 14 2018

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

commit ef846477a1d1d87864b84c2a01de1c798ee72954
Author: Kent Tamura <tkent@chromium.org>
Date: Mon May 14 08:53:24 2018

Qualify #includes in files generated by the IDL compiler except for primary headers.

* normaliza_and_sort_includes():
 - Fix the normalization. The output directory is now
   "gen/third_party/blink/", not "gen/blink/".

 - produce repo-root-relative paths instead of blink-relative paths.

Bug:  830255 ,  842413 
Change-Id: Ia99af5830f3cc67cb25e3e26ff24d0941f38effa
Reviewed-on: https://chromium-review.googlesource.com/1056967
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558228}
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/scripts/code_generator.py
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/array_buffer_or_array_buffer_view_or_dictionary.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/array_buffer_or_array_buffer_view_or_dictionary.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/boolean_or_element_sequence.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/boolean_or_element_sequence.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/boolean_or_string_or_unrestricted_double.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/boolean_or_string_or_unrestricted_double.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/boolean_or_test_callback_interface.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/boolean_or_test_callback_interface.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/byte_string_or_node_list.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/byte_string_or_node_list.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/byte_string_sequence_sequence_or_byte_string_byte_string_record.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/byte_string_sequence_sequence_or_byte_string_byte_string_record.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_double_or_null_sequence.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_double_or_null_sequence.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_double_sequence.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_double_sequence.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_long_or_boolean_sequence.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_long_or_boolean_sequence.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_string.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_string.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_string_or_double_or_string_sequence.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/double_or_string_or_double_or_string_sequence.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/element_sequence_or_byte_string_double_or_string_record.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/element_sequence_or_byte_string_double_or_string_record.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/float_or_boolean.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/float_or_boolean.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/long_or_boolean.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/long_or_boolean.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/long_or_test_dictionary.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/long_or_test_dictionary.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/long_sequence_or_event.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/long_sequence_or_event.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/nested_union_type.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/nested_union_type.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/node_or_node_list.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/node_or_node_list.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/origin_trial_features_for_core.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/string_or_array_buffer_or_array_buffer_view.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/string_or_array_buffer_or_array_buffer_view.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/string_or_double.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/string_or_double.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/string_or_string_sequence.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/string_or_string_sequence.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_dictionary.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_dictionary.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_dictionary_derived_implemented_as.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_enum_or_double.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_enum_or_double.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_enum_or_test_enum_or_null_sequence.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_enum_or_test_enum_or_null_sequence.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_enum_or_test_enum_sequence.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_enum_or_test_enum_sequence.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_2_or_uint8_array.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_2_or_uint8_array.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_event_init.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_garbage_collected_or_string.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_garbage_collected_or_string.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_or_long.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_or_long.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_or_test_interface_empty.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_interface_or_test_interface_empty.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/test_permissive_dictionary.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/unrestricted_double_or_string.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/unrestricted_double_or_string.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/unsigned_long_long_or_boolean_or_test_callback_interface.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/unsigned_long_long_or_boolean_or_test_callback_interface.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_any_callback_function_optional_any_arg.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_any_callback_function_optional_any_arg.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_array_buffer.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_array_buffer.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_array_buffer_view.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_array_buffer_view.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_data_view.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_data_view.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_long_callback_function.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_long_callback_function.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_string_sequence_callback_function_long_sequence_arg.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_string_sequence_callback_function_long_sequence_arg.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_svg_test_interface.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_svg_test_interface.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_attribute_getters.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_attribute_getters.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_callback_functions.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_callback_functions.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_callback_interface.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_callback_interface.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_constants.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_constants.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_dictionary.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_dictionary.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_dictionary_derived.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_dictionary_derived.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_integer_indexed.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_integer_indexed.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_integer_indexed_global.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_integer_indexed_global.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_integer_indexed_primary_global.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_integer_indexed_primary_global.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_2.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_2.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_3.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_3.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_check_security.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_check_security.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_conditional_secure_context.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_conditional_secure_context.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_constructor.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_constructor.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_constructor_2.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_constructor_2.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_constructor_3.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_constructor_3.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_constructor_4.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_constructor_4.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_custom_constructor.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_custom_constructor.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_document.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_document.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_empty.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_empty.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_event_init.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_event_init.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_event_init_constructor.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_event_init_constructor.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_event_target.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_event_target.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_garbage_collected.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_garbage_collected.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_named_constructor.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_named_constructor.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_named_constructor_2.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_named_constructor_2.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_node.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_node.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_origin_trial_enabled.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_origin_trial_enabled.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_secure_context.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_interface_secure_context.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_legacy_callback_interface.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_legacy_callback_interface.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_node.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_node.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_object.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_object.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_permissive_dictionary.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_permissive_dictionary.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_special_operations.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_special_operations.h
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings/tests/results/core/v8_test_special_operations_not_enumerable.cc
[modify] https://crrev.com/ef846477a1d1d87864b84c2a01de1c798ee72954/third_party/blink/renderer/bindings

Comment 5 by tkent@chromium.org, May 14 2018

Status: Fixed (was: Started)

Comment 6 by thakis@chromium.org, May 15 2018

It's much better now, but there are still a handful left: https://cs.chromium.org/search/?q=%23include.*out%5C/debug&type=cs

Is cs just stale, or is there more to do?

Comment 7 by peria@chromium.org, May 15 2018

I can't find with cs, nor in my local gen/ directory.

Sign in to add a comment