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

Side by Side Diff: third_party/WebKit/Source/web/WebViewImpl.h

Issue 2530883002: Refactor overlay fullscreen video handling into a single callback (Closed)
Patch Set: rebase Created 4 years 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) 2010 Google Inc. All rights reserved. 2 * Copyright (C) 2010 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 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 } 442 }
443 float fakePageScaleAnimationPageScaleForTesting() const { 443 float fakePageScaleAnimationPageScaleForTesting() const {
444 return m_fakePageScaleAnimationPageScaleFactor; 444 return m_fakePageScaleAnimationPageScaleFactor;
445 } 445 }
446 bool fakePageScaleAnimationUseAnchorForTesting() const { 446 bool fakePageScaleAnimationUseAnchorForTesting() const {
447 return m_fakePageScaleAnimationUseAnchor; 447 return m_fakePageScaleAnimationUseAnchor;
448 } 448 }
449 449
450 void enterFullscreenForElement(Element*); 450 void enterFullscreenForElement(Element*);
451 void exitFullscreen(LocalFrame*); 451 void exitFullscreen(LocalFrame*);
452 void fullscreenElementChanged(Element*, Element*);
452 453
453 // Exposed for the purpose of overriding device metrics. 454 // Exposed for the purpose of overriding device metrics.
454 void sendResizeEventAndRepaint(); 455 void sendResizeEventAndRepaint();
455 456
456 // Exposed for testing purposes. 457 // Exposed for testing purposes.
457 bool hasHorizontalScrollbar(); 458 bool hasHorizontalScrollbar();
458 bool hasVerticalScrollbar(); 459 bool hasVerticalScrollbar();
459 460
460 // Exposed for tests. 461 // Exposed for tests.
461 unsigned numLinkHighlights() { return m_linkHighlights.size(); } 462 unsigned numLinkHighlights() { return m_linkHighlights.size(); }
(...skipping 272 matching lines...) Expand 10 before | Expand all | Expand 10 after
734 Persistent<ResizeViewportAnchor> m_resizeViewportAnchor; 735 Persistent<ResizeViewportAnchor> m_resizeViewportAnchor;
735 }; 736 };
736 737
737 // We have no ways to check if the specified WebView is an instance of 738 // We have no ways to check if the specified WebView is an instance of
738 // WebViewImpl because WebViewImpl is the only implementation of WebView. 739 // WebViewImpl because WebViewImpl is the only implementation of WebView.
739 DEFINE_TYPE_CASTS(WebViewImpl, WebView, webView, true, true); 740 DEFINE_TYPE_CASTS(WebViewImpl, WebView, webView, true, true);
740 741
741 } // namespace blink 742 } // namespace blink
742 743
743 #endif 744 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/FullscreenController.cpp ('k') | third_party/WebKit/Source/web/WebViewImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698