Revert of Implement webview.captureVisibleRegion()

Revert of Implement webview.captureVisibleRegion() (patchset #9 id:160001 of https://codereview.chromium.org/1582053002/ )

Reason for revert:
The test WebViewAPITest.TestCaptureVisibleRegion is failing on several windows bots:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/45040
https://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/42690
https://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/builds/64394

Original issue's description:
> Implement webview.captureVisibleRegion()
>
> This CL implements  webview.captureVisibleRegion(), an extension/apps
> API to allow WebView users to capture screenshots of the contents
> displayed in a WebView. The surfaces contents capture has been plumbed
> via RenderWidgetHostViewChildFrame so this implementation should not
> require changes when WebView switches to using OOPIF.
>
> As part of the implementation, there are two notable refactors:
>
> 1) CaptureWebContentsFunction has been refactored into
> WebContentsCaptureClient to remove the extensions::AsyncExtensionFunction
> dependence so that this code can be used by both tabs.captureVisibleTab
> and webview.captureVisibleRegion, and
>
> 2) common code from DelegatedFrameHost has ben moved to
> content/browser/compositor/surface_utils.* in order to avoid duplication
> as both DelegatedFrameHost and RenderWidgetHostViewChildFrame now
> use the code.
>
> BUG=326755
>
> Committed: https://crrev.com/5b8cdcc82fc9da7bab15fd3dbf65df843dc73589
> Cr-Commit-Position: refs/heads/master@{#370565}

TBR=rockot@chromium.org,creis@chromium.org,kenrb@chromium.org,nick@chromium.org,fsamuel@chromium.org,wjmaclean@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=326755
Committed: https://crrev.com/b30b1aaf04cb1cae5c455461c8c5c418dd7c2aea
Cr-Commit-Position: refs/heads/master@{#370641}

[tsergeant, 2016-01-21 06:49:06]

Created Revert of Implement webview.captureVisibleRegion()

[commit-bot: I haz the power, 2016-01-21 06:49:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1614703003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1614703003/1

[commit-bot: I haz the power, 2016-01-21 06:51:07]

Description was changed from

==========
Revert of Implement webview.captureVisibleRegion() (patchset #9 id:160001 of
https://codereview.chromium.org/1582053002/ )

Reason for revert:
The test WebViewAPITest.TestCaptureVisibleRegion is failing on several windows
bots:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...
https://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/build...
https://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/bu...

Original issue's description:
> Implement webview.captureVisibleRegion()
> 
> This CL implements  webview.captureVisibleRegion(), an extension/apps
> API to allow WebView users to capture screenshots of the contents
> displayed in a WebView. The surfaces contents capture has been plumbed
> via RenderWidgetHostViewChildFrame so this implementation should not
> require changes when WebView switches to using OOPIF.
> 
> As part of the implementation, there are two notable refactors:
> 
> 1) CaptureWebContentsFunction has been refactored into
> WebContentsCaptureClient to remove the extensions::AsyncExtensionFunction
> dependence so that this code can be used by both tabs.captureVisibleTab
> and webview.captureVisibleRegion, and
> 
> 2) common code from DelegatedFrameHost has ben moved to
> content/browser/compositor/surface_utils.* in order to avoid duplication
> as both DelegatedFrameHost and RenderWidgetHostViewChildFrame now
> use the code.
> 
> BUG=326755
> 
> Committed: https://crrev.com/5b8cdcc82fc9da7bab15fd3dbf65df843dc73589
> Cr-Commit-Position: refs/heads/master@{#370565}

TBR=rockot@chromium.org,creis@chromium.org,kenrb@chromium.org,nick@chromium.o...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=326755
==========

to

==========
Revert of Implement webview.captureVisibleRegion() (patchset #9 id:160001 of
https://codereview.chromium.org/1582053002/ )

Reason for revert:
The test WebViewAPITest.TestCaptureVisibleRegion is failing on several windows
bots:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...
https://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/build...
https://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/bu...

Original issue's description:
> Implement webview.captureVisibleRegion()
> 
> This CL implements  webview.captureVisibleRegion(), an extension/apps
> API to allow WebView users to capture screenshots of the contents
> displayed in a WebView. The surfaces contents capture has been plumbed
> via RenderWidgetHostViewChildFrame so this implementation should not
> require changes when WebView switches to using OOPIF.
> 
> As part of the implementation, there are two notable refactors:
> 
> 1) CaptureWebContentsFunction has been refactored into
> WebContentsCaptureClient to remove the extensions::AsyncExtensionFunction
> dependence so that this code can be used by both tabs.captureVisibleTab
> and webview.captureVisibleRegion, and
> 
> 2) common code from DelegatedFrameHost has ben moved to
> content/browser/compositor/surface_utils.* in order to avoid duplication
> as both DelegatedFrameHost and RenderWidgetHostViewChildFrame now
> use the code.
> 
> BUG=326755
> 
> Committed: https://crrev.com/5b8cdcc82fc9da7bab15fd3dbf65df843dc73589
> Cr-Commit-Position: refs/heads/master@{#370565}

TBR=rockot@chromium.org,creis@chromium.org,kenrb@chromium.org,nick@chromium.o...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=326755
==========

[commit-bot: I haz the power, 2016-01-21 06:51:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-21 06:51:58]

Description was changed from

==========
Revert of Implement webview.captureVisibleRegion() (patchset #9 id:160001 of
https://codereview.chromium.org/1582053002/ )

Reason for revert:
The test WebViewAPITest.TestCaptureVisibleRegion is failing on several windows
bots:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...
https://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/build...
https://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/bu...

Original issue's description:
> Implement webview.captureVisibleRegion()
> 
> This CL implements  webview.captureVisibleRegion(), an extension/apps
> API to allow WebView users to capture screenshots of the contents
> displayed in a WebView. The surfaces contents capture has been plumbed
> via RenderWidgetHostViewChildFrame so this implementation should not
> require changes when WebView switches to using OOPIF.
> 
> As part of the implementation, there are two notable refactors:
> 
> 1) CaptureWebContentsFunction has been refactored into
> WebContentsCaptureClient to remove the extensions::AsyncExtensionFunction
> dependence so that this code can be used by both tabs.captureVisibleTab
> and webview.captureVisibleRegion, and
> 
> 2) common code from DelegatedFrameHost has ben moved to
> content/browser/compositor/surface_utils.* in order to avoid duplication
> as both DelegatedFrameHost and RenderWidgetHostViewChildFrame now
> use the code.
> 
> BUG=326755
> 
> Committed: https://crrev.com/5b8cdcc82fc9da7bab15fd3dbf65df843dc73589
> Cr-Commit-Position: refs/heads/master@{#370565}

TBR=rockot@chromium.org,creis@chromium.org,kenrb@chromium.org,nick@chromium.o...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=326755
==========

to

==========
Revert of Implement webview.captureVisibleRegion() (patchset #9 id:160001 of
https://codereview.chromium.org/1582053002/ )

Reason for revert:
The test WebViewAPITest.TestCaptureVisibleRegion is failing on several windows
bots:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...
https://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/build...
https://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/bu...

Original issue's description:
> Implement webview.captureVisibleRegion()
>
> This CL implements  webview.captureVisibleRegion(), an extension/apps
> API to allow WebView users to capture screenshots of the contents
> displayed in a WebView. The surfaces contents capture has been plumbed
> via RenderWidgetHostViewChildFrame so this implementation should not
> require changes when WebView switches to using OOPIF.
>
> As part of the implementation, there are two notable refactors:
>
> 1) CaptureWebContentsFunction has been refactored into
> WebContentsCaptureClient to remove the extensions::AsyncExtensionFunction
> dependence so that this code can be used by both tabs.captureVisibleTab
> and webview.captureVisibleRegion, and
>
> 2) common code from DelegatedFrameHost has ben moved to
> content/browser/compositor/surface_utils.* in order to avoid duplication
> as both DelegatedFrameHost and RenderWidgetHostViewChildFrame now
> use the code.
>
> BUG=326755
>
> Committed: https://crrev.com/5b8cdcc82fc9da7bab15fd3dbf65df843dc73589
> Cr-Commit-Position: refs/heads/master@{#370565}

TBR=rockot@chromium.org,creis@chromium.org,kenrb@chromium.org,nick@chromium.o...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=326755

Committed: https://crrev.com/b30b1aaf04cb1cae5c455461c8c5c418dd7c2aea
Cr-Commit-Position: refs/heads/master@{#370641}
==========

[commit-bot: I haz the power, 2016-01-21 06:51:59]

Patchset 1 (id:??) landed as
https://crrev.com/b30b1aaf04cb1cae5c455461c8c5c418dd7c2aea
Cr-Commit-Position: refs/heads/master@{#370641}