Chromium Code Reviews
DescriptionWorkaround for flaky ax_enums.h errors when compiling under app_list_presenter
ui/views/view.h #includes ax_enums.h and app_list_presenter #includes
view.h. This means ax_enums.h needs to exist before anything in
app_list_presenter is compiled, even though app_list_presenter doesn't
use anything from ax_enums.h.
An alternative might be to make views.gyp:views a hard_dependency of
everything that depends on it. However, it seems unfair to force that,
since it's really only ax_gen that is the "hard" dependency, and it's
already correctly marked as a hard_dependency.
Exploring a nicer fix in http://crbug.com/612382, which involves view.h
simply not #including ax_enums.h
BUG=611898
Committed: https://crrev.com/93fa270c60338957b15cefb20e5553fb67cb2ba3
Cr-Commit-Position: refs/heads/master@{#394074}
Patch Set 1 #Patch Set 2 : more dots #
Messages
Total messages: 10 (5 generated)
|
||||||||||||||||||||||||||||