[ChromeDriver] Detect page loading status when the page is not loaded at all.

[ChromeDriver] Detect page loading status when the page is not loaded at all.

BUG=chromedriver:815
Committed: https://crrev.com/af41684421a077cb0d3f87c904de83ce13423b47
Cr-Commit-Position: refs/heads/master@{#297468}

[stgao, 2014-09-25 20:48:18]

Patchset #1 (id:1) has been deleted

[stgao, 2014-09-26 20:24:54]

Patchset #1 (id:20001) has been deleted

[stgao, 2014-09-26 20:39:11]

stgao@chromium.org changed reviewers:
+ samuong@chromium.org

[samuong, 2014-09-29 17:20:32]

Thanks Shuotao. This breaks a few of the unit tests for me, does this happen for
you too?

4 tests failed:
    NavigationTracker.OnSuccessfulNavigateStillWaiting
    NavigationTracker.UnknownStateFailsToDetermineState
    NavigationTracker.UnknownStateForcesStart
    NavigationTracker.UnknownStateForcesStartReceivesStop

[stgao, 2014-09-30 00:08:28]

Added a new unittest and fixed existing unittest.

[samuong, 2014-09-30 04:37:10]

On 2014/09/30 00:08:28, Shuotao wrote:
> Added a new unittest and fixed existing unittest.

thanks, lgtm with a nit

[samuong, 2014-09-30 04:37:17]

https://codereview.chromium.org/594383005/diff/60001/chrome/test/chromedriver...
File chrome/test/chromedriver/chrome/navigation_tracker_unittest.cc (right):

https://codereview.chromium.org/594383005/diff/60001/chrome/test/chromedriver...
chrome/test/chromedriver/chrome/navigation_tracker_unittest.cc:316: if
(has_empty_base_url_) {
remove braces?

[stgao, 2014-09-30 17:57:47]

https://codereview.chromium.org/594383005/diff/60001/chrome/test/chromedriver...
File chrome/test/chromedriver/chrome/navigation_tracker_unittest.cc (right):

https://codereview.chromium.org/594383005/diff/60001/chrome/test/chromedriver...
chrome/test/chromedriver/chrome/navigation_tracker_unittest.cc:316: if
(has_empty_base_url_) {
On 2014/09/30 04:37:17, samuong wrote:
> remove braces?

Done.

[stgao, 2014-09-30 17:57:50]

The CQ bit was checked by stgao@chromium.org

[commit-bot: I haz the power, 2014-09-30 17:59:08]

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

[commit-bot: I haz the power, 2014-09-30 18:57:36]

Committed patchset #3 (id:80001) as b33fbf9958ff11e6b28d3803123bf488af48ce14

[commit-bot: I haz the power, 2014-09-30 18:58:45]

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