Allow passing args to UWP app from command line

support/hololens/ServoApp/App.cpp

@@ -32,9 +32,8 @@ App::App() {
 }
 
 void App::createRootFrame(
-    bool prelaunchActivated,
+    Frame &rootFrame, bool prelaunchActivated,
     winrt::Windows::Foundation::IInspectable const &args) {
-  Frame rootFrame{nullptr};
   auto content = Window::Current().Content();
   if (content) {
     rootFrame = content.try_as<Frame>();
@@ -63,13 +62,22 @@ void App::createRootFrame(
 }
 
 void App::OnLaunched(LaunchActivatedEventArgs const &e) {
-  this->createRootFrame(e.PrelaunchActivated(), box_value(e.Arguments()));
+  Frame rootFrame{nullptr};
+  this->createRootFrame(rootFrame, e.PrelaunchActivated(),
+                        box_value(e.Arguments()));
 }
 
 void App::OnActivated(IActivatedEventArgs const &args) {
   if (args.Kind() == Windows::ApplicationModel::Activation::ActivationKind::
                          CommandLineLaunch) {
-    return this->createRootFrame(false, nullptr);
+    auto cmdLineArgs{args.as<Windows::ApplicationModel::Activation::
+                                 CommandLineActivatedEventArgs>()};
+    auto cmdLineStr = cmdLineArgs.Operation().Arguments();
+    Frame rootFrame{nullptr};
+    this->createRootFrame(rootFrame, false, nullptr);
+    auto page = rootFrame.Content().try_as<BrowserPage>();
+    page->SetArgs(cmdLineStr);
+    return;
   }
 
   if (args.Kind() ==
@@ -82,10 +90,7 @@ void App::OnActivated(IActivatedEventArgs const &args) {
     auto content = Window::Current().Content();
     bool isRunning = content != nullptr;
     if (!isRunning) {
-      rootFrame = Frame();
+      this->createRootFrame(rootFrame, false, nullptr);
-      rootFrame.Navigate(xaml_typename<ServoApp::BrowserPage>());
-      Window::Current().Content(rootFrame);
-      Window::Current().Activate();
     } else {
       rootFrame = content.try_as<Frame>();
     }

support/hololens/ServoApp/App.h

@@ -9,12 +9,12 @@ namespace winrt::ServoApp::implementation {
 struct App : AppT<App> {
   App();
 
-  void createRootFrame(bool prelaunchActivated,
+  void createRootFrame(winrt::Windows::UI::Xaml::Controls::Frame &, bool,
-                       winrt::Windows::Foundation::IInspectable const &args);
+                       winrt::Windows::Foundation::IInspectable const &);
   void OnLaunched(
       Windows::ApplicationModel::Activation::LaunchActivatedEventArgs const &);
   void App::OnActivated(
-      Windows::ApplicationModel::Activation::IActivatedEventArgs const &args);
+      Windows::ApplicationModel::Activation::IActivatedEventArgs const &);
   void OnSuspending(IInspectable const &,
                     Windows::ApplicationModel::SuspendingEventArgs const &);
   void OnNavigationFailed(

support/hololens/ServoApp/BrowserPage.cpp

@@ -67,6 +67,8 @@ void BrowserPage::SetTransientMode(bool transient) {
                                        : Visibility::Visible);
 }
 
+void BrowserPage::SetArgs(hstring args) { servoControl().SetArgs(args); }
+
 void BrowserPage::Shutdown() { servoControl().Shutdown(); }
 
 /**** USER INTERACTIONS WITH UI ****/

support/hololens/ServoApp/BrowserPage.h

@@ -29,6 +29,7 @@ public:
   void Shutdown();
   void LoadServoURI(Windows::Foundation::Uri uri);
   void SetTransientMode(bool);
+  void SetArgs(hstring);
 
 private:
   void BindServoEvents();

support/hololens/ServoApp/ServoControl/Servo.cpp

@@ -56,12 +56,12 @@ const char* get_clipboard_contents() {
   return nullptr;
 }
 
-Servo::Servo(hstring url, GLsizei width, GLsizei height, float dpi,
+Servo::Servo(hstring url, hstring args, GLsizei width, GLsizei height, float dpi,
              ServoDelegate &aDelegate)
     : mWindowHeight(height), mWindowWidth(width), mDelegate(aDelegate) {
 
   capi::CInitOptions o;
-  o.args = "--pref dom.webxr.enabled";
+  o.args = *hstring2char(args);
   o.url = *hstring2char(url);
   o.width = mWindowWidth;
   o.height = mWindowHeight;

support/hololens/ServoApp/ServoControl/Servo.h

@@ -43,7 +43,7 @@ protected:
 
 class Servo {
 public:
-  Servo(hstring, GLsizei, GLsizei, float, ServoDelegate &);
+  Servo(hstring, hstring, GLsizei, GLsizei, float, ServoDelegate &);
   ~Servo();
   ServoDelegate &Delegate() { return mDelegate; }
 

support/hololens/ServoApp/ServoControl/ServoControl.cpp

@@ -52,8 +52,7 @@ void ServoControl::OnLoaded(IInspectable const &, RoutedEventArgs const &) {
       });
   panel.ManipulationDelta(
       std::bind(&ServoControl::OnSurfaceManipulationDelta, this, _1, _2));
-  Panel().SizeChanged(
+  Panel().SizeChanged(std::bind(&ServoControl::OnSurfaceResized, this, _1, _2));
-      std::bind(&ServoControl::OnSurfaceResized, this, _1, _2));
   InitializeConditionVariable(&mGLCondVar);
   InitializeCriticalSection(&mGLLock);
   CreateRenderSurface();
@@ -166,7 +165,8 @@ void ServoControl::Loop() {
   if (mServo == nullptr) {
     log("Entering loop");
     ServoDelegate *sd = static_cast<ServoDelegate *>(this);
-    mServo = std::make_unique<Servo>(mInitialURL, panelWidth, panelHeight, mDPI, *sd);
+    mServo = std::make_unique<Servo>(mInitialURL, mArgs, panelWidth, panelHeight, mDPI,
+                                     *sd);
   } else {
     // FIXME: this will fail since create_task didn't pick the thread
     // where Servo was running initially.
@@ -273,9 +273,7 @@ void ServoControl::WakeUp() {
 
 bool ServoControl::OnServoAllowNavigation(hstring uri) {
   if (mTransient) {
-    RunOnUIThread([=] {
+    RunOnUIThread([=] { Launcher::LaunchUriAsync(Uri{uri}); });
-      Launcher::LaunchUriAsync(Uri{uri});
-    });
   }
   return !mTransient;
 }
@@ -288,7 +286,8 @@ void ServoControl::OnServoAnimatingChanged(bool animating) {
 }
 
 void ServoControl::OnServoIMEStateChanged(bool aShow) {
-  // FIXME: https://docs.microsoft.com/en-us/windows/win32/winauto/uiauto-implementingtextandtextrange
+  // FIXME:
+  // https://docs.microsoft.com/en-us/windows/win32/winauto/uiauto-implementingtextandtextrange
 }
 
 template <typename Callable> void ServoControl::RunOnUIThread(Callable cb) {

support/hololens/ServoApp/ServoControl/ServoControl.h

@@ -72,6 +72,8 @@ struct ServoControl : ServoControlT<ServoControl>, public servo::ServoDelegate {
 
   void SetTransientMode(bool transient) { mTransient = transient; }
 
+  void SetArgs(hstring args) { mArgs = args; }
+
   virtual void WakeUp();
   virtual void OnServoLoadStarted();
   virtual void OnServoLoadEnded();
@@ -139,6 +141,7 @@ private:
   CRITICAL_SECTION mGLLock;
   CONDITION_VARIABLE mGLCondVar;
   std::unique_ptr<Concurrency::task<void>> mLoopTask;
+  hstring mArgs;
 };
 } // namespace winrt::ServoApp::implementation
 

support/hololens/ServoApp/ServoControl/ServoControl.idl

@@ -11,6 +11,7 @@ namespace ServoApp {
       void Stop();
       Windows.Foundation.Uri LoadURIOrSearch(String url);
       void SetTransientMode(Boolean transient);
+      void SetArgs(String args);
       void Shutdown();
       event EventDelegate OnLoadStarted;
       event EventDelegate OnLoadEnded;