Revert of Enforce capability spec renderer <--> browser

Revert of Enforce capability spec renderer <--> browser (patchset #14 id:260001 of https://codereview.chromium.org/2259903002/ )

Reason for revert:
Introduces crashes in Mojo on all Windows perf bots: https://bugs.chromium.org/p/chromium/issues/detail?id=647006

Original issue's description:
> Enforce capability spec renderer <--> browser.
> 
> This implements:
> 
> - allows the content embedder to merge a manifest into the base ones provided by content, allowing it to add to the list of interfaces exposed to the renderer.
> - adds such an overlay to chrome/browser
> 
> So.. from now on if you add an interface to either the browser/renderer, you'll need to list it in the manifest. I wonder how we should broadcast this. Also as a separate step we should require these manifests get security review.
> 
> R=rockot@chromium.org,tsepez@chromium.org
> 
> Committed: https://crrev.com/4ea17059bf78553528f3ffb4a9de84a447622fd3
> Committed: https://crrev.com/807a926c9ca02ac8ae1b52e82537834ef78fb9ba
> Committed: https://crrev.com/91c25a3cedc4fadcb8c84e91cc59a8f90411058a
> Cr-Original-Original-Commit-Position: refs/heads/master@{#418282}
> Cr-Original-Commit-Position: refs/heads/master@{#418398}
> Cr-Commit-Position: refs/heads/master@{#418582}

TBR=rockot@chromium.org,tsepez@chromium.org,ben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

[oystein (OOO til 10th of July), 2016-09-14 20:36:38]

The CQ bit was checked by oysteine@chromium.org

[oystein (OOO til 10th of July), 2016-09-14 20:36:38]

Created Revert of Enforce capability spec renderer <--> browser

[commit-bot: I haz the power, 2016-09-14 20:37:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-14 20:38:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-14 20:38:49]

Failed to apply patch for chrome/browser/browser_resources.grd:
While running git apply --index -3 -p1;
  error: patch failed: chrome/browser/browser_resources.grd:576
  error: repository lacks the necessary blob to fall back on 3-way merge.
  error: chrome/browser/browser_resources.grd: patch does not apply

Patch:       chrome/browser/browser_resources.grd
Index: chrome/browser/browser_resources.grd
diff --git a/chrome/browser/browser_resources.grd
b/chrome/browser/browser_resources.grd
index
56897bd6d5e5d8d0662b22730d153178b8a3b25f..65daf56508d305dbcdd481d2d7cd5d647a4e12c0
100644
--- a/chrome/browser/browser_resources.grd
+++ b/chrome/browser/browser_resources.grd
@@ -576,7 +576,6 @@
         <include name="IDR_IME_WINDOW_CLOSE_C"
file="resources\input_ime\ime_window_close_click.png" type="BINDATA" />
         <include name="IDR_IME_WINDOW_CLOSE_H"
file="resources\input_ime\ime_window_close_hover.png" type="BINDATA" />
       </if>
-      <include name="IDR_CONTENT_BROWSER_MANIFEST_OVERLAY"
file="content_browser_manifest_overlay.json" type="BINDATA" />
     </includes>
   </release>
 </grit>

[oystein (OOO til 10th of July), 2016-09-14 20:38:50]

The CQ bit was unchecked by oysteine@chromium.org

[oystein (OOO til 10th of July), 2016-09-14 20:39:12]

Description was changed from

==========
Revert of Enforce capability spec renderer <--> browser (patchset #14 id:260001
of https://codereview.chromium.org/2259903002/ )

Reason for revert:
Introduces crashes in Mojo on all Windows perf bots:
https://bugs.chromium.org/p/chromium/issues/detail?id=647006

Original issue's description:
> Enforce capability spec renderer <--> browser.
> 
> This implements:
> 
> - allows the content embedder to merge a manifest into the base ones provided
by content, allowing it to add to the list of interfaces exposed to the
renderer.
> - adds such an overlay to chrome/browser
> 
> So.. from now on if you add an interface to either the browser/renderer,
you'll need to list it in the manifest. I wonder how we should broadcast this.
Also as a separate step we should require these manifests get security review.
> 
> R=rockot@chromium.org,tsepez@chromium.org
> 
> Committed: https://crrev.com/4ea17059bf78553528f3ffb4a9de84a447622fd3
> Committed: https://crrev.com/807a926c9ca02ac8ae1b52e82537834ef78fb9ba
> Committed: https://crrev.com/91c25a3cedc4fadcb8c84e91cc59a8f90411058a
> Cr-Original-Original-Commit-Position: refs/heads/master@{#418282}
> Cr-Original-Commit-Position: refs/heads/master@{#418398}
> Cr-Commit-Position: refs/heads/master@{#418582}

TBR=rockot@chromium.org,tsepez@chromium.org,ben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of Enforce capability spec renderer <--> browser (patchset #14 id:260001
of https://codereview.chromium.org/2259903002/ )

Reason for revert:
Introduces crashes in Mojo on all Windows perf bots:
https://bugs.chromium.org/p/chromium/issues/detail?id=647006

Original issue's description:
> Enforce capability spec renderer <--> browser.
> 
> This implements:
> 
> - allows the content embedder to merge a manifest into the base ones provided
by content, allowing it to add to the list of interfaces exposed to the
renderer.
> - adds such an overlay to chrome/browser
> 
> So.. from now on if you add an interface to either the browser/renderer,
you'll need to list it in the manifest. I wonder how we should broadcast this.
Also as a separate step we should require these manifests get security review.
> 
> R=rockot@chromium.org,tsepez@chromium.org
> 
> Committed: https://crrev.com/4ea17059bf78553528f3ffb4a9de84a447622fd3
> Committed: https://crrev.com/807a926c9ca02ac8ae1b52e82537834ef78fb9ba
> Committed: https://crrev.com/91c25a3cedc4fadcb8c84e91cc59a8f90411058a
> Cr-Original-Original-Commit-Position: refs/heads/master@{#418282}
> Cr-Original-Commit-Position: refs/heads/master@{#418398}
> Cr-Commit-Position: refs/heads/master@{#418582}

TBR=rockot@chromium.org,tsepez@chromium.org,ben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

[oystein (OOO til 10th of July), 2016-09-14 20:42:20]

(Closed; already reverted)