Commit 29736ca2 authored by bdash's avatar bdash

2007-01-10 Mitz Pettel <mitz@webkit.org>

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=11775
          'Show URLs in Tool Tips' preference is ignored

        * WebView/WebHTMLView.mm:
        (-[WebHTMLView _resetCachedWebPreferences:]):
        (-[WebHTMLView setDataSource:]): Added a call to _resetCachedWebPreferences:.
        Added an assertion that the view is not closed, instead of reopening it.
        Reopening should not occur, now that <http://bugs.webkit.org/show_bug.cgi?id=12087>
        is fixed.


git-svn-id: svn://svn.chromium.org/blink/trunk@18756 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 80478fb8
2007-01-10 Mitz Pettel <mitz@webkit.org>
Reviewed by Darin.
- fix http://bugs.webkit.org/show_bug.cgi?id=11775
'Show URLs in Tool Tips' preference is ignored
* WebView/WebHTMLView.mm:
(-[WebHTMLView _resetCachedWebPreferences:]):
(-[WebHTMLView setDataSource:]): Added a call to _resetCachedWebPreferences:.
Added an assertion that the view is not closed, instead of reopening it.
Reopening should not occur, now that <http://bugs.webkit.org/show_bug.cgi?id=12087>
is fixed.
2007-01-10 Beth Dakin <bdakin@apple.com> 2007-01-10 Beth Dakin <bdakin@apple.com>
Reviewed by John. Reviewed by John.
......
...@@ -1901,9 +1901,9 @@ static WebHTMLView *lastHitView = nil; ...@@ -1901,9 +1901,9 @@ static WebHTMLView *lastHitView = nil;
{ {
WebPreferences *preferences = [[self _webView] preferences]; WebPreferences *preferences = [[self _webView] preferences];
// Check for nil because we might not yet have an associated webView when this is called // Check for nil because we might not yet have an associated webView when this is called
if (preferences == nil) { if (preferences == nil)
preferences = [WebPreferences standardPreferences]; preferences = [WebPreferences standardPreferences];
}
_private->showsURLsInToolTips = [preferences showsURLsInToolTips]; _private->showsURLsInToolTips = [preferences showsURLsInToolTips];
} }
...@@ -3073,13 +3073,14 @@ done: ...@@ -3073,13 +3073,14 @@ done:
- (void)setDataSource:(WebDataSource *)dataSource - (void)setDataSource:(WebDataSource *)dataSource
{ {
ASSERT(dataSource); ASSERT(dataSource);
if (_private->dataSource == dataSource) if (_private->dataSource != dataSource) {
return; ASSERT(!_private->dataSource);
ASSERT(!_private->dataSource); ASSERT(!_private->closed);
_private->closed = NO; // setting a data source reopens a closed view _private->dataSource = [dataSource retain];
_private->dataSource = [dataSource retain]; [_private->pluginController setDataSource:dataSource];
[_private->pluginController setDataSource:dataSource]; [self addMouseMovedObserver];
[self addMouseMovedObserver]; }
[self _resetCachedWebPreferences:nil];
} }
- (void)dataSourceUpdated:(WebDataSource *)dataSource - (void)dataSourceUpdated:(WebDataSource *)dataSource
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment