Fix Android ContentShell's incorrect behavior when opening link with window.open.

Fix Android ContentShell's incorrect behavior when opening link with window.open.

Fixed the two following issues:
- When open a link with 'window.open', new shell content is not rendered
  correctly, the content is always DEFAULT_SHELL_URL(http://www.google.com).
- When ContentShell activity is stopped and restarted, the rendered content is
  clear and reload to DEFAULT_SHELL_URL.

BUG=165745
TEST=manual: verified link in news.google.com is opened and rendered correctly


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=173370

[Shouqun Liu, 2012-12-13 08:41:41]

When open a link with "window.open" or stop/restart the shell activity, the
rendered content is always cleared and reload to DEFAULT_SHELL_URL.
This is not expected,  mActiveShell.loadUrl(mStartupUrl) should only be called
when ContentShell starts up.

[Jay Civelli, 2012-12-13 18:12:24]

https://codereview.chromium.org/11574003/diff/1/content/shell/android/java/sr...
File content/shell/android/java/src/org/chromium/content_shell/ShellManager.java
(right):

https://codereview.chromium.org/11574003/diff/1/content/shell/android/java/sr...
content/shell/android/java/src/org/chromium/content_shell/ShellManager.java:44: 
sStartup = false;
would be clearer.

[Shouqun Liu, 2012-12-14 00:30:44]

On 2012/12/13 18:12:24, Jay Civelli wrote:
>
https://codereview.chromium.org/11574003/diff/1/content/shell/android/java/sr...
> File
content/shell/android/java/src/org/chromium/content_shell/ShellManager.java
> (right):
> 
>
https://codereview.chromium.org/11574003/diff/1/content/shell/android/java/sr...
>
content/shell/android/java/src/org/chromium/content_shell/ShellManager.java:44: 
> sStartup = false;
> would be clearer.

Yes, updated the patch.

[Jay Civelli, 2012-12-14 16:58:23]

LGTM, Ted should probably review as well.

[Ted C, 2012-12-14 17:00:51]

lgtm

[commit-bot: I haz the power, 2012-12-15 03:39:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shouqun.liu@intel.com/11574003/3

[commit-bot: I haz the power, 2012-12-15 08:00:23]

Retried try job too often on win_rel for step(s) browser_tests

[commit-bot: I haz the power, 2012-12-15 09:30:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shouqun.liu@intel.com/11574003/3

[commit-bot: I haz the power, 2012-12-15 12:55:45]

Retried try job too often on win_rel for step(s) browser_tests

[commit-bot: I haz the power, 2012-12-15 15:04:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shouqun.liu@intel.com/11574003/3

[commit-bot: I haz the power, 2012-12-15 18:15:41]

Retried try job too often on win_rel for step(s) browser_tests

[commit-bot: I haz the power, 2012-12-16 02:07:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shouqun.liu@intel.com/11574003/3

[commit-bot: I haz the power, 2012-12-16 05:13:45]

Change committed as 173370