From 6256e21b43e8c31da3c8ecc81621dd2c8362bc2a Mon Sep 17 00:00:00 2001
From: "finnur@chromium.org"
 <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Mon, 29 Jun 2009 20:22:41 +0000
Subject: Fixing: DataSource is used before null check in GetLoadTimes.

BUG=None
TEST=None

Review URL: http://codereview.chromium.org/150036

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19518 0039d316-1c4b-4281-b951-d872f2087c98
---
 chrome/renderer/loadtimes_extension_bindings.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'chrome')

diff --git a/chrome/renderer/loadtimes_extension_bindings.cc b/chrome/renderer/loadtimes_extension_bindings.cc
index 5a65af7..11b5d89 100644
--- a/chrome/renderer/loadtimes_extension_bindings.cc
+++ b/chrome/renderer/loadtimes_extension_bindings.cc
@@ -67,9 +67,9 @@ class LoadTimesExtensionWrapper : public v8::Extension {
     WebFrame* frame = WebFrame::RetrieveFrameForEnteredContext();
     if (frame) {
       WebDataSource* data_source = frame->GetDataSource();
-      NavigationState* navigation_state =
-          NavigationState::FromDataSource(data_source);
       if (data_source) {
+        NavigationState* navigation_state =
+            NavigationState::FromDataSource(data_source);
         v8::Local<v8::Object> load_times = v8::Object::New();
         load_times->Set(
             v8::String::New("requestTime"),
-- 
cgit v1.1