Add a pref name for share targets, and store their manifest data.

chrome/browser/extensions/bookmark_app_helper.cc

Index: chrome/browser/extensions/bookmark_app_helper.cc
diff --git a/chrome/browser/extensions/bookmark_app_helper.cc b/chrome/browser/extensions/bookmark_app_helper.cc
index a70c1b294760af858479e5a0fa4d389438adb6c1..3551a18b62c80df49952f120fe568d78e8fddd0a 100644
--- a/chrome/browser/extensions/bookmark_app_helper.cc
+++ b/chrome/browser/extensions/bookmark_app_helper.cc
@@ -28,8 +28,10 @@
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/web_applications/web_app.h"
+#include "chrome/browser/webshare/share_target_pref_helper.cc"

[Matt Giuca, 2017/01/23 03:00:57]

You can't include a .cc file. Need a header file.

(Did you make a header file but forgot to add it, or is there only a .cc file?)

[constantina, 2017/01/24 02:03:42]

Ah yeah, I just wanted to get it to work. Added header file and included it
everywhere necessary.

 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/extensions/manifest_handlers/app_launch_info.h"
+#include "chrome/common/origin_trials/chrome_origin_trial_policy.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/platform_locale_settings.h"
 #include "components/prefs/pref_service.h"
@@ -583,6 +585,12 @@ void BookmarkAppHelper::OnDidGetManifest(const GURL& manifest_url,
 
   UpdateWebAppInfoFromManifest(manifest, &web_app_info_);
 
+  if (!ChromeOriginTrialPolicy().IsFeatureDisabled("WebShare")) {
+    AddShareTargetToPrefs(manifest_url.GetOrigin().spec(),
+                       manifest.share_target.url_template,

[mlamouri (slow - plz ping), 2017/01/23 00:24:06]

Following up from my comment on the manifest's CL, maybe this should be:
```
if (!ChromeOriginTrialPolicy().IsFeatureDisabled("WebShare") &&
manifest.share_target.has_value()) {
  AddShareTargetToPrefs(manifest_url.GetOrigin().spec(),
                        manifest.share_target.value().url_template,
                        profile_->GetPrefs());
}
```

[Matt Giuca, 2017/01/23 03:00:57]

Needs formatting (some alignment issue).

[constantina, 2017/01/24 02:03:42]

Done.

[constantina, 2017/01/24 02:03:42]

Done.

+                       profile_->GetPrefs());
+  }
+
   // Add urls from the WebApplicationInfo.
   std::vector<GURL> web_app_info_icon_urls;
   for (std::vector<WebApplicationInfo::IconInfo>::const_iterator it =