1468590 - Preemptively fix unified bustage in layout/base

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Daniel Holbert [:dholbert]]

This bug is one in a series of "fix latent missing-#include/declaration/namespace compile errors which are currently being hidden by unified builds" bugs.  (This is worth doing periodically, because otherwise these issues are just landmines that get discovered when we add new .cpp files or make another change that reshuffles the unification.)

(I'm using this as an opportunity to try the phabricator code-review workflow, too.)

This bug covers bustage in layout/base when you disable unified builds there (e.g. removing UNIFIED_ from the moz.build file in that directory). This bustage currently consists of the following four issues on my machine:

../../../mozilla/layout/base/AccessibleCaretEventHub.cpp:554:32: error: use of undeclared identifier 'MouseEventBinding'; did you mean 'dom::MouseEventBinding'?
  mManager->SetLastInputSource(MouseEventBinding::MOZ_SOURCE_TOUCH);
                               ^~~~~~~~~~~~~~~~~
                               dom::MouseEventBinding


../../../mozilla/layout/base/PositionedEventTargeting.cpp:580:9: error: use of undeclared identifier 'MouseEventBinding'; did you mean 'dom::MouseEventBinding'?
        MouseEventBinding::MOZ_SOURCE_TOUCH) {
        ^~~~~~~~~~~~~~~~~
        dom::MouseEventBinding


0:19.55 In file included from ../../../mozilla/layout/base/ShapeUtils.cpp:7:
../../dist/include/mozilla/ShapeUtils.h:91:10: error: no template named 'nsTArray'; did you mean 'IsArray'?
  static nsTArray<nsPoint> ComputePolygonVertices(
         ^~~~~~~~
         IsArray


../../../mozilla/layout/base/nsLayoutUtils.cpp:9199:12: error: cannot initialize a variable of type 'mozilla::dom::Element *' with an rvalue of type 'mozilla::dom::HTMLBodyElement *'
  Element* bodyElement = document ? document->GetBodyElement() : nullptr;
           ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Comment 1

[Phabricator Automation]

Attachment: Bug 1468590: Preemptively fix unified build bustage in layout/base.

This commit just adds some #include directives and "using namespace" diretives.

(The "using namespace mozilla::dom;" are the important namespace directives
here, though I've included "using namespace mozilla;" alongside it as well, for
completeness/consistency.)

Comment 2

[Phabricator Automation]

Comment on attachment 8985223 [details]
Bug 1468590: Preemptively fix unified build bustage in layout/base.

Emilio Cobos Álvarez [:emilio] has approved the revision.

https://phabricator.services.mozilla.com/D1647

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c5507887c5f6
Preemptively fix unified build bustage in layout/base. r=emilio

Comment 4

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/c5507887c5f6