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

Side by Side Diff: content/browser/renderer_host/render_widget_host_view_aura_unittest.cc

Issue 2731793002: DelegatedFrameHost should not allocate a new local surface id on frame eviction (Closed)
Patch Set: c Created 3 years, 9 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
« no previous file with comments | « content/browser/renderer_host/delegated_frame_host.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/render_widget_host_view_aura.h" 5 #include "content/browser/renderer_host/render_widget_host_view_aura.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <tuple> 10 #include <tuple>
(...skipping 403 matching lines...) Expand 10 before | Expand all | Expand 10 after
414 last_copy_request_->texture_mailbox().sync_token()); 414 last_copy_request_->texture_mailbox().sync_token());
415 gl_helper->ResizeTexture(texture, window()->bounds().size()); 415 gl_helper->ResizeTexture(texture, window()->bounds().size());
416 gl_helper->DeleteTexture(texture); 416 gl_helper->DeleteTexture(texture);
417 } 417 }
418 } 418 }
419 419
420 cc::SurfaceId surface_id() const { 420 cc::SurfaceId surface_id() const {
421 return GetDelegatedFrameHost()->SurfaceIdForTesting(); 421 return GetDelegatedFrameHost()->SurfaceIdForTesting();
422 } 422 }
423 423
424 const cc::LocalSurfaceId& GetLocalSurfaceId() const { 424 bool HasFrameData() const {
425 return GetDelegatedFrameHost()->LocalSurfaceIdForTesting(); 425 return GetDelegatedFrameHost()->HasFrameForTesting();
426 } 426 }
427 427
428 bool HasFrameData() const { return GetLocalSurfaceId().is_valid(); }
429
430 bool released_front_lock_active() const { 428 bool released_front_lock_active() const {
431 return GetDelegatedFrameHost()->ReleasedFrontLockActiveForTesting(); 429 return GetDelegatedFrameHost()->ReleasedFrontLockActiveForTesting();
432 } 430 }
433 431
434 void ReclaimResources(const cc::ReturnedResourceArray& resources) { 432 void ReclaimResources(const cc::ReturnedResourceArray& resources) {
435 GetDelegatedFrameHost()->ReclaimResources(resources); 433 GetDelegatedFrameHost()->ReclaimResources(resources);
436 } 434 }
437 435
438 void ResetCompositor() { GetDelegatedFrameHost()->ResetCompositor(); } 436 void ResetCompositor() { GetDelegatedFrameHost()->ResetCompositor(); }
439 437
(...skipping 4375 matching lines...) Expand 10 before | Expand all | Expand 10 after
4815 // There is no composition in the beginning. 4813 // There is no composition in the beginning.
4816 EXPECT_FALSE(has_composition_text()); 4814 EXPECT_FALSE(has_composition_text());
4817 SetHasCompositionTextToTrue(); 4815 SetHasCompositionTextToTrue();
4818 view->ImeCancelComposition(); 4816 view->ImeCancelComposition();
4819 // The composition must have been canceled. 4817 // The composition must have been canceled.
4820 EXPECT_FALSE(has_composition_text()); 4818 EXPECT_FALSE(has_composition_text());
4821 } 4819 }
4822 } 4820 }
4823 4821
4824 } // namespace content 4822 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/renderer_host/delegated_frame_host.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698