Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(68)

Side by Side Diff: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

Issue 2786673002: Separate ContentSettingsClient out from LocalFrameClient (Closed)
Patch Set: added class-level comment Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 552 matching lines...) Expand 10 before | Expand all | Expand 10 after
563 563
564 WebVector<WebIconURL> WebLocalFrameImpl::iconURLs(int iconTypesMask) const { 564 WebVector<WebIconURL> WebLocalFrameImpl::iconURLs(int iconTypesMask) const {
565 // The URL to the icon may be in the header. As such, only 565 // The URL to the icon may be in the header. As such, only
566 // ask the loader for the icon if it's finished loading. 566 // ask the loader for the icon if it's finished loading.
567 if (frame()->document()->loadEventFinished()) 567 if (frame()->document()->loadEventFinished())
568 return frame()->document()->iconURLs(iconTypesMask); 568 return frame()->document()->iconURLs(iconTypesMask);
569 return WebVector<WebIconURL>(); 569 return WebVector<WebIconURL>();
570 } 570 }
571 571
572 void WebLocalFrameImpl::setContentSettingsClient( 572 void WebLocalFrameImpl::setContentSettingsClient(
573 WebContentSettingsClient* contentSettingsClient) { 573 WebContentSettingsClient* client) {
574 m_contentSettingsClient = contentSettingsClient; 574 m_contentSettingsClient.setClient(client);
575 } 575 }
576 576
577 void WebLocalFrameImpl::setSharedWorkerRepositoryClient( 577 void WebLocalFrameImpl::setSharedWorkerRepositoryClient(
578 WebSharedWorkerRepositoryClient* client) { 578 WebSharedWorkerRepositoryClient* client) {
579 m_sharedWorkerRepositoryClient = 579 m_sharedWorkerRepositoryClient =
580 SharedWorkerRepositoryClientImpl::create(client); 580 SharedWorkerRepositoryClientImpl::create(client);
581 } 581 }
582 582
583 ScrollableArea* WebLocalFrameImpl::layoutViewportScrollableArea() const { 583 ScrollableArea* WebLocalFrameImpl::layoutViewportScrollableArea() const {
584 if (FrameView* view = frameView()) 584 if (FrameView* view = frameView())
(...skipping 971 matching lines...) Expand 10 before | Expand all | Expand 10 after
1556 WebLocalFrameImpl::WebLocalFrameImpl( 1556 WebLocalFrameImpl::WebLocalFrameImpl(
1557 WebTreeScopeType scope, 1557 WebTreeScopeType scope,
1558 WebFrameClient* client, 1558 WebFrameClient* client,
1559 blink::InterfaceProvider* interfaceProvider, 1559 blink::InterfaceProvider* interfaceProvider,
1560 blink::InterfaceRegistry* interfaceRegistry) 1560 blink::InterfaceRegistry* interfaceRegistry)
1561 : WebLocalFrame(scope), 1561 : WebLocalFrame(scope),
1562 m_localFrameClientImpl(LocalFrameClientImpl::create(this)), 1562 m_localFrameClientImpl(LocalFrameClientImpl::create(this)),
1563 m_frameWidget(0), 1563 m_frameWidget(0),
1564 m_client(client), 1564 m_client(client),
1565 m_autofillClient(0), 1565 m_autofillClient(0),
1566 m_contentSettingsClient(0),
1567 m_inputEventsScaleFactorForEmulation(1), 1566 m_inputEventsScaleFactorForEmulation(1),
1568 m_interfaceProvider(interfaceProvider), 1567 m_interfaceProvider(interfaceProvider),
1569 m_interfaceRegistry(interfaceRegistry), 1568 m_interfaceRegistry(interfaceRegistry),
1570 m_webDevToolsFrontend(0), 1569 m_webDevToolsFrontend(0),
1571 m_inputMethodController(new WebInputMethodControllerImpl(this)), 1570 m_inputMethodController(new WebInputMethodControllerImpl(this)),
1572 m_textCheckerClient(new TextCheckerClientImpl(this)), 1571 m_textCheckerClient(new TextCheckerClientImpl(this)),
1573 m_selfKeepAlive(this) { 1572 m_selfKeepAlive(this) {
1574 DCHECK(m_client); 1573 DCHECK(m_client);
1575 frameCount++; 1574 frameCount++;
1576 } 1575 }
(...skipping 944 matching lines...) Expand 10 before | Expand all | Expand 10 after
2521 createMarkup(endPosition, startPosition, AnnotateForInterchange, 2520 createMarkup(endPosition, startPosition, AnnotateForInterchange,
2522 ConvertBlocksToInlines::NotConvert, ResolveNonLocalURLs); 2521 ConvertBlocksToInlines::NotConvert, ResolveNonLocalURLs);
2523 } 2522 }
2524 } 2523 }
2525 2524
2526 TextCheckerClient& WebLocalFrameImpl::textCheckerClient() const { 2525 TextCheckerClient& WebLocalFrameImpl::textCheckerClient() const {
2527 return *m_textCheckerClient; 2526 return *m_textCheckerClient;
2528 } 2527 }
2529 2528
2530 } // namespace blink 2529 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/WebLocalFrameImpl.h ('k') | third_party/WebKit/Source/web/WebViewImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698