diff options
author | danakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-06 04:37:17 +0000 |
---|---|---|
committer | danakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-06 04:37:17 +0000 |
commit | 3b9f2df0b78911631b92fff4c174a37fd8077482 (patch) | |
tree | 7eceed9124cdad273658f9c813a51ccd37bfcdf0 /cc/scheduler_state_machine.cc | |
parent | 5b3065b13476000a1940aeabfec125c736d687da (diff) | |
download | chromium_src-3b9f2df0b78911631b92fff4c174a37fd8077482.zip chromium_src-3b9f2df0b78911631b92fff4c174a37fd8077482.tar.gz chromium_src-3b9f2df0b78911631b92fff4c174a37fd8077482.tar.bz2 |
cc: Finish the rename from cc::GraphicsContext to cc::OutputSurface
It is far too confusing to deal with OutputSurface code when half the time it is
called a GraphicsContext in correctly. Cleaning this up so I can think about the
code properly as I upstream the Ubercomp CL.
NOTRY=true
R=jamesr,piman
BUG=146080
Review URL: https://chromiumcodereview.appspot.com/11450019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171403 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/scheduler_state_machine.cc')
-rw-r--r-- | cc/scheduler_state_machine.cc | 38 |
1 files changed, 19 insertions, 19 deletions
diff --git a/cc/scheduler_state_machine.cc b/cc/scheduler_state_machine.cc index 7b2ce7b..e8404476 100644 --- a/cc/scheduler_state_machine.cc +++ b/cc/scheduler_state_machine.cc @@ -28,7 +28,7 @@ SchedulerStateMachine::SchedulerStateMachine() , m_canDraw(false) , m_drawIfPossibleFailed(false) , m_textureState(LAYER_TEXTURE_STATE_UNLOCKED) - , m_contextState(CONTEXT_ACTIVE) + , m_outputSurfaceState(OUTPUT_SURFACE_ACTIVE) { } @@ -53,7 +53,7 @@ std::string SchedulerStateMachine::toString() base::StringAppendF(&str, "m_canDraw = %d; ", m_canDraw); base::StringAppendF(&str, "m_drawIfPossibleFailed = %d; ", m_drawIfPossibleFailed); base::StringAppendF(&str, "m_textureState = %d; ", m_textureState); - base::StringAppendF(&str, "m_contextState = %d; ", m_contextState); + base::StringAppendF(&str, "m_outputSurfaceState = %d; ", m_outputSurfaceState); return str; } @@ -93,7 +93,7 @@ bool SchedulerStateMachine::shouldDraw() const return false; if (hasDrawnThisFrame()) return false; - if (m_contextState != CONTEXT_ACTIVE) + if (m_outputSurfaceState != OUTPUT_SURFACE_ACTIVE) return false; return true; } @@ -120,13 +120,13 @@ SchedulerStateMachine::Action SchedulerStateMachine::nextAction() const return ACTION_ACQUIRE_LAYER_TEXTURES_FOR_MAIN_THREAD; switch (m_commitState) { case COMMIT_STATE_IDLE: - if (m_contextState != CONTEXT_ACTIVE && m_needsForcedRedraw) + if (m_outputSurfaceState != OUTPUT_SURFACE_ACTIVE && m_needsForcedRedraw) return ACTION_DRAW_FORCED; - if (m_contextState != CONTEXT_ACTIVE && m_needsForcedCommit) + if (m_outputSurfaceState != OUTPUT_SURFACE_ACTIVE && m_needsForcedCommit) return ACTION_BEGIN_FRAME; - if (m_contextState == CONTEXT_LOST) - return ACTION_BEGIN_CONTEXT_RECREATION; - if (m_contextState == CONTEXT_RECREATING) + if (m_outputSurfaceState == OUTPUT_SURFACE_LOST) + return ACTION_BEGIN_OUTPUT_SURFACE_RECREATION; + if (m_outputSurfaceState == OUTPUT_SURFACE_RECREATING) return ACTION_NONE; if (shouldDraw()) return m_needsForcedRedraw ? ACTION_DRAW_FORCED : ACTION_DRAW_IF_POSSIBLE; @@ -143,7 +143,7 @@ SchedulerStateMachine::Action SchedulerStateMachine::nextAction() const return ACTION_COMMIT; case COMMIT_STATE_WAITING_FOR_FIRST_DRAW: - if (shouldDraw() || m_contextState == CONTEXT_LOST) + if (shouldDraw() || m_outputSurfaceState == OUTPUT_SURFACE_LOST) return m_needsForcedRedraw ? ACTION_DRAW_FORCED : ACTION_DRAW_IF_POSSIBLE; // COMMIT_STATE_WAITING_FOR_FIRST_DRAW wants to enforce a draw. If m_canDraw is false // or textures are not available, proceed to the next step (similar as in COMMIT_STATE_IDLE). @@ -201,10 +201,10 @@ void SchedulerStateMachine::updateState(Action action) m_textureState = LAYER_TEXTURE_STATE_UNLOCKED; return; - case ACTION_BEGIN_CONTEXT_RECREATION: + case ACTION_BEGIN_OUTPUT_SURFACE_RECREATION: DCHECK(m_commitState == COMMIT_STATE_IDLE); - DCHECK(m_contextState == CONTEXT_LOST); - m_contextState = CONTEXT_RECREATING; + DCHECK(m_outputSurfaceState == OUTPUT_SURFACE_LOST); + m_outputSurfaceState = OUTPUT_SURFACE_RECREATING; return; case ACTION_ACQUIRE_LAYER_TEXTURES_FOR_MAIN_THREAD: @@ -232,7 +232,7 @@ bool SchedulerStateMachine::vsyncCallbackNeeded() const if (m_needsForcedRedraw) return true; - return m_needsRedraw && m_visible && m_contextState == CONTEXT_ACTIVE; + return m_needsRedraw && m_visible && m_outputSurfaceState == OUTPUT_SURFACE_ACTIVE; } void SchedulerStateMachine::didEnterVSync() @@ -307,17 +307,17 @@ void SchedulerStateMachine::beginFrameAborted() } } -void SchedulerStateMachine::didLoseContext() +void SchedulerStateMachine::didLoseOutputSurface() { - if (m_contextState == CONTEXT_LOST || m_contextState == CONTEXT_RECREATING) + if (m_outputSurfaceState == OUTPUT_SURFACE_LOST || m_outputSurfaceState == OUTPUT_SURFACE_RECREATING) return; - m_contextState = CONTEXT_LOST; + m_outputSurfaceState = OUTPUT_SURFACE_LOST; } -void SchedulerStateMachine::didRecreateContext() +void SchedulerStateMachine::didRecreateOutputSurface() { - DCHECK(m_contextState == CONTEXT_RECREATING); - m_contextState = CONTEXT_ACTIVE; + DCHECK(m_outputSurfaceState == OUTPUT_SURFACE_RECREATING); + m_outputSurfaceState = OUTPUT_SURFACE_ACTIVE; setNeedsCommit(); } |