My favorites | Sign in
Logo
             
New issue | Search
for
| Advanced search | Search tips
Issue 7625: regression : Window.open shows "about:blank" in the omnibox
5 people starred this issue and may be notified of changes. Back to list
Status:  Verified
Owner:  brettw@chromium.org
Closed:  Feb 2009
Cc:  anantha@chromium.org
Type-Bug
Pri-1
OS-All
Mstone-2.0
Regression
Area-Internals


Sign in to add a comment
 
Reported by phistuck, Feb 12, 2009
Chrome Version       : 2.0.162.0
    Firefox 3: OK
         IE 7:OK

What steps will reproduce the problem?
1. Type javascript:function go() 
{window.open('http://google.com');return;}go()
or
javascript:window.open('http://google.com');
2. Press Enter.

What is the expected result?
Google. And that the omnibox will show the corresponding URL.

What happens instead?
Google. But the URL is "about:blank".
Comment 1 by dhhwai, Feb 12, 2009
And another problem:

EXPECTED RESULT: Tab title/titlebar should display either the <title> of the page, or 
the root filename if the page has no title

WHAT HAPPENS INSTEAD: Tab title/titlebar displays Untitled
Comment 2 by dhhwai, Feb 12, 2009
This works fine in 1.0.154.48, but problem in 2.0.162.0

Add label: Regression
Comment 3 by phistuck, Feb 12, 2009
Probably derives from the same issue.
Comment 4 by phistuck, Feb 12, 2009
Also derives from this issue - the Web Inspector cannot be activated.
Comment 5 by jasneet@chromium.org, Feb 17, 2009
 Issue 7675  has been merged into this issue.
Cc: anan...@chromium.org
Comment 6 by jasneet@chromium.org, Feb 17, 2009
(No comment was entered for this change.)
Summary: regression : Window.open shows "about:blank" in the omnibox
Status: Untriaged
Comment 7 by mal.chromium, Feb 18, 2009
Not sure if this is UI or backend... somewhere we're failing to notify that the page 
title has changed.
Labels: -Pri-2 -Area-Misc Pri-1 Area-BrowserUI Area-BrowserBackend Mstone-2.0 Regression
Comment 8 by phistuck, Feb 18, 2009
Mmm.. title and URL.
Comment 9 by mal.chromium, Feb 18, 2009
(No comment was entered for this change.)
Status: Assigned
Owner: bre...@chromium.org
Labels: -Area-BrowserUI
Comment 10 by mal.chromium, Feb 19, 2009
 Issue 7873  has been merged into this issue.
Comment 11 by brettw@chromium.org, Feb 19, 2009
(No comment was entered for this change.)
Status: Started
Comment 12 by brettw@chromium.org, Feb 20, 2009
WebKit isn't triggering our detection in glue when a new navigation occurs. This load 
type seems to be FrameLoadTypeRedirectWithLockedBackForwardList and we trigger off of 
the back forward list. I'm going to research how this used to work.
Comment 13 by bugdroid1@chromium.org, Feb 21, 2009
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=10147 

------------------------------------------------------------------------
r10147 | brettw@chromium.org | 2009-02-21 08:21:35 -0800 (Sat, 21 Feb 2009) | 5 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/deps/third_party/WebKit/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp?r1=10147&r2=10146

Add missing bool parameter to changeLocation(). This causes us to not update
the back/forward list and current location since it thinks history is locked.

BUG=7625
Review URL: http://codereview.chromium.org/30001
------------------------------------------------------------------------

Comment 14 by bugdroid1@chromium.org, Feb 21, 2009
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=10148 

------------------------------------------------------------------------
r10148 | brettw@chromium.org | 2009-02-21 08:49:36 -0800 (Sat, 21 Feb 2009) | 4 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/DEPS?r1=10148&r2=10147

Pull deps to get about:blank fix.

BUG=7625
Review URL: http://codereview.chromium.org/27035
------------------------------------------------------------------------

Comment 15 by brettw@chromium.org, Feb 21, 2009
(No comment was entered for this change.)
Status: Fixed
Comment 16 by mberkow...@chromium.org, Feb 23, 2009
Verified in Chromium 2.0.167.0 (Developer Build 10179).
Status: Verified
Comment 17 by mal.chromium, Dec 18, 2009
(No comment was entered for this change.)
Labels: -Area-BrowserBackend Area-Internals
Sign in to add a comment

Powered by Google Project Hosting