Revert of Optimize render passes with single quads

Revert of Optimize render passes with single quads (patchset #13 id:240001 of https://codereview.chromium.org/1960543002/ )

Reason for revert:
Context menu is broken on Linux:
https://drive.google.com/file/d/0B7RLEApsOKBLNGZDai1JT0kwR2M/view?usp=sharing

BUG=619185
Original issue's description:
> Optimize render passes with single quads
>
> Many effects (masks, replicas, filters) generate render passes for
> simplicity in the code.  However, in the cases where the pass contains a
> single quad with a texture, that resulting texture could just be used as
> the input texture instead of the render pass itself.
>
> The complication is mostly that render passes and tile textures are
> flipped relative to each other (oops) and so some knowledge of this has
> to leak into drawing render passes.
>
> This is done by detecting such render passes inside of DirectRenderer,
> storing the TileQuad that would have been drawn, skipping allocating the
> pass and rendering it, and then calling a slightly modified version of
> DrawRenderPassQuad with the TileQuad's resource.  The check for which
> render passes can be supported is conservative to start.
>
> This optimization usually will not be supported on mac because skia
> does not support textures with texture rectangle targets as input.
>
> BUG=254639, 606672
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/ff3dc65b0f7845184458fb25b3d566fa079cd232
> Cr-Commit-Position: refs/heads/master@{#399060}

TBR=ericrk@chromium.org,senorblanco@chromium.org,enne@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=254639, 606672
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/95a0c4522dbb71c522c09798e7dbc77a7285d1ea
Cr-Commit-Position: refs/heads/master@{#399308}

[kozy, 2016-06-10 22:03:58]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2016-06-10 22:03:59]

Created Revert of Optimize render passes with single quads

[commit-bot: I haz the power, 2016-06-10 22:04:55]

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

[commit-bot: I haz the power, 2016-06-10 22:05:36]

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

[commit-bot: I haz the power, 2016-06-10 22:05:37]

Failed to apply patch for third_party/WebKit/LayoutTests/TestExpectations:
While running git apply --index -3 -p1;
  error: patch failed: third_party/WebKit/LayoutTests/TestExpectations:1282
  error: repository lacks the necessary blob to fall back on 3-way merge.
  error: third_party/WebKit/LayoutTests/TestExpectations: patch does not apply

Patch:       third_party/WebKit/LayoutTests/TestExpectations
Index: third_party/WebKit/LayoutTests/TestExpectations
diff --git a/third_party/WebKit/LayoutTests/TestExpectations
b/third_party/WebKit/LayoutTests/TestExpectations
index
aaef47cb41321704f5a213044266e7504010c494..e9a04d504efeac83964b6f8995b930752798de05
100644
--- a/third_party/WebKit/LayoutTests/TestExpectations
+++ b/third_party/WebKit/LayoutTests/TestExpectations
@@ -1282,29 +1282,6 @@
 
 crbug.com/588103
fast/xmlhttprequest/xmlhttprequest-responsetype-arraybuffer.html [ Pass Failure
]
 
-crbug.com/254639 compositing/masks/direct-image-mask.html [ NeedsRebaseline ]
-crbug.com/254639 compositing/reflections/nested-reflection-anchor-point.html [
NeedsRebaseline ]
-crbug.com/254639 compositing/reflections/nested-reflection-animated.html [
NeedsRebaseline ]
-crbug.com/254639 compositing/reflections/nested-reflection-opacity.html [
NeedsRebaseline ]
-crbug.com/254639 css3/blending/effect-background-blend-mode-stacking.html [
NeedsRebaseline ]
-crbug.com/254639 css3/filters/composited-reflected.html [ NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-all-on-background-hw.html [
NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-brightness-clamping-hw.html [
NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-brightness-hw.html [ NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-contrast-hw.html [ NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-grayscale-hw.html [ NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-hue-rotate-hw.html [ NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-reference-colorspace-hw.html [
NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-reference-hidpi-hw.html [ NeedsRebaseline
]
-crbug.com/254639 css3/filters/effect-reference-hw.html [ NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-reference-ordering-hw.html [
NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-reference-subregion-hw.html [
NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-reference-zoom-hw.html [ NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-saturate-hw.html [ NeedsRebaseline ]
-crbug.com/254639 css3/filters/effect-sepia-hw.html [ NeedsRebaseline ]
-crbug.com/254639 compositing/reflections/nested-reflection-transformed.html [
NeedsRebaseline ]
-crbug.com/254639 compositing/reflections/nested-reflection-transformed2.html [
NeedsRebaseline ]
-
 crbug.com/594672 fast/events/iframe-object-onload.html [ Failure Pass ]
 crbug.com/594672 fast/events/scale-and-scroll-iframe-body.html [ Failure Pass ]
 crbug.com/594672 fast/events/updateLayoutForHitTest.html [ Failure Pass ]

[commit-bot: I haz the power, 2016-06-10 22:06:06]

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

[commit-bot: I haz the power, 2016-06-10 22:06:07]

CQ bit was unchecked

[danakj, 2016-06-10 22:10:31]

danakj@chromium.org changed reviewers:
+ danakj@chromium.org

[danakj, 2016-06-10 22:10:31]

Does the context menu use multiple render passes? --ui-show-layer-borders will
give layers borders. Blue means a separate render pass.

[danakj, 2016-06-10 22:11:16]

On 2016/06/10 22:10:31, danakj wrote:
> Does the context menu use multiple render passes? --ui-show-layer-borders will
> give layers borders. Blue means a separate render pass.

(Is there a bug? One wasn't added here)

[kozy, 2016-06-10 22:15:24]

On 2016/06/10 22:10:31, danakj wrote:
> Does the context menu use multiple render passes? --ui-show-layer-borders will
> give layers borders. Blue means a separate render pass.

It has orange border and it works with --ui-show-layer-borders:
https://drive.google.com/a/chromium.org/file/d/0B7RLEApsOKBLRkRqQlZQY2F5SFk/v...

[danakj, 2016-06-10 22:17:54]

On Fri, Jun 10, 2016 at 3:15 PM, <kozyatinskiy@chromium.org> wrote:

> On 2016/06/10 22:10:31, danakj wrote:
> > Does the context menu use multiple render passes?
> --ui-show-layer-borders will
> > give layers borders. Blue means a separate render pass.
>
> It has orange border and it works with --ui-show-layer-borders:
>
>
https://drive.google.com/a/chromium.org/file/d/0B7RLEApsOKBLRkRqQlZQY2F5SFk/v...
>

Orange means it's not a separate render pass. How sure are you it's this CL?

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[kozy, 2016-06-10 22:19:55]

Description was changed from

==========
Revert of Optimize render passes with single quads (patchset #13 id:240001 of
https://codereview.chromium.org/1960543002/ )

Reason for revert:
Context menu is broken on Linux:
https://drive.google.com/file/d/0B7RLEApsOKBLNGZDai1JT0kwR2M/view?usp=sharing

Original issue's description:
> Optimize render passes with single quads
> 
> Many effects (masks, replicas, filters) generate render passes for
> simplicity in the code.  However, in the cases where the pass contains a
> single quad with a texture, that resulting texture could just be used as
> the input texture instead of the render pass itself.
> 
> The complication is mostly that render passes and tile textures are
> flipped relative to each other (oops) and so some knowledge of this has
> to leak into drawing render passes.
> 
> This is done by detecting such render passes inside of DirectRenderer,
> storing the TileQuad that would have been drawn, skipping allocating the
> pass and rendering it, and then calling a slightly modified version of
> DrawRenderPassQuad with the TileQuad's resource.  The check for which
> render passes can be supported is conservative to start.
> 
> This optimization usually will not be supported on mac because skia
> does not support textures with texture rectangle targets as input.
> 
> BUG=254639,606672
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/ff3dc65b0f7845184458fb25b3d566fa079cd232
> Cr-Commit-Position: refs/heads/master@{#399060}

TBR=ericrk@chromium.org,senorblanco@chromium.org,enne@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=254639,606672
==========

to

==========
Revert of Optimize render passes with single quads (patchset #13 id:240001 of
https://codereview.chromium.org/1960543002/ )

Reason for revert:
Context menu is broken on Linux:
https://drive.google.com/file/d/0B7RLEApsOKBLNGZDai1JT0kwR2M/view?usp=sharing

BUG=619185

Original issue's description:
> Optimize render passes with single quads
> 
> Many effects (masks, replicas, filters) generate render passes for
> simplicity in the code.  However, in the cases where the pass contains a
> single quad with a texture, that resulting texture could just be used as
> the input texture instead of the render pass itself.
> 
> The complication is mostly that render passes and tile textures are
> flipped relative to each other (oops) and so some knowledge of this has
> to leak into drawing render passes.
> 
> This is done by detecting such render passes inside of DirectRenderer,
> storing the TileQuad that would have been drawn, skipping allocating the
> pass and rendering it, and then calling a slightly modified version of
> DrawRenderPassQuad with the TileQuad's resource.  The check for which
> render passes can be supported is conservative to start.
> 
> This optimization usually will not be supported on mac because skia
> does not support textures with texture rectangle targets as input.
> 
> BUG=254639,606672
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/ff3dc65b0f7845184458fb25b3d566fa079cd232
> Cr-Commit-Position: refs/heads/master@{#399060}

TBR=ericrk@chromium.org,senorblanco@chromium.org,enne@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=254639,606672
==========

[kozy, 2016-06-10 22:21:56]

On 2016/06/10 22:17:54, danakj wrote:
> On Fri, Jun 10, 2016 at 3:15 PM, <mailto:kozyatinskiy@chromium.org> wrote:
> 
> > On 2016/06/10 22:10:31, danakj wrote:
> > > Does the context menu use multiple render passes?
> > --ui-show-layer-borders will
> > > give layers borders. Blue means a separate render pass.
> >
> > It has orange border and it works with --ui-show-layer-borders:
> >
> >
>
https://drive.google.com/a/chromium.org/file/d/0B7RLEApsOKBLRkRqQlZQY2F5SFk/v...
> >
> 
> Orange means it's not a separate render pass. How sure are you it's this CL?
> 
> -- 
> You received this message because you are subscribed to the Google Groups
> "Chromium-reviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

I've bisected it and then reverted on my machine and it solves the problem
locally.
With --ui-show-layer-borders menus is rendered correctly, without flag it's
rendered as black box.

[danakj, 2016-06-10 22:26:30]

On Fri, Jun 10, 2016 at 3:21 PM, <kozyatinskiy@chromium.org> wrote:

> On 2016/06/10 22:17:54, danakj wrote:
> > On Fri, Jun 10, 2016 at 3:15 PM, <mailto:kozyatinskiy@chromium.org>
> wrote:
> >
> > > On 2016/06/10 22:10:31, danakj wrote:
> > > > Does the context menu use multiple render passes?
> > > --ui-show-layer-borders will
> > > > give layers borders. Blue means a separate render pass.
> > >
> > > It has orange border and it works with --ui-show-layer-borders:
> > >
> > >
> >
>
>
https://drive.google.com/a/chromium.org/file/d/0B7RLEApsOKBLRkRqQlZQY2F5SFk/v...
> > >
> >
> > Orange means it's not a separate render pass. How sure are you it's this
> CL?
> >
> > --
> > You received this message because you are subscribed to the Google Groups
> > "Chromium-reviews" group.
> > To unsubscribe from this group and stop receiving emails from it, send an
> email
> > to mailto:chromium-reviews+unsubscribe@chromium.org.
>
> I've bisected it and then reverted on my machine and it solves the problem
> locally.
> With --ui-show-layer-borders menus is rendered correctly, without flag it's
> rendered as black box.
>

OK thanks. :) I think you'll have to revert 2 more CLs to revert this. They
came after on https://bugs.chromium.org/p/chromium/issues/detail?id=254639.


>
> https://codereview.chromium.org/2058013003/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[kozy, 2016-06-10 22:36:42]

Description was changed from

==========
Revert of Optimize render passes with single quads (patchset #13 id:240001 of
https://codereview.chromium.org/1960543002/ )

Reason for revert:
Context menu is broken on Linux:
https://drive.google.com/file/d/0B7RLEApsOKBLNGZDai1JT0kwR2M/view?usp=sharing

BUG=619185

Original issue's description:
> Optimize render passes with single quads
> 
> Many effects (masks, replicas, filters) generate render passes for
> simplicity in the code.  However, in the cases where the pass contains a
> single quad with a texture, that resulting texture could just be used as
> the input texture instead of the render pass itself.
> 
> The complication is mostly that render passes and tile textures are
> flipped relative to each other (oops) and so some knowledge of this has
> to leak into drawing render passes.
> 
> This is done by detecting such render passes inside of DirectRenderer,
> storing the TileQuad that would have been drawn, skipping allocating the
> pass and rendering it, and then calling a slightly modified version of
> DrawRenderPassQuad with the TileQuad's resource.  The check for which
> render passes can be supported is conservative to start.
> 
> This optimization usually will not be supported on mac because skia
> does not support textures with texture rectangle targets as input.
> 
> BUG=254639,606672
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/ff3dc65b0f7845184458fb25b3d566fa079cd232
> Cr-Commit-Position: refs/heads/master@{#399060}

TBR=ericrk@chromium.org,senorblanco@chromium.org,enne@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=254639,606672
==========

to

==========
Revert of Optimize render passes with single quads (patchset #13 id:240001 of
https://codereview.chromium.org/1960543002/ )

Reason for revert:
Context menu is broken on Linux:
https://drive.google.com/file/d/0B7RLEApsOKBLNGZDai1JT0kwR2M/view?usp=sharing

BUG=619185

Original issue's description:
> Optimize render passes with single quads
> 
> Many effects (masks, replicas, filters) generate render passes for
> simplicity in the code.  However, in the cases where the pass contains a
> single quad with a texture, that resulting texture could just be used as
> the input texture instead of the render pass itself.
> 
> The complication is mostly that render passes and tile textures are
> flipped relative to each other (oops) and so some knowledge of this has
> to leak into drawing render passes.
> 
> This is done by detecting such render passes inside of DirectRenderer,
> storing the TileQuad that would have been drawn, skipping allocating the
> pass and rendering it, and then calling a slightly modified version of
> DrawRenderPassQuad with the TileQuad's resource.  The check for which
> render passes can be supported is conservative to start.
> 
> This optimization usually will not be supported on mac because skia
> does not support textures with texture rectangle targets as input.
> 
> BUG=254639,606672
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/ff3dc65b0f7845184458fb25b3d566fa079cd232
> Cr-Commit-Position: refs/heads/master@{#399060}

TBR=ericrk@chromium.org,senorblanco@chromium.org,enne@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=254639,606672
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[kozy, 2016-06-10 22:36:58]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-06-10 22:37:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2058013003/270001

[commit-bot: I haz the power, 2016-06-10 22:38:05]

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

[commit-bot: I haz the power, 2016-06-10 22:38:06]

Failed to apply patch for third_party/WebKit/LayoutTests/TestExpectations:
While running git apply --index -3 -p1;
  error: patch failed: third_party/WebKit/LayoutTests/TestExpectations:1465
  Falling back to three-way merge...
  Applied patch to 'third_party/WebKit/LayoutTests/TestExpectations' with
conflicts.
  U third_party/WebKit/LayoutTests/TestExpectations

Patch:       third_party/WebKit/LayoutTests/TestExpectations
Index: third_party/WebKit/LayoutTests/TestExpectations
diff --git a/third_party/WebKit/LayoutTests/TestExpectations
b/third_party/WebKit/LayoutTests/TestExpectations
index
7ae50dd71ee9bf823113ac78f6c9a54768c038ac..4433206d4c27ce1530d0f168c7dd43600cd542a9
100644
--- a/third_party/WebKit/LayoutTests/TestExpectations
+++ b/third_party/WebKit/LayoutTests/TestExpectations
@@ -42,10 +42,6 @@ crbug.com/504613 crbug.com/524248
paint/images/image-backgrounds-not-antialiased
 crbug.com/504613 crbug.com/524248
virtual/spv2/paint/images/image-backgrounds-not-antialiased.html [ Skip ]
 crbug.com/502531 fast/borders/border-antialiasing.html [ Failure ]
 
-# Fails on Linux and Win.
-crbug.com/254639 css3/filters/effect-brightness-hw.html [ Failure Pass ]
-crbug.com/254639 css3/filters/effect-hue-rotate-hw.html [ Failure Pass ]
-
 crbug.com/600008 [ Win ]
fast/repaint/window-resize-centered-inline-under-fixed-pos.html [ Failure Pass ]
 crbug.com/600008 [ Win ] fast/repaint/window-resize-vertical-writing-mode.html
[ Failure Pass ]
 crbug.com/600008 [ Mac Win ]
fast/repaint/window-resize-background-image-non-fixed.html [ Failure Pass ] 
@@ -1465,3 +1461,26 @@ crbug.com/613663
imported/wpt/quirks-mode/table-cell-width-calculation.html [ Fa
 crbug.com/614509 [ Win ] inspector/elements/styles-2/pseudo-elements.html [
Pass Timeout ]
 
 crbug.com/618399 fast/js/regress/to-int32-boolean.html [ Pass Timeout ]
+
+crbug.com/254639 compositing/masks/direct-image-mask.html [ NeedsRebaseline ]
+crbug.com/254639 compositing/reflections/nested-reflection-anchor-point.html [
NeedsRebaseline ]
+crbug.com/254639 compositing/reflections/nested-reflection-animated.html [
NeedsRebaseline ]
+crbug.com/254639 compositing/reflections/nested-reflection-opacity.html [
NeedsRebaseline ]
+crbug.com/254639 css3/blending/effect-background-blend-mode-stacking.html [
NeedsRebaseline ]
+crbug.com/254639 css3/filters/composited-reflected.html [ NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-all-on-background-hw.html [
NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-brightness-clamping-hw.html [
NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-brightness-hw.html [ NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-contrast-hw.html [ NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-grayscale-hw.html [ NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-hue-rotate-hw.html [ NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-reference-colorspace-hw.html [
NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-reference-hidpi-hw.html [ NeedsRebaseline
]
+crbug.com/254639 css3/filters/effect-reference-hw.html [ NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-reference-ordering-hw.html [
NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-reference-subregion-hw.html [
NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-reference-zoom-hw.html [ NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-saturate-hw.html [ NeedsRebaseline ]
+crbug.com/254639 css3/filters/effect-sepia-hw.html [ NeedsRebaseline ]
+crbug.com/254639 compositing/reflections/nested-reflection-transformed.html [
NeedsRebaseline ]
+crbug.com/254639 compositing/reflections/nested-reflection-transformed2.html [
NeedsRebaseline ]

[commit-bot: I haz the power, 2016-06-10 22:38:25]

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

[commit-bot: I haz the power, 2016-06-10 22:38:27]

CQ bit was unchecked

[kozy, 2016-06-10 22:51:07]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-06-10 22:52:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2058013003/290001

[commit-bot: I haz the power, 2016-06-10 22:52:52]

Committed patchset #3 (id:290001)

[commit-bot: I haz the power, 2016-06-10 22:54:41]

Description was changed from

==========
Revert of Optimize render passes with single quads (patchset #13 id:240001 of
https://codereview.chromium.org/1960543002/ )

Reason for revert:
Context menu is broken on Linux:
https://drive.google.com/file/d/0B7RLEApsOKBLNGZDai1JT0kwR2M/view?usp=sharing

BUG=619185

Original issue's description:
> Optimize render passes with single quads
> 
> Many effects (masks, replicas, filters) generate render passes for
> simplicity in the code.  However, in the cases where the pass contains a
> single quad with a texture, that resulting texture could just be used as
> the input texture instead of the render pass itself.
> 
> The complication is mostly that render passes and tile textures are
> flipped relative to each other (oops) and so some knowledge of this has
> to leak into drawing render passes.
> 
> This is done by detecting such render passes inside of DirectRenderer,
> storing the TileQuad that would have been drawn, skipping allocating the
> pass and rendering it, and then calling a slightly modified version of
> DrawRenderPassQuad with the TileQuad's resource.  The check for which
> render passes can be supported is conservative to start.
> 
> This optimization usually will not be supported on mac because skia
> does not support textures with texture rectangle targets as input.
> 
> BUG=254639,606672
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/ff3dc65b0f7845184458fb25b3d566fa079cd232
> Cr-Commit-Position: refs/heads/master@{#399060}

TBR=ericrk@chromium.org,senorblanco@chromium.org,enne@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=254639,606672
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Revert of Optimize render passes with single quads (patchset #13 id:240001 of
https://codereview.chromium.org/1960543002/ )

Reason for revert:
Context menu is broken on Linux:
https://drive.google.com/file/d/0B7RLEApsOKBLNGZDai1JT0kwR2M/view?usp=sharing

BUG=619185

Original issue's description:
> Optimize render passes with single quads
>
> Many effects (masks, replicas, filters) generate render passes for
> simplicity in the code.  However, in the cases where the pass contains a
> single quad with a texture, that resulting texture could just be used as
> the input texture instead of the render pass itself.
>
> The complication is mostly that render passes and tile textures are
> flipped relative to each other (oops) and so some knowledge of this has
> to leak into drawing render passes.
>
> This is done by detecting such render passes inside of DirectRenderer,
> storing the TileQuad that would have been drawn, skipping allocating the
> pass and rendering it, and then calling a slightly modified version of
> DrawRenderPassQuad with the TileQuad's resource.  The check for which
> render passes can be supported is conservative to start.
>
> This optimization usually will not be supported on mac because skia
> does not support textures with texture rectangle targets as input.
>
> BUG=254639,606672
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/ff3dc65b0f7845184458fb25b3d566fa079cd232
> Cr-Commit-Position: refs/heads/master@{#399060}

TBR=ericrk@chromium.org,senorblanco@chromium.org,enne@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=254639,606672
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/95a0c4522dbb71c522c09798e7dbc77a7285d1ea
Cr-Commit-Position: refs/heads/master@{#399308}
==========

[commit-bot: I haz the power, 2016-06-10 22:54:42]

Patchset 3 (id:??) landed as
https://crrev.com/95a0c4522dbb71c522c09798e7dbc77a7285d1ea
Cr-Commit-Position: refs/heads/master@{#399308}