Navigation transitions: Exposed "insertCSS" to WebContents

Navigation transitions: Exposed "insertCSS" to WebContents

The navigation transition needs to apply some styles during the transition, so we need to expose this call to java.

Design doc: https://docs.google.com/a/chromium.org/document/d/17jg1RRL3RI969cLwbKBIcoGDsPwqaEdBxafGNYGwiY4/edit# 
Implementation details: https://docs.google.com/a/chromium.org/document/d/1kREPtFJaeLoDKwrfmrYTD7DHCdxX1RzFBga2gNY8lyE/edit#heading=h.bng2kpmyvxq5
BUG=370696
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282504

[Yaron, 2014-07-08 23:49:37]

Could you move this to WebContents.java and web_contents_android.{h,cc}?

I know that CVC is historically where we've added these accessors but since this
looks like a new API exposed to java, I think the most logical place is to put
directly in java WebContents.

(we're in the process of moving more WebContents functionality from CVC to
there).

Also, how/where is this API invoked? I see the attached doc but I guess I missed
that.

[shatch, 2014-07-09 18:58:34]

On 2014/07/08 23:49:37, Yaron wrote:
> Could you move this to WebContents.java and web_contents_android.{h,cc}?
> 
> I know that CVC is historically where we've added these accessors but since
this
> looks like a new API exposed to java, I think the most logical place is to put
> directly in java WebContents.
> 
> (we're in the process of moving more WebContents functionality from CVC to
> there).
> 
> Also, how/where is this API invoked? I see the attached doc but I guess I
missed
> that.

Sure! Moved everything in new snapshot, ptal.

[Yaron, 2014-07-09 20:30:25]

lgtm

[shatch, 2014-07-09 20:57:27]

The CQ bit was checked by simonhatch@chromium.org

[commit-bot: I haz the power, 2014-07-09 20:59:20]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/simonhatch@chromium.org/375073002/20001

[commit-bot: I haz the power, 2014-07-10 00:47:11]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-10 00:50:16]

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

[commit-bot: I haz the power, 2014-07-10 00:50:17]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[shatch, 2014-07-10 16:32:40]

Hi Daniel, ptal, looking for an owner for

content/browser/web_contents/web_contents_android.*

[no sievers, 2014-07-10 19:15:09]

On 2014/07/10 16:32:40, shatch wrote:
> Hi Daniel, ptal, looking for an owner for
> 
> content/browser/web_contents/web_contents_android.*

lgtm

[shatch, 2014-07-10 19:30:43]

The CQ bit was checked by simonhatch@chromium.org

[commit-bot: I haz the power, 2014-07-10 19:32:44]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/simonhatch@chromium.org/375073002/20001

[commit-bot: I haz the power, 2014-07-10 19:36:14]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-10 19:38:45]

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

[commit-bot: I haz the power, 2014-07-10 19:38:46]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)

[shatch, 2014-07-10 20:58:33]

The CQ bit was checked by simonhatch@chromium.org

[commit-bot: I haz the power, 2014-07-10 20:59:34]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/simonhatch@chromium.org/375073002/40001

[commit-bot: I haz the power, 2014-07-11 02:35:39]

Change committed as 282504