Revert of [devtools] Support different encodings for Page.CaptureScreenshot.

Revert of [devtools] Support different encodings for Page.CaptureScreenshot. (patchset #8 id:480001 of https://codereview.chromium.org/2592983002/ )

Reason for revert:
Causing failures on "Mac10.11 Tests" builder:

https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.11%20Tests/builds/6881

[ RUN      ] GrabWindowSnapshotTest.TestGrabWindowSnapshot
../../ui/snapshot/snapshot_mac_unittest.mm:45: Failure
Value of: CGImageGetWidth([rep CGImage])
  Actual: 1
Expected: 400 * scaleFactor
Which is: 400
../../ui/snapshot/snapshot_mac_unittest.mm:49: Failure
Expected: (red + green + blue) >= (3.0), actual: 0 vs 3
[  FAILED  ] GrabWindowSnapshotTest.TestGrabWindowSnapshot (31 ms)


Original issue's description:
> [devtools] Support different encodings for Page.CaptureScreenshot.
> 
> This adds support for jpeg encoding of screenshots, similar to that for
> screencasts. For this purpose, the patch modifies ui::Grab*Snapshot*
> methods to return a gfx::Image instead of a png. Utility methods can
> then encode the gfx::Image to jpeg/png depending on its
> platform-specific backing representation.
> 
> BUG=676282
> 
> Review-Url: https://codereview.chromium.org/2592983002
> Cr-Commit-Position: refs/heads/master@{#445730}
> Committed: https://chromium.googlesource.com/chromium/src/+/b6438b1ca6462f3cc014cd1bf18410957101a119

TBR=enne@chromium.org,avi@chromium.org,dtrainor@chromium.org,oshima@chromium.org,pfeldman@chromium.org,sadrul@chromium.org,dpranke@chromium.org,eseckler@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=676282
Review-Url: https://codereview.chromium.org/2650903003
Cr-Commit-Position: refs/heads/master@{#445741}
Committed: https://chromium.googlesource.com/chromium/src/+/1ae4ca9c8372d75a7cd617911c4e0bd4c65de33a

[mohsen, 2017-01-24 16:31:44]

The CQ bit was checked by mohsen@chromium.org

[mohsen, 2017-01-24 16:31:45]

Created Revert of [devtools] Support different encodings for
Page.CaptureScreenshot.

[commit-bot: I haz the power, 2017-01-24 16:31:57]

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, 2017-01-24 16:33:00]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485275504803610, "parent_rev":
"3d4141d40471d274b0b66d2e938821137bcc0f9c", "commit_rev":
"1ae4ca9c8372d75a7cd617911c4e0bd4c65de33a"}

[commit-bot: I haz the power, 2017-01-24 16:33:33]

Description was changed from

==========
Revert of [devtools] Support different encodings for Page.CaptureScreenshot.
(patchset #8 id:480001 of https://codereview.chromium.org/2592983002/ )

Reason for revert:
Causing failures on "Mac10.11 Tests" builder:

https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.11%20Tests...

[ RUN      ] GrabWindowSnapshotTest.TestGrabWindowSnapshot
../../ui/snapshot/snapshot_mac_unittest.mm:45: Failure
Value of: CGImageGetWidth([rep CGImage])
  Actual: 1
Expected: 400 * scaleFactor
Which is: 400
../../ui/snapshot/snapshot_mac_unittest.mm:49: Failure
Expected: (red + green + blue) >= (3.0), actual: 0 vs 3
[  FAILED  ] GrabWindowSnapshotTest.TestGrabWindowSnapshot (31 ms)


Original issue's description:
> [devtools] Support different encodings for Page.CaptureScreenshot.
> 
> This adds support for jpeg encoding of screenshots, similar to that for
> screencasts. For this purpose, the patch modifies ui::Grab*Snapshot*
> methods to return a gfx::Image instead of a png. Utility methods can
> then encode the gfx::Image to jpeg/png depending on its
> platform-specific backing representation.
> 
> BUG=676282
> 
> Review-Url: https://codereview.chromium.org/2592983002
> Cr-Commit-Position: refs/heads/master@{#445730}
> Committed:
https://chromium.googlesource.com/chromium/src/+/b6438b1ca6462f3cc014cd1bf184...

TBR=enne@chromium.org,avi@chromium.org,dtrainor@chromium.org,oshima@chromium....
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=676282
==========

to

==========
Revert of [devtools] Support different encodings for Page.CaptureScreenshot.
(patchset #8 id:480001 of https://codereview.chromium.org/2592983002/ )

Reason for revert:
Causing failures on "Mac10.11 Tests" builder:

https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.11%20Tests...

[ RUN      ] GrabWindowSnapshotTest.TestGrabWindowSnapshot
../../ui/snapshot/snapshot_mac_unittest.mm:45: Failure
Value of: CGImageGetWidth([rep CGImage])
  Actual: 1
Expected: 400 * scaleFactor
Which is: 400
../../ui/snapshot/snapshot_mac_unittest.mm:49: Failure
Expected: (red + green + blue) >= (3.0), actual: 0 vs 3
[  FAILED  ] GrabWindowSnapshotTest.TestGrabWindowSnapshot (31 ms)


Original issue's description:
> [devtools] Support different encodings for Page.CaptureScreenshot.
> 
> This adds support for jpeg encoding of screenshots, similar to that for
> screencasts. For this purpose, the patch modifies ui::Grab*Snapshot*
> methods to return a gfx::Image instead of a png. Utility methods can
> then encode the gfx::Image to jpeg/png depending on its
> platform-specific backing representation.
> 
> BUG=676282
> 
> Review-Url: https://codereview.chromium.org/2592983002
> Cr-Commit-Position: refs/heads/master@{#445730}
> Committed:
https://chromium.googlesource.com/chromium/src/+/b6438b1ca6462f3cc014cd1bf184...

TBR=enne@chromium.org,avi@chromium.org,dtrainor@chromium.org,oshima@chromium....
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=676282

Review-Url: https://codereview.chromium.org/2650903003
Cr-Commit-Position: refs/heads/master@{#445741}
Committed:
https://chromium.googlesource.com/chromium/src/+/1ae4ca9c8372d75a7cd617911c4e...
==========

[commit-bot: I haz the power, 2017-01-24 16:33:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/1ae4ca9c8372d75a7cd617911c4e...