Remove BlimpUiContextFactory

Remove BlimpUiContextFactory

This removes the override of the ContextFactory and replaces it by just
dropping the OnAcceleratedWidgetAvailable call on the floor.  This
prevents GetSharedBitmap/ResizeDisplay/CreateOutputSurface from being
called at all rather than ignoring them when they got called.

R=sievers@chromium.org,haibinlu@chromium.org

Committed: https://crrev.com/ae96391ad67ebb128d0e0082bdd9633989fd370b
Cr-Commit-Position: refs/heads/master@{#383156}

[enne (OOO), 2016-03-24 18:26:45]

This appears to do the right thing for me when I run locally.  It seems like the
main reason that BlimpUiContextFactory exists is to avoid a bunch of calls, but
stopping them higher up seems even better if it's possible.

[no sievers, 2016-03-24 18:37:53]

LGTM

[no sievers, 2016-03-24 18:40:25]

Can you remove the "content/browser/compositor" DEPS entry again now for the
blimp engine, so that it stays restricted to using the public api?

[enne (OOO), 2016-03-24 18:41:57]

Description was changed from

==========
Remove BlimpUiContextFactory

This removes the override of the ContextFactory and replaces it by just
dropping the OnAcceleratedWidgetAvailable call on the floor.  This
prevents GetSharedBitmap/ResizeDisplay/CreateOutputSurface from being
called at all rather than ignoring them when they got called.

It's also awfully confusing to have two global accessors for the
"global" ContextFactory via ImageTransportFactory::GetContextFactory and
aura::Env::GetContextFactory, which may return different results.  An
improved solution here is to make one of them definitive
(ImageTransportFactory) and remove the other (aura::Env).

R=sievers@chromium.org,haibinlu@chromium.org
==========

to

==========
Remove BlimpUiContextFactory

This removes the override of the ContextFactory and replaces it by just
dropping the OnAcceleratedWidgetAvailable call on the floor.  This
prevents GetSharedBitmap/ResizeDisplay/CreateOutputSurface from being
called at all rather than ignoring them when they got called.

R=sievers@chromium.org,haibinlu@chromium.org
==========

[haibinlu, 2016-03-24 18:44:04]

lgtm

Thanks!

[enne (OOO), 2016-03-24 18:55:21]

On 2016/03/24 at 18:40:25, sievers wrote:
> Can you remove the "content/browser/compositor" DEPS entry again now for the
blimp engine, so that it stays restricted to using the public api?

Done, thanks!

[enne (OOO), 2016-03-24 21:45:10]

The CQ bit was checked by enne@chromium.org

[enne (OOO), 2016-03-24 21:45:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
haibinlu@chromium.org, sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1830293003/#ps20001
(title: "Remove blink/engine deps")

[commit-bot: I haz the power, 2016-03-24 21:45:36]

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

[commit-bot: I haz the power, 2016-03-24 21:51:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-24 21:52:01]

Description was changed from

==========
Remove BlimpUiContextFactory

This removes the override of the ContextFactory and replaces it by just
dropping the OnAcceleratedWidgetAvailable call on the floor.  This
prevents GetSharedBitmap/ResizeDisplay/CreateOutputSurface from being
called at all rather than ignoring them when they got called.

R=sievers@chromium.org,haibinlu@chromium.org
==========

to

==========
Remove BlimpUiContextFactory

This removes the override of the ContextFactory and replaces it by just
dropping the OnAcceleratedWidgetAvailable call on the floor.  This
prevents GetSharedBitmap/ResizeDisplay/CreateOutputSurface from being
called at all rather than ignoring them when they got called.

R=sievers@chromium.org,haibinlu@chromium.org

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

[commit-bot: I haz the power, 2016-03-24 21:52:02]

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