handoff: Fix the origin so that it correctly reflects the sender.

components/handoff/handoff_manager.mm

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/handoff/handoff_manager.h"
 
 #include "base/logging.h"
 #include "base/mac/scoped_nsobject.h"
-#include "components/handoff/handoff_utility.h"
 #include "net/base/mac/url_conversions.h"
 
 #if defined(OS_IOS)
 #include "base/ios/ios_util.h"
 #endif
 
 #if defined(OS_MACOSX) && !defined(OS_IOS)
 #include "base/mac/mac_util.h"
 #include "base/mac/sdk_forward_declarations.h"
 #endif
 
 @interface HandoffManager ()
 
 // The active user activity.
 @property(nonatomic, retain) NSUserActivity* userActivity;
 
 // Whether the URL of the current tab should be exposed for Handoff.
 - (BOOL)shouldUseActiveURL;
 
 // Updates the active NSUserActivity.
 - (void)updateUserActivity;
 
 @end
 
 @implementation HandoffManager
 
 @synthesize userActivity = _userActivity;
 
 - (instancetype)init {
+  NOTREACHED();
+  return nil;
+}
+
+- (instancetype)initWithOrigin:(handoff::Origin)origin {
+  DCHECK(origin == handoff::ORIGIN_IOS || origin == handoff::ORIGIN_MAC);
   self = [super init];

[Avi (use Gerrit), 2015/03/26 18:14:04]

Can we #ifdef so that if this is built for iOS we automatically get ORIGIN_IOS
and if built for Mac we get ORIGIN_MAC?

[erikchen, 2015/03/26 18:22:36]

Good suggestion, done.

   if (self) {
     _propertyReleaser_HandoffManager.Init(self, [HandoffManager class]);
+    _origin = origin;
   }
   return self;
 }
 
 - (void)updateActiveURL:(const GURL&)url {
 #if defined(OS_IOS)
   // Handoff is only available on iOS 8+.
   DCHECK(base::ios::IsRunningOnIOS8OrLater());
 #endif
 
@@ skipping 25 matching lines @@
 
   // Invalidate the old user activity and make a new one.
   [self.userActivity invalidate];
 
   Class aClass = NSClassFromString(@"NSUserActivity");
   NSUserActivity* userActivity = [[aClass performSelector:@selector(alloc)]
       performSelector:@selector(initWithActivityType:)
            withObject:handoff::kChromeHandoffActivityType];
   self.userActivity = base::scoped_nsobject<NSUserActivity>(userActivity);
   self.userActivity.webpageURL = net::NSURLWithGURL(_activeURL);
-  self.userActivity.userInfo = @{ handoff::kOriginKey : handoff::kOriginiOS };
+  NSString* origin = handoff::StringFromOrigin(_origin);
+  DCHECK(origin);
+  self.userActivity.userInfo = @{ handoff::kOriginKey : origin };
   [self.userActivity becomeCurrent];
 }
 
 @end