Adding NativeWindow base class with no activity.

Adding NativeWindow base class with context.

This is a change from the previous CL (https://codereview.chromium.org/10916160/). It turns out that not all the Android objects that create NativeWindow will have an activity, so we need this change.

BUG=116131
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=158159

[aurimas (slooooooooow), 2012-09-20 23:30:39]

Hey Dave, 

This is the same CL as downstream. 

Aurimas

[aurimas (slooooooooow), 2012-09-20 23:35:33]

Hey Nico,

Could you take a look at this CL?

Thanks,
Aurimas

[Nico, 2012-09-21 02:10:15]

lgtm, but it's probably good if someone on clank takes alook too.

https://codereview.chromium.org/10962002/diff/2001/ui/android/java/src/org/ch...
File ui/android/java/src/org/chromium/ui/gfx/ActivityNativeWindow.java (right):

https://codereview.chromium.org/10962002/diff/2001/ui/android/java/src/org/ch...
ui/android/java/src/org/chromium/ui/gfx/ActivityNativeWindow.java:74: if (error
!= null) Toast.makeText(mActivity, error, Toast.LENGTH_SHORT).show();
nit: linebreak after if

[aurimas (slooooooooow), 2012-09-21 15:51:28]

Hey Ben and Jay,

Would you mind looking over this CL?

Aurimas

https://codereview.chromium.org/10962002/diff/2001/ui/android/java/src/org/ch...
File ui/android/java/src/org/chromium/ui/gfx/ActivityNativeWindow.java (right):

https://codereview.chromium.org/10962002/diff/2001/ui/android/java/src/org/ch...
ui/android/java/src/org/chromium/ui/gfx/ActivityNativeWindow.java:74: if (error
!= null) Toast.makeText(mActivity, error, Toast.LENGTH_SHORT).show();
On 2012/09/21 02:10:15, Nico wrote:
> nit: linebreak after if

Done.

[benm (inactive), 2012-09-21 15:56:27]

lgtm

[Jay Civelli, 2012-09-21 21:30:45]

lgtm

[aurimas (slooooooooow), 2012-09-21 21:46:44]

Hey Yaron,

Could you take a look at my CL? I need a chrome/android owner.

Thanks,
Aurimas

[aurimas (slooooooooow), 2012-09-21 21:47:45]

Hey Scott,

Could you take a look at my ui/ changes?

Thanks,
Aurimas

[Yaron, 2012-09-21 22:18:12]

chrome/android lgtm

we should probably have a ui/android/OWNERS file

[sky, 2012-09-21 23:21:46]

LGTM

[commit-bot: I haz the power, 2012-09-21 23:26:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/aurimas@chromium.org/10962002/12001

[commit-bot: I haz the power, 2012-09-22 02:48:02]

Change committed as 158159