869517 - Fix JSFlatString RootKind compilation error when building with --enable-root-analysis

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Chris Peterson [:cpeterson]]

Attachment: define-JSFlatString-RootKind.patch

When I build with --enable-root-analysis on OSX, I get the following compilation error because RootKind<JSFlatString *> is undefined:


../../dist/include/js/RootingAPI.h:382:46: error: incomplete type 'JSFlatString' named in nested name specifier
  static ThingRootKind rootKind() { return T::rootKind(); }
                                           ^~~
../../dist/include/js/RootingAPI.h:389:57: note: in instantiation of member function 'js::RootKind<JSFlatString *>::rootKind' requested here
  static ThingRootKind kind() { return RootKind<T *>::rootKind(); }
                                                      ^
../../dist/include/js/RootingAPI.h:490:54: note: in instantiation of member function 'js::RootMethods<JSFlatString *>::kind' requested here
      js::ThingRootKind kind = js::RootMethods<T>::kind();
                                                   ^
../../dist/include/js/RootingAPI.h:411:9: note: in instantiation of member function 'JS::Rooted<JSFlatString *>::commonInit' requested here
      commonInit(cx->thingGCRooters);
      ^
../../dist/include/js/RootingAPI.h:428:9: note: in instantiation of member function 'JS::Rooted<JSFlatString *>::init' requested here
      init(cx);
      ^
/Users/chris/Code/mozilla/central/dom/base/nsJSTimeoutHandler.cpp:188:29: note: in instantiation of member function 'JS::Rooted<JSFlatString *>::Rooted' requested here
JS::Rooted<JSFlatString*> expr(cx);
                          ^
../../dist/include/jspubtd.h:186:45: note: forward declaration of 'JSFlatString'
class                                       JSFlatString;
                                          ^
1 error generated.

Comment 1

[Chris Peterson [:cpeterson]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/51b1dc857800

Comment 2

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/51b1dc857800