Closed Bug 1271477 Opened 4 years ago Closed 4 years ago

widget/gtk/WidgetStyleCache.cpp:191:29: error: 'PodArrayZero' was not declared in this scope (--disable-dbus)

Categories

(Core :: Widget: Gtk, defect)

Unspecified
FreeBSD
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: jbeich, Unassigned)

References

Details

(Keywords: regression)

Attachments

(2 files)

In file included from widget/gtk/Unified_cpp_widget_gtk0.cpp:29:
widget/gtk/WidgetStyleCache.cpp:191:3: error: use of undeclared identifier
      'PodArrayZero'; did you mean 'mozilla::PodArrayZero'?
  PodArrayZero(sStyleStorage);
  ^~~~~~~~~~~~
  mozilla::PodArrayZero
dist/include/mozilla/PodOperations.h:65:1: note: 'mozilla::PodArrayZero'
      declared here
PodArrayZero(T (&aT)[N])
^
In file included from widget/gtk/Unified_cpp_widget_gtk0.cpp:29:
widget/gtk/WidgetStyleCache.cpp:198:3: error: use of undeclared identifier
      'PodArrayZero'; did you mean 'mozilla::PodArrayZero'?
  PodArrayZero(sWidgetStorage);
  ^~~~~~~~~~~~
  mozilla::PodArrayZero
dist/include/mozilla/PodOperations.h:65:1: note: 'mozilla::PodArrayZero'
      declared here
PodArrayZero(T (&aT)[N])
^
2 errors generated.
In file included from widget/gtk/Unified_cpp_widget_gtk1.cpp:128:
In file included from widget/gtk/nsWidgetFactory.cpp:34:
widget/gtk/WakeLockListener.h:17:10: fatal error: 'mozilla/ipc/DBusConnectionRefPtr.h'
      file not found
#include "mozilla/ipc/DBusConnectionRefPtr.h"
         ^
1 error
Blocks: 1266054
Attached patch comment 0 fixSplinter Review
Apologies for mixing two separate issues but they're a bit hard to untangle due to namespace pollution caused by unified build.
Attachment #8750562 - Flags: review?(karlt)
Attached patch comment 2 fixSplinter Review
Attachment #8750563 - Flags: review?(karlt)
Comment on attachment 8750562 [details] [diff] [review]
comment 0 fix

Thanks!
Attachment #8750562 - Flags: review?(karlt) → review+
Attachment #8750563 - Flags: review?(karlt) → review+
(In reply to Jan Beich from comment #5)
> Created attachment 8750563 [details] [diff] [review]
> comment 2 fix

Thanks for fixing this issue.
Keywords: checkin-needed
Keywords: regression
Comment on attachment 8750562 [details] [diff] [review]
comment 0 fix

Chasing https://hg.mozilla.org/releases/mozilla-aurora/pushloghtml?changeset=cb42beedc07e

Approval Request Comment
[Feature/regressing bug #]: bug 1234158 regression
[User impact if declined]: --disable-dbus build broken
[Describe test coverage new/current, TreeHerder]: m-c
[Risks and why]: Low, can only break build on Linux
[String/UUID change made/needed]: None
Attachment #8750562 - Flags: approval-mozilla-aurora?
Comment on attachment 8750562 [details] [diff] [review]
comment 0 fix

Fix the build with some options, tiny risk, taking it.
Attachment #8750562 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Only comment 0 fix (the approved patch), comment 2 fix dependency (bug 1266054) wasn't backported/nominated for Aurora48 yet.
Flags: needinfo?(jbeich)
https://hg.mozilla.org/releases/mozilla-aurora/rev/94c2c4e54006
I'm setting status48 to 'fixed', but this will make it difficult for us to see if/when the comment2fix patch is ready to be uplifted. If/when you request uplift for it, please try to remember to set status48 back to 'affected' to make it easier to see this in our uplift queries. :)
Flags: needinfo?(jbeich)
jan, so does comment 0 or comment 2 fix (or both need to be uplifted ?)
Flags: needinfo?(jbeich)
Bug 1266054 wasn't backported to FF48, so there was no need to backport comment 2 fix as well.
Flags: needinfo?(jbeich)
You need to log in before you can comment on or make changes to this bug.